lkml.org 
[lkml]   [2022]   [Oct]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH 8/8] blk-iocost: Get ioc_now inside weight_updated
Date
The ioc_now parameter of weight_updated is only needed if we need call
propagate_weights. Move ioc_now inside weight_updated to remove
unnecessary get of ioc_now from outside of weight_updated.

Signed-off-by: Kemeng Shi <shikemeng@huawei.com>
---
block/blk-iocost.c | 18 +++++++++---------
1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/block/blk-iocost.c b/block/blk-iocost.c
index 4815e676733d..e2aaf133deb8 100644
--- a/block/blk-iocost.c
+++ b/block/blk-iocost.c
@@ -1220,18 +1220,21 @@ static u32 current_hweight_max(struct ioc_gq *iocg)
return max_t(u32, hwm, 1);
}

-static void weight_updated(struct ioc_gq *iocg, struct ioc_now *now)
+static void weight_updated(struct ioc_gq *iocg)
{
struct ioc *ioc = iocg->ioc;
struct blkcg_gq *blkg = iocg_to_blkg(iocg);
struct ioc_cgrp *iocc = blkcg_to_iocc(blkg->blkcg);
+ struct ioc_now now;
u32 weight;

lockdep_assert_held(&ioc->lock);

weight = iocg->cfg_weight ?: iocc->dfl_weight;
- if (weight != iocg->weight && iocg->active)
- propagate_weights(iocg, weight, iocg->inuse, true, now);
+ if (weight != iocg->weight && iocg->active) {
+ ioc_now(iocg->ioc, &now);
+ propagate_weights(iocg, weight, iocg->inuse, true, &now);
+ }
iocg->weight = weight;
}

@@ -2968,7 +2971,7 @@ static void ioc_pd_init(struct blkg_policy_data *pd)
}

spin_lock_irqsave(&ioc->lock, flags);
- weight_updated(iocg, NULL);
+ weight_updated(iocg);
spin_unlock_irqrestore(&ioc->lock, flags);
}

@@ -3053,7 +3056,6 @@ static ssize_t ioc_weight_write(struct kernfs_open_file *of, char *buf,
struct blkcg *blkcg = css_to_blkcg(of_css(of));
struct ioc_cgrp *iocc = blkcg_to_iocc(blkcg);
struct blkg_conf_ctx ctx;
- struct ioc_now now;
struct ioc_gq *iocg;
u32 v;
int ret;
@@ -3074,8 +3076,7 @@ static ssize_t ioc_weight_write(struct kernfs_open_file *of, char *buf,

if (iocg) {
spin_lock(&iocg->ioc->lock);
- ioc_now(iocg->ioc, &now);
- weight_updated(iocg, &now);
+ weight_updated(iocg);
spin_unlock(&iocg->ioc->lock);
}
}
@@ -3101,8 +3102,7 @@ static ssize_t ioc_weight_write(struct kernfs_open_file *of, char *buf,

spin_lock(&iocg->ioc->lock);
iocg->cfg_weight = v * WEIGHT_ONE;
- ioc_now(iocg->ioc, &now);
- weight_updated(iocg, &now);
+ weight_updated(iocg);
spin_unlock(&iocg->ioc->lock);

blkg_conf_finish(&ctx);
--
2.30.0
\
 
 \ /
  Last update: 2022-10-17 04:01    [W:1.329 / U:0.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site