lkml.org 
[lkml]   [2020]   [May]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH] exfat: add the dummy mount options to be backward compatible with staging/exfat
Date
> On Thu, May 21, 2020 at 10:44:28AM -0500, Eric Sandeen wrote:
> > On 5/21/20 9:05 AM, Namjae Jeon wrote:
> > > As Ubuntu and Fedora release new version used kernel version equal
> > > to or higher than v5.4, They started to support kernel exfat filesystem.
> > >
> > > Linus Torvalds reported mount error with new version of exfat on Fedora.
> > >
> > > exfat: Unknown parameter 'namecase'
> > >
> > > This is because there is a difference in mount option between old
> > > staging/exfat and new exfat.
> > > And utf8, debug, and codepage options as well as namecase have been
> > > removed from new exfat.
> > >
> > > This patch add the dummy mount options as deprecated option to be
> > > backward compatible with old one.
> >
> > Wow, it seems wild that we'd need to maintain compatibility with
> > options which only ever existed in a different codebase in a staging
> > driver (what's the point of staging if every interface that makes it
> > that far has to be maintained in perpetuity?)
> >
> > Often, when things are deprecated, they are eventually removed.
> > Perhaps a future removal date stated in this commit, or in
> > Documentation/..../exfat.txt would be good as a reminder to eventually remove this?
>
> For NFS, 'intr' has been deprecated since December 2007 and has been printing a warning since June
> 2008. How long until we delete it?
>
> > > static const struct constant_table exfat_param_enums[] = { @@
> > > -223,6 +229,10 @@ static const struct fs_parameter_spec exfat_parameters[] = {
> > > fsparam_enum("errors", Opt_errors, exfat_param_enums),
> > > fsparam_flag("discard", Opt_discard),
> > > fsparam_s32("time_offset", Opt_time_offset),
> > > + fsparam_flag("utf8", Opt_utf8),
> > > + fsparam_flag("debug", Opt_debug),
> > > + fsparam_u32("namecase", Opt_namecase),
> > > + fsparam_u32("codepage", Opt_codepage),
>
> __fsparam(NULL, "utf8", Opt_utf8, fs_param_deprecated, NULL),
> __fsparam(NULL, "debug", Opt_debug, fs_param_deprecated, NULL),
> __fsparam(fs_param_is_u32, "namecase", Opt_namecase,
> fs_param_deprecated, NULL),
> __fsparam(fs_param_is_u32, "codepage", Opt_codepage,
> fs_param_deprecated, NULL),
>
> > > @@ -278,6 +288,18 @@ static int exfat_parse_param(struct fs_context *fc, struct fs_parameter
> *param)
> > > return -EINVAL;
> > > opts->time_offset = result.int_32;
> > > break;
> > > + case Opt_utf8:
> > > + pr_warn("exFAT-fs: 'utf8' mount option is deprecated and has no effect\n");
> > > + break;
> > > + case Opt_debug:
> > > + pr_warn("exFAT-fs: 'debug' mount option is deprecated and has no effect\n");
> > > + break;
> > > + case Opt_namecase:
> > > + pr_warn("exFAT-fs: 'namecase' mount option is deprecated and has no effect\n");
> > > + break;
> > > + case Opt_codepage:
> > > + pr_warn("exFAT-fs: 'codepage' mount option is deprecated and has no effect\n");
> > > + break;
>
> and then you don't need this hunk because the fs parser will print the deprecated message for you.
Fixed it on v3.
Thanks for your review!


\
 
 \ /
  Last update: 2020-05-22 01:44    [W:0.035 / U:0.284 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site