DPDK patches and discussions
 help / color / mirror / Atom feed
From: Ciara Loftus <ciara.loftus@intel.com>
To: dev@dpdk.org
Cc: Ciara Loftus <ciara.loftus@intel.com>
Subject: [PATCH v2 0/7] net/intel: fixes and improvements to rx path selection
Date: Wed, 15 Oct 2025 10:07:16 +0000	[thread overview]
Message-ID: <20251015100723.1603296-1-ciara.loftus@intel.com> (raw)
In-Reply-To: <20251014084517.1407407-1-ciara.loftus@intel.com>

This series contains a number of fixes and improvements to the logic
concerned with selecting an rx path in the intel drivers.

Patch 1 fixes incorrect behaviour in the ci_rxq_vec_capable function
which would disqualify rxqs that have the timestamp offload enabled from
selecting vector paths. This was incorrect because iavf vector paths
support timestamp offload.

Patch 2 removes a reduntant check from the ci_rxq_vec_capable function
that disqualifies rxqs that have the buffer split offload enabled from
selecting vector paths. This check is performed during the common rx
path selection for three of the drivers and during dev_configure for the
fourth, so the check in ci_rxq_vec_capable is unneeded. Although it is
similar this patch was kept separate from patch 1 because it is only
relevant after the three aformention drivers adopted the common rx path
selection infrastructure, whereas patch 1 is a fix for an issue that
existed prior to that.

Patches 3 and 4 make fixes to the iavf rx path definitions, ensuring the
correct offloads and features are defined for all paths.

The final three patches reformat the arrays in the i40e iavf and ice
drivers that hold information for each Rx path that are used in the
common selection process, in an attempt to improve readability.

v2:
* Added a new patch (2) that removes the reduntant check for the buffer
split offload in the common function.
* Removed the useless defines IAVF_RX_NO_OFFLOADS and ICE_RX_NO_OFFLOADS
* Use a newline for closing braces in the rx path infos arrays
* Let a simd width of zero represent scalar paths instead of 64 which
lessens the number of fields that need to be initialised for scalar path
definitions

Ciara Loftus (7):
  net/intel: fix Rx vector capability detection
  net/intel: remove redundant Rx offload check
  net/iavf: fix Rx paths feature definitions
  net/iavf: fix Rx path selection for scalar flex bulk alloc
  net/iavf: reformat the Rx path infos array
  net/i40e: reformat the Rx path infos array
  net/ice: reformat the Rx path infos array

 drivers/net/intel/common/rx.h                 |   7 +-
 drivers/net/intel/i40e/i40e_rxtx.c            | 149 ++++++--
 drivers/net/intel/i40e/i40e_rxtx_vec_common.h |   2 +-
 drivers/net/intel/iavf/iavf.h                 |   1 +
 drivers/net/intel/iavf/iavf_rxtx.c            | 335 ++++++++++++++----
 drivers/net/intel/iavf/iavf_rxtx.h            |   2 -
 drivers/net/intel/ice/ice_rxtx.c              | 147 ++++++--
 drivers/net/intel/ice/ice_rxtx.h              |   1 -
 drivers/net/intel/ice/ice_rxtx_vec_common.h   |   2 +-
 .../net/intel/ixgbe/ixgbe_rxtx_vec_common.c   |   2 +-
 10 files changed, 498 insertions(+), 150 deletions(-)

-- 
2.34.1


  parent reply	other threads:[~2025-10-15 10:07 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-14  8:45 [PATCH 0/6] " Ciara Loftus
2025-10-14  8:45 ` [PATCH 1/6] net/intel: fix Rx vector capability detection Ciara Loftus
2025-10-14 14:16   ` Bruce Richardson
2025-10-14 14:59     ` Loftus, Ciara
2025-10-14  8:45 ` [PATCH 2/6] net/iavf: fix Rx paths feature definitions Ciara Loftus
2025-10-14 14:26   ` Bruce Richardson
2025-10-14  8:45 ` [PATCH 3/6] net/iavf: fix Rx path selection for scalar flex bulk alloc Ciara Loftus
2025-10-14 14:33   ` Bruce Richardson
2025-10-14  8:45 ` [PATCH 4/6] net/iavf: reformat the Rx path infos array Ciara Loftus
2025-10-14 14:38   ` Bruce Richardson
2025-10-14  8:45 ` [PATCH 5/6] net/i40e: " Ciara Loftus
2025-10-14 14:38   ` Bruce Richardson
2025-10-14  8:45 ` [PATCH 6/6] net/ice: " Ciara Loftus
2025-10-14 14:39   ` Bruce Richardson
2025-10-15 10:07 ` Ciara Loftus [this message]
2025-10-15 10:07   ` [PATCH v2 1/7] net/intel: fix Rx vector capability detection Ciara Loftus
2025-10-15 10:07   ` [PATCH v2 2/7] net/intel: remove redundant Rx offload check Ciara Loftus
2025-10-15 10:07   ` [PATCH v2 3/7] net/iavf: fix Rx paths feature definitions Ciara Loftus
2025-10-15 10:07   ` [PATCH v2 4/7] net/iavf: fix Rx path selection for scalar flex bulk alloc Ciara Loftus
2025-10-15 10:07   ` [PATCH v2 5/7] net/iavf: reformat the Rx path infos array Ciara Loftus
2025-10-15 10:07   ` [PATCH v2 6/7] net/i40e: " Ciara Loftus
2025-10-15 10:07   ` [PATCH v2 7/7] net/ice: " Ciara Loftus

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=20251015100723.1603296-1-ciara.loftus@intel.com \
    --to=ciara.loftus@intel.com \
    --cc=dev@dpdk.org \
    /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).