| From 8fb186cacbb4f5ae75270a021429cbb6a97c2b1b Mon Sep 17 00:00:00 2001 |
| From: Sasha Levin <sashal@kernel.org> |
| Date: Mon, 3 Feb 2020 11:40:56 +0100 |
| Subject: block, bfq: get extra ref to prevent a queue from being freed during |
| a group move |
| |
| From: Paolo Valente <paolo.valente@linaro.org> |
| |
| [ Upstream commit ecedd3d7e19911ab8fe42f17b77c0a30fe7f4db3 ] |
| |
| In bfq_bfqq_move(), the bfq_queue, say Q, to be moved to a new group |
| may happen to be deactivated in the scheduling data structures of the |
| source group (and then activated in the destination group). If Q is |
| referred only by the data structures in the source group when the |
| deactivation happens, then Q is freed upon the deactivation. |
| |
| This commit addresses this issue by getting an extra reference before |
| the possible deactivation, and releasing this extra reference after Q |
| has been moved. |
| |
| Tested-by: Chris Evich <cevich@redhat.com> |
| Tested-by: Oleksandr Natalenko <oleksandr@natalenko.name> |
| Signed-off-by: Paolo Valente <paolo.valente@linaro.org> |
| Signed-off-by: Jens Axboe <axboe@kernel.dk> |
| Signed-off-by: Sasha Levin <sashal@kernel.org> |
| --- |
| block/bfq-cgroup.c | 8 ++++++++ |
| 1 file changed, 8 insertions(+) |
| |
| diff --git a/block/bfq-cgroup.c b/block/bfq-cgroup.c |
| index e7919e76a27c2..db2a14215aeea 100644 |
| --- a/block/bfq-cgroup.c |
| +++ b/block/bfq-cgroup.c |
| @@ -651,6 +651,12 @@ void bfq_bfqq_move(struct bfq_data *bfqd, struct bfq_queue *bfqq, |
| bfq_bfqq_expire(bfqd, bfqd->in_service_queue, |
| false, BFQQE_PREEMPTED); |
| |
| + /* |
| + * get extra reference to prevent bfqq from being freed in |
| + * next possible deactivate |
| + */ |
| + bfqq->ref++; |
| + |
| if (bfq_bfqq_busy(bfqq)) |
| bfq_deactivate_bfqq(bfqd, bfqq, false, false); |
| else if (entity->on_st) |
| @@ -670,6 +676,8 @@ void bfq_bfqq_move(struct bfq_data *bfqd, struct bfq_queue *bfqq, |
| |
| if (!bfqd->in_service_queue && !bfqd->rq_in_driver) |
| bfq_schedule_dispatch(bfqd); |
| + /* release extra ref taken above */ |
| + bfq_put_queue(bfqq); |
| } |
| |
| /** |
| -- |
| 2.20.1 |
| |