diff options
| -rw-r--r-- | drivers/char/sysrq.c | 2 | ||||
| -rw-r--r-- | include/asm-generic/mutex-null.h | 15 | ||||
| -rw-r--r-- | include/linux/debug_locks.h | 69 | ||||
| -rw-r--r-- | include/linux/init_task.h | 1 | ||||
| -rw-r--r-- | include/linux/mm.h | 8 | ||||
| -rw-r--r-- | include/linux/mutex-debug.h | 12 | ||||
| -rw-r--r-- | include/linux/mutex.h | 6 | ||||
| -rw-r--r-- | include/linux/rtmutex.h | 10 | ||||
| -rw-r--r-- | include/linux/sched.h | 4 | ||||
| -rw-r--r-- | init/main.c | 8 | ||||
| -rw-r--r-- | kernel/exit.c | 5 | ||||
| -rw-r--r-- | kernel/fork.c | 4 | ||||
| -rw-r--r-- | kernel/mutex-debug.c | 51 | ||||
| -rw-r--r-- | kernel/mutex-debug.h | 82 | ||||
| -rw-r--r-- | kernel/mutex.c | 52 | ||||
| -rw-r--r-- | kernel/mutex.h | 17 | ||||
| -rw-r--r-- | kernel/rtmutex-debug.c | 302 | ||||
| -rw-r--r-- | kernel/rtmutex-debug.h | 8 | ||||
| -rw-r--r-- | kernel/rtmutex.c | 46 | ||||
| -rw-r--r-- | kernel/rtmutex.h | 3 | ||||
| -rw-r--r-- | kernel/sched.c | 16 | ||||
| -rw-r--r-- | lib/Makefile | 2 | ||||
| -rw-r--r-- | lib/debug_locks.c | 45 | ||||
| -rw-r--r-- | lib/spinlock_debug.c | 62 | ||||
| -rw-r--r-- | mm/vmalloc.c | 2 |
25 files changed, 265 insertions, 567 deletions
diff --git a/drivers/char/sysrq.c b/drivers/char/sysrq.c index a064ee9181c0..e31f079400fa 100644 --- a/drivers/char/sysrq.c +++ b/drivers/char/sysrq.c | |||
| @@ -151,7 +151,7 @@ static struct sysrq_key_op sysrq_mountro_op = { | |||
| 151 | static void sysrq_handle_showlocks(int key, struct pt_regs *pt_regs, | 151 | static void sysrq_handle_showlocks(int key, struct pt_regs *pt_regs, |
| 152 | struct tty_struct *tty) | 152 | struct tty_struct *tty) |
| 153 | { | 153 | { |
| 154 | mutex_debug_show_all_locks(); | 154 | debug_show_all_locks(); |
| 155 | } | 155 | } |
| 156 | static struct sysrq_key_op sysrq_showlocks_op = { | 156 | static struct sysrq_key_op sysrq_showlocks_op = { |
| 157 | .handler = sysrq_handle_showlocks, | 157 | .handler = sysrq_handle_showlocks, |
diff --git a/include/asm-generic/mutex-null.h b/include/asm-generic/mutex-null.h index 5cf8b7ce0c45..254a126ede5c 100644 --- a/include/asm-generic/mutex-null.h +++ b/include/asm-generic/mutex-null.h | |||
| @@ -10,15 +10,10 @@ | |||
| 10 | #ifndef _ASM_GENERIC_MUTEX_NULL_H | 10 | #ifndef _ASM_GENERIC_MUTEX_NULL_H |
| 11 | #define _ASM_GENERIC_MUTEX_NULL_H | 11 | #define _ASM_GENERIC_MUTEX_NULL_H |
| 12 | 12 | ||
| 13 | /* extra parameter only needed for mutex debugging: */ | 13 | #define __mutex_fastpath_lock(count, fail_fn) fail_fn(count) |
| 14 | #ifndef __IP__ | 14 | #define __mutex_fastpath_lock_retval(count, fail_fn) fail_fn(count) |
| 15 | # define __IP__ | 15 | #define __mutex_fastpath_unlock(count, fail_fn) fail_fn(count) |
| 16 | #endif | 16 | #define __mutex_fastpath_trylock(count, fail_fn) fail_fn(count) |
| 17 | 17 | #define __mutex_slowpath_needs_to_unlock() 1 | |
| 18 | #define __mutex_fastpath_lock(count, fail_fn) fail_fn(count __RET_IP__) | ||
| 19 | #define __mutex_fastpath_lock_retval(count, fail_fn) fail_fn(count __RET_IP__) | ||
| 20 | #define __mutex_fastpath_unlock(count, fail_fn) fail_fn(count __RET_IP__) | ||
| 21 | #define __mutex_fastpath_trylock(count, fail_fn) fail_fn(count) | ||
| 22 | #define __mutex_slowpath_needs_to_unlock() 1 | ||
| 23 | 18 | ||
| 24 | #endif | 19 | #endif |
diff --git a/include/linux/debug_locks.h b/include/linux/debug_locks.h new file mode 100644 index 000000000000..6a7047851e48 --- /dev/null +++ b/include/linux/debug_locks.h | |||
| @@ -0,0 +1,69 @@ | |||
| 1 | #ifndef __LINUX_DEBUG_LOCKING_H | ||
| 2 | #define __LINUX_DEBUG_LOCKING_H | ||
| 3 | |||
| 4 | extern int debug_locks; | ||
| 5 | extern int debug_locks_silent; | ||
| 6 | |||
| 7 | /* | ||
| 8 | * Generic 'turn off all lock debugging' function: | ||
| 9 | */ | ||
| 10 | extern int debug_locks_off(void); | ||
| 11 | |||
| 12 | /* | ||
| 13 | * In the debug case we carry the caller's instruction pointer into | ||
| 14 | * other functions, but we dont want the function argument overhead | ||
| 15 | * in the nondebug case - hence these macros: | ||
| 16 | */ | ||
| 17 | #define _RET_IP_ (unsigned long)__builtin_return_address(0) | ||
| 18 | #define _THIS_IP_ ({ __label__ __here; __here: (unsigned long)&&__here; }) | ||
| 19 | |||
| 20 | #define DEBUG_LOCKS_WARN_ON(c) \ | ||
| 21 | ({ \ | ||
| 22 | int __ret = 0; \ | ||
| 23 | \ | ||
| 24 | if (unlikely(c)) { \ | ||
| 25 | if (debug_locks_off()) \ | ||
| 26 | WARN_ON(1); \ | ||
| 27 | __ret = 1; \ | ||
| 28 | } \ | ||
| 29 | __ret; \ | ||
| 30 | }) | ||
| 31 | |||
| 32 | #ifdef CONFIG_SMP | ||
| 33 | # define SMP_DEBUG_LOCKS_WARN_ON(c) DEBUG_LOCKS_WARN_ON(c) | ||
| 34 | #else | ||
| 35 | # define SMP_DEBUG_LOCKS_WARN_ON(c) do { } while (0) | ||
| 36 | #endif | ||
| 37 | |||
| 38 | #ifdef CONFIG_DEBUG_LOCKING_API_SELFTESTS | ||
| 39 | extern void locking_selftest(void); | ||
| 40 | #else | ||
| 41 | # define locking_selftest() do { } while (0) | ||
| 42 | #endif | ||
| 43 | |||
| 44 | #ifdef CONFIG_LOCKDEP | ||
| 45 | extern void debug_show_all_locks(void); | ||
| 46 | extern void debug_show_held_locks(struct task_struct *task); | ||
| 47 | extern void debug_check_no_locks_freed(const void *from, unsigned long len); | ||
| 48 | extern void debug_check_no_locks_held(struct task_struct *task); | ||
| 49 | #else | ||
| 50 | static inline void debug_show_all_locks(void) | ||
| 51 | { | ||
| 52 | } | ||
| 53 | |||
| 54 | static inline void debug_show_held_locks(struct task_struct *task) | ||
| 55 | { | ||
| 56 | } | ||
| 57 | |||
| 58 | static inline void | ||
| 59 | debug_check_no_locks_freed(const void *from, unsigned long len) | ||
| 60 | { | ||
| 61 | } | ||
| 62 | |||
| 63 | static inline void | ||
| 64 | debug_check_no_locks_held(struct task_struct *task) | ||
| 65 | { | ||
| 66 | } | ||
| 67 | #endif | ||
| 68 | |||
| 69 | #endif | ||
diff --git a/include/linux/init_task.h b/include/linux/init_task.h index 3a256957fb56..678c1a90380d 100644 --- a/include/linux/init_task.h +++ b/include/linux/init_task.h | |||
| @@ -124,7 +124,6 @@ extern struct group_info init_groups; | |||
| 124 | .cpu_timers = INIT_CPU_TIMERS(tsk.cpu_timers), \ | 124 | .cpu_timers = INIT_CPU_TIMERS(tsk.cpu_timers), \ |
| 125 | .fs_excl = ATOMIC_INIT(0), \ | 125 | .fs_excl = ATOMIC_INIT(0), \ |
| 126 | .pi_lock = SPIN_LOCK_UNLOCKED, \ | 126 | .pi_lock = SPIN_LOCK_UNLOCKED, \ |
| 127 | INIT_RT_MUTEXES(tsk) \ | ||
| 128 | } | 127 | } |
| 129 | 128 | ||
| 130 | 129 | ||
diff --git a/include/linux/mm.h b/include/linux/mm.h index 75179529e399..990957e0929f 100644 --- a/include/linux/mm.h +++ b/include/linux/mm.h | |||
| @@ -14,6 +14,7 @@ | |||
| 14 | #include <linux/prio_tree.h> | 14 | #include <linux/prio_tree.h> |
| 15 | #include <linux/fs.h> | 15 | #include <linux/fs.h> |
| 16 | #include <linux/mutex.h> | 16 | #include <linux/mutex.h> |
| 17 | #include <linux/debug_locks.h> | ||
| 17 | 18 | ||
| 18 | struct mempolicy; | 19 | struct mempolicy; |
| 19 | struct anon_vma; | 20 | struct anon_vma; |
| @@ -1034,13 +1035,6 @@ static inline void vm_stat_account(struct mm_struct *mm, | |||
| 1034 | } | 1035 | } |
| 1035 | #endif /* CONFIG_PROC_FS */ | 1036 | #endif /* CONFIG_PROC_FS */ |
| 1036 | 1037 | ||
| 1037 | static inline void | ||
| 1038 | debug_check_no_locks_freed(const void *from, unsigned long len) | ||
| 1039 | { | ||
| 1040 | mutex_debug_check_no_locks_freed(from, len); | ||
| 1041 | rt_mutex_debug_check_no_locks_freed(from, len); | ||
| 1042 | } | ||
| 1043 | |||
| 1044 | #ifndef CONFIG_DEBUG_PAGEALLOC | 1038 | #ifndef CONFIG_DEBUG_PAGEALLOC |
| 1045 | static inline void | 1039 | static inline void |
| 1046 | kernel_map_pages(struct page *page, int numpages, int enable) | 1040 | kernel_map_pages(struct page *page, int numpages, int enable) |
diff --git a/include/linux/mutex-debug.h b/include/linux/mutex-debug.h index 8b5769f00467..70a26091fc73 100644 --- a/include/linux/mutex-debug.h +++ b/include/linux/mutex-debug.h | |||
| @@ -7,17 +7,11 @@ | |||
| 7 | * Mutexes - debugging helpers: | 7 | * Mutexes - debugging helpers: |
| 8 | */ | 8 | */ |
| 9 | 9 | ||
| 10 | #define __DEBUG_MUTEX_INITIALIZER(lockname) \ | 10 | #define __DEBUG_MUTEX_INITIALIZER(lockname) \ |
| 11 | , .held_list = LIST_HEAD_INIT(lockname.held_list), \ | 11 | , .magic = &lockname |
| 12 | .name = #lockname , .magic = &lockname | ||
| 13 | 12 | ||
| 14 | #define mutex_init(sem) __mutex_init(sem, __FUNCTION__) | 13 | #define mutex_init(sem) __mutex_init(sem, __FILE__":"#sem) |
| 15 | 14 | ||
| 16 | extern void FASTCALL(mutex_destroy(struct mutex *lock)); | 15 | extern void FASTCALL(mutex_destroy(struct mutex *lock)); |
| 17 | 16 | ||
| 18 | extern void mutex_debug_show_all_locks(void); | ||
| 19 | extern void mutex_debug_show_held_locks(struct task_struct *filter); | ||
| 20 | extern void mutex_debug_check_no_locks_held(struct task_struct *task); | ||
| 21 | extern void mutex_debug_check_no_locks_freed(const void *from, unsigned long len); | ||
| 22 | |||
| 23 | #endif | 17 | #endif |
diff --git a/include/linux/mutex.h b/include/linux/mutex.h index f1ac507fa20d..caafecd5e366 100644 --- a/include/linux/mutex.h +++ b/include/linux/mutex.h | |||
| @@ -50,8 +50,6 @@ struct mutex { | |||
| 50 | struct list_head wait_list; | 50 | struct list_head wait_list; |
| 51 | #ifdef CONFIG_DEBUG_MUTEXES | 51 | #ifdef CONFIG_DEBUG_MUTEXES |
| 52 | struct thread_info *owner; | 52 | struct thread_info *owner; |
| 53 | struct list_head held_list; | ||
| 54 | unsigned long acquire_ip; | ||
| 55 | const char *name; | 53 | const char *name; |
| 56 | void *magic; | 54 | void *magic; |
| 57 | #endif | 55 | #endif |
| @@ -76,10 +74,6 @@ struct mutex_waiter { | |||
| 76 | # define __DEBUG_MUTEX_INITIALIZER(lockname) | 74 | # define __DEBUG_MUTEX_INITIALIZER(lockname) |
| 77 | # define mutex_init(mutex) __mutex_init(mutex, NULL) | 75 | # define mutex_init(mutex) __mutex_init(mutex, NULL) |
| 78 | # define mutex_destroy(mutex) do { } while (0) | 76 | # define mutex_destroy(mutex) do { } while (0) |
| 79 | # define mutex_debug_show_all_locks() do { } while (0) | ||
| 80 | # define mutex_debug_show_held_locks(p) do { } while (0) | ||
| 81 | # define mutex_debug_check_no_locks_held(task) do { } while (0) | ||
| 82 | # define mutex_debug_check_no_locks_freed(from, len) do { } while (0) | ||
| 83 | #endif | 77 | #endif |
| 84 | 78 | ||
| 85 | #define __MUTEX_INITIALIZER(lockname) \ | 79 | #define __MUTEX_INITIALIZER(lockname) \ |
diff --git a/include/linux/rtmutex.h b/include/linux/rtmutex.h index fa4a3b82ba70..5d41dee82f80 100644 --- a/include/linux/rtmutex.h +++ b/include/linux/rtmutex.h | |||
| @@ -29,8 +29,6 @@ struct rt_mutex { | |||
| 29 | struct task_struct *owner; | 29 | struct task_struct *owner; |
| 30 | #ifdef CONFIG_DEBUG_RT_MUTEXES | 30 | #ifdef CONFIG_DEBUG_RT_MUTEXES |
| 31 | int save_state; | 31 | int save_state; |
| 32 | struct list_head held_list_entry; | ||
| 33 | unsigned long acquire_ip; | ||
| 34 | const char *name, *file; | 32 | const char *name, *file; |
| 35 | int line; | 33 | int line; |
| 36 | void *magic; | 34 | void *magic; |
| @@ -98,14 +96,6 @@ extern int rt_mutex_trylock(struct rt_mutex *lock); | |||
| 98 | 96 | ||
| 99 | extern void rt_mutex_unlock(struct rt_mutex *lock); | 97 | extern void rt_mutex_unlock(struct rt_mutex *lock); |
| 100 | 98 | ||
| 101 | #ifdef CONFIG_DEBUG_RT_MUTEXES | ||
| 102 | # define INIT_RT_MUTEX_DEBUG(tsk) \ | ||
| 103 | .held_list_head = LIST_HEAD_INIT(tsk.held_list_head), \ | ||
| 104 | .held_list_lock = SPIN_LOCK_UNLOCKED | ||
| 105 | #else | ||
| 106 | # define INIT_RT_MUTEX_DEBUG(tsk) | ||
| 107 | #endif | ||
| 108 | |||
| 109 | #ifdef CONFIG_RT_MUTEXES | 99 | #ifdef CONFIG_RT_MUTEXES |
| 110 | # define INIT_RT_MUTEXES(tsk) \ | 100 | # define INIT_RT_MUTEXES(tsk) \ |
| 111 | .pi_waiters = PLIST_HEAD_INIT(tsk.pi_waiters, tsk.pi_lock), \ | 101 | .pi_waiters = PLIST_HEAD_INIT(tsk.pi_waiters, tsk.pi_lock), \ |
diff --git a/include/linux/sched.h b/include/linux/sched.h index aaf723308ed4..bdabeee10a78 100644 --- a/include/linux/sched.h +++ b/include/linux/sched.h | |||
| @@ -865,10 +865,6 @@ struct task_struct { | |||
| 865 | struct plist_head pi_waiters; | 865 | struct plist_head pi_waiters; |
| 866 | /* Deadlock detection and priority inheritance handling */ | 866 | /* Deadlock detection and priority inheritance handling */ |
| 867 | struct rt_mutex_waiter *pi_blocked_on; | 867 | struct rt_mutex_waiter *pi_blocked_on; |
| 868 | # ifdef CONFIG_DEBUG_RT_MUTEXES | ||
| 869 | spinlock_t held_list_lock; | ||
| 870 | struct list_head held_list_head; | ||
| 871 | # endif | ||
| 872 | #endif | 868 | #endif |
| 873 | 869 | ||
| 874 | #ifdef CONFIG_DEBUG_MUTEXES | 870 | #ifdef CONFIG_DEBUG_MUTEXES |
diff --git a/init/main.c b/init/main.c index d604dfef82b8..fc73e1cd8614 100644 --- a/init/main.c +++ b/init/main.c | |||
| @@ -47,6 +47,7 @@ | |||
| 47 | #include <linux/key.h> | 47 | #include <linux/key.h> |
| 48 | #include <linux/unwind.h> | 48 | #include <linux/unwind.h> |
| 49 | #include <linux/buffer_head.h> | 49 | #include <linux/buffer_head.h> |
| 50 | #include <linux/debug_locks.h> | ||
| 50 | 51 | ||
| 51 | #include <asm/io.h> | 52 | #include <asm/io.h> |
| 52 | #include <asm/bugs.h> | 53 | #include <asm/bugs.h> |
| @@ -511,6 +512,13 @@ asmlinkage void __init start_kernel(void) | |||
| 511 | console_init(); | 512 | console_init(); |
| 512 | if (panic_later) | 513 | if (panic_later) |
| 513 | panic(panic_later, panic_param); | 514 | panic(panic_later, panic_param); |
| 515 | /* | ||
| 516 | * Need to run this when irqs are enabled, because it wants | ||
| 517 | * to self-test [hard/soft]-irqs on/off lock inversion bugs | ||
| 518 | * too: | ||
| 519 | */ | ||
| 520 | locking_selftest(); | ||
| 521 | |||
| 514 | #ifdef CONFIG_BLK_DEV_INITRD | 522 | #ifdef CONFIG_BLK_DEV_INITRD |
| 515 | if (initrd_start && !initrd_below_start_ok && | 523 | if (initrd_start && !initrd_below_start_ok && |
| 516 | initrd_start < min_low_pfn << PAGE_SHIFT) { | 524 | initrd_start < min_low_pfn << PAGE_SHIFT) { |
diff --git a/kernel/exit.c b/kernel/exit.c index 7f7ef2258553..c595db14cf25 100644 --- a/kernel/exit.c +++ b/kernel/exit.c | |||
| @@ -933,10 +933,9 @@ fastcall NORET_TYPE void do_exit(long code) | |||
| 933 | if (unlikely(current->pi_state_cache)) | 933 | if (unlikely(current->pi_state_cache)) |
| 934 | kfree(current->pi_state_cache); | 934 | kfree(current->pi_state_cache); |
| 935 | /* | 935 | /* |
| 936 | * If DEBUG_MUTEXES is on, make sure we are holding no locks: | 936 | * Make sure we are holding no locks: |
| 937 | */ | 937 | */ |
| 938 | mutex_debug_check_no_locks_held(tsk); | 938 | debug_check_no_locks_held(tsk); |
| 939 | rt_mutex_debug_check_no_locks_held(tsk); | ||
| 940 | 939 | ||
| 941 | if (tsk->io_context) | 940 | if (tsk->io_context) |
| 942 | exit_io_context(); | 941 | exit_io_context(); |
diff --git a/kernel/fork.c b/kernel/fork.c index 9064bf9e131b..1cd46a4fb0d3 100644 --- a/kernel/fork.c +++ b/kernel/fork.c | |||
| @@ -919,10 +919,6 @@ static inline void rt_mutex_init_task(struct task_struct *p) | |||
| 919 | spin_lock_init(&p->pi_lock); | 919 | spin_lock_init(&p->pi_lock); |
| 920 | plist_head_init(&p->pi_waiters, &p->pi_lock); | 920 | plist_head_init(&p->pi_waiters, &p->pi_lock); |
| 921 | p->pi_blocked_on = NULL; | 921 | p->pi_blocked_on = NULL; |
| 922 | # ifdef CONFIG_DEBUG_RT_MUTEXES | ||
| 923 | spin_lock_init(&p->held_list_lock); | ||
| 924 | INIT_LIST_HEAD(&p->held_list_head); | ||
| 925 | # endif | ||
| 926 | #endif | 922 | #endif |
| 927 | } | 923 | } |
| 928 | 924 | ||
diff --git a/kernel/mutex-debug.c b/kernel/mutex-debug.c index a92de145ed0d..5569766a1ea2 100644 --- a/kernel/mutex-debug.c +++ b/kernel/mutex-debug.c | |||
| @@ -20,52 +20,19 @@ | |||
| 20 | #include <linux/spinlock.h> | 20 | #include <linux/spinlock.h> |
| 21 | #include <linux/kallsyms.h> | 21 | #include <linux/kallsyms.h> |
| 22 | #include <linux/interrupt.h> | 22 | #include <linux/interrupt.h> |
| 23 | #include <linux/debug_locks.h> | ||
| 23 | 24 | ||
| 24 | #include "mutex-debug.h" | 25 | #include "mutex-debug.h" |
| 25 | 26 | ||
| 26 | /* | 27 | /* |
| 27 | * We need a global lock when we walk through the multi-process | ||
| 28 | * lock tree. Only used in the deadlock-debugging case. | ||
| 29 | */ | ||
| 30 | DEFINE_SPINLOCK(debug_mutex_lock); | ||
| 31 | |||
| 32 | /* | ||
| 33 | * All locks held by all tasks, in a single global list: | ||
| 34 | */ | ||
| 35 | LIST_HEAD(debug_mutex_held_locks); | ||
| 36 | |||
| 37 | /* | ||
| 38 | * In the debug case we carry the caller's instruction pointer into | ||
| 39 | * other functions, but we dont want the function argument overhead | ||
| 40 | * in the nondebug case - hence these macros: | ||
| 41 | */ | ||
| 42 | #define __IP_DECL__ , unsigned long ip | ||
| 43 | #define __IP__ , ip | ||
| 44 | #define __RET_IP__ , (unsigned long)__builtin_return_address(0) | ||
| 45 | |||
| 46 | /* | ||
| 47 | * "mutex debugging enabled" flag. We turn it off when we detect | ||
| 48 | * the first problem because we dont want to recurse back | ||
| 49 | * into the tracing code when doing error printk or | ||
| 50 | * executing a BUG(): | ||
| 51 | */ | ||
| 52 | int debug_mutex_on = 1; | ||
| 53 | |||
| 54 | /* | ||
| 55 | * Must be called with lock->wait_lock held. | 28 | * Must be called with lock->wait_lock held. |
| 56 | */ | 29 | */ |
| 57 | void debug_mutex_set_owner(struct mutex *lock, | 30 | void debug_mutex_set_owner(struct mutex *lock, struct thread_info *new_owner) |
| 58 | struct thread_info *new_owner __IP_DECL__) | ||
| 59 | { | 31 | { |
| 60 | lock->owner = new_owner; | 32 | lock->owner = new_owner; |
| 61 | DEBUG_LOCKS_WARN_ON(!list_empty(&lock->held_list)); | ||
| 62 | if (debug_mutex_on) { | ||
| 63 | list_add_tail(&lock->held_list, &debug_mutex_held_locks); | ||
| 64 | lock->acquire_ip = ip; | ||
| 65 | } | ||
| 66 | } | 33 | } |
| 67 | 34 | ||
| 68 | void debug_mutex_init_waiter(struct mutex_waiter *waiter) | 35 | void debug_mutex_lock_common(struct mutex *lock, struct mutex_waiter *waiter) |
| 69 | { | 36 | { |
| 70 | memset(waiter, MUTEX_DEBUG_INIT, sizeof(*waiter)); | 37 | memset(waiter, MUTEX_DEBUG_INIT, sizeof(*waiter)); |
| 71 | waiter->magic = waiter; | 38 | waiter->magic = waiter; |
| @@ -87,9 +54,10 @@ void debug_mutex_free_waiter(struct mutex_waiter *waiter) | |||
| 87 | } | 54 | } |
| 88 | 55 | ||
| 89 | void debug_mutex_add_waiter(struct mutex *lock, struct mutex_waiter *waiter, | 56 | void debug_mutex_add_waiter(struct mutex *lock, struct mutex_waiter *waiter, |
| 90 | struct thread_info *ti __IP_DECL__) | 57 | struct thread_info *ti) |
| 91 | { | 58 | { |
| 92 | SMP_DEBUG_LOCKS_WARN_ON(!spin_is_locked(&lock->wait_lock)); | 59 | SMP_DEBUG_LOCKS_WARN_ON(!spin_is_locked(&lock->wait_lock)); |
| 60 | |||
| 93 | /* Mark the current thread as blocked on the lock: */ | 61 | /* Mark the current thread as blocked on the lock: */ |
| 94 | ti->task->blocked_on = waiter; | 62 | ti->task->blocked_on = waiter; |
| 95 | waiter->lock = lock; | 63 | waiter->lock = lock; |
| @@ -109,13 +77,10 @@ void mutex_remove_waiter(struct mutex *lock, struct mutex_waiter *waiter, | |||
| 109 | 77 | ||
| 110 | void debug_mutex_unlock(struct mutex *lock) | 78 | void debug_mutex_unlock(struct mutex *lock) |
| 111 | { | 79 | { |
| 80 | DEBUG_LOCKS_WARN_ON(lock->owner != current_thread_info()); | ||
| 112 | DEBUG_LOCKS_WARN_ON(lock->magic != lock); | 81 | DEBUG_LOCKS_WARN_ON(lock->magic != lock); |
| 113 | DEBUG_LOCKS_WARN_ON(!lock->wait_list.prev && !lock->wait_list.next); | 82 | DEBUG_LOCKS_WARN_ON(!lock->wait_list.prev && !lock->wait_list.next); |
| 114 | DEBUG_LOCKS_WARN_ON(lock->owner != current_thread_info()); | 83 | DEBUG_LOCKS_WARN_ON(lock->owner != current_thread_info()); |
| 115 | if (debug_mutex_on) { | ||
| 116 | DEBUG_LOCKS_WARN_ON(list_empty(&lock->held_list)); | ||
| 117 | list_del_init(&lock->held_list); | ||
| 118 | } | ||
| 119 | } | 84 | } |
| 120 | 85 | ||
| 121 | void debug_mutex_init(struct mutex *lock, const char *name) | 86 | void debug_mutex_init(struct mutex *lock, const char *name) |
| @@ -123,10 +88,8 @@ void debug_mutex_init(struct mutex *lock, const char *name) | |||
| 123 | /* | 88 | /* |
| 124 | * Make sure we are not reinitializing a held lock: | 89 | * Make sure we are not reinitializing a held lock: |
| 125 | */ | 90 | */ |
| 126 | mutex_debug_check_no_locks_freed((void *)lock, sizeof(*lock)); | 91 | debug_check_no_locks_freed((void *)lock, sizeof(*lock)); |
| 127 | lock->owner = NULL; | 92 | lock->owner = NULL; |
| 128 | INIT_LIST_HEAD(&lock->held_list); | ||
| 129 | lock->name = name; | ||
| 130 | lock->magic = lock; | 93 | lock->magic = lock; |
| 131 | } | 94 | } |
| 132 | 95 | ||
diff --git a/kernel/mutex-debug.h b/kernel/mutex-debug.h index bdab13a9ee26..babfbdfc534b 100644 --- a/kernel/mutex-debug.h +++ b/kernel/mutex-debug.h | |||
| @@ -10,102 +10,44 @@ | |||
| 10 | * More details are in kernel/mutex-debug.c. | 10 | * More details are in kernel/mutex-debug.c. |
| 11 | */ | 11 | */ |
| 12 | 12 | ||
| 13 | extern spinlock_t debug_mutex_lock; | ||
| 14 | extern struct list_head debug_mutex_held_locks; | ||
| 15 | extern int debug_mutex_on; | ||
| 16 | |||
| 17 | /* | ||
| 18 | * In the debug case we carry the caller's instruction pointer into | ||
| 19 | * other functions, but we dont want the function argument overhead | ||
| 20 | * in the nondebug case - hence these macros: | ||
| 21 | */ | ||
| 22 | #define __IP_DECL__ , unsigned long ip | ||
| 23 | #define __IP__ , ip | ||
| 24 | #define __RET_IP__ , (unsigned long)__builtin_return_address(0) | ||
| 25 | |||
| 26 | /* | 13 | /* |
| 27 | * This must be called with lock->wait_lock held. | 14 | * This must be called with lock->wait_lock held. |
| 28 | */ | 15 | */ |
| 29 | extern void debug_mutex_set_owner(struct mutex *lock, | 16 | extern void |
| 30 | struct thread_info *new_owner __IP_DECL__); | 17 | debug_mutex_set_owner(struct mutex *lock, struct thread_info *new_owner); |
| 31 | 18 | ||
| 32 | static inline void debug_mutex_clear_owner(struct mutex *lock) | 19 | static inline void debug_mutex_clear_owner(struct mutex *lock) |
| 33 | { | 20 | { |
| 34 | lock->owner = NULL; | 21 | lock->owner = NULL; |
| 35 | } | 22 | } |
| 36 | 23 | ||
| 37 | extern void debug_mutex_init_waiter(struct mutex_waiter *waiter); | 24 | extern void debug_mutex_lock_common(struct mutex *lock, |
| 25 | struct mutex_waiter *waiter); | ||
| 38 | extern void debug_mutex_wake_waiter(struct mutex *lock, | 26 | extern void debug_mutex_wake_waiter(struct mutex *lock, |
| 39 | struct mutex_waiter *waiter); | 27 | struct mutex_waiter *waiter); |
| 40 | extern void debug_mutex_free_waiter(struct mutex_waiter *waiter); | 28 | extern void debug_mutex_free_waiter(struct mutex_waiter *waiter); |
| 41 | extern void debug_mutex_add_waiter(struct mutex *lock, | 29 | extern void debug_mutex_add_waiter(struct mutex *lock, |
| 42 | struct mutex_waiter *waiter, | 30 | struct mutex_waiter *waiter, |
| 43 | struct thread_info *ti __IP_DECL__); | 31 | struct thread_info *ti); |
| 44 | extern void mutex_remove_waiter(struct mutex *lock, struct mutex_waiter *waiter, | 32 | extern void mutex_remove_waiter(struct mutex *lock, struct mutex_waiter *waiter, |
| 45 | struct thread_info *ti); | 33 | struct thread_info *ti); |
| 46 | extern void debug_mutex_unlock(struct mutex *lock); | 34 | extern void debug_mutex_unlock(struct mutex *lock); |
| 47 | extern void debug_mutex_init(struct mutex *lock, const char *name); | 35 | extern void debug_mutex_init(struct mutex *lock, const char *name, |
| 48 | 36 | struct lock_class_key *key); | |
| 49 | #define debug_spin_lock_save(lock, flags) \ | ||
| 50 | do { \ | ||
| 51 | local_irq_save(flags); \ | ||
| 52 | if (debug_mutex_on) \ | ||
| 53 | spin_lock(lock); \ | ||
| 54 | } while (0) | ||
| 55 | |||
| 56 | #define debug_spin_unlock_restore(lock, flags) \ | ||
| 57 | do { \ | ||
| 58 | if (debug_mutex_on) \ | ||
| 59 | spin_unlock(lock); \ | ||
| 60 | local_irq_restore(flags); \ | ||
| 61 | preempt_check_resched(); \ | ||
| 62 | } while (0) | ||
| 63 | 37 | ||
| 64 | #define spin_lock_mutex(lock, flags) \ | 38 | #define spin_lock_mutex(lock, flags) \ |
| 65 | do { \ | 39 | do { \ |
| 66 | struct mutex *l = container_of(lock, struct mutex, wait_lock); \ | 40 | struct mutex *l = container_of(lock, struct mutex, wait_lock); \ |
| 67 | \ | 41 | \ |
| 68 | DEBUG_LOCKS_WARN_ON(in_interrupt()); \ | 42 | DEBUG_LOCKS_WARN_ON(in_interrupt()); \ |
| 69 | debug_spin_lock_save(&debug_mutex_lock, flags); \ | 43 | local_irq_save(flags); \ |
| 70 | spin_lock(lock); \ | 44 | __raw_spin_lock(&(lock)->raw_lock); \ |
| 71 | DEBUG_LOCKS_WARN_ON(l->magic != l); \ | 45 | DEBUG_LOCKS_WARN_ON(l->magic != l); \ |
| 72 | } while (0) | 46 | } while (0) |
| 73 | 47 | ||
| 74 | #define spin_unlock_mutex(lock, flags) \ | 48 | #define spin_unlock_mutex(lock, flags) \ |
| 75 | do { \ | 49 | do { \ |
| 76 | spin_unlock(lock); \ | 50 | __raw_spin_unlock(&(lock)->raw_lock); \ |
| 77 | debug_spin_unlock_restore(&debug_mutex_lock, flags); \ | 51 | local_irq_restore(flags); \ |
| 52 | preempt_check_resched(); \ | ||
| 78 | } while (0) | 53 | } while (0) |
| 79 | |||
| 80 | #define DEBUG_OFF() \ | ||
| 81 | do { \ | ||
| 82 | if (debug_mutex_on) { \ | ||
| 83 | debug_mutex_on = 0; \ | ||
| 84 | console_verbose(); \ | ||
| 85 | if (spin_is_locked(&debug_mutex_lock)) \ | ||
| 86 | spin_unlock(&debug_mutex_lock); \ | ||
| 87 | } \ | ||
| 88 | } while (0) | ||
| 89 | |||
| 90 | #define DEBUG_BUG() \ | ||
| 91 | do { \ | ||
| 92 | if (debug_mutex_on) { \ | ||
| 93 | DEBUG_OFF(); \ | ||
| 94 | BUG(); \ | ||
| 95 | } \ | ||
| 96 | } while (0) | ||
| 97 | |||
| 98 | #define DEBUG_LOCKS_WARN_ON(c) \ | ||
| 99 | do { \ | ||
| 100 | if (unlikely(c && debug_mutex_on)) { \ | ||
| 101 | DEBUG_OFF(); \ | ||
| 102 | WARN_ON(1); \ | ||
| 103 | } \ | ||
| 104 | } while (0) | ||
| 105 | |||
| 106 | #ifdef CONFIG_SMP | ||
| 107 | # define SMP_DEBUG_LOCKS_WARN_ON(c) DEBUG_LOCKS_WARN_ON(c) | ||
| 108 | #else | ||
| 109 | # define SMP_DEBUG_LOCKS_WARN_ON(c) do { } while (0) | ||
| 110 | #endif | ||
| 111 | |||
diff --git a/kernel/mutex.c b/kernel/mutex.c index 101ceeb38925..3aad0b7992f4 100644 --- a/kernel/mutex.c +++ b/kernel/mutex.c | |||
| @@ -17,6 +17,7 @@ | |||
| 17 | #include <linux/module.h> | 17 | #include <linux/module.h> |
| 18 | #include <linux/spinlock.h> | 18 | #include <linux/spinlock.h> |
| 19 | #include <linux/interrupt.h> | 19 | #include <linux/interrupt.h> |
| 20 | #include <linux/debug_locks.h> | ||
| 20 | 21 | ||
| 21 | /* | 22 | /* |
| 22 | * In the DEBUG case we are using the "NULL fastpath" for mutexes, | 23 | * In the DEBUG case we are using the "NULL fastpath" for mutexes, |
| @@ -38,7 +39,7 @@ | |||
| 38 | * | 39 | * |
| 39 | * It is not allowed to initialize an already locked mutex. | 40 | * It is not allowed to initialize an already locked mutex. |
| 40 | */ | 41 | */ |
| 41 | void fastcall __mutex_init(struct mutex *lock, const char *name) | 42 | __always_inline void fastcall __mutex_init(struct mutex *lock, const char *name) |
| 42 | { | 43 | { |
| 43 | atomic_set(&lock->count, 1); | 44 | atomic_set(&lock->count, 1); |
| 44 | spin_lock_init(&lock->wait_lock); | 45 | spin_lock_init(&lock->wait_lock); |
| @@ -56,7 +57,7 @@ EXPORT_SYMBOL(__mutex_init); | |||
| 56 | * branch is predicted by the CPU as default-untaken. | 57 | * branch is predicted by the CPU as default-untaken. |
| 57 | */ | 58 | */ |
| 58 | static void fastcall noinline __sched | 59 | static void fastcall noinline __sched |
| 59 | __mutex_lock_slowpath(atomic_t *lock_count __IP_DECL__); | 60 | __mutex_lock_slowpath(atomic_t *lock_count); |
| 60 | 61 | ||
| 61 | /*** | 62 | /*** |
| 62 | * mutex_lock - acquire the mutex | 63 | * mutex_lock - acquire the mutex |
| @@ -79,7 +80,7 @@ __mutex_lock_slowpath(atomic_t *lock_count __IP_DECL__); | |||
| 79 | * | 80 | * |
| 80 | * This function is similar to (but not equivalent to) down(). | 81 | * This function is similar to (but not equivalent to) down(). |
| 81 | */ | 82 | */ |
| 82 | void fastcall __sched mutex_lock(struct mutex *lock) | 83 | void inline fastcall __sched mutex_lock(struct mutex *lock) |
| 83 | { | 84 | { |
| 84 | might_sleep(); | 85 | might_sleep(); |
| 85 | /* | 86 | /* |
| @@ -92,7 +93,7 @@ void fastcall __sched mutex_lock(struct mutex *lock) | |||
| 92 | EXPORT_SYMBOL(mutex_lock); | 93 | EXPORT_SYMBOL(mutex_lock); |
| 93 | 94 | ||
| 94 | static void fastcall noinline __sched | 95 | static void fastcall noinline __sched |
| 95 | __mutex_unlock_slowpath(atomic_t *lock_count __IP_DECL__); | 96 | __mutex_unlock_slowpath(atomic_t *lock_count); |
| 96 | 97 | ||
| 97 | /*** | 98 | /*** |
| 98 | * mutex_unlock - release the mutex | 99 | * mutex_unlock - release the mutex |
| @@ -120,18 +121,17 @@ EXPORT_SYMBOL(mutex_unlock); | |||
| 120 | * Lock a mutex (possibly interruptible), slowpath: | 121 | * Lock a mutex (possibly interruptible), slowpath: |
| 121 | */ | 122 | */ |
| 122 | static inline int __sched | 123 | static inline int __sched |
| 123 | __mutex_lock_common(struct mutex *lock, long state __IP_DECL__) | 124 | __mutex_lock_common(struct mutex *lock, long state, unsigned int subclass) |
| 124 | { | 125 | { |
| 125 | struct task_struct *task = current; | 126 | struct task_struct *task = current; |
| 126 | struct mutex_waiter waiter; | 127 | struct mutex_waiter waiter; |
| 127 | unsigned int old_val; | 128 | unsigned int old_val; |
| 128 | unsigned long flags; | 129 | unsigned long flags; |
| 129 | 130 | ||
| 130 | debug_mutex_init_waiter(&waiter); | ||
| 131 | |||
| 132 | spin_lock_mutex(&lock->wait_lock, flags); | 131 | spin_lock_mutex(&lock->wait_lock, flags); |
| 133 | 132 | ||
| 134 | debug_mutex_add_waiter(lock, &waiter, task->thread_info, ip); | 133 | debug_mutex_lock_common(lock, &waiter); |
| 134 | debug_mutex_add_waiter(lock, &waiter, task->thread_info); | ||
| 135 | 135 | ||
| 136 | /* add waiting tasks to the end of the waitqueue (FIFO): */ | 136 | /* add waiting tasks to the end of the waitqueue (FIFO): */ |
| 137 | list_add_tail(&waiter.list, &lock->wait_list); | 137 | list_add_tail(&waiter.list, &lock->wait_list); |
| @@ -173,7 +173,7 @@ __mutex_lock_common(struct mutex *lock, long state __IP_DECL__) | |||
| 173 | 173 | ||
| 174 | /* got the lock - rejoice! */ | 174 | /* got the lock - rejoice! */ |
| 175 | mutex_remove_waiter(lock, &waiter, task->thread_info); | 175 | mutex_remove_waiter(lock, &waiter, task->thread_info); |
| 176 | debug_mutex_set_owner(lock, task->thread_info __IP__); | 176 | debug_mutex_set_owner(lock, task->thread_info); |
| 177 | 177 | ||
| 178 | /* set it to 0 if there are no waiters left: */ | 178 | /* set it to 0 if there are no waiters left: */ |
| 179 | if (likely(list_empty(&lock->wait_list))) | 179 | if (likely(list_empty(&lock->wait_list))) |
| @@ -183,32 +183,28 @@ __mutex_lock_common(struct mutex *lock, long state __IP_DECL__) | |||
| 183 | 183 | ||
| 184 | debug_mutex_free_waiter(&waiter); | 184 | debug_mutex_free_waiter(&waiter); |
| 185 | 185 | ||
| 186 | DEBUG_LOCKS_WARN_ON(list_empty(&lock->held_list)); | ||
| 187 | DEBUG_LOCKS_WARN_ON(lock->owner != task->thread_info); | ||
| 188 | |||
| 189 | return 0; | 186 | return 0; |
| 190 | } | 187 | } |
| 191 | 188 | ||
| 192 | static void fastcall noinline __sched | 189 | static void fastcall noinline __sched |
| 193 | __mutex_lock_slowpath(atomic_t *lock_count __IP_DECL__) | 190 | __mutex_lock_slowpath(atomic_t *lock_count) |
| 194 | { | 191 | { |
| 195 | struct mutex *lock = container_of(lock_count, struct mutex, count); | 192 | struct mutex *lock = container_of(lock_count, struct mutex, count); |
| 196 | 193 | ||
| 197 | __mutex_lock_common(lock, TASK_UNINTERRUPTIBLE __IP__); | 194 | __mutex_lock_common(lock, TASK_UNINTERRUPTIBLE, 0); |
| 198 | } | 195 | } |
| 199 | 196 | ||
| 200 | /* | 197 | /* |
| 201 | * Release the lock, slowpath: | 198 | * Release the lock, slowpath: |
| 202 | */ | 199 | */ |
| 203 | static fastcall noinline void | 200 | static fastcall inline void |
| 204 | __mutex_unlock_slowpath(atomic_t *lock_count __IP_DECL__) | 201 | __mutex_unlock_common_slowpath(atomic_t *lock_count) |
| 205 | { | 202 | { |
| 206 | struct mutex *lock = container_of(lock_count, struct mutex, count); | 203 | struct mutex *lock = container_of(lock_count, struct mutex, count); |
| 207 | unsigned long flags; | 204 | unsigned long flags; |
| 208 | 205 | ||
| 209 | DEBUG_LOCKS_WARN_ON(lock->owner != current_thread_info()); | ||
| 210 | |||
| 211 | spin_lock_mutex(&lock->wait_lock, flags); | 206 | spin_lock_mutex(&lock->wait_lock, flags); |
| 207 | debug_mutex_unlock(lock); | ||
| 212 | 208 | ||
| 213 | /* | 209 | /* |
| 214 | * some architectures leave the lock unlocked in the fastpath failure | 210 | * some architectures leave the lock unlocked in the fastpath failure |
| @@ -218,8 +214,6 @@ __mutex_unlock_slowpath(atomic_t *lock_count __IP_DECL__) | |||
| 218 | if (__mutex_slowpath_needs_to_unlock()) | 214 | if (__mutex_slowpath_needs_to_unlock()) |
| 219 | atomic_set(&lock->count, 1); | 215 | atomic_set(&lock->count, 1); |
| 220 | 216 | ||
| 221 | debug_mutex_unlock(lock); | ||
| 222 | |||
| 223 | if (!list_empty(&lock->wait_list)) { | 217 | if (!list_empty(&lock->wait_list)) { |
| 224 | /* get the first entry from the wait-list: */ | 218 | /* get the first entry from the wait-list: */ |
| 225 | struct mutex_waiter *waiter = | 219 | struct mutex_waiter *waiter = |
| @@ -237,11 +231,20 @@ __mutex_unlock_slowpath(atomic_t *lock_count __IP_DECL__) | |||
| 237 | } | 231 | } |
| 238 | 232 | ||
| 239 | /* | 233 | /* |
| 234 | * Release the lock, slowpath: | ||
| 235 | */ | ||
| 236 | static fastcall noinline void | ||
| 237 | __mutex_unlock_slowpath(atomic_t *lock_count) | ||
| 238 | { | ||
| 239 | __mutex_unlock_common_slowpath(lock_count); | ||
| 240 | } | ||
| 241 | |||
| 242 | /* | ||
| 240 | * Here come the less common (and hence less performance-critical) APIs: | 243 | * Here come the less common (and hence less performance-critical) APIs: |
| 241 | * mutex_lock_interruptible() and mutex_trylock(). | 244 | * mutex_lock_interruptible() and mutex_trylock(). |
| 242 | */ | 245 | */ |
| 243 | static int fastcall noinline __sched | 246 | static int fastcall noinline __sched |
| 244 | __mutex_lock_interruptible_slowpath(atomic_t *lock_count __IP_DECL__); | 247 | __mutex_lock_interruptible_slowpath(atomic_t *lock_count); |
| 245 | 248 | ||
| 246 | /*** | 249 | /*** |
| 247 | * mutex_lock_interruptible - acquire the mutex, interruptable | 250 | * mutex_lock_interruptible - acquire the mutex, interruptable |
| @@ -264,11 +267,11 @@ int fastcall __sched mutex_lock_interruptible(struct mutex *lock) | |||
| 264 | EXPORT_SYMBOL(mutex_lock_interruptible); | 267 | EXPORT_SYMBOL(mutex_lock_interruptible); |
| 265 | 268 | ||
| 266 | static int fastcall noinline __sched | 269 | static int fastcall noinline __sched |
| 267 | __mutex_lock_interruptible_slowpath(atomic_t *lock_count __IP_DECL__) | 270 | __mutex_lock_interruptible_slowpath(atomic_t *lock_count) |
| 268 | { | 271 | { |
| 269 | struct mutex *lock = container_of(lock_count, struct mutex, count); | 272 | struct mutex *lock = container_of(lock_count, struct mutex, count); |
| 270 | 273 | ||
| 271 | return __mutex_lock_common(lock, TASK_INTERRUPTIBLE __IP__); | 274 | return __mutex_lock_common(lock, TASK_INTERRUPTIBLE, 0); |
| 272 | } | 275 | } |
| 273 | 276 | ||
| 274 | /* | 277 | /* |
| @@ -285,7 +288,8 @@ static inline int __mutex_trylock_slowpath(atomic_t *lock_count) | |||
| 285 | 288 | ||
| 286 | prev = atomic_xchg(&lock->count, -1); | 289 | prev = atomic_xchg(&lock->count, -1); |
| 287 | if (likely(prev == 1)) | 290 | if (likely(prev == 1)) |
| 288 | debug_mutex_set_owner(lock, current_thread_info() __RET_IP__); | 291 | debug_mutex_set_owner(lock, current_thread_info()); |
| 292 | |||
| 289 | /* Set it back to 0 if there are no waiters: */ | 293 | /* Set it back to 0 if there are no waiters: */ |
| 290 | if (likely(list_empty(&lock->wait_list))) | 294 | if (likely(list_empty(&lock->wait_list))) |
| 291 | atomic_set(&lock->count, 0); | 295 | atomic_set(&lock->count, 0); |
diff --git a/kernel/mutex.h b/kernel/mutex.h index 7e1ed48d1a6c..aeb2d916aa0e 100644 --- a/kernel/mutex.h +++ b/kernel/mutex.h | |||
| @@ -16,22 +16,15 @@ | |||
| 16 | #define mutex_remove_waiter(lock, waiter, ti) \ | 16 | #define mutex_remove_waiter(lock, waiter, ti) \ |
| 17 | __list_del((waiter)->list.prev, (waiter)->list.next) | 17 | __list_del((waiter)->list.prev, (waiter)->list.next) |
| 18 | 18 | ||
| 19 | #define DEBUG_LOCKS_WARN_ON(c) do { } while (0) | ||
| 20 | #define debug_mutex_set_owner(lock, new_owner) do { } while (0) | 19 | #define debug_mutex_set_owner(lock, new_owner) do { } while (0) |
| 21 | #define debug_mutex_clear_owner(lock) do { } while (0) | 20 | #define debug_mutex_clear_owner(lock) do { } while (0) |
| 22 | #define debug_mutex_init_waiter(waiter) do { } while (0) | ||
| 23 | #define debug_mutex_wake_waiter(lock, waiter) do { } while (0) | 21 | #define debug_mutex_wake_waiter(lock, waiter) do { } while (0) |
| 24 | #define debug_mutex_free_waiter(waiter) do { } while (0) | 22 | #define debug_mutex_free_waiter(waiter) do { } while (0) |
| 25 | #define debug_mutex_add_waiter(lock, waiter, ti, ip) do { } while (0) | 23 | #define debug_mutex_add_waiter(lock, waiter, ti) do { } while (0) |
| 26 | #define debug_mutex_unlock(lock) do { } while (0) | 24 | #define debug_mutex_unlock(lock) do { } while (0) |
| 27 | #define debug_mutex_init(lock, name) do { } while (0) | 25 | #define debug_mutex_init(lock, name) do { } while (0) |
| 28 | 26 | ||
| 29 | /* | 27 | static inline void |
| 30 | * Return-address parameters/declarations. They are very useful for | 28 | debug_mutex_lock_common(struct mutex *lock, struct mutex_waiter *waiter) |
| 31 | * debugging, but add overhead in the !DEBUG case - so we go the | 29 | { |
| 32 | * trouble of using this not too elegant but zero-cost solution: | 30 | } |
| 33 | */ | ||
| 34 | #define __IP_DECL__ | ||
| 35 | #define __IP__ | ||
| 36 | #define __RET_IP__ | ||
| 37 | |||
diff --git a/kernel/rtmutex-debug.c b/kernel/rtmutex-debug.c index 4aa8a2c9f453..353a853bc390 100644 --- a/kernel/rtmutex-debug.c +++ b/kernel/rtmutex-debug.c | |||
| @@ -26,6 +26,7 @@ | |||
| 26 | #include <linux/interrupt.h> | 26 | #include <linux/interrupt.h> |
| 27 | #include <linux/plist.h> | 27 | #include <linux/plist.h> |
| 28 | #include <linux/fs.h> | 28 | #include <linux/fs.h> |
| 29 | #include <linux/debug_locks.h> | ||
| 29 | 30 | ||
| 30 | #include "rtmutex_common.h" | 31 | #include "rtmutex_common.h" |
| 31 | 32 | ||
| @@ -45,8 +46,6 @@ do { \ | |||
| 45 | console_verbose(); \ | 46 | console_verbose(); \ |
| 46 | if (spin_is_locked(¤t->pi_lock)) \ | 47 | if (spin_is_locked(¤t->pi_lock)) \ |
| 47 | spin_unlock(¤t->pi_lock); \ | 48 | spin_unlock(¤t->pi_lock); \ |
| 48 | if (spin_is_locked(¤t->held_list_lock)) \ | ||
| 49 | spin_unlock(¤t->held_list_lock); \ | ||
| 50 | } \ | 49 | } \ |
| 51 | } while (0) | 50 | } while (0) |
| 52 | 51 | ||
| @@ -105,14 +104,6 @@ static void printk_task(task_t *p) | |||
| 105 | printk("<none>"); | 104 | printk("<none>"); |
| 106 | } | 105 | } |
| 107 | 106 | ||
| 108 | static void printk_task_short(task_t *p) | ||
| 109 | { | ||
| 110 | if (p) | ||
| 111 | printk("%s/%d [%p, %3d]", p->comm, p->pid, p, p->prio); | ||
| 112 | else | ||
| 113 | printk("<none>"); | ||
| 114 | } | ||
| 115 | |||
| 116 | static void printk_lock(struct rt_mutex *lock, int print_owner) | 107 | static void printk_lock(struct rt_mutex *lock, int print_owner) |
| 117 | { | 108 | { |
| 118 | if (lock->name) | 109 | if (lock->name) |
| @@ -128,222 +119,6 @@ static void printk_lock(struct rt_mutex *lock, int print_owner) | |||
| 128 | printk_task(rt_mutex_owner(lock)); | 119 | printk_task(rt_mutex_owner(lock)); |
| 129 | printk("\n"); | 120 | printk("\n"); |
| 130 | } | 121 | } |
| 131 | if (rt_mutex_owner(lock)) { | ||
| 132 | printk("... acquired at: "); | ||
| 133 | print_symbol("%s\n", lock->acquire_ip); | ||
| 134 | } | ||
| 135 | } | ||
| 136 | |||
| 137 | static void printk_waiter(struct rt_mutex_waiter *w) | ||
| 138 | { | ||
| 139 | printk("-------------------------\n"); | ||
| 140 | printk("| waiter struct %p:\n", w); | ||
| 141 | printk("| w->list_entry: [DP:%p/%p|SP:%p/%p|PRI:%d]\n", | ||
| 142 | w->list_entry.plist.prio_list.prev, w->list_entry.plist.prio_list.next, | ||
| 143 | w->list_entry.plist.node_list.prev, w->list_entry.plist.node_list.next, | ||
| 144 | w->list_entry.prio); | ||
| 145 | printk("| w->pi_list_entry: [DP:%p/%p|SP:%p/%p|PRI:%d]\n", | ||
| 146 | w->pi_list_entry.plist.prio_list.prev, w->pi_list_entry.plist.prio_list.next, | ||
| 147 | w->pi_list_entry.plist.node_list.prev, w->pi_list_entry.plist.node_list.next, | ||
| 148 | w->pi_list_entry.prio); | ||
| 149 | printk("\n| lock:\n"); | ||
| 150 | printk_lock(w->lock, 1); | ||
| 151 | printk("| w->ti->task:\n"); | ||
| 152 | printk_task(w->task); | ||
| 153 | printk("| blocked at: "); | ||
| 154 | print_symbol("%s\n", w->ip); | ||
| 155 | printk("-------------------------\n"); | ||
| 156 | } | ||
| 157 | |||
| 158 | static void show_task_locks(task_t *p) | ||
| 159 | { | ||
| 160 | switch (p->state) { | ||
| 161 | case TASK_RUNNING: printk("R"); break; | ||
| 162 | case TASK_INTERRUPTIBLE: printk("S"); break; | ||
| 163 | case TASK_UNINTERRUPTIBLE: printk("D"); break; | ||
| 164 | case TASK_STOPPED: printk("T"); break; | ||
| 165 | case EXIT_ZOMBIE: printk("Z"); break; | ||
| 166 | case EXIT_DEAD: printk("X"); break; | ||
| 167 | default: printk("?"); break; | ||
| 168 | } | ||
| 169 | printk_task(p); | ||
| 170 | if (p->pi_blocked_on) { | ||
| 171 | struct rt_mutex *lock = p->pi_blocked_on->lock; | ||
| 172 | |||
| 173 | printk(" blocked on:"); | ||
| 174 | printk_lock(lock, 1); | ||
| 175 | } else | ||
| 176 | printk(" (not blocked)\n"); | ||
| 177 | } | ||
| 178 | |||
| 179 | void rt_mutex_show_held_locks(task_t *task, int verbose) | ||
| 180 | { | ||
| 181 | struct list_head *curr, *cursor = NULL; | ||
| 182 | struct rt_mutex *lock; | ||
| 183 | task_t *t; | ||
| 184 | unsigned long flags; | ||
| 185 | int count = 0; | ||
| 186 | |||
| 187 | if (!rt_trace_on) | ||
| 188 | return; | ||
| 189 | |||
| 190 | if (verbose) { | ||
| 191 | printk("------------------------------\n"); | ||
| 192 | printk("| showing all locks held by: | ("); | ||
| 193 | printk_task_short(task); | ||
| 194 | printk("):\n"); | ||
| 195 | printk("------------------------------\n"); | ||
| 196 | } | ||
| 197 | |||
| 198 | next: | ||
| 199 | spin_lock_irqsave(&task->held_list_lock, flags); | ||
| 200 | list_for_each(curr, &task->held_list_head) { | ||
| 201 | if (cursor && curr != cursor) | ||
| 202 | continue; | ||
| 203 | lock = list_entry(curr, struct rt_mutex, held_list_entry); | ||
| 204 | t = rt_mutex_owner(lock); | ||
| 205 | WARN_ON(t != task); | ||
| 206 | count++; | ||
| 207 | cursor = curr->next; | ||
| 208 | spin_unlock_irqrestore(&task->held_list_lock, flags); | ||
| 209 | |||
| 210 | printk("\n#%03d: ", count); | ||
| 211 | printk_lock(lock, 0); | ||
| 212 | goto next; | ||
| 213 | } | ||
| 214 | spin_unlock_irqrestore(&task->held_list_lock, flags); | ||
| 215 | |||
| 216 | printk("\n"); | ||
| 217 | } | ||
| 218 | |||
| 219 | void rt_mutex_show_all_locks(void) | ||
| 220 | { | ||
| 221 | task_t *g, *p; | ||
| 222 | int count = 10; | ||
| 223 | int unlock = 1; | ||
| 224 | |||
| 225 | printk("\n"); | ||
| 226 | printk("----------------------\n"); | ||
| 227 | printk("| showing all tasks: |\n"); | ||
| 228 | printk("----------------------\n"); | ||
| 229 | |||
| 230 | /* | ||
| 231 | * Here we try to get the tasklist_lock as hard as possible, | ||
| 232 | * if not successful after 2 seconds we ignore it (but keep | ||
| 233 | * trying). This is to enable a debug printout even if a | ||
| 234 | * tasklist_lock-holding task deadlocks or crashes. | ||
| 235 | */ | ||
| 236 | retry: | ||
| 237 | if (!read_trylock(&tasklist_lock)) { | ||
| 238 | if (count == 10) | ||
| 239 | printk("hm, tasklist_lock locked, retrying... "); | ||
| 240 | if (count) { | ||
| 241 | count--; | ||
| 242 | printk(" #%d", 10-count); | ||
| 243 | mdelay(200); | ||
| 244 | goto retry; | ||
| 245 | } | ||
| 246 | printk(" ignoring it.\n"); | ||
| 247 | unlock = 0; | ||
| 248 | } | ||
| 249 | if (count != 10) | ||
| 250 | printk(" locked it.\n"); | ||
| 251 | |||
| 252 | do_each_thread(g, p) { | ||
| 253 | show_task_locks(p); | ||
| 254 | if (!unlock) | ||
| 255 | if (read_trylock(&tasklist_lock)) | ||
| 256 | unlock = 1; | ||
| 257 | } while_each_thread(g, p); | ||
| 258 | |||
| 259 | printk("\n"); | ||
| 260 | |||
| 261 | printk("-----------------------------------------\n"); | ||
| 262 | printk("| showing all locks held in the system: |\n"); | ||
| 263 | printk("-----------------------------------------\n"); | ||
| 264 | |||
| 265 | do_each_thread(g, p) { | ||
| 266 | rt_mutex_show_held_locks(p, 0); | ||
| 267 | if (!unlock) | ||
| 268 | if (read_trylock(&tasklist_lock)) | ||
| 269 | unlock = 1; | ||
| 270 | } while_each_thread(g, p); | ||
| 271 | |||
| 272 | |||
| 273 | printk("=============================================\n\n"); | ||
| 274 | |||
| 275 | if (unlock) | ||
| 276 | read_unlock(&tasklist_lock); | ||
| 277 | } | ||
| 278 | |||
| 279 | void rt_mutex_debug_check_no_locks_held(task_t *task) | ||
| 280 | { | ||
| 281 | struct rt_mutex_waiter *w; | ||
| 282 | struct list_head *curr; | ||
| 283 | struct rt_mutex *lock; | ||
| 284 | |||
| 285 | if (!rt_trace_on) | ||
| 286 | return; | ||
| 287 | if (!rt_prio(task->normal_prio) && rt_prio(task->prio)) { | ||
| 288 | printk("BUG: PI priority boost leaked!\n"); | ||
| 289 | printk_task(task); | ||
| 290 | printk("\n"); | ||
| 291 | } | ||
| 292 | if (list_empty(&task->held_list_head)) | ||
| 293 | return; | ||
| 294 | |||
| 295 | spin_lock(&task->pi_lock); | ||
| 296 | plist_for_each_entry(w, &task->pi_waiters, pi_list_entry) { | ||
| 297 | TRACE_OFF(); | ||
| 298 | |||
| 299 | printk("hm, PI interest held at exit time? Task:\n"); | ||
| 300 | printk_task(task); | ||
| 301 | printk_waiter(w); | ||
| 302 | return; | ||
| 303 | } | ||
| 304 | spin_unlock(&task->pi_lock); | ||
| 305 | |||
| 306 | list_for_each(curr, &task->held_list_head) { | ||
| 307 | lock = list_entry(curr, struct rt_mutex, held_list_entry); | ||
| 308 | |||
| 309 | printk("BUG: %s/%d, lock held at task exit time!\n", | ||
| 310 | task->comm, task->pid); | ||
| 311 | printk_lock(lock, 1); | ||
| 312 | if (rt_mutex_owner(lock) != task) | ||
| 313 | printk("exiting task is not even the owner??\n"); | ||
| 314 | } | ||
| 315 | } | ||
| 316 | |||
| 317 | int rt_mutex_debug_check_no_locks_freed(const void *from, unsigned long len) | ||
| 318 | { | ||
| 319 | const void *to = from + len; | ||
| 320 | struct list_head *curr; | ||
| 321 | struct rt_mutex *lock; | ||
| 322 | unsigned long flags; | ||
| 323 | void *lock_addr; | ||
| 324 | |||
| 325 | if (!rt_trace_on) | ||
| 326 | return 0; | ||
| 327 | |||
| 328 | spin_lock_irqsave(¤t->held_list_lock, flags); | ||
| 329 | list_for_each(curr, ¤t->held_list_head) { | ||
| 330 | lock = list_entry(curr, struct rt_mutex, held_list_entry); | ||
| 331 | lock_addr = lock; | ||
| 332 | if (lock_addr < from || lock_addr >= to) | ||
| 333 | continue; | ||
| 334 | TRACE_OFF(); | ||
| 335 | |||
| 336 | printk("BUG: %s/%d, active lock [%p(%p-%p)] freed!\n", | ||
| 337 | current->comm, current->pid, lock, from, to); | ||
| 338 | dump_stack(); | ||
| 339 | printk_lock(lock, 1); | ||
| 340 | if (rt_mutex_owner(lock) != current) | ||
| 341 | printk("freeing task is not even the owner??\n"); | ||
| 342 | return 1; | ||
| 343 | } | ||
| 344 | spin_unlock_irqrestore(¤t->held_list_lock, flags); | ||
| 345 | |||
| 346 | return 0; | ||
| 347 | } | 122 | } |
| 348 | 123 | ||
| 349 | void rt_mutex_debug_task_free(struct task_struct *task) | 124 | void rt_mutex_debug_task_free(struct task_struct *task) |
| @@ -395,85 +170,41 @@ void debug_rt_mutex_print_deadlock(struct rt_mutex_waiter *waiter) | |||
| 395 | current->comm, current->pid); | 170 | current->comm, current->pid); |
| 396 | printk_lock(waiter->lock, 1); | 171 | printk_lock(waiter->lock, 1); |
| 397 | 172 | ||
| 398 | printk("... trying at: "); | ||
| 399 | print_symbol("%s\n", waiter->ip); | ||
| 400 | |||
| 401 | printk("\n2) %s/%d is blocked on this lock:\n", task->comm, task->pid); | 173 | printk("\n2) %s/%d is blocked on this lock:\n", task->comm, task->pid); |
| 402 | printk_lock(waiter->deadlock_lock, 1); | 174 | printk_lock(waiter->deadlock_lock, 1); |
| 403 | 175 | ||
| 404 | rt_mutex_show_held_locks(current, 1); | 176 | debug_show_held_locks(current); |
| 405 | rt_mutex_show_held_locks(task, 1); | 177 | debug_show_held_locks(task); |
| 406 | 178 | ||
| 407 | printk("\n%s/%d's [blocked] stackdump:\n\n", task->comm, task->pid); | 179 | printk("\n%s/%d's [blocked] stackdump:\n\n", task->comm, task->pid); |
| 408 | show_stack(task, NULL); | 180 | show_stack(task, NULL); |
| 409 | printk("\n%s/%d's [current] stackdump:\n\n", | 181 | printk("\n%s/%d's [current] stackdump:\n\n", |
| 410 | current->comm, current->pid); | 182 | current->comm, current->pid); |
| 411 | dump_stack(); | 183 | dump_stack(); |
| 412 | rt_mutex_show_all_locks(); | 184 | debug_show_all_locks(); |
| 185 | |||
| 413 | printk("[ turning off deadlock detection." | 186 | printk("[ turning off deadlock detection." |
| 414 | "Please report this trace. ]\n\n"); | 187 | "Please report this trace. ]\n\n"); |
| 415 | local_irq_disable(); | 188 | local_irq_disable(); |
| 416 | } | 189 | } |
| 417 | 190 | ||
| 418 | void debug_rt_mutex_lock(struct rt_mutex *lock __IP_DECL__) | 191 | void debug_rt_mutex_lock(struct rt_mutex *lock) |
| 419 | { | 192 | { |
| 420 | unsigned long flags; | ||
| 421 | |||
| 422 | if (rt_trace_on) { | ||
| 423 | TRACE_WARN_ON_LOCKED(!list_empty(&lock->held_list_entry)); | ||
| 424 | |||
| 425 | spin_lock_irqsave(¤t->held_list_lock, flags); | ||
| 426 | list_add_tail(&lock->held_list_entry, ¤t->held_list_head); | ||
| 427 | spin_unlock_irqrestore(¤t->held_list_lock, flags); | ||
| 428 | |||
| 429 | lock->acquire_ip = ip; | ||
| 430 | } | ||
| 431 | } | 193 | } |
| 432 | 194 | ||
| 433 | void debug_rt_mutex_unlock(struct rt_mutex *lock) | 195 | void debug_rt_mutex_unlock(struct rt_mutex *lock) |
| 434 | { | 196 | { |
| 435 | unsigned long flags; | 197 | TRACE_WARN_ON_LOCKED(rt_mutex_owner(lock) != current); |
| 436 | |||
| 437 | if (rt_trace_on) { | ||
| 438 | TRACE_WARN_ON_LOCKED(rt_mutex_owner(lock) != current); | ||
| 439 | TRACE_WARN_ON_LOCKED(list_empty(&lock->held_list_entry)); | ||
| 440 | |||
| 441 | spin_lock_irqsave(¤t->held_list_lock, flags); | ||
| 442 | list_del_init(&lock->held_list_entry); | ||
| 443 | spin_unlock_irqrestore(¤t->held_list_lock, flags); | ||
| 444 | } | ||
| 445 | } | 198 | } |
| 446 | 199 | ||
| 447 | void debug_rt_mutex_proxy_lock(struct rt_mutex *lock, | 200 | void |
| 448 | struct task_struct *powner __IP_DECL__) | 201 | debug_rt_mutex_proxy_lock(struct rt_mutex *lock, struct task_struct *powner) |
| 449 | { | 202 | { |
| 450 | unsigned long flags; | ||
| 451 | |||
| 452 | if (rt_trace_on) { | ||
| 453 | TRACE_WARN_ON_LOCKED(!list_empty(&lock->held_list_entry)); | ||
| 454 | |||
| 455 | spin_lock_irqsave(&powner->held_list_lock, flags); | ||
| 456 | list_add_tail(&lock->held_list_entry, &powner->held_list_head); | ||
| 457 | spin_unlock_irqrestore(&powner->held_list_lock, flags); | ||
| 458 | |||
| 459 | lock->acquire_ip = ip; | ||
| 460 | } | ||
| 461 | } | 203 | } |
| 462 | 204 | ||
| 463 | void debug_rt_mutex_proxy_unlock(struct rt_mutex *lock) | 205 | void debug_rt_mutex_proxy_unlock(struct rt_mutex *lock) |
| 464 | { | 206 | { |
| 465 | unsigned long flags; | 207 | TRACE_WARN_ON_LOCKED(!rt_mutex_owner(lock)); |
| 466 | |||
| 467 | if (rt_trace_on) { | ||
| 468 | struct task_struct *owner = rt_mutex_owner(lock); | ||
| 469 | |||
| 470 | TRACE_WARN_ON_LOCKED(!owner); | ||
| 471 | TRACE_WARN_ON_LOCKED(list_empty(&lock->held_list_entry)); | ||
| 472 | |||
| 473 | spin_lock_irqsave(&owner->held_list_lock, flags); | ||
| 474 | list_del_init(&lock->held_list_entry); | ||
| 475 | spin_unlock_irqrestore(&owner->held_list_lock, flags); | ||
| 476 | } | ||
| 477 | } | 208 | } |
| 478 | 209 | ||
| 479 | void debug_rt_mutex_init_waiter(struct rt_mutex_waiter *waiter) | 210 | void debug_rt_mutex_init_waiter(struct rt_mutex_waiter *waiter) |
| @@ -493,14 +224,11 @@ void debug_rt_mutex_free_waiter(struct rt_mutex_waiter *waiter) | |||
| 493 | 224 | ||
| 494 | void debug_rt_mutex_init(struct rt_mutex *lock, const char *name) | 225 | void debug_rt_mutex_init(struct rt_mutex *lock, const char *name) |
| 495 | { | 226 | { |
| 496 | void *addr = lock; | 227 | /* |
| 497 | 228 | * Make sure we are not reinitializing a held lock: | |
| 498 | if (rt_trace_on) { | 229 | */ |
| 499 | rt_mutex_debug_check_no_locks_freed(addr, | 230 | debug_check_no_locks_freed((void *)lock, sizeof(*lock)); |
| 500 | sizeof(struct rt_mutex)); | 231 | lock->name = name; |
| 501 | INIT_LIST_HEAD(&lock->held_list_entry); | ||
| 502 | lock->name = name; | ||
| 503 | } | ||
| 504 | } | 232 | } |
| 505 | 233 | ||
| 506 | void rt_mutex_deadlock_account_lock(struct rt_mutex *lock, task_t *task) | 234 | void rt_mutex_deadlock_account_lock(struct rt_mutex *lock, task_t *task) |
diff --git a/kernel/rtmutex-debug.h b/kernel/rtmutex-debug.h index 7612fbc62d70..14193d596d78 100644 --- a/kernel/rtmutex-debug.h +++ b/kernel/rtmutex-debug.h | |||
| @@ -9,20 +9,16 @@ | |||
| 9 | * This file contains macros used solely by rtmutex.c. Debug version. | 9 | * This file contains macros used solely by rtmutex.c. Debug version. |
| 10 | */ | 10 | */ |
| 11 | 11 | ||
| 12 | #define __IP_DECL__ , unsigned long ip | ||
| 13 | #define __IP__ , ip | ||
| 14 | #define __RET_IP__ , (unsigned long)__builtin_return_address(0) | ||
| 15 | |||
| 16 | extern void | 12 | extern void |
| 17 | rt_mutex_deadlock_account_lock(struct rt_mutex *lock, struct task_struct *task); | 13 | rt_mutex_deadlock_account_lock(struct rt_mutex *lock, struct task_struct *task); |
| 18 | extern void rt_mutex_deadlock_account_unlock(struct task_struct *task); | 14 | extern void rt_mutex_deadlock_account_unlock(struct task_struct *task); |
| 19 | extern void debug_rt_mutex_init_waiter(struct rt_mutex_waiter *waiter); | 15 | extern void debug_rt_mutex_init_waiter(struct rt_mutex_waiter *waiter); |
| 20 | extern void debug_rt_mutex_free_waiter(struct rt_mutex_waiter *waiter); | 16 | extern void debug_rt_mutex_free_waiter(struct rt_mutex_waiter *waiter); |
| 21 | extern void debug_rt_mutex_init(struct rt_mutex *lock, const char *name); | 17 | extern void debug_rt_mutex_init(struct rt_mutex *lock, const char *name); |
| 22 | extern void debug_rt_mutex_lock(struct rt_mutex *lock __IP_DECL__); | 18 | extern void debug_rt_mutex_lock(struct rt_mutex *lock); |
| 23 | extern void debug_rt_mutex_unlock(struct rt_mutex *lock); | 19 | extern void debug_rt_mutex_unlock(struct rt_mutex *lock); |
| 24 | extern void debug_rt_mutex_proxy_lock(struct rt_mutex *lock, | 20 | extern void debug_rt_mutex_proxy_lock(struct rt_mutex *lock, |
| 25 | struct task_struct *powner __IP_DECL__); | 21 | struct task_struct *powner); |
| 26 | extern void debug_rt_mutex_proxy_unlock(struct rt_mutex *lock); | 22 | extern void debug_rt_mutex_proxy_unlock(struct rt_mutex *lock); |
| 27 | extern void debug_rt_mutex_deadlock(int detect, struct rt_mutex_waiter *waiter, | 23 | extern void debug_rt_mutex_deadlock(int detect, struct rt_mutex_waiter *waiter, |
| 28 | struct rt_mutex *lock); | 24 | struct rt_mutex *lock); |
diff --git a/kernel/rtmutex.c b/kernel/rtmutex.c index 45d61016da57..91b699aa658b 100644 --- a/kernel/rtmutex.c +++ b/kernel/rtmutex.c | |||
| @@ -161,8 +161,7 @@ static int rt_mutex_adjust_prio_chain(task_t *task, | |||
| 161 | int deadlock_detect, | 161 | int deadlock_detect, |
| 162 | struct rt_mutex *orig_lock, | 162 | struct rt_mutex *orig_lock, |
| 163 | struct rt_mutex_waiter *orig_waiter, | 163 | struct rt_mutex_waiter *orig_waiter, |
| 164 | struct task_struct *top_task | 164 | struct task_struct *top_task) |
| 165 | __IP_DECL__) | ||
| 166 | { | 165 | { |
| 167 | struct rt_mutex *lock; | 166 | struct rt_mutex *lock; |
| 168 | struct rt_mutex_waiter *waiter, *top_waiter = orig_waiter; | 167 | struct rt_mutex_waiter *waiter, *top_waiter = orig_waiter; |
| @@ -357,7 +356,7 @@ static inline int try_to_steal_lock(struct rt_mutex *lock) | |||
| 357 | * | 356 | * |
| 358 | * Must be called with lock->wait_lock held. | 357 | * Must be called with lock->wait_lock held. |
| 359 | */ | 358 | */ |
| 360 | static int try_to_take_rt_mutex(struct rt_mutex *lock __IP_DECL__) | 359 | static int try_to_take_rt_mutex(struct rt_mutex *lock) |
| 361 | { | 360 | { |
| 362 | /* | 361 | /* |
| 363 | * We have to be careful here if the atomic speedups are | 362 | * We have to be careful here if the atomic speedups are |
| @@ -384,7 +383,7 @@ static int try_to_take_rt_mutex(struct rt_mutex *lock __IP_DECL__) | |||
| 384 | return 0; | 383 | return 0; |
| 385 | 384 | ||
| 386 | /* We got the lock. */ | 385 | /* We got the lock. */ |
| 387 | debug_rt_mutex_lock(lock __IP__); | 386 | debug_rt_mutex_lock(lock); |
| 388 | 387 | ||
| 389 | rt_mutex_set_owner(lock, current, 0); | 388 | rt_mutex_set_owner(lock, current, 0); |
| 390 | 389 | ||
| @@ -402,8 +401,7 @@ static int try_to_take_rt_mutex(struct rt_mutex *lock __IP_DECL__) | |||
| 402 | */ | 401 | */ |
| 403 | static int task_blocks_on_rt_mutex(struct rt_mutex *lock, | 402 | static int task_blocks_on_rt_mutex(struct rt_mutex *lock, |
| 404 | struct rt_mutex_waiter *waiter, | 403 | struct rt_mutex_waiter *waiter, |
| 405 | int detect_deadlock | 404 | int detect_deadlock) |
| 406 | __IP_DECL__) | ||
| 407 | { | 405 | { |
| 408 | struct rt_mutex_waiter *top_waiter = waiter; | 406 | struct rt_mutex_waiter *top_waiter = waiter; |
| 409 | task_t *owner = rt_mutex_owner(lock); | 407 | task_t *owner = rt_mutex_owner(lock); |
| @@ -454,7 +452,7 @@ static int task_blocks_on_rt_mutex(struct rt_mutex *lock, | |||
| 454 | spin_unlock(&lock->wait_lock); | 452 | spin_unlock(&lock->wait_lock); |
| 455 | 453 | ||
| 456 | res = rt_mutex_adjust_prio_chain(owner, detect_deadlock, lock, waiter, | 454 | res = rt_mutex_adjust_prio_chain(owner, detect_deadlock, lock, waiter, |
| 457 | current __IP__); | 455 | current); |
| 458 | 456 | ||
| 459 | spin_lock(&lock->wait_lock); | 457 | spin_lock(&lock->wait_lock); |
| 460 | 458 | ||
| @@ -526,7 +524,7 @@ static void wakeup_next_waiter(struct rt_mutex *lock) | |||
| 526 | * Must be called with lock->wait_lock held | 524 | * Must be called with lock->wait_lock held |
| 527 | */ | 525 | */ |
| 528 | static void remove_waiter(struct rt_mutex *lock, | 526 | static void remove_waiter(struct rt_mutex *lock, |
| 529 | struct rt_mutex_waiter *waiter __IP_DECL__) | 527 | struct rt_mutex_waiter *waiter) |
| 530 | { | 528 | { |
| 531 | int first = (waiter == rt_mutex_top_waiter(lock)); | 529 | int first = (waiter == rt_mutex_top_waiter(lock)); |
| 532 | int boost = 0; | 530 | int boost = 0; |
| @@ -568,7 +566,7 @@ static void remove_waiter(struct rt_mutex *lock, | |||
| 568 | 566 | ||
| 569 | spin_unlock(&lock->wait_lock); | 567 | spin_unlock(&lock->wait_lock); |
| 570 | 568 | ||
| 571 | rt_mutex_adjust_prio_chain(owner, 0, lock, NULL, current __IP__); | 569 | rt_mutex_adjust_prio_chain(owner, 0, lock, NULL, current); |
| 572 | 570 | ||
| 573 | spin_lock(&lock->wait_lock); | 571 | spin_lock(&lock->wait_lock); |
| 574 | } | 572 | } |
| @@ -595,7 +593,7 @@ void rt_mutex_adjust_pi(struct task_struct *task) | |||
| 595 | get_task_struct(task); | 593 | get_task_struct(task); |
| 596 | spin_unlock_irqrestore(&task->pi_lock, flags); | 594 | spin_unlock_irqrestore(&task->pi_lock, flags); |
| 597 | 595 | ||
| 598 | rt_mutex_adjust_prio_chain(task, 0, NULL, NULL, task __RET_IP__); | 596 | rt_mutex_adjust_prio_chain(task, 0, NULL, NULL, task); |
| 599 | } | 597 | } |
| 600 | 598 | ||
| 601 | /* | 599 | /* |
| @@ -604,7 +602,7 @@ void rt_mutex_adjust_pi(struct task_struct *task) | |||
| 604 | static int __sched | 602 | static int __sched |
| 605 | rt_mutex_slowlock(struct rt_mutex *lock, int state, | 603 | rt_mutex_slowlock(struct rt_mutex *lock, int state, |
| 606 | struct hrtimer_sleeper *timeout, | 604 | struct hrtimer_sleeper *timeout, |
| 607 | int detect_deadlock __IP_DECL__) | 605 | int detect_deadlock) |
| 608 | { | 606 | { |
| 609 | struct rt_mutex_waiter waiter; | 607 | struct rt_mutex_waiter waiter; |
| 610 | int ret = 0; | 608 | int ret = 0; |
| @@ -615,7 +613,7 @@ rt_mutex_slowlock(struct rt_mutex *lock, int state, | |||
| 615 | spin_lock(&lock->wait_lock); | 613 | spin_lock(&lock->wait_lock); |
| 616 | 614 | ||
| 617 | /* Try to acquire the lock again: */ | 615 | /* Try to acquire the lock again: */ |
| 618 | if (try_to_take_rt_mutex(lock __IP__)) { | 616 | if (try_to_take_rt_mutex(lock)) { |
| 619 | spin_unlock(&lock->wait_lock); | 617 | spin_unlock(&lock->wait_lock); |
| 620 | return 0; | 618 | return 0; |
| 621 | } | 619 | } |
| @@ -629,7 +627,7 @@ rt_mutex_slowlock(struct rt_mutex *lock, int state, | |||
| 629 | 627 | ||
| 630 | for (;;) { | 628 | for (;;) { |
| 631 | /* Try to acquire the lock: */ | 629 | /* Try to acquire the lock: */ |
| 632 | if (try_to_take_rt_mutex(lock __IP__)) | 630 | if (try_to_take_rt_mutex(lock)) |
| 633 | break; | 631 | break; |
| 634 | 632 | ||
| 635 | /* | 633 | /* |
| @@ -653,7 +651,7 @@ rt_mutex_slowlock(struct rt_mutex *lock, int state, | |||
| 653 | */ | 651 | */ |
| 654 | if (!waiter.task) { | 652 | if (!waiter.task) { |
| 655 | ret = task_blocks_on_rt_mutex(lock, &waiter, | 653 | ret = task_blocks_on_rt_mutex(lock, &waiter, |
| 656 | detect_deadlock __IP__); | 654 | detect_deadlock); |
| 657 | /* | 655 | /* |
| 658 | * If we got woken up by the owner then start loop | 656 | * If we got woken up by the owner then start loop |
| 659 | * all over without going into schedule to try | 657 | * all over without going into schedule to try |
| @@ -680,7 +678,7 @@ rt_mutex_slowlock(struct rt_mutex *lock, int state, | |||
| 680 | set_current_state(TASK_RUNNING); | 678 | set_current_state(TASK_RUNNING); |
| 681 | 679 | ||
| 682 | if (unlikely(waiter.task)) | 680 | if (unlikely(waiter.task)) |
| 683 | remove_waiter(lock, &waiter __IP__); | 681 | remove_waiter(lock, &waiter); |
| 684 | 682 | ||
| 685 | /* | 683 | /* |
| 686 | * try_to_take_rt_mutex() sets the waiter bit | 684 | * try_to_take_rt_mutex() sets the waiter bit |
| @@ -711,7 +709,7 @@ rt_mutex_slowlock(struct rt_mutex *lock, int state, | |||
| 711 | * Slow path try-lock function: | 709 | * Slow path try-lock function: |
| 712 | */ | 710 | */ |
| 713 | static inline int | 711 | static inline int |
| 714 | rt_mutex_slowtrylock(struct rt_mutex *lock __IP_DECL__) | 712 | rt_mutex_slowtrylock(struct rt_mutex *lock) |
| 715 | { | 713 | { |
| 716 | int ret = 0; | 714 | int ret = 0; |
| 717 | 715 | ||
| @@ -719,7 +717,7 @@ rt_mutex_slowtrylock(struct rt_mutex *lock __IP_DECL__) | |||
| 719 | 717 | ||
| 720 | if (likely(rt_mutex_owner(lock) != current)) { | 718 | if (likely(rt_mutex_owner(lock) != current)) { |
| 721 | 719 | ||
| 722 | ret = try_to_take_rt_mutex(lock __IP__); | 720 | ret = try_to_take_rt_mutex(lock); |
| 723 | /* | 721 | /* |
| 724 | * try_to_take_rt_mutex() sets the lock waiters | 722 | * try_to_take_rt_mutex() sets the lock waiters |
| 725 | * bit unconditionally. Clean this up. | 723 | * bit unconditionally. Clean this up. |
| @@ -769,13 +767,13 @@ rt_mutex_fastlock(struct rt_mutex *lock, int state, | |||
| 769 | int detect_deadlock, | 767 | int detect_deadlock, |
| 770 | int (*slowfn)(struct rt_mutex *lock, int state, | 768 | int (*slowfn)(struct rt_mutex *lock, int state, |
| 771 | struct hrtimer_sleeper *timeout, | 769 | struct hrtimer_sleeper *timeout, |
| 772 | int detect_deadlock __IP_DECL__)) | 770 | int detect_deadlock)) |
| 773 | { | 771 | { |
| 774 | if (!detect_deadlock && likely(rt_mutex_cmpxchg(lock, NULL, current))) { | 772 | if (!detect_deadlock && likely(rt_mutex_cmpxchg(lock, NULL, current))) { |
| 775 | rt_mutex_deadlock_account_lock(lock, current); | 773 | rt_mutex_deadlock_account_lock(lock, current); |
| 776 | return 0; | 774 | return 0; |
| 777 | } else | 775 | } else |
| 778 | return slowfn(lock, state, NULL, detect_deadlock __RET_IP__); | 776 | return slowfn(lock, state, NULL, detect_deadlock); |
| 779 | } | 777 | } |
| 780 | 778 | ||
| 781 | static inline int | 779 | static inline int |
| @@ -783,24 +781,24 @@ rt_mutex_timed_fastlock(struct rt_mutex *lock, int state, | |||
| 783 | struct hrtimer_sleeper *timeout, int detect_deadlock, | 781 | struct hrtimer_sleeper *timeout, int detect_deadlock, |
| 784 | int (*slowfn)(struct rt_mutex *lock, int state, | 782 | int (*slowfn)(struct rt_mutex *lock, int state, |
| 785 | struct hrtimer_sleeper *timeout, | 783 | struct hrtimer_sleeper *timeout, |
| 786 | int detect_deadlock __IP_DECL__)) | 784 | int detect_deadlock)) |
| 787 | { | 785 | { |
| 788 | if (!detect_deadlock && likely(rt_mutex_cmpxchg(lock, NULL, current))) { | 786 | if (!detect_deadlock && likely(rt_mutex_cmpxchg(lock, NULL, current))) { |
| 789 | rt_mutex_deadlock_account_lock(lock, current); | 787 | rt_mutex_deadlock_account_lock(lock, current); |
| 790 | return 0; | 788 | return 0; |
| 791 | } else | 789 | } else |
| 792 | return slowfn(lock, state, timeout, detect_deadlock __RET_IP__); | 790 | return slowfn(lock, state, timeout, detect_deadlock); |
| 793 | } | 791 | } |
| 794 | 792 | ||
| 795 | static inline int | 793 | static inline int |
| 796 | rt_mutex_fasttrylock(struct rt_mutex *lock, | 794 | rt_mutex_fasttrylock(struct rt_mutex *lock, |
| 797 | int (*slowfn)(struct rt_mutex *lock __IP_DECL__)) | 795 | int (*slowfn)(struct rt_mutex *lock)) |
| 798 | { | 796 | { |
| 799 | if (likely(rt_mutex_cmpxchg(lock, NULL, current))) { | 797 | if (likely(rt_mutex_cmpxchg(lock, NULL, current))) { |
| 800 | rt_mutex_deadlock_account_lock(lock, current); | 798 | rt_mutex_deadlock_account_lock(lock, current); |
| 801 | return 1; | 799 | return 1; |
| 802 | } | 800 | } |
| 803 | return slowfn(lock __RET_IP__); | 801 | return slowfn(lock); |
| 804 | } | 802 | } |
| 805 | 803 | ||
| 806 | static inline void | 804 | static inline void |
| @@ -948,7 +946,7 @@ void rt_mutex_init_proxy_locked(struct rt_mutex *lock, | |||
| 948 | struct task_struct *proxy_owner) | 946 | struct task_struct *proxy_owner) |
| 949 | { | 947 | { |
| 950 | __rt_mutex_init(lock, NULL); | 948 | __rt_mutex_init(lock, NULL); |
| 951 | debug_rt_mutex_proxy_lock(lock, proxy_owner __RET_IP__); | 949 | debug_rt_mutex_proxy_lock(lock, proxy_owner); |
| 952 | rt_mutex_set_owner(lock, proxy_owner, 0); | 950 | rt_mutex_set_owner(lock, proxy_owner, 0); |
| 953 | rt_mutex_deadlock_account_lock(lock, proxy_owner); | 951 | rt_mutex_deadlock_account_lock(lock, proxy_owner); |
| 954 | } | 952 | } |
diff --git a/kernel/rtmutex.h b/kernel/rtmutex.h index 1e0fca13ff72..a1a1dd06421d 100644 --- a/kernel/rtmutex.h +++ b/kernel/rtmutex.h | |||
| @@ -10,9 +10,6 @@ | |||
| 10 | * Non-debug version. | 10 | * Non-debug version. |
| 11 | */ | 11 | */ |
| 12 | 12 | ||
| 13 | #define __IP_DECL__ | ||
| 14 | #define __IP__ | ||
| 15 | #define __RET_IP__ | ||
| 16 | #define rt_mutex_deadlock_check(l) (0) | 13 | #define rt_mutex_deadlock_check(l) (0) |
| 17 | #define rt_mutex_deadlock_account_lock(m, t) do { } while (0) | 14 | #define rt_mutex_deadlock_account_lock(m, t) do { } while (0) |
| 18 | #define rt_mutex_deadlock_account_unlock(l) do { } while (0) | 15 | #define rt_mutex_deadlock_account_unlock(l) do { } while (0) |
diff --git a/kernel/sched.c b/kernel/sched.c index d5e37072ea54..48c1faa60a67 100644 --- a/kernel/sched.c +++ b/kernel/sched.c | |||
| @@ -30,6 +30,7 @@ | |||
| 30 | #include <linux/capability.h> | 30 | #include <linux/capability.h> |
| 31 | #include <linux/completion.h> | 31 | #include <linux/completion.h> |
| 32 | #include <linux/kernel_stat.h> | 32 | #include <linux/kernel_stat.h> |
| 33 | #include <linux/debug_locks.h> | ||
| 33 | #include <linux/security.h> | 34 | #include <linux/security.h> |
| 34 | #include <linux/notifier.h> | 35 | #include <linux/notifier.h> |
| 35 | #include <linux/profile.h> | 36 | #include <linux/profile.h> |
| @@ -3142,12 +3143,13 @@ void fastcall add_preempt_count(int val) | |||
| 3142 | /* | 3143 | /* |
| 3143 | * Underflow? | 3144 | * Underflow? |
| 3144 | */ | 3145 | */ |
| 3145 | BUG_ON((preempt_count() < 0)); | 3146 | if (DEBUG_LOCKS_WARN_ON((preempt_count() < 0))) |
| 3147 | return; | ||
| 3146 | preempt_count() += val; | 3148 | preempt_count() += val; |
| 3147 | /* | 3149 | /* |
| 3148 | * Spinlock count overflowing soon? | 3150 | * Spinlock count overflowing soon? |
| 3149 | */ | 3151 | */ |
| 3150 | BUG_ON((preempt_count() & PREEMPT_MASK) >= PREEMPT_MASK-10); | 3152 | DEBUG_LOCKS_WARN_ON((preempt_count() & PREEMPT_MASK) >= PREEMPT_MASK-10); |
| 3151 | } | 3153 | } |
| 3152 | EXPORT_SYMBOL(add_preempt_count); | 3154 | EXPORT_SYMBOL(add_preempt_count); |
| 3153 | 3155 | ||
| @@ -3156,11 +3158,15 @@ void fastcall sub_preempt_count(int val) | |||
| 3156 | /* | 3158 | /* |
| 3157 | * Underflow? | 3159 | * Underflow? |
| 3158 | */ | 3160 | */ |
| 3159 | BUG_ON(val > preempt_count()); | 3161 | if (DEBUG_LOCKS_WARN_ON(val > preempt_count())) |
| 3162 | return; | ||
| 3160 | /* | 3163 | /* |
| 3161 | * Is the spinlock portion underflowing? | 3164 | * Is the spinlock portion underflowing? |
| 3162 | */ | 3165 | */ |
| 3163 | BUG_ON((val < PREEMPT_MASK) && !(preempt_count() & PREEMPT_MASK)); | 3166 | if (DEBUG_LOCKS_WARN_ON((val < PREEMPT_MASK) && |
| 3167 | !(preempt_count() & PREEMPT_MASK))) | ||
| 3168 | return; | ||
| 3169 | |||
| 3164 | preempt_count() -= val; | 3170 | preempt_count() -= val; |
| 3165 | } | 3171 | } |
| 3166 | EXPORT_SYMBOL(sub_preempt_count); | 3172 | EXPORT_SYMBOL(sub_preempt_count); |
| @@ -4690,7 +4696,7 @@ void show_state(void) | |||
| 4690 | } while_each_thread(g, p); | 4696 | } while_each_thread(g, p); |
| 4691 | 4697 | ||
| 4692 | read_unlock(&tasklist_lock); | 4698 | read_unlock(&tasklist_lock); |
| 4693 | mutex_debug_show_all_locks(); | 4699 | debug_show_all_locks(); |
| 4694 | } | 4700 | } |
| 4695 | 4701 | ||
| 4696 | /** | 4702 | /** |
diff --git a/lib/Makefile b/lib/Makefile index 10c13c9d7824..4f5d01922f82 100644 --- a/lib/Makefile +++ b/lib/Makefile | |||
| @@ -11,7 +11,7 @@ lib-$(CONFIG_SMP) += cpumask.o | |||
| 11 | 11 | ||
| 12 | lib-y += kobject.o kref.o kobject_uevent.o klist.o | 12 | lib-y += kobject.o kref.o kobject_uevent.o klist.o |
| 13 | 13 | ||
| 14 | obj-y += sort.o parser.o halfmd4.o iomap_copy.o | 14 | obj-y += sort.o parser.o halfmd4.o iomap_copy.o debug_locks.o |
| 15 | 15 | ||
| 16 | ifeq ($(CONFIG_DEBUG_KOBJECT),y) | 16 | ifeq ($(CONFIG_DEBUG_KOBJECT),y) |
| 17 | CFLAGS_kobject.o += -DDEBUG | 17 | CFLAGS_kobject.o += -DDEBUG |
diff --git a/lib/debug_locks.c b/lib/debug_locks.c new file mode 100644 index 000000000000..0ef01d14727c --- /dev/null +++ b/lib/debug_locks.c | |||
| @@ -0,0 +1,45 @@ | |||
| 1 | /* | ||
| 2 | * lib/debug_locks.c | ||
| 3 | * | ||
| 4 | * Generic place for common debugging facilities for various locks: | ||
| 5 | * spinlocks, rwlocks, mutexes and rwsems. | ||
| 6 | * | ||
| 7 | * Started by Ingo Molnar: | ||
| 8 | * | ||
| 9 | * Copyright (C) 2006 Red Hat, Inc., Ingo Molnar <mingo@redhat.com> | ||
| 10 | */ | ||
| 11 | #include <linux/rwsem.h> | ||
| 12 | #include <linux/mutex.h> | ||
| 13 | #include <linux/module.h> | ||
| 14 | #include <linux/spinlock.h> | ||
| 15 | #include <linux/debug_locks.h> | ||
| 16 | |||
| 17 | /* | ||
| 18 | * We want to turn all lock-debugging facilities on/off at once, | ||
| 19 | * via a global flag. The reason is that once a single bug has been | ||
| 20 | * detected and reported, there might be cascade of followup bugs | ||
| 21 | * that would just muddy the log. So we report the first one and | ||
| 22 | * shut up after that. | ||
| 23 | */ | ||
| 24 | int debug_locks = 1; | ||
| 25 | |||
| 26 | /* | ||
| 27 | * The locking-testsuite uses <debug_locks_silent> to get a | ||
| 28 | * 'silent failure': nothing is printed to the console when | ||
| 29 | * a locking bug is detected. | ||
| 30 | */ | ||
| 31 | int debug_locks_silent; | ||
| 32 | |||
| 33 | /* | ||
| 34 | * Generic 'turn off all lock debugging' function: | ||
| 35 | */ | ||
| 36 | int debug_locks_off(void) | ||
| 37 | { | ||
| 38 | if (xchg(&debug_locks, 0)) { | ||
| 39 | if (!debug_locks_silent) { | ||
| 40 | console_verbose(); | ||
| 41 | return 1; | ||
| 42 | } | ||
| 43 | } | ||
| 44 | return 0; | ||
| 45 | } | ||
diff --git a/lib/spinlock_debug.c b/lib/spinlock_debug.c index 93c15ee3f8ea..3de2ccf48ac6 100644 --- a/lib/spinlock_debug.c +++ b/lib/spinlock_debug.c | |||
| @@ -8,38 +8,35 @@ | |||
| 8 | 8 | ||
| 9 | #include <linux/spinlock.h> | 9 | #include <linux/spinlock.h> |
| 10 | #include <linux/interrupt.h> | 10 | #include <linux/interrupt.h> |
| 11 | #include <linux/debug_locks.h> | ||
| 11 | #include <linux/delay.h> | 12 | #include <linux/delay.h> |
| 13 | #include <linux/module.h> | ||
| 12 | 14 | ||
| 13 | static void spin_bug(spinlock_t *lock, const char *msg) | 15 | static void spin_bug(spinlock_t *lock, const char *msg) |
| 14 | { | 16 | { |
| 15 | static long print_once = 1; | ||
| 16 | struct task_struct *owner = NULL; | 17 | struct task_struct *owner = NULL; |
| 17 | 18 | ||
| 18 | if (xchg(&print_once, 0)) { | 19 | if (!debug_locks_off()) |
| 19 | if (lock->owner && lock->owner != SPINLOCK_OWNER_INIT) | 20 | return; |
| 20 | owner = lock->owner; | 21 | |
| 21 | printk(KERN_EMERG "BUG: spinlock %s on CPU#%d, %s/%d\n", | 22 | if (lock->owner && lock->owner != SPINLOCK_OWNER_INIT) |
| 22 | msg, raw_smp_processor_id(), | 23 | owner = lock->owner; |
| 23 | current->comm, current->pid); | 24 | printk(KERN_EMERG "BUG: spinlock %s on CPU#%d, %s/%d\n", |
| 24 | printk(KERN_EMERG " lock: %p, .magic: %08x, .owner: %s/%d, " | 25 | msg, raw_smp_processor_id(), |
| 25 | ".owner_cpu: %d\n", | 26 | current->comm, current->pid); |
| 26 | lock, lock->magic, | 27 | printk(KERN_EMERG " lock: %p, .magic: %08x, .owner: %s/%d, " |
| 27 | owner ? owner->comm : "<none>", | 28 | ".owner_cpu: %d\n", |
| 28 | owner ? owner->pid : -1, | 29 | lock, lock->magic, |
| 29 | lock->owner_cpu); | 30 | owner ? owner->comm : "<none>", |
| 30 | dump_stack(); | 31 | owner ? owner->pid : -1, |
| 31 | #ifdef CONFIG_SMP | 32 | lock->owner_cpu); |
| 32 | /* | 33 | dump_stack(); |
| 33 | * We cannot continue on SMP: | ||
| 34 | */ | ||
| 35 | // panic("bad locking"); | ||
| 36 | #endif | ||
| 37 | } | ||
| 38 | } | 34 | } |
| 39 | 35 | ||
| 40 | #define SPIN_BUG_ON(cond, lock, msg) if (unlikely(cond)) spin_bug(lock, msg) | 36 | #define SPIN_BUG_ON(cond, lock, msg) if (unlikely(cond)) spin_bug(lock, msg) |
| 41 | 37 | ||
| 42 | static inline void debug_spin_lock_before(spinlock_t *lock) | 38 | static inline void |
| 39 | debug_spin_lock_before(spinlock_t *lock) | ||
| 43 | { | 40 | { |
| 44 | SPIN_BUG_ON(lock->magic != SPINLOCK_MAGIC, lock, "bad magic"); | 41 | SPIN_BUG_ON(lock->magic != SPINLOCK_MAGIC, lock, "bad magic"); |
| 45 | SPIN_BUG_ON(lock->owner == current, lock, "recursion"); | 42 | SPIN_BUG_ON(lock->owner == current, lock, "recursion"); |
| @@ -118,20 +115,13 @@ void _raw_spin_unlock(spinlock_t *lock) | |||
| 118 | 115 | ||
| 119 | static void rwlock_bug(rwlock_t *lock, const char *msg) | 116 | static void rwlock_bug(rwlock_t *lock, const char *msg) |
| 120 | { | 117 | { |
| 121 | static long print_once = 1; | 118 | if (!debug_locks_off()) |
| 122 | 119 | return; | |
| 123 | if (xchg(&print_once, 0)) { | 120 | |
| 124 | printk(KERN_EMERG "BUG: rwlock %s on CPU#%d, %s/%d, %p\n", | 121 | printk(KERN_EMERG "BUG: rwlock %s on CPU#%d, %s/%d, %p\n", |
| 125 | msg, raw_smp_processor_id(), current->comm, | 122 | msg, raw_smp_processor_id(), current->comm, |
| 126 | current->pid, lock); | 123 | current->pid, lock); |
| 127 | dump_stack(); | 124 | dump_stack(); |
| 128 | #ifdef CONFIG_SMP | ||
| 129 | /* | ||
| 130 | * We cannot continue on SMP: | ||
| 131 | */ | ||
| 132 | panic("bad locking"); | ||
| 133 | #endif | ||
| 134 | } | ||
| 135 | } | 125 | } |
| 136 | 126 | ||
| 137 | #define RWLOCK_BUG_ON(cond, lock, msg) if (unlikely(cond)) rwlock_bug(lock, msg) | 127 | #define RWLOCK_BUG_ON(cond, lock, msg) if (unlikely(cond)) rwlock_bug(lock, msg) |
diff --git a/mm/vmalloc.c b/mm/vmalloc.c index 35f8553f893a..7b450798b458 100644 --- a/mm/vmalloc.c +++ b/mm/vmalloc.c | |||
| @@ -330,6 +330,8 @@ void __vunmap(void *addr, int deallocate_pages) | |||
| 330 | return; | 330 | return; |
| 331 | } | 331 | } |
| 332 | 332 | ||
| 333 | debug_check_no_locks_freed(addr, area->size); | ||
| 334 | |||
| 333 | if (deallocate_pages) { | 335 | if (deallocate_pages) { |
| 334 | int i; | 336 | int i; |
| 335 | 337 | ||
