From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-vk0-x244.google.com ([2607:f8b0:400c:c05::244]) by bombadil.infradead.org with esmtps (Exim 4.85_2 #1 (Red Hat Linux)) id 1bxSxD-0003ue-90 for barebox@lists.infradead.org; Fri, 21 Oct 2016 06:05:20 +0000 Received: by mail-vk0-x244.google.com with SMTP id b186so4278437vkb.3 for ; Thu, 20 Oct 2016 23:04:57 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <20161004194020.vmmm35546csmbvnv@pengutronix.de> References: <20161004101048.12970-1-ejo@pengutronix.de> <20161004101048.12970-4-ejo@pengutronix.de> <20161004194020.vmmm35546csmbvnv@pengutronix.de> From: Yegor Yefremov Date: Fri, 21 Oct 2016 08:04:35 +0200 Message-ID: List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH 3/5] fs: squashfs: port zlib compression support from kernel To: Sascha Hauer Cc: barebox , Enrico Jorns Hi Enrico, On Tue, Oct 4, 2016 at 9:40 PM, Sascha Hauer wrote: > On Tue, Oct 04, 2016 at 12:10:46PM +0200, Enrico Jorns wrote: >> As this is the default compression method for squashfs, make this the >> default in kconfig selection, too >> >> Signed-off-by: Enrico Jorns >> --- >> fs/squashfs/Kconfig | 20 ++++++- >> fs/squashfs/Makefile | 1 + >> fs/squashfs/zlib_wrapper.c | 132 +++++++++++++++++++++++++++++++++++++++++++++ >> 3 files changed, 152 insertions(+), 1 deletion(-) >> create mode 100644 fs/squashfs/zlib_wrapper.c >> >> diff --git a/fs/squashfs/Kconfig b/fs/squashfs/Kconfig >> index d2de168..1cb1ac5 100644 >> --- a/fs/squashfs/Kconfig >> +++ b/fs/squashfs/Kconfig >> @@ -17,6 +17,19 @@ menuconfig FS_SQUASHFS >> embedded systems where low overhead is needed. Further information >> and tools are available from http://squashfs.sourceforge.net. >> >> +config SQUASHFS_ZLIB >> + bool "Include support for ZLIB compressed file systems" >> + depends on FS_SQUASHFS >> + select ZLIB >> + default y >> + help >> + ZLIB compression is the standard compression used by Squashfs >> + file systems. It offers a good trade-off between compression >> + achieved and the amount of CPU time and memory necessary to >> + compress and decompress. >> + >> + If unsure, say Y. >> + >> config SQUASHFS_LZ4 >> bool "Include support for LZ4 compressed file systems" >> depends on FS_SQUASHFS >> @@ -30,9 +43,10 @@ config SQUASHFS_LZ4 >> LZ4 is not the standard compression used in Squashfs and so most >> file systems will be readable without selecting this option. >> >> + If unsure, say N. >> + >> config SQUASHFS_LZO >> bool "Include support for LZO compressed file systems" >> - default y >> depends on FS_SQUASHFS >> select LZO_DECOMPRESS >> help >> @@ -44,6 +58,8 @@ config SQUASHFS_LZO >> LZO is not the standard compression used in Squashfs and so most >> file systems will be readable without selecting this option. >> >> + If unsure, say N. >> + >> config SQUASHFS_XZ >> bool "Include support for XZ compressed file systems" >> default y >> @@ -57,3 +73,5 @@ config SQUASHFS_XZ >> >> XZ is not the standard compression used in Squashfs and so most >> file systems will be readable without selecting this option. >> + >> + If unsure, say N. > > Here unrelated Kconfig entries are changed. Also in the end we get > "default y" for SQUASHFS_XZ along with the help "If unsure, say N." > which seems inconsistent. Care to send v2? @Sascha what is the plan as to the default compression selection? Leave XZ or set it to ZLIB? I'm open for both options. Thanks. Yegor _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox