From: Thomas Monjalon <thomas@monjalon.net>
To: Bing Zhao <bingz@nvidia.com>, Raslan Darawsheh <rasland@nvidia.com>
Cc: Slava Ovsiienko <viacheslavo@nvidia.com>,
"dev@dpdk.org" <dev@dpdk.org>, Ori Kam <orika@nvidia.com>,
Dariusz Sosnowski <dsosnowski@nvidia.com>,
Suanming Mou <suanmingm@nvidia.com>,
Matan Azrad <matan@nvidia.com>
Subject: Re: [PATCH v2] net/mlx5: add metadata split for compatibility API
Date: Thu, 08 Aug 2024 13:27:15 +0200 [thread overview]
Message-ID: <3297286.oiGErgHkdL@thomas> (raw)
In-Reply-To: <MN0PR12MB60560F757C78580147C022F1CFDB2@MN0PR12MB6056.namprd12.prod.outlook.com>
09/07/2024 11:36, Raslan Darawsheh:
> Hi,
>
> From: Bing Zhao <bingz@nvidia.com>
> Sent: Thursday, July 4, 2024 1:26 PM
> To: Slava Ovsiienko; dev@dpdk.org; Raslan Darawsheh
> Cc: Ori Kam; Dariusz Sosnowski; Suanming Mou; Matan Azrad
> Subject: [PATCH v2] net/mlx5: add metadata split for compatibility API
>
> The method will not create any new flow rule implicitly during split
> stage, but only generate needed items, actions and attributes based
> on the detection.
>
> All the rules will still be created in the flow_hw_list_create().
>
> In the meanwhile, once the mark action is specified in the FDB rule,
> a new rule in the NIC Rx will be created to:
> 1. match the mark value on REG_C_x in FDB and set it into Rx
> flow tag field.
> 2. copy the metadata in REG_C_x' into the REG_B.
>
> If there is no mark, the default rule with only copying metadata
> will be hit if there is Queue or RSS action in the NIC Rx rule.
>
> Regarding the NIC Tx, only the metadata is relevant and it will be
> copied in NIC Tx from REG_A into some REG_C_x. The current HWS
> implementation already has already supported in the default copy
> rule or the default SQ miss rule in the NIC Tx root table.
>
> Signed-off-by: Bing Zhao <bingz@nvidia.com>
> Acked-by: Dariusz Sosnowski <dsosnowski@nvidia.com>
>
> Patch applied to next-net-mlx,
It looks like this patch was missed,
probably not pushed in the branch.
Please send a v3 for integration in 24.11.
next prev parent reply other threads:[~2024-08-08 11:27 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-13 12:51 [PATCH] net/mlx5: add non-template flow metadata split Bing Zhao
2024-07-04 10:26 ` [PATCH v2] net/mlx5: add metadata split for compatibility API Bing Zhao
2024-07-09 9:36 ` Raslan Darawsheh
2024-08-08 11:27 ` Thomas Monjalon [this message]
2024-08-29 8:58 ` Raslan Darawsheh
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=3297286.oiGErgHkdL@thomas \
--to=thomas@monjalon.net \
--cc=bingz@nvidia.com \
--cc=dev@dpdk.org \
--cc=dsosnowski@nvidia.com \
--cc=matan@nvidia.com \
--cc=orika@nvidia.com \
--cc=rasland@nvidia.com \
--cc=suanmingm@nvidia.com \
--cc=viacheslavo@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.