Re: update8 [PATCH 2/5] init: bzip2 or lzma -compressed kernels and initrds

Alain Knaff
2009-01-04T23:12:22Z

H. Peter Anvin wrote:
> Hi Alain,
>
> Looking pretty good now. The only issues that I see so far are:
>
> - Putting the kernel compression selection in init/Kconfig isn't going
> to work too well, since it affects all architectures, and not all
> architectures even do their own decompression (for quite a few
> architectures it's the boot loader's responsibility.)
>
> I see two options here: either have this be parameterized by the arch
> Kconfig files, or simply move this chunk into arch/*/Kconfig as part
> of the arch-enablement patches. The latter is probably the simplest,
> even if it means some replicated code.
>
> Let me know what you think

What do you mean "parametrized by the arch Kconfig files"? Does this
mean that it would still be in the global files, but would be enabled by
some kind of switch or preprocessor macro in the arch dependant file? If
easily doable, this would IMHO be preferable.

However, if not easily feasible, I'm also ok with duplication.

> -- I can do this pretty easily while
> importing, so you don't need to submit a full new patchset.

Thanks, that would be nice.

> - I did a followon patch (attached) to change the ramdisk compression
> search to table driven. Since I already did the followon patch, don't
> worry about it.
>
> -hpa
>
>

Ok

Thanks,

Alain