On 2/16/11 3:49 PM, Ted Ts'o wrote: > On Wed, Feb 16, 2011 at 01:27:39PM -0600, Eric Sandeen wrote: >> On 2/16/11 1:15 PM, Andreas Dilger wrote: >>> On 2011-02-16, at 11:21, Eric Sandeen wrote: >>>> User namespace xattrs are generally useful, and I think extN >>>> is the only filesystem requiring a special mount option to >>>> enable them, when xattrs are otherwise available. So this >>>> change sets that mount option into the defaults. >>>> >>> >>> Wouldn't it be more useful to change this in the kernel, instead of only changing it in the superblock for new filesystems? >>> >> >> I thought about that; maybe it should happen in both places? > > Yes, agreed. Maybe we should do the same thing with posix ACL's as > well? ACL's can only take away access as compared to the unix > permission bits, so it's safe to enable ACL's. > > - Ted > Whoops, I didn't realize that acls were in the same boat, of "config on and then turn on with a mount option" Ok, sure, adding that makes sense to me too... Is sticking it in the _initialize() function ok with you? It's a little odd, but ... -Eric -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html