DPDK patches and discussions
 help / color / mirror / Atom feed
From: David Marchand <david.marchand@redhat.com>
To: "Loftus, Ciara" <ciara.loftus@intel.com>,
	Kevin Traynor <ktraynor@redhat.com>,
	 Luca Boccassi <luca.boccassi@gmail.com>,
	"Xueming(Steven) Li" <xuemingl@nvidia.com>
Cc: "dev@dpdk.org" <dev@dpdk.org>,
	"Richardson, Bruce" <bruce.richardson@intel.com>,
	dpdk stable <stable@dpdk.org>
Subject: Re: [PATCH v3 10/15] net/ice: remove unsupported Rx offload
Date: Mon, 15 Sep 2025 10:59:43 +0200	[thread overview]
Message-ID: <CAJFAV8xp1Bckbudei4JFAFAacFYi8wTd50TnWXT0BqFSOMixSg@mail.gmail.com> (raw)
In-Reply-To: <DM3PPF7D18F34A18F628B6488BAB7ADD7608E15A@DM3PPF7D18F34A1.namprd11.prod.outlook.com>

On Mon, 15 Sept 2025 at 10:45, Loftus, Ciara <ciara.loftus@intel.com> wrote:
>
> >
> > On Mon, 18 Aug 2025 at 13:00, Ciara Loftus <ciara.loftus@intel.com> wrote:
> > >
> > > The offload RTE_ETH_RX_OFFLOAD_SCTP_CKSUM is not supported in the ice
> > > driver so remove the one erroneous use of it.
> > >
> > > Signed-off-by: Ciara Loftus <ciara.loftus@intel.com>
> > > Acked-by: Bruce Richardson <bruce.richardson@intel.com>
> >
> > I just noticed this fix in main.
> > Fixes: 808a17b3c1e6 ("net/ice: add Rx AVX512 offload path")
> >
> > This looks like something worth fixing in LTS.
> > Was it intentional not marking it for backport?
>
> Apologies, it was not intentional not marking it for backport. I should have added that.

The fault is not yours.
Let's blame the reviewer instead ;-).


> Shall I submit a new patch to be backported or can this one be used?

It is ok.
Just, the scripts won't catch this fix, so let's ping people.
@LTS maintainers, fyi.


-- 
David Marchand


  reply	other threads:[~2025-09-15  9:00 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-18 10:58 [PATCH v3 00/15] net/intel: rx path selection simplification Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 01/15] net/ice: use the same Rx path across process types Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 02/15] net/iavf: rename Rx/Tx function type variables Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 03/15] net/iavf: use the same Rx path across process types Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 04/15] net/i40e: " Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 05/15] net/intel: introduce common vector capability function Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 06/15] net/ice: use the new " Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 07/15] net/iavf: " Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 08/15] net/i40e: " Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 09/15] net/iavf: remove redundant field from iavf adapter struct Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 10/15] net/ice: remove unsupported Rx offload Ciara Loftus
2025-09-15  8:09   ` David Marchand
2025-09-15  8:45     ` Loftus, Ciara
2025-09-15  8:59       ` David Marchand [this message]
2025-09-15  9:39         ` Kevin Traynor
2025-08-18 10:59 ` [PATCH v3 11/15] net/iavf: reorder enum of Rx function types Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 12/15] net/intel: introduce infrastructure for Rx path selection Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 13/15] net/ice: use the common Rx path selection infrastructure Ciara Loftus
2025-08-20  9:47   ` Bruce Richardson
2025-08-18 10:59 ` [PATCH v3 14/15] net/iavf: " Ciara Loftus
2025-08-20  9:49   ` Bruce Richardson
2025-08-18 10:59 ` [PATCH v3 15/15] net/i40e: " Ciara Loftus
2025-08-20  9:51   ` Bruce Richardson
2025-08-20 12:15 ` [PATCH v3 00/15] net/intel: rx path selection simplification Bruce Richardson

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=CAJFAV8xp1Bckbudei4JFAFAacFYi8wTd50TnWXT0BqFSOMixSg@mail.gmail.com \
    --to=david.marchand@redhat.com \
    --cc=bruce.richardson@intel.com \
    --cc=ciara.loftus@intel.com \
    --cc=dev@dpdk.org \
    --cc=ktraynor@redhat.com \
    --cc=luca.boccassi@gmail.com \
    --cc=stable@dpdk.org \
    --cc=xuemingl@nvidia.com \
    /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).