Re: [PATCH v2] bootp: add native DHCPv4 support

[Mark Rutland]

Hi,

On Tue, Mar 22, 2016 at 09:26:20PM +0300, Andrei Borzenkov wrote:
> This patch adds support for native DHCPv4 and removes requirement for
> BOOTP compatibility support in DHCP server.

Sorry for the delay in testing this; I'd meant to take a look at this
but it managed to fall off my radar.

I've just given this a spin atop of 7a5b301e3adb8e05 ("build: Use
AC_HEADER_MAJOR to find device macros"), with a small conflict manually
fixed up (I assume that the NULL check for inter still stands).

Otherwise, this seems to work perfectly for me in my BOOT-free work
environment on an AArch64 UEFI platform:

grub> insmod efinet
grub> net_ls_cards
efinet0 00:02:f7:00:61:79
grub> net_dhcp efinet0
grub> net_default_server=10.1.205.151
grub> linux (tftp)/Image.gz
grub> boot
EFI stub: Booting Linux Kernel...
EFI stub: Using DTB from configuration table
EFI stub: Exiting boot services and installing virtual address map...
[    0.000000] Booting Linux on physical CPU 0x100
[    0.000000] Linux version 4.6.0-rc5-next-20160427+ (address@hidden) (gcc 
version 5.1.1 20150608 (Linaro GCC 5.1-2015.08) ) #8 SMP PREEMPT Wed Apr 27 
14:53:05 BST 2016
...

As previously, I don't have a BOOTP server to hand to test with; sorry
for not being able to test that.

It would be great to see this committed!

Thanks,
Mark.