From: Emilio G. Cota <cota@braap.org>
Date: Fri, 14 Oct 2016 09:54:51 +0000 (+0200)
Subject: qht-bench: relax test_start/stop atomic accesses
X-Git-Tag: Tizen_Studio_1.3_Release_p2.3.2~10^2~14^2~5^2~117^2~40
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=977ec47de06bdcb24f01c93bc125b7c6c221a1c5;p=sdk%2Femulator%2Fqemu.git

qht-bench: relax test_start/stop atomic accesses

test_start/stop are used only as flags to loop on. Barriers are unnecessary,
since no dependent data is transferred among threads apart from the flags
themselves.

This commit relaxes the three accesses to test_start/stop that were
not yet relaxed.

Signed-off-by: Emilio G. Cota <cota@braap.org>
---

diff --git a/tests/qht-bench.c b/tests/qht-bench.c
index 76360a0cf5..2afa09d859 100644
--- a/tests/qht-bench.c
+++ b/tests/qht-bench.c
@@ -193,7 +193,7 @@ static void *thread_func(void *p)
     rcu_register_thread();
 
     atomic_inc(&n_ready_threads);
-    while (!atomic_mb_read(&test_start)) {
+    while (!atomic_read(&test_start)) {
         cpu_relax();
     }
 
@@ -393,11 +393,11 @@ static void run_test(void)
     while (atomic_read(&n_ready_threads) != n_rw_threads + n_rz_threads) {
         cpu_relax();
     }
-    atomic_mb_set(&test_start, true);
+    atomic_set(&test_start, true);
     do {
         remaining = sleep(duration);
     } while (remaining);
-    atomic_mb_set(&test_stop, true);
+    atomic_set(&test_stop, true);
 
     for (i = 0; i < n_rw_threads; i++) {
         qemu_thread_join(&rw_threads[i]);