From: Andreas Dilger <adilger@sun.com>
Subject: Re: [2.6 patch] FS_MBCACHE: don't needlessly make it built-in
Date: Sun, 17 Aug 2008 12:04:31 -0700
Message-ID: <20080817190431.GG4199@webber.adilger.int>
References: <20080803184325.GC7442@cs181140183.pp.htv.fi>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7BIT
Cc: Andreas Gruenbacher <agruen@suse.de>, linux-ext4@vger.kernel.org
To: Adrian Bunk <bunk@kernel.org>
In-reply-to: <20080803184325.GC7442@cs181140183.pp.htv.fi>
Content-disposition: inline
Sender: linux-ext4-owner@vger.kernel.org

On Aug 03, 2008  21:43 +0300, Adrian Bunk wrote:
> Assume you have:
> - one or more of ext2/3/4 statically built into your kernel
> - none of these with extended attributes enabled and
> - want to add onother one of ext2/3/4 modular and with
>   extended attributes enabled
> 
> then you currently have to reboot to use it since this results in 
> CONFIG_FS_MBCACHE=y.
> 
> That's not a common issue, but I just ran into it and since there's no 
> reason to get a built-in mbcache in this case this patch fixes it.

To be honest, I'd like an option to disable MBCACHE entirely.  This
code is of no use if the EAs on an inode are not identical (i.e. if
anything other than ACLs are in use) and is also not useful if the
xattrs fit into the large inodes in ext4 (and ext3 if the filesystem
is formatted with this option).

Cheers, Andreas
--
Andreas Dilger
Sr. Staff Engineer, Lustre Group
Sun Microsystems of Canada, Inc.