| Message ID | 20240131094917.850-1-chenhx.fnst@fujitsu.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [v3] nfs: fix regression in handling of fsc= option in NFSv4 | expand |
On Wed, 2024-01-31 at 17:49 +0800, Chen Hanxiao wrote: > Setting the uniquifier for fscache via the fsc= mount > option is currently broken in NFSv4. > > Fix this by passing fscache_uniq to root_fc if possible. > > Signed-off-by: Chen Hanxiao <chenhx.fnst@fujitsu.com> > --- > v3: > properly set param_fsc.size > v2: > use kmemdup_nul instead of snprintf > > fs/nfs/nfs4super.c | 24 ++++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > > diff --git a/fs/nfs/nfs4super.c b/fs/nfs/nfs4super.c > index d09bcfd7db89..8da5a9c000f4 100644 > --- a/fs/nfs/nfs4super.c > +++ b/fs/nfs/nfs4super.c > @@ -145,6 +145,7 @@ static int do_nfs4_mount(struct nfs_server *server, > const char *export_path) > { > struct nfs_fs_context *root_ctx; > + struct nfs_fs_context *ctx; > struct fs_context *root_fc; > struct vfsmount *root_mnt; > struct dentry *dentry; > @@ -157,6 +158,12 @@ static int do_nfs4_mount(struct nfs_server *server, > .dirfd = -1, > }; > > + struct fs_parameter param_fsc = { > + .key = "fsc", > + .type = fs_value_is_string, > + .dirfd = -1, > + }; > + > if (IS_ERR(server)) > return PTR_ERR(server); > > @@ -168,9 +175,26 @@ static int do_nfs4_mount(struct nfs_server *server, > kfree(root_fc->source); > root_fc->source = NULL; > > + ctx = nfs_fc2context(fc); > root_ctx = nfs_fc2context(root_fc); > root_ctx->internal = true; > root_ctx->server = server; > + > + if (ctx->fscache_uniq) { > + len = strlen(ctx->fscache_uniq); > + param_fsc.size = len; > + param_fsc.string = kmemdup_nul(ctx->fscache_uniq, len, GFP_KERNEL); > + if (param_fsc.string == NULL) { > + put_fs_context(root_fc); > + return -ENOMEM; > + } > + ret = vfs_parse_fs_param(root_fc, ¶m_fsc); > + kfree(param_fsc.string); > + if (ret < 0) { > + put_fs_context(root_fc); > + return ret; > + } > + } > /* We leave export_path unset as it's not used to find the root. */ > > len = strlen(hostname) + 5; Looks good: Reviewed-by: Jeff Layton <jlayton@kernel.org>
diff --git a/fs/nfs/nfs4super.c b/fs/nfs/nfs4super.c index d09bcfd7db89..8da5a9c000f4 100644 --- a/fs/nfs/nfs4super.c +++ b/fs/nfs/nfs4super.c @@ -145,6 +145,7 @@ static int do_nfs4_mount(struct nfs_server *server, const char *export_path) { struct nfs_fs_context *root_ctx; + struct nfs_fs_context *ctx; struct fs_context *root_fc; struct vfsmount *root_mnt; struct dentry *dentry; @@ -157,6 +158,12 @@ static int do_nfs4_mount(struct nfs_server *server, .dirfd = -1, }; + struct fs_parameter param_fsc = { + .key = "fsc", + .type = fs_value_is_string, + .dirfd = -1, + }; + if (IS_ERR(server)) return PTR_ERR(server); @@ -168,9 +175,26 @@ static int do_nfs4_mount(struct nfs_server *server, kfree(root_fc->source); root_fc->source = NULL; + ctx = nfs_fc2context(fc); root_ctx = nfs_fc2context(root_fc); root_ctx->internal = true; root_ctx->server = server; + + if (ctx->fscache_uniq) { + len = strlen(ctx->fscache_uniq); + param_fsc.size = len; + param_fsc.string = kmemdup_nul(ctx->fscache_uniq, len, GFP_KERNEL); + if (param_fsc.string == NULL) { + put_fs_context(root_fc); + return -ENOMEM; + } + ret = vfs_parse_fs_param(root_fc, ¶m_fsc); + kfree(param_fsc.string); + if (ret < 0) { + put_fs_context(root_fc); + return ret; + } + } /* We leave export_path unset as it's not used to find the root. */ len = strlen(hostname) + 5;
Setting the uniquifier for fscache via the fsc= mount option is currently broken in NFSv4. Fix this by passing fscache_uniq to root_fc if possible. Signed-off-by: Chen Hanxiao <chenhx.fnst@fujitsu.com> --- v3: properly set param_fsc.size v2: use kmemdup_nul instead of snprintf fs/nfs/nfs4super.c | 24 ++++++++++++++++++++++++ 1 file changed, 24 insertions(+)