diff mbox series

[v3,05/12] test/bpf: use compiler atomics for calculation

Message ID	20211117082200.1029621-6-joyce.kong@arm.com (mailing list archive)
State	Accepted, archived
Delegated to:	David Marchand
Headers	From: Joyce Kong <joyce.kong@arm.com> To: Konstantin Ananyev <konstantin.ananyev@intel.com> Cc: dev@dpdk.org, honnappa.nagarahalli@arm.com, nd@arm.com, Joyce Kong <joyce.kong@arm.com>, Ruifeng Wang <ruifeng.wang@arm.com> Subject: [PATCH v3 05/12] test/bpf: use compiler atomics for calculation Date: Wed, 17 Nov 2021 08:21:53 +0000 Message-Id: <20211117082200.1029621-6-joyce.kong@arm.com> In-Reply-To: <20211117082200.1029621-1-joyce.kong@arm.com> References: <20211116094205.750359-1-joyce.kong@arm.com> <20211117082200.1029621-1-joyce.kong@arm.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: list Errors-To: dev-bounces@dpdk.org
Series	use compiler atomic builtins for app modules \| [v3,00/12] use compiler atomic builtins for app modules [v3,01/12] test/pmd_perf: use compiler atomic builtins for polling sync [v3,02/12] test/ring_perf: use compiler atomic builtins for lcores sync [v3,03/12] test/timer: use compiler atomic builtins for sync [v3,04/12] test/stack_perf: use compiler atomics for lcore sync [v3,05/12] test/bpf: use compiler atomics for calculation [v3,06/12] test/func_reentrancy: use compiler atomics for data sync [v3,07/12] app/eventdev: use compiler atomics for shared data sync [v3,08/12] app/crypto: use compiler atomic builtins for display sync [v3,09/12] app/compress: use compiler atomic builtins for display sync [v3,10/12] app/testpmd: remove atomic operations for port status [v3,11/12] app/bbdev: use compiler atomics for shared data sync [v3,12/12] app: remove unnecessary include of atomic header file

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

Joyce Kong Nov. 17, 2021, 8:21 a.m. UTC

  Convert rte_atomic usages to compiler atomic built-ins
for calculation in bpf test cases.

Signed-off-by: Joyce Kong <joyce.kong@arm.com>
Reviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>
---
 app/test/test_bpf.c | 28 ++++++++++++++--------------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff mbox series

Patch

diff --git a/app/test/test_bpf.c b/app/test/test_bpf.c
index e3e9a1b0b5..b8be1e3d30 100644
--- a/app/test/test_bpf.c
+++ b/app/test/test_bpf.c
@@ -1569,32 +1569,32 @@  test_xadd1_check(uint64_t rc, const void *arg)
 	memset(&dfe, 0, sizeof(dfe));
 
 	rv = 1;
-	rte_atomic32_add((rte_atomic32_t *)&dfe.u32, rv);
-	rte_atomic64_add((rte_atomic64_t *)&dfe.u64, rv);
+	__atomic_fetch_add(&dfe.u32, rv, __ATOMIC_RELAXED);
+	__atomic_fetch_add(&dfe.u64, rv, __ATOMIC_RELAXED);
 
 	rv = -1;
-	rte_atomic32_add((rte_atomic32_t *)&dfe.u32, rv);
-	rte_atomic64_add((rte_atomic64_t *)&dfe.u64, rv);
+	__atomic_fetch_add(&dfe.u32, rv, __ATOMIC_RELAXED);
+	__atomic_fetch_add(&dfe.u64, rv, __ATOMIC_RELAXED);
 
 	rv = (int32_t)TEST_FILL_1;
-	rte_atomic32_add((rte_atomic32_t *)&dfe.u32, rv);
-	rte_atomic64_add((rte_atomic64_t *)&dfe.u64, rv);
+	__atomic_fetch_add(&dfe.u32, rv, __ATOMIC_RELAXED);
+	__atomic_fetch_add(&dfe.u64, rv, __ATOMIC_RELAXED);
 
 	rv = TEST_MUL_1;
-	rte_atomic32_add((rte_atomic32_t *)&dfe.u32, rv);
-	rte_atomic64_add((rte_atomic64_t *)&dfe.u64, rv);
+	__atomic_fetch_add(&dfe.u32, rv, __ATOMIC_RELAXED);
+	__atomic_fetch_add(&dfe.u64, rv, __ATOMIC_RELAXED);
 
 	rv = TEST_MUL_2;
-	rte_atomic32_add((rte_atomic32_t *)&dfe.u32, rv);
-	rte_atomic64_add((rte_atomic64_t *)&dfe.u64, rv);
+	__atomic_fetch_add(&dfe.u32, rv, __ATOMIC_RELAXED);
+	__atomic_fetch_add(&dfe.u64, rv, __ATOMIC_RELAXED);
 
 	rv = TEST_JCC_2;
-	rte_atomic32_add((rte_atomic32_t *)&dfe.u32, rv);
-	rte_atomic64_add((rte_atomic64_t *)&dfe.u64, rv);
+	__atomic_fetch_add(&dfe.u32, rv, __ATOMIC_RELAXED);
+	__atomic_fetch_add(&dfe.u64, rv, __ATOMIC_RELAXED);
 
 	rv = TEST_JCC_3;
-	rte_atomic32_add((rte_atomic32_t *)&dfe.u32, rv);
-	rte_atomic64_add((rte_atomic64_t *)&dfe.u64, rv);
+	__atomic_fetch_add(&dfe.u32, rv, __ATOMIC_RELAXED);
+	__atomic_fetch_add(&dfe.u64, rv, __ATOMIC_RELAXED);
 
 	return cmp_res(__func__, 1, rc, &dfe, dft, sizeof(dfe));
 }