From: SeongJae Park Date: Tue, 22 Mar 2022 21:48:55 +0000 (-0700) Subject: mm/damon/reclaim: use damon_select_ops() instead of damon_{v,p}a_set_operations() X-Git-Tag: v6.1-rc5~1778^2~22 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4d69c3457821100a39fa8c6e0c23ed910bb6c29d;p=platform%2Fkernel%2Flinux-starfive.git mm/damon/reclaim: use damon_select_ops() instead of damon_{v,p}a_set_operations() This commit makes DAMON_RECLAIM to select the registered monitoring operations for the physical address space instead of setting it on its own. This allows DAMON_RECLAIM be independent of DAMON_PADDR, but leave the dependency as is, because it's the only one monitoring operations it use, and therefore it makes no sense to build DAMON_RECLAIM without DAMON_PADDR. Link: https://lkml.kernel.org/r/20220215184603.1479-5-sj@kernel.org Signed-off-by: SeongJae Park Cc: David Rientjes Cc: Xin Hao Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds --- diff --git a/mm/damon/reclaim.c b/mm/damon/reclaim.c index 3c93095..b53d9c2 100644 --- a/mm/damon/reclaim.c +++ b/mm/damon/reclaim.c @@ -384,7 +384,9 @@ static int __init damon_reclaim_init(void) if (!ctx) return -ENOMEM; - damon_pa_set_operations(ctx); + if (damon_select_ops(ctx, DAMON_OPS_PADDR)) + return -EINVAL; + ctx->callback.after_aggregation = damon_reclaim_after_aggregation; target = damon_new_target();