X-Git-Url: https://git.liburcu.org/?p=urcu.git;a=blobdiff_plain;f=urcu-defer.c;h=92635ed3cca4c1f45390bfac5348714722f56547;hp=3cc1a0c0b32b9132673b47a361ea116f9a8bf338;hb=0ecb3fde04135d76545ce55d80abea9aef465b54;hpb=ec8e44cf2ba50dcb421af40b67e74650bb5057f6

diff --git a/urcu-defer.c b/urcu-defer.c
index 3cc1a0c..92635ed 100644
--- a/urcu-defer.c
+++ b/urcu-defer.c
@@ -37,14 +37,14 @@
 /* Do not #define _LGPL_SOURCE to ensure we can emit the wrapper symbols */
 #include "urcu-defer.h"
 
-void __attribute__((destructor)) urcu_defer_exit(void);
+void __attribute__((destructor)) rcu_defer_exit(void);
 
 extern void synchronize_rcu(void);
 
 /*
- * urcu_defer_mutex nests inside defer_thread_mutex.
+ * rcu_defer_mutex nests inside defer_thread_mutex.
  */
-static pthread_mutex_t urcu_defer_mutex = PTHREAD_MUTEX_INITIALIZER;
+static pthread_mutex_t rcu_defer_mutex = PTHREAD_MUTEX_INITIALIZER;
 static pthread_mutex_t defer_thread_mutex = PTHREAD_MUTEX_INITIALIZER;
 
 static int defer_thread_futex;
@@ -57,7 +57,7 @@ static struct defer_queue __thread defer_queue;
 static LIST_HEAD(registry);
 static pthread_t tid_defer;
 
-static void internal_urcu_lock(pthread_mutex_t *mutex)
+static void mutex_lock(pthread_mutex_t *mutex)
 {
 	int ret;
 
@@ -80,7 +80,7 @@ static void internal_urcu_lock(pthread_mutex_t *mutex)
 #endif /* #else #ifndef DISTRUST_SIGNALS_EXTREME */
 }
 
-static void internal_urcu_unlock(pthread_mutex_t *mutex)
+static void mutex_unlock(pthread_mutex_t *mutex)
 {
 	int ret;
 
@@ -108,12 +108,12 @@ static unsigned long rcu_defer_num_callbacks(void)
 	unsigned long num_items = 0, head;
 	struct defer_queue *index;
 
-	internal_urcu_lock(&urcu_defer_mutex);
+	mutex_lock(&rcu_defer_mutex);
 	list_for_each_entry(index, &registry, list) {
 		head = LOAD_SHARED(index->head);
 		num_items += head - index->tail;
 	}
-	internal_urcu_unlock(&urcu_defer_mutex);
+	mutex_unlock(&rcu_defer_mutex);
 	return num_items;
 }
 
@@ -184,9 +184,9 @@ static void _rcu_defer_barrier_thread(void)
 
 void rcu_defer_barrier_thread(void)
 {
-	internal_urcu_lock(&urcu_defer_mutex);
+	mutex_lock(&rcu_defer_mutex);
 	_rcu_defer_barrier_thread();
-	internal_urcu_unlock(&urcu_defer_mutex);
+	mutex_unlock(&rcu_defer_mutex);
 }
 
 /*
@@ -210,7 +210,7 @@ void rcu_defer_barrier(void)
 	if (list_empty(&registry))
 		return;
 
-	internal_urcu_lock(&urcu_defer_mutex);
+	mutex_lock(&rcu_defer_mutex);
 	list_for_each_entry(index, &registry, list) {
 		index->last_head = LOAD_SHARED(index->head);
 		num_items += index->last_head - index->tail;
@@ -226,21 +226,15 @@ void rcu_defer_barrier(void)
 	list_for_each_entry(index, &registry, list)
 		rcu_defer_barrier_queue(index, index->last_head);
 end:
-	internal_urcu_unlock(&urcu_defer_mutex);
+	mutex_unlock(&rcu_defer_mutex);
 }
 
 /*
  * _defer_rcu - Queue a RCU callback.
  */
-void _defer_rcu_ratelimit(void (*fct)(void *p), void *p, int (*rl)(void *p))
+void _defer_rcu(void (*fct)(void *p), void *p)
 {
 	unsigned long head, tail;
-	int sync;
-
-	/*
-	 * Verify if we reached the rate limiter threshold.
-	 */
-	sync = rl ? rl(p) : 0;
 
 	/*
 	 * Head is only modified by ourself. Tail can be modified by reclamation
@@ -253,7 +247,7 @@ void _defer_rcu_ratelimit(void (*fct)(void *p), void *p, int (*rl)(void *p))
 	 * If queue is full, or reached threshold. Empty queue ourself.
 	 * Worse-case: must allow 2 supplementary entries for fct pointer.
 	 */
-	if (unlikely(sync || (head - tail >= DEFER_QUEUE_SIZE - 2))) {
+	if (unlikely(head - tail >= DEFER_QUEUE_SIZE - 2)) {
 		assert(head - tail <= DEFER_QUEUE_SIZE);
 		rcu_defer_barrier_thread();
 		assert(head - LOAD_SHARED(defer_queue.tail) == 0);
@@ -321,9 +315,9 @@ void *thr_defer(void *args)
  * library wrappers to be used by non-LGPL compatible source code.
  */
 
-void defer_rcu_ratelimit(void (*fct)(void *p), void *p, int (*rl)(void *p))
+void defer_rcu(void (*fct)(void *p), void *p)
 {
-	_defer_rcu_ratelimit(fct, p, rl);
+	_defer_rcu(fct, p);
 }
 
 static void start_defer_thread(void)
@@ -353,36 +347,36 @@ void rcu_defer_register_thread(void)
 	assert(defer_queue.q == NULL);
 	defer_queue.q = malloc(sizeof(void *) * DEFER_QUEUE_SIZE);
 
-	internal_urcu_lock(&defer_thread_mutex);
-	internal_urcu_lock(&urcu_defer_mutex);
+	mutex_lock(&defer_thread_mutex);
+	mutex_lock(&rcu_defer_mutex);
 	was_empty = list_empty(&registry);
 	list_add(&defer_queue.list, &registry);
-	internal_urcu_unlock(&urcu_defer_mutex);
+	mutex_unlock(&rcu_defer_mutex);
 
 	if (was_empty)
 		start_defer_thread();
-	internal_urcu_unlock(&defer_thread_mutex);
+	mutex_unlock(&defer_thread_mutex);
 }
 
 void rcu_defer_unregister_thread(void)
 {
 	int is_empty;
 
-	internal_urcu_lock(&defer_thread_mutex);
-	internal_urcu_lock(&urcu_defer_mutex);
+	mutex_lock(&defer_thread_mutex);
+	mutex_lock(&rcu_defer_mutex);
 	list_del(&defer_queue.list);
 	_rcu_defer_barrier_thread();
 	free(defer_queue.q);
 	defer_queue.q = NULL;
 	is_empty = list_empty(&registry);
-	internal_urcu_unlock(&urcu_defer_mutex);
+	mutex_unlock(&rcu_defer_mutex);
 
 	if (is_empty)
 		stop_defer_thread();
-	internal_urcu_unlock(&defer_thread_mutex);
+	mutex_unlock(&defer_thread_mutex);
 }
 
-void urcu_defer_exit(void)
+void rcu_defer_exit(void)
 {
 	assert(list_empty(&registry));
 }