Switch atomic release API from __sync to __atomic builtin. 30/19930/3
authorSirshak Das <sirshak.das@arm.com>
Tue, 28 May 2019 13:46:27 +0000 (08:46 -0500)
committerDamjan Marion <dmarion@me.com>
Wed, 5 Jun 2019 13:52:01 +0000 (13:52 +0000)
__sync_lock_release switched to __atomic_store for code consitency,
although both generate same instructions with current compilers.

Change-Id: I37d320509e43a4c2b8a49af6346dc4a43ca2f535
Signed-off-by: Sirshak Das <sirshak.das@arm.com>
Reviewed-by: Lijian Zhang <Lijian.Zhang@arm.com>
Reviewed-by: Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>
src/vppinfra/atomics.h

index 1d480af..df8e534 100644 (file)
@@ -38,7 +38,7 @@
 #define clib_atomic_bool_cmp_and_swap(addr,old,new) __sync_bool_compare_and_swap(addr, old, new)
 
 #define clib_atomic_test_and_set(a) __atomic_exchange_n(a, 1, __ATOMIC_ACQUIRE)
-#define clib_atomic_release(a) __sync_lock_release(a)
+#define clib_atomic_release(a) __atomic_store_n(a, 0, __ATOMIC_RELEASE)
 
 #define clib_atomic_fence_rel() __atomic_thread_fence(__ATOMIC_RELEASE);