Sorry, I misunderstood your comment and update it in patch v5.

Regards,
Li Zhang

-----Original Message-----
From: Thomas Monjalon <tho...@monjalon.net> 
Sent: Tuesday, October 26, 2021 6:16 PM
To: Li Zhang <l...@nvidia.com>
Cc: Ori Kam <or...@nvidia.com>; Slava Ovsiienko <viachesl...@nvidia.com>; Matan 
Azrad <ma...@nvidia.com>; Shahaf Shuler <shah...@nvidia.com>; Shun Hao 
<sh...@nvidia.com>; dev@dpdk.org; Raslan Darawsheh <rasl...@nvidia.com>; Roni 
Bar Yanai <ron...@nvidia.com>; sta...@dpdk.org
Subject: Re: [PATCH v4] doc: document a limitation for a meter with RSS action

External email: Use caution opening links or attachments


26/10/2021 12:10, Li Zhang:
> A meter policy with RSS/Queue action is not supported when dv_xmeta_en 
> enabled.
>
> When dv_xmeta_en enabled in legacy creating flow, it will split into 
> two flows (one set_tag with jump flow and one RSS/queue action flow).
> For meter policy as termination table, it cannot split flow and cannot 
> support dv_xmeta_en enabled.

Thank you for the explanation.

> Fixes: 51ec04d ("net/mlx5: connect meter policy to created flows")
> Cc: sta...@dpdk.org
>
> Signed-off-by: Li Zhang <l...@nvidia.com>
> Acked-by: Matan Azrad <ma...@nvidia.com>
> ---
>  doc/guides/nics/mlx5.rst | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/doc/guides/nics/mlx5.rst b/doc/guides/nics/mlx5.rst index 
> 47709d93b3..2f46ed0263 100644
> --- a/doc/guides/nics/mlx5.rst
> +++ b/doc/guides/nics/mlx5.rst
> @@ -435,6 +435,7 @@ Limitations
>       - yellow: QUEUE, RSS, PORT_ID, REPRESENTED_PORT, JUMP, DROP, MARK and 
> SET_TAG.
>       - RED: must be DROP.
>    - Policy actions of RSS for green and yellow should have the same 
> configuration except queues.
> +  - Policy with RSS/queue action is not supported when dv_xmeta_en enabled.

Did you understand my comment about the formatting?
You must add quotes like that: ``dv_xmeta_en`` Look at other occurences in the 
doc.




Reply via email to