Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756116Ab0KXW0X (ORCPT ); Wed, 24 Nov 2010 17:26:23 -0500 Received: from smtp1.linux-foundation.org ([140.211.169.13]:46235 "EHLO smtp1.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755980Ab0KXW0V (ORCPT ); Wed, 24 Nov 2010 17:26:21 -0500 Date: Wed, 24 Nov 2010 14:25:49 -0800 From: Andrew Morton To: Lasse Collin Cc: linux-kernel@vger.kernel.org, linux-embedded@vger.kernel.org, "H. Peter Anvin" , Alain Knaff , Albin Tonnerre , Phillip Lougher Subject: Re: [PATCH RFC 1/3] Decompressors: Add XZ decompressor module Message-Id: <20101124142549.c1b9ad14.akpm@linux-foundation.org> In-Reply-To: <201011242251.52927.lasse.collin@tukaani.org> References: <201011242251.52927.lasse.collin@tukaani.org> X-Mailer: Sylpheed 3.0.2 (GTK+ 2.20.1; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2225 Lines: 62 On Wed, 24 Nov 2010 22:51:52 +0200 Lasse Collin wrote: > From: Lasse Collin > > In userspace, the .lzma format has become mostly a legacy > file format that got superseded by the .xz format. Similarly, > LZMA Utils was superseded by XZ Utils. > > These patches add support for XZ decompression into > the kernel. Most of the code is as is from XZ Embedded > . It was written for > the Linux kernel but is usable in other projects too. > > Advantages of XZ over the current LZMA code in the kernel: > - Nice API that can be used by other kernel modules; it's > not limited to kernel, initramfs, and initrd decompression. > - Integrity check support (CRC32) > - BCJ filters improve compression of executable code on > certain architectures. These together with LZMA2 can > produce a few percent smaller kernel or Squashfs images > than plain LZMA without making the decompression slower. > > This patch: Add the main decompression code (xz_dec), testing > module (xz_dec_test), wrapper script (xz_wrap.sh) for the xz > command line tool, and documentation. The xz_dec module is > enough to have a usable XZ decompressor e.g. for Squashfs. I'm not seeing any documentation which tells me how to create, install and execute xs-compressed kernels. There are new makefile targets? > > ... > > +#define bcj_x86_test_msbyte(b) ((b) == 0x00 || (b) == 0xFF) This should be written in C. It looks nicer, and so bcj_x86_test_msbyte(*p++) won't explode. > +static noinline_for_stack size_t bcj_x86( hm, but it uses little stack space. > +static noinline_for_stack size_t bcj_x86( > + struct xz_dec_bcj *s, uint8_t *buf, size_t size) The preferred style is static noinline_for_stack size_t bcj_x86(struct xz_dec_bcj *s, uint8_t *buf, size_t size) or static noinline_for_stack size_t bcj_x86(struct xz_dec_bcj *s, uint8_t *buf, size_t size) (lots of dittoes) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/