[PATCH 4/8] mtip32xx: use blk_mq_queue_tag_busy_iter

From: Jianchao Wang
Date: Fri Mar 15 2019 - 05:06:57 EST


blk_mq_tagset_busy_iter is not safe that it could get stale request
in tags->rqs[]. Use blk_mq_queue_tag_busy_iter here.

Signed-off-by: Jianchao Wang <jianchao.w.wang@xxxxxxxxxx>
---
drivers/block/mtip32xx/mtip32xx.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/block/mtip32xx/mtip32xx.c b/drivers/block/mtip32xx/mtip32xx.c
index 88e8440..6e356f7 100644
--- a/drivers/block/mtip32xx/mtip32xx.c
+++ b/drivers/block/mtip32xx/mtip32xx.c
@@ -2770,13 +2770,13 @@ static int mtip_service_thread(void *data)

blk_mq_quiesce_queue(dd->queue);

- blk_mq_tagset_busy_iter(&dd->tags, mtip_queue_cmd, dd);
+ blk_mq_queue_tag_busy_iter(dd->queue, mtip_queue_cmd, dd, true);

set_bit(MTIP_PF_ISSUE_CMDS_BIT, &dd->port->flags);

if (mtip_device_reset(dd))
- blk_mq_tagset_busy_iter(&dd->tags,
- mtip_abort_cmd, dd);
+ blk_mq_queue_tag_busy_iter(dd->queue,
+ mtip_abort_cmd, dd, true);

clear_bit(MTIP_PF_TO_ACTIVE_BIT, &dd->port->flags);

@@ -3907,7 +3907,7 @@ static int mtip_block_remove(struct driver_data *dd)

blk_freeze_queue_start(dd->queue);
blk_mq_quiesce_queue(dd->queue);
- blk_mq_tagset_busy_iter(&dd->tags, mtip_no_dev_cleanup, dd);
+ blk_mq_queue_tag_busy_iter(dd->queue, mtip_no_dev_cleanup, dd, true);
blk_mq_unquiesce_queue(dd->queue);

/*
--
2.7.4