patches for DPDK stable branches
 help / color / mirror / Atom feed
From: Luca Boccassi <luca.boccassi@gmail.com>
To: Jiawen Wu <jiawenwu@trustnetic.com>
Cc: stable@dpdk.org
Subject: Re: [PATCH 22.11] net/txgbe: fix to create FDIR filter for tunnel packet
Date: Tue, 15 Jul 2025 11:04:12 +0100	[thread overview]
Message-ID: <CAMw=ZnSu79zyUu-fU4sB9CWbdtEjASYbumhYeRhm473FtxFA1Q@mail.gmail.com> (raw)
In-Reply-To: <20250715034849.1297724-1-jiawenwu@trustnetic.com>

On Tue, 15 Jul 2025 at 04:24, Jiawen Wu <jiawenwu@trustnetic.com> wrote:
>
> [ upstream commit a1851465f8252ee75a26d05b9b2d3dca7023e8f2 ]
>
> Fix to create FDIR rules for VXLAN/GRE/NVGRE/GENEVE packets, they will
> match the rules in the inner layers.
>
> Fixes: b973ee26747a ("net/txgbe: parse flow director filter")
> Cc: stable@dpdk.org
>
> Signed-off-by: Jiawen Wu <jiawenwu@trustnetic.com>
> ---
>  doc/guides/nics/features/txgbe.ini |   2 +
>  drivers/net/txgbe/txgbe_ethdev.h   |   1 -
>  drivers/net/txgbe/txgbe_flow.c     | 585 +++++++++++++++++++++++------
>  3 files changed, 478 insertions(+), 110 deletions(-)

Hi,

Thanks for the patch, but this doesn't solve the problem, it still
fails to build:

../drivers/net/txgbe/txgbe_flow.c: In function ‘txgbe_parse_fdir_filter_tunnel’:
../drivers/net/txgbe/txgbe_flow.c:2599:60: error: request for member
‘a’ in something not a structure or union
 2599 |                                 if
(ipv6_mask->hdr.src_addr.a[j] == UINT8_MAX) {
      |                                                            ^
../drivers/net/txgbe/txgbe_flow.c:2601:67: error: request for member
‘a’ in something not a structure or union
 2601 |                                 } else if
(ipv6_mask->hdr.src_addr.a[j] != 0) {
      |                                                                   ^
../drivers/net/txgbe/txgbe_flow.c:2612:60: error: request for member
‘a’ in something not a structure or union
 2612 |                                 if
(ipv6_mask->hdr.dst_addr.a[j] == UINT8_MAX) {
      |                                                            ^
../drivers/net/txgbe/txgbe_flow.c:2614:67: error: request for member
‘a’ in something not a structure or union
 2614 |                                 } else if
(ipv6_mask->hdr.dst_addr.a[j] != 0) {


Could you please test that it compiles and works, and send a new
revision? Thanks

      reply	other threads:[~2025-07-15 10:04 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-07-15  3:48 Jiawen Wu
2025-07-15 10:04 ` Luca Boccassi [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='CAMw=ZnSu79zyUu-fU4sB9CWbdtEjASYbumhYeRhm473FtxFA1Q@mail.gmail.com' \
    --to=luca.boccassi@gmail.com \
    --cc=jiawenwu@trustnetic.com \
    --cc=stable@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).