summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHuan Yang <link@vivo.com>2023-09-20 09:57:27 +0800
committerAndrew Morton <akpm@linux-foundation.org>2023-10-16 15:44:38 -0700
commit987ffa5a3858bee448dc791cf6f596790aea52a8 (patch)
treee7284c9e20fc9753eccfb0d8eeb490171ca870f8
parent8c9ae56dc73b5ae48a14000b96292bd4f2aeb710 (diff)
downloadlwn-987ffa5a3858bee448dc791cf6f596790aea52a8.tar.gz
lwn-987ffa5a3858bee448dc791cf6f596790aea52a8.zip
mm/damon/core: remove unnecessary si_meminfo invoke.
si_meminfo() will read and assign more info not just free/ram pages. For just DAMOS_WMARK_FREE_MEM_RATE use, only get free and ram pages is ok to save cpu. Link: https://lkml.kernel.org/r/20230920015727.4482-1-link@vivo.com Signed-off-by: Huan Yang <link@vivo.com> Reviewed-by: SeongJae Park <sj@kernel.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
-rw-r--r--mm/damon/core.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/mm/damon/core.c b/mm/damon/core.c
index 5eb649bd002f..9f4f7c378cf3 100644
--- a/mm/damon/core.c
+++ b/mm/damon/core.c
@@ -1326,12 +1326,10 @@ static bool kdamond_need_stop(struct damon_ctx *ctx)
static unsigned long damos_wmark_metric_value(enum damos_wmark_metric metric)
{
- struct sysinfo i;
-
switch (metric) {
case DAMOS_WMARK_FREE_MEM_RATE:
- si_meminfo(&i);
- return i.freeram * 1000 / i.totalram;
+ return global_zone_page_state(NR_FREE_PAGES) * 1000 /
+ totalram_pages();
default:
break;
}