virtio_blk: Don't bother validating blocksize

The block queue limits validation does this for us now.

Reviewed-by: Christoph Hellwig <hch@lst.de>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: John Garry <john.g.garry@oracle.com>
Reviewed-by: Damien Le Moal <dlemoal@kernel.org>
Link: https://lore.kernel.org/r/20240708091651.177447-5-john.g.garry@oracle.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>

diff --git a/drivers/block/virtio_blk.c b/drivers/block/virtio_blk.c
index f11b0c3b26256daa03841b8c9c49320d4e884857..e3147a611151034dd343ccf550574eea9ade8953 100644 (file)
--- a/drivers/block/virtio_blk.c
+++ b/drivers/block/virtio_blk.c
@@ -1289,18 +1289,9 @@ static int virtblk_read_limits(struct virtio_blk *vblk,
        lim->max_segment_size = max_size;
 
        /* Host can optionally specify the block size of the device */
-       err = virtio_cread_feature(vdev, VIRTIO_BLK_F_BLK_SIZE,
+       virtio_cread_feature(vdev, VIRTIO_BLK_F_BLK_SIZE,
                                   struct virtio_blk_config, blk_size,
                                   &lim->logical_block_size);
-       if (!err) {
-               err = blk_validate_block_size(lim->logical_block_size);
-               if (err) {
-                       dev_err(&vdev->dev,
-                               "virtio_blk: invalid block size: 0x%x\n",
-                               lim->logical_block_size);
-                       return err;
-               }
-       }
 
        /* Use topology information if available */
        err = virtio_cread_feature(vdev, VIRTIO_BLK_F_TOPOLOGY,