On Thu, Apr 17, 2014 at 5:02 AM, Peter Maydell <peter.mayd...@linaro.org> wrote: > On 2 April 2014 13:47, Peter Maydell <peter.mayd...@linaro.org> wrote: >> On 2 April 2014 13:11, Peter Crosthwaite <peter.crosthwa...@xilinx.com> >> wrote: >>> Like others, I have been carrying this change locally. Good to see it up! >> >> Why are you all booting raw Images anyway (just out of curiosity)? > > Given the recent feedback from the kernel mailing list (viz "don't boot > Image unless you really know what you're doing, the correct load > image may change randomly depending what other board support > is in a multikernel image, etc") maybe we should just remove the > Image booting support instead? Clearly nobody who hasn't locally > patched QEMU is using it at the moment...
Including AArch64, too? :) It happens to be correct ATM, but really we should be looking the Image header to determine the offset. I heard Mark Rutland has some patches to do TEXT_OFFSET randomization in fact. The choice of 0x10000 is not really a good one either as this forces the decompressor to move itself out of the way first. I guess it is a good choice if your goal is to test the worst possible code path for the decompressor. Rob