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]
Date:   Thu, 2 Jun 2022 13:04:21 +0200
From:   Lukas Czerner <lczerner@...hat.com>
To:     Yang Xu <xuyang2018.jy@...itsu.com>
Cc:     linux-ext4@...r.kernel.org
Subject: Re: [PATCH] ext4: Remove deprecated flag for noacl and noxattr_user
 mount options

On Thu, Jun 02, 2022 at 06:01:39PM +0800, Yang Xu wrote:
> Since kernel commit f70486055ee3 ("ext4: try to deprecate noacl and
> noxattr_user mount options"), we deprecated these two mount options
> because no other filesystem used the
> 
> But now, acl has been used by ext4 ext2 btrfs f2fs ocfs2 and noxattr_user
> has been used by ext4 ext2 f2fs ocfs2.

And many other fs don't have it. Is that your only reason for dropping
the deprecation? I can easily imagine that those fs got it because ext4
had it at the time.

Moreover the deprecation message has been there for 10 years, have we
seen anyone actually complaining that they want to keep it?

Why not to just remove the option. I don't have a strong opinion either
way, but it would be nice to remove stuff we don't need. Do you have a
use case for it? If not, can we make it Opt_removed?

-Lukas

> 
> I think it is time to remove deprecated flag for them.
> 
> Signed-off-by: Yang Xu <xuyang2018.jy@...itsu.com>
> ---
>  fs/ext4/super.c | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/fs/ext4/super.c b/fs/ext4/super.c
> index 450c918d68fc..8a0cc8815ee7 100644
> --- a/fs/ext4/super.c
> +++ b/fs/ext4/super.c
> @@ -2116,10 +2116,6 @@ static int ext4_parse_param(struct fs_context *fc, struct fs_parameter *param)
>  		else
>  			return note_qf_name(fc, GRPQUOTA, param);
>  #endif
> -	case Opt_noacl:
> -	case Opt_nouser_xattr:
> -		ext4_msg(NULL, KERN_WARNING, deprecated_msg, param->key, "3.5");
> -		break;
>  	case Opt_sb:
>  		if (fc->purpose == FS_CONTEXT_FOR_RECONFIGURE) {
>  			ext4_msg(NULL, KERN_WARNING,
> -- 
> 2.23.0
> 

Powered by blists - more mailing lists