diff options
author | zhengbin <zhengbin13@huawei.com> | 2019-07-23 22:10:42 +0800 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-10-05 13:13:38 +0200 |
commit | 5e1feedf563e3d9dd8b8de38a924ef4dfc542efc (patch) | |
tree | da3f3dcc65589f7b7f9d011725c8514b7092f693 /block | |
parent | 503ac6efb245b10162c19da5f25d8b3e52c67ffd (diff) | |
download | linux-stable-5e1feedf563e3d9dd8b8de38a924ef4dfc542efc.tar.gz linux-stable-5e1feedf563e3d9dd8b8de38a924ef4dfc542efc.tar.bz2 linux-stable-5e1feedf563e3d9dd8b8de38a924ef4dfc542efc.zip |
blk-mq: Fix memory leak in blk_mq_init_allocated_queue error handling
[ Upstream commit 73d9c8d4c0017e21e1ff519474ceb1450484dc9a ]
If blk_mq_init_allocated_queue->elevator_init_mq fails, need to release
the previously requested resources.
Fixes: d34849913819 ("blk-mq-sched: allow setting of default IO scheduler")
Signed-off-by: zhengbin <zhengbin13@huawei.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'block')
-rw-r--r-- | block/blk-mq.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/block/blk-mq.c b/block/blk-mq.c index 68106a41f90d..f934e8afe5b4 100644 --- a/block/blk-mq.c +++ b/block/blk-mq.c @@ -2853,6 +2853,8 @@ static unsigned int nr_hw_queues(struct blk_mq_tag_set *set) struct request_queue *blk_mq_init_allocated_queue(struct blk_mq_tag_set *set, struct request_queue *q) { + int ret = -ENOMEM; + /* mark the queue as mq asap */ q->mq_ops = set->ops; @@ -2914,17 +2916,18 @@ struct request_queue *blk_mq_init_allocated_queue(struct blk_mq_tag_set *set, blk_mq_map_swqueue(q); if (!(set->flags & BLK_MQ_F_NO_SCHED)) { - int ret; - ret = elevator_init_mq(q); if (ret) - return ERR_PTR(ret); + goto err_tag_set; } return q; +err_tag_set: + blk_mq_del_queue_tag_set(q); err_hctxs: kfree(q->queue_hw_ctx); + q->nr_hw_queues = 0; err_sys_init: blk_mq_sysfs_deinit(q); err_poll: |