[PATCH v3 00/12] use compiler atomic builtins for app modules
David Marchand
david.marchand at redhat.com
Wed Nov 17 11:02:02 CET 2021
On Wed, Nov 17, 2021 at 9:22 AM Joyce Kong <joyce.kong at arm.com> wrote:
>
> Since atomic operations have been adopted in DPDK now[1],
> change rte_atomicNN_xxx APIs to compiler atomic built-ins
> in app modules[2].
>
> [1] https://www.dpdk.org/blog/2021/03/26/dpdk-adopts-the-c11-memory-model/
> [2] https://doc.dpdk.org/guides/rel_notes/deprecation.html
>
> v3:
> 1. In pmd_perf test case, move the initialization of polling
> start before calling rte_eal_remote_launch, so the update
> is visible to the worker threads.(Honnappa Nagarahalli)
> 2. Remove the rest rte_atomic.h which miss in v2.(David Marchand)
>
> v2:
> By Honnappa Nagarahalli:
> 1. Replace the RELAXED barriers with suitable ones for shared
> data sync in pmd_perf and timer test cases.
> 2. Avoid unnecessary atomic operations in compress and testpmd
> modules.
> 3. Fix some typo.
>
> Joyce Kong (12):
> test/pmd_perf: use compiler atomic builtins for polling sync
> test/ring_perf: use compiler atomic builtins for lcores sync
> test/timer: use compiler atomic builtins for sync
> test/stack_perf: use compiler atomics for lcore sync
> test/bpf: use compiler atomics for calculation
> test/func_reentrancy: use compiler atomics for data sync
> app/eventdev: use compiler atomics for shared data sync
> app/crypto: use compiler atomic builtins for display sync
> app/compress: use compiler atomic builtins for display sync
> app/testpmd: remove atomic operations for port status
> app/bbdev: use compiler atomics for shared data sync
> app: remove unnecessary include of atomic header file
There were cleanups of unneeded rte_atomic.h inclusion along the series:
I moved all of them to the last patch so that patches focus on what
their commitlog describes.
Series applied, thanks.
--
David Marchand
More information about the dev
mailing list