diff options
author | Sebastian Andrzej Siewior <bigeasy@linutronix.de> | 2021-03-26 16:29:32 +0100 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2021-03-29 15:57:02 +0200 |
commit | 6d41c675a5394057f6fb1dc97cc0a0e360f2c2f8 (patch) | |
tree | 95fa0eca643d79adabe7957638c32d271b3fdeca /kernel/locking/rtmutex_common.h | |
parent | locking/rtmutex: Remove rtmutex deadlock tester leftovers (diff) | |
download | linux-6d41c675a5394057f6fb1dc97cc0a0e360f2c2f8.tar.xz linux-6d41c675a5394057f6fb1dc97cc0a0e360f2c2f8.zip |
locking/rtmutex: Remove output from deadlock detector
The rtmutex specific deadlock detector predates lockdep coverage of rtmutex
and since commit f5694788ad8da ("rt_mutex: Add lockdep annotations") it
contains a lot of redundant functionality:
- lockdep will detect an potential deadlock before rtmutex-debug
has a chance to do so
- the deadlock debugging is restricted to rtmutexes which are not
associated to futexes and have an active waiter, which is covered by
lockdep already
Remove the redundant functionality and move actual deadlock WARN() into the
deadlock code path. The latter needs a seperate cleanup.
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lore.kernel.org/r/20210326153943.320398604@linutronix.de
Diffstat (limited to 'kernel/locking/rtmutex_common.h')
-rw-r--r-- | kernel/locking/rtmutex_common.h | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/kernel/locking/rtmutex_common.h b/kernel/locking/rtmutex_common.h index aa047436dadf..badb2a2803aa 100644 --- a/kernel/locking/rtmutex_common.h +++ b/kernel/locking/rtmutex_common.h @@ -29,10 +29,6 @@ struct rt_mutex_waiter { struct rb_node pi_tree_entry; struct task_struct *task; struct rt_mutex *lock; -#ifdef CONFIG_DEBUG_RT_MUTEXES - struct pid *deadlock_task_pid; - struct rt_mutex *deadlock_lock; -#endif int prio; u64 deadline; }; |