diff mbox series

[3/5] block, bfq: remove redundant max_t() in bfq_bfqq_charge_time()

Message ID 20221102114354.162-4-Yuwei.Guan@zeekrlife.com
State New
Headers
Series Some cleanup patches for block bfq |

Commit Message

Yuwei Guan Nov. 2, 2022, 11:43 a.m. UTC 
  The "tot_serv_to_charge" is great than or equal to "entity->service",
so no need to do max_t() again.

Signed-off-by: Yuwei Guan <Yuwei.Guan@zeekrlife.com>
---
 block/bfq-wf2q.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/block/bfq-wf2q.c b/block/bfq-wf2q.c
index 8fc3da4c23bb..3da08f4be633 100644
--- a/block/bfq-wf2q.c
+++ b/block/bfq-wf2q.c
@@ -867,8 +867,7 @@  void bfq_bfqq_charge_time(struct bfq_data *bfqd, struct bfq_queue *bfqq,
 	if (tot_serv_to_charge > entity->budget)
 		entity->budget = tot_serv_to_charge;
 
-	bfq_bfqq_served(bfqq,
-			max_t(int, 0, tot_serv_to_charge - entity->service));
+	bfq_bfqq_served(bfqq, tot_serv_to_charge - entity->service);
 }
 
 static void bfq_update_fin_time_enqueue(struct bfq_entity *entity,