All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.