There is no point for this wrapper at all.

Signed-off-by: Thomas Gleixner <t...@linutronix.de>
---
 kernel/locking/rtmutex.c |   11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

--- a/kernel/locking/rtmutex.c
+++ b/kernel/locking/rtmutex.c
@@ -343,14 +343,9 @@ static void rt_mutex_adjust_prio(struct
 static bool rt_mutex_cond_detect_deadlock(struct rt_mutex_waiter *waiter,
                                          enum rtmutex_chainwalk chwalk)
 {
-       /*
-        * This is just a wrapper function for the following call,
-        * because debug_rt_mutex_detect_deadlock() smells like a magic
-        * debug feature and I wanted to keep the cond function in the
-        * main source file along with the comments instead of having
-        * two of the same in the headers.
-        */
-       return debug_rt_mutex_detect_deadlock(waiter, chwalk);
+       if (IS_ENABLED(CONFIG_DEBUG_RT_MUTEX))
+               return waiter != NULL;
+       return chwalk == RT_MUTEX_FULL_CHAINWALK;
 }
 
 /*

Reply via email to