> On Feb 24, 2019, at 1:41 AM, Dekel Peled <dek...@mellanox.com> wrote:
> 
> Original patch changed logic of function mlx4_flow_merge_eth().
> The setting of flow->promisc was wrongly removed.
> This patch adds the removed setting of flow->promisc, to restore
> the required behavior.
> 
> Fixes: c0d239263156 ("net/mlx4: support flow w/o ETH spec and with VLAN")
> Cc: sta...@dpdk.org
> 
> ---
> v2: Correct typo in 'stable' email address.
> ---
> 
> Signed-off-by: Dekel Peled <dek...@mellanox.com>
> ---

Acked-by: Yongseok Koh <ys...@mellanox.com>
 
Thanks

> drivers/net/mlx4/mlx4_flow.c | 1 +
> 1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/net/mlx4/mlx4_flow.c b/drivers/net/mlx4/mlx4_flow.c
> index 3abde30..b7af324 100644
> --- a/drivers/net/mlx4/mlx4_flow.c
> +++ b/drivers/net/mlx4/mlx4_flow.c
> @@ -250,6 +250,7 @@ struct mlx4_drop {
>       if (!mask) {
>               eth->val.dst_mac[0] = 0xff;
>               flow->ibv_attr->type = IBV_FLOW_ATTR_ALL_DEFAULT;
> +             flow->promisc = 1;
>               return 0;
>       }
>       memcpy(eth->val.dst_mac, spec->dst.addr_bytes, ETHER_ADDR_LEN);
> -- 
> 1.8.3.1
> 

Reply via email to