Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel
From: Nicolas Pitre
Date: Wed Feb 27 2013 - 12:17:01 EST
On Wed, 27 Feb 2013, Joe Perches wrote:
> On Wed, 2013-02-27 at 16:31 +0000, Russell King - ARM Linux wrote:
> > On Wed, Feb 27, 2013 at 07:49:12AM -0800, Joe Perches wrote:
> > > On Wed, 2013-02-27 at 09:56 +0000, Russell King - ARM Linux wrote:
> > > > On Tue, Feb 26, 2013 at 05:40:34PM -0800, Joe Perches wrote:
> > > > > On Tue, 2013-02-26 at 22:10 +0000, Russell King - ARM Linux wrote:
> > > > > > So... for a selected kernel version of a particular size, can we please
> > > > > > have a comparison between the new LZO code and this LZ4 code, so that
> > > > > > we can see whether it's worth updating the LZO code or replacing the
> > > > > > LZO code with LZ4?
> > > > >
> > > > > How could it be questionable that it's worth updating the LZO code?
> > > >
> > > > Please read the comments against the previous posting of these patches
> > > > where I first stated this argument - and with agreement from those
> > > > following the thread. The thread started on 26 Jan 2013. Thanks.
> > >
> > > https://lkml.org/lkml/2013/1/29/145
> > >
> > > I did not and do not see significant value in
> > > adding LZ4 given Markus' LZO improvements.
> >
> > Sorry, a 66% increase in decompression speed over the updated LZO code
> > isn't "significant value" ?
>
> We disagree.
>
> > I'm curious - what in your mind qualifies "significant value" ?
>
> faster boot time. smaller, faster overall code.
Sorry, but you certainly successfully got me confused, and probably
others as well.
RMK says that "66% increase in decompression speed over LZO" is
significant. You apparently disagree with that.
Then you say that faster boot time is significant.
Again, can you (or anyone else) provide comprehensive test results in a
single email with both compression methods?
Nicolas
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/