btrfs: relocation: return bool from btrfs_should_ignore_reloc_root
authorDavid Sterba <dsterba@suse.com>
Fri, 22 Sep 2023 11:07:25 +0000 (13:07 +0200)
committerDavid Sterba <dsterba@suse.com>
Thu, 12 Oct 2023 14:44:13 +0000 (16:44 +0200)
btrfs_should_ignore_reloc_root() is a predicate so it should return
bool.

Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Reviewed-by: Qu Wenruo <wqu@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/relocation.c
fs/btrfs/relocation.h

index 75463377f418e072fa575dc46a37e0177b87eacd..d1dcbb15baa7634ca9c2e7b87d99cdf778eb50f4 100644 (file)
@@ -325,31 +325,30 @@ static bool have_reloc_root(struct btrfs_root *root)
        return true;
 }
 
-int btrfs_should_ignore_reloc_root(struct btrfs_root *root)
+bool btrfs_should_ignore_reloc_root(struct btrfs_root *root)
 {
        struct btrfs_root *reloc_root;
 
        if (!test_bit(BTRFS_ROOT_SHAREABLE, &root->state))
-               return 0;
+               return false;
 
        /* This root has been merged with its reloc tree, we can ignore it */
        if (reloc_root_is_dead(root))
-               return 1;
+               return true;
 
        reloc_root = root->reloc_root;
        if (!reloc_root)
-               return 0;
+               return false;
 
        if (btrfs_header_generation(reloc_root->commit_root) ==
            root->fs_info->running_transaction->transid)
-               return 0;
+               return false;
        /*
-        * if there is reloc tree and it was created in previous
-        * transaction backref lookup can find the reloc tree,
-        * so backref node for the fs tree root is useless for
-        * relocation.
+        * If there is reloc tree and it was created in previous transaction
+        * backref lookup can find the reloc tree, so backref node for the fs
+        * tree root is useless for relocation.
         */
-       return 1;
+       return true;
 }
 
 /*
index 77d69f6ae967c285edac5f847eb1e8b3b54b2fe3..af749c780b4e74bf2746765adc405263bb1aa085 100644 (file)
@@ -18,7 +18,7 @@ int btrfs_reloc_post_snapshot(struct btrfs_trans_handle *trans,
                              struct btrfs_pending_snapshot *pending);
 int btrfs_should_cancel_balance(struct btrfs_fs_info *fs_info);
 struct btrfs_root *find_reloc_root(struct btrfs_fs_info *fs_info, u64 bytenr);
-int btrfs_should_ignore_reloc_root(struct btrfs_root *root);
+bool btrfs_should_ignore_reloc_root(struct btrfs_root *root);
 u64 btrfs_get_reloc_bg_bytenr(struct btrfs_fs_info *fs_info);
 
 #endif