btrfs: make btrfs_can_overcommit() return bool instead of int

It's a boolean function, so switch its return type to bool.

Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

diff --git a/fs/btrfs/space-info.c b/fs/btrfs/space-info.c
index c0bad6914bb7b1c1a1a4e88da430c7170bf9c35c..0fdf60f05228b6fb1c37a8579dda4cd483149a77 100644 (file)
--- a/fs/btrfs/space-info.c
+++ b/fs/btrfs/space-info.c
@@ -490,22 +490,20 @@ static u64 calc_available_free_space(const struct btrfs_space_info *space_info,
        return avail;
 }
 
-int btrfs_can_overcommit(const struct btrfs_space_info *space_info, u64 bytes,
-                        enum btrfs_reserve_flush_enum flush)
+bool btrfs_can_overcommit(const struct btrfs_space_info *space_info, u64 bytes,
+                         enum btrfs_reserve_flush_enum flush)
 {
        u64 avail;
        u64 used;
 
        /* Don't overcommit when in mixed mode */
        if (space_info->flags & BTRFS_BLOCK_GROUP_DATA)
-               return 0;
+               return false;
 
        used = btrfs_space_info_used(space_info, true);
        avail = calc_available_free_space(space_info, flush);
 
-       if (used + bytes < space_info->total_bytes + avail)
-               return 1;
-       return 0;
+       return (used + bytes < space_info->total_bytes + avail);
 }
 
 static void remove_ticket(struct btrfs_space_info *space_info,

diff --git a/fs/btrfs/space-info.h b/fs/btrfs/space-info.h
index 2fad2e4c22524247a5d5a16bad320e2adbcc8db0..d97b0799649f3819d16c69991291c1aed32ae302 100644 (file)
--- a/fs/btrfs/space-info.h
+++ b/fs/btrfs/space-info.h
@@ -282,8 +282,8 @@ int btrfs_reserve_metadata_bytes(struct btrfs_space_info *space_info,
                                 u64 orig_bytes,
                                 enum btrfs_reserve_flush_enum flush);
 void btrfs_try_granting_tickets(struct btrfs_space_info *space_info);
-int btrfs_can_overcommit(const struct btrfs_space_info *space_info, u64 bytes,
-                        enum btrfs_reserve_flush_enum flush);
+bool btrfs_can_overcommit(const struct btrfs_space_info *space_info, u64 bytes,
+                         enum btrfs_reserve_flush_enum flush);
 
 static inline void btrfs_space_info_free_bytes_may_use(
                                struct btrfs_space_info *space_info,