Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Joe Perches
(removed Richard Purdie and Albin Tonnerre as their email addresses seem to be bounding) > While recently asking someone to enable VFP debugging, so I could help > sort out a problem they had reported, this is the debug output I was > greeted by thanks to your meddling: :) Meddling... You sound

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Andrew Morton
On Wed, 27 Feb 2013 09:51:39 + Russell King - ARM Linux wrote: > On Wed, Feb 27, 2013 at 04:36:47PM +0900, Kyungsik Lee wrote: > > Compiler: Linaro ARM gcc 4.6.2 > > 2. ARMv7, 1.7GHz based board > >Kernel: linux 3.7 > >Uncompressed Kernel Size: 14MB > > Compressed Size Decom

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Russell King - ARM Linux
On Wed, Feb 27, 2013 at 09:39:47AM -0800, Joe Perches wrote: > On Wed, 2013-02-27 at 12:16 -0500, Nicolas Pitre wrote: > > On Wed, 27 Feb 2013, Joe Perches wrote: > > > On Wed, 2013-02-27 at 16:31 +, Russell King - ARM Linux wrote: > > > > On Wed, Feb 27, 2013 at 07:49:12AM -0800, Joe Perches w

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Nicolas Pitre
On Wed, 27 Feb 2013, Joe Perches wrote: > On Wed, 2013-02-27 at 12:16 -0500, Nicolas Pitre wrote: > > RMK says that "66% increase in decompression speed over LZO" is > > significant. You apparently disagree with that. > > Yeah, I can see how that can be interpreted. > I'm referring only to the

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Joe Perches
On Wed, 2013-02-27 at 12:16 -0500, Nicolas Pitre wrote: > On Wed, 27 Feb 2013, Joe Perches wrote: > > On Wed, 2013-02-27 at 16:31 +, 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 +, Russell King - ARM L

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Russell King - ARM Linux
On Wed, Feb 27, 2013 at 09:04:48AM -0800, Joe Perches wrote: > On Wed, 2013-02-27 at 16:31 +, 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 +, Russell King - ARM Linux wrote: > > > > On Tue, Feb 26, 2013 at

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Nicolas Pitre
On Wed, 27 Feb 2013, Joe Perches wrote: > On Wed, 2013-02-27 at 16:31 +, 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 +, Russell King - ARM Linux wrote: > > > > On Tue, Feb 26, 2013 at 05:40:34PM -0800, J

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Joe Perches
On Wed, 2013-02-27 at 16:31 +, 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 +, Russell King - ARM Linux wrote: > > > On Tue, Feb 26, 2013 at 05:40:34PM -0800, Joe Perches wrote: > > > > On Tue, 2013-02-26 at 2

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Borislav Petkov
On Wed, Feb 27, 2013 at 04:31:18PM +, Russell King - ARM Linux wrote: > I'm not saying that the LZO code should not be updated. I'm saying > that the kernel boot time decompressor is not a play ground for an > ever increasing number of "my favourite compression method" crap. > We don't need fou

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Russell King - ARM Linux
On Wed, Feb 27, 2013 at 07:49:12AM -0800, Joe Perches wrote: > On Wed, 2013-02-27 at 09:56 +, 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 +, Russell King - ARM Linux wrote: > > > > So... for a selected ke

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Nicolas Pitre
On Wed, 27 Feb 2013, Joe Perches wrote: > https://lkml.org/lkml/2013/1/29/145 Connecting to lkml.org (lkml.org)|87.253.128.182|:443... connected. HTTP request sent, awaiting response... 500 Server Error > I did not and do not see significant value in > adding LZ4 given Markus' LZO improvements.

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Joe Perches
On Wed, 2013-02-27 at 09:56 +, 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 +, Russell King - ARM Linux wrote: > > > So... for a selected kernel version of a particular size, can we please > > > have a compari

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Nicolas Pitre
On Wed, 27 Feb 2013, Johannes Stezenbach wrote: > On Wed, Feb 27, 2013 at 09:51:39AM +, Russell King - ARM Linux wrote: > > On Wed, Feb 27, 2013 at 04:36:47PM +0900, Kyungsik Lee wrote: > > > Compiler: Linaro ARM gcc 4.6.2 > > > 2. ARMv7, 1.7GHz based board > > >Kernel: linux 3.7 > > >

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Kyungsik Lee
On Wed, Feb 27, 2013 at 09:51:39AM +, Russell King - ARM Linux wrote: > On Wed, Feb 27, 2013 at 04:36:47PM +0900, Kyungsik Lee wrote: > > Compiler: Linaro ARM gcc 4.6.2 > > 2. ARMv7, 1.7GHz based board > >Kernel: linux 3.7 > >Uncompressed Kernel Size: 14MB > > Compressed Size

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Johannes Stezenbach
On Wed, Feb 27, 2013 at 09:51:39AM +, Russell King - ARM Linux wrote: > On Wed, Feb 27, 2013 at 04:36:47PM +0900, Kyungsik Lee wrote: > > Compiler: Linaro ARM gcc 4.6.2 > > 2. ARMv7, 1.7GHz based board > >Kernel: linux 3.7 > >Uncompressed Kernel Size: 14MB > > Compressed Size

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Russell King - ARM Linux
On Tue, Feb 26, 2013 at 05:40:34PM -0800, Joe Perches wrote: > On Tue, 2013-02-26 at 22:10 +, 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 whet

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-27 Thread Russell King - ARM Linux
On Wed, Feb 27, 2013 at 04:36:47PM +0900, Kyungsik Lee wrote: > Compiler: Linaro ARM gcc 4.6.2 > 2. ARMv7, 1.7GHz based board >Kernel: linux 3.7 >Uncompressed Kernel Size: 14MB > Compressed Size Decompression Speed > LZO 6.0MB34.1MB/sOld > ---

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-26 Thread Kyungsik Lee
On Tue, Feb 26, 2013 at 09:33:22PM +0100, Markus F.X.J. Oberhumer wrote: > On 2013-02-26 07:24, Kyungsik Lee wrote: > > Hi, > > > > [...] > > > > Through the benchmark, it was found that -Os Compiler flag for > > decompress.o brought better decompression performance in most of cases > > (ex, diff

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-26 Thread Joe Perches
On Tue, 2013-02-26 at 22:10 +, 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

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-26 Thread Russell King - ARM Linux
On Tue, Feb 26, 2013 at 10:58:02PM +0100, Peter Korsgaard wrote: > > "Nicolas" == Nicolas Pitre writes: > > Hi, > > >> Did you actually *try* the new LZO version and the patch (which is > attached > >> once again) as explained in https://lkml.org/lkml/2013/2/3/367 ? > >> > >> Because t

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-26 Thread Nicolas Pitre
On Tue, 26 Feb 2013, Peter Korsgaard wrote: > > "Nicolas" == Nicolas Pitre writes: > > Hi, > > >> Did you actually *try* the new LZO version and the patch (which is > attached > >> once again) as explained in https://lkml.org/lkml/2013/2/3/367 ? > >> > >> Because the new LZO version i

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-26 Thread Peter Korsgaard
> "Nicolas" == Nicolas Pitre writes: Hi, >> Did you actually *try* the new LZO version and the patch (which is attached >> once again) as explained in https://lkml.org/lkml/2013/2/3/367 ? >> >> Because the new LZO version is faster than LZ4 in my testing, at least >> when comparing app

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-26 Thread Nicolas Pitre
On Tue, 26 Feb 2013, Markus F.X.J. Oberhumer wrote: > On 2013-02-26 07:24, Kyungsik Lee wrote: > > Hi, > > > > [...] > > > > Through the benchmark, it was found that -Os Compiler flag for > > decompress.o brought better decompression performance in most of cases > > (ex, different compiler and h

Re: [RFC PATCH v2 0/4] Add support for LZ4-compressed kernel

2013-02-26 Thread Markus F.X.J. Oberhumer
On 2013-02-26 07:24, Kyungsik Lee wrote: > Hi, > > [...] > > Through the benchmark, it was found that -Os Compiler flag for > decompress.o brought better decompression performance in most of cases > (ex, different compiler and hardware spec.) in ARM architecture. > > Lastly, CONFIG_HAVE_EFFICIEN