[efi] Do not return EFI_NOT_READY from our ReceiveFilters() method
authorMichael Brown <mcb30@ipxe.org>
Tue, 1 Sep 2015 18:20:28 +0000 (19:20 +0100)
committerMichael Brown <mcb30@ipxe.org>
Tue, 1 Sep 2015 20:24:02 +0000 (21:24 +0100)
commit355da7b1336114564ce442b7d9bc4485f86d0be5
tree7c2800d39c160e65b81511a374029b6ff5af8289
parent226d5f2cafc258b979c090b19503eafa77c9a1a2
[efi] Do not return EFI_NOT_READY from our ReceiveFilters() method

Our SNP ReceiveFilters() method is a no-op, since we always (if
possible) use promiscuous mode for all network cards.  The method
currently returns EFI_NOT_READY if the SNP interfaces are claimed for
use by iPXE, as with all other SNP methods.

The WDS bootstrap wdsmgfw.efi attempts to use both the PXE Base Code
protocol and the Simple Network Protocol simultaneously.  This is
fundamentally broken, since use of the PXE Base Code protocol requires
us to disable the use of SNP (by claiming the interfaces for use by
iPXE), otherwise MnpDxe swallows all of the received packets before
our PXE Base Code's UdpRead() method is able to return them.

The root cause of this problem is that, as with BIOS PXE, the network
booting portions of the UEFI specification are less of a specification
and more of an application note sketchily describing how the original
hacked-together Intel implementation works.  No sane design would ever
have included the UdpWrite() and UdpRead() methods.

Work around these fundamental conceptual flaws by unconditionally
returning success from efi_snp_receive_filters().

Signed-off-by: Michael Brown <mcb30@ipxe.org>
src/interface/efi/efi_snp.c