Commit 8e34bac5 authored by SeongJae Park's avatar SeongJae Park Committed by Andrew Morton
Browse files

mm/damon/dbgfs-test: skip dbgfs_set_targets() test if PADDR is not registered 

The test depends on registration of DAMON_OPS_PADDR.  It would be
registered only when CONFIG_DAMON_PADDR is set.  DAMON core kunit tests do
fake ops registration for such case.  However, the functions for such fake
ops registration is not available to DAMON debugfs interface.  Just skip
the test in the case.

Link: https://lkml.kernel.org/r/20240827030336.7930-7-sj@kernel.org


Fixes: 999b9467 ("mm/damon/dbgfs-test: fix is_target_id() change")
Signed-off-by: default avatarSeongJae Park <sj@kernel.org>
Cc: Brendan Higgins <brendanhiggins@google.com>
Cc: David Gow <davidgow@google.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent e43772dc

mm/damon/dbgfs-test.h

+5 −0
Original line number Diff line number Diff line
@@ -73,6 +73,11 @@ static void damon_dbgfs_test_set_targets(struct kunit *test) 
	struct damon_ctx *ctx = dbgfs_new_ctx();

	char buf[64];



	if (!damon_is_registered_ops(DAMON_OPS_PADDR)) {

		dbgfs_destroy_ctx(ctx);

		kunit_skip(test, "PADDR not registered");

	}



	/* Make DAMON consider target has no pid */

	damon_select_ops(ctx, DAMON_OPS_PADDR);