summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorAl Viro <viro@zeniv.linux.org.uk>2019-12-21 05:52:55 +0100
committerAl Viro <viro@zeniv.linux.org.uk>2020-02-07 20:48:34 +0100
commit7f5d38141e309bb4ba995d9726928af85a299c50 (patch)
tree06400fc56ab3282c8a137b8655d1dba7eb5925d5 /fs
parentswitch rbd and libceph to p_log-based primitives (diff)
downloadlinux-7f5d38141e309bb4ba995d9726928af85a299c50.tar.xz
linux-7f5d38141e309bb4ba995d9726928af85a299c50.zip
new primitive: __fs_parse()
fs_parse() analogue taking p_log instead of fs_context. fs_parse() turned into a wrapper, callers in ceph_common and rbd switched to __fs_parse(). As the result, fs_parse() never gets NULL fs_context and neither do fs_context-based logging primitives Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs')
-rw-r--r--fs/fs_parser.c21
1 files changed, 15 insertions, 6 deletions
diff --git a/fs/fs_parser.c b/fs/fs_parser.c
index dc5c6737c280..dadb6582874d 100644
--- a/fs/fs_parser.c
+++ b/fs/fs_parser.c
@@ -80,7 +80,7 @@ static const struct fs_parameter_spec *fs_lookup_key(
* unknown parameters are okay and -EINVAL if there was a conversion issue or
* the parameter wasn't recognised and unknowns aren't okay.
*/
-int fs_parse(struct fs_context *fc,
+int __fs_parse(struct p_log *log,
const struct fs_parameter_description *desc,
struct fs_parameter *param,
struct fs_parse_result *result)
@@ -113,8 +113,7 @@ int fs_parse(struct fs_context *fc,
}
if (p->flags & fs_param_deprecated)
- warnf(fc, "%s: Deprecated parameter '%s'",
- desc->name, param->key);
+ warn_plog(log, "Deprecated parameter '%s'", param->key);
if (result->negated)
goto okay;
@@ -152,8 +151,8 @@ int fs_parse(struct fs_context *fc,
switch (p->type) {
case fs_param_is_flag:
if (param->type != fs_value_is_flag)
- return invalf(fc, "%s: Unexpected value for '%s'",
- desc->name, param->key);
+ return inval_plog(log, "Unexpected value for '%s'",
+ param->key);
result->boolean = true;
goto okay;
@@ -238,10 +237,20 @@ okay:
return p->opt;
bad_value:
- return invalf(fc, "%s: Bad value for '%s'", desc->name, param->key);
+ return inval_plog(log, "Bad value for '%s'", param->key);
unknown_parameter:
return -ENOPARAM;
}
+EXPORT_SYMBOL(__fs_parse);
+
+int fs_parse(struct fs_context *fc,
+ const struct fs_parameter_description *desc,
+ struct fs_parameter *param,
+ struct fs_parse_result *result)
+{
+ struct p_log log = {.prefix = desc->name, .log = fc->log};
+ return __fs_parse(&log, desc, param, result);
+}
EXPORT_SYMBOL(fs_parse);
/**