ext4: remove unused parameter biop in ext4_issue_discard()
authorWenchao Hao <haowenchao2@huawei.com>
Mon, 26 Feb 2024 08:17:31 +0000 (16:17 +0800)
committerTheodore Ts'o <tytso@mit.edu>
Thu, 7 Mar 2024 18:32:54 +0000 (13:32 -0500)
all caller of ext4_issue_discard() would set biop to NULL since
'commit 55cdd0af2bc5 ("ext4: get discard out of jbd2 commit kthread
contex")', it's unnecessary to keep this parameter any more.

Signed-off-by: Wenchao Hao <haowenchao2@huawei.com>
Reviewed-by: Ritesh Harjani (IBM) <ritesh.list@gmail.com>
Reviewed-by: Zhang Yi <yi.zhang@huawei.com>
Link: https://lore.kernel.org/r/20240226081731.3224470-1-haowenchao2@huawei.com
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
fs/ext4/mballoc.c

index 85a91a6..12b3f19 100644 (file)
@@ -3832,8 +3832,7 @@ void ext4_mb_release(struct super_block *sb)
 }
 
 static inline int ext4_issue_discard(struct super_block *sb,
-               ext4_group_t block_group, ext4_grpblk_t cluster, int count,
-               struct bio **biop)
+               ext4_group_t block_group, ext4_grpblk_t cluster, int count)
 {
        ext4_fsblk_t discard_block;
 
@@ -3842,13 +3841,8 @@ static inline int ext4_issue_discard(struct super_block *sb,
        count = EXT4_C2B(EXT4_SB(sb), count);
        trace_ext4_discard_blocks(sb,
                        (unsigned long long) discard_block, count);
-       if (biop) {
-               return __blkdev_issue_discard(sb->s_bdev,
-                       (sector_t)discard_block << (sb->s_blocksize_bits - 9),
-                       (sector_t)count << (sb->s_blocksize_bits - 9),
-                       GFP_NOFS, biop);
-       } else
-               return sb_issue_discard(sb, discard_block, count, GFP_NOFS, 0);
+
+       return sb_issue_discard(sb, discard_block, count, GFP_NOFS, 0);
 }
 
 static void ext4_free_data_in_buddy(struct super_block *sb,
@@ -6495,7 +6489,7 @@ do_more:
        } else {
                if (test_opt(sb, DISCARD)) {
                        err = ext4_issue_discard(sb, block_group, bit,
-                                                count_clusters, NULL);
+                                                count_clusters);
                        /*
                         * Ignore EOPNOTSUPP error. This is consistent with
                         * what happens when using journal.
@@ -6752,7 +6746,7 @@ __acquires(bitlock)
         */
        mb_mark_used(e4b, &ex);
        ext4_unlock_group(sb, group);
-       ret = ext4_issue_discard(sb, group, start, count, NULL);
+       ret = ext4_issue_discard(sb, group, start, count);
        ext4_lock_group(sb, group);
        mb_free_blocks(NULL, e4b, start, ex.fe_len);
        return ret;