Commit 513a2437 authored by Huan Yang's avatar Huan Yang Committed by Ze Zuo
Browse files

mm/damon/core: remove unnecessary si_meminfo invoke. 

mainline inclusion
from mainline-v6.7-rc1
commit 987ffa5a3858bee448dc791cf6f596790aea52a8
category: bugfix
bugzilla: https://gitee.com/src-openeuler/kernel/issues/IBDTU7

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=987ffa5a3858bee448dc791cf6f596790aea52a8

--------------------------------

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: default avatarHuan Yang <link@vivo.com>
Reviewed-by: default avatarSeongJae Park <sj@kernel.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarZe Zuo <zuoze1@huawei.com>
parent 16a0cbac

mm/damon/core.c

+2 −4
Original line number Diff line number Diff line
@@ -1261,12 +1261,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;

	}