DPDK-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] doc/mlx5: document modify field limitation on group 0
@ 2026-07-30 15:44 Maayan Kashani
  2026-08-10  7:08 ` Bing Zhao
  2026-08-31 15:41 ` Raslan Darawsheh
  0 siblings, 2 replies; 3+ messages in thread
From: Maayan Kashani @ 2026-07-30 15:44 UTC (permalink / raw)
  To: dev
  Cc: mkashani, rasland, Dariusz Sosnowski, Viacheslav Ovsiienko,
	Bing Zhao, Ori Kam, Suanming Mou, Matan Azrad

Document that on the root table, two sequential MODIFY_FIELD actions
where the second uses a field modified by the first as src are not
supported: the second action is handled before the first change is
applied. Only immediate src types (VALUE/POINTER) are allowed on
group 0 for sync HWS non-template and DV flows.

Signed-off-by: Maayan Kashani <mkashani@nvidia.com>
---
 doc/guides/nics/mlx5.rst | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/doc/guides/nics/mlx5.rst b/doc/guides/nics/mlx5.rst
index 9b035b878e0..aa751669444 100644
--- a/doc/guides/nics/mlx5.rst
+++ b/doc/guides/nics/mlx5.rst
@@ -3425,6 +3425,12 @@ Limitations
 
 #. In template tables of group 0, the modify action must be fully masked.
 
+#. On group 0 (root table), when using sync API, two sequential
+   ``RTE_FLOW_ACTION_TYPE_MODIFY_FIELD`` actions where the first modifies a
+   packet or metadata field and the second uses that field as ``src`` are not
+   supported. On the root table the second action is handled before the first
+   change is applied, so the source still reads the original field value.
+
 #. Modification of an arbitrary place in a packet via
    the special ``RTE_FLOW_FIELD_START`` field ID is not supported.
 
-- 
2.21.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* RE: [PATCH] doc/mlx5: document modify field limitation on group 0
  2026-07-30 15:44 [PATCH] doc/mlx5: document modify field limitation on group 0 Maayan Kashani
@ 2026-08-10  7:08 ` Bing Zhao
  2026-08-31 15:41 ` Raslan Darawsheh
  1 sibling, 0 replies; 3+ messages in thread
From: Bing Zhao @ 2026-08-10  7:08 UTC (permalink / raw)
  To: Maayan Kashani, dev@dpdk.org
  Cc: Raslan Darawsheh, Dariusz Sosnowski, Slava Ovsiienko, Ori Kam,
	Suanming Mou, Matan Azrad

Hi,

> -----Original Message-----
> From: Maayan Kashani <mkashani@nvidia.com>
> Sent: Thursday, July 30, 2026 11:45 PM
> To: dev@dpdk.org
> Cc: Maayan Kashani <mkashani@nvidia.com>; Raslan Darawsheh
> <rasland@nvidia.com>; Dariusz Sosnowski <dsosnowski@nvidia.com>; Slava
> Ovsiienko <viacheslavo@nvidia.com>; Bing Zhao <bingz@nvidia.com>; Ori Kam
> <orika@nvidia.com>; Suanming Mou <suanmingm@nvidia.com>; Matan Azrad
> <matan@nvidia.com>
> Subject: [PATCH] doc/mlx5: document modify field limitation on group 0
> 
> Document that on the root table, two sequential MODIFY_FIELD actions where
> the second uses a field modified by the first as src are not
> supported: the second action is handled before the first change is
> applied. Only immediate src types (VALUE/POINTER) are allowed on group 0
> for sync HWS non-template and DV flows.
> 
> Signed-off-by: Maayan Kashani <mkashani@nvidia.com>
> ---
>  doc/guides/nics/mlx5.rst | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/doc/guides/nics/mlx5.rst b/doc/guides/nics/mlx5.rst index
> 9b035b878e0..aa751669444 100644
> --- a/doc/guides/nics/mlx5.rst
> +++ b/doc/guides/nics/mlx5.rst
> @@ -3425,6 +3425,12 @@ Limitations
> 
>  #. In template tables of group 0, the modify action must be fully masked.
> 
> +#. On group 0 (root table), when using sync API, two sequential
> +   ``RTE_FLOW_ACTION_TYPE_MODIFY_FIELD`` actions where the first modifies
> a
> +   packet or metadata field and the second uses that field as ``src`` are
> not
> +   supported. On the root table the second action is handled before the
> first
> +   change is applied, so the source still reads the original field value.
> +
>  #. Modification of an arbitrary place in a packet via
>     the special ``RTE_FLOW_FIELD_START`` field ID is not supported.
> 
> --
> 2.21.0

Acked-by: Bing Zhao <bingz@nvidia.com>


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] doc/mlx5: document modify field limitation on group 0
  2026-07-30 15:44 [PATCH] doc/mlx5: document modify field limitation on group 0 Maayan Kashani
  2026-08-10  7:08 ` Bing Zhao
@ 2026-08-31 15:41 ` Raslan Darawsheh
  1 sibling, 0 replies; 3+ messages in thread
From: Raslan Darawsheh @ 2026-08-31 15:41 UTC (permalink / raw)
  To: Maayan Kashani, dev
  Cc: Dariusz Sosnowski, Viacheslav Ovsiienko, Bing Zhao, Ori Kam,
	Suanming Mou, Matan Azrad

Hi,


On 30/07/2026 6:44 PM, Maayan Kashani wrote:
> Document that on the root table, two sequential MODIFY_FIELD actions
> where the second uses a field modified by the first as src are not
> supported: the second action is handled before the first change is
> applied. Only immediate src types (VALUE/POINTER) are allowed on
> group 0 for sync HWS non-template and DV flows.
> 
> Signed-off-by: Maayan Kashani <mkashani@nvidia.com>
> ---

Patch applied to next-net-mlx,

Kindest regards
Raslan Darawsheh


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-08-31 15:41 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-30 15:44 [PATCH] doc/mlx5: document modify field limitation on group 0 Maayan Kashani
2026-08-10  7:08 ` Bing Zhao
2026-08-31 15:41 ` Raslan Darawsheh

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox