[PATCH 2/2] bcachefs: Log message when incompat version requested but not enabled

[Kent Overstreet]

Signed-off-by: Kent Overstreet <kent.overstr...@linux.dev>
---
 fs/bcachefs/bcachefs.h |  2 ++
 fs/bcachefs/fs-ioctl.c |  2 +-
 fs/bcachefs/super-io.c | 20 ++++++++++++++++++--
 fs/bcachefs/super.c    |  1 +
 4 files changed, 22 insertions(+), 3 deletions(-)

diff --git a/fs/bcachefs/bcachefs.h b/fs/bcachefs/bcachefs.h
index 5cb0fc384ac0..75f7408da173 100644
--- a/fs/bcachefs/bcachefs.h
+++ b/fs/bcachefs/bcachefs.h
@@ -788,6 +788,8 @@ struct bch_fs {
                unsigned long   errors_silent[BITS_TO_LONGS(BCH_FSCK_ERR_MAX)];
                u64             btrees_lost_data;
        }                       sb;
+       DARRAY(enum bcachefs_metadata_version)
+                               incompat_versions_requested;
 
 #ifdef CONFIG_UNICODE
        struct unicode_map      *cf_encoding;
diff --git a/fs/bcachefs/fs-ioctl.c b/fs/bcachefs/fs-ioctl.c
index c1553e44e049..14886e1d4d6d 100644
--- a/fs/bcachefs/fs-ioctl.c
+++ b/fs/bcachefs/fs-ioctl.c
@@ -69,7 +69,7 @@ static int bch2_inode_flags_set(struct btree_trans *trans,
                if (ret < 0)
                        return ret;
 
-               ret = 
bch2_request_incompat_feature(c,bcachefs_metadata_version_casefolding);
+               ret = bch2_request_incompat_feature(c, 
bcachefs_metadata_version_casefolding);
                if (ret)
                        return ret;
 
diff --git a/fs/bcachefs/super-io.c b/fs/bcachefs/super-io.c
index e27422b6d9c6..25b6bce05c3c 100644
--- a/fs/bcachefs/super-io.c
+++ b/fs/bcachefs/super-io.c
@@ -73,14 +73,30 @@ int bch2_set_version_incompat(struct bch_fs *c, enum 
bcachefs_metadata_version v
                ? 0
                : -BCH_ERR_may_not_use_incompat_feature;
 
+       mutex_lock(&c->sb_lock);
        if (!ret) {
-               mutex_lock(&c->sb_lock);
                SET_BCH_SB_VERSION_INCOMPAT(c->disk_sb.sb,
                        max(BCH_SB_VERSION_INCOMPAT(c->disk_sb.sb), version));
                bch2_write_super(c);
-               mutex_unlock(&c->sb_lock);
+       } else {
+               darray_for_each(c->incompat_versions_requested, i)
+                       if (version == *i)
+                               goto out;
+
+               darray_push(&c->incompat_versions_requested, version);
+               struct printbuf buf = PRINTBUF;
+               prt_str(&buf, "requested incompat feature ");
+               bch2_version_to_text(&buf, version);
+               prt_str(&buf, " currently not enabled");
+               prt_printf(&buf, "\n  set version_upgrade=incompat to enable");
+
+               bch_notice(c, "%s", buf.buf);
+               printbuf_exit(&buf);
        }
 
+out:
+       mutex_unlock(&c->sb_lock);
+
        return ret;
 }
 
diff --git a/fs/bcachefs/super.c b/fs/bcachefs/super.c
index f1eef5e7698b..e8a17ed1615d 100644
--- a/fs/bcachefs/super.c
+++ b/fs/bcachefs/super.c
@@ -589,6 +589,7 @@ static void __bch2_fs_free(struct bch_fs *c)
                free_percpu(c->online_reserved);
        }
 
+       darray_exit(&c->incompat_versions_requested);
        darray_exit(&c->btree_roots_extra);
        free_percpu(c->pcpu);
        free_percpu(c->usage);
-- 
2.49.0