On Mon, Sep 14, 2026 at 03:18:57PM +0100, Loftus, Ciara wrote:
> > Subject: [PATCH 13/13] net/ice: enable context desc offloads for vector Tx
> > 
> > Use the context descriptor Tx vector paths from intel/common in ice
> > driver to enable more offloads on the vector path for ice driver.
> > 
> > Signed-off-by: Bruce Richardson <[email protected]>
> > ---
> >  doc/guides/rel_notes/release_26_11.rst      |  6 ++++
> >  drivers/net/intel/ice/ice_ethdev.h          |  3 ++
> >  drivers/net/intel/ice/ice_rxtx.c            | 31 ++++++++++++++++++---
> >  drivers/net/intel/ice/ice_rxtx.h            | 11 ++++++++
> >  drivers/net/intel/ice/ice_rxtx_vec_avx2.c   | 24 ++++++++++++++++
> >  drivers/net/intel/ice/ice_rxtx_vec_avx512.c | 24 ++++++++++++++++
> >  6 files changed, 95 insertions(+), 4 deletions(-)
> > 
> 
> <snip>
> 
> > diff --git a/drivers/net/intel/ice/ice_rxtx.h 
> > b/drivers/net/intel/ice/ice_rxtx.h
> > index 999b6b30d6..bee8a464f5 100644
> > --- a/drivers/net/intel/ice/ice_rxtx.h
> > +++ b/drivers/net/intel/ice/ice_rxtx.h
> > @@ -135,6 +135,12 @@
> >     RTE_ETH_TX_OFFLOAD_UDP_CKSUM |          \
> >     RTE_ETH_TX_OFFLOAD_TCP_CKSUM |          \
> >     RTE_ETH_TX_OFFLOAD_SCTP_CKSUM)
> > +/* vector ctx offload path: QinQ outer tag + outer/tunnel checksum via ctx
> > desc */
> > +#define ICE_TX_VECTOR_CTX_OFFLOAD_OFFLOADS (       \
> > +   ICE_TX_VECTOR_OFFLOAD_OFFLOADS |        \
> > +   RTE_ETH_TX_OFFLOAD_OUTER_IPV4_CKSUM |   \
> > +   RTE_ETH_TX_OFFLOAD_OUTER_UDP_CKSUM |    \
> > +   RTE_ETH_TX_OFFLOAD_QINQ_INSERT)
> 
> Now that the paths and thus offloads are shared, is it worth creating
> common OFFLOADS masks? I assume
> IAVF_TX_VECTOR_CTX_OFFLOAD_OFFLOADS is an exact match for
> ICE_TX_VECTOR_CTX_OFFLOAD_OFFLOADS
> 
Yes, it is probably worth doing. Not sure if it's best in this patchset or
not. Let me investigate.

/Bruce

Reply via email to