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
next prev 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).