public inbox for dev@dpdk.org
 help / color / mirror / Atom feed
From: "Morten Brørup" <mb@smartsharesystems.com>
To: "Stephen Hemminger" <stephen@networkplumber.org>,
	"Zhichao Zeng" <zhichaox.zeng@intel.com>
Cc: <dev@dpdk.org>, <stable@dpdk.org>,
	"Konstantin Ananyev" <konstantin.ananyev@huawei.com>,
	"Ferruh Yigit" <ferruh.yigit@amd.com>,
	"Shahaf Shuler" <shahafs@nvidia.com>,
	"Bruce Richardson" <bruce.richardson@intel.com>
Subject: RE: [PATCH] examples/ip_reassembly: remove conflicting Tx offload
Date: Wed, 14 Jan 2026 08:35:05 +0100	[thread overview]
Message-ID: <98CBD80474FA8B44BF855DF32C47DC35F65658@smartserver.smartshare.dk> (raw)
In-Reply-To: <20260113222742.0e17c7c4@phoenix.local>

+CC Bruce

> From: Stephen Hemminger [mailto:stephen@networkplumber.org]
> 
> On Fri, 19 Sep 2025 15:12:13 +0800
> Zhichao Zeng <zhichaox.zeng@intel.com> wrote:
> 
> > IP packet reassembly requires the MULTI_SEGS offload to be enabled in
> order
> > to handle fragmented packets across multiple memory segments.
> However, the
> > MBUF_FAST_FREE offload conflicts with MULTI_SEGS and cannot be
> enabled
> > simultaneously.
> >
> > Remove the MBUF_FAST_FREE offload to avoid conflicts with MULTI_SEGS.
> >
> > Fixes: 9df56e69ee3f ("examples/ip_reassembly: convert to new ethdev
> offloads API")
> > Cc: stable@dpdk.org
> >
> > Signed-off-by: Zhichao Zeng <zhichaox.zeng@intel.com>
> >
> 
> Later Tech Board discussion concluded that fast free and multi segs are
> not
> conflicting flags. Dropping this patch.

Let's postpone rather than drop this patch.

The documentation still says that FAST_FREE does not allow segmented packets.
And it has been reported that some drivers fail when using both flags.
The Tech Board wishes to get rid of the mutual exclusivity, to support jumbo frames with FAST_FREE; but the mutual exclusivity is still under investigation, and no final conclusion has been reached yet.
For now, it has been concluded that the mbuf library is not the reason for mutual exclusivity, but it has not yet been concluded if is required for driver implementation purposes.


      reply	other threads:[~2026-01-14  7:35 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-19  7:12 [PATCH] examples/ip_reassembly: remove conflicting Tx offload Zhichao Zeng
2025-09-19  7:13 ` Konstantin Ananyev
2025-09-28  9:04 ` Xu, HailinX
2026-01-14  6:27 ` Stephen Hemminger
2026-01-14  7:35   ` Morten Brørup [this message]

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=98CBD80474FA8B44BF855DF32C47DC35F65658@smartserver.smartshare.dk \
    --to=mb@smartsharesystems.com \
    --cc=bruce.richardson@intel.com \
    --cc=dev@dpdk.org \
    --cc=ferruh.yigit@amd.com \
    --cc=konstantin.ananyev@huawei.com \
    --cc=shahafs@nvidia.com \
    --cc=stable@dpdk.org \
    --cc=stephen@networkplumber.org \
    --cc=zhichaox.zeng@intel.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