diff mbox series

[v2,08/18] btrfs: add fs_parameter definitions

Message ID 24771d34a49eb428d6415197f390bf41dcef495c.1699470345.git.josef@toxicpanda.com (mailing list archive)
State New, archived
Headers show
Series btrfs: convert to the new mount API | expand

Commit Message

Josef Bacik Nov. 8, 2023, 7:08 p.m. UTC
In order to convert to the new mount api we have to change how we do the
mount option parsing.  For now we're going to duplicate these helpers to
make it easier to follow, and then remove the old code once everything
is in place.  This patch contains the re-definiton of all of our mount
options into the new fs_parameter_spec format.

Signed-off-by: Josef Bacik <josef@toxicpanda.com>
---
 fs/btrfs/super.c | 128 ++++++++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 127 insertions(+), 1 deletion(-)

Comments

David Sterba Nov. 14, 2023, 5:49 p.m. UTC | #1
On Wed, Nov 08, 2023 at 02:08:43PM -0500, Josef Bacik wrote:
> In order to convert to the new mount api we have to change how we do the
> mount option parsing.  For now we're going to duplicate these helpers to
> make it easier to follow, and then remove the old code once everything
> is in place.  This patch contains the re-definiton of all of our mount
> options into the new fs_parameter_spec format.
> 
> Signed-off-by: Josef Bacik <josef@toxicpanda.com>
> ---
>  fs/btrfs/super.c | 128 ++++++++++++++++++++++++++++++++++++++++++++++-
>  1 file changed, 127 insertions(+), 1 deletion(-)
> 
> diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c
> index d7070269e3ea..0e9cb9ed6508 100644
> --- a/fs/btrfs/super.c
> +++ b/fs/btrfs/super.c
> @@ -27,6 +27,7 @@
>  #include <linux/crc32c.h>
>  #include <linux/btrfs.h>
>  #include <linux/security.h>
> +#include <linux/fs_parser.h>
>  #include "messages.h"
>  #include "delayed-inode.h"
>  #include "ctree.h"
> @@ -132,7 +133,7 @@ enum {
>  	/* Debugging options */
>  	Opt_enospc_debug, Opt_noenospc_debug,
>  #ifdef CONFIG_BTRFS_DEBUG
> -	Opt_fragment_data, Opt_fragment_metadata, Opt_fragment_all,
> +	Opt_fragment, Opt_fragment_data, Opt_fragment_metadata, Opt_fragment_all,
>  #endif
>  #ifdef CONFIG_BTRFS_FS_REF_VERIFY
>  	Opt_ref_verify,
> @@ -222,6 +223,131 @@ static const match_table_t rescue_tokens = {
>  	{Opt_err, NULL},
>  };
>  
> +enum {
> +	Opt_fatal_errors_panic,
> +	Opt_fatal_errors_bug,
> +};
> +
> +static const struct constant_table btrfs_parameter_fatal_errors[] = {
> +	{ "panic", Opt_fatal_errors_panic },
> +	{ "bug", Opt_fatal_errors_bug },
> +	{}
> +};
> +
> +enum {
> +	Opt_discard_sync,
> +	Opt_discard_async,
> +};
> +
> +static const struct constant_table btrfs_parameter_discard[] = {
> +	{ "sync", Opt_discard_sync },
> +	{ "async", Opt_discard_async },
> +	{}
> +};
> +
> +enum {
> +	Opt_space_cache_v1,
> +	Opt_space_cache_v2,
> +};
> +
> +static const struct constant_table btrfs_parameter_space_cache[] = {
> +	{ "v1", Opt_space_cache_v1 },
> +	{ "v2", Opt_space_cache_v2 },
> +	{}
> +};
> +
> +enum {
> +	Opt_rescue_usebackuproot,
> +	Opt_rescue_nologreplay,
> +	Opt_rescue_ignorebadroots,
> +	Opt_rescue_ignoredatacsums,
> +	Opt_rescue_parameter_all,
> +};
> +
> +static const struct constant_table btrfs_parameter_rescue[] = {
> +	{ "usebackuproot", Opt_rescue_usebackuproot },
> +	{ "nologreplay", Opt_rescue_nologreplay },
> +	{ "ignorebadroots", Opt_rescue_ignorebadroots },
> +	{ "ibadroots", Opt_rescue_ignorebadroots },
> +	{ "ignoredatacsums", Opt_rescue_ignoredatacsums },
> +	{ "idatacsums", Opt_rescue_ignoredatacsums },
> +	{ "all", Opt_rescue_parameter_all },
> +	{}
> +};
> +
> +#ifdef CONFIG_BTRFS_DEBUG
> +enum {
> +	Opt_fragment_parameter_data,
> +	Opt_fragment_parameter_metadata,
> +	Opt_fragment_parameter_all,
> +};
> +
> +static const struct constant_table btrfs_parameter_fragment[] = {
> +	{ "data", Opt_fragment_parameter_data },
> +	{ "metadata", Opt_fragment_parameter_metadata },
> +	{ "all", Opt_fragment_parameter_all },
> +	{}
> +};
> +#endif
> +
> +static const struct fs_parameter_spec btrfs_fs_parameters[] __maybe_unused = {
> +	fsparam_flag_no("acl", Opt_acl),
> +	fsparam_flag("clear_cache", Opt_clear_cache),
> +	fsparam_u32("commit", Opt_commit_interval),
> +	fsparam_flag("compress", Opt_compress),
> +	fsparam_string("compress", Opt_compress_type),
> +	fsparam_flag("compress-force", Opt_compress_force),
> +	fsparam_string("compress-force", Opt_compress_force_type),
> +	fsparam_flag("degraded", Opt_degraded),
> +	fsparam_string("device", Opt_device),
> +	fsparam_enum("fatal_errors", Opt_fatal_errors, btrfs_parameter_fatal_errors),
> +	fsparam_flag_no("flushoncommit", Opt_flushoncommit),
> +	fsparam_flag_no("inode_cache", Opt_inode_cache),

I think it's a good opportunity to remove this option completely, its
functionality is gone since 5.11, that's eough time. I'm fine with
removing it both before or after the API switch.

> +	fsparam_string("max_inline", Opt_max_inline),
> +	fsparam_flag_no("barrier", Opt_barrier),
> +	fsparam_flag_no("datacow", Opt_datacow),
> +	fsparam_flag_no("datasum", Opt_datasum),
> +	fsparam_flag_no("autodefrag", Opt_defrag),
> +	fsparam_flag_no("discard", Opt_discard),
> +	fsparam_enum("discard", Opt_discard_mode, btrfs_parameter_discard),
> +	fsparam_u32("metadata_ratio", Opt_ratio),
> +	fsparam_flag("rescan_uuid_tree", Opt_rescan_uuid_tree),
> +	fsparam_flag("skip_balance", Opt_skip_balance),
> +	fsparam_flag_no("space_cache", Opt_space_cache),
> +	fsparam_enum("space_cache", Opt_space_cache_version, btrfs_parameter_space_cache),
> +	fsparam_flag_no("ssd", Opt_ssd),
> +	fsparam_flag_no("ssd_spread", Opt_ssd_spread),
> +	fsparam_string("subvol", Opt_subvol),
> +	fsparam_flag("subvol=", Opt_subvol_empty),
> +	fsparam_u64("subvolid", Opt_subvolid),
> +	fsparam_u32("thread_pool", Opt_thread_pool),
> +	fsparam_flag_no("treelog", Opt_treelog),
> +	fsparam_flag("user_subvol_rm_allowed", Opt_user_subvol_rm_allowed),
> +
> +	/* Rescue options */
> +	fsparam_enum("rescue", Opt_rescue, btrfs_parameter_rescue),
> +	/* Deprecated, with alias rescue=nologreplay */
> +	__fsparam(NULL, "nologreplay", Opt_nologreplay, fs_param_deprecated,
> +		  NULL),
> +	/* Deprecated, with alias rescue=usebackuproot */
> +	__fsparam(NULL, "usebackuproot", Opt_usebackuproot, fs_param_deprecated,
> +		  NULL),
> +
> +	/* Deprecated options */
> +	__fsparam(NULL, "recovery", Opt_recovery,
> +		  fs_param_neg_with_no|fs_param_deprecated, NULL),

Same here, 'recovery' obsoleted by the rescue options in 4.5.

> +
> +	/* Debugging options */
> +	fsparam_flag_no("enospc_debug", Opt_enospc_debug),
> +#ifdef CONFIG_BTRFS_DEBUG
> +	fsparam_enum("fragment", Opt_fragment, btrfs_parameter_fragment),
> +#endif
> +#ifdef CONFIG_BTRFS_FS_REF_VERIFY
> +	fsparam_flag("ref_verify", Opt_ref_verify),
> +#endif
> +	{}
> +};
> +
>  static bool check_ro_option(struct btrfs_fs_info *fs_info, unsigned long opt,
>  			    const char *opt_name)
>  {
> -- 
> 2.41.0
>
diff mbox series

Patch

diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c
index d7070269e3ea..0e9cb9ed6508 100644
--- a/fs/btrfs/super.c
+++ b/fs/btrfs/super.c
@@ -27,6 +27,7 @@ 
 #include <linux/crc32c.h>
 #include <linux/btrfs.h>
 #include <linux/security.h>
+#include <linux/fs_parser.h>
 #include "messages.h"
 #include "delayed-inode.h"
 #include "ctree.h"
@@ -132,7 +133,7 @@  enum {
 	/* Debugging options */
 	Opt_enospc_debug, Opt_noenospc_debug,
 #ifdef CONFIG_BTRFS_DEBUG
-	Opt_fragment_data, Opt_fragment_metadata, Opt_fragment_all,
+	Opt_fragment, Opt_fragment_data, Opt_fragment_metadata, Opt_fragment_all,
 #endif
 #ifdef CONFIG_BTRFS_FS_REF_VERIFY
 	Opt_ref_verify,
@@ -222,6 +223,131 @@  static const match_table_t rescue_tokens = {
 	{Opt_err, NULL},
 };
 
+enum {
+	Opt_fatal_errors_panic,
+	Opt_fatal_errors_bug,
+};
+
+static const struct constant_table btrfs_parameter_fatal_errors[] = {
+	{ "panic", Opt_fatal_errors_panic },
+	{ "bug", Opt_fatal_errors_bug },
+	{}
+};
+
+enum {
+	Opt_discard_sync,
+	Opt_discard_async,
+};
+
+static const struct constant_table btrfs_parameter_discard[] = {
+	{ "sync", Opt_discard_sync },
+	{ "async", Opt_discard_async },
+	{}
+};
+
+enum {
+	Opt_space_cache_v1,
+	Opt_space_cache_v2,
+};
+
+static const struct constant_table btrfs_parameter_space_cache[] = {
+	{ "v1", Opt_space_cache_v1 },
+	{ "v2", Opt_space_cache_v2 },
+	{}
+};
+
+enum {
+	Opt_rescue_usebackuproot,
+	Opt_rescue_nologreplay,
+	Opt_rescue_ignorebadroots,
+	Opt_rescue_ignoredatacsums,
+	Opt_rescue_parameter_all,
+};
+
+static const struct constant_table btrfs_parameter_rescue[] = {
+	{ "usebackuproot", Opt_rescue_usebackuproot },
+	{ "nologreplay", Opt_rescue_nologreplay },
+	{ "ignorebadroots", Opt_rescue_ignorebadroots },
+	{ "ibadroots", Opt_rescue_ignorebadroots },
+	{ "ignoredatacsums", Opt_rescue_ignoredatacsums },
+	{ "idatacsums", Opt_rescue_ignoredatacsums },
+	{ "all", Opt_rescue_parameter_all },
+	{}
+};
+
+#ifdef CONFIG_BTRFS_DEBUG
+enum {
+	Opt_fragment_parameter_data,
+	Opt_fragment_parameter_metadata,
+	Opt_fragment_parameter_all,
+};
+
+static const struct constant_table btrfs_parameter_fragment[] = {
+	{ "data", Opt_fragment_parameter_data },
+	{ "metadata", Opt_fragment_parameter_metadata },
+	{ "all", Opt_fragment_parameter_all },
+	{}
+};
+#endif
+
+static const struct fs_parameter_spec btrfs_fs_parameters[] __maybe_unused = {
+	fsparam_flag_no("acl", Opt_acl),
+	fsparam_flag("clear_cache", Opt_clear_cache),
+	fsparam_u32("commit", Opt_commit_interval),
+	fsparam_flag("compress", Opt_compress),
+	fsparam_string("compress", Opt_compress_type),
+	fsparam_flag("compress-force", Opt_compress_force),
+	fsparam_string("compress-force", Opt_compress_force_type),
+	fsparam_flag("degraded", Opt_degraded),
+	fsparam_string("device", Opt_device),
+	fsparam_enum("fatal_errors", Opt_fatal_errors, btrfs_parameter_fatal_errors),
+	fsparam_flag_no("flushoncommit", Opt_flushoncommit),
+	fsparam_flag_no("inode_cache", Opt_inode_cache),
+	fsparam_string("max_inline", Opt_max_inline),
+	fsparam_flag_no("barrier", Opt_barrier),
+	fsparam_flag_no("datacow", Opt_datacow),
+	fsparam_flag_no("datasum", Opt_datasum),
+	fsparam_flag_no("autodefrag", Opt_defrag),
+	fsparam_flag_no("discard", Opt_discard),
+	fsparam_enum("discard", Opt_discard_mode, btrfs_parameter_discard),
+	fsparam_u32("metadata_ratio", Opt_ratio),
+	fsparam_flag("rescan_uuid_tree", Opt_rescan_uuid_tree),
+	fsparam_flag("skip_balance", Opt_skip_balance),
+	fsparam_flag_no("space_cache", Opt_space_cache),
+	fsparam_enum("space_cache", Opt_space_cache_version, btrfs_parameter_space_cache),
+	fsparam_flag_no("ssd", Opt_ssd),
+	fsparam_flag_no("ssd_spread", Opt_ssd_spread),
+	fsparam_string("subvol", Opt_subvol),
+	fsparam_flag("subvol=", Opt_subvol_empty),
+	fsparam_u64("subvolid", Opt_subvolid),
+	fsparam_u32("thread_pool", Opt_thread_pool),
+	fsparam_flag_no("treelog", Opt_treelog),
+	fsparam_flag("user_subvol_rm_allowed", Opt_user_subvol_rm_allowed),
+
+	/* Rescue options */
+	fsparam_enum("rescue", Opt_rescue, btrfs_parameter_rescue),
+	/* Deprecated, with alias rescue=nologreplay */
+	__fsparam(NULL, "nologreplay", Opt_nologreplay, fs_param_deprecated,
+		  NULL),
+	/* Deprecated, with alias rescue=usebackuproot */
+	__fsparam(NULL, "usebackuproot", Opt_usebackuproot, fs_param_deprecated,
+		  NULL),
+
+	/* Deprecated options */
+	__fsparam(NULL, "recovery", Opt_recovery,
+		  fs_param_neg_with_no|fs_param_deprecated, NULL),
+
+	/* Debugging options */
+	fsparam_flag_no("enospc_debug", Opt_enospc_debug),
+#ifdef CONFIG_BTRFS_DEBUG
+	fsparam_enum("fragment", Opt_fragment, btrfs_parameter_fragment),
+#endif
+#ifdef CONFIG_BTRFS_FS_REF_VERIFY
+	fsparam_flag("ref_verify", Opt_ref_verify),
+#endif
+	{}
+};
+
 static bool check_ro_option(struct btrfs_fs_info *fs_info, unsigned long opt,
 			    const char *opt_name)
 {