Commit f32e4eaf authored by John Garry's avatar John Garry Committed by Jens Axboe

blk-mq: Pass driver tags to blk_mq_clear_rq_mapping()

Function blk_mq_clear_rq_mapping() will be used for shared sbitmap tags
in future, so pass a driver tags pointer instead of the tagset container
and HW queue index.
Signed-off-by: default avatarJohn Garry <john.garry@huawei.com>
Reviewed-by: default avatarHannes Reinecke <hare@suse.de>
Reviewed-by: default avatarMing Lei <ming.lei@redhat.com>
Link: https://lore.kernel.org/r/1633429419-228500-8-git-send-email-john.garry@huawei.comSigned-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent 1820f4f0
...@@ -2311,10 +2311,9 @@ static size_t order_to_size(unsigned int order) ...@@ -2311,10 +2311,9 @@ static size_t order_to_size(unsigned int order)
} }
/* called before freeing request pool in @tags */ /* called before freeing request pool in @tags */
static void blk_mq_clear_rq_mapping(struct blk_mq_tag_set *set, static void blk_mq_clear_rq_mapping(struct blk_mq_tags *drv_tags,
struct blk_mq_tags *tags, unsigned int hctx_idx) struct blk_mq_tags *tags)
{ {
struct blk_mq_tags *drv_tags = set->tags[hctx_idx];
struct page *page; struct page *page;
unsigned long flags; unsigned long flags;
...@@ -2323,7 +2322,7 @@ static void blk_mq_clear_rq_mapping(struct blk_mq_tag_set *set, ...@@ -2323,7 +2322,7 @@ static void blk_mq_clear_rq_mapping(struct blk_mq_tag_set *set,
unsigned long end = start + order_to_size(page->private); unsigned long end = start + order_to_size(page->private);
int i; int i;
for (i = 0; i < set->queue_depth; i++) { for (i = 0; i < drv_tags->nr_tags; i++) {
struct request *rq = drv_tags->rqs[i]; struct request *rq = drv_tags->rqs[i];
unsigned long rq_addr = (unsigned long)rq; unsigned long rq_addr = (unsigned long)rq;
...@@ -2347,8 +2346,11 @@ static void blk_mq_clear_rq_mapping(struct blk_mq_tag_set *set, ...@@ -2347,8 +2346,11 @@ static void blk_mq_clear_rq_mapping(struct blk_mq_tag_set *set,
void blk_mq_free_rqs(struct blk_mq_tag_set *set, struct blk_mq_tags *tags, void blk_mq_free_rqs(struct blk_mq_tag_set *set, struct blk_mq_tags *tags,
unsigned int hctx_idx) unsigned int hctx_idx)
{ {
struct blk_mq_tags *drv_tags;
struct page *page; struct page *page;
drv_tags = set->tags[hctx_idx];
if (tags->static_rqs && set->ops->exit_request) { if (tags->static_rqs && set->ops->exit_request) {
int i; int i;
...@@ -2362,7 +2364,7 @@ void blk_mq_free_rqs(struct blk_mq_tag_set *set, struct blk_mq_tags *tags, ...@@ -2362,7 +2364,7 @@ void blk_mq_free_rqs(struct blk_mq_tag_set *set, struct blk_mq_tags *tags,
} }
} }
blk_mq_clear_rq_mapping(set, tags, hctx_idx); blk_mq_clear_rq_mapping(drv_tags, tags);
while (!list_empty(&tags->page_list)) { while (!list_empty(&tags->page_list)) {
page = list_first_entry(&tags->page_list, struct page, lru); page = list_first_entry(&tags->page_list, struct page, lru);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment