btrfs: refactor block group replication factor calculation to a helper
authorDavid Sterba <dsterba@suse.com>
Fri, 13 Jul 2018 18:46:30 +0000 (20:46 +0200)
committerDavid Sterba <dsterba@suse.com>
Mon, 6 Aug 2018 11:12:53 +0000 (13:12 +0200)
There are many places that open code the duplicity factor of the block
group profiles, create a common helper. This can be easily extended for
more copies.

Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/extent-tree.c
fs/btrfs/super.c
fs/btrfs/volumes.c
fs/btrfs/volumes.h

index 4f19bdca3214dea23e4a6cbfa7139c918c23d68d..fd109bfd528da869e6fa0c899df128addf16ba1f 100644 (file)
@@ -3931,11 +3931,7 @@ static void update_space_info(struct btrfs_fs_info *info, u64 flags,
        struct btrfs_space_info *found;
        int factor;
 
-       if (flags & (BTRFS_BLOCK_GROUP_DUP | BTRFS_BLOCK_GROUP_RAID1 |
-                    BTRFS_BLOCK_GROUP_RAID10))
-               factor = 2;
-       else
-               factor = 1;
+       factor = btrfs_bg_type_to_factor(flags);
 
        found = __find_space_info(info, flags);
        ASSERT(found);
@@ -4576,6 +4572,7 @@ static int can_overcommit(struct btrfs_fs_info *fs_info,
        u64 space_size;
        u64 avail;
        u64 used;
+       int factor;
 
        /* Don't overcommit when in mixed mode. */
        if (space_info->flags & BTRFS_BLOCK_GROUP_DATA)
@@ -4610,10 +4607,8 @@ static int can_overcommit(struct btrfs_fs_info *fs_info,
         * doesn't include the parity drive, so we don't have to
         * change the math
         */
-       if (profile & (BTRFS_BLOCK_GROUP_DUP |
-                      BTRFS_BLOCK_GROUP_RAID1 |
-                      BTRFS_BLOCK_GROUP_RAID10))
-               avail >>= 1;
+       factor = btrfs_bg_type_to_factor(profile);
+       avail = div_u64(avail, factor);
 
        /*
         * If we aren't flushing all things, let us overcommit up to
@@ -6094,12 +6089,8 @@ static int update_block_group(struct btrfs_trans_handle *trans,
                cache = btrfs_lookup_block_group(info, bytenr);
                if (!cache)
                        return -ENOENT;
-               if (cache->flags & (BTRFS_BLOCK_GROUP_DUP |
-                                   BTRFS_BLOCK_GROUP_RAID1 |
-                                   BTRFS_BLOCK_GROUP_RAID10))
-                       factor = 2;
-               else
-                       factor = 1;
+               factor = btrfs_bg_type_to_factor(cache->flags);
+
                /*
                 * If this block group has free space cache written out, we
                 * need to make sure to load it if we are removing space.  This
@@ -9359,13 +9350,7 @@ u64 btrfs_account_ro_block_groups_free_space(struct btrfs_space_info *sinfo)
                        continue;
                }
 
-               if (block_group->flags & (BTRFS_BLOCK_GROUP_RAID1 |
-                                         BTRFS_BLOCK_GROUP_RAID10 |
-                                         BTRFS_BLOCK_GROUP_DUP))
-                       factor = 2;
-               else
-                       factor = 1;
-
+               factor = btrfs_bg_type_to_factor(block_group->flags);
                free_bytes += (block_group->key.offset -
                               btrfs_block_group_used(&block_group->item)) *
                               factor;
@@ -10175,12 +10160,7 @@ int btrfs_remove_block_group(struct btrfs_trans_handle *trans,
 
        memcpy(&key, &block_group->key, sizeof(key));
        index = btrfs_bg_flags_to_raid_index(block_group->flags);
-       if (block_group->flags & (BTRFS_BLOCK_GROUP_DUP |
-                                 BTRFS_BLOCK_GROUP_RAID1 |
-                                 BTRFS_BLOCK_GROUP_RAID10))
-               factor = 2;
-       else
-               factor = 1;
+       factor = btrfs_bg_type_to_factor(block_group->flags);
 
        /* make sure this block group isn't part of an allocation cluster */
        cluster = &fs_info->data_alloc_cluster;
index d7a54c648c5fa88c11aea606d84a06350483efa5..efe8b03ce380ee43e8045e836e349b097f2aed83 100644 (file)
@@ -2107,14 +2107,9 @@ static int btrfs_statfs(struct dentry *dentry, struct kstatfs *buf)
                                btrfs_account_ro_block_groups_free_space(found);
 
                        for (i = 0; i < BTRFS_NR_RAID_TYPES; i++) {
-                               if (!list_empty(&found->block_groups[i])) {
-                                       switch (i) {
-                                       case BTRFS_RAID_DUP:
-                                       case BTRFS_RAID_RAID1:
-                                       case BTRFS_RAID_RAID10:
-                                               factor = 2;
-                                       }
-                               }
+                               if (!list_empty(&found->block_groups[i]))
+                                       factor = btrfs_bg_type_to_factor(
+                                               btrfs_raid_array[i].bg_flag);
                        }
                }
 
index 0b9023f52b2af5b03269b9afe98e948a94a13715..eca67529e53670a9958996e5602c88d1dce2c9ca 100644 (file)
@@ -7292,3 +7292,14 @@ void btrfs_reset_fs_info_ptr(struct btrfs_fs_info *fs_info)
                fs_devices = fs_devices->seed;
        }
 }
+
+/*
+ * Multiplicity factor for simple profiles: DUP, RAID1-like and RAID10.
+ */
+int btrfs_bg_type_to_factor(u64 flags)
+{
+       if (flags & (BTRFS_BLOCK_GROUP_DUP | BTRFS_BLOCK_GROUP_RAID1 |
+                    BTRFS_BLOCK_GROUP_RAID10))
+               return 2;
+       return 1;
+}
index 06d8bb7dd55763016cf52500028ca56e59b9e4f3..6943aab9bdd705edad35145559d2817950620bcf 100644 (file)
@@ -559,4 +559,6 @@ void btrfs_reset_fs_info_ptr(struct btrfs_fs_info *fs_info);
 bool btrfs_check_rw_degradable(struct btrfs_fs_info *fs_info,
                                        struct btrfs_device *failing_dev);
 
+int btrfs_bg_type_to_factor(u64 flags);
+
 #endif