X-Git-Url: https://git.liburcu.org/?p=urcu.git;a=blobdiff_plain;f=test_urcu.c;h=3a28474561977675c6932663055c3cb8d2e2fc19;hp=93bc013b4a3a3575e598ef10eb61096871db7aa1;hb=710b09b7ba5203ea868240c1746d0ac0fc65f884;hpb=cf380c2fd2f85e3fba4826991fb748255bdc9b76

diff --git a/test_urcu.c b/test_urcu.c
index 93bc013..3a28474 100644
--- a/test_urcu.c
+++ b/test_urcu.c
@@ -40,6 +40,8 @@ struct test_array {
 	int a;
 };
 
+static int no_writer_delay;
+
 static struct test_array *test_rcu_pointer;
 
 static unsigned long duration;
@@ -63,6 +65,9 @@ static int test_duration(void)
 #define NR_READ 10
 #define NR_WRITE 9
 
+static unsigned long long __thread nr_writes;
+static unsigned long long __thread nr_reads;
+
 pthread_mutex_t rcu_copy_mutex = PTHREAD_MUTEX_INITIALIZER;
 
 void rcu_copy_mutex_lock(void)
@@ -86,9 +91,45 @@ void rcu_copy_mutex_unlock(void)
 	}
 }
 
-void *thr_reader(void *arg)
+/*
+ * malloc/free are reusing memory areas too quickly, which does not let us
+ * test races appropriately. Use a large circular array for allocations.
+ * ARRAY_SIZE is larger than NR_WRITE, which insures we never run over our tail.
+ */
+#define ARRAY_SIZE (1048576 * NR_WRITE)
+#define ARRAY_POISON 0xDEADBEEF
+static int array_index;
+static struct test_array test_array[ARRAY_SIZE];
+
+static struct test_array *test_array_alloc(void)
+{
+	struct test_array *ret;
+	int index;
+
+	rcu_copy_mutex_lock();
+	index = array_index % ARRAY_SIZE;
+	assert(test_array[index].a == ARRAY_POISON ||
+		test_array[index].a == 0);
+	ret = &test_array[index];
+	array_index++;
+	if (array_index == ARRAY_SIZE)
+		array_index = 0;
+	rcu_copy_mutex_unlock();
+	return ret;
+}
+
+static void test_array_free(struct test_array *ptr)
+{
+	if (!ptr)
+		return;
+	rcu_copy_mutex_lock();
+	ptr->a = ARRAY_POISON;
+	rcu_copy_mutex_unlock();
+}
+
+void *thr_reader(void *_count)
 {
-	int qparity;
+	unsigned long long *count = _count;
 	struct test_array *local_ptr;
 
 	printf("thread_begin %s, thread id : %lx, tid %lu\n",
@@ -97,87 +138,107 @@ void *thr_reader(void *arg)
 	urcu_register_thread();
 
 	for (;;) {
-		rcu_read_lock(&qparity);
+		rcu_read_lock();
 		local_ptr = rcu_dereference(test_rcu_pointer);
+		debug_yield_read();
 		if (local_ptr)
 			assert(local_ptr->a == 8);
-		rcu_read_unlock(&qparity);
+		rcu_read_unlock();
+		nr_reads++;
 		if (!test_duration())
 			break;
 	}
 
 	urcu_unregister_thread();
 
+	*count = nr_reads;
 	printf("thread_end %s, thread id : %lx, tid %lu\n",
 			"reader", pthread_self(), (unsigned long)gettid());
 	return ((void*)1);
 
 }
 
-void *thr_writer(void *arg)
+void *thr_writer(void *_count)
 {
+	unsigned long long *count = _count;
 	struct test_array *new, *old;
 
 	printf("thread_begin %s, thread id : %lx, tid %lu\n",
 			"writer", pthread_self(), (unsigned long)gettid());
 
 	for (;;) {
-		new = malloc(sizeof(struct test_array));
+		new = test_array_alloc();
 		rcu_copy_mutex_lock();
 		old = test_rcu_pointer;
 		if (old)
 			assert(old->a == 8);
 		new->a = 8;
-		old = urcu_publish_content((void **)&test_rcu_pointer, new);
+		old = urcu_publish_content(&test_rcu_pointer, new);
 		rcu_copy_mutex_unlock();
 		/* can be done after unlock */
 		if (old)
 			old->a = 0;
-		free(old);
+		test_array_free(old);
+		nr_writes++;
 		if (!test_duration())
 			break;
-		usleep(1);
+		if (!no_writer_delay)
+			usleep(1);
 	}
 
 	printf("thread_end %s, thread id : %lx, tid %lu\n",
 			"writer", pthread_self(), (unsigned long)gettid());
+	*count = nr_writes;
 	return ((void*)2);
 }
 
+void show_usage(int argc, char **argv)
+{
+	printf("Usage : %s duration (s)", argv[0]);
+#ifdef DEBUG_YIELD
+	printf(" [-r] [-w] (yield reader and/or writer)");
+#endif
+	printf(" [-n] (disable writer delay)");
+	printf("\n");
+}
+
 int main(int argc, char **argv)
 {
 	int err;
 	pthread_t tid_reader[NR_READ], tid_writer[NR_WRITE];
 	void *tret;
+	unsigned long long count_reader[NR_READ], count_writer[NR_WRITE];
+	unsigned long long tot_reads = 0, tot_writes = 0;
 	int i;
 
 	if (argc < 2) {
-		printf("Usage : %s duration (s) [-r] [-w] "
-		       "(yield reader and/or writer)\n", argv[0]);
+		show_usage(argc, argv);
 		return -1;
 	}
 
 	err = sscanf(argv[1], "%lu", &duration);
 	if (err != 1) {
-		printf("Usage : %s duration (s) [-r] [-w] "
-		       "(yield reader and/or writer)\n", argv[0]);
+		show_usage(argc, argv);
 		return -1;
 	}
 
-#ifdef DEBUG_YIELD
 	for (i = 2; i < argc; i++) {
 		if (argv[i][0] != '-')
 			continue;
 		switch (argv[i][1]) {
+#ifdef DEBUG_YIELD
 		case 'r':
 			yield_active |= YIELD_READ;
 			break;
 		case 'w':
 			yield_active |= YIELD_WRITE;
 			break;
+#endif
+		case 'n':
+			no_writer_delay = 1;
+			break;
 		}
 	}
-#endif
 
 	printf("running test for %lu seconds.\n", duration);
 	start_time = time(NULL);
@@ -185,12 +246,14 @@ int main(int argc, char **argv)
 			"main", pthread_self(), (unsigned long)gettid());
 
 	for (i = 0; i < NR_READ; i++) {
-		err = pthread_create(&tid_reader[i], NULL, thr_reader, NULL);
+		err = pthread_create(&tid_reader[i], NULL, thr_reader,
+				     &count_reader[i]);
 		if (err != 0)
 			exit(1);
 	}
 	for (i = 0; i < NR_WRITE; i++) {
-		err = pthread_create(&tid_writer[i], NULL, thr_writer, NULL);
+		err = pthread_create(&tid_writer[i], NULL, thr_writer,
+				     &count_writer[i]);
 		if (err != 0)
 			exit(1);
 	}
@@ -199,13 +262,17 @@ int main(int argc, char **argv)
 		err = pthread_join(tid_reader[i], &tret);
 		if (err != 0)
 			exit(1);
+		tot_reads += count_reader[i];
 	}
 	for (i = 0; i < NR_WRITE; i++) {
 		err = pthread_join(tid_writer[i], &tret);
 		if (err != 0)
 			exit(1);
+		tot_writes += count_writer[i];
 	}
-	free(test_rcu_pointer);
+	printf("total number of reads : %llu, writes %llu\n", tot_reads,
+	       tot_writes);
+	test_array_free(test_rcu_pointer);
 
 	return 0;
 }