On Mon, Feb 13, 2012 at 4:34 PM, Tom Gundersen <t...@jklm.no> wrote:
> This increases compression efficiency, which might be useful on an
> image containing a lot of modules.
>
> For the usecase I have in mind, this reduced the size by about a quarter.
+1 from me, I'd only suggest being explicit about your use case here
in the commit message, as in 1 year you aren't going to remember why
you said this (and everyone else is completely in the dark).

"For large recovery images containing several modules" or something.

>
> Signed-off-by: Tom Gundersen <t...@jklm.no>
> ---
>  mkinitcpio |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/mkinitcpio b/mkinitcpio
> index 677f608..82e2d25 100755
> --- a/mkinitcpio
> +++ b/mkinitcpio
> @@ -349,6 +349,8 @@ if (( ${#ADDED_MODULES[*]} )); then
>     msg "Generating module dependencies"
>     install -m644 -t "$BUILDROOT/lib/modules/$KERNELVERSION" \
>         "$BASEDIR/lib/modules/$KERNELVERSION"/modules.{builtin,order}
> +    # improve overall compression by storing each module uncompressed
> +    gunzip -r "$BUILDROOT/lib/modules/$KERNELVERSION"
>     depmod -b "$BUILDROOT" "${KERNELVERSION}"
>
>     # remove all non-binary module.* files (except devname for on-demand 
> module loading)
> --
> 1.7.9
>

Reply via email to