[ltt-dev] [PATCH resent 3/7] urcu-qsbr: move private definitions to .c file

Paolo Bonzini pbonzini at redhat.com
Wed Aug 17 04:31:30 EDT 2011


The reader does not and should not know about RCU_QS_ACTIVE_ATTEMPTS.
Move the definition to urcu-qsbr.c.  Also remove KICK_READER_LOOPS.

Signed-off-by: Paolo Bonzini <pbonzini at redhat.com>
Reviewed-by: Mathieu Desnoyers <mathieu.desnoyers at efficios.com>
---
 urcu-qsbr.c             |    5 +++++
 urcu/static/urcu-qsbr.h |   12 ------------
 2 files changed, 5 insertions(+), 12 deletions(-)

diff --git a/urcu-qsbr.c b/urcu-qsbr.c
index 87cf41d..a239be0 100644
--- a/urcu-qsbr.c
+++ b/urcu-qsbr.c
@@ -53,6 +53,11 @@ int32_t gp_futex;
 unsigned long rcu_gp_ctr = RCU_GP_ONLINE;
 
 /*
+ * Active attempts to check for reader Q.S. before calling futex().
+ */
+#define RCU_QS_ACTIVE_ATTEMPTS 100
+
+/*
  * Written to only by each individual reader. Read by both the reader and the
  * writers.
  */
diff --git a/urcu/static/urcu-qsbr.h b/urcu/static/urcu-qsbr.h
index c46a7be..5b7adac 100644
--- a/urcu/static/urcu-qsbr.h
+++ b/urcu/static/urcu-qsbr.h
@@ -55,18 +55,6 @@ extern "C" {
  * This is required to permit relinking with newer versions of the library.
  */
 
-/*
- * If a reader is really non-cooperative and refuses to commit its
- * rcu_reader.ctr count to memory (there is no barrier in the reader
- * per-se), kick it after a few loops waiting for it.
- */
-#define KICK_READER_LOOPS 10000
-
-/*
- * Active attempts to check for reader Q.S. before calling futex().
- */
-#define RCU_QS_ACTIVE_ATTEMPTS 100
-
 #ifdef DEBUG_RCU
 #define rcu_assert(args...)	assert(args)
 #else
-- 
1.7.6






More information about the lttng-dev mailing list