UPSTREAM: mm/damon/core: split out scheme stat update logic into a new function
The function for applying a given DAMON scheme action to a given DAMON region, 'damos_apply_scheme()' is not quite short. Make it better to read by splitting out the stat update logic into a new function. Link: https://lkml.kernel.org/r/20221026225943.100429-4-sj@kernel.org Signed-off-by: SeongJae Park <sj@kernel.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> (cherry picked from commit d1cbbf621fc25950938be74a228ef518d05d93a1) Bug: 300502883 Change-Id: I1502a102cdd6959494c249b3633ff97af6ccb94c Signed-off-by: cui yangpei <cuiyangpei@xiaomi.com>
This commit is contained in:
parent
0b0a43029e
commit
43475d9708
1 changed files with 11 additions and 5 deletions
|
|
@ -755,6 +755,16 @@ static bool damos_skip_charged_region(struct damon_target *t,
|
|||
return false;
|
||||
}
|
||||
|
||||
static void damos_update_stat(struct damos *s,
|
||||
unsigned long sz_tried, unsigned long sz_applied)
|
||||
{
|
||||
s->stat.nr_tried++;
|
||||
s->stat.sz_tried += sz_tried;
|
||||
if (sz_applied)
|
||||
s->stat.nr_applied++;
|
||||
s->stat.sz_applied += sz_applied;
|
||||
}
|
||||
|
||||
static void damos_apply_scheme(struct damon_ctx *c, struct damon_target *t,
|
||||
struct damon_region *r, struct damos *s)
|
||||
{
|
||||
|
|
@ -786,11 +796,7 @@ static void damos_apply_scheme(struct damon_ctx *c, struct damon_target *t,
|
|||
r->age = 0;
|
||||
|
||||
update_stat:
|
||||
s->stat.nr_tried++;
|
||||
s->stat.sz_tried += sz;
|
||||
if (sz_applied)
|
||||
s->stat.nr_applied++;
|
||||
s->stat.sz_applied += sz_applied;
|
||||
damos_update_stat(s, sz, sz_applied);
|
||||
}
|
||||
|
||||
static void damon_do_apply_schemes(struct damon_ctx *c,
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue