ntfs3.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
* [PATCH v2] fs/ntfs3: Remove deprecated mount options nls
@ 2021-09-21 17:19 Kari Argillander
  2021-09-23 16:12 ` Konstantin Komarov
  0 siblings, 1 reply; 2+ messages in thread
From: Kari Argillander @ 2021-09-21 17:19 UTC (permalink / raw)
  To: Konstantin Komarov, ntfs3; +Cc: Kari Argillander, linux-kernel, Pali Rohár

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 <kari.argillander@gmail.com>
---
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


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH v2] fs/ntfs3: Remove deprecated mount options nls
  2021-09-21 17:19 [PATCH v2] fs/ntfs3: Remove deprecated mount options nls Kari Argillander
@ 2021-09-23 16:12 ` Konstantin Komarov
  0 siblings, 0 replies; 2+ messages in thread
From: Konstantin Komarov @ 2021-09-23 16:12 UTC (permalink / raw)
  To: Kari Argillander, ntfs3; +Cc: linux-kernel, Pali Rohár



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 <kari.argillander@gmail.com>
> ---
> 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!

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-09-23 16:12 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-21 17:19 [PATCH v2] fs/ntfs3: Remove deprecated mount options nls Kari Argillander
2021-09-23 16:12 ` Konstantin Komarov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).