Commit 60fa9355 authored by SeongJae Park's avatar SeongJae Park Committed by Andrew Morton
Browse files

mm/damon/core: implement per-scheme ops-handled filter-passed bytes stat 

Implement a new per-DAMOS scheme statistic field, namely
sz_ops_filter_passed, using the changed damon_operations->apply_scheme()
interface.  It counts total bytes of memory that given DAMOS action tried
to be applied, and passed the operations layer handled region-internal
filters of the scheme.  DAMON API users can access it using DAMON-internal
safe access features such as damon_call() and/or damos_walk().

Link: https://lkml.kernel.org/r/20250106193401.109161-8-sj@kernel.org


Signed-off-by: default avatarSeongJae Park <sj@kernel.org>
Cc: Jonathan Corbet <corbet@lwn.net>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent c0cb9d91

include/linux/damon.h

+3 −0
Original line number Diff line number Diff line
@@ -286,6 +286,8 @@ struct damos_watermarks { 
 * @sz_tried:	Total size of regions that the scheme is tried to be applied.

 * @nr_applied:	Total number of regions that the scheme is applied.

 * @sz_applied:	Total size of regions that the scheme is applied.

 * @sz_ops_filter_passed:

 *		Total bytes that passed ops layer-handled DAMOS filters.

 * @qt_exceeds: Total number of times the quota of the scheme has exceeded.

 *

 * "Tried an action to a region" in this context means the DAMOS core logic
@@ -310,6 +312,7 @@ struct damos_stat { 
	unsigned long sz_tried;

	unsigned long nr_applied;

	unsigned long sz_applied;

	unsigned long sz_ops_filter_passed;

	unsigned long qt_exceeds;

};

mm/damon/core.c

+4 −2
Original line number Diff line number Diff line
@@ -1362,13 +1362,15 @@ static bool damos_skip_charged_region(struct damon_target *t, 
}



static void damos_update_stat(struct damos *s,

		unsigned long sz_tried, unsigned long sz_applied)

		unsigned long sz_tried, unsigned long sz_applied,

		unsigned long sz_ops_filter_passed)

{

	s->stat.nr_tried++;

	s->stat.sz_tried += sz_tried;

	if (sz_applied)

		s->stat.nr_applied++;

	s->stat.sz_applied += sz_applied;

	s->stat.sz_ops_filter_passed += sz_ops_filter_passed;

}



static bool __damos_filter_out(struct damon_ctx *ctx, struct damon_target *t,
@@ -1586,7 +1588,7 @@ static void damos_apply_scheme(struct damon_ctx *c, struct damon_target *t, 
		r->age = 0;



update_stat:

	damos_update_stat(s, sz, sz_applied);

	damos_update_stat(s, sz, sz_applied, sz_ops_filter_passed);

}



static void damon_do_apply_schemes(struct damon_ctx *c,