net/mlx5: revert multicast rule verbs flow type
authorShahaf Shuler <shahafs@mellanox.com>
Tue, 13 Feb 2018 12:31:55 +0000 (14:31 +0200)
committerThomas Monjalon <thomas@monjalon.net>
Tue, 13 Feb 2018 16:00:37 +0000 (17:00 +0100)
commitc0ff2fb81464a1bbb1a4e0fa1614c1d11e0ebbf4
tree83b313f2ca03f6013fc7ccf55d24e5d6cdbf6981
parent8c5bca92c92cce90239ca813f293f3df53427775
net/mlx5: revert multicast rule verbs flow type

This is to revert the following commits:
commit da646bd93888 ("net/mlx5: fix all multi verification code position")
commit 0a40a1363a4d ("net/mlx5: fix flow type for allmulti rules")

The last one introduced a bug in the following diff:
@ -1262,6 +1274,7 @@ struct ibv_spec_header {
                eth.val.ether_type &= eth.mask.ether_type;
        }
        mlx5_flow_create_copy(parser, &eth, eth_size);
+       parser->allmulti = eth.val.dst_mac[0] & 1;
        return 0;
 }

As broadcast rules will be considered of type allmulti as well.

The patch was originally intended to enable VF to receive all multicast
traffic by using the IBV_FLOW_ATTR_MC_DEFAULT flow type.
Since the support was removed from the kernel there is no point with
fixing this issue, hence the revert.

Fixes: da646bd93888 ("net/mlx5: fix all multi verification code position")
Fixes: 0a40a1363a4d ("net/mlx5: fix flow type for allmulti rules")
Cc: stable@dpdk.org
Signed-off-by: Shahaf Shuler <shahafs@mellanox.com>
Acked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
drivers/net/mlx5/mlx5_flow.c