> -----Original Message-----
> From: Dekel Peled <dek...@mellanox.com>
> Sent: Thursday, November 7, 2019 17:04
> To: Matan Azrad <ma...@mellanox.com>; Shahaf Shuler
> <shah...@mellanox.com>; Slava Ovsiienko <viachesl...@mellanox.com>
> Cc: Ori Kam <or...@mellanox.com>; dev@dpdk.org
> Subject: [PATCH] net/mlx5: fix condition to create default rule
>
> Previous patch added creation of a default flow rule on port start.
> Rule is created under the condition that device is in eswitch mode, and is not
> a VF, to make sure rule is created only once.
> In Bluefield, where PF representor is used, this condition is not sufficient.
> Rule is created twice, causing loss of traffic.
>
> This patch updates this condition, adding check that device is also not a
> representor.
>
> Fixes: b67b4ecbde22 ("net/mlx5: skip table zero to improve insertion rate")
>
> Signed-off-by: Dekel Peled <dek...@mellanox.com>
> ---
> drivers/net/mlx5/mlx5_trigger.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/mlx5/mlx5_trigger.c
> b/drivers/net/mlx5/mlx5_trigger.c index cafab25..f4dc047 100644
> --- a/drivers/net/mlx5/mlx5_trigger.c
> +++ b/drivers/net/mlx5/mlx5_trigger.c
> @@ -420,7 +420,7 @@
> }
> mlx5_txq_release(dev, i);
> }
> - if (priv->config.dv_esw_en && !priv->config.vf)
> + if (priv->config.dv_esw_en && !priv->config.vf && !priv-
> >representor)
> if (!mlx5_flow_create_esw_table_zero_flow(dev))
> goto error;
> if (priv->isolated)
> --
> 1.8.3.1
Acked-by: Viacheslav Ovsiienko <viachesl...@mellanox.com>
