ext4: remove the 'group' parameter of ext4_trim_extent
authorWang Jianchao <wangjianchao@kuaishou.com>
Sat, 24 Jul 2021 07:41:20 +0000 (15:41 +0800)
committerTheodore Ts'o <tytso@mit.edu>
Thu, 12 Aug 2021 17:11:25 +0000 (13:11 -0400)
Get rid of the 'group' parameter of ext4_trim_extent as we can get
it from the 'e4b'.

Reviewed-by: Andreas Dilger <adilger@dilger.ca>
Signed-off-by: Wang Jianchao <wangjianchao@kuaishou.com>
Reviewed-by: Jan Kara <jack@suse.cz>
Link: https://lore.kernel.org/r/20210724074124.25731-2-jianchao.wan9@gmail.com
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
fs/ext4/mballoc.c

index 089c958..018d5d3 100644 (file)
@@ -6183,19 +6183,19 @@ error_return:
  * @sb:                super block for the file system
  * @start:     starting block of the free extent in the alloc. group
  * @count:     number of blocks to TRIM
- * @group:     alloc. group we are working with
  * @e4b:       ext4 buddy for the group
  *
  * Trim "count" blocks starting at "start" in the "group". To assure that no
  * one will allocate those blocks, mark it as used in buddy bitmap. This must
  * be called with under the group lock.
  */
-static int ext4_trim_extent(struct super_block *sb, int start, int count,
-                            ext4_group_t group, struct ext4_buddy *e4b)
+static int ext4_trim_extent(struct super_block *sb,
+               int start, int count, struct ext4_buddy *e4b)
 __releases(bitlock)
 __acquires(bitlock)
 {
        struct ext4_free_extent ex;
+       ext4_group_t group = e4b->bd_group;
        int ret = 0;
 
        trace_ext4_trim_extent(sb, group, start, count);
@@ -6271,8 +6271,7 @@ ext4_trim_all_free(struct super_block *sb, ext4_group_t group,
                next = mb_find_next_bit(bitmap, max + 1, start);
 
                if ((next - start) >= minblocks) {
-                       ret = ext4_trim_extent(sb, start,
-                                              next - start, group, &e4b);
+                       ret = ext4_trim_extent(sb, start, next - start, &e4b);
                        if (ret && ret != -EOPNOTSUPP)
                                break;
                        ret = 0;