DPDK patches and discussions
 help / color / mirror / Atom feed
From: Xueming Li <xuemingl@nvidia.com>
To: <dev@dpdk.org>
Cc: <xuemingl@nvidia.com>, Ferruh Yigit <ferruh.yigit@intel.com>,
	"Andrew Rybchenko" <andrew.rybchenko@oktetlabs.ru>,
	"Singh, Aman Deep" <aman.deep.singh@intel.com>,
	Thomas Monjalon <thomas@monjalon.net>
Subject: [dpdk-dev] [PATCH v5 0/2] ethdev: change queue release callback
Date: Sat, 18 Sep 2021 20:35:23 +0800	[thread overview]
Message-ID: <20210918123525.135129-1-xuemingl@nvidia.com> (raw)
In-Reply-To: <20210727034134.20556-1-xuemingl@nvidia.com>

This patch is a preparation of shared Rx queue feature[1]. Rxq object
could be shared among ports of a share group, a shared rxq object can't
tell which port it belongs to. This is the motativation to clean up
queue release callback API to use queue ID.

v2:
  included new NFP PMD driver
v3:
- commit message update
- split allowing empty queue release callback to another patch
v4:
- use helper funtction to release a single queue
- move driver empty callback cleanup to first patch
v5:
- fix release callback parameter name
  
[1]
https://mails.dpdk.org/archives/dev/2021-July/215575.html


