mm/damon/reclaim: remove unnecessary code for online tuning
authorSeongJae Park <sj@kernel.org>
Tue, 18 Jun 2024 18:18:07 +0000 (11:18 -0700)
committerAndrew Morton <akpm@linux-foundation.org>
Thu, 4 Jul 2024 02:30:14 +0000 (19:30 -0700)
DAMON_RECLAIM contains code for handling of online DAMON parameters update
edge cases.  It is no more necessary since damon_commit_ctx() takes care
of the cases.  Remove the unnecessary code.

Link: https://lkml.kernel.org/r/20240618181809.82078-11-sj@kernel.org
Signed-off-by: SeongJae Park <sj@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
mm/damon/reclaim.c

index be7f04b00d0cf5ac5dc25dbf0686a733ad0e0e00..9e0077a9404e2ce64ff7142938e0536687dc7b60 100644 (file)
@@ -181,23 +181,11 @@ static struct damos *damon_reclaim_new_scheme(void)
                        NUMA_NO_NODE);
 }
 
-static void damon_reclaim_copy_quota_status(struct damos_quota *dst,
-               struct damos_quota *src)
-{
-       dst->total_charged_sz = src->total_charged_sz;
-       dst->total_charged_ns = src->total_charged_ns;
-       dst->charged_sz = src->charged_sz;
-       dst->charged_from = src->charged_from;
-       dst->charge_target_from = src->charge_target_from;
-       dst->charge_addr_from = src->charge_addr_from;
-       dst->esz_bp = src->esz_bp;
-}
-
 static int damon_reclaim_apply_parameters(void)
 {
        struct damon_ctx *param_ctx;
        struct damon_target *param_target;
-       struct damos *scheme, *old_scheme;
+       struct damos *scheme;
        struct damos_quota_goal *goal;
        struct damos_filter *filter;
        int err;
@@ -214,11 +202,6 @@ static int damon_reclaim_apply_parameters(void)
        scheme = damon_reclaim_new_scheme();
        if (!scheme)
                goto out;
-       if (!list_empty(&ctx->schemes)) {
-               damon_for_each_scheme(old_scheme, ctx)
-                       damon_reclaim_copy_quota_status(&scheme->quota,
-                                       &old_scheme->quota);
-       }
        damon_set_schemes(ctx, &scheme, 1);
 
        if (quota_mem_pressure_us) {