On Wed, Nov 18, 2015 at 03:05:06PM +0100, Marcel Ziswiler wrote: > The address range check may overflow if the memory region is located at > the top of the 32-bit address space. This can e.g. be seen on TK1 if > using the E1000 gigabit Ethernet driver where start and size are both > 0x80000000 leading to the following messages: > > Apalis TK1 # tftpboot $loadaddr test_file > Using e1000#0 device > TFTP from server 192.168.10.1; our IP address is 192.168.10.2 > Filename 'test_file'. > Load address: 0x80408000 > Loading: pci_hose_phys_to_bus: invalid physical address > > This patch fixes this by changing the order of the addition vs. > subtraction in the range check just like already done in > __pci_hose_bus_to_phys(). > > Reported-by: Ivan Mercier <[email protected]> > Signed-off-by: Marcel Ziswiler <[email protected]> > Reviewed-by: Bin Meng <[email protected]> > Reviewed-by: Stephen Warren <[email protected]> > Acked-by: Simon Glass <[email protected]>
Applied to u-boot/master, thanks! -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
