tree:
https://git.kernel.org/pub/scm/linux/kernel/git/peterz/queue.git locking/next
head: cbbbf3810e4e62ffc809fae78080d8ea260d7a81
commit: 7b15db327cde4c9557f7c54e3ea25942a663078c [7/11] locking/lockdep: Mark
local_lock_t
config: m68k-randconfig-r035-20201214 (attached as .config)
compiler: m68k-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/peterz/queue.git/commit/?...
git remote add peterz-queue
https://git.kernel.org/pub/scm/linux/kernel/git/peterz/queue.git
git fetch --no-tags peterz-queue locking/next
git checkout 7b15db327cde4c9557f7c54e3ea25942a663078c
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=m68k
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 >>):
fs/squashfs/decompressor_multi_percpu.c: In function
'squashfs_decompressor_create':
> fs/squashfs/decompressor_multi_percpu.c:46:32: error: macro
"lockdep_init_map_type" passed 7 arguments, but takes just 6
46 |
local_lock_init(&stream->lock);
| ^
In file included from include/linux/spinlock.h:59,
from include/linux/mmzone.h:8,
from include/linux/gfp.h:6,
from include/linux/slab.h:15,
from fs/squashfs/decompressor_multi_percpu.c:8:
include/linux/lockdep.h:350: note: macro "lockdep_init_map_type" defined
here
350 | # define lockdep_init_map_type(lock, name, key, sub, inner, outer) \
|
In file included from include/linux/local_lock.h:5,
from include/linux/radix-tree.h:20,
from include/linux/fs.h:15,
from include/linux/buffer_head.h:12,
from fs/squashfs/decompressor_multi_percpu.c:10:
include/linux/local_lock_internal.h:34:2: error: 'lockdep_init_map_type'
undeclared (first use in this function); did you mean 'lockdep_wait_type'?
34 | lockdep_init_map_type(&(lock)->dep_map, #lock, &__key, 0, \
| ^~~~~~~~~~~~~~~~~~~~~
include/linux/local_lock.h:10:32: note: in expansion of macro
'__local_lock_init'
10 | #define local_lock_init(lock) __local_lock_init(lock)
| ^~~~~~~~~~~~~~~~~
fs/squashfs/decompressor_multi_percpu.c:46:3: note: in expansion of macro
'local_lock_init'
46 | local_lock_init(&stream->lock);
| ^~~~~~~~~~~~~~~
include/linux/local_lock_internal.h:34:2: note: each undeclared identifier is reported
only once for each function it appears in
34 | lockdep_init_map_type(&(lock)->dep_map, #lock, &__key, 0, \
| ^~~~~~~~~~~~~~~~~~~~~
include/linux/local_lock.h:10:32: note: in expansion of macro
'__local_lock_init'
10 | #define local_lock_init(lock) __local_lock_init(lock)
| ^~~~~~~~~~~~~~~~~
fs/squashfs/decompressor_multi_percpu.c:46:3: note: in expansion of macro
'local_lock_init'
46 | local_lock_init(&stream->lock);
| ^~~~~~~~~~~~~~~
include/linux/local_lock_internal.h:31:31: warning: unused variable '__key'
[-Wunused-variable]
31 | static struct lock_class_key __key; \
| ^~~~~
include/linux/local_lock.h:10:32: note: in expansion of macro
'__local_lock_init'
10 | #define local_lock_init(lock) __local_lock_init(lock)
| ^~~~~~~~~~~~~~~~~
fs/squashfs/decompressor_multi_percpu.c:46:3: note: in expansion of macro
'local_lock_init'
46 | local_lock_init(&stream->lock);
| ^~~~~~~~~~~~~~~
At top level:
include/linux/local_lock_internal.h:31:31: warning: '__key' defined but not
used [-Wunused-variable]
31 | static struct lock_class_key __key; \
| ^~~~~
include/linux/local_lock.h:10:32: note: in expansion of macro
'__local_lock_init'
10 | #define local_lock_init(lock) __local_lock_init(lock)
| ^~~~~~~~~~~~~~~~~
fs/squashfs/decompressor_multi_percpu.c:46:3: note: in expansion of macro
'local_lock_init'
46 | local_lock_init(&stream->lock);
| ^~~~~~~~~~~~~~~
vim +/lockdep_init_map_type +46 fs/squashfs/decompressor_multi_percpu.c
d208383d640727b Phillip Lougher 2013-11-18 27
d208383d640727b Phillip Lougher 2013-11-18 28 void *squashfs_decompressor_create(struct
squashfs_sb_info *msblk,
d208383d640727b Phillip Lougher 2013-11-18 29 void *comp_opts)
d208383d640727b Phillip Lougher 2013-11-18 30 {
d208383d640727b Phillip Lougher 2013-11-18 31 struct squashfs_stream *stream;
d208383d640727b Phillip Lougher 2013-11-18 32 struct squashfs_stream __percpu
*percpu;
d208383d640727b Phillip Lougher 2013-11-18 33 int err, cpu;
d208383d640727b Phillip Lougher 2013-11-18 34
d208383d640727b Phillip Lougher 2013-11-18 35 percpu = alloc_percpu(struct
squashfs_stream);
d208383d640727b Phillip Lougher 2013-11-18 36 if (percpu == NULL)
d208383d640727b Phillip Lougher 2013-11-18 37 return ERR_PTR(-ENOMEM);
d208383d640727b Phillip Lougher 2013-11-18 38
d208383d640727b Phillip Lougher 2013-11-18 39 for_each_possible_cpu(cpu) {
d208383d640727b Phillip Lougher 2013-11-18 40 stream = per_cpu_ptr(percpu, cpu);
d208383d640727b Phillip Lougher 2013-11-18 41 stream->stream =
msblk->decompressor->init(msblk, comp_opts);
d208383d640727b Phillip Lougher 2013-11-18 42 if (IS_ERR(stream->stream)) {
d208383d640727b Phillip Lougher 2013-11-18 43 err = PTR_ERR(stream->stream);
d208383d640727b Phillip Lougher 2013-11-18 44 goto out;
d208383d640727b Phillip Lougher 2013-11-18 45 }
fd56200a16c72c7 Julia Cartwright 2020-05-27 @46 local_lock_init(&stream->lock);
d208383d640727b Phillip Lougher 2013-11-18 47 }
d208383d640727b Phillip Lougher 2013-11-18 48
d208383d640727b Phillip Lougher 2013-11-18 49 kfree(comp_opts);
d208383d640727b Phillip Lougher 2013-11-18 50 return (__force void *) percpu;
d208383d640727b Phillip Lougher 2013-11-18 51
d208383d640727b Phillip Lougher 2013-11-18 52 out:
d208383d640727b Phillip Lougher 2013-11-18 53 for_each_possible_cpu(cpu) {
d208383d640727b Phillip Lougher 2013-11-18 54 stream = per_cpu_ptr(percpu, cpu);
d208383d640727b Phillip Lougher 2013-11-18 55 if
(!IS_ERR_OR_NULL(stream->stream))
d208383d640727b Phillip Lougher 2013-11-18 56
msblk->decompressor->free(stream->stream);
d208383d640727b Phillip Lougher 2013-11-18 57 }
d208383d640727b Phillip Lougher 2013-11-18 58 free_percpu(percpu);
d208383d640727b Phillip Lougher 2013-11-18 59 return ERR_PTR(err);
d208383d640727b Phillip Lougher 2013-11-18 60 }
d208383d640727b Phillip Lougher 2013-11-18 61
:::::: The code at line 46 was first introduced by commit
:::::: fd56200a16c72c7c3ec3e54e06160dfaa5b8dee8 squashfs: Make use of local lock in
multi_cpu decompressor
:::::: TO: Julia Cartwright <julia(a)ni.com>
:::::: CC: Ingo Molnar <mingo(a)kernel.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org