2014-01-28 16:14:46

by Mark Brown

[permalink] [raw]
Subject: Possible stable fix for decompressor speed

The changes in patch 1431574a1c4c (lib/decompressors: fix "no limit"
output buffer length) avoid doing decompression a byte at a time when
decompressing into high physical addresses using a small, well isolated
change. The patch can give a very noticable improvement in kernel boot
times on affected systems, for example with one ARM system this reduces
the total time to boot the kernel by more than a third.

Would you consider this patch for stable? It doesn't quite fit within
the criteria but you've indicated in the past that such isolated and
well supported changs can be acceptable anyway. If it's not OK for
stable it seems like it should be a good candidate for LTSI.


Attachments:
(No filename) (681.00 B)
signature.asc (836.00 B)
Digital signature
Download all attachments

2014-01-28 16:26:59

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: Possible stable fix for decompressor speed

On Tue, Jan 28, 2014 at 04:14:28PM +0000, Mark Brown wrote:
> The changes in patch 1431574a1c4c (lib/decompressors: fix "no limit"
> output buffer length) avoid doing decompression a byte at a time when
> decompressing into high physical addresses using a small, well isolated
> change. The patch can give a very noticable improvement in kernel boot
> times on affected systems, for example with one ARM system this reduces
> the total time to boot the kernel by more than a third.
>
> Would you consider this patch for stable? It doesn't quite fit within
> the criteria but you've indicated in the past that such isolated and
> well supported changs can be acceptable anyway. If it's not OK for
> stable it seems like it should be a good candidate for LTSI.

It's a one-line change that seems well tested and provides a noticed
speedup, so yes, I can take it for the 3.10-stable tree, thanks.

greg k-h

2014-02-03 13:59:20

by Luis Henriques

[permalink] [raw]
Subject: Re: Possible stable fix for decompressor speed

On Tue, Jan 28, 2014 at 04:14:28PM +0000, Mark Brown wrote:
> The changes in patch 1431574a1c4c (lib/decompressors: fix "no limit"
> output buffer length) avoid doing decompression a byte at a time when
> decompressing into high physical addresses using a small, well isolated
> change. The patch can give a very noticable improvement in kernel boot
> times on affected systems, for example with one ARM system this reduces
> the total time to boot the kernel by more than a third.
>
> Would you consider this patch for stable? It doesn't quite fit within
> the criteria but you've indicated in the past that such isolated and
> well supported changs can be acceptable anyway. If it's not OK for
> stable it seems like it should be a good candidate for LTSI.

Thanks Mark, I'm queuing this for the 3.5 and the 3.11 kernels as well.

Cheers,
--
Luis