[PATCH] Clean up ARM compressed loader

Russell King - ARM Linux linux at arm.linux.org.uk
Wed Feb 24 19:01:14 EST 2010


On Thu, Feb 25, 2010 at 12:57:20AM +0100, Hector Martin wrote:
> Russell King - ARM Linux wrote:
> > On Wed, Feb 24, 2010 at 06:34:49PM -0500, Nicolas Pitre wrote:
> >> What about simply not compiling the decompressor with -fPIC when using 
> >> ZBOOT_ROM=y?  That would certainly solve the problem with the only 
> >> restriction that such kernel images won't be bootable from RAM which is 
> >> probably an acceptable compromize.
> > 
> > Unfortunately, that doesn't solve the stack-bashing with ZBOOT_ROM=n.
> 
> Yes it does, that's exactly what my first version of the patch did. Once
> you get rid of the partial relocation used for ROM builds (with split
> text/bss) you don't need -Dstatic=, and once you get rid of that you
> solve the stack-bashing. The ROM build becomes a bog standard
> non-relocatable ROM image (with the usual LMA/VMA linker script stuff to
> copy initialized data to RAM), and the RAM build becomes a bog standard
> relocatable image (a single contiguous blob including
> text/rodata/data/bss) that doesn't suffer from any issues when you move
> it around.

Did you bother to read my previous reply explaining that this is not
a hack for the toolchain?  It sounds to me like you didn't.



More information about the linux-arm-kernel mailing list