]> git.sesse.net Git - bcachefs-tools-debian/blobdiff - libbcachefs/opts.c
Update bcachefs sources to a8115093df bcachefs: Fix divide by zero in rebalance_work()
[bcachefs-tools-debian] / libbcachefs / opts.c
index 77fbb7d2194e956738e91385066f99b7019fee24..960bb247f3a0ab8b84c64e8f439738ffdf76b3b6 100644 (file)
@@ -5,19 +5,25 @@
 #include "bcachefs.h"
 #include "compress.h"
 #include "disk_groups.h"
+#include "error.h"
 #include "opts.h"
 #include "super-io.h"
 #include "util.h"
 
-#define x(t, n) [n] = #t,
+#define x(t, n, ...) [n] = #t,
 
-const char * const bch2_metadata_versions[] = {
-       BCH_METADATA_VERSIONS()
+const char * const bch2_error_actions[] = {
+       BCH_ERROR_ACTIONS()
        NULL
 };
 
-const char * const bch2_error_actions[] = {
-       BCH_ERROR_ACTIONS()
+const char * const bch2_fsck_fix_opts[] = {
+       BCH_FIX_ERRORS_OPTS()
+       NULL
+};
+
+const char * const bch2_version_upgrade_opts[] = {
+       BCH_VERSION_UPGRADE_OPTS()
        NULL
 };
 
@@ -33,6 +39,7 @@ const char * const bch2_sb_compat[] = {
 
 const char * const bch2_btree_ids[] = {
        BCH_BTREE_IDS()
+       "interior btree node",
        NULL
 };
 
@@ -88,6 +95,37 @@ const char * const bch2_fs_usage_types[] = {
 
 #undef x
 
+static int bch2_opt_fix_errors_parse(struct bch_fs *c, const char *val, u64 *res,
+                                    struct printbuf *err)
+{
+       if (!val) {
+               *res = FSCK_FIX_yes;
+       } else {
+               int ret = match_string(bch2_fsck_fix_opts, -1, val);
+
+               if (ret < 0 && err)
+                       prt_str(err, "fix_errors: invalid selection");
+               if (ret < 0)
+                       return ret;
+               *res = ret;
+       }
+
+       return 0;
+}
+
+static void bch2_opt_fix_errors_to_text(struct printbuf *out,
+                                       struct bch_fs *c,
+                                       struct bch_sb *sb,
+                                       u64 v)
+{
+       prt_str(out, bch2_fsck_fix_opts[v]);
+}
+
+#define bch2_opt_fix_errors (struct bch_opt_fn) {      \
+       .parse = bch2_opt_fix_errors_parse,             \
+       .to_text = bch2_opt_fix_errors_to_text,         \
+}
+
 const char * const bch2_d_types[BCH_DT_MAX] = {
        [DT_UNKNOWN]    = "unknown",
        [DT_FIFO]       = "fifo",
@@ -166,11 +204,9 @@ const struct bch_option bch2_opt_table[] = {
 #define OPT_UINT(_min, _max)   .type = BCH_OPT_UINT,                   \
                                .min = _min, .max = _max
 #define OPT_STR(_choices)      .type = BCH_OPT_STR,                    \
-                               .min = 0, .max = ARRAY_SIZE(_choices),\
+                               .min = 0, .max = ARRAY_SIZE(_choices),  \
                                .choices = _choices
-#define OPT_FN(_fn)            .type = BCH_OPT_FN,                     \
-                               .parse = _fn##_parse,                   \
-                               .to_text = _fn##_to_text
+#define OPT_FN(_fn)            .type = BCH_OPT_FN, .fn = _fn
 
 #define x(_name, _bits, _flags, _type, _sb_opt, _default, _hint, _help)        \
        [Opt_##_name] = {                                               \
@@ -228,28 +264,28 @@ int bch2_opt_validate(const struct bch_option *opt, u64 v, struct printbuf *err)
 {
        if (v < opt->min) {
                if (err)
-                       pr_buf(err, "%s: too small (min %llu)",
+                       prt_printf(err, "%s: too small (min %llu)",
                               opt->attr.name, opt->min);
                return -ERANGE;
        }
 
        if (opt->max && v >= opt->max) {
                if (err)
-                       pr_buf(err, "%s: too big (max %llu)",
+                       prt_printf(err, "%s: too big (max %llu)",
                               opt->attr.name, opt->max);
                return -ERANGE;
        }
 
        if ((opt->flags & OPT_SB_FIELD_SECTORS) && (v & 511)) {
                if (err)
-                       pr_buf(err, "%s: not a multiple of 512",
+                       prt_printf(err, "%s: not a multiple of 512",
                               opt->attr.name);
                return -EINVAL;
        }
 
        if ((opt->flags & OPT_MUST_BE_POW_2) && !is_power_of_2(v)) {
                if (err)
-                       pr_buf(err, "%s: must be a power of two",
+                       prt_printf(err, "%s: must be a power of two",
                               opt->attr.name);
                return -EINVAL;
        }
@@ -266,31 +302,61 @@ int bch2_opt_parse(struct bch_fs *c,
 
        switch (opt->type) {
        case BCH_OPT_BOOL:
-               ret = kstrtou64(val, 10, res);
-               if (ret < 0)
+               if (val) {
+                       ret = kstrtou64(val, 10, res);
+               } else {
+                       ret = 0;
+                       *res = 1;
+               }
+
+               if (ret < 0 || (*res != 0 && *res != 1)) {
+                       if (err)
+                               prt_printf(err, "%s: must be bool", opt->attr.name);
                        return ret;
+               }
                break;
        case BCH_OPT_UINT:
+               if (!val) {
+                       prt_printf(err, "%s: required value",
+                                  opt->attr.name);
+                       return -EINVAL;
+               }
+
                ret = opt->flags & OPT_HUMAN_READABLE
                        ? bch2_strtou64_h(val, res)
                        : kstrtou64(val, 10, res);
-               if (ret < 0)
+               if (ret < 0) {
+                       if (err)
+                               prt_printf(err, "%s: must be a number",
+                                          opt->attr.name);
                        return ret;
+               }
                break;
        case BCH_OPT_STR:
+               if (!val) {
+                       prt_printf(err, "%s: required value",
+                                  opt->attr.name);
+                       return -EINVAL;
+               }
+
                ret = match_string(opt->choices, -1, val);
-               if (ret < 0)
+               if (ret < 0) {
+                       if (err)
+                               prt_printf(err, "%s: invalid selection",
+                                          opt->attr.name);
                        return ret;
+               }
 
                *res = ret;
                break;
        case BCH_OPT_FN:
-               if (!c)
-                       return 0;
-
-               ret = opt->parse(c, val, res);
-               if (ret < 0)
+               ret = opt->fn.parse(c, val, res, err);
+               if (ret < 0) {
+                       if (err)
+                               prt_printf(err, "%s: parse error",
+                                          opt->attr.name);
                        return ret;
+               }
        }
 
        return bch2_opt_validate(opt, *res, err);
@@ -303,31 +369,31 @@ void bch2_opt_to_text(struct printbuf *out,
 {
        if (flags & OPT_SHOW_MOUNT_STYLE) {
                if (opt->type == BCH_OPT_BOOL) {
-                       pr_buf(out, "%s%s",
+                       prt_printf(out, "%s%s",
                               v ? "" : "no",
                               opt->attr.name);
                        return;
                }
 
-               pr_buf(out, "%s=", opt->attr.name);
+               prt_printf(out, "%s=", opt->attr.name);
        }
 
        switch (opt->type) {
        case BCH_OPT_BOOL:
        case BCH_OPT_UINT:
                if (opt->flags & OPT_HUMAN_READABLE)
-                       bch2_hprint(out, v);
+                       prt_human_readable_u64(out, v);
                else
-                       pr_buf(out, "%lli", v);
+                       prt_printf(out, "%lli", v);
                break;
        case BCH_OPT_STR:
                if (flags & OPT_SHOW_FULL_LIST)
-                       bch2_string_opt_to_text(out, opt->choices, v);
+                       prt_string_option(out, opt->choices, v);
                else
-                       pr_buf(out, opt->choices[v]);
+                       prt_str(out, opt->choices[v]);
                break;
        case BCH_OPT_FN:
-               opt->to_text(out, c, sb, v);
+               opt->fn.to_text(out, c, sb, v);
                break;
        default:
                BUG();
@@ -379,6 +445,13 @@ int bch2_parse_mount_opts(struct bch_fs *c, struct bch_opts *opts,
        if (!options)
                return 0;
 
+       /*
+        * sys_fsconfig() is now occasionally providing us with option lists
+        * starting with a comma - weird.
+        */
+       if (*options == ',')
+               options++;
+
        copied_opts = kstrdup(options, GFP_KERNEL);
        if (!copied_opts)
                return -1;
@@ -388,31 +461,19 @@ int bch2_parse_mount_opts(struct bch_fs *c, struct bch_opts *opts,
                name    = strsep(&opt, "=");
                val     = opt;
 
-               if (val) {
-                       id = bch2_mount_opt_lookup(name);
-                       if (id < 0)
-                               goto bad_opt;
-
-                       ret = bch2_opt_parse(c, &bch2_opt_table[id], val, &v, &err);
-                       if (ret < 0)
-                               goto bad_val;
-               } else {
-                       id = bch2_mount_opt_lookup(name);
-                       v = 1;
+               id = bch2_mount_opt_lookup(name);
 
-                       if (id < 0 &&
-                           !strncmp("no", name, 2)) {
-                               id = bch2_mount_opt_lookup(name + 2);
-                               v = 0;
-                       }
-
-                       if (id < 0)
-                               goto bad_opt;
-
-                       if (bch2_opt_table[id].type != BCH_OPT_BOOL)
-                               goto no_val;
+               /* Check for the form "noopt", negation of a boolean opt: */
+               if (id < 0 &&
+                   !val &&
+                   !strncmp("no", name, 2)) {
+                       id = bch2_mount_opt_lookup(name + 2);
+                       val = "0";
                }
 
+               if (id < 0)
+                       goto bad_opt;
+
                if (!(bch2_opt_table[id].flags & OPT_MOUNT))
                        goto bad_opt;
 
@@ -425,6 +486,10 @@ int bch2_parse_mount_opts(struct bch_fs *c, struct bch_opts *opts,
                    !IS_ENABLED(CONFIG_BCACHEFS_QUOTA))
                        goto bad_opt;
 
+               ret = bch2_opt_parse(c, &bch2_opt_table[id], val, &v, &err);
+               if (ret < 0)
+                       goto bad_val;
+
                bch2_opt_set_by_id(opts, id, v);
        }
 
@@ -439,10 +504,6 @@ bad_val:
        pr_err("Invalid mount option %s", err.buf);
        ret = -1;
        goto out;
-no_val:
-       pr_err("Mount option %s requires a value", name);
-       ret = -1;
-       goto out;
 out:
        kfree(copied_opts_start);
        printbuf_exit(&err);
@@ -514,33 +575,11 @@ void bch2_opt_set_sb(struct bch_fs *c, const struct bch_option *opt, u64 v)
 
 struct bch_io_opts bch2_opts_to_inode_opts(struct bch_opts src)
 {
-       struct bch_io_opts ret = { 0 };
-#define x(_name, _bits)                                        \
-       if (opt_defined(src, _name))                                    \
-               opt_set(ret, _name, src._name);
-       BCH_INODE_OPTS()
-#undef x
-       return ret;
-}
-
-struct bch_opts bch2_inode_opts_to_opts(struct bch_io_opts src)
-{
-       struct bch_opts ret = { 0 };
-#define x(_name, _bits)                                        \
-       if (opt_defined(src, _name))                                    \
-               opt_set(ret, _name, src._name);
-       BCH_INODE_OPTS()
-#undef x
-       return ret;
-}
-
-void bch2_io_opts_apply(struct bch_io_opts *dst, struct bch_io_opts src)
-{
-#define x(_name, _bits)                                        \
-       if (opt_defined(src, _name))                                    \
-               opt_set(*dst, _name, src._name);
+       return (struct bch_io_opts) {
+#define x(_name, _bits)        ._name = src._name,
        BCH_INODE_OPTS()
 #undef x
+       };
 }
 
 bool bch2_opt_is_inode_opt(enum bch_opt_id id)