Fix: urcu-qsbr: reversed logic on RCU_DEBUG

* Dmitri Shubin <sbn@tbricks.com> wrote:
> Shouldn't the condition in line 94 actually be
>
>      94  #if (!defined(BUILD_QSBR_LIB) && !defined(RCU_DEBUG))
>
> So when RCU_DEBUG is _not_ defined we get static inlines for
> rcu_read_{,un}lock() ?

Indeed!

Reported-by: Dmitri Shubin <sbn@tbricks.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

diff --git a/urcu-qsbr.h b/urcu-qsbr.h
index bb0523c904916062ae1ee4843c8ee5085dd8ee8f..b4a28a78a6a84490611f1a12277197f4a02092d6 100644 (file)
--- a/urcu-qsbr.h
+++ b/urcu-qsbr.h
@@ -91,7 +91,7 @@ extern "C" {
  * library debugging & tracing features we could come up with.
  */
 
  * library debugging & tracing features we could come up with.
  */
 

-#if (!defined(BUILD_QSBR_LIB) && defined(RCU_DEBUG))
+#if (!defined(BUILD_QSBR_LIB) && !defined(RCU_DEBUG))

 
 static inline void rcu_read_lock(void)
 {
 
 static inline void rcu_read_lock(void)
 {