md/md-linear: cleanup linear_add()
authorYu Kuai <yukuai3@huawei.com>
Tue, 10 Oct 2023 15:19:54 +0000 (23:19 +0800)
committerSong Liu <song@kernel.org>
Wed, 11 Oct 2023 01:49:51 +0000 (18:49 -0700)
Now that caller already suspend the array, there is no need to suspend
array in liner_add().

Note that mddev_suspend/resume() is not used anymore.

Signed-off-by: Yu Kuai <yukuai3@huawei.com>
Signed-off-by: Song Liu <song@kernel.org>
Link: https://lore.kernel.org/r/20231010151958.145896-16-yukuai1@huaweicloud.com
drivers/md/md-linear.c

index ae2826e9645bde10b104d3e8fb2a2d61b1e63beb..8eca7693b793fe94cb3f1e630d1e498894576ef5 100644 (file)
@@ -183,7 +183,6 @@ static int linear_add(struct mddev *mddev, struct md_rdev *rdev)
         * in linear_congested(), therefore kfree_rcu() is used to free
         * oldconf until no one uses it anymore.
         */
-       mddev_suspend(mddev);
        oldconf = rcu_dereference_protected(mddev->private,
                        lockdep_is_held(&mddev->reconfig_mutex));
        mddev->raid_disks++;
@@ -192,7 +191,6 @@ static int linear_add(struct mddev *mddev, struct md_rdev *rdev)
        rcu_assign_pointer(mddev->private, newconf);
        md_set_array_sectors(mddev, linear_size(mddev, 0, 0));
        set_capacity_and_notify(mddev->gendisk, mddev->array_sectors);
-       mddev_resume(mddev);
        kfree_rcu(oldconf, rcu);
        return 0;
 }