[hch-block:alloc_disk-part3 59/60] arch/um/drivers/ubd_kern.c:247:7: warning: variable 'major' set but not used
by kernel test robot
tree: git://git.infradead.org/users/hch/block.git alloc_disk-part3
head: 92eea9c8466f585a79e99bef97404389d1e6eb87
commit: 6e11b3a51b107394b94b87170401d748a7d1bf17 [59/60] ubd: remove the code to register as the legacy IDE driver
config: um-randconfig-r014-20210531 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
git remote add hch-block git://git.infradead.org/users/hch/block.git
git fetch --no-tags hch-block alloc_disk-part3
git checkout 6e11b3a51b107394b94b87170401d748a7d1bf17
# save the attached .config to linux build tree
make W=1 ARCH=um
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
arch/um/drivers/ubd_kern.c: In function 'ubd_setup_common':
>> arch/um/drivers/ubd_kern.c:247:7: warning: variable 'major' set but not used [-Wunused-but-set-variable]
247 | int major;
| ^~~~~
vim +/major +247 arch/um/drivers/ubd_kern.c
^1da177e4c3f41 Linus Torvalds 2005-04-16 231
d8d7c28ec0b50a Paolo 'Blaisorblade' Giarrusso 2006-10-30 232 /* If *index_out == -1 at exit, the passed option was a general one;
d8d7c28ec0b50a Paolo 'Blaisorblade' Giarrusso 2006-10-30 233 * otherwise, the str pointer is used (and owned) inside ubd_devs array, so it
d8d7c28ec0b50a Paolo 'Blaisorblade' Giarrusso 2006-10-30 234 * should not be freed on exit.
d8d7c28ec0b50a Paolo 'Blaisorblade' Giarrusso 2006-10-30 235 */
f28169d2000177 Jeff Dike 2007-02-10 236 static int ubd_setup_common(char *str, int *index_out, char **error_out)
^1da177e4c3f41 Linus Torvalds 2005-04-16 237 {
7d314e346d6081 Paolo 'Blaisorblade' Giarrusso 2006-10-30 238 struct ubd *ubd_dev;
^1da177e4c3f41 Linus Torvalds 2005-04-16 239 struct openflags flags = global_openflags;
ef3ba87cb7c911 Christopher Obbard 2020-11-23 240 char *file, *backing_file, *serial;
b8831a1d2c78c0 Jeff Dike 2007-02-10 241 int n, err = 0, i;
^1da177e4c3f41 Linus Torvalds 2005-04-16 242
^1da177e4c3f41 Linus Torvalds 2005-04-16 243 if(index_out) *index_out = -1;
^1da177e4c3f41 Linus Torvalds 2005-04-16 244 n = *str;
^1da177e4c3f41 Linus Torvalds 2005-04-16 245 if(n == '='){
^1da177e4c3f41 Linus Torvalds 2005-04-16 246 char *end;
^1da177e4c3f41 Linus Torvalds 2005-04-16 @247 int major;
^1da177e4c3f41 Linus Torvalds 2005-04-16 248
^1da177e4c3f41 Linus Torvalds 2005-04-16 249 str++;
^1da177e4c3f41 Linus Torvalds 2005-04-16 250 if(!strcmp(str, "sync")){
^1da177e4c3f41 Linus Torvalds 2005-04-16 251 global_openflags = of_sync(global_openflags);
9ca55299f2ee0a Daniel Walter 2019-04-02 252 return err;
^1da177e4c3f41 Linus Torvalds 2005-04-16 253 }
b8831a1d2c78c0 Jeff Dike 2007-02-10 254
b8831a1d2c78c0 Jeff Dike 2007-02-10 255 err = -EINVAL;
^1da177e4c3f41 Linus Torvalds 2005-04-16 256 major = simple_strtoul(str, &end, 0);
^1da177e4c3f41 Linus Torvalds 2005-04-16 257 if((*end != '\0') || (end == str)){
f28169d2000177 Jeff Dike 2007-02-10 258 *error_out = "Didn't parse major number";
9ca55299f2ee0a Daniel Walter 2019-04-02 259 return err;
^1da177e4c3f41 Linus Torvalds 2005-04-16 260 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 261
6e11b3a51b1073 Christoph Hellwig 2021-05-04 262 pr_warn("fake major not supported any more\n");
6e11b3a51b1073 Christoph Hellwig 2021-05-04 263 return 0;
^1da177e4c3f41 Linus Torvalds 2005-04-16 264 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 265
^1da177e4c3f41 Linus Torvalds 2005-04-16 266 n = parse_unit(&str);
^1da177e4c3f41 Linus Torvalds 2005-04-16 267 if(n < 0){
f28169d2000177 Jeff Dike 2007-02-10 268 *error_out = "Couldn't parse device number";
f28169d2000177 Jeff Dike 2007-02-10 269 return -EINVAL;
^1da177e4c3f41 Linus Torvalds 2005-04-16 270 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 271 if(n >= MAX_DEV){
f28169d2000177 Jeff Dike 2007-02-10 272 *error_out = "Device number out of range";
f28169d2000177 Jeff Dike 2007-02-10 273 return 1;
^1da177e4c3f41 Linus Torvalds 2005-04-16 274 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 275
f28169d2000177 Jeff Dike 2007-02-10 276 err = -EBUSY;
d7fb2c3865ca0f Paolo 'Blaisorblade' Giarrusso 2006-10-30 277 mutex_lock(&ubd_lock);
^1da177e4c3f41 Linus Torvalds 2005-04-16 278
7d314e346d6081 Paolo 'Blaisorblade' Giarrusso 2006-10-30 279 ubd_dev = &ubd_devs[n];
7d314e346d6081 Paolo 'Blaisorblade' Giarrusso 2006-10-30 280 if(ubd_dev->file != NULL){
f28169d2000177 Jeff Dike 2007-02-10 281 *error_out = "Device is already configured";
^1da177e4c3f41 Linus Torvalds 2005-04-16 282 goto out;
^1da177e4c3f41 Linus Torvalds 2005-04-16 283 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 284
^1da177e4c3f41 Linus Torvalds 2005-04-16 285 if (index_out)
^1da177e4c3f41 Linus Torvalds 2005-04-16 286 *index_out = n;
^1da177e4c3f41 Linus Torvalds 2005-04-16 287
f28169d2000177 Jeff Dike 2007-02-10 288 err = -EINVAL;
50109b5a03b402 Anton Ivanov 2018-11-14 289 for (i = 0; i < sizeof("rscdt="); i++) {
^1da177e4c3f41 Linus Torvalds 2005-04-16 290 switch (*str) {
^1da177e4c3f41 Linus Torvalds 2005-04-16 291 case 'r':
^1da177e4c3f41 Linus Torvalds 2005-04-16 292 flags.w = 0;
^1da177e4c3f41 Linus Torvalds 2005-04-16 293 break;
^1da177e4c3f41 Linus Torvalds 2005-04-16 294 case 's':
^1da177e4c3f41 Linus Torvalds 2005-04-16 295 flags.s = 1;
^1da177e4c3f41 Linus Torvalds 2005-04-16 296 break;
^1da177e4c3f41 Linus Torvalds 2005-04-16 297 case 'd':
7d314e346d6081 Paolo 'Blaisorblade' Giarrusso 2006-10-30 298 ubd_dev->no_cow = 1;
^1da177e4c3f41 Linus Torvalds 2005-04-16 299 break;
6c29256c5703ad Jeff Dike 2006-03-27 300 case 'c':
7d314e346d6081 Paolo 'Blaisorblade' Giarrusso 2006-10-30 301 ubd_dev->shared = 1;
6c29256c5703ad Jeff Dike 2006-03-27 302 break;
50109b5a03b402 Anton Ivanov 2018-11-14 303 case 't':
50109b5a03b402 Anton Ivanov 2018-11-14 304 ubd_dev->no_trim = 1;
50109b5a03b402 Anton Ivanov 2018-11-14 305 break;
^1da177e4c3f41 Linus Torvalds 2005-04-16 306 case '=':
^1da177e4c3f41 Linus Torvalds 2005-04-16 307 str++;
^1da177e4c3f41 Linus Torvalds 2005-04-16 308 goto break_loop;
^1da177e4c3f41 Linus Torvalds 2005-04-16 309 default:
f28169d2000177 Jeff Dike 2007-02-10 310 *error_out = "Expected '=' or flag letter "
50109b5a03b402 Anton Ivanov 2018-11-14 311 "(r, s, c, t or d)";
^1da177e4c3f41 Linus Torvalds 2005-04-16 312 goto out;
^1da177e4c3f41 Linus Torvalds 2005-04-16 313 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 314 str++;
^1da177e4c3f41 Linus Torvalds 2005-04-16 315 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 316
^1da177e4c3f41 Linus Torvalds 2005-04-16 317 if (*str == '=')
f28169d2000177 Jeff Dike 2007-02-10 318 *error_out = "Too many flags specified";
^1da177e4c3f41 Linus Torvalds 2005-04-16 319 else
f28169d2000177 Jeff Dike 2007-02-10 320 *error_out = "Missing '='";
^1da177e4c3f41 Linus Torvalds 2005-04-16 321 goto out;
^1da177e4c3f41 Linus Torvalds 2005-04-16 322
^1da177e4c3f41 Linus Torvalds 2005-04-16 323 break_loop:
ef3ba87cb7c911 Christopher Obbard 2020-11-23 324 file = strsep(&str, ",:");
ef3ba87cb7c911 Christopher Obbard 2020-11-23 325 if (*file == '\0')
ef3ba87cb7c911 Christopher Obbard 2020-11-23 326 file = NULL;
ef3ba87cb7c911 Christopher Obbard 2020-11-23 327
ef3ba87cb7c911 Christopher Obbard 2020-11-23 328 backing_file = strsep(&str, ",:");
94c41b3a7c370b Hajime Tazaki 2020-12-21 329 if (backing_file && *backing_file == '\0')
ef3ba87cb7c911 Christopher Obbard 2020-11-23 330 backing_file = NULL;
^1da177e4c3f41 Linus Torvalds 2005-04-16 331
ef3ba87cb7c911 Christopher Obbard 2020-11-23 332 serial = strsep(&str, ",:");
94c41b3a7c370b Hajime Tazaki 2020-12-21 333 if (serial && *serial == '\0')
ef3ba87cb7c911 Christopher Obbard 2020-11-23 334 serial = NULL;
^1da177e4c3f41 Linus Torvalds 2005-04-16 335
ef3ba87cb7c911 Christopher Obbard 2020-11-23 336 if (backing_file && ubd_dev->no_cow) {
f28169d2000177 Jeff Dike 2007-02-10 337 *error_out = "Can't specify both 'd' and a cow file";
f28169d2000177 Jeff Dike 2007-02-10 338 goto out;
f28169d2000177 Jeff Dike 2007-02-10 339 }
ef3ba87cb7c911 Christopher Obbard 2020-11-23 340
f28169d2000177 Jeff Dike 2007-02-10 341 err = 0;
ef3ba87cb7c911 Christopher Obbard 2020-11-23 342 ubd_dev->file = file;
7d314e346d6081 Paolo 'Blaisorblade' Giarrusso 2006-10-30 343 ubd_dev->cow.file = backing_file;
ef3ba87cb7c911 Christopher Obbard 2020-11-23 344 ubd_dev->serial = serial;
7d314e346d6081 Paolo 'Blaisorblade' Giarrusso 2006-10-30 345 ubd_dev->boot_openflags = flags;
^1da177e4c3f41 Linus Torvalds 2005-04-16 346 out:
d7fb2c3865ca0f Paolo 'Blaisorblade' Giarrusso 2006-10-30 347 mutex_unlock(&ubd_lock);
f28169d2000177 Jeff Dike 2007-02-10 348 return err;
^1da177e4c3f41 Linus Torvalds 2005-04-16 349 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 350
:::::: The code at line 247 was first introduced by commit
:::::: 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 Linux-2.6.12-rc2
:::::: TO: Linus Torvalds <torvalds(a)ppc970.osdl.org>
:::::: CC: Linus Torvalds <torvalds(a)ppc970.osdl.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
11 months, 4 weeks
[asm-generic:clkdev 1/5] arch/mips/ar7/clock.c:428:28: error: array type has incomplete element type 'struct clkdev_table'
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/arnd/asm-generic.git clkdev
head: 3c150d2578b204c2e18c7ac6d967b43a614befbe
commit: 991cb086b5cfcd6ad953a64c6ce384bbd8595ded [1/5] mips: ar7: convert to clkdev_lookup
config: mips-ar7_defconfig (attached as .config)
compiler: mipsel-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://git.kernel.org/pub/scm/linux/kernel/git/arnd/asm-generic.git/comm...
git remote add asm-generic https://git.kernel.org/pub/scm/linux/kernel/git/arnd/asm-generic.git
git fetch --no-tags asm-generic clkdev
git checkout 991cb086b5cfcd6ad953a64c6ce384bbd8595ded
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=mips
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
>> arch/mips/ar7/clock.c:428:28: error: array type has incomplete element type 'struct clkdev_table'
428 | static struct clkdev_table clkdev_table[] = {
| ^~~~~~~~~~~~
In file included from arch/mips/ar7/clock.c:17:
>> include/linux/clkdev.h:28:3: error: field name not in record or union initializer
28 | .dev_id = d, \
| ^
arch/mips/ar7/clock.c:429:2: note: in expansion of macro 'CLKDEV_INIT'
429 | CLKDEV_INIT(NULL, "bus", &bus_clk),
| ^~~~~~~~~~~
include/linux/clkdev.h:28:3: note: (near initialization for 'clkdev_table')
28 | .dev_id = d, \
| ^
arch/mips/ar7/clock.c:429:2: note: in expansion of macro 'CLKDEV_INIT'
429 | CLKDEV_INIT(NULL, "bus", &bus_clk),
| ^~~~~~~~~~~
include/linux/clkdev.h:29:3: error: field name not in record or union initializer
29 | .con_id = n, \
| ^
arch/mips/ar7/clock.c:429:2: note: in expansion of macro 'CLKDEV_INIT'
429 | CLKDEV_INIT(NULL, "bus", &bus_clk),
| ^~~~~~~~~~~
include/linux/clkdev.h:29:3: note: (near initialization for 'clkdev_table')
29 | .con_id = n, \
| ^
arch/mips/ar7/clock.c:429:2: note: in expansion of macro 'CLKDEV_INIT'
429 | CLKDEV_INIT(NULL, "bus", &bus_clk),
| ^~~~~~~~~~~
include/linux/clkdev.h:30:3: error: field name not in record or union initializer
30 | .clk = c, \
| ^
arch/mips/ar7/clock.c:429:2: note: in expansion of macro 'CLKDEV_INIT'
429 | CLKDEV_INIT(NULL, "bus", &bus_clk),
| ^~~~~~~~~~~
include/linux/clkdev.h:30:3: note: (near initialization for 'clkdev_table')
30 | .clk = c, \
| ^
arch/mips/ar7/clock.c:429:2: note: in expansion of macro 'CLKDEV_INIT'
429 | CLKDEV_INIT(NULL, "bus", &bus_clk),
| ^~~~~~~~~~~
>> include/linux/clkdev.h:28:3: error: field name not in record or union initializer
28 | .dev_id = d, \
| ^
arch/mips/ar7/clock.c:431:2: note: in expansion of macro 'CLKDEV_INIT'
431 | CLKDEV_INIT("cpmac.0", "cpmac", &vbus_clk),
| ^~~~~~~~~~~
include/linux/clkdev.h:28:3: note: (near initialization for 'clkdev_table')
28 | .dev_id = d, \
| ^
arch/mips/ar7/clock.c:431:2: note: in expansion of macro 'CLKDEV_INIT'
431 | CLKDEV_INIT("cpmac.0", "cpmac", &vbus_clk),
| ^~~~~~~~~~~
include/linux/clkdev.h:29:3: error: field name not in record or union initializer
29 | .con_id = n, \
| ^
arch/mips/ar7/clock.c:431:2: note: in expansion of macro 'CLKDEV_INIT'
431 | CLKDEV_INIT("cpmac.0", "cpmac", &vbus_clk),
| ^~~~~~~~~~~
include/linux/clkdev.h:29:3: note: (near initialization for 'clkdev_table')
29 | .con_id = n, \
| ^
arch/mips/ar7/clock.c:431:2: note: in expansion of macro 'CLKDEV_INIT'
431 | CLKDEV_INIT("cpmac.0", "cpmac", &vbus_clk),
| ^~~~~~~~~~~
include/linux/clkdev.h:30:3: error: field name not in record or union initializer
30 | .clk = c, \
| ^
arch/mips/ar7/clock.c:431:2: note: in expansion of macro 'CLKDEV_INIT'
431 | CLKDEV_INIT("cpmac.0", "cpmac", &vbus_clk),
| ^~~~~~~~~~~
include/linux/clkdev.h:30:3: note: (near initialization for 'clkdev_table')
30 | .clk = c, \
| ^
arch/mips/ar7/clock.c:431:2: note: in expansion of macro 'CLKDEV_INIT'
431 | CLKDEV_INIT("cpmac.0", "cpmac", &vbus_clk),
| ^~~~~~~~~~~
>> include/linux/clkdev.h:28:3: error: field name not in record or union initializer
28 | .dev_id = d, \
| ^
arch/mips/ar7/clock.c:432:2: note: in expansion of macro 'CLKDEV_INIT'
432 | CLKDEV_INIT("cpmac.1", "cpmac", &vbus_clk),
| ^~~~~~~~~~~
include/linux/clkdev.h:28:3: note: (near initialization for 'clkdev_table')
28 | .dev_id = d, \
| ^
arch/mips/ar7/clock.c:432:2: note: in expansion of macro 'CLKDEV_INIT'
432 | CLKDEV_INIT("cpmac.1", "cpmac", &vbus_clk),
| ^~~~~~~~~~~
include/linux/clkdev.h:29:3: error: field name not in record or union initializer
29 | .con_id = n, \
| ^
arch/mips/ar7/clock.c:432:2: note: in expansion of macro 'CLKDEV_INIT'
432 | CLKDEV_INIT("cpmac.1", "cpmac", &vbus_clk),
| ^~~~~~~~~~~
include/linux/clkdev.h:29:3: note: (near initialization for 'clkdev_table')
29 | .con_id = n, \
| ^
arch/mips/ar7/clock.c:432:2: note: in expansion of macro 'CLKDEV_INIT'
432 | CLKDEV_INIT("cpmac.1", "cpmac", &vbus_clk),
| ^~~~~~~~~~~
include/linux/clkdev.h:30:3: error: field name not in record or union initializer
30 | .clk = c, \
| ^
arch/mips/ar7/clock.c:432:2: note: in expansion of macro 'CLKDEV_INIT'
432 | CLKDEV_INIT("cpmac.1", "cpmac", &vbus_clk),
| ^~~~~~~~~~~
include/linux/clkdev.h:30:3: note: (near initialization for 'clkdev_table')
30 | .clk = c, \
| ^
arch/mips/ar7/clock.c:432:2: note: in expansion of macro 'CLKDEV_INIT'
432 | CLKDEV_INIT("cpmac.1", "cpmac", &vbus_clk),
| ^~~~~~~~~~~
>> include/linux/clkdev.h:28:3: error: field name not in record or union initializer
28 | .dev_id = d, \
| ^
arch/mips/ar7/clock.c:433:2: note: in expansion of macro 'CLKDEV_INIT'
433 | CLKDEV_INIT(NULL, "cpu", &cpu_clk)
| ^~~~~~~~~~~
include/linux/clkdev.h:28:3: note: (near initialization for 'clkdev_table')
28 | .dev_id = d, \
| ^
arch/mips/ar7/clock.c:433:2: note: in expansion of macro 'CLKDEV_INIT'
433 | CLKDEV_INIT(NULL, "cpu", &cpu_clk)
| ^~~~~~~~~~~
include/linux/clkdev.h:29:3: error: field name not in record or union initializer
29 | .con_id = n, \
| ^
arch/mips/ar7/clock.c:433:2: note: in expansion of macro 'CLKDEV_INIT'
433 | CLKDEV_INIT(NULL, "cpu", &cpu_clk)
| ^~~~~~~~~~~
include/linux/clkdev.h:29:3: note: (near initialization for 'clkdev_table')
29 | .con_id = n, \
| ^
arch/mips/ar7/clock.c:433:2: note: in expansion of macro 'CLKDEV_INIT'
433 | CLKDEV_INIT(NULL, "cpu", &cpu_clk)
| ^~~~~~~~~~~
include/linux/clkdev.h:30:3: error: field name not in record or union initializer
30 | .clk = c, \
| ^
arch/mips/ar7/clock.c:433:2: note: in expansion of macro 'CLKDEV_INIT'
433 | CLKDEV_INIT(NULL, "cpu", &cpu_clk)
| ^~~~~~~~~~~
include/linux/clkdev.h:30:3: note: (near initialization for 'clkdev_table')
30 | .clk = c, \
| ^
arch/mips/ar7/clock.c:433:2: note: in expansion of macro 'CLKDEV_INIT'
433 | CLKDEV_INIT(NULL, "cpu", &cpu_clk)
| ^~~~~~~~~~~
>> include/linux/clkdev.h:27:2: error: expected '}' before '{' token
27 | { \
| ^
arch/mips/ar7/clock.c:434:2: note: in expansion of macro 'CLKDEV_INIT'
434 | CLKDEV_INIT(NULL, "dsp", &dsp_clk),
| ^~~~~~~~~~~
arch/mips/ar7/clock.c:428:45: note: to match this '{'
428 | static struct clkdev_table clkdev_table[] = {
| ^
In file included from include/linux/bits.h:22,
from include/linux/bitops.h:6,
from include/linux/kernel.h:12,
from arch/mips/ar7/clock.c:8:
arch/mips/ar7/clock.c: In function 'ar7_init_clocks':
>> include/linux/build_bug.h:16:51: error: bit-field '<anonymous>' width not an integer constant
16 | #define BUILD_BUG_ON_ZERO(e) ((int)(sizeof(struct { int:(-!!(e)); })))
| ^
include/linux/compiler.h:240:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
240 | #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
| ^~~~~~~~~~~~~~~~~
include/linux/kernel.h:42:59: note: in expansion of macro '__must_be_array'
42 | #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
| ^~~~~~~~~~~~~~~
arch/mips/ar7/clock.c:455:34: note: in expansion of macro 'ARRAY_SIZE'
455 | clkdev_add_table(&clkdev_table, ARRAY_SIZE(clkdev_table);
| ^~~~~~~~~~
>> arch/mips/ar7/clock.c:455:58: error: expected ')' before ';' token
455 | clkdev_add_table(&clkdev_table, ARRAY_SIZE(clkdev_table);
| ^
>> arch/mips/ar7/clock.c:455:59: error: expected ';' before '}' token
455 | clkdev_add_table(&clkdev_table, ARRAY_SIZE(clkdev_table);
| ^
| ;
456 | }
| ~
At top level:
arch/mips/ar7/clock.c:428:28: warning: 'clkdev_table' defined but not used [-Wunused-variable]
428 | static struct clkdev_table clkdev_table[] = {
| ^~~~~~~~~~~~
vim +428 arch/mips/ar7/clock.c
427
> 428 static struct clkdev_table clkdev_table[] = {
429 CLKDEV_INIT(NULL, "bus", &bus_clk),
430 /* cpmac and vbus share the same rate */
431 CLKDEV_INIT("cpmac.0", "cpmac", &vbus_clk),
432 CLKDEV_INIT("cpmac.1", "cpmac", &vbus_clk),
433 CLKDEV_INIT(NULL, "cpu", &cpu_clk)
434 CLKDEV_INIT(NULL, "dsp", &dsp_clk),
435 CLKDEV_INIT(NULL, "vbus", &vbus_clk),
436 };
437
438 void __init ar7_init_clocks(void)
439 {
440 switch (ar7_chip_id()) {
441 case AR7_CHIP_7100:
442 case AR7_CHIP_7200:
443 tnetd7200_init_clocks();
444 break;
445 case AR7_CHIP_7300:
446 dsp_clk.rate = tnetd7300_dsp_clock();
447 tnetd7300_init_clocks();
448 break;
449 default:
450 break;
451 }
452 /* adjust vbus clock rate */
453 vbus_clk.rate = bus_clk.rate / 2;
454
> 455 clkdev_add_table(&clkdev_table, ARRAY_SIZE(clkdev_table);
456 }
457
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
11 months, 4 weeks
Re: [PATCH net-next 8/8] nfp: flower-ct: add tc merge functionality
by kernel test robot
Hi Simon,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on net-next/master]
url: https://github.com/0day-ci/linux/commits/Simon-Horman/Introduce-conntrack...
base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git af9207adb6d9986be6ed64e76705cf513087e724
config: arm64-randconfig-r022-20210531 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project bc6799f2f79f0ae87e9f1ebf9d25ba799fbd25a9)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install arm64 cross compiling tool for clang build
# apt-get install binutils-aarch64-linux-gnu
# https://github.com/0day-ci/linux/commit/ddeb710e846c772a8cb45e071c15e37bd...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Simon-Horman/Introduce-conntrack-offloading-to-the-nfp-driver/20210528-224456
git checkout ddeb710e846c772a8cb45e071c15e37bdbd2000c
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> drivers/net/ethernet/netronome/nfp/flower/conntrack.c:364:11: warning: variable 'ct_list' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]
else if (ct_entry1->type == CT_TYPE_POST_CT)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/ethernet/netronome/nfp/flower/conntrack.c:367:46: note: uninitialized use occurs here
list_for_each_entry_safe(ct_entry2, ct_tmp, ct_list,
^~~~~~~
include/linux/list.h:715:30: note: expanded from macro 'list_for_each_entry_safe'
for (pos = list_first_entry(head, typeof(*pos), member), \
^~~~
include/linux/list.h:522:14: note: expanded from macro 'list_first_entry'
list_entry((ptr)->next, type, member)
^~~
include/linux/list.h:511:15: note: expanded from macro 'list_entry'
container_of(ptr, type, member)
^~~
include/linux/kernel.h:703:26: note: expanded from macro 'container_of'
void *__mptr = (void *)(ptr); \
^~~
drivers/net/ethernet/netronome/nfp/flower/conntrack.c:364:7: note: remove the 'if' if its condition is always true
else if (ct_entry1->type == CT_TYPE_POST_CT)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/ethernet/netronome/nfp/flower/conntrack.c:360:27: note: initialize the variable 'ct_list' to silence this warning
struct list_head *ct_list;
^
= NULL
1 warning generated.
vim +364 drivers/net/ethernet/netronome/nfp/flower/conntrack.c
353
354 static void
355 nfp_ct_merge_tc_entries(struct nfp_fl_ct_flow_entry *ct_entry1,
356 struct nfp_fl_ct_zone_entry *zt_src,
357 struct nfp_fl_ct_zone_entry *zt_dst)
358 {
359 struct nfp_fl_ct_flow_entry *ct_entry2, *ct_tmp;
360 struct list_head *ct_list;
361
362 if (ct_entry1->type == CT_TYPE_PRE_CT)
363 ct_list = &zt_src->post_ct_list;
> 364 else if (ct_entry1->type == CT_TYPE_POST_CT)
365 ct_list = &zt_src->pre_ct_list;
366
367 list_for_each_entry_safe(ct_entry2, ct_tmp, ct_list,
368 list_node) {
369 nfp_ct_do_tc_merge(zt_dst, ct_entry2, ct_entry1);
370 }
371 }
372
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
11 months, 4 weeks
[linux-stable-rc:linux-4.4.y 906/2041] arch/powerpc/kernel/setup_64.c:922:6: error: no previous prototype for 'uaccess_flush_enable'
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-4.4.y
head: c26ec7873e268a06dcb98b71ba8a2db18cf7a25c
commit: f579da2a8c318e5fd1bef6ad5c300386eff9fe7d [906/2041] powerpc/64s: flush L1D after user accesses
config: powerpc64-randconfig-r011-20210530 (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 7.5.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gi...
git remote add linux-stable-rc https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
git fetch --no-tags linux-stable-rc linux-4.4.y
git checkout f579da2a8c318e5fd1bef6ad5c300386eff9fe7d
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-7.5.0 make.cross ARCH=powerpc64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
Note: the linux-stable-rc/linux-4.4.y HEAD c26ec7873e268a06dcb98b71ba8a2db18cf7a25c builds fine.
It only hurts bisectibility.
All errors (new ones prefixed by >>):
In file included from include/linux/hugetlb.h:432:0,
from arch/powerpc/kernel/setup_64.c:38:
arch/powerpc/include/asm/hugetlb.h: In function 'huge_ptep_clear_flush':
arch/powerpc/include/asm/hugetlb.h:134:8: error: variable 'pte' set but not used [-Werror=unused-but-set-variable]
pte_t pte;
^~~
In file included from arch/powerpc/include/uapi/asm/kvm_para.h:81:0,
from arch/powerpc/include/asm/kvm_para.h:22,
from include/uapi/linux/kvm_para.h:30,
from include/linux/kvm_para.h:4,
from include/linux/kvm_host.h:31,
from arch/powerpc/include/asm/kvm_ppc.h:30,
from arch/powerpc/kernel/setup_64.c:70:
arch/powerpc/include/asm/epapr_hcalls.h: In function 'epapr_hypercall':
arch/powerpc/include/asm/epapr_hcalls.h:469:2: error: 'register' is not at beginning of declaration [-Werror=old-style-declaration]
unsigned long register r0 asm("r0");
^~~~~~~~
arch/powerpc/include/asm/epapr_hcalls.h:470:2: error: 'register' is not at beginning of declaration [-Werror=old-style-declaration]
unsigned long register r3 asm("r3") = in[0];
^~~~~~~~
arch/powerpc/include/asm/epapr_hcalls.h:471:2: error: 'register' is not at beginning of declaration [-Werror=old-style-declaration]
unsigned long register r4 asm("r4") = in[1];
^~~~~~~~
arch/powerpc/include/asm/epapr_hcalls.h:472:2: error: 'register' is not at beginning of declaration [-Werror=old-style-declaration]
unsigned long register r5 asm("r5") = in[2];
^~~~~~~~
arch/powerpc/include/asm/epapr_hcalls.h:473:2: error: 'register' is not at beginning of declaration [-Werror=old-style-declaration]
unsigned long register r6 asm("r6") = in[3];
^~~~~~~~
arch/powerpc/include/asm/epapr_hcalls.h:474:2: error: 'register' is not at beginning of declaration [-Werror=old-style-declaration]
unsigned long register r7 asm("r7") = in[4];
^~~~~~~~
arch/powerpc/include/asm/epapr_hcalls.h:475:2: error: 'register' is not at beginning of declaration [-Werror=old-style-declaration]
unsigned long register r8 asm("r8") = in[5];
^~~~~~~~
arch/powerpc/include/asm/epapr_hcalls.h:476:2: error: 'register' is not at beginning of declaration [-Werror=old-style-declaration]
unsigned long register r9 asm("r9") = in[6];
^~~~~~~~
arch/powerpc/include/asm/epapr_hcalls.h:477:2: error: 'register' is not at beginning of declaration [-Werror=old-style-declaration]
unsigned long register r10 asm("r10") = in[7];
^~~~~~~~
arch/powerpc/include/asm/epapr_hcalls.h:478:2: error: 'register' is not at beginning of declaration [-Werror=old-style-declaration]
unsigned long register r11 asm("r11") = nr;
^~~~~~~~
arch/powerpc/include/asm/epapr_hcalls.h:479:2: error: 'register' is not at beginning of declaration [-Werror=old-style-declaration]
unsigned long register r12 asm("r12");
^~~~~~~~
In file included from arch/powerpc/kernel/setup_64.c:70:0:
arch/powerpc/include/asm/kvm_ppc.h: In function 'kvmppc_get_last_inst':
arch/powerpc/include/asm/kvm_ppc.h:287:27: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
if (vcpu->arch.last_inst == KVM_INST_FETCH_FAILED)
^~
arch/powerpc/kernel/setup_64.c: At top level:
arch/powerpc/kernel/setup_64.c:254:13: error: no previous prototype for 'early_setup' [-Werror=missing-prototypes]
void __init early_setup(unsigned long dt_ptr)
^~~~~~~~~~~
arch/powerpc/kernel/setup_64.c:334:6: error: no previous prototype for 'early_setup_secondary' [-Werror=missing-prototypes]
void early_setup_secondary(void)
^~~~~~~~~~~~~~~~~~~~~
arch/powerpc/kernel/setup_64.c:479:13: error: no previous prototype for 'setup_system' [-Werror=missing-prototypes]
void __init setup_system(void)
^~~~~~~~~~~~
In file included from include/linux/rcupdate.h:40:0,
from include/linux/rbtree.h:34,
from include/linux/sched.h:22,
from arch/powerpc/kernel/setup_64.c:17:
arch/powerpc/kernel/setup_64.c: In function 'irqstack_early_init':
include/linux/cpumask.h:223:9: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
(cpu) < nr_cpu_ids;)
^
include/linux/cpumask.h:743:36: note: in expansion of macro 'for_each_cpu'
#define for_each_possible_cpu(cpu) for_each_cpu((cpu), cpu_possible_mask)
^~~~~~~~~~~~
arch/powerpc/kernel/setup_64.c:624:2: note: in expansion of macro 'for_each_possible_cpu'
for_each_possible_cpu(i) {
^~~~~~~~~~~~~~~~~~~~~
arch/powerpc/kernel/setup_64.c: In function 'emergency_stack_init':
include/linux/cpumask.h:223:9: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
(cpu) < nr_cpu_ids;)
^
include/linux/cpumask.h:743:36: note: in expansion of macro 'for_each_cpu'
#define for_each_possible_cpu(cpu) for_each_cpu((cpu), cpu_possible_mask)
^~~~~~~~~~~~
arch/powerpc/kernel/setup_64.c:682:2: note: in expansion of macro 'for_each_possible_cpu'
for_each_possible_cpu(i) {
^~~~~~~~~~~~~~~~~~~~~
arch/powerpc/kernel/setup_64.c: In function 'setup_per_cpu_areas':
include/linux/cpumask.h:223:9: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
(cpu) < nr_cpu_ids;)
^
include/linux/cpumask.h:743:36: note: in expansion of macro 'for_each_cpu'
#define for_each_possible_cpu(cpu) for_each_cpu((cpu), cpu_possible_mask)
^~~~~~~~~~~~
arch/powerpc/kernel/setup_64.c:802:2: note: in expansion of macro 'for_each_possible_cpu'
for_each_possible_cpu(cpu) {
^~~~~~~~~~~~~~~~~~~~~
arch/powerpc/kernel/setup_64.c: At top level:
arch/powerpc/kernel/setup_64.c:910:6: error: no previous prototype for 'entry_flush_enable' [-Werror=missing-prototypes]
void entry_flush_enable(bool enable)
^~~~~~~~~~~~~~~~~~
>> arch/powerpc/kernel/setup_64.c:922:6: error: no previous prototype for 'uaccess_flush_enable' [-Werror=missing-prototypes]
void uaccess_flush_enable(bool enable)
^~~~~~~~~~~~~~~~~~~~
arch/powerpc/kernel/setup_64.c: In function 'rfi_flush_debugfs_init':
arch/powerpc/kernel/setup_64.c:1084:41: error: 'powerpc_debugfs_root' undeclared (first use in this function); did you mean 'arch_debugfs_dir'?
debugfs_create_file("rfi_flush", 0600, powerpc_debugfs_root, NULL, &fops_rfi_flush);
^~~~~~~~~~~~~~~~~~~~
arch_debugfs_dir
arch/powerpc/kernel/setup_64.c:1084:41: note: each undeclared identifier is reported only once for each function it appears in
cc1: all warnings being treated as errors
vim +/uaccess_flush_enable +922 arch/powerpc/kernel/setup_64.c
921
> 922 void uaccess_flush_enable(bool enable)
923 {
924 if (enable) {
925 do_uaccess_flush_fixups(enabled_flush_types);
926 if (static_key_initialized)
927 static_branch_enable(&uaccess_flush_key);
928 else
929 printk(KERN_DEBUG "uaccess-flush: deferring static key until after static key initialization\n");
930 on_each_cpu(do_nothing, NULL, 1);
931 } else {
932 static_branch_disable(&uaccess_flush_key);
933 do_uaccess_flush_fixups(L1D_FLUSH_NONE);
934 }
935
936 uaccess_flush = enable;
937 }
938
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
11 months, 4 weeks
Re: [PATCHv3 for-next 05/20] RDMA/rtrs: Change MAX_SESS_QUEUE_DEPTH
by kernel test robot
Hi Jack,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on rdma/for-next]
[also build test WARNING on v5.13-rc3 next-20210528]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Jack-Wang/RTRS-update-for-5-14/2...
base: https://git.kernel.org/pub/scm/linux/kernel/git/rdma/rdma.git for-next
config: x86_64-randconfig-a012-20210526 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 6505c630407c5feec818f0bb1c284f9eeebf2071)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# https://github.com/0day-ci/linux/commit/66f95f659060028d1f0f91473ad1c16a6...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Jack-Wang/RTRS-update-for-5-14/20210528-193313
git checkout 66f95f659060028d1f0f91473ad1c16a6595fcac
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> drivers/infiniband/ulp/rtrs/rtrs-clt.c:1786:19: warning: result of comparison of constant 'MAX_SESS_QUEUE_DEPTH' (65536) with expression of type 'u16' (aka 'unsigned short') is always false [-Wtautological-constant-out-of-range-compare]
if (queue_depth > MAX_SESS_QUEUE_DEPTH) {
~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:56:47: note: expanded from macro 'if'
#define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
^~~~
include/linux/compiler.h:58:86: note: expanded from macro '__trace_if_var'
#define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) : __trace_if_value(cond))
^~~~
include/linux/compiler.h:69:3: note: expanded from macro '__trace_if_value'
(cond) ? \
^~~~
>> drivers/infiniband/ulp/rtrs/rtrs-clt.c:1786:19: warning: result of comparison of constant 'MAX_SESS_QUEUE_DEPTH' (65536) with expression of type 'u16' (aka 'unsigned short') is always false [-Wtautological-constant-out-of-range-compare]
if (queue_depth > MAX_SESS_QUEUE_DEPTH) {
~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:56:47: note: expanded from macro 'if'
#define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
^~~~
include/linux/compiler.h:58:52: note: expanded from macro '__trace_if_var'
#define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) : __trace_if_value(cond))
^~~~
>> drivers/infiniband/ulp/rtrs/rtrs-clt.c:1786:19: warning: result of comparison of constant 'MAX_SESS_QUEUE_DEPTH' (65536) with expression of type 'u16' (aka 'unsigned short') is always false [-Wtautological-constant-out-of-range-compare]
if (queue_depth > MAX_SESS_QUEUE_DEPTH) {
~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:56:47: note: expanded from macro 'if'
#define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
^~~~
include/linux/compiler.h:58:61: note: expanded from macro '__trace_if_var'
#define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) : __trace_if_value(cond))
^~~~
3 warnings generated.
vim +1786 drivers/infiniband/ulp/rtrs/rtrs-clt.c
6a98d71daea186 Jack Wang 2020-05-11 1750
6a98d71daea186 Jack Wang 2020-05-11 1751 static int rtrs_rdma_conn_established(struct rtrs_clt_con *con,
6a98d71daea186 Jack Wang 2020-05-11 1752 struct rdma_cm_event *ev)
6a98d71daea186 Jack Wang 2020-05-11 1753 {
6a98d71daea186 Jack Wang 2020-05-11 1754 struct rtrs_clt_sess *sess = to_clt_sess(con->c.sess);
6a98d71daea186 Jack Wang 2020-05-11 1755 struct rtrs_clt *clt = sess->clt;
6a98d71daea186 Jack Wang 2020-05-11 1756 const struct rtrs_msg_conn_rsp *msg;
6a98d71daea186 Jack Wang 2020-05-11 1757 u16 version, queue_depth;
6a98d71daea186 Jack Wang 2020-05-11 1758 int errno;
6a98d71daea186 Jack Wang 2020-05-11 1759 u8 len;
6a98d71daea186 Jack Wang 2020-05-11 1760
6a98d71daea186 Jack Wang 2020-05-11 1761 msg = ev->param.conn.private_data;
6a98d71daea186 Jack Wang 2020-05-11 1762 len = ev->param.conn.private_data_len;
6a98d71daea186 Jack Wang 2020-05-11 1763 if (len < sizeof(*msg)) {
6a98d71daea186 Jack Wang 2020-05-11 1764 rtrs_err(clt, "Invalid RTRS connection response\n");
6a98d71daea186 Jack Wang 2020-05-11 1765 return -ECONNRESET;
6a98d71daea186 Jack Wang 2020-05-11 1766 }
6a98d71daea186 Jack Wang 2020-05-11 1767 if (le16_to_cpu(msg->magic) != RTRS_MAGIC) {
6a98d71daea186 Jack Wang 2020-05-11 1768 rtrs_err(clt, "Invalid RTRS magic\n");
6a98d71daea186 Jack Wang 2020-05-11 1769 return -ECONNRESET;
6a98d71daea186 Jack Wang 2020-05-11 1770 }
6a98d71daea186 Jack Wang 2020-05-11 1771 version = le16_to_cpu(msg->version);
6a98d71daea186 Jack Wang 2020-05-11 1772 if (version >> 8 != RTRS_PROTO_VER_MAJOR) {
6a98d71daea186 Jack Wang 2020-05-11 1773 rtrs_err(clt, "Unsupported major RTRS version: %d, expected %d\n",
6a98d71daea186 Jack Wang 2020-05-11 1774 version >> 8, RTRS_PROTO_VER_MAJOR);
6a98d71daea186 Jack Wang 2020-05-11 1775 return -ECONNRESET;
6a98d71daea186 Jack Wang 2020-05-11 1776 }
6a98d71daea186 Jack Wang 2020-05-11 1777 errno = le16_to_cpu(msg->errno);
6a98d71daea186 Jack Wang 2020-05-11 1778 if (errno) {
6a98d71daea186 Jack Wang 2020-05-11 1779 rtrs_err(clt, "Invalid RTRS message: errno %d\n",
6a98d71daea186 Jack Wang 2020-05-11 1780 errno);
6a98d71daea186 Jack Wang 2020-05-11 1781 return -ECONNRESET;
6a98d71daea186 Jack Wang 2020-05-11 1782 }
6a98d71daea186 Jack Wang 2020-05-11 1783 if (con->c.cid == 0) {
6a98d71daea186 Jack Wang 2020-05-11 1784 queue_depth = le16_to_cpu(msg->queue_depth);
6a98d71daea186 Jack Wang 2020-05-11 1785
6a98d71daea186 Jack Wang 2020-05-11 @1786 if (queue_depth > MAX_SESS_QUEUE_DEPTH) {
6a98d71daea186 Jack Wang 2020-05-11 1787 rtrs_err(clt, "Invalid RTRS message: queue=%d\n",
6a98d71daea186 Jack Wang 2020-05-11 1788 queue_depth);
6a98d71daea186 Jack Wang 2020-05-11 1789 return -ECONNRESET;
6a98d71daea186 Jack Wang 2020-05-11 1790 }
6a98d71daea186 Jack Wang 2020-05-11 1791 if (!sess->rbufs || sess->queue_depth < queue_depth) {
6a98d71daea186 Jack Wang 2020-05-11 1792 kfree(sess->rbufs);
6a98d71daea186 Jack Wang 2020-05-11 1793 sess->rbufs = kcalloc(queue_depth, sizeof(*sess->rbufs),
6a98d71daea186 Jack Wang 2020-05-11 1794 GFP_KERNEL);
6a98d71daea186 Jack Wang 2020-05-11 1795 if (!sess->rbufs)
6a98d71daea186 Jack Wang 2020-05-11 1796 return -ENOMEM;
6a98d71daea186 Jack Wang 2020-05-11 1797 }
6a98d71daea186 Jack Wang 2020-05-11 1798 sess->queue_depth = queue_depth;
6a98d71daea186 Jack Wang 2020-05-11 1799 sess->max_hdr_size = le32_to_cpu(msg->max_hdr_size);
6a98d71daea186 Jack Wang 2020-05-11 1800 sess->max_io_size = le32_to_cpu(msg->max_io_size);
6a98d71daea186 Jack Wang 2020-05-11 1801 sess->flags = le32_to_cpu(msg->flags);
6a98d71daea186 Jack Wang 2020-05-11 1802 sess->chunk_size = sess->max_io_size + sess->max_hdr_size;
6a98d71daea186 Jack Wang 2020-05-11 1803
6a98d71daea186 Jack Wang 2020-05-11 1804 /*
6a98d71daea186 Jack Wang 2020-05-11 1805 * Global queue depth and IO size is always a minimum.
6a98d71daea186 Jack Wang 2020-05-11 1806 * If while a reconnection server sends us a value a bit
6a98d71daea186 Jack Wang 2020-05-11 1807 * higher - client does not care and uses cached minimum.
6a98d71daea186 Jack Wang 2020-05-11 1808 *
6a98d71daea186 Jack Wang 2020-05-11 1809 * Since we can have several sessions (paths) restablishing
6a98d71daea186 Jack Wang 2020-05-11 1810 * connections in parallel, use lock.
6a98d71daea186 Jack Wang 2020-05-11 1811 */
6a98d71daea186 Jack Wang 2020-05-11 1812 mutex_lock(&clt->paths_mutex);
6a98d71daea186 Jack Wang 2020-05-11 1813 clt->queue_depth = min_not_zero(sess->queue_depth,
6a98d71daea186 Jack Wang 2020-05-11 1814 clt->queue_depth);
6a98d71daea186 Jack Wang 2020-05-11 1815 clt->max_io_size = min_not_zero(sess->max_io_size,
6a98d71daea186 Jack Wang 2020-05-11 1816 clt->max_io_size);
6a98d71daea186 Jack Wang 2020-05-11 1817 mutex_unlock(&clt->paths_mutex);
6a98d71daea186 Jack Wang 2020-05-11 1818
6a98d71daea186 Jack Wang 2020-05-11 1819 /*
6a98d71daea186 Jack Wang 2020-05-11 1820 * Cache the hca_port and hca_name for sysfs
6a98d71daea186 Jack Wang 2020-05-11 1821 */
6a98d71daea186 Jack Wang 2020-05-11 1822 sess->hca_port = con->c.cm_id->port_num;
6a98d71daea186 Jack Wang 2020-05-11 1823 scnprintf(sess->hca_name, sizeof(sess->hca_name),
6a98d71daea186 Jack Wang 2020-05-11 1824 sess->s.dev->ib_dev->name);
6a98d71daea186 Jack Wang 2020-05-11 1825 sess->s.src_addr = con->c.cm_id->route.addr.src_addr;
03e9b33a0fd677 Md Haris Iqbal 2021-02-12 1826 /* set for_new_clt, to allow future reconnect on any path */
03e9b33a0fd677 Md Haris Iqbal 2021-02-12 1827 sess->for_new_clt = 1;
6a98d71daea186 Jack Wang 2020-05-11 1828 }
6a98d71daea186 Jack Wang 2020-05-11 1829
6a98d71daea186 Jack Wang 2020-05-11 1830 return 0;
6a98d71daea186 Jack Wang 2020-05-11 1831 }
6a98d71daea186 Jack Wang 2020-05-11 1832
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
11 months, 4 weeks
drivers/spi/spi-s3c64xx.c:1461:34: warning: unused variable 's3c64xx_spi_dt_match'
by kernel test robot
Hi Masahiro,
First bad commit (maybe != root cause):
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 8124c8a6b35386f73523d27eacb71b5364a68c4c
commit: ea29b20a828511de3348334e529a3d046a180416 init/Kconfig: make COMPILE_TEST depend on HAS_IOMEM
date: 3 months ago
config: s390-randconfig-r014-20210531 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project bc6799f2f79f0ae87e9f1ebf9d25ba799fbd25a9)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install s390 cross compiling tool for clang build
# apt-get install binutils-s390x-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout ea29b20a828511de3348334e529a3d046a180416
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=s390
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:119:21: note: expanded from macro '__swab32'
___constant_swab32(x) : \
^
include/uapi/linux/swab.h:19:12: note: expanded from macro '___constant_swab32'
(((__u32)(x) & (__u32)0x000000ffUL) << 24) | \
^
In file included from drivers/spi/spi-s3c64xx.c:11:
In file included from include/linux/dma-mapping.h:10:
In file included from include/linux/scatterlist.h:9:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:119:21: note: expanded from macro '__swab32'
___constant_swab32(x) : \
^
include/uapi/linux/swab.h:20:12: note: expanded from macro '___constant_swab32'
(((__u32)(x) & (__u32)0x0000ff00UL) << 8) | \
^
In file included from drivers/spi/spi-s3c64xx.c:11:
In file included from include/linux/dma-mapping.h:10:
In file included from include/linux/scatterlist.h:9:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:119:21: note: expanded from macro '__swab32'
___constant_swab32(x) : \
^
include/uapi/linux/swab.h:21:12: note: expanded from macro '___constant_swab32'
(((__u32)(x) & (__u32)0x00ff0000UL) >> 8) | \
^
In file included from drivers/spi/spi-s3c64xx.c:11:
In file included from include/linux/dma-mapping.h:10:
In file included from include/linux/scatterlist.h:9:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:119:21: note: expanded from macro '__swab32'
___constant_swab32(x) : \
^
include/uapi/linux/swab.h:22:12: note: expanded from macro '___constant_swab32'
(((__u32)(x) & (__u32)0xff000000UL) >> 24)))
^
In file included from drivers/spi/spi-s3c64xx.c:11:
In file included from include/linux/dma-mapping.h:10:
In file included from include/linux/scatterlist.h:9:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:120:12: note: expanded from macro '__swab32'
__fswab32(x))
^
In file included from drivers/spi/spi-s3c64xx.c:11:
In file included from include/linux/dma-mapping.h:10:
In file included from include/linux/scatterlist.h:9:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:501:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
__raw_writeb(value, PCI_IOBASE + addr);
~~~~~~~~~~ ^
include/asm-generic/io.h:511:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
__raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
~~~~~~~~~~ ^
include/asm-generic/io.h:521:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
__raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
~~~~~~~~~~ ^
include/asm-generic/io.h:609:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
readsb(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:617:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
readsw(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:625:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
readsl(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:634:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
writesb(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:643:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
writesw(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:652:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
writesl(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
>> drivers/spi/spi-s3c64xx.c:1461:34: warning: unused variable 's3c64xx_spi_dt_match' [-Wunused-const-variable]
static const struct of_device_id s3c64xx_spi_dt_match[] = {
^
21 warnings generated.
--
include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:119:21: note: expanded from macro '__swab32'
___constant_swab32(x) : \
^
include/uapi/linux/swab.h:20:12: note: expanded from macro '___constant_swab32'
(((__u32)(x) & (__u32)0x0000ff00UL) << 8) | \
^
In file included from drivers/gpu/drm/rockchip/rockchip_vop_reg.c:19:
In file included from drivers/gpu/drm/rockchip/rockchip_drm_drv.h:12:
In file included from include/drm/drm_fb_helper.h:35:
In file included from include/drm/drm_client.h:6:
In file included from include/linux/dma-buf-map.h:9:
In file included from include/linux/io.h:13:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:119:21: note: expanded from macro '__swab32'
___constant_swab32(x) : \
^
include/uapi/linux/swab.h:21:12: note: expanded from macro '___constant_swab32'
(((__u32)(x) & (__u32)0x00ff0000UL) >> 8) | \
^
In file included from drivers/gpu/drm/rockchip/rockchip_vop_reg.c:19:
In file included from drivers/gpu/drm/rockchip/rockchip_drm_drv.h:12:
In file included from include/drm/drm_fb_helper.h:35:
In file included from include/drm/drm_client.h:6:
In file included from include/linux/dma-buf-map.h:9:
In file included from include/linux/io.h:13:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:119:21: note: expanded from macro '__swab32'
___constant_swab32(x) : \
^
include/uapi/linux/swab.h:22:12: note: expanded from macro '___constant_swab32'
(((__u32)(x) & (__u32)0xff000000UL) >> 24)))
^
In file included from drivers/gpu/drm/rockchip/rockchip_vop_reg.c:19:
In file included from drivers/gpu/drm/rockchip/rockchip_drm_drv.h:12:
In file included from include/drm/drm_fb_helper.h:35:
In file included from include/drm/drm_client.h:6:
In file included from include/linux/dma-buf-map.h:9:
In file included from include/linux/io.h:13:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:120:12: note: expanded from macro '__swab32'
__fswab32(x))
^
In file included from drivers/gpu/drm/rockchip/rockchip_vop_reg.c:19:
In file included from drivers/gpu/drm/rockchip/rockchip_drm_drv.h:12:
In file included from include/drm/drm_fb_helper.h:35:
In file included from include/drm/drm_client.h:6:
In file included from include/linux/dma-buf-map.h:9:
In file included from include/linux/io.h:13:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:501:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
__raw_writeb(value, PCI_IOBASE + addr);
~~~~~~~~~~ ^
include/asm-generic/io.h:511:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
__raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
~~~~~~~~~~ ^
include/asm-generic/io.h:521:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
__raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
~~~~~~~~~~ ^
include/asm-generic/io.h:609:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
readsb(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:617:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
readsw(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:625:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
readsl(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:634:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
writesb(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:643:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
writesw(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:652:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
writesl(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
>> drivers/gpu/drm/rockchip/rockchip_vop_reg.c:1038:34: warning: unused variable 'vop_driver_dt_match' [-Wunused-const-variable]
static const struct of_device_id vop_driver_dt_match[] = {
^
21 warnings generated.
--
include/uapi/linux/swab.h:19:12: note: expanded from macro '___constant_swab32'
(((__u32)(x) & (__u32)0x000000ffUL) << 24) | \
^
In file included from drivers/gpu/drm/rockchip/analogix_dp-rockchip.c:15:
In file included from include/linux/regmap.h:20:
In file included from include/linux/iopoll.h:14:
In file included from include/linux/io.h:13:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:119:21: note: expanded from macro '__swab32'
___constant_swab32(x) : \
^
include/uapi/linux/swab.h:20:12: note: expanded from macro '___constant_swab32'
(((__u32)(x) & (__u32)0x0000ff00UL) << 8) | \
^
In file included from drivers/gpu/drm/rockchip/analogix_dp-rockchip.c:15:
In file included from include/linux/regmap.h:20:
In file included from include/linux/iopoll.h:14:
In file included from include/linux/io.h:13:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:119:21: note: expanded from macro '__swab32'
___constant_swab32(x) : \
^
include/uapi/linux/swab.h:21:12: note: expanded from macro '___constant_swab32'
(((__u32)(x) & (__u32)0x00ff0000UL) >> 8) | \
^
In file included from drivers/gpu/drm/rockchip/analogix_dp-rockchip.c:15:
In file included from include/linux/regmap.h:20:
In file included from include/linux/iopoll.h:14:
In file included from include/linux/io.h:13:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:119:21: note: expanded from macro '__swab32'
___constant_swab32(x) : \
^
include/uapi/linux/swab.h:22:12: note: expanded from macro '___constant_swab32'
(((__u32)(x) & (__u32)0xff000000UL) >> 24)))
^
In file included from drivers/gpu/drm/rockchip/analogix_dp-rockchip.c:15:
In file included from include/linux/regmap.h:20:
In file included from include/linux/iopoll.h:14:
In file included from include/linux/io.h:13:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:120:12: note: expanded from macro '__swab32'
__fswab32(x))
^
In file included from drivers/gpu/drm/rockchip/analogix_dp-rockchip.c:15:
In file included from include/linux/regmap.h:20:
In file included from include/linux/iopoll.h:14:
In file included from include/linux/io.h:13:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:501:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
__raw_writeb(value, PCI_IOBASE + addr);
~~~~~~~~~~ ^
include/asm-generic/io.h:511:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
__raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
~~~~~~~~~~ ^
include/asm-generic/io.h:521:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
__raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
~~~~~~~~~~ ^
include/asm-generic/io.h:609:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
readsb(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:617:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
readsw(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:625:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
readsl(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:634:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
writesb(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:643:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
writesw(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:652:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
writesl(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
>> drivers/gpu/drm/rockchip/analogix_dp-rockchip.c:457:34: warning: unused variable 'rockchip_dp_dt_ids' [-Wunused-const-variable]
static const struct of_device_id rockchip_dp_dt_ids[] = {
^
21 warnings generated.
..
vim +/s3c64xx_spi_dt_match +1461 drivers/spi/spi-s3c64xx.c
a5238e360b715e Thomas Abraham 2012-07-13 1460
2b90807549e5d1 Thomas Abraham 2012-07-13 @1461 static const struct of_device_id s3c64xx_spi_dt_match[] = {
a3b924df8fa908 Mateusz Krawczuk 2013-09-23 1462 { .compatible = "samsung,s3c2443-spi",
a3b924df8fa908 Mateusz Krawczuk 2013-09-23 1463 .data = (void *)&s3c2443_spi_port_config,
a3b924df8fa908 Mateusz Krawczuk 2013-09-23 1464 },
a3b924df8fa908 Mateusz Krawczuk 2013-09-23 1465 { .compatible = "samsung,s3c6410-spi",
a3b924df8fa908 Mateusz Krawczuk 2013-09-23 1466 .data = (void *)&s3c6410_spi_port_config,
a3b924df8fa908 Mateusz Krawczuk 2013-09-23 1467 },
a3b924df8fa908 Mateusz Krawczuk 2013-09-23 1468 { .compatible = "samsung,s5pv210-spi",
a3b924df8fa908 Mateusz Krawczuk 2013-09-23 1469 .data = (void *)&s5pv210_spi_port_config,
a3b924df8fa908 Mateusz Krawczuk 2013-09-23 1470 },
2b90807549e5d1 Thomas Abraham 2012-07-13 1471 { .compatible = "samsung,exynos4210-spi",
2b90807549e5d1 Thomas Abraham 2012-07-13 1472 .data = (void *)&exynos4_spi_port_config,
2b90807549e5d1 Thomas Abraham 2012-07-13 1473 },
bf77cba95f8c06 Padmavathi Venna 2014-11-06 1474 { .compatible = "samsung,exynos7-spi",
bf77cba95f8c06 Padmavathi Venna 2014-11-06 1475 .data = (void *)&exynos7_spi_port_config,
bf77cba95f8c06 Padmavathi Venna 2014-11-06 1476 },
7990b00819e765 Andi Shyti 2016-07-12 1477 { .compatible = "samsung,exynos5433-spi",
7990b00819e765 Andi Shyti 2016-07-12 1478 .data = (void *)&exynos5433_spi_port_config,
7990b00819e765 Andi Shyti 2016-07-12 1479 },
2b90807549e5d1 Thomas Abraham 2012-07-13 1480 { },
2b90807549e5d1 Thomas Abraham 2012-07-13 1481 };
2b90807549e5d1 Thomas Abraham 2012-07-13 1482 MODULE_DEVICE_TABLE(of, s3c64xx_spi_dt_match);
2b90807549e5d1 Thomas Abraham 2012-07-13 1483
:::::: The code at line 1461 was first introduced by commit
:::::: 2b90807549e5d1f700e523ddd098651ecfc18e65 spi: s3c64xx: add device tree support
:::::: TO: Thomas Abraham <thomas.abraham(a)linaro.org>
:::::: CC: Kukjin Kim <kgene.kim(a)samsung.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
11 months, 4 weeks
[chrome-os:chromeos-5.10 1/3] drivers/remoteproc/mtk_scp.c:854:37: warning: unused variable 'mt8183_of_data'
by kernel test robot
tree: https://chromium.googlesource.com/chromiumos/third_party/kernel chromeos-5.10
head: cd482ccbd833906f2594059d719bced156a7fe9d
commit: e2ab415f43ccbb60df9c64cf69ba6881d13738ad [1/3] FIXUP: FROMGIT: remoteproc/mtk_scp: surround DT device IDs with CONFIG_OF
config: x86_64-randconfig-a013-20210531 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project bc6799f2f79f0ae87e9f1ebf9d25ba799fbd25a9)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
git remote add chrome-os https://chromium.googlesource.com/chromiumos/third_party/kernel
git fetch --no-tags chrome-os chromeos-5.10
git checkout e2ab415f43ccbb60df9c64cf69ba6881d13738ad
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> drivers/remoteproc/mtk_scp.c:854:37: warning: unused variable 'mt8183_of_data' [-Wunused-const-variable]
static const struct mtk_scp_of_data mt8183_of_data = {
^
>> drivers/remoteproc/mtk_scp.c:866:37: warning: unused variable 'mt8192_of_data' [-Wunused-const-variable]
static const struct mtk_scp_of_data mt8192_of_data = {
^
2 warnings generated.
vim +/mt8183_of_data +854 drivers/remoteproc/mtk_scp.c
63c13d61eafe46 Erin Lo 2019-11-12 853
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 @854 static const struct mtk_scp_of_data mt8183_of_data = {
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 855 .scp_before_load = mt8183_scp_before_load,
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 856 .scp_irq_handler = mt8183_scp_irq_handler,
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 857 .scp_reset_assert = mt8183_scp_reset_assert,
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 858 .scp_reset_deassert = mt8183_scp_reset_deassert,
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 859 .scp_stop = mt8183_scp_stop,
1f52378f9cd441 Tzung-Bi Shih 2021-01-27 860 .scp_da_to_va = mt8183_scp_da_to_va,
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 861 .host_to_scp_reg = MT8183_HOST_TO_SCP,
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 862 .host_to_scp_int_bit = MT8183_HOST_IPC_INT_BIT,
2d98bf34a9d96b Tzung-Bi Shih 2020-12-02 863 .ipi_buf_offset = 0x7bdb0,
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 864 };
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 865
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 @866 static const struct mtk_scp_of_data mt8192_of_data = {
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 867 .scp_before_load = mt8192_scp_before_load,
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 868 .scp_irq_handler = mt8192_scp_irq_handler,
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 869 .scp_reset_assert = mt8192_scp_reset_assert,
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 870 .scp_reset_deassert = mt8192_scp_reset_deassert,
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 871 .scp_stop = mt8192_scp_stop,
1f52378f9cd441 Tzung-Bi Shih 2021-01-27 872 .scp_da_to_va = mt8192_scp_da_to_va,
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 873 .host_to_scp_reg = MT8192_GIPC_IN_SET,
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 874 .host_to_scp_int_bit = MT8192_HOST_IPC_INT_BIT,
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 875 };
fd0b6c1ff85a48 Pi-Hsun Shih 2020-09-21 876
:::::: The code at line 854 was first introduced by commit
:::::: fd0b6c1ff85a489bcf1bcf58af64da1aeffd39f0 remoteproc/mediatek: Add support for mt8192 SCP
:::::: TO: Pi-Hsun Shih <pihsun(a)chromium.org>
:::::: CC: Bjorn Andersson <bjorn.andersson(a)linaro.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
11 months, 4 weeks