On Wed, Nov 17, 2021 at 9:22 AM Joyce Kong <joyce.k...@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
