public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Frank Sorenson <frank@tuxrocks.com>
To: Christian Leber <christian@leber.de>
Cc: linux-kernel@vger.kernel.org, akpm@osdl.org
Subject: Re: [PATCH 2/2] lzma support: lzma compressed kernel image
Date: Mon, 13 Jun 2005 20:10:25 -0600	[thread overview]
Message-ID: <42AE3C91.4090904@tuxrocks.com> (raw)
In-Reply-To: <20050612223150.GA26370@core.home>

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Christian Leber wrote:
> Building the lzma tool cut and paste style:
> 
> mkdir build-lzma
> wget http://www.7-zip.org/dl/lzma417.tar.bz2
> tar xjf lzma417.tar.bz2
> cd SRC/7zip/Compress/LZMA_Alone/
> make
> cp lzma ../../../../../
> cd ../../../../../

Other than the coding style issues which need to be addressed, the
patches appear to work as advertised.  lzma reduced my kernel by
approximately 25%, so I'd say it looks promising.

I do have a gripe about the need for the external lzma program when
building the kernel.  gzip can reasonably be expected to be on people's
systems, but lzma is (currently, at least) rather obscure, and isn't
likely to be on very many systems.

I think one or more of the following ought to happen:
- - Modify the help text in the Kconfig option to show people how to
obtain, compile, and install lzma (and warn them they'll need to install
it).
- - Detect that the lzma application isn't present, and fall back to gzip
(with a warning) if lzma fails.
- - If we can embed the decompressor into the boot-time kernel, can't we
put a compressor into the kernel source, and avoid the need for the
external program?
- - ...

Frank
- --
Frank Sorenson - KD7TZK
Systems Manager, Computer Science Department
Brigham Young University
frank@tuxrocks.com
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.6 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org

iD8DBQFCrjyRaI0dwg4A47wRAk3PAKCZiMzdEP1UefLN3ETO/c64zrjedACfaz+T
RSAM/qgCehDj2Gs6e03T3RY=
=oIaY
-----END PGP SIGNATURE-----

  parent reply	other threads:[~2005-06-14  1:12 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-06-07 21:41 [PATCH 2/2] lzma support: lzma compressed kernel image Christian Leber
2005-06-12 22:31 ` Christian Leber
2005-06-13  4:25   ` Willy Tarreau
2005-06-14  2:10   ` Frank Sorenson [this message]
2005-06-14 10:31     ` Christian Leber
2005-06-15 21:27       ` Frank Sorenson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=42AE3C91.4090904@tuxrocks.com \
    --to=frank@tuxrocks.com \
    --cc=akpm@osdl.org \
    --cc=christian@leber.de \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox