From: Patrick Robb <probb@iol.unh.edu>
To: Bruce Richardson <bruce.richardson@intel.com>
Cc: Dean Marx <dmarx@iol.unh.edu>, dev@dpdk.org, techboard@dpdk.org
Subject: Re: [RFC PATCH] doc: clarify VLAN and QinQ stripping behaviour
Date: Tue, 15 Jul 2025 17:15:34 -0400 [thread overview]
Message-ID: <CAJvnSUBSZo8OAZ+_xjFm3=5FG2EBJ2UiF6KfkPb_iNz2_1aoyg@mail.gmail.com> (raw)
In-Reply-To: <aHYHj9vtdXm8MgY_@bricha3-mobl1.ger.corp.intel.com>
[-- Attachment #1: Type: text/plain, Size: 1718 bytes --]
On Tue, Jul 15, 2025 at 3:47 AM Bruce Richardson <bruce.richardson@intel.com>
wrote:
>
>
> Thanks. Let me know how the testing otherwise goes. If only Intel NICs are
> supporting QinQ, then it can't be that widely used a feature yet, so we may
> be able to tweak things a bit if it diverges from what is expected.
>
Yeah, there seems to be fairly broad support for VLAN strip, but much less
for QinQ strip. So, we can verify "correct" VLAN stripping behavior given
the various packet profiles we discussed previously like double vlan,
vlan + qinq, single qinq etc.
For QinQ strip like Dean said, most PMDs dont support this currently, with
Intel PMDs being an exception among the NICs we have at UNH. We took a look
at the DPDK networking drivers support table which reflects this (see the
QinQ offload row). https://doc.dpdk.org/guides/nics/overview.html
So, it looks like there are a few (ZTE DXDH is an example) that are also
supporting QinQ Offload, but not many right now.
> I also wonder if the definition of expected behaviour is preventing other
> NICs from implementing the feature?
>
> Can you also check with VLAN stripping enabled? My biggest issue with the
> behaviour description of QinQ strip only removing outer tag is that it
> implies for QinQ traffoc that VLAN strip alone should strip inner tag
> without
> removing outer.
Sure, we can check this case. across the NICs at UNH. I have a sync
wth Dean tomorrow morning for this, at the end of which we'll share back to
you what we have.
> That doesn't make sense to me - and I'm not sure if NIC
> hardware supports such features of removing inner headers while leaving
> outer.
>
> /Bruce
>
[-- Attachment #2: Type: text/html, Size: 2550 bytes --]
next prev parent reply other threads:[~2025-07-15 21:21 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-07-14 13:30 Bruce Richardson
2025-07-14 15:06 ` Stephen Hemminger
2025-07-14 15:11 ` Bruce Richardson
2025-07-14 16:33 ` Morten Brørup
2025-07-14 16:49 ` Bruce Richardson
2025-07-15 15:04 ` Morten Brørup
2025-07-15 16:20 ` Bruce Richardson
2025-07-14 20:09 ` Dean Marx
2025-07-14 21:41 ` Patrick Robb
2025-07-15 7:47 ` Bruce Richardson
2025-07-15 21:15 ` Patrick Robb [this message]
2025-07-16 10:11 ` 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='CAJvnSUBSZo8OAZ+_xjFm3=5FG2EBJ2UiF6KfkPb_iNz2_1aoyg@mail.gmail.com' \
--to=probb@iol.unh.edu \
--cc=bruce.richardson@intel.com \
--cc=dev@dpdk.org \
--cc=dmarx@iol.unh.edu \
--cc=techboard@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).