On Sat, Jan 26, 2019 at 03:25:12PM +0100, Heinrich Schuchardt wrote:
On x86_64 the size of the file u-boot loaded by the tftp test has grown in size such that when loading the file to 0x200000 it overwrites a memory area reserved for PCI.
If no load address is specified for tftp do not use the ram base address (or if zero 0x200000) but the default address.
Signed-off-by: Heinrich Schuchardt xypron.glpk@gmx.de
Currently there is a bug in net/tftp.c introduced by commit a156c47e39ad ("tftp: prevent overwriting reserved memory") which does not allow loading to the second DRAM bank. Simon Goldschmidt is working to fix it.
Yes, a real bug and good job sorting it out.
Due to this bug with the proposed patch we see a Travis CI error for vexpress_ca15_tc2_defconfig and vexpress_ca9x4_defconfig.
It's breaking with whatever additional changes you're testing. Please just put this change in your WIP branch until Simon has fixed the problem you've found. Alternatively, fork uboot-test-hooks and make your WIP trees use that fork which specifies a different address to use for tftp? Or am I missing something else? Thanks!