btrfs: handle btrfs_update_reloc_root failure in prepare_to_merge
authorJosef Bacik <josef@toxicpanda.com>
Fri, 12 Mar 2021 20:25:19 +0000 (15:25 -0500)
committerDavid Sterba <dsterba@suse.com>
Mon, 19 Apr 2021 15:25:21 +0000 (17:25 +0200)
btrfs_update_reloc_root will will return errors in the future, so handle
an error properly in prepare_to_merge.

Signed-off-by: Josef Bacik <josef@toxicpanda.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/relocation.c

index 69f170b..ca97412 100644 (file)
@@ -1870,10 +1870,21 @@ again:
                 */
                if (!err)
                        btrfs_set_root_refs(&reloc_root->root_item, 1);
-               btrfs_update_reloc_root(trans, root);
+               ret = btrfs_update_reloc_root(trans, root);
 
+               /*
+                * Even if we have an error we need this reloc root back on our
+                * list so we can clean up properly.
+                */
                list_add(&reloc_root->root_list, &reloc_roots);
                btrfs_put_root(root);
+
+               if (ret) {
+                       btrfs_abort_transaction(trans, ret);
+                       if (!err)
+                               err = ret;
+                       break;
+               }
        }
 
        list_splice(&reloc_roots, &rc->reloc_roots);