linux-ext4 - Re: [PATCH] ext4: enable acls and user

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <20110221134642.GC6584@quack.suse.cz>
Date:	Mon, 21 Feb 2011 14:46:42 +0100
From:	Jan Kara <jack@...e.cz>
To:	Eric Sandeen <sandeen@...hat.com>
Cc:	ext4 development <linux-ext4@...r.kernel.org>,
	Jan Kara <jack@...e.cz>
Subject: Re: [PATCH] ext4: enable acls and user_xattr by default

  Hi Eric,

On Fri 18-02-11 14:31:01, Eric Sandeen wrote:
> There's no good reason to require the extra step of providing
> a mount option for acl or user_xattr once the feature is configured
> on; no other filesystem that I know of requires this.
> 
> Userspace patches have set these options in default mount options,
> and this patch makes them default in the kernel.  At some point
> we can start to deprecate the options, perhaps.
> 
> For now I've removed default mount option checks in show_options()
> to be explicit about what's set, since it's changing the default,
> but I'm open to alternatives if desired.
> 
> Signed-off-by: Eric Sandeen <sandeen@...hat.com>
> ---
> 
> p.s. I've got ext2 & ext3 patches too - Jan, is this ok with
> you for ext2/3 as well?
  I was thinking about this for a while. I definitely agree with the change
of default mount options in e2fsprogs (so that user_xattr and acl will be
turned on by default). But after such change is there really any bigger
point in changing the kernel defaults? Users won't have to specify options
for new filesystems because of new default mount options setting (so the
change does not bring any simplification) and for old filesystems the admin
has proper options in /etc/fstab and if not perhaps he didn't want e.g.
user_xattr to be supported so suddently turning the support on with newer
kernel will confuse them)?

So if we wanted to do such a transition for consistency with other
filesystems, we should maybe first start warning that people should use
nouser_xattr and noacl when they really didn't want them?

								Honza
-- 
Jan Kara <jack@...e.cz>
SUSE Labs, CR
--
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