From: Ruifeng Wang <Ruifeng.Wang@arm.com>
To: Tyler Retzlaff <roretzla@linux.microsoft.com>,
"dev@dpdk.org" <dev@dpdk.org>
Cc: Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>,
"thomas@monjalon.net" <thomas@monjalon.net>,
"bruce.richardson@intel.com" <bruce.richardson@intel.com>,
"mb@smartsharesystems.com" <mb@smartsharesystems.com>,
"maxime.coquelin@redhat.com" <maxime.coquelin@redhat.com>,
nd <nd@arm.com>
Subject: RE: [PATCH v2 00/17] use __atomic operations returning previous value
Date: Fri, 3 Mar 2023 01:54:48 +0000 [thread overview]
Message-ID: <AS8PR08MB70806B2236AC74787AB204CD9EB39@AS8PR08MB7080.eurprd08.prod.outlook.com> (raw)
In-Reply-To: <1677773902-5167-1-git-send-email-roretzla@linux.microsoft.com>
> -----Original Message-----
> From: Tyler Retzlaff <roretzla@linux.microsoft.com>
> Sent: Friday, March 3, 2023 12:18 AM
> To: dev@dpdk.org
> Cc: Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>; thomas@monjalon.net;
> bruce.richardson@intel.com; mb@smartsharesystems.com; Ruifeng Wang <Ruifeng.Wang@arm.com>;
> maxime.coquelin@redhat.com; Tyler Retzlaff <roretzla@linux.microsoft.com>
> Subject: [PATCH v2 00/17] use __atomic operations returning previous value
>
> This series replaces uses of __atomic_{add,and,or,sub,xor}_fetch with
> __atomic_fetch_{add,and,or,sub,xor} intrinsics. The latter omits generation of code that
> is only needed if the returned value is actually used.
>
> Additionally, this change simplifies adapting dpdk to standard atomics planned for 23.07
> since __atomic_fetch_xxx can be trivially replaced with atomic_fetch_xxx whereas
> __atomic_xxx_fetch has no standard equivalent.
>
>
> v2:
> net/iavf patch had one incorrect change to __atomic_fetch_sub
> from __atomic_sub_fetch the new value was assigned to a variable
> on the previous line so revert the change from the patch.
>
> Series-acked-by: Morten Brørup <mb@smartsharesystems.com>
>
> Tyler Retzlaff (17):
> vhost: use previous value atomic fetch operations
> telemetry: use previous value atomic fetch operations
> stack: use previous value atomic fetch operations
> eal: use previous value atomic fetch operations
> distributor: use previous value atomic fetch operations
> bbdev: use previous value atomic fetch operations
> examples/vhost: use previous value atomic fetch operations
> net/virtio: use previous value atomic fetch operations
> net/octeontx: use previous value atomic fetch operations
> net/mlx5: use previous value atomic fetch operations
> net/iavf: use previous value atomic fetch operations
> net/cxgbe: use previous value atomic fetch operations
> drivers/event: use previous value atomic fetch operations
> dma/skeleton: use previous value atomic fetch operations
> drivers/common: use previous value atomic fetch operations
> app/test: use previous value atomic fetch operations
> test-eventdev: use previous value atomic fetch operations
>
> app/test-eventdev/test_order_common.h | 2 +-
> app/test/test_lcores.c | 2 +-
> app/test/test_service_cores.c | 4 ++--
> drivers/common/cnxk/roc_nix_inl_dev.c | 2 +-
> drivers/common/mlx5/mlx5_common_mr.c | 2 +-
> drivers/common/mlx5/mlx5_common_utils.c | 10 +++++-----
> drivers/common/mlx5/mlx5_malloc.c | 16 ++++++++--------
> drivers/dma/skeleton/skeleton_dmadev.c | 2 +-
> drivers/event/cnxk/cnxk_eventdev_selftest.c | 12 ++++++------
> drivers/event/cnxk/cnxk_tim_worker.h | 6 +++---
> drivers/event/dsw/dsw_event.c | 6 +++---
> drivers/event/octeontx/timvf_worker.h | 6 +++---
> drivers/net/cxgbe/clip_tbl.c | 2 +-
> drivers/net/cxgbe/cxgbe_main.c | 12 ++++++------
> drivers/net/cxgbe/l2t.c | 4 ++--
> drivers/net/cxgbe/mps_tcam.c | 2 +-
> drivers/net/cxgbe/smt.c | 4 ++--
> drivers/net/iavf/iavf_vchnl.c | 4 ++--
> drivers/net/mlx5/linux/mlx5_verbs.c | 2 +-
> drivers/net/mlx5/mlx5_flow.c | 6 +++---
> drivers/net/mlx5/mlx5_flow_dv.c | 4 ++--
> drivers/net/mlx5/mlx5_flow_flex.c | 6 +++---
> drivers/net/mlx5/mlx5_flow_hw.c | 10 +++++-----
> drivers/net/mlx5/mlx5_flow_meter.c | 20 ++++++++++----------
> drivers/net/mlx5/mlx5_rx.h | 2 +-
> drivers/net/octeontx/octeontx_ethdev.c | 2 +-
> drivers/net/virtio/virtio_user/virtio_user_dev.c | 2 +-
> examples/vhost/main.c | 12 ++++++------
> examples/vhost/virtio_net.c | 4 ++--
> lib/bbdev/rte_bbdev.c | 2 +-
> lib/distributor/rte_distributor.c | 2 +-
> lib/eal/common/eal_common_trace.c | 8 ++++----
> lib/eal/common/rte_service.c | 8 ++++----
> lib/eal/ppc/include/rte_atomic.h | 16 ++++++++--------
> lib/stack/rte_stack_lf_c11.h | 2 +-
> lib/telemetry/telemetry.c | 6 +++---
> lib/vhost/virtio_net.c | 6 +++---
> 37 files changed, 109 insertions(+), 109 deletions(-)
>
> --
> 1.8.3.1
Series-acked-by: Ruifeng Wang <ruifeng.wang@arm.com>
next prev parent reply other threads:[~2023-03-03 1:55 UTC|newest]
Thread overview: 47+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-02 0:47 [PATCH 00/17] use __atomic operations returning previous value Tyler Retzlaff
2023-03-02 0:47 ` [PATCH 01/17] vhost: use previous value atomic fetch operations Tyler Retzlaff
2023-03-02 14:47 ` Maxime Coquelin
2023-03-02 0:47 ` [PATCH 02/17] telemetry: " Tyler Retzlaff
2023-03-02 9:34 ` Bruce Richardson
2023-03-02 0:47 ` [PATCH 03/17] stack: " Tyler Retzlaff
2023-03-02 0:47 ` [PATCH 04/17] eal: " Tyler Retzlaff
2023-03-02 0:47 ` [PATCH 05/17] distributor: " Tyler Retzlaff
2023-03-02 0:47 ` [PATCH 06/17] bbdev: " Tyler Retzlaff
2023-03-02 0:47 ` [PATCH 07/17] examples/vhost: " Tyler Retzlaff
2023-03-02 14:47 ` Maxime Coquelin
2023-03-02 0:47 ` [PATCH 08/17] net/virtio: " Tyler Retzlaff
2023-03-02 14:48 ` Maxime Coquelin
2023-03-02 0:47 ` [PATCH 09/17] net/octeontx: " Tyler Retzlaff
2023-03-02 0:47 ` [PATCH 10/17] net/mlx5: " Tyler Retzlaff
2023-03-02 0:47 ` [PATCH 11/17] net/iavf: " Tyler Retzlaff
2023-03-02 9:28 ` Ruifeng Wang
2023-03-02 15:50 ` Tyler Retzlaff
2023-03-02 0:47 ` [PATCH 12/17] net/cxgbe: " Tyler Retzlaff
2023-03-02 0:47 ` [PATCH 13/17] drivers/event: " Tyler Retzlaff
2023-03-02 0:47 ` [PATCH 14/17] dma/skeleton: " Tyler Retzlaff
2023-03-02 0:47 ` [PATCH 15/17] drivers/common: " Tyler Retzlaff
2023-03-02 0:47 ` [PATCH 16/17] app/test: " Tyler Retzlaff
2023-03-02 0:47 ` [PATCH 17/17] test-eventdev: " Tyler Retzlaff
2023-03-02 14:40 ` [PATCH 00/17] use __atomic operations returning previous value Morten Brørup
2023-03-02 16:18 ` [PATCH v2 " Tyler Retzlaff
2023-03-02 16:18 ` [PATCH v2 01/17] vhost: use previous value atomic fetch operations Tyler Retzlaff
2023-03-02 16:18 ` [PATCH v2 02/17] telemetry: " Tyler Retzlaff
2023-03-02 16:18 ` [PATCH v2 03/17] stack: " Tyler Retzlaff
2023-03-02 16:18 ` [PATCH v2 04/17] eal: " Tyler Retzlaff
2023-03-02 16:18 ` [PATCH v2 05/17] distributor: " Tyler Retzlaff
2023-03-02 16:18 ` [PATCH v2 06/17] bbdev: " Tyler Retzlaff
2023-03-02 16:18 ` [PATCH v2 07/17] examples/vhost: " Tyler Retzlaff
2023-03-02 16:18 ` [PATCH v2 08/17] net/virtio: " Tyler Retzlaff
2023-03-02 16:18 ` [PATCH v2 09/17] net/octeontx: " Tyler Retzlaff
2023-03-02 16:18 ` [PATCH v2 10/17] net/mlx5: " Tyler Retzlaff
2023-03-02 16:18 ` [PATCH v2 11/17] net/iavf: " Tyler Retzlaff
2023-03-02 16:18 ` [PATCH v2 12/17] net/cxgbe: " Tyler Retzlaff
2023-03-02 16:18 ` [PATCH v2 13/17] drivers/event: " Tyler Retzlaff
2023-03-13 7:02 ` [EXT] " Pavan Nikhilesh Bhagavatula
2023-03-02 16:18 ` [PATCH v2 14/17] dma/skeleton: " Tyler Retzlaff
2023-03-02 16:18 ` [PATCH v2 15/17] drivers/common: " Tyler Retzlaff
2023-03-02 16:18 ` [PATCH v2 16/17] app/test: " Tyler Retzlaff
2023-03-02 16:18 ` [PATCH v2 17/17] test-eventdev: " Tyler Retzlaff
2023-03-03 1:54 ` Ruifeng Wang [this message]
2023-03-03 2:33 ` [PATCH v2 00/17] use __atomic operations returning previous value fengchengwen
2023-04-25 9:10 ` David Marchand
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=AS8PR08MB70806B2236AC74787AB204CD9EB39@AS8PR08MB7080.eurprd08.prod.outlook.com \
--to=ruifeng.wang@arm.com \
--cc=Honnappa.Nagarahalli@arm.com \
--cc=bruce.richardson@intel.com \
--cc=dev@dpdk.org \
--cc=maxime.coquelin@redhat.com \
--cc=mb@smartsharesystems.com \
--cc=nd@arm.com \
--cc=roretzla@linux.microsoft.com \
--cc=thomas@monjalon.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.