loop: fix zero sized loop for block special file

By default, /dev/sda is block special file from devtmpfs, getattr will
return file size as zero, causing loop failed for raw block device.

We can add bdev_statx() to return device size, however this may
introduce changes that are not acknowledged by user. Fix this problem by
reverting changes for block special file, file mapping host is set to
bdev inode while opening, and use i_size_read() directly to get device
size.

Fixes: 47b71abd5846 ("loop: use vfs_getattr_nosec for accurate file size")
Reported-by: kernel test robot <oliver.sang@intel.com>
Closes: https://lore.kernel.org/oe-lkp/202508200409.b2459c02-lkp@intel.com
Signed-off-by: Yu Kuai <yukuai3@huawei.com>
Reviewed-by: Ming Lei <ming.lei@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Link: https://lore.kernel.org/r/20250825093205.3684121-1-yukuai1@huaweicloud.com
[axboe: fix spelling error]
Signed-off-by: Jens Axboe <axboe@kernel.dk>

diff --git a/drivers/block/loop.c b/drivers/block/loop.c
index 57263c273f0fb4bf2fb1b6612618f290c4dea581..053a086d547ec6d61b9b3422dfba44ca6bad8733 100644 (file)
--- a/drivers/block/loop.c
+++ b/drivers/block/loop.c
@@ -139,20 +139,26 @@ static int part_shift;
 
 static loff_t lo_calculate_size(struct loop_device *lo, struct file *file)
 {
-       struct kstat stat;
        loff_t loopsize;
        int ret;
 
-       /*
-        * Get the accurate file size. This provides better results than
-        * cached inode data, particularly for network filesystems where
-        * metadata may be stale.
-        */
-       ret = vfs_getattr_nosec(&file->f_path, &stat, STATX_SIZE, 0);
-       if (ret)
-               return 0;
+       if (S_ISBLK(file_inode(file)->i_mode)) {
+               loopsize = i_size_read(file->f_mapping->host);
+       } else {
+               struct kstat stat;
+
+               /*
+                * Get the accurate file size. This provides better results than
+                * cached inode data, particularly for network filesystems where
+                * metadata may be stale.
+                */
+               ret = vfs_getattr_nosec(&file->f_path, &stat, STATX_SIZE, 0);
+               if (ret)
+                       return 0;
+
+               loopsize = stat.size;
+       }
 
-       loopsize = stat.size;
        if (lo->lo_offset > 0)
                loopsize -= lo->lo_offset;
        /* offset is beyond i_size, weird but possible */