diff mbox series

[v2,16/16] rcu: use previous value atomic fetch operations

Message ID	1678914945-10638-17-git-send-email-roretzla@linux.microsoft.com (mailing list archive)
State	Superseded, archived
Delegated to:	Thomas Monjalon
Headers	From: Tyler Retzlaff <roretzla@linux.microsoft.com> To: dev@dpdk.org Cc: Honnappa.Nagarahalli@arm.com, Ruifeng.Wang@arm.com, thomas@monjalon.net, Tyler Retzlaff <roretzla@linux.microsoft.com> Subject: [PATCH v2 16/16] rcu: use previous value atomic fetch operations Date: Wed, 15 Mar 2023 14:15:45 -0700 Message-Id: <1678914945-10638-17-git-send-email-roretzla@linux.microsoft.com> In-Reply-To: <1678914945-10638-1-git-send-email-roretzla@linux.microsoft.com> References: <1678486530-20688-1-git-send-email-roretzla@linux.microsoft.com> <1678914945-10638-1-git-send-email-roretzla@linux.microsoft.com> Precedence: list Errors-To: dev-bounces@dpdk.org
Series	replace __atomic operations returning new value \| [v2,00/16] replace __atomic operations returning new value [v2,01/16] app/test: use previous value atomic fetch operations [v2,02/16] common/cnxk: use previous value atomic fetch operations [v2,03/16] common/mlx5: use previous value atomic fetch operations [v2,04/16] drivers/event: use previous value atomic fetch operations [v2,05/16] net/af_xdp: use previous value atomic fetch operations [v2,06/16] net/cnxk: use previous value atomic fetch operations [v2,07/16] net/cxgbe: use previous value atomic fetch operations [v2,08/16] net/iavf: use previous value atomic fetch operations [v2,09/16] net/mlx5: use previous value atomic fetch operations [v2,10/16] net/octeontx: use previous value atomic fetch operations [v2,11/16] raw/ifpga: use previous value atomic fetch operations [v2,12/16] bbdev: use previous value atomic fetch operations [v2,13/16] eal: use previous value atomic fetch operations [v2,14/16] ipsec: use previous value atomic fetch operations [v2,15/16] mbuf: use previous value atomic fetch operations [v2,16/16] rcu: use previous value atomic fetch operations

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK
ci/loongarch-compilation	success	Compilation OK
ci/loongarch-unit-testing	success	Unit Testing PASS
ci/Intel-compilation	success	Compilation OK
ci/iol-mellanox-Performance	success	Performance Testing PASS
ci/iol-broadcom-Functional	success	Functional Testing PASS
ci/iol-intel-Functional	success	Functional Testing PASS
ci/intel-Testing	success	Testing PASS
ci/iol-broadcom-Performance	success	Performance Testing PASS
ci/iol-intel-Performance	success	Performance Testing PASS
ci/iol-x86_64-compile-testing	success	Testing PASS
ci/iol-aarch64-unit-testing	success	Testing PASS
ci/github-robot: build	success	github build: passed
ci/iol-aarch64-compile-testing	success	Testing PASS
ci/iol-abi-testing	success	Testing PASS
ci/intel-Functional	fail	Functional issues

Commit Message

Tyler Retzlaff March 15, 2023, 9:15 p.m. UTC

  Use __atomic_fetch_{add,and,or,sub,xor} instead of
__atomic_{add,and,or,sub,xor}_fetch adding the necessary code to
allow consumption of the resulting value.

Signed-off-by: Tyler Retzlaff <roretzla@linux.microsoft.com>
---
 lib/rcu/rte_rcu_qsbr.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff mbox series

Patch

diff --git a/lib/rcu/rte_rcu_qsbr.h b/lib/rcu/rte_rcu_qsbr.h
index ccae5d5..1aa078e 100644
--- a/lib/rcu/rte_rcu_qsbr.h
+++ b/lib/rcu/rte_rcu_qsbr.h
@@ -462,7 +462,7 @@  struct rte_rcu_qsbr_dq_parameters {
 	 * structure are visible to the workers before the token
 	 * update is visible.
 	 */
-	t = __atomic_add_fetch(&v->token, 1, __ATOMIC_RELEASE);
+	t = __atomic_fetch_add(&v->token, 1, __ATOMIC_RELEASE) + 1;
 
 	return t;
 }