From 7e6bad80c8eedbe4a382294151d8e0318d50d19a Mon Sep 17 00:00:00 2001 From: Mathieu Desnoyers Date: Mon, 1 May 2023 10:06:38 -0400 Subject: [PATCH] rcu.h: remove duplicated fence Already implied by SEQ_CST. Signed-off-by: Mathieu Desnoyers --- src/rcu.h | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) 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); } -- 2.34.1