linux-ext4 - Re: [2.6 patch] FS_MBCACHE: don't needlessly make it built-in

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Sun, 17 Aug 2008 22:20:05 -0500
From:	Eric Sandeen <sandeen@...hat.com>
To:	Andreas Dilger <adilger@....com>
CC:	Adrian Bunk <bunk@...nel.org>,
	Andreas Gruenbacher <agruen@...e.de>,
	linux-ext4@...r.kernel.org
Subject: Re: [2.6 patch] FS_MBCACHE: don't needlessly make it built-in

Andreas Dilger wrote:
> 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)

or selinux....

> 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).

... which is now the default.

a config option sounds reasonable to me, too.  I think by the time EAs
spill out of the inode, the chance of them being identical is pretty
small?  (i.e. a single acl set, or selinux context may be common, but if
you have enough to not fit in the inode it's more likely to be a
mishmash of things?)

-Eric


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

--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html