diff options
| author | Ming Lei <[email protected]> | 2021-10-21 22:59:18 +0800 |
|---|---|---|
| committer | Jens Axboe <[email protected]> | 2021-11-02 08:40:10 -0600 |
| commit | a1c2f7e7f25c9d35d3bf046f99682c5373b20fa2 (patch) | |
| tree | b342bc219e30aeb909d4ed7a619b10c09cd97e24 | |
| parent | b22809092c70099f4d8c3b6f3d34c5bc89b300ea (diff) | |
dm: don't stop request queue after the dm device is suspended
For fixing queue quiesce race between driver and block layer(elevator
switch, update nr_requests, ...), we need to support concurrent quiesce
and unquiesce, which requires the two call to be balanced.
__bind() is only called from dm_swap_table() in which dm device has been
suspended already, so not necessary to stop queue again. With this way,
request queue quiesce and unquiesce can be balanced.
Reported-by: Yi Zhang <[email protected]>
Fixes: e70feb8b3e68 ("blk-mq: support concurrent queue quiesce/unquiesce")
Signed-off-by: Ming Lei <[email protected]>
Acked-by: Mike Snitzer <[email protected]>
Tested-by: Yi Zhang <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Jens Axboe <[email protected]>
| -rw-r--r-- | drivers/md/dm.c | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/drivers/md/dm.c b/drivers/md/dm.c index 8b91f4f0e053..be0eb2e1dd11 100644 --- a/drivers/md/dm.c +++ b/drivers/md/dm.c @@ -1927,16 +1927,6 @@ static struct dm_table *__bind(struct mapped_device *md, struct dm_table *t, dm_table_event_callback(t, event_callback, md); - /* - * The queue hasn't been stopped yet, if the old table type wasn't - * for request-based during suspension. So stop it to prevent - * I/O mapping before resume. - * This must be done before setting the queue restrictions, - * because request-based dm may be run just after the setting. - */ - if (request_based) - dm_stop_queue(q); - if (request_based) { /* * Leverage the fact that request-based DM targets are |