From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from qw-out-1920.google.com ([74.125.92.144]) by bombadil.infradead.org with esmtp (Exim 4.69 #1 (Red Hat Linux)) id 1NAOzY-0003is-Li for kexec@lists.infradead.org; Tue, 17 Nov 2009 14:25:17 +0000 Received: by qw-out-1920.google.com with SMTP id 5so5391qwc.24 for ; Tue, 17 Nov 2009 06:25:11 -0800 (PST) From: Florian Fainelli Subject: Re: [PATCH] add support for loading lzma compressed kernels Date: Tue, 17 Nov 2009 15:23:15 +0100 References: <200911160053.10339.florian@openwrt.org> <20091117030429.GC12541@verge.net.au> <200911171508.29971.florian@openwrt.org> In-Reply-To: <200911171508.29971.florian@openwrt.org> MIME-Version: 1.0 Message-Id: <200911171523.15419.florian@openwrt.org> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: kexec-bounces@lists.infradead.org Errors-To: kexec-bounces+dwmw2=infradead.org@lists.infradead.org To: kexec@lists.infradead.org Cc: Simon Horman , "Eric W. Biederman" On Tuesday 17 November 2009 15:08:29 Florian Fainelli wrote: > Hello Simon, > > On Tuesday 17 November 2009 04:04:38 Simon Horman wrote: > > On Mon, Nov 16, 2009 at 12:53:06AM +0100, Florian Fainelli wrote: > > > Hi Eric, > > > > > > This patch allows one to load a lzma compressed kernel using kexec -l. > > > As I wanted the lzma code to be very similar to the existing > > > zlib slurp_decompress I took lzread and associated routines > > > from the cpio lzma support. Tested on my x86 laptop using the > > > following commands: > > > > > > lzma e bzImage bzImage.lzma > > > kexec -l bzImage.lzma > > > > > > Having lzma support is particularly useful on some embedded > > > systems on which we have the kernel already lzma compressed > > > and available on a mtd partition. > > > > > > Signed-off-by: Florian Fainelli > > > > Should lzma_code_ be lzma_code. The former doesn't seem to work with > > liblzma 4.999.9beta+20091016-1 from Debian. > > You are right it's actually lzma_code (without the trailing _). > > > > + AC_MSG_NOTICE([lzma support disabled]))) > > > > The trailing "fi" line appears to be missing. > > Fixed too. > [snip] > > > Does this imply that zlib compression isn't supported if > > lzma compression support is enabled? > > Indeed, we might want to support both at runtime. Would you agree with the > following proposal: > > - rename slurp_decompress_file to zlib/lzma_decompress_file > - in case gzopen fails, do not die, but return NULL > - test the return value of zlib_decompress_file and try > lzma_decompress_file We would also have to modify the call sites of slurp_decompress file this might become pretty heavy if we support more decompression algorithms. What do you think? -- WBR, Florian _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec