Re: [PATCH 2/2] lzma support: lzma compressed kernel image

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



-----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
[email protected]
-----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-----
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux