[PATCH v2] ARM: support XZ compressed kernels

Russell King - ARM Linux linux at arm.linux.org.uk
Fri Jul 22 05:57:38 EDT 2011


On Fri, Jul 22, 2011 at 11:52:19AM +0200, Imre Kaloz wrote:
> On Fri, 22 Jul 2011 11:16:04 +0200, Russell King - ARM Linux <linux at arm.linux.org.uk> wrote:
>
>> On Tue, Jul 19, 2011 at 12:07:17PM +0100, Russell King - ARM Linux wrote:
>>> On Tue, Jul 19, 2011 at 12:02:35PM +0100, Russell King - ARM Linux wrote:
>>> > On Sat, Jul 16, 2011 at 04:07:34PM +0200, Imre Kaloz wrote:
>>> > > On Fri, 15 Jul 2011 18:25:19 +0200, Russell King - ARM Linux <linux at arm.linux.org.uk> wrote:
>>> > >
>>> > >> On Tue, Jul 12, 2011 at 02:56:09PM +0200, Imre Kaloz wrote:
>>> > >>> Wire up support for the XZ decompressor
>>> > >>
>>> > >> Ok.
>>> > >
>>> > > Thanks, added to the patch system.
>>> >
>>> > FYI:
>>> >
>>> > Patching 7001/1...
>>> > <stdin>:56: space before tab in indent.
>>> >                 $(addprefix $(obj)/, $(OBJS)) $(lib1funcs) $(ashldi3) FORCE
>>> > warning: 1 line applied after fixing whitespace errors.
>>>
>>> BTW, has anyone outside of the ARM community seen the patch to
>>> lib/xz/xz_dec_stream.c adding the linux/kernel.h include?  How
>>> well tested is that change on other architectures?
>>
>> Ok, no reply to this, so I'm dropping your patch for the merge window.
>>
>
> Guessed I'm not the one outside the ARM community, so.. I can say it
> didn't break neither ppc or mips for me, but have no idea who's feedback
> are you waiting for.

What I'm concerned about is the apparant lack of exposure of the patch
to people outside of ARM, especially as it touches lib/xz/xz_dec_stream.c.

I have no idea whether that breaks some architecture or not.

Is Lasse Collin (the original contributer of that file) aware of this
change?

What is the additional include fixing on ARM?  Why isn't this include
needed on other architectures?  And why isn't any of this mentioned in
the patch commit log?



More information about the linux-arm-kernel mailing list