Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1378384

[PATCH] lockdep: Include print_collision and helper functions in DEBUG_LOCKDEP

From Yang Shi <yang.shi@linaro.org>
Newsgroups linux.kernel
Subject [PATCH] lockdep: Include print_collision and helper functions in DEBUG_LOCKDEP
Date 2016-04-14 03:30 +0200
Message-ID <rnEgy-22Q-1@gated-at.bofh.it> (permalink)
Organization linux.* mail to news gateway

Show all headers | View raw


pint_collosion and helper functions are just used when DEBUG_LOCKDEP is
selected, so include them in #ifdef CONFIG_DEBUG_LOCKDEP to avoid the below
"defined but not used" warning when DEBUG_LOCKDEP is off:

kernel/locking/lockdep.c:2051:13: warning: 'print_collision' defined but not used [-Wunused-function]
 static void print_collision(struct task_struct *curr,

Signed-off-by: Yang Shi <yang.shi@linaro.org>
---
 kernel/locking/lockdep.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/kernel/locking/lockdep.c b/kernel/locking/lockdep.c
index 2324ba5..ed94109 100644
--- a/kernel/locking/lockdep.c
+++ b/kernel/locking/lockdep.c
@@ -1999,6 +1999,7 @@ static inline int get_first_held_lock(struct task_struct *curr,
 	return ++i;
 }
 
+#ifdef CONFIG_DEBUG_LOCKDEP
 /*
  * Returns the next chain_key iteration
  */
@@ -2069,6 +2070,7 @@ static void print_collision(struct task_struct *curr,
 	printk("\nstack backtrace:\n");
 	dump_stack();
 }
+#endif
 
 /*
  * Checks whether the chain and the current held locks are consistent
-- 
2.0.2

Back to linux.kernel | Previous | Next | Find similar | Unroll thread


Thread

[PATCH] lockdep: Include print_collision and helper functions in DEBUG_LOCKDEP Yang Shi <yang.shi@linaro.org> - 2016-04-14 03:30 +0200

csiph-web