[PATCH] Clean up ARM compressed loader

Uwe Kleine-König u.kleine-koenig at pengutronix.de
Wed Feb 24 03:51:26 EST 2010


Hello,

On Wed, Feb 24, 2010 at 03:23:37AM +0100, Hector Martin wrote:
> The -Dstatic= stuff in the Makefile is a hack and subtly breaks the
> current inflate implementation, because the fixed inflate tables are
> included into a function and removing static places them in the stack,
> which bloats the stack and breaks references after the function returns.
> So get rid of the hack.
> 
> Instead, clean up the stub loader and unify it. The loader is now
> exactly the same regardless of whether you've enabled CONFIG_ZBOOT_ROM
> or not, and runs from RAM in PIC mode. CONFIG_ZBOOT_ROM simply enables
> an extra section of stub code that lets the zImage boot from ROM by
> copying itself to RAM first.
This is a big regression as this introduces two new copies.  And as
CONFIG_ZBOOT_ROM isn't primary about saving RAM at the early boot stages
but for fast booting this is not acceptable.
 
Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |



More information about the linux-arm-kernel mailing list