From: kernel test robot <lkp@intel.com>
To: Suhaas Joshi <suhaas@s-joshi.in>,
sj@kernel.org, akpm@linux-foundation.org,
aethernet65535@gmail.com
Cc: oe-kbuild-all@lists.linux.dev, damon@lists.linux.dev,
linux-mm@kvack.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 2/2] mm/damon/core: Copy only installed probe in damon_split_region_at()
Date: Fri, 9 Oct 2026 10:40:07 +0800 [thread overview]
Message-ID: <202610091039.fhMXCn7d-lkp@intel.com> (raw)
In-Reply-To: <20261008151944.113714-3-suhaas@s-joshi.in>
Hi Suhaas,
kernel test robot noticed the following build errors:
[auto build test ERROR on akpm-mm/mm-everything]
url: https://github.com/intel-lab-lkp/linux/commits/Suhaas-Joshi/mm-damon-core-Copy-only-installed-probe-in-damon_split_region_at/20261008-204926
base: https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git mm-everything
patch link: https://lore.kernel.org/r/20261008151944.113714-3-suhaas%40s-joshi.in
patch subject: [PATCH v2 2/2] mm/damon/core: Copy only installed probe in damon_split_region_at()
config: csky-randconfig-1001-20261009 (https://download.01.org/0day-ci/archive/20261009/202610091039.fhMXCn7d-lkp@intel.com/config)
compiler: csky-linux-gcc (GCC) 9.5.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20261009/202610091039.fhMXCn7d-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202610091039.fhMXCn7d-lkp@intel.com/
All error/warnings (new ones prefixed by >>):
In file included from mm/damon/core.c:4295:
mm/damon/tests/core-kunit.h: In function 'damon_test_last_region_quota_reset':
>> mm/damon/tests/core-kunit.h:2519:38: error: passing argument 1 of 'damos_skip_charged_region' from incompatible pointer type [-Werror=incompatible-pointer-types]
2519 | skipped = damos_skip_charged_region(t, r2, &s, min_region_sz);
| ^
| |
| struct damon_target *
mm/damon/core.c:2500:57: note: expected 'struct damon_ctx *' but argument is of type 'struct damon_target *'
2500 | static bool damos_skip_charged_region(struct damon_ctx *ctx,
| ~~~~~~~~~~~~~~~~~~^~~
In file included from mm/damon/core.c:4295:
mm/damon/tests/core-kunit.h:2519:41: error: passing argument 2 of 'damos_skip_charged_region' from incompatible pointer type [-Werror=incompatible-pointer-types]
2519 | skipped = damos_skip_charged_region(t, r2, &s, min_region_sz);
| ^~
| |
| struct damon_region *
mm/damon/core.c:2501:32: note: expected 'struct damon_target *' but argument is of type 'struct damon_region *'
2501 | struct damon_target *t,
| ~~~~~~~~~~~~~~~~~~~~~^
In file included from mm/damon/core.c:4295:
mm/damon/tests/core-kunit.h:2519:45: error: passing argument 3 of 'damos_skip_charged_region' from incompatible pointer type [-Werror=incompatible-pointer-types]
2519 | skipped = damos_skip_charged_region(t, r2, &s, min_region_sz);
| ^~
| |
| struct damos *
mm/damon/core.c:2502:32: note: expected 'struct damon_region *' but argument is of type 'struct damos *'
2502 | struct damon_region *r, struct damos *s,
| ~~~~~~~~~~~~~~~~~~~~~^
In file included from mm/damon/core.c:4295:
>> mm/damon/tests/core-kunit.h:2519:49: warning: passing argument 4 of 'damos_skip_charged_region' makes pointer from integer without a cast [-Wint-conversion]
2519 | skipped = damos_skip_charged_region(t, r2, &s, min_region_sz);
| ^~~~~~~~~~~~~
| |
| long unsigned int
mm/damon/core.c:2502:49: note: expected 'struct damos *' but argument is of type 'long unsigned int'
2502 | struct damon_region *r, struct damos *s,
| ~~~~~~~~~~~~~~^
In file included from mm/damon/core.c:4295:
>> mm/damon/tests/core-kunit.h:2519:12: error: too few arguments to function 'damos_skip_charged_region'
2519 | skipped = damos_skip_charged_region(t, r2, &s, min_region_sz);
| ^~~~~~~~~~~~~~~~~~~~~~~~~
mm/damon/core.c:2500:13: note: declared here
2500 | static bool damos_skip_charged_region(struct damon_ctx *ctx,
| ^~~~~~~~~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +/damos_skip_charged_region +2519 mm/damon/tests/core-kunit.h
79500f5843f453 Donggeun Yoo 2026-09-28 2489
18793c04abe665 Liew Rui Yan 2026-10-05 2490 static void damon_test_last_region_quota_reset(struct kunit *test)
18793c04abe665 Liew Rui Yan 2026-10-05 2491 {
18793c04abe665 Liew Rui Yan 2026-10-05 2492 struct damos s = {0};
18793c04abe665 Liew Rui Yan 2026-10-05 2493 struct damon_target *t;
18793c04abe665 Liew Rui Yan 2026-10-05 2494 struct damon_region *r1, *r2;
18793c04abe665 Liew Rui Yan 2026-10-05 2495 unsigned long min_region_sz = 1;
18793c04abe665 Liew Rui Yan 2026-10-05 2496 bool skipped;
18793c04abe665 Liew Rui Yan 2026-10-05 2497
18793c04abe665 Liew Rui Yan 2026-10-05 2498 t = damon_new_target();
18793c04abe665 Liew Rui Yan 2026-10-05 2499 if (!t)
18793c04abe665 Liew Rui Yan 2026-10-05 2500 kunit_skip(test, "target alloc fail");
18793c04abe665 Liew Rui Yan 2026-10-05 2501
18793c04abe665 Liew Rui Yan 2026-10-05 2502 r1 = damon_new_region(0, 100);
18793c04abe665 Liew Rui Yan 2026-10-05 2503 if (!r1) {
18793c04abe665 Liew Rui Yan 2026-10-05 2504 damon_free_target(t);
18793c04abe665 Liew Rui Yan 2026-10-05 2505 kunit_skip(test, "region 1 alloc fail");
18793c04abe665 Liew Rui Yan 2026-10-05 2506 }
18793c04abe665 Liew Rui Yan 2026-10-05 2507 damon_add_region(r1, t);
18793c04abe665 Liew Rui Yan 2026-10-05 2508
18793c04abe665 Liew Rui Yan 2026-10-05 2509 r2 = damon_new_region(100, 200);
18793c04abe665 Liew Rui Yan 2026-10-05 2510 if (!r2) {
18793c04abe665 Liew Rui Yan 2026-10-05 2511 damon_free_target(t);
18793c04abe665 Liew Rui Yan 2026-10-05 2512 kunit_skip(test, "region 2 alloc fail");
18793c04abe665 Liew Rui Yan 2026-10-05 2513 }
18793c04abe665 Liew Rui Yan 2026-10-05 2514 damon_add_region(r2, t);
18793c04abe665 Liew Rui Yan 2026-10-05 2515
18793c04abe665 Liew Rui Yan 2026-10-05 2516 s.quota.charge_target_from = t;
18793c04abe665 Liew Rui Yan 2026-10-05 2517 s.quota.charge_addr_from = r1->ar.end;
18793c04abe665 Liew Rui Yan 2026-10-05 2518
18793c04abe665 Liew Rui Yan 2026-10-05 @2519 skipped = damos_skip_charged_region(t, r2, &s, min_region_sz);
18793c04abe665 Liew Rui Yan 2026-10-05 2520
18793c04abe665 Liew Rui Yan 2026-10-05 2521 /* 'r2' is not processed, it should not skip */
18793c04abe665 Liew Rui Yan 2026-10-05 2522 KUNIT_EXPECT_EQ(test, skipped, false);
18793c04abe665 Liew Rui Yan 2026-10-05 2523
18793c04abe665 Liew Rui Yan 2026-10-05 2524 /* 'r2' is last region, it should reset charge_{target,addr}_from */
18793c04abe665 Liew Rui Yan 2026-10-05 2525 KUNIT_EXPECT_PTR_EQ(test, s.quota.charge_target_from, NULL);
18793c04abe665 Liew Rui Yan 2026-10-05 2526 KUNIT_EXPECT_EQ(test, s.quota.charge_addr_from, 0);
18793c04abe665 Liew Rui Yan 2026-10-05 2527
18793c04abe665 Liew Rui Yan 2026-10-05 2528 damon_free_target(t);
18793c04abe665 Liew Rui Yan 2026-10-05 2529 }
18793c04abe665 Liew Rui Yan 2026-10-05 2530
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2026-10-09 2:40 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20261008151944.113714-1-suhaas@s-joshi.in>
2026-10-08 15:19 ` [PATCH v2 1/2] mm/damon/core: Use only installed probe in damon_merge_two_regions() Suhaas Joshi
2026-10-08 15:33 ` Suhaas Joshi
2026-10-09 7:46 ` SJ Park
2026-10-09 7:43 ` SJ Park
2026-10-08 15:19 ` [PATCH v2 2/2] mm/damon/core: Copy only installed probe in damon_split_region_at() Suhaas Joshi
2026-10-09 2:40 ` kernel test robot [this message]
2026-10-09 8:21 ` SJ Park
2026-10-09 15:43 ` Suhaas Joshi
2026-10-09 4:06 ` kernel test robot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202610091039.fhMXCn7d-lkp@intel.com \
--to=lkp@intel.com \
--cc=aethernet65535@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=damon@lists.linux.dev \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=sj@kernel.org \
--cc=suhaas@s-joshi.in \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®