Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752942AbXFDJAa (ORCPT ); Mon, 4 Jun 2007 05:00:30 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751634AbXFDJAV (ORCPT ); Mon, 4 Jun 2007 05:00:21 -0400 Received: from lazybastard.de ([212.112.238.170]:59536 "EHLO longford.lazybastard.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751210AbXFDJAT (ORCPT ); Mon, 4 Jun 2007 05:00:19 -0400 Date: Mon, 4 Jun 2007 10:54:48 +0200 From: =?utf-8?B?SsO2cm4=?= Engel To: Arnd Bergmann Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org, akpm@osdl.org, Sam Ravnborg , John Stoffel , David Woodhouse , Jamie Lokier , Artem Bityutskiy , CaT , Jan Engelhardt , Evgeniy Polyakov , David Weinehall , Willy Tarreau , Kyle Moffett , Dongjun Shin , Pavel Machek , Bill Davidsen , Thomas Gleixner , Albert Cahalan , Pekka Enberg , Roland Dreier , Ondrej Zajicek , Ulisses Furquim Subject: Re: [Patch 06/18] fs/logfs/compr.c Message-ID: <20070604085447.GA14823@lazybastard.org> References: <20070603183845.GA8952@lazybastard.org> <20070603184355.GG8952@lazybastard.org> <200706032358.44870.arnd@arndb.de> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <200706032358.44870.arnd@arndb.de> User-Agent: Mutt/1.5.9i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1289 Lines: 32 On Sun, 3 June 2007 23:58:43 +0200, Arnd Bergmann wrote: > On Sunday 03 June 2007, Jörn Engel wrote: > > +#define COMPR_LEVEL 3 > > + > > +static DEFINE_MUTEX(compr_mutex); > > +static struct z_stream_s stream; > > Is there a particular reason to choose '3' as the only compression > level? Should this perhaps be a per-superblock option instead? There is no particular reason. '3' should be a reasonable value for most people. If actual users want to change this value, I can make it a mount option as well. Right now I'm just lazy and doubt the merits. > Also, I thought I saw discussion about making the mutex and > stream per-superblock, but don't remember if the idea was discarded. > If it was, you might want to add it to the won't-happen list. It was more or less discarded. As long as the sweet spot for LogFS is small systems, saving memory is more important than multithreaded performance. Will add it to the list. Jörn -- Joern's library part 2: http://www.art.net/~hopkins/Don/unix-haters/tirix/embarrassing-memo.html - 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/