From: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Date: Tue, 8 Oct 2013 13:23:10 +0000 (-0400)
Subject: Fix: urcu-qsbr: reversed logic on RCU_DEBUG
X-Git-Tag: v0.8.1~6
X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=e7bf48b4c997109356801f02e1c47d832004a102;p=urcu.git

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 bb0523c..b4a28a7 100644
--- a/urcu-qsbr.h
+++ b/urcu-qsbr.h
@@ -91,7 +91,7 @@ extern "C" {
  * 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)
 {