]> git.ipfire.org Git - thirdparty/ipxe.git/commit
[ioapi] Fail ioremap() when attempting to map a zero bus address
authorMichael Brown <mcb30@ipxe.org>
Wed, 16 Jul 2014 14:27:16 +0000 (15:27 +0100)
committerMichael Brown <mcb30@ipxe.org>
Wed, 16 Jul 2014 14:39:59 +0000 (15:39 +0100)
commitae778091cac3e6324d0f4876bcc12869679c26e4
treefb1c85795bf6a92cf21c89332ec954a9e1169b5c
parent8a42a369420817763fcb200a12cec84562143a13
[ioapi] Fail ioremap() when attempting to map a zero bus address

When a 32-bit iPXE binary is running on a system which allocates PCI
memory BARs above 4GB, our PCI subsystem will return the base address
for any such BARs as zero (with a warning message if DEBUG=pci is
enabled).  Currently, ioremap() will happily map an address pointing
to the start of physical memory, providing no sensible indication of
failure.

Fix by always returning NULL if we are asked to ioremap() a zero bus
address.

With a totally flat memory model (e.g. under EFI), this provides an
accurate failure indication since no PCI peripheral will be mapped to
the zero bus address.

With the librm memory model, there is the possibility of a spurious
NULL return from ioremap() if the bus address happens to be equal to
virt_offset.  Under the current virtual memory map, the NULL virtual
address will always be the start of .textdata, and so this problem
cannot occur; a NULL return from ioremap() will always be an accurate
failure indication.

Debugged-by: Anton D. Kachalov <mouse@yandex-team.ru>
Signed-off-by: Michael Brown <mcb30@ipxe.org>
src/arch/x86/include/ipxe/x86_io.h