@@ -3995,7 +3995,7 @@ static int mtip_block_remove(struct driver_data *dd)
dd->disk->disk_name);
blk_freeze_queue_start(dd->queue);
- blk_mq_stop_hw_queues(dd->queue);
+ blk_mq_quiesce_queue(dd->queue);
blk_mq_tagset_busy_iter(&dd->tags, mtip_no_dev_cleanup, dd);
We still need to unquiesce queue for avoiding hanging blk_freeze_queue()
in blk_cleanup_queue() since any new request queued during quiescing
can't be dispatched to driver/device.
Yes, already added it in v2.
There are other blk_mq_tagset_busy_iter() in mtip_service_thread(),
which looks need quiesce too.
Wasn't sure about those two... I agree it looks like quiescing is
needed, will add in v2.
This case is even worse, because both
mtip_abort_cmd() and mtip_queue_cmd() do not check if the req is
started.
Its easy enough to add.