block: factor out common part in blkdev_fallocate()

Only the flags passed to blkdev_issue_zeroout() differ among the two
zeroing branches in blkdev_fallocate(). Therefore, do cleanup by
factoring them out.

Signed-off-by: Zhang Yi <yi.zhang@huawei.com>
Link: https://lore.kernel.org/20250619111806.3546162-8-yi.zhang@huaweicloud.com
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: "Martin K. Petersen" <martin.petersen@oracle.com>
Signed-off-by: Christian Brauner <brauner@kernel.org>

diff --git a/block/fops.c b/block/fops.c
index 1309861d4c2c4b88be375136eb83e34299ef7616..e1c921549d28979453880652dd38106a9a767998 100644 (file)
--- a/block/fops.c
+++ b/block/fops.c
@@ -850,6 +850,7 @@ static long blkdev_fallocate(struct file *file, int mode, loff_t start,
        struct block_device *bdev = I_BDEV(inode);
        loff_t end = start + len - 1;
        loff_t isize;
+       unsigned int flags;
        int error;
 
        /* Fail if we don't recognize the flags. */
@@ -877,34 +878,29 @@ static long blkdev_fallocate(struct file *file, int mode, loff_t start,
        inode_lock(inode);
        filemap_invalidate_lock(inode->i_mapping);
 
-       /*
-        * Invalidate the page cache, including dirty pages, for valid
-        * de-allocate mode calls to fallocate().
-        */
        switch (mode) {
        case FALLOC_FL_ZERO_RANGE:
        case FALLOC_FL_ZERO_RANGE | FALLOC_FL_KEEP_SIZE:
-               error = truncate_bdev_range(bdev, file_to_blk_mode(file), start, end);
-               if (error)
-                       goto fail;
-
-               error = blkdev_issue_zeroout(bdev, start >> SECTOR_SHIFT,
-                                            len >> SECTOR_SHIFT, GFP_KERNEL,
-                                            BLKDEV_ZERO_NOUNMAP);
+               flags = BLKDEV_ZERO_NOUNMAP;
                break;
        case FALLOC_FL_PUNCH_HOLE | FALLOC_FL_KEEP_SIZE:
-               error = truncate_bdev_range(bdev, file_to_blk_mode(file), start, end);
-               if (error)
-                       goto fail;
-
-               error = blkdev_issue_zeroout(bdev, start >> SECTOR_SHIFT,
-                                            len >> SECTOR_SHIFT, GFP_KERNEL,
-                                            BLKDEV_ZERO_NOFALLBACK);
+               flags = BLKDEV_ZERO_NOFALLBACK;
                break;
        default:
                error = -EOPNOTSUPP;
+               goto fail;
        }
 
+       /*
+        * Invalidate the page cache, including dirty pages, for valid
+        * de-allocate mode calls to fallocate().
+        */
+       error = truncate_bdev_range(bdev, file_to_blk_mode(file), start, end);
+       if (error)
+               goto fail;
+
+       error = blkdev_issue_zeroout(bdev, start >> SECTOR_SHIFT,
+                                    len >> SECTOR_SHIFT, GFP_KERNEL, flags);
  fail:
        filemap_invalidate_unlock(inode->i_mapping);
        inode_unlock(inode);