From: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Date: Mon, 1 May 2023 14:06:38 +0000 (-0400)
Subject: rcu.h: remove duplicated fence
X-Git-Url: http://git.efficios.com/?p=libside.git;a=commitdiff_plain;h=7e6bad80c8eedbe4a382294151d8e0318d50d19a

rcu.h: remove duplicated fence

Already implied by SEQ_CST.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
---

diff --git a/src/rcu.h b/src/rcu.h
index 86b9e24..69a07c7 100644
--- a/src/rcu.h
+++ b/src/rcu.h
@@ -143,11 +143,10 @@ void side_rcu_read_end(struct side_rcu_gp_state *gp_state, struct side_rcu_read_
 	/* Fallback to atomic increment and SEQ_CST. */
 	(void) __atomic_add_fetch(&begin_cpu_count->end, 1, __ATOMIC_SEQ_CST);
 	/*
-	 * This barrier (F) is paired with SEQ_CST barrier or
-	 * membarrier() at (G). It orders increment of the begin/end
-	 * counters before load/store to the futex.
+	 * This barrier (F) implied by SEQ_CST is paired with SEQ_CST
+	 * barrier or membarrier() at (G). It orders increment of the
+	 * begin/end counters before load/store to the futex.
 	 */
-	__atomic_thread_fence(__ATOMIC_SEQ_CST);
 end:
 	side_rcu_wake_up_gp(gp_state);
 }