diff options
| author | Paolo Valente <paolo.valente@linaro.org> | 2020-02-03 11:40:57 +0100 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2020-02-03 06:58:15 -0700 |
| commit | 33a16a9804688b2f4c4281ec31bc393ef2645ae4 (patch) | |
| tree | d0582697320c2a862c429649d31dcffaa0346b66 /block/bfq-cgroup.c | |
| parent | ecedd3d7e19911ab8fe42f17b77c0a30fe7f4db3 (diff) | |
| download | lwn-33a16a9804688b2f4c4281ec31bc393ef2645ae4.tar.gz lwn-33a16a9804688b2f4c4281ec31bc393ef2645ae4.zip | |
block, bfq: extend incomplete name of field on_st
The flag on_st in the bfq_entity data structure is true if the entity
is on a service tree or is in service. Yet the name of the field,
confusingly, does not mention the second, very important case. Extend
the name to mention the second case too.
Tested-by: Oleksandr Natalenko <oleksandr@natalenko.name>
Signed-off-by: Paolo Valente <paolo.valente@linaro.org>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'block/bfq-cgroup.c')
| -rw-r--r-- | block/bfq-cgroup.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/block/bfq-cgroup.c b/block/bfq-cgroup.c index 8ab7f18ff8cb..c818c64766e5 100644 --- a/block/bfq-cgroup.c +++ b/block/bfq-cgroup.c @@ -659,7 +659,7 @@ void bfq_bfqq_move(struct bfq_data *bfqd, struct bfq_queue *bfqq, if (bfq_bfqq_busy(bfqq)) bfq_deactivate_bfqq(bfqd, bfqq, false, false); - else if (entity->on_st) + else if (entity->on_st_or_in_serv) bfq_put_idle_entity(bfq_entity_service_tree(entity), entity); bfqg_and_blkg_put(bfqq_group(bfqq)); |