erofs: clean up parsing of fscache related options

... to avoid the mess of conditional preprocessing as we are continually
adding fscache related mount options.

Reviewd-by: Gao Xiang <hsiangkao@linux.alibaba.com>
Reviewed-by: Yue Hu <huyue2@coolpad.com>
Reviewed-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jingbo Xu <jefflexu@linux.alibaba.com>
Link: https://lore.kernel.org/r/20230112065431.124926-3-jefflexu@linux.alibaba.com
Signed-off-by: Gao Xiang <hsiangkao@linux.alibaba.com>
This commit is contained in:
Jingbo Xu 2023-01-12 14:54:31 +08:00 committed by Gao Xiang
parent b22c7b9718
commit e02ac3e732

View File

@ -577,26 +577,25 @@ static int erofs_fc_parse_param(struct fs_context *fc,
} }
++ctx->devs->extra_devices; ++ctx->devs->extra_devices;
break; break;
case Opt_fsid:
#ifdef CONFIG_EROFS_FS_ONDEMAND #ifdef CONFIG_EROFS_FS_ONDEMAND
case Opt_fsid:
kfree(ctx->fsid); kfree(ctx->fsid);
ctx->fsid = kstrdup(param->string, GFP_KERNEL); ctx->fsid = kstrdup(param->string, GFP_KERNEL);
if (!ctx->fsid) if (!ctx->fsid)
return -ENOMEM; return -ENOMEM;
#else
errorfc(fc, "fsid option not supported");
#endif
break; break;
case Opt_domain_id: case Opt_domain_id:
#ifdef CONFIG_EROFS_FS_ONDEMAND
kfree(ctx->domain_id); kfree(ctx->domain_id);
ctx->domain_id = kstrdup(param->string, GFP_KERNEL); ctx->domain_id = kstrdup(param->string, GFP_KERNEL);
if (!ctx->domain_id) if (!ctx->domain_id)
return -ENOMEM; return -ENOMEM;
#else
errorfc(fc, "domain_id option not supported");
#endif
break; break;
#else
case Opt_fsid:
case Opt_domain_id:
errorfc(fc, "%s option not supported", erofs_fs_parameters[opt].name);
break;
#endif
default: default:
return -ENOPARAM; return -ENOPARAM;
} }