Repair mode will commit transaction which will make us
fail to load log tree anymore.

Give a warning to common users, if they really want to
coninue, we will clear out log tree.

Signed-off-by: Wang Shilong <wangsl.f...@cn.fujitsu.com>
---
 cmds-check.c | 33 +++++++++++++++++++++++++++++++++
 1 file changed, 33 insertions(+)

diff --git a/cmds-check.c b/cmds-check.c
index 93f9ae6..15806a4 100644
--- a/cmds-check.c
+++ b/cmds-check.c
@@ -6615,6 +6615,22 @@ out:
        return ret;
 }
 
+static int zero_log_tree(struct btrfs_root *root)
+{
+       struct btrfs_trans_handle *trans;
+       int ret;
+
+       trans = btrfs_start_transaction(root, 1);
+       if (IS_ERR(trans)) {
+               ret = PTR_ERR(trans);
+               return ret;
+       }
+       btrfs_set_super_log_root(root->fs_info->super_copy, 0);
+       btrfs_set_super_log_root_level(root->fs_info->super_copy, 0);
+       ret = btrfs_commit_transaction(trans, root);
+       return ret;
+}
+
 static struct option long_options[] = {
        { "super", 1, NULL, 's' },
        { "repair", 0, NULL, 0 },
@@ -6719,6 +6735,23 @@ int cmd_check(int argc, char **argv)
        }
 
        root = info->fs_root;
+       /*
+        * repair mode will force us to commit transaction which
+        * will make us fail to load log tree when mounting.
+        */
+       if (repair && btrfs_super_log_root(info->super_copy)) {
+               ret = ask_user("repair mode will force to clear out log tree, 
Are you sure?");
+               if (!ret) {
+                       ret = 1;
+                       goto close_out;
+               }
+               ret = zero_log_tree(root);
+               if (ret) {
+                       fprintf(stderr, "fail to zero log tree\n");
+                       goto close_out;
+               }
+       }
+
        uuid_unparse(info->super_copy->fsid, uuidbuf);
        printf("Checking filesystem on %s\nUUID: %s\n", argv[optind], uuidbuf);
 
-- 
1.9.0

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to