summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAsier Gutierrez <gutierrez.asier@huawei-partners.com>2026-02-13 17:50:32 +0300
committerAndrew Morton <akpm@linux-foundation.org>2026-04-05 23:53:00 +0300
commitc9cb94c6b85a2854ae03c874331b0880ee735441 (patch)
treeda9b3038858759aefc709430aeeed55899922bae
parent8a552d68a86ef0e6fb2ff4af13031a5e82c0f1d0 (diff)
downloadlinux-c9cb94c6b85a2854ae03c874331b0880ee735441.tar.xz
mm/damon: remove unused target param of get_scheme_score()
damon_target is not used by get_scheme_score operations, nor with virtual neither with physical addresses. Link: https://lkml.kernel.org/r/20260213145032.1740407-1-gutierrez.asier@huawei-partners.com Signed-off-by: Asier Gutierrez <gutierrez.asier@huawei-partners.com> Reviewed-by: SeongJae Park <sj@kernel.org> Cc: Kefeng Wang <wangkefeng.wang@huawei.com> Cc: Quanmin Yan <yanquanmin1@huawei.com> Cc: ze zuo <zuoze1@huawei.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
-rw-r--r--include/linux/damon.h3
-rw-r--r--mm/damon/core.c10
-rw-r--r--mm/damon/paddr.c3
-rw-r--r--mm/damon/vaddr.c3
4 files changed, 8 insertions, 11 deletions
diff --git a/include/linux/damon.h b/include/linux/damon.h
index be3d198043ff..60e6da3012fa 100644
--- a/include/linux/damon.h
+++ b/include/linux/damon.h
@@ -647,8 +647,7 @@ struct damon_operations {
void (*prepare_access_checks)(struct damon_ctx *context);
unsigned int (*check_accesses)(struct damon_ctx *context);
int (*get_scheme_score)(struct damon_ctx *context,
- struct damon_target *t, struct damon_region *r,
- struct damos *scheme);
+ struct damon_region *r, struct damos *scheme);
unsigned long (*apply_scheme)(struct damon_ctx *context,
struct damon_target *t, struct damon_region *r,
struct damos *scheme, unsigned long *sz_filter_passed);
diff --git a/mm/damon/core.c b/mm/damon/core.c
index 3e1890d64d06..0e5ada441b05 100644
--- a/mm/damon/core.c
+++ b/mm/damon/core.c
@@ -1689,15 +1689,15 @@ static bool __damos_valid_target(struct damon_region *r, struct damos *s)
r->age <= s->pattern.max_age_region;
}
-static bool damos_valid_target(struct damon_ctx *c, struct damon_target *t,
- struct damon_region *r, struct damos *s)
+static bool damos_valid_target(struct damon_ctx *c, struct damon_region *r,
+ struct damos *s)
{
bool ret = __damos_valid_target(r, s);
if (!ret || !s->quota.esz || !c->ops.get_scheme_score)
return ret;
- return c->ops.get_scheme_score(c, t, r, s) >= s->quota.min_score;
+ return c->ops.get_scheme_score(c, r, s) >= s->quota.min_score;
}
/*
@@ -2021,7 +2021,7 @@ static void damon_do_apply_schemes(struct damon_ctx *c,
s->max_nr_snapshots <= s->stat.nr_snapshots)
continue;
- if (damos_valid_target(c, t, r, s))
+ if (damos_valid_target(c, r, s))
damos_apply_scheme(c, t, r, s);
if (damon_is_last_region(r, t))
@@ -2319,7 +2319,7 @@ static void damos_adjust_quota(struct damon_ctx *c, struct damos *s)
damon_for_each_region(r, t) {
if (!__damos_valid_target(r, s))
continue;
- score = c->ops.get_scheme_score(c, t, r, s);
+ score = c->ops.get_scheme_score(c, r, s);
c->regions_score_histogram[score] +=
damon_sz_region(r);
if (score > max_score)
diff --git a/mm/damon/paddr.c b/mm/damon/paddr.c
index 9bfe48826840..5cdcc5037cbc 100644
--- a/mm/damon/paddr.c
+++ b/mm/damon/paddr.c
@@ -343,8 +343,7 @@ static unsigned long damon_pa_apply_scheme(struct damon_ctx *ctx,
}
static int damon_pa_scheme_score(struct damon_ctx *context,
- struct damon_target *t, struct damon_region *r,
- struct damos *scheme)
+ struct damon_region *r, struct damos *scheme)
{
switch (scheme->action) {
case DAMOS_PAGEOUT:
diff --git a/mm/damon/vaddr.c b/mm/damon/vaddr.c
index 729b7ffd3565..4d6d8251d419 100644
--- a/mm/damon/vaddr.c
+++ b/mm/damon/vaddr.c
@@ -985,8 +985,7 @@ static unsigned long damon_va_apply_scheme(struct damon_ctx *ctx,
}
static int damon_va_scheme_score(struct damon_ctx *context,
- struct damon_target *t, struct damon_region *r,
- struct damos *scheme)
+ struct damon_region *r, struct damos *scheme)
{
switch (scheme->action) {