Commit 33a16a98 authored by Paolo Valente's avatar Paolo Valente Committed by Jens Axboe

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: default avatarOleksandr Natalenko <oleksandr@natalenko.name>
Signed-off-by: default avatarPaolo Valente <paolo.valente@linaro.org>
Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent ecedd3d7
...@@ -659,7 +659,7 @@ void bfq_bfqq_move(struct bfq_data *bfqd, struct bfq_queue *bfqq, ...@@ -659,7 +659,7 @@ void bfq_bfqq_move(struct bfq_data *bfqd, struct bfq_queue *bfqq,
if (bfq_bfqq_busy(bfqq)) if (bfq_bfqq_busy(bfqq))
bfq_deactivate_bfqq(bfqd, bfqq, false, false); 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); bfq_put_idle_entity(bfq_entity_service_tree(entity), entity);
bfqg_and_blkg_put(bfqq_group(bfqq)); bfqg_and_blkg_put(bfqq_group(bfqq));
......
...@@ -1059,7 +1059,7 @@ bfq_bfqq_resume_state(struct bfq_queue *bfqq, struct bfq_data *bfqd, ...@@ -1059,7 +1059,7 @@ bfq_bfqq_resume_state(struct bfq_queue *bfqq, struct bfq_data *bfqd,
static int bfqq_process_refs(struct bfq_queue *bfqq) static int bfqq_process_refs(struct bfq_queue *bfqq)
{ {
return bfqq->ref - bfqq->allocated - bfqq->entity.on_st - return bfqq->ref - bfqq->allocated - bfqq->entity.on_st_or_in_serv -
(bfqq->weight_counter != NULL); (bfqq->weight_counter != NULL);
} }
......
...@@ -150,7 +150,7 @@ struct bfq_entity { ...@@ -150,7 +150,7 @@ struct bfq_entity {
* Flag, true if the entity is on a tree (either the active or * Flag, true if the entity is on a tree (either the active or
* the idle one of its service_tree) or is in service. * the idle one of its service_tree) or is in service.
*/ */
bool on_st; bool on_st_or_in_serv;
/* B-WF2Q+ start and finish timestamps [sectors/weight] */ /* B-WF2Q+ start and finish timestamps [sectors/weight] */
u64 start, finish; u64 start, finish;
......
...@@ -645,7 +645,7 @@ static void bfq_forget_entity(struct bfq_service_tree *st, ...@@ -645,7 +645,7 @@ static void bfq_forget_entity(struct bfq_service_tree *st,
{ {
struct bfq_queue *bfqq = bfq_entity_to_bfqq(entity); struct bfq_queue *bfqq = bfq_entity_to_bfqq(entity);
entity->on_st = false; entity->on_st_or_in_serv = false;
st->wsum -= entity->weight; st->wsum -= entity->weight;
if (bfqq && !is_in_service) if (bfqq && !is_in_service)
bfq_put_queue(bfqq); bfq_put_queue(bfqq);
...@@ -999,7 +999,7 @@ static void __bfq_activate_entity(struct bfq_entity *entity, ...@@ -999,7 +999,7 @@ static void __bfq_activate_entity(struct bfq_entity *entity,
*/ */
bfq_get_entity(entity); bfq_get_entity(entity);
entity->on_st = true; entity->on_st_or_in_serv = true;
} }
#ifdef CONFIG_BFQ_GROUP_IOSCHED #ifdef CONFIG_BFQ_GROUP_IOSCHED
...@@ -1165,7 +1165,10 @@ bool __bfq_deactivate_entity(struct bfq_entity *entity, bool ins_into_idle_tree) ...@@ -1165,7 +1165,10 @@ bool __bfq_deactivate_entity(struct bfq_entity *entity, bool ins_into_idle_tree)
struct bfq_service_tree *st; struct bfq_service_tree *st;
bool is_in_service; bool is_in_service;
if (!entity->on_st) /* entity never activated, or already inactive */ if (!entity->on_st_or_in_serv) /*
* entity never activated, or
* already inactive
*/
return false; return false;
/* /*
...@@ -1620,7 +1623,7 @@ bool __bfq_bfqd_reset_in_service(struct bfq_data *bfqd) ...@@ -1620,7 +1623,7 @@ bool __bfq_bfqd_reset_in_service(struct bfq_data *bfqd)
* service tree either, then release the service reference to * service tree either, then release the service reference to
* the queue it represents (taken with bfq_get_entity). * the queue it represents (taken with bfq_get_entity).
*/ */
if (!in_serv_entity->on_st) { if (!in_serv_entity->on_st_or_in_serv) {
/* /*
* If no process is referencing in_serv_bfqq any * If no process is referencing in_serv_bfqq any
* longer, then the service reference may be the only * longer, then the service reference may be the only
......
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