summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJean Delvare <khali@linux-fr.org>2011-07-16 17:42:00 +0200
committerIngo Molnar <mingo@elte.hu>2011-07-21 09:36:09 +0200
commit4582c0a4866ea70c35aa9279e1f91834d3348a93 (patch)
treee180e664a88a996512a98a69098ef59f92e18c0a
parent732375c6a5a4cc825b676c922d547aba96b8ce15 (diff)
downloadlinux-3.10-4582c0a4866ea70c35aa9279e1f91834d3348a93.tar.gz
linux-3.10-4582c0a4866ea70c35aa9279e1f91834d3348a93.tar.bz2
linux-3.10-4582c0a4866ea70c35aa9279e1f91834d3348a93.zip
mutex: Make mutex_destroy() an inline function
The non-debug variant of mutex_destroy is a no-op, currently implemented as a macro which does nothing. This approach fails to check the type of the parameter, so an error would only show when debugging gets enabled. Using an inline function instead, offers type checking for earlier bug catching. Signed-off-by: Jean Delvare <khali@linux-fr.org> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Link: http://lkml.kernel.org/r/20110716174200.41002352@endymion.delvare Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r--include/linux/mutex.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/mutex.h b/include/linux/mutex.h
index a940fe435ac..7f87217e9d1 100644
--- a/include/linux/mutex.h
+++ b/include/linux/mutex.h
@@ -92,7 +92,7 @@ do { \
\
__mutex_init((mutex), #mutex, &__key); \
} while (0)
-# define mutex_destroy(mutex) do { } while (0)
+static inline void mutex_destroy(struct mutex *lock) {}
#endif
#ifdef CONFIG_DEBUG_LOCK_ALLOC