block: mtip32xx: don't touch q->queue_hw_ctx

Ming Lei · axboe · commit de0328d3a253 · 2022-03-08T17:57:19.000-07:00
q->queue_hw_ctx is really one blk-mq internal structure for retrieving hctx via its index, not supposed to be used by drivers. Meantime drivers can get the tags structure easily from tagset. Reviewed-by: Hannes Reinecke <hare@suse.de> Reviewed-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Ming Lei <ming.lei@redhat.com> Link: https://lore.kernel.org/r/20220308073219.91173-5-ming.lei@redhat.com Signed-off-by: Jens Axboe <axboe@kernel.dk>
diff --git a/drivers/block/mtip32xx/mtip32xx.c b/drivers/block/mtip32xx/mtip32xx.c
@@ -160,9 +160,7 @@ static bool mtip_check_surprise_removal(struct driver_data *dd)
 static struct mtip_cmd *mtip_cmd_from_tag(struct driver_data *dd,
 					  unsigned int tag)
 {
-	struct blk_mq_hw_ctx *hctx = dd->queue->queue_hw_ctx[0];
-
-	return blk_mq_rq_to_pdu(blk_mq_tag_to_rq(hctx->tags, tag));
+	return blk_mq_rq_to_pdu(blk_mq_tag_to_rq(dd->tags.tags[0], tag));
 }
 
 /*

Original file line number	Diff line number	Diff line change
`@@ -160,9 +160,7 @@ static bool mtip_check_surprise_removal(struct driver_data *dd)`
`160`	`160`	`static struct mtip_cmd mtip_cmd_from_tag(struct driver_data dd,`
`161`	`161`	`unsigned int tag)`
`162`	`162`	`{`
`163`		`- struct blk_mq_hw_ctx *hctx = dd->queue->queue_hw_ctx[0];`
`164`		`-`
`165`		`- return blk_mq_rq_to_pdu(blk_mq_tag_to_rq(hctx->tags, tag));`
	`163`	`+ return blk_mq_rq_to_pdu(blk_mq_tag_to_rq(dd->tags.tags[0], tag));`
`166`	`164`	`}`
`167`	`165`
`168`	`166`	`/*`