From f6da0ec3fc06b963e4e564995befa28d800f71ed Mon Sep 17 00:00:00 2001
From: kehongzhang <75783504+kehongzhang@users.noreply.github.com>
Date: Tue, 22 Dec 2020 20:40:37 +0800
Subject: [PATCH] Update atomics.h

The fetchadd64_acquire only have load acquire barrier. It may lead to load/store before the synchronize could be speculated after of load/store after the synchronize.
So, change fetchadd64_acquire to fetchadd64_acquire_release to ensure load/store order.
---
 benchmarks/lockhammer/include/atomics.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/benchmarks/lockhammer/include/atomics.h b/benchmarks/lockhammer/include/atomics.h
index c1ae409..100c0c5 100644
--- a/benchmarks/lockhammer/include/atomics.h
+++ b/benchmarks/lockhammer/include/atomics.h
@@ -494,7 +494,7 @@ void synchronize_threads(uint64_t *barrier, unsigned long nthrds)
     uint64_t tmp_sense = ~global_sense & SENSE_BIT_MASK;
     uint32_t local_sense = (uint32_t)(tmp_sense >> 32);
 
-    uint64_t old_barrier = fetchadd64_acquire(barrier, 2);
+    uint64_t old_barrier = fetchadd64_acquire_release(barrier, 2);
     if (old_barrier == (((nthrds - 1) * 2) | global_sense)) {
         // Make sure the store gets observed by the system. Reset count
         // to zero and flip the sense bit.