[U-Boot] dhcp command not populating dnsip environment variable
In the latest version of u-boot, the dhcp command appears not to populate the environment variable dnsip. This used to be the behaviour.
The dns command works, if dnsip is populated manually.
Is the a Kconfig variable controlling this? Or is there a new environment variable?
Thanks
Duncan Hare
On Thu, Mar 15, 2018 at 3:25 PM, Duncan Hare dh@synoia.com wrote:
In the latest version of u-boot, the dhcp command appears not to populate the environment variable dnsip. This used to be the behaviour.
Is it possible that your DHCP server isn't sending it? It seems like the code isn't changed.
The dns command works, if dnsip is populated manually.
Is the a Kconfig variable controlling this? Or is there a new environment variable?
Thanks
Duncan Hare _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
From: Joe Hershberger joe.hershberger@ni.com To: Duncan Hare dh@synoia.com Cc: u-boot u-boot@lists.denx.de; Joe Hershberger joe.hershberger@ni.com Sent: Thursday, March 15, 2018 1:29 PM Subject: Re: [U-Boot] dhcp command not populating dnsip environment variable
On Thu, Mar 15, 2018 at 3:25 PM, Duncan Hare dh@synoia.com wrote:
In the latest version of u-boot, the dhcp command appears not to populate the environment variable dnsip. This used to be the behaviour.
Is it possible that your DHCP server isn't sending it? It seems like the code isn't changed.
The dns command works, if dnsip is populated manually.
Is the a Kconfig variable controlling this? Or is there a new environment variable?
Thanks
Duncan Hare _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Joe Thanks, you gave me an idea.
I fired up wireshark See attached.
Old release: Before Kconfig work options requested (many) including DNS server address
After: No options
I see no where in u-boot to set DHCP options.
Kconfig is a good idea. HOWEVER, the previous defaults should be selected as the base set. Choosing The NULL set is a very unfriendly choice.
What do we do to resolve this? Duncan Hare
714 931 7952
On Thu, Mar 15, 2018 at 3:25 PM, Duncan Hare dh@synoia.com wrote:
In the latest version of u-boot, the dhcp command appears not to populate the environment variable dnsip. This used to be the behaviour.
_______________________________________________ The bootp.c code was changed with the addition of Kconfig control of Bootp.c (of which dhcp is a part).There does not appear to be the cond in net/ Kconfig to manage the Bootp config in menuconfig. The change log at the top of bootp.c has no note to indicate the module was changed. Adding the note would appears to be the proper process. How many other source code modules have been changed without a change note?Finding this has taken two or three days. I can find no entries in .config nor in menuconfig to configure the bootp options.
Would you like me to prepare a patch for net/Kconfig? Duncan Hare
Hi Duncan,
On Fri, Mar 16, 2018 at 12:18 PM, Duncan Hare dh@synoia.com wrote:
On Thu, Mar 15, 2018 at 3:25 PM, Duncan Hare dh@synoia.com wrote:
In the latest version of u-boot, the dhcp command appears not to populate the environment variable dnsip. This used to be the behaviour.
The bootp.c code was changed with the addition of Kconfig control of Bootp.c (of which dhcp is a part).There does not appear to be the cond in net/ Kconfig to manage the Bootp config in menuconfig.
Are you referring to commit 3b3ea2c56ec4bc5588281fd103c744e608f8b25c ("Kconfig: cmd: Make networking command dependent on NET")?
The change log at the top of bootp.c has no note to indicate the module was changed. Adding the note would appears to be the proper process. How many other source code modules have been changed without a change note?Finding this has taken two or three days. I can find no entries in .config nor in menuconfig to configure the bootp options.
commit d021e942107a1f7304a879cec99286ca462f7be3 ("Convert CONFIG_BOOTP_BOOTPATH et al to Kconfig") moved CONFIG_BOOTP_DNS, but not CONFIG_BOOTP_DNS2 for some reason. The README was changed to delete the description of both, though.
Would you like me to prepare a patch for net/Kconfig?
Sure, that sounds great.
Duncan Hare
Thanks, -Joe
participants (2)
-
Duncan Hare -
Joe Hershberger