Xueming Li (2):
  ethdev: make queue release callback optional
  ethdev: change queue release callback

 app/test/virtual_pmd.c                    | 12 ----
 drivers/net/af_packet/rte_eth_af_packet.c |  7 --
 drivers/net/af_xdp/rte_eth_af_xdp.c       |  7 --
 drivers/net/atlantic/atl_ethdev.h         |  4 +-
 drivers/net/atlantic/atl_rxtx.c           | 24 +++----
 drivers/net/avp/avp_ethdev.c              | 34 +++------
 drivers/net/axgbe/axgbe_dev.c             |  2 +-
 drivers/net/axgbe/axgbe_rxtx.c            |  8 +--
 drivers/net/axgbe/axgbe_rxtx.h            |  4 +-
 drivers/net/bnx2x/bnx2x_rxtx.c            |  8 +--
 drivers/net/bnx2x/bnx2x_rxtx.h            |  4 +-
 drivers/net/bnxt/bnxt_reps.c              | 20 +++---
 drivers/net/bnxt/bnxt_reps.h              |  4 +-
 drivers/net/bnxt/bnxt_ring.c              |  2 +-
 drivers/net/bnxt/bnxt_rxq.c               | 12 ++--
 drivers/net/bnxt/bnxt_rxq.h               |  2 +-
 drivers/net/bnxt/bnxt_txq.c               | 15 ++--
 drivers/net/bnxt/bnxt_txq.h               |  2 +-
 drivers/net/bonding/rte_eth_bond_pmd.c    |  8 ++-
 drivers/net/cnxk/cnxk_ethdev.c            | 33 ++++-----
 drivers/net/cxgbe/cxgbe_ethdev.c          | 12 ++--
 drivers/net/cxgbe/cxgbe_pfvf.h            |  4 +-
 drivers/net/dpaa/dpaa_ethdev.c            | 13 ----
 drivers/net/dpaa2/dpaa2_ethdev.c          | 11 +--
 drivers/net/e1000/e1000_ethdev.h          |  8 +--
 drivers/net/e1000/em_rxtx.c               | 12 ++--
 drivers/net/e1000/igb_rxtx.c              | 12 ++--
 drivers/net/ena/ena_ethdev.c              | 18 +++--
 drivers/net/enetc/enetc_ethdev.c          | 12 ++--
 drivers/net/enic/enic_ethdev.c            |  8 ++-
 drivers/net/enic/enic_vf_representor.c    |  8 ++-
 drivers/net/failsafe/failsafe_ops.c       | 42 +++++------
 drivers/net/fm10k/fm10k_ethdev.c          | 14 ++--
 drivers/net/hinic/hinic_pmd_ethdev.c      | 20 +++---
 drivers/net/hns3/hns3_rxtx.c              | 25 ++++---
 drivers/net/hns3/hns3_rxtx.h              |  4 +-
 drivers/net/i40e/i40e_fdir.c              |  8 +--
 drivers/net/i40e/i40e_rxtx.c              | 40 +++++++----
 drivers/net/i40e/i40e_rxtx.h              |  6 +-
 drivers/net/iavf/iavf_rxtx.c              | 12 ++--
 drivers/net/iavf/iavf_rxtx.h              |  4 +-
 drivers/net/ice/ice_dcf_ethdev.c          |  4 +-
 drivers/net/ice/ice_ethdev.c              |  4 +-
 drivers/net/ice/ice_rxtx.c                | 12 ++++
 drivers/net/ice/ice_rxtx.h                |  2 +
 drivers/net/igc/igc_ethdev.c              |  4 +-
 drivers/net/igc/igc_txrx.c                | 12 ++--
 drivers/net/igc/igc_txrx.h                |  4 +-
 drivers/net/ionic/ionic_lif.c             |  4 +-
 drivers/net/ionic/ionic_rxtx.c            | 14 ++--
 drivers/net/ionic/ionic_rxtx.h            |  4 +-
 drivers/net/ipn3ke/ipn3ke_representor.c   | 12 ----
 drivers/net/ixgbe/ixgbe_ethdev.h          |  4 +-
 drivers/net/ixgbe/ixgbe_rxtx.c            | 12 ++--
 drivers/net/kni/rte_eth_kni.c             |  7 --
 drivers/net/liquidio/lio_ethdev.c         | 24 ++++---
 drivers/net/liquidio/lio_ethdev.h         |  4 +-
 drivers/net/liquidio/lio_rxtx.c           |  4 +-
 drivers/net/memif/rte_eth_memif.c         | 23 ++++--
 drivers/net/mlx4/mlx4.c                   |  4 +-
 drivers/net/mlx4/mlx4_rxq.c               | 27 +++----
 drivers/net/mlx4/mlx4_rxtx.h              |  4 +-
 drivers/net/mlx4/mlx4_txq.c               | 27 +++----
 drivers/net/mlx5/mlx5_rx.h                |  2 +-
 drivers/net/mlx5/mlx5_rxq.c               | 21 +++---
 drivers/net/mlx5/mlx5_tx.h                |  2 +-
 drivers/net/mlx5/mlx5_txq.c               | 25 +++----
 drivers/net/mvneta/mvneta_ethdev.c        |  4 +-
 drivers/net/mvneta/mvneta_rxtx.c          | 22 +++---
 drivers/net/mvneta/mvneta_rxtx.h          |  4 +-
 drivers/net/mvpp2/mrvl_ethdev.c           | 20 +++---
 drivers/net/netvsc/hn_rxtx.c              | 10 +--
 drivers/net/netvsc/hn_var.h               |  4 +-
 drivers/net/netvsc/hn_vf.c                | 14 ++--
 drivers/net/nfb/nfb_ethdev.c              |  4 +-
 drivers/net/nfb/nfb_rx.c                  |  5 +-
 drivers/net/nfb/nfb_rx.h                  |  8 ++-
 drivers/net/nfb/nfb_tx.c                  |  5 +-
 drivers/net/nfb/nfb_tx.h                  |  8 ++-
 drivers/net/nfp/nfp_rxtx.c                | 30 ++++----
 drivers/net/nfp/nfp_rxtx.h                |  4 +-
 drivers/net/ngbe/ngbe_ethdev.h            |  4 +-
 drivers/net/ngbe/ngbe_rxtx.c              | 12 ++--
 drivers/net/null/rte_eth_null.c           | 22 ++++--
 drivers/net/octeontx/octeontx_ethdev.c    | 18 +++--
 drivers/net/octeontx2/otx2_ethdev.c       | 59 ++++++----------
 drivers/net/octeontx_ep/otx_ep_ethdev.c   | 20 +++---
 drivers/net/pcap/pcap_ethdev.c            |  7 --
 drivers/net/pfe/pfe_ethdev.c              | 14 ----
 drivers/net/qede/qede_ethdev.c            | 20 ++++--
 drivers/net/ring/rte_eth_ring.c           |  4 --
 drivers/net/sfc/sfc_ethdev.c              |  8 +--
 drivers/net/szedata2/rte_eth_szedata2.c   | 50 ++++++-------
 drivers/net/tap/rte_eth_tap.c             |  8 +--
 drivers/net/thunderx/nicvf_ethdev.c       | 28 ++++----
 drivers/net/txgbe/txgbe_ethdev.h          |  4 +-
 drivers/net/txgbe/txgbe_rxtx.c            | 12 ++--
 drivers/net/vhost/rte_eth_vhost.c         | 14 ++--
 drivers/net/virtio/virtio_ethdev.c        |  8 ---
 drivers/net/vmxnet3/vmxnet3_ethdev.c      | 14 ++--
 drivers/net/vmxnet3/vmxnet3_ethdev.h      |  4 +-
 drivers/net/vmxnet3/vmxnet3_rxtx.c        |  8 +--
 lib/ethdev/ethdev_driver.h                |  3 +-
 lib/ethdev/rte_ethdev.c                   | 86 ++++++++++-------------
 104 files changed, 631 insertions(+), 709 deletions(-)

-- 
2.33.0


  parent reply	other threads:[~2021-09-18 12:36 UTC|newest]

