mbox series

[v1,00/12] use compiler atomic builtins for app

Message ID 20210802101847.3462-1-joyce.kong@arm.com (mailing list archive)
Headers show
Series use compiler atomic builtins for app | expand

Message

Joyce Kong Aug. 2, 2021, 10:18 a.m. UTC
Since atomic operations have been adopted in DPDK now[1],
change rte_atomicNN_xxx APIs to compiler's atomic built-ins
in app module[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

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 atomic for data sync
  app/eventdev: use compiler atomic builtins for packets sync
  app/crypto: use compiler atomic builtins for display sync
  app/compress: use compiler atomic builtins for display sync
  app/testpmd: use compiler atomic builtins for port sync
  app/bbdev: use compiler atomics for thread sync
  app: remove unnecessary include of atomic

 app/proc-info/main.c                          |   1 -
 app/test-bbdev/test_bbdev_perf.c              | 135 ++++++++----------
 .../comp_perf_test_common.h                   |   2 +-
 .../comp_perf_test_cyclecount.c               |  10 +-
 .../comp_perf_test_throughput.c               |  11 +-
 .../comp_perf_test_verify.c                   |   6 +-
 app/test-crypto-perf/cperf_test_latency.c     |   6 +-
 .../cperf_test_pmd_cyclecount.c               |   9 +-
 app/test-crypto-perf/cperf_test_throughput.c  |   9 +-
 app/test-crypto-perf/cperf_test_verify.c      |   9 +-
 app/test-eventdev/evt_main.c                  |   1 -
 app/test-eventdev/test_order_atq.c            |   4 +-
 app/test-eventdev/test_order_common.c         |   4 +-
 app/test-eventdev/test_order_common.h         |   8 +-
 app/test-eventdev/test_order_queue.c          |   4 +-
 app/test-pipeline/config.c                    |   1 -
 app/test-pipeline/init.c                      |   1 -
 app/test-pipeline/main.c                      |   1 -
 app/test-pipeline/runtime.c                   |   1 -
 app/test-pmd/cmdline.c                        |   1 -
 app/test-pmd/config.c                         |   1 -
 app/test-pmd/csumonly.c                       |   1 -
 app/test-pmd/flowgen.c                        |   1 -
 app/test-pmd/icmpecho.c                       |   1 -
 app/test-pmd/iofwd.c                          |   1 -
 app/test-pmd/macfwd.c                         |   1 -
 app/test-pmd/macswap.c                        |   1 -
 app/test-pmd/parameters.c                     |   1 -
 app/test-pmd/rxonly.c                         |   1 -
 app/test-pmd/testpmd.c                        |  75 ++++++----
 app/test-pmd/txonly.c                         |   1 -
 app/test/test_barrier.c                       |   1 -
 app/test/test_bpf.c                           |  28 ++--
 app/test/test_func_reentrancy.c               |  27 ++--
 app/test/test_mbuf.c                          |   1 -
 app/test/test_mp_secondary.c                  |   1 -
 app/test/test_pmd_perf.c                      |  12 +-
 app/test/test_ring.c                          |   1 -
 app/test/test_ring_perf.c                     |   9 +-
 app/test/test_stack_perf.c                    |  14 +-
 app/test/test_timer.c                         |  28 ++--
 app/test/test_timer_secondary.c               |   1 -
 42 files changed, 213 insertions(+), 219 deletions(-)

Comments

Joyce Kong Oct. 21, 2021, 6:35 a.m. UTC | #1
Hi all,

Would you please help review the patch series?
Thanks!

Best Regards,
Joyce

> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Joyce Kong
> Sent: Monday, August 2, 2021 6:19 PM
> To: thomas@monjalon.net; david.marchand@redhat.com; Honnappa
> Nagarahalli <Honnappa.Nagarahalli@arm.com>; Ruifeng Wang
> <Ruifeng.Wang@arm.com>; konstantin.ananyev@intel.com;
> rsanford@akamai.com; erik.g.carrillo@intel.com; olivier.matz@6wind.com;
> yipeng1.wang@intel.com; sameh.gobriel@intel.com;
> bruce.richardson@intel.com; vladimir.medvedkin@intel.com;
> anatoly.burakov@intel.com; andrew.rybchenko@oktetlabs.ru;
> jerinj@marvell.com; declan.doherty@intel.com; ciara.power@intel.com;
> xiaoyun.li@intel.com; nicolas.chautru@intel.com;
> maryam.tahhan@intel.com; reshma.pattan@intel.com;
> cristian.dumitrescu@intel.com
> Cc: dev@dpdk.org; nd <nd@arm.com>
> Subject: [dpdk-dev] [PATCH v1 00/12] use compiler atomic builtins for app
> 
> Since atomic operations have been adopted in DPDK now[1], change
> rte_atomicNN_xxx APIs to compiler's atomic built-ins in app module[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
> 
> 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 atomic for data sync
>   app/eventdev: use compiler atomic builtins for packets sync
>   app/crypto: use compiler atomic builtins for display sync
>   app/compress: use compiler atomic builtins for display sync
>   app/testpmd: use compiler atomic builtins for port sync
>   app/bbdev: use compiler atomics for thread sync
>   app: remove unnecessary include of atomic
> 
>  app/proc-info/main.c                          |   1 -
>  app/test-bbdev/test_bbdev_perf.c              | 135 ++++++++----------
>  .../comp_perf_test_common.h                   |   2 +-
>  .../comp_perf_test_cyclecount.c               |  10 +-
>  .../comp_perf_test_throughput.c               |  11 +-
>  .../comp_perf_test_verify.c                   |   6 +-
>  app/test-crypto-perf/cperf_test_latency.c     |   6 +-
>  .../cperf_test_pmd_cyclecount.c               |   9 +-
>  app/test-crypto-perf/cperf_test_throughput.c  |   9 +-
>  app/test-crypto-perf/cperf_test_verify.c      |   9 +-
>  app/test-eventdev/evt_main.c                  |   1 -
>  app/test-eventdev/test_order_atq.c            |   4 +-
>  app/test-eventdev/test_order_common.c         |   4 +-
>  app/test-eventdev/test_order_common.h         |   8 +-
>  app/test-eventdev/test_order_queue.c          |   4 +-
>  app/test-pipeline/config.c                    |   1 -
>  app/test-pipeline/init.c                      |   1 -
>  app/test-pipeline/main.c                      |   1 -
>  app/test-pipeline/runtime.c                   |   1 -
>  app/test-pmd/cmdline.c                        |   1 -
>  app/test-pmd/config.c                         |   1 -
>  app/test-pmd/csumonly.c                       |   1 -
>  app/test-pmd/flowgen.c                        |   1 -
>  app/test-pmd/icmpecho.c                       |   1 -
>  app/test-pmd/iofwd.c                          |   1 -
>  app/test-pmd/macfwd.c                         |   1 -
>  app/test-pmd/macswap.c                        |   1 -
>  app/test-pmd/parameters.c                     |   1 -
>  app/test-pmd/rxonly.c                         |   1 -
>  app/test-pmd/testpmd.c                        |  75 ++++++----
>  app/test-pmd/txonly.c                         |   1 -
>  app/test/test_barrier.c                       |   1 -
>  app/test/test_bpf.c                           |  28 ++--
>  app/test/test_func_reentrancy.c               |  27 ++--
>  app/test/test_mbuf.c                          |   1 -
>  app/test/test_mp_secondary.c                  |   1 -
>  app/test/test_pmd_perf.c                      |  12 +-
>  app/test/test_ring.c                          |   1 -
>  app/test/test_ring_perf.c                     |   9 +-
>  app/test/test_stack_perf.c                    |  14 +-
>  app/test/test_timer.c                         |  28 ++--
>  app/test/test_timer_secondary.c               |   1 -
>  42 files changed, 213 insertions(+), 219 deletions(-)
> 
> --
> 2.17.1