| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <201710240404.dBXMGWWf%fengguang.wu@intel.com>
Date: Tue, 24 Oct 2017 04:22:22 +0800
From: kbuild test robot <lkp@...el.com>
To: Mark Rutland <mark.rutland@....com>
Cc: kbuild-all@...org, linux-kernel@...r.kernel.org,
Mark Rutland <mark.rutland@....com>,
Peter Zijlstra <peterz@...radead.org>,
Ingo Molnar <mingo@...hat.com>
Subject: Re: [PATCH] locking/spinlock/debug: snapshot lock fields
Hi Mark,
[auto build test WARNING on tip/locking/core]
[also build test WARNING on v4.14-rc6 next-20171018]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Mark-Rutland/locking-spinlock-debug-snapshot-lock-fields/20171024-035933
config: i386-randconfig-x009-201743 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All warnings (new ones prefixed by >>):
In file included from include/linux/linkage.h:4:0,
from include/linux/preempt.h:9,
from include/linux/spinlock.h:50,
from kernel/locking/spinlock_debug.c:9:
kernel/locking/spinlock_debug.c: In function 'debug_spin_unlock':
kernel/locking/spinlock_debug.c:124:14: error: implicit declaration of function 'arch_spin_value_unlocked' [-Werror=implicit-function-declaration]
SPIN_BUG_ON(arch_spin_value_unlocked(lock.raw_lock), lockp, lock,
^
include/linux/compiler.h:156:30: note: in definition of macro '__trace_if'
if (__builtin_constant_p(!!(cond)) ? !!(cond) : \
^~~~
>> kernel/locking/spinlock_debug.c:81:2: note: in expansion of macro 'if'
if (unlikely(cond)) spin_bug(lockp, lock, msg)
^~
include/linux/compiler.h:146:24: note: in expansion of macro '__branch_check__'
# define unlikely(x) (__branch_check__(x, 0, __builtin_constant_p(x)))
^~~~~~~~~~~~~~~~
kernel/locking/spinlock_debug.c:81:6: note: in expansion of macro 'unlikely'
if (unlikely(cond)) spin_bug(lockp, lock, msg)
^~~~~~~~
kernel/locking/spinlock_debug.c:124:2: note: in expansion of macro 'SPIN_BUG_ON'
SPIN_BUG_ON(arch_spin_value_unlocked(lock.raw_lock), lockp, lock,
^~~~~~~~~~~
kernel/locking/spinlock_debug.c: In function 'do_raw_spin_trylock':
kernel/locking/spinlock_debug.c:154:49: error: macro "SPIN_BUG_ON" requires 4 arguments, but only 3 given
SPIN_BUG_ON(!ret, lock, "trylock failure on UP");
^
kernel/locking/spinlock_debug.c:154:2: error: 'SPIN_BUG_ON' undeclared (first use in this function)
SPIN_BUG_ON(!ret, lock, "trylock failure on UP");
^~~~~~~~~~~
kernel/locking/spinlock_debug.c:154:2: note: each undeclared identifier is reported only once for each function it appears in
cc1: some warnings being treated as errors
vim +/if +81 kernel/locking/spinlock_debug.c
> 9 #include <linux/spinlock.h>
10 #include <linux/nmi.h>
11 #include <linux/interrupt.h>
12 #include <linux/debug_locks.h>
13 #include <linux/delay.h>
14 #include <linux/export.h>
15
16 void __raw_spin_lock_init(raw_spinlock_t *lock, const char *name,
17 struct lock_class_key *key)
18 {
19 #ifdef CONFIG_DEBUG_LOCK_ALLOC
20 /*
21 * Make sure we are not reinitializing a held lock:
22 */
23 debug_check_no_locks_freed((void *)lock, sizeof(*lock));
24 lockdep_init_map(&lock->dep_map, name, key, 0);
25 #endif
26 lock->raw_lock = (arch_spinlock_t)__ARCH_SPIN_LOCK_UNLOCKED;
27 lock->magic = SPINLOCK_MAGIC;
28 lock->owner = SPINLOCK_OWNER_INIT;
29 lock->owner_cpu = -1;
30 }
31
32 EXPORT_SYMBOL(__raw_spin_lock_init);
33
34 void __rwlock_init(rwlock_t *lock, const char *name,
35 struct lock_class_key *key)
36 {
37 #ifdef CONFIG_DEBUG_LOCK_ALLOC
38 /*
39 * Make sure we are not reinitializing a held lock:
40 */
41 debug_check_no_locks_freed((void *)lock, sizeof(*lock));
42 lockdep_init_map(&lock->dep_map, name, key, 0);
43 #endif
44 lock->raw_lock = (arch_rwlock_t) __ARCH_RW_LOCK_UNLOCKED;
45 lock->magic = RWLOCK_MAGIC;
46 lock->owner = SPINLOCK_OWNER_INIT;
47 lock->owner_cpu = -1;
48 }
49
50 EXPORT_SYMBOL(__rwlock_init);
51
52 static void spin_dump(raw_spinlock_t *lockp, raw_spinlock_t lock,
53 const char *msg)
54 {
55 struct task_struct *owner = NULL;
56
57 if (lock.owner && lock.owner != SPINLOCK_OWNER_INIT)
58 owner = lock.owner;
59 printk(KERN_EMERG "BUG: spinlock %s on CPU#%d, %s/%d\n",
60 msg, raw_smp_processor_id(),
61 current->comm, task_pid_nr(current));
62 printk(KERN_EMERG " lock: %pS, .magic: %08x, .owner: %s/%d, "
63 ".owner_cpu: %d\n",
64 lockp, lock.magic,
65 owner ? owner->comm : "<none>",
66 owner ? task_pid_nr(owner) : -1,
67 lock.owner_cpu);
68 dump_stack();
69 }
70
71 static void spin_bug(raw_spinlock_t *lockp, raw_spinlock_t lock,
72 const char *msg)
73 {
74 if (!debug_locks_off())
75 return;
76
77 spin_dump(lockp, lock, msg);
78 }
79
80 #define SPIN_BUG_ON(cond, lockp, lock, msg) \
> 81 if (unlikely(cond)) spin_bug(lockp, lock, msg)
82
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
Download attachment ".config.gz" of type "application/gzip" (29326 bytes)
Powered by blists - more mailing lists