Thread overview: 63+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-27  3:41 [dpdk-dev] [RFC] " Xueming Li
2021-07-28  7:40 ` Andrew Rybchenko
2021-08-09 14:39   ` Singh, Aman Deep
2021-08-09 15:31     ` Ferruh Yigit
2021-08-10  8:03       ` Xueming(Steven) Li
2021-08-10  8:54         ` Ferruh Yigit
2021-08-10  9:07           ` Xueming(Steven) Li
2021-08-11 11:57             ` Ferruh Yigit
2021-08-11 12:13               ` Xueming(Steven) Li
2021-08-12 14:29                 ` Xueming(Steven) Li
2021-09-26 11:25               ` Xueming(Steven) Li
2021-08-11 13:45 ` [dpdk-dev] [PATCH v1] " Xueming Li
2021-09-15 13:02 ` [dpdk-dev] [PATCH v2] " Xueming Li
2021-09-15 13:36   ` Xueming(Steven) Li
2021-09-16  8:09   ` Thomas Monjalon
2021-09-16 15:43     ` Xueming(Steven) Li
2021-09-16 15:50       ` Thomas Monjalon
2021-09-17  9:40         ` Xueming(Steven) Li
2021-09-17  9:39 ` [dpdk-dev] [PATCH v3 0/2] " Xueming Li
2021-09-17  9:39   ` [dpdk-dev] [PATCH v3 1/2] ethdev: queue release callback optional Xueming Li
2021-09-17 11:29     ` Andrew Rybchenko
2021-09-17 11:53       ` Andrew Rybchenko
2021-09-17 14:33         ` Xueming(Steven) Li
2021-09-17  9:39   ` [dpdk-dev] [PATCH v3 2/2] ethdev: change queue release callback Xueming Li
2021-09-17 11:49     ` Andrew Rybchenko
2021-09-17 14:31       ` Xueming(Steven) Li
2021-09-17 14:28 ` [dpdk-dev] [PATCH v4 0/2] " Xueming Li
2021-09-17 14:28   ` [dpdk-dev] [PATCH v4 1/2] ethdev: make queue release callback optional Xueming Li
2021-09-18  6:44     ` Andrew Rybchenko
2021-10-05 22:00       ` Thomas Monjalon
2021-09-17 14:28   ` [dpdk-dev] [PATCH v4 2/2] ethdev: change queue release callback Xueming Li
2021-09-18  6:50     ` Andrew Rybchenko
2021-09-18 12:39       ` Xueming(Steven) Li
2021-09-18 12:35 ` Xueming Li [this message]
2021-09-18 12:35   ` [dpdk-dev] [PATCH v5 1/2] ethdev: make queue release callback optional Xueming Li
2021-09-21 16:23     ` Ferruh Yigit
2021-09-18 12:35   ` [dpdk-dev] [PATCH v5 2/2] ethdev: change queue release callback Xueming Li
2021-09-21 18:13     ` Ferruh Yigit
2021-09-22  9:35       ` Xueming(Steven) Li
2021-09-22 10:57         ` Ferruh Yigit
2021-09-22 12:54           ` Xueming(Steven) Li
2021-09-29 13:57             ` Xueming(Steven) Li
2021-10-05 16:38               ` Ferruh Yigit
2021-10-06  7:55                 ` Xueming(Steven) Li
2021-10-06  8:04                   ` Ferruh Yigit
2021-10-06 11:19                     ` Xueming(Steven) Li
     [not found]           ` <2d2e9329b076c022418efd7b38ff280cf3ed1af4.camel@nvidia.com>
     [not found]             ` <56f7537a-bfc0-e4b8-72e8-c382ef0e2dbd@huawei.com>
     [not found]               ` <8e2c2f96265dc17af0564befb3918f1a8ea5154a.camel@nvidia.com>
2021-09-29 14:04                 ` [dpdk-dev] Fwd: " Xueming(Steven) Li
2021-09-30 15:17 ` [dpdk-dev] [PATCH v6 0/2] " Xueming Li
2021-09-30 15:17   ` [dpdk-dev] [PATCH v6 1/2] ethdev: make queue release callback optional Xueming Li
2021-10-05 22:04     ` Thomas Monjalon
2021-09-30 15:17   ` [dpdk-dev] [PATCH v6 2/2] ethdev: change queue release callback Xueming Li
2021-10-03  7:38     ` Matan Azrad
2021-10-03 21:00       ` Ajit Khaparde
2021-10-06 10:21     ` Somnath Kotur
2021-10-06 11:18 ` [dpdk-dev] [PATCH v7 0/2] " Xueming Li
2021-10-06 11:18   ` [dpdk-dev] [PATCH v7 1/2] ethdev: make queue release callback optional Xueming Li
2021-10-06 15:38     ` Hemant Agrawal
2021-10-08  8:16     ` Xu, Rosen
2021-10-06 11:18   ` [dpdk-dev] [PATCH v7 2/2] ethdev: change queue release callback Xueming Li
2021-10-06 17:20     ` Ferruh Yigit
2021-10-11  8:28     ` Thomas Monjalon
2021-10-11 13:11       ` Ferruh Yigit
2021-10-06 17:25   ` [dpdk-dev] [PATCH v7 0/2] " Ferruh Yigit

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=20210918123525.135129-1-xuemingl@nvidia.com \
    --to=xuemingl@nvidia.com \
    --cc=aman.deep.singh@intel.com \
    --cc=andrew.rybchenko@oktetlabs.ru \
    --cc=dev@dpdk.org \
    --cc=ferruh.yigit@intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).