Btrfs-progs: fsck: reset balance after reiniting extent root
authorWang Shilong <wangsl.fnst@cn.fujitsu.com>
Thu, 6 Mar 2014 05:53:57 +0000 (13:53 +0800)
committerChris Mason <clm@fb.com>
Fri, 21 Mar 2014 13:23:35 +0000 (06:23 -0700)
reset balance need cow block which will insert extent item into
extent tree. If we do this before reinitting extent root, we may
encounter EEIXST.

Signed-off-by: Wang Shilong <wangsl.fnst@cn.fujitsu.com>
Signed-off-by: David Sterba <dsterba@suse.cz>
Signed-off-by: Chris Mason <clm@fb.com>
cmds-check.c

index 3cf59b6..8a3f2cd 100644 (file)
@@ -6253,12 +6253,6 @@ static int reinit_extent_tree(struct btrfs_trans_handle *trans,
                return ret;
        }
 
-       ret = reset_balance(trans, fs_info);
-       if (ret) {
-               fprintf(stderr, "error reseting the pending balance\n");
-               return ret;
-       }
-
        /* Ok we can allocate now, reinit the extent root */
        ret = btrfs_fsck_reinit_root(trans, fs_info->extent_root, 0);
        if (ret) {
@@ -6293,7 +6287,11 @@ static int reinit_extent_tree(struct btrfs_trans_handle *trans,
                btrfs_extent_post_op(trans, fs_info->extent_root);
        }
 
-       return 0;
+       ret = reset_balance(trans, fs_info);
+       if (ret)
+               fprintf(stderr, "error reseting the pending balance\n");
+
+       return ret;
 }
 
 static int recow_extent_buffer(struct btrfs_root *root, struct extent_buffer *eb)