Hi,
Wolfgang Denk wrote:
Dear Michal Simek,
In message 1314786967-23261-1-git-send-email-monstr@monstr.eu you wrote:
Patch: "Put common autoload code into auto_load() function" (sha1: 093498669e77597635a24f326f11efeab213d394) is not simple code cleanup but code change which introduce new bug.
If autoload variable is not setup it worked as autoload=yes.
Yes, and this is intentionally, because it's what poople expect when they run the "dhcpboot" command.
ok.
Currently if autoload is not setup dhcp sends request in forever loop.
That's not correct. Only until a DHCP server provides the needed configurationinformation so TFTP gets started.
That's what I get/see on my platforms.
There are two options how to fix it:
- Move TftpStart() which is in this patch
- Change functionality if autoload is not setup, set NetSate and ends.
That would change behavious in a way that is not wanted.
Agree that point 2 change behavior. Point 1 just fix c&p error.
Regards, Michal