md/raid*: Fix the set_queue_limits implementations
authorBart Van Assche <bvanassche@acm.org>
Wed, 12 Feb 2025 17:11:07 +0000 (09:11 -0800)
committerYu Kuai <yukuai@kernel.org>
Thu, 13 Feb 2025 17:29:33 +0000 (01:29 +0800)
queue_limits_cancel_update() must only be called if
queue_limits_start_update() is called first. Remove the
queue_limits_cancel_update() calls from the raid*_set_limits() functions
because there is no corresponding queue_limits_start_update() call.

Cc: Christoph Hellwig <hch@lst.de>
Fixes: c6e56cf6b2e7 ("block: move integrity information into queue_limits")
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Link: https://lore.kernel.org/linux-raid/20250212171108.3483150-1-bvanassche@acm.org/
Signed-off-by: Yu Kuai <yukuai@kernel.org>
drivers/md/raid0.c
drivers/md/raid1.c
drivers/md/raid10.c

index 8fc9339..70bcc3c 100644 (file)
@@ -386,10 +386,8 @@ static int raid0_set_limits(struct mddev *mddev)
        lim.io_opt = lim.io_min * mddev->raid_disks;
        lim.features |= BLK_FEAT_ATOMIC_WRITES;
        err = mddev_stack_rdev_limits(mddev, &lim, MDDEV_STACK_INTEGRITY);
-       if (err) {
-               queue_limits_cancel_update(mddev->gendisk->queue);
+       if (err)
                return err;
-       }
        return queue_limits_set(mddev->gendisk->queue, &lim);
 }
 
index 9d57a88..10ea3af 100644 (file)
@@ -3219,10 +3219,8 @@ static int raid1_set_limits(struct mddev *mddev)
        lim.max_write_zeroes_sectors = 0;
        lim.features |= BLK_FEAT_ATOMIC_WRITES;
        err = mddev_stack_rdev_limits(mddev, &lim, MDDEV_STACK_INTEGRITY);
-       if (err) {
-               queue_limits_cancel_update(mddev->gendisk->queue);
+       if (err)
                return err;
-       }
        return queue_limits_set(mddev->gendisk->queue, &lim);
 }
 
index efe93b9..15b9ae5 100644 (file)
@@ -4020,10 +4020,8 @@ static int raid10_set_queue_limits(struct mddev *mddev)
        lim.io_opt = lim.io_min * raid10_nr_stripes(conf);
        lim.features |= BLK_FEAT_ATOMIC_WRITES;
        err = mddev_stack_rdev_limits(mddev, &lim, MDDEV_STACK_INTEGRITY);
-       if (err) {
-               queue_limits_cancel_update(mddev->gendisk->queue);
+       if (err)
                return err;
-       }
        return queue_limits_set(mddev->gendisk->queue, &lim);
 }