md: factor out a helper to sync mddev
There are no functional changes, prepare to sync mddev in array_state_store(). Signed-off-by: Li Nan <linan122@huawei.com> Signed-off-by: Song Liu <song@kernel.org> Link: https://lore.kernel.org/r/20240226031444.3606764-7-linan666@huaweicloud.com
This commit is contained in:
+21
-11
@@ -529,6 +529,24 @@ void mddev_resume(struct mddev *mddev)
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(mddev_resume);
|
||||
|
||||
/* sync bdev before setting device to readonly or stopping raid*/
|
||||
static int mddev_set_closing_and_sync_blockdev(struct mddev *mddev, int opener_num)
|
||||
{
|
||||
mutex_lock(&mddev->open_mutex);
|
||||
if (mddev->pers && atomic_read(&mddev->openers) > opener_num) {
|
||||
mutex_unlock(&mddev->open_mutex);
|
||||
return -EBUSY;
|
||||
}
|
||||
if (test_and_set_bit(MD_CLOSING, &mddev->flags)) {
|
||||
mutex_unlock(&mddev->open_mutex);
|
||||
return -EBUSY;
|
||||
}
|
||||
mutex_unlock(&mddev->open_mutex);
|
||||
|
||||
sync_blockdev(mddev->gendisk->part0);
|
||||
return 0;
|
||||
}
|
||||
|
||||
/*
|
||||
* Generic flush handling for md
|
||||
*/
|
||||
@@ -7658,17 +7676,9 @@ static int md_ioctl(struct block_device *bdev, blk_mode_t mode,
|
||||
/* Need to flush page cache, and ensure no-one else opens
|
||||
* and writes
|
||||
*/
|
||||
mutex_lock(&mddev->open_mutex);
|
||||
if (mddev->pers && atomic_read(&mddev->openers) > 1) {
|
||||
mutex_unlock(&mddev->open_mutex);
|
||||
return -EBUSY;
|
||||
}
|
||||
if (test_and_set_bit(MD_CLOSING, &mddev->flags)) {
|
||||
mutex_unlock(&mddev->open_mutex);
|
||||
return -EBUSY;
|
||||
}
|
||||
mutex_unlock(&mddev->open_mutex);
|
||||
sync_blockdev(bdev);
|
||||
err = mddev_set_closing_and_sync_blockdev(mddev, 1);
|
||||
if (err)
|
||||
return err;
|
||||
}
|
||||
|
||||
if (!md_is_rdwr(mddev))
|
||||
|
||||
Reference in New Issue
Block a user