From: Raslan Darawsheh Date: Tue, 5 Dec 2017 09:37:50 +0000 (+0200) Subject: net/mlx5: fix flow type for allmulti rules X-Git-Tag: spdx-start~578 X-Git-Url: http://git.droids-corp.org/?a=commitdiff_plain;h=0a40a1363a4d;p=dpdk.git net/mlx5: fix flow type for allmulti rules Chnaged ibv_flow_attr type for allmulti rule to IBV_FLOW_ATTR_MC_DEFAULT instead of IBV_FLOW_ATTR_NORMAL, in case allmulti was enabled. Fixes: 272733b5 ("net/mlx5: use flow to enable unicast traffic") Cc: stable@dpdk.org Signed-off-by: Raslan Darawsheh Acked-by: Nelio Laranjeiro --- diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c index 1eda836710..6605cfd9f5 100644 --- a/drivers/net/mlx5/mlx5_flow.c +++ b/drivers/net/mlx5/mlx5_flow.c @@ -429,6 +429,7 @@ static const struct mlx5_flow_items mlx5_flow_items[] = { /** Structure to pass to the conversion function. */ struct mlx5_flow_parse { uint32_t inner; /**< Set once VXLAN is encountered. */ + uint32_t allmulti:1; /**< Set once allmulti dst MAC is encountered. */ uint32_t create:1; /**< Whether resources should remain after a validate. */ uint32_t drop:1; /**< Target is a drop queue. */ @@ -1165,6 +1166,17 @@ exit_free: } } } + if (parser->allmulti && + parser->layer == HASH_RXQ_ETH) { + for (i = 0; i != hash_rxq_init_n; ++i) { + if (!parser->queue[i].ibv_attr) + continue; + if (parser->queue[i].ibv_attr->num_of_specs != 1) + break; + parser->queue[i].ibv_attr->type = + IBV_FLOW_ATTR_MC_DEFAULT; + } + } return ret; exit_enomem: for (i = 0; i != hash_rxq_init_n; ++i) { @@ -1262,6 +1274,7 @@ mlx5_flow_create_eth(const struct rte_flow_item *item, eth.val.ether_type &= eth.mask.ether_type; } mlx5_flow_create_copy(parser, ð, eth_size); + parser->allmulti = eth.val.dst_mac[0] & 1; return 0; }