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 <ivan.merc...@nexvision.fr> > Signed-off-by: Marcel Ziswiler <marcel.ziswi...@toradex.com> > Reviewed-by: Bin Meng <bmeng...@gmail.com> > Reviewed-by: Stephen Warren <swar...@nvidia.com> > Acked-by: Simon Glass <s...@chromium.org>
Applied to u-boot/master, thanks! -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
