2021-09-21 17:20:25

by Kari Argillander

[permalink] [raw]
Subject: [PATCH v2] fs/ntfs3: Remove deprecated mount options nls

Some discussion has been spoken that this deprecated mount options
should be removed before 5.15 lands. This driver is not never seen day
light so it was decided that nls mount option has to be removed. We have
always possibility to add this if needed.

One possible need is example if current ntfs driver will be taken out of
kernel and ntfs3 needs to support mount options what it has.

Signed-off-by: Kari Argillander <[email protected]>
---
Patch is compile and checkpatch tested.

v2:
-Do not delete {}
---
fs/ntfs3/super.c | 4 ----
1 file changed, 4 deletions(-)

diff --git a/fs/ntfs3/super.c b/fs/ntfs3/super.c
index 6a535b144ff9..800897777eb0 100644
--- a/fs/ntfs3/super.c
+++ b/fs/ntfs3/super.c
@@ -246,10 +246,6 @@ static const struct fs_parameter_spec ntfs_fs_parameters[] = {
fsparam_flag_no("prealloc", Opt_prealloc),
fsparam_flag_no("acsrules", Opt_noacsrules),
fsparam_string("iocharset", Opt_iocharset),
-
- __fsparam(fs_param_is_string,
- "nls", Opt_iocharset,
- fs_param_deprecated, NULL),
{}
};


base-commit: 6354467245ff8dd04b54e39790f2ee4d21d5419e
--
2.25.1


2021-09-23 16:15:28

by Konstantin Komarov

[permalink] [raw]
Subject: Re: [PATCH v2] fs/ntfs3: Remove deprecated mount options nls



On 21.09.2021 20:19, Kari Argillander wrote:
> Some discussion has been spoken that this deprecated mount options
> should be removed before 5.15 lands. This driver is not never seen day
> light so it was decided that nls mount option has to be removed. We have
> always possibility to add this if needed.
>
> One possible need is example if current ntfs driver will be taken out of
> kernel and ntfs3 needs to support mount options what it has.
>
> Signed-off-by: Kari Argillander <[email protected]>
> ---
> Patch is compile and checkpatch tested.
>
> v2:
> -Do not delete {}
> ---
> fs/ntfs3/super.c | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/fs/ntfs3/super.c b/fs/ntfs3/super.c
> index 6a535b144ff9..800897777eb0 100644
> --- a/fs/ntfs3/super.c
> +++ b/fs/ntfs3/super.c
> @@ -246,10 +246,6 @@ static const struct fs_parameter_spec ntfs_fs_parameters[] = {
> fsparam_flag_no("prealloc", Opt_prealloc),
> fsparam_flag_no("acsrules", Opt_noacsrules),
> fsparam_string("iocharset", Opt_iocharset),
> -
> - __fsparam(fs_param_is_string,
> - "nls", Opt_iocharset,
> - fs_param_deprecated, NULL),
> {}
> };
>
>
> base-commit: 6354467245ff8dd04b54e39790f2ee4d21d5419e
>

Applied, thanks!