linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: u.kleine-koenig@pengutronix.de (Uwe Kleine-König)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] ARM: Add Kconfig option to use mkimage -T kernel_noload
Date: Wed, 29 Feb 2012 21:30:56 +0100	[thread overview]
Message-ID: <20120229203056.GP14173@pengutronix.de> (raw)
In-Reply-To: <4F4E86A1.8010506@bluewatersys.com>

On Thu, Mar 01, 2012 at 09:12:17AM +1300, Andre Renaud wrote:
> On 01/03/12 08:59, Stephen Warren wrote:
> > Uwe Kleine-K?nig wrote at Wednesday, February 29, 2012 12:44 PM:
> > ...
> >> If you want to give incentive for U-Boot to improve, drop the target
> >> today. And note that at least people caring about boot time must not use
> >> the kernel's uImage target anyhow.
> > 
> > If you enable the new config option in this patch, then the performance
> > issue is solved; U-Boot doesn't copy the kernel image any more, and the
> > kernel decompressor can write directly to the appropriate location without
> > moving the image first (assuming your board boot script loads the uImage
> > to a non-conflicting address).
> 
> I may have missed part of this, but isn't one of the points regarding
> this that the zImage decompressor always runs with data cache disabled,
> resulting in a slow decompress, where as if the U-Boot decompressor is
> used (ie, gzipping the Image, and telling U-Boot to decompress), then it
> can run with caches enabled, improving boot speed?
This is wrong. The zImage decompressor runs with caches on. The
advantage that U-Boot (maybe) has when doing the decompression itself is
that the cache for reading the zImage is already hot when U-Boot copied it
from NAND to RAM first. (I don't know if U-Boot can decompress directly
from NAND without writing the Image to RAM first?!)

> Thus the relocation issue is not really the speed hit, rather it is the
> image decompression.
I'm sure that letting U-Boot decompress an image that first has to be
moved to prevent it being overwritten during decompression is slower
than jumping into zImage if the image isn't relocated.

Best regards
Uwe

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

  parent reply	other threads:[~2012-02-29 20:30 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-29  0:03 [PATCH 1/2] ARM: Add Kconfig option to use mkimage -T kernel_noload Stephen Warren
2012-02-29  0:03 ` [PATCH 2/2] ARM: tegra: Add zreladdr etc to Makefile.boot for Tegra30 Stephen Warren
2012-02-29 12:29 ` [PATCH 1/2] ARM: Add Kconfig option to use mkimage -T kernel_noload Jean-Christophe PLAGNIOL-VILLARD
2012-02-29 16:58   ` Stephen Warren
2012-02-29 18:14     ` Jean-Christophe PLAGNIOL-VILLARD
2012-02-29 18:33       ` Tim Bird
2012-02-29 18:50         ` Jean-Christophe PLAGNIOL-VILLARD
2012-03-12 17:40           ` Marek Vasut
2012-03-12 17:44             ` Jean-Christophe PLAGNIOL-VILLARD
2012-02-29 19:12         ` Uwe Kleine-König
2012-02-29 19:17           ` Stephen Warren
2012-02-29 19:44             ` Uwe Kleine-König
2012-02-29 19:59               ` Stephen Warren
2012-02-29 20:12                 ` Andre Renaud
2012-02-29 20:16                   ` Stephen Warren
2012-02-29 20:19                   ` Russell King - ARM Linux
2012-02-29 20:25                     ` Andre Renaud
2012-02-29 20:29                       ` Russell King - ARM Linux
2012-02-29 20:39                       ` Uwe Kleine-König
2012-02-29 20:45                         ` Russell King - ARM Linux
2012-02-29 20:59                           ` Uwe Kleine-König
2012-03-01  4:55                             ` Jean-Christophe PLAGNIOL-VILLARD
2012-03-12 17:42                             ` Marek Vasut
2012-02-29 21:27                           ` Stephen Warren
2012-02-29 22:28                             ` Andre Renaud
2012-02-29 20:30                   ` Uwe Kleine-König [this message]
2012-02-29 23:53               ` Ryan Mallon
2012-03-12 17:44                 ` Marek Vasut
2012-03-01 15:34               ` Mark Brown
2012-03-01 17:01 ` Stephen Warren
2012-03-05  4:21   ` Olof Johansson
2012-03-05 20:29   ` Nicolas Pitre

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120229203056.GP14173@pengutronix.de \
    --to=u.kleine-koenig@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).