Re: [PATCH] mm/damon/core: simplify the parameter passing for region split operation

From: kernel test robot
Date: Fri Aug 12 2022 - 08:50:57 EST


Hi,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on akpm-mm/mm-everything]

url: https://github.com/intel-lab-lkp/linux/commits/xiakaixu1987-gmail-com/mm-damon-core-simplify-the-parameter-passing-for-region-split-operation/20220812-154316
base: https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git mm-everything
config: i386-allyesconfig (https://download.01.org/0day-ci/archive/20220812/202208122001.69lQAxjD-lkp@xxxxxxxxx/config)
compiler: gcc-11 (Debian 11.3.0-3) 11.3.0
reproduce (this is a W=1 build):
# https://github.com/intel-lab-lkp/linux/commit/26b7d93b0f64c71d6d324a4a37d85641b55eaec2
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review xiakaixu1987-gmail-com/mm-damon-core-simplify-the-parameter-passing-for-region-split-operation/20220812-154316
git checkout 26b7d93b0f64c71d6d324a4a37d85641b55eaec2
# save the config file
mkdir build_dir && cp config build_dir/.config
make W=1 O=build_dir ARCH=i386 SHELL=/bin/bash

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@xxxxxxxxx>

All errors (new ones prefixed by >>):

In file included from mm/damon/core.c:1218:
mm/damon/core-test.h: In function 'damon_test_split_at':
>> mm/damon/core-test.h:129:31: error: passing argument 1 of 'damon_split_region_at' from incompatible pointer type [-Werror=incompatible-pointer-types]
129 | damon_split_region_at(c, t, r, 25);
| ^
| |
| struct damon_ctx *
mm/damon/core.c:930:56: note: expected 'struct damon_target *' but argument is of type 'struct damon_ctx *'
930 | static void damon_split_region_at(struct damon_target *t,
| ~~~~~~~~~~~~~~~~~~~~~^
In file included from mm/damon/core.c:1218:
mm/damon/core-test.h:129:34: error: passing argument 2 of 'damon_split_region_at' from incompatible pointer type [-Werror=incompatible-pointer-types]
129 | damon_split_region_at(c, t, r, 25);
| ^
| |
| struct damon_target *
mm/damon/core.c:931:56: note: expected 'struct damon_region *' but argument is of type 'struct damon_target *'
931 | struct damon_region *r, unsigned long sz_r)
| ~~~~~~~~~~~~~~~~~~~~~^
In file included from mm/damon/core.c:1218:
mm/damon/core-test.h:129:37: warning: passing argument 3 of 'damon_split_region_at' makes integer from pointer without a cast [-Wint-conversion]
129 | damon_split_region_at(c, t, r, 25);
| ^
| |
| struct damon_region *
mm/damon/core.c:931:73: note: expected 'long unsigned int' but argument is of type 'struct damon_region *'
931 | struct damon_region *r, unsigned long sz_r)
| ~~~~~~~~~~~~~~^~~~
In file included from mm/damon/core.c:1218:
>> mm/damon/core-test.h:129:9: error: too many arguments to function 'damon_split_region_at'
129 | damon_split_region_at(c, t, r, 25);
| ^~~~~~~~~~~~~~~~~~~~~
mm/damon/core.c:930:13: note: declared here
930 | static void damon_split_region_at(struct damon_target *t,
| ^~~~~~~~~~~~~~~~~~~~~
In file included from mm/damon/core.c:1218:
mm/damon/core-test.h: In function 'damon_test_split_regions_of':
>> mm/damon/core-test.h:222:32: error: passing argument 1 of 'damon_split_regions_of' from incompatible pointer type [-Werror=incompatible-pointer-types]
222 | damon_split_regions_of(c, t, 2);
| ^
| |
| struct damon_ctx *
mm/damon/core.c:948:57: note: expected 'struct damon_target *' but argument is of type 'struct damon_ctx *'
948 | static void damon_split_regions_of(struct damon_target *t, int nr_subs)
| ~~~~~~~~~~~~~~~~~~~~~^
In file included from mm/damon/core.c:1218:
mm/damon/core-test.h:222:35: warning: passing argument 2 of 'damon_split_regions_of' makes integer from pointer without a cast [-Wint-conversion]
222 | damon_split_regions_of(c, t, 2);
| ^
| |
| struct damon_target *
mm/damon/core.c:948:64: note: expected 'int' but argument is of type 'struct damon_target *'
948 | static void damon_split_regions_of(struct damon_target *t, int nr_subs)
| ~~~~^~~~~~~
In file included from mm/damon/core.c:1218:
>> mm/damon/core-test.h:222:9: error: too many arguments to function 'damon_split_regions_of'
222 | damon_split_regions_of(c, t, 2);
| ^~~~~~~~~~~~~~~~~~~~~~
mm/damon/core.c:948:13: note: declared here
948 | static void damon_split_regions_of(struct damon_target *t, int nr_subs)
| ^~~~~~~~~~~~~~~~~~~~~~
In file included from mm/damon/core.c:1218:
mm/damon/core-test.h:229:32: error: passing argument 1 of 'damon_split_regions_of' from incompatible pointer type [-Werror=incompatible-pointer-types]
229 | damon_split_regions_of(c, t, 4);
| ^
| |
| struct damon_ctx *
mm/damon/core.c:948:57: note: expected 'struct damon_target *' but argument is of type 'struct damon_ctx *'
948 | static void damon_split_regions_of(struct damon_target *t, int nr_subs)
| ~~~~~~~~~~~~~~~~~~~~~^
In file included from mm/damon/core.c:1218:
mm/damon/core-test.h:229:35: warning: passing argument 2 of 'damon_split_regions_of' makes integer from pointer without a cast [-Wint-conversion]
229 | damon_split_regions_of(c, t, 4);
| ^
| |
| struct damon_target *
mm/damon/core.c:948:64: note: expected 'int' but argument is of type 'struct damon_target *'
948 | static void damon_split_regions_of(struct damon_target *t, int nr_subs)
| ~~~~^~~~~~~
In file included from mm/damon/core.c:1218:
mm/damon/core-test.h:229:9: error: too many arguments to function 'damon_split_regions_of'
229 | damon_split_regions_of(c, t, 4);
| ^~~~~~~~~~~~~~~~~~~~~~
mm/damon/core.c:948:13: note: declared here
948 | static void damon_split_regions_of(struct damon_target *t, int nr_subs)
| ^~~~~~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors


vim +/damon_split_region_at +129 mm/damon/core-test.h

17ccae8bb5c928 SeongJae Park 2021-09-07 119
17ccae8bb5c928 SeongJae Park 2021-09-07 120 static void damon_test_split_at(struct kunit *test)
17ccae8bb5c928 SeongJae Park 2021-09-07 121 {
17ccae8bb5c928 SeongJae Park 2021-09-07 122 struct damon_ctx *c = damon_new_ctx();
17ccae8bb5c928 SeongJae Park 2021-09-07 123 struct damon_target *t;
17ccae8bb5c928 SeongJae Park 2021-09-07 124 struct damon_region *r;
17ccae8bb5c928 SeongJae Park 2021-09-07 125
1971bd630452e9 SeongJae Park 2022-03-22 126 t = damon_new_target();
17ccae8bb5c928 SeongJae Park 2021-09-07 127 r = damon_new_region(0, 100);
17ccae8bb5c928 SeongJae Park 2021-09-07 128 damon_add_region(r, t);
17ccae8bb5c928 SeongJae Park 2021-09-07 @129 damon_split_region_at(c, t, r, 25);
17ccae8bb5c928 SeongJae Park 2021-09-07 130 KUNIT_EXPECT_EQ(test, r->ar.start, 0ul);
17ccae8bb5c928 SeongJae Park 2021-09-07 131 KUNIT_EXPECT_EQ(test, r->ar.end, 25ul);
17ccae8bb5c928 SeongJae Park 2021-09-07 132
17ccae8bb5c928 SeongJae Park 2021-09-07 133 r = damon_next_region(r);
17ccae8bb5c928 SeongJae Park 2021-09-07 134 KUNIT_EXPECT_EQ(test, r->ar.start, 25ul);
17ccae8bb5c928 SeongJae Park 2021-09-07 135 KUNIT_EXPECT_EQ(test, r->ar.end, 100ul);
17ccae8bb5c928 SeongJae Park 2021-09-07 136
17ccae8bb5c928 SeongJae Park 2021-09-07 137 damon_free_target(t);
17ccae8bb5c928 SeongJae Park 2021-09-07 138 damon_destroy_ctx(c);
17ccae8bb5c928 SeongJae Park 2021-09-07 139 }
17ccae8bb5c928 SeongJae Park 2021-09-07 140
17ccae8bb5c928 SeongJae Park 2021-09-07 141 static void damon_test_merge_two(struct kunit *test)
17ccae8bb5c928 SeongJae Park 2021-09-07 142 {
17ccae8bb5c928 SeongJae Park 2021-09-07 143 struct damon_target *t;
17ccae8bb5c928 SeongJae Park 2021-09-07 144 struct damon_region *r, *r2, *r3;
17ccae8bb5c928 SeongJae Park 2021-09-07 145 int i;
17ccae8bb5c928 SeongJae Park 2021-09-07 146
1971bd630452e9 SeongJae Park 2022-03-22 147 t = damon_new_target();
17ccae8bb5c928 SeongJae Park 2021-09-07 148 r = damon_new_region(0, 100);
17ccae8bb5c928 SeongJae Park 2021-09-07 149 r->nr_accesses = 10;
17ccae8bb5c928 SeongJae Park 2021-09-07 150 damon_add_region(r, t);
17ccae8bb5c928 SeongJae Park 2021-09-07 151 r2 = damon_new_region(100, 300);
17ccae8bb5c928 SeongJae Park 2021-09-07 152 r2->nr_accesses = 20;
17ccae8bb5c928 SeongJae Park 2021-09-07 153 damon_add_region(r2, t);
17ccae8bb5c928 SeongJae Park 2021-09-07 154
17ccae8bb5c928 SeongJae Park 2021-09-07 155 damon_merge_two_regions(t, r, r2);
17ccae8bb5c928 SeongJae Park 2021-09-07 156 KUNIT_EXPECT_EQ(test, r->ar.start, 0ul);
17ccae8bb5c928 SeongJae Park 2021-09-07 157 KUNIT_EXPECT_EQ(test, r->ar.end, 300ul);
17ccae8bb5c928 SeongJae Park 2021-09-07 158 KUNIT_EXPECT_EQ(test, r->nr_accesses, 16u);
17ccae8bb5c928 SeongJae Park 2021-09-07 159
17ccae8bb5c928 SeongJae Park 2021-09-07 160 i = 0;
17ccae8bb5c928 SeongJae Park 2021-09-07 161 damon_for_each_region(r3, t) {
17ccae8bb5c928 SeongJae Park 2021-09-07 162 KUNIT_EXPECT_PTR_EQ(test, r, r3);
17ccae8bb5c928 SeongJae Park 2021-09-07 163 i++;
17ccae8bb5c928 SeongJae Park 2021-09-07 164 }
17ccae8bb5c928 SeongJae Park 2021-09-07 165 KUNIT_EXPECT_EQ(test, i, 1);
17ccae8bb5c928 SeongJae Park 2021-09-07 166
17ccae8bb5c928 SeongJae Park 2021-09-07 167 damon_free_target(t);
17ccae8bb5c928 SeongJae Park 2021-09-07 168 }
17ccae8bb5c928 SeongJae Park 2021-09-07 169
17ccae8bb5c928 SeongJae Park 2021-09-07 170 static struct damon_region *__nth_region_of(struct damon_target *t, int idx)
17ccae8bb5c928 SeongJae Park 2021-09-07 171 {
17ccae8bb5c928 SeongJae Park 2021-09-07 172 struct damon_region *r;
17ccae8bb5c928 SeongJae Park 2021-09-07 173 unsigned int i = 0;
17ccae8bb5c928 SeongJae Park 2021-09-07 174
17ccae8bb5c928 SeongJae Park 2021-09-07 175 damon_for_each_region(r, t) {
17ccae8bb5c928 SeongJae Park 2021-09-07 176 if (i++ == idx)
17ccae8bb5c928 SeongJae Park 2021-09-07 177 return r;
17ccae8bb5c928 SeongJae Park 2021-09-07 178 }
17ccae8bb5c928 SeongJae Park 2021-09-07 179
17ccae8bb5c928 SeongJae Park 2021-09-07 180 return NULL;
17ccae8bb5c928 SeongJae Park 2021-09-07 181 }
17ccae8bb5c928 SeongJae Park 2021-09-07 182
17ccae8bb5c928 SeongJae Park 2021-09-07 183 static void damon_test_merge_regions_of(struct kunit *test)
17ccae8bb5c928 SeongJae Park 2021-09-07 184 {
17ccae8bb5c928 SeongJae Park 2021-09-07 185 struct damon_target *t;
17ccae8bb5c928 SeongJae Park 2021-09-07 186 struct damon_region *r;
17ccae8bb5c928 SeongJae Park 2021-09-07 187 unsigned long sa[] = {0, 100, 114, 122, 130, 156, 170, 184};
17ccae8bb5c928 SeongJae Park 2021-09-07 188 unsigned long ea[] = {100, 112, 122, 130, 156, 170, 184, 230};
17ccae8bb5c928 SeongJae Park 2021-09-07 189 unsigned int nrs[] = {0, 0, 10, 10, 20, 30, 1, 2};
17ccae8bb5c928 SeongJae Park 2021-09-07 190
17ccae8bb5c928 SeongJae Park 2021-09-07 191 unsigned long saddrs[] = {0, 114, 130, 156, 170};
17ccae8bb5c928 SeongJae Park 2021-09-07 192 unsigned long eaddrs[] = {112, 130, 156, 170, 230};
17ccae8bb5c928 SeongJae Park 2021-09-07 193 int i;
17ccae8bb5c928 SeongJae Park 2021-09-07 194
1971bd630452e9 SeongJae Park 2022-03-22 195 t = damon_new_target();
17ccae8bb5c928 SeongJae Park 2021-09-07 196 for (i = 0; i < ARRAY_SIZE(sa); i++) {
17ccae8bb5c928 SeongJae Park 2021-09-07 197 r = damon_new_region(sa[i], ea[i]);
17ccae8bb5c928 SeongJae Park 2021-09-07 198 r->nr_accesses = nrs[i];
17ccae8bb5c928 SeongJae Park 2021-09-07 199 damon_add_region(r, t);
17ccae8bb5c928 SeongJae Park 2021-09-07 200 }
17ccae8bb5c928 SeongJae Park 2021-09-07 201
17ccae8bb5c928 SeongJae Park 2021-09-07 202 damon_merge_regions_of(t, 9, 9999);
17ccae8bb5c928 SeongJae Park 2021-09-07 203 /* 0-112, 114-130, 130-156, 156-170 */
17ccae8bb5c928 SeongJae Park 2021-09-07 204 KUNIT_EXPECT_EQ(test, damon_nr_regions(t), 5u);
17ccae8bb5c928 SeongJae Park 2021-09-07 205 for (i = 0; i < 5; i++) {
17ccae8bb5c928 SeongJae Park 2021-09-07 206 r = __nth_region_of(t, i);
17ccae8bb5c928 SeongJae Park 2021-09-07 207 KUNIT_EXPECT_EQ(test, r->ar.start, saddrs[i]);
17ccae8bb5c928 SeongJae Park 2021-09-07 208 KUNIT_EXPECT_EQ(test, r->ar.end, eaddrs[i]);
17ccae8bb5c928 SeongJae Park 2021-09-07 209 }
17ccae8bb5c928 SeongJae Park 2021-09-07 210 damon_free_target(t);
17ccae8bb5c928 SeongJae Park 2021-09-07 211 }
17ccae8bb5c928 SeongJae Park 2021-09-07 212
17ccae8bb5c928 SeongJae Park 2021-09-07 213 static void damon_test_split_regions_of(struct kunit *test)
17ccae8bb5c928 SeongJae Park 2021-09-07 214 {
17ccae8bb5c928 SeongJae Park 2021-09-07 215 struct damon_ctx *c = damon_new_ctx();
17ccae8bb5c928 SeongJae Park 2021-09-07 216 struct damon_target *t;
17ccae8bb5c928 SeongJae Park 2021-09-07 217 struct damon_region *r;
17ccae8bb5c928 SeongJae Park 2021-09-07 218
1971bd630452e9 SeongJae Park 2022-03-22 219 t = damon_new_target();
17ccae8bb5c928 SeongJae Park 2021-09-07 220 r = damon_new_region(0, 22);
17ccae8bb5c928 SeongJae Park 2021-09-07 221 damon_add_region(r, t);
17ccae8bb5c928 SeongJae Park 2021-09-07 @222 damon_split_regions_of(c, t, 2);
2e014660b3e4b7 SeongJae Park 2021-10-28 223 KUNIT_EXPECT_LE(test, damon_nr_regions(t), 2u);
17ccae8bb5c928 SeongJae Park 2021-09-07 224 damon_free_target(t);
17ccae8bb5c928 SeongJae Park 2021-09-07 225
1971bd630452e9 SeongJae Park 2022-03-22 226 t = damon_new_target();
17ccae8bb5c928 SeongJae Park 2021-09-07 227 r = damon_new_region(0, 220);
17ccae8bb5c928 SeongJae Park 2021-09-07 228 damon_add_region(r, t);
17ccae8bb5c928 SeongJae Park 2021-09-07 229 damon_split_regions_of(c, t, 4);
2e014660b3e4b7 SeongJae Park 2021-10-28 230 KUNIT_EXPECT_LE(test, damon_nr_regions(t), 4u);
17ccae8bb5c928 SeongJae Park 2021-09-07 231 damon_free_target(t);
17ccae8bb5c928 SeongJae Park 2021-09-07 232 damon_destroy_ctx(c);
17ccae8bb5c928 SeongJae Park 2021-09-07 233 }
17ccae8bb5c928 SeongJae Park 2021-09-07 234

--
0-DAY CI Kernel Test Service
https://01.org/lkp