net/mlx5: fix all multi verification code position
[dpdk.git] / drivers / net / mlx5 / mlx5_flow.c
index 466d74b..20c4185 100644 (file)
 #pragma GCC diagnostic error "-Wpedantic"
 #endif
 
-#include <rte_ethdev.h>
+#include <rte_ethdev_driver.h>
 #include <rte_flow.h>
 #include <rte_flow_driver.h>
 #include <rte_malloc.h>
+#include <rte_ip.h>
 
 #include "mlx5.h"
+#include "mlx5_defs.h"
 #include "mlx5_prm.h"
 
 /* Define minimal priority for control plane flows. */
@@ -250,11 +252,8 @@ struct rte_flow {
        uint8_t rss_key[40]; /**< copy of the RSS key. */
        struct ibv_counter_set *cs; /**< Holds the counters for the rule. */
        struct mlx5_flow_counter_stats counter_stats;/**<The counter stats. */
-       union {
-               struct mlx5_flow frxq[RTE_DIM(hash_rxq_init)];
-               /**< Flow with Rx queue. */
-               struct mlx5_flow_drop drxq; /**< Flow with drop Rx queue. */
-       };
+       struct mlx5_flow frxq[RTE_DIM(hash_rxq_init)];
+       /**< Flow with Rx queue. */
 };
 
 /** Static initializer for items. */
@@ -432,6 +431,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. */
@@ -444,20 +444,12 @@ struct mlx5_flow_parse {
        uint8_t rss_key[40]; /**< copy of the RSS key. */
        enum hash_rxq_type layer; /**< Last pattern layer detected. */
        struct ibv_counter_set *cs; /**< Holds the counter set for the rule */
-       union {
-               struct {
-                       struct ibv_flow_attr *ibv_attr;
-                       /**< Pointer to Verbs attributes. */
-                       unsigned int offset;
-                       /**< Current position or total size of the attribute. */
-               } queue[RTE_DIM(hash_rxq_init)];
-               struct {
-                       struct ibv_flow_attr *ibv_attr;
-                       /**< Pointer to Verbs attributes. */
-                       unsigned int offset;
-                       /**< Current position or total size of the attribute. */
-               } drop_q;
-       };
+       struct {
+               struct ibv_flow_attr *ibv_attr;
+               /**< Pointer to Verbs attributes. */
+               unsigned int offset;
+               /**< Current position or total size of the attribute. */
+       } queue[RTE_DIM(hash_rxq_init)];
 };
 
 static const struct rte_flow_ops mlx5_flow_ops = {
@@ -578,9 +570,15 @@ priv_flow_convert_rss_conf(struct priv *priv,
                           struct mlx5_flow_parse *parser,
                           const struct rte_eth_rss_conf *rss_conf)
 {
-       const struct rte_eth_rss_conf *rss =
-               rss_conf ? rss_conf : &priv->rss_conf;
+       const struct rte_eth_rss_conf *rss;
 
+       if (rss_conf) {
+               if (rss_conf->rss_hf & MLX5_RSS_HF_MASK)
+                       return EINVAL;
+               rss = rss_conf;
+       } else {
+               rss = &priv->rss_conf;
+       }
        if (rss->rss_key_len > 40)
                return EINVAL;
        parser->rss_conf.rss_key_len = rss->rss_key_len;
@@ -781,7 +779,7 @@ priv_flow_convert_actions(struct priv *priv,
                } else if (actions->type == RTE_FLOW_ACTION_TYPE_FLAG) {
                        parser->mark = 1;
                } else if (actions->type == RTE_FLOW_ACTION_TYPE_COUNT &&
-                          priv->counter_set_supported) {
+                          priv->config.flow_counter_en) {
                        parser->count = 1;
                } else {
                        goto exit_action_not_supported;
@@ -827,12 +825,8 @@ priv_flow_convert_items_validate(struct priv *priv,
 
        (void)priv;
        /* Initialise the offsets to start after verbs attribute. */
-       if (parser->drop) {
-               parser->drop_q.offset = sizeof(struct ibv_flow_attr);
-       } else {
-               for (i = 0; i != hash_rxq_init_n; ++i)
-                       parser->queue[i].offset = sizeof(struct ibv_flow_attr);
-       }
+       for (i = 0; i != hash_rxq_init_n; ++i)
+               parser->queue[i].offset = sizeof(struct ibv_flow_attr);
        for (; items->type != RTE_FLOW_ITEM_TYPE_END; ++items) {
                const struct mlx5_flow_items *token = NULL;
                unsigned int n;
@@ -868,15 +862,17 @@ priv_flow_convert_items_validate(struct priv *priv,
                        }
                        parser->inner = IBV_FLOW_SPEC_INNER;
                }
-               if (parser->drop) {
-                       parser->drop_q.offset += cur_item->dst_sz;
-               } else if (parser->queues_n == 1) {
+               if (parser->drop || parser->queues_n == 1) {
                        parser->queue[HASH_RXQ_ETH].offset += cur_item->dst_sz;
                } else {
                        for (n = 0; n != hash_rxq_init_n; ++n)
                                parser->queue[n].offset += cur_item->dst_sz;
                }
        }
+       if (parser->drop) {
+               parser->queue[HASH_RXQ_ETH].offset +=
+                       sizeof(struct ibv_flow_spec_action_drop);
+       }
        if (parser->mark) {
                for (i = 0; i != hash_rxq_init_n; ++i)
                        parser->queue[i].offset +=
@@ -885,12 +881,8 @@ priv_flow_convert_items_validate(struct priv *priv,
        if (parser->count) {
                unsigned int size = sizeof(struct ibv_flow_spec_counter_action);
 
-               if (parser->drop) {
-                       parser->drop_q.offset += size;
-               } else {
-                       for (i = 0; i != hash_rxq_init_n; ++i)
-                               parser->queue[i].offset += size;
-               }
+               for (i = 0; i != hash_rxq_init_n; ++i)
+                       parser->queue[i].offset += size;
        }
        return 0;
 exit_item_not_supported:
@@ -1102,15 +1094,7 @@ priv_flow_convert(struct priv *priv,
         * Second step.
         * Allocate the memory space to store verbs specifications.
         */
-       if (parser->drop) {
-               parser->drop_q.ibv_attr =
-                       priv_flow_convert_allocate(priv, attr->priority,
-                                                  parser->drop_q.offset,
-                                                  error);
-               if (!parser->drop_q.ibv_attr)
-                       return ENOMEM;
-               parser->drop_q.offset = sizeof(struct ibv_flow_attr);
-       } else if (parser->queues_n == 1) {
+       if (parser->drop || parser->queues_n == 1) {
                unsigned int priority =
                        attr->priority +
                        hash_rxq_init[HASH_RXQ_ETH].flow_priority;
@@ -1172,15 +1156,7 @@ priv_flow_convert(struct priv *priv,
         * Last step. Complete missing specification to reach the RSS
         * configuration.
         */
-       if (parser->drop) {
-               /*
-                * Drop queue priority needs to be adjusted to
-                * their most specific layer priority.
-                */
-               parser->drop_q.ibv_attr->priority =
-                       attr->priority +
-                       hash_rxq_init[parser->layer].flow_priority;
-       } else if (parser->queues_n > 1) {
+       if (parser->queues_n > 1) {
                priv_flow_convert_finalise(priv, parser);
        } else {
                /*
@@ -1192,13 +1168,20 @@ priv_flow_convert(struct priv *priv,
                        attr->priority +
                        hash_rxq_init[parser->layer].flow_priority;
        }
+       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;
+               }
+       }
 exit_free:
        /* Only verification is expected, all resources should be released. */
        if (!parser->create) {
-               if (parser->drop) {
-                       rte_free(parser->drop_q.ibv_attr);
-                       parser->drop_q.ibv_attr = NULL;
-               }
                for (i = 0; i != hash_rxq_init_n; ++i) {
                        if (parser->queue[i].ibv_attr) {
                                rte_free(parser->queue[i].ibv_attr);
@@ -1240,14 +1223,6 @@ mlx5_flow_create_copy(struct mlx5_flow_parse *parser, void *src,
        unsigned int i;
        void *dst;
 
-       if (parser->drop) {
-               dst = (void *)((uintptr_t)parser->drop_q.ibv_attr +
-                               parser->drop_q.offset);
-               memcpy(dst, src, size);
-               ++parser->drop_q.ibv_attr->num_of_specs;
-               parser->drop_q.offset += size;
-               return;
-       }
        for (i = 0; i != hash_rxq_init_n; ++i) {
                if (!parser->queue[i].ibv_attr)
                        continue;
@@ -1311,6 +1286,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, eth_size);
+       parser->allmulti = eth.val.dst_mac[0] & 1;
        return 0;
 }
 
@@ -1340,14 +1316,6 @@ mlx5_flow_create_vlan(const struct rte_flow_item *item,
                if (!mask)
                        mask = default_mask;
 
-               if (parser->drop) {
-                       eth = (void *)((uintptr_t)parser->drop_q.ibv_attr +
-                                      parser->drop_q.offset - eth_size);
-                       eth->val.vlan_tag = spec->tci;
-                       eth->mask.vlan_tag = mask->tci;
-                       eth->val.vlan_tag &= eth->mask.vlan_tag;
-                       return 0;
-               }
                for (i = 0; i != hash_rxq_init_n; ++i) {
                        if (!parser->queue[i].ibv_attr)
                                continue;
@@ -1443,6 +1411,8 @@ mlx5_flow_create_ipv6(const struct rte_flow_item *item,
                parser->layer = HASH_RXQ_IPV6;
        if (spec) {
                unsigned int i;
+               uint32_t vtc_flow_val;
+               uint32_t vtc_flow_mask;
 
                if (!mask)
                        mask = default_mask;
@@ -1454,7 +1424,20 @@ mlx5_flow_create_ipv6(const struct rte_flow_item *item,
                       RTE_DIM(ipv6.mask.src_ip));
                memcpy(&ipv6.mask.dst_ip, mask->hdr.dst_addr,
                       RTE_DIM(ipv6.mask.dst_ip));
-               ipv6.mask.flow_label = mask->hdr.vtc_flow;
+               vtc_flow_val = rte_be_to_cpu_32(spec->hdr.vtc_flow);
+               vtc_flow_mask = rte_be_to_cpu_32(mask->hdr.vtc_flow);
+               ipv6.val.flow_label =
+                       rte_cpu_to_be_32((vtc_flow_val & IPV6_HDR_FL_MASK) >>
+                                        IPV6_HDR_FL_SHIFT);
+               ipv6.val.traffic_class = (vtc_flow_val & IPV6_HDR_TC_MASK) >>
+                                        IPV6_HDR_TC_SHIFT;
+               ipv6.val.next_hdr = spec->hdr.proto;
+               ipv6.val.hop_limit = spec->hdr.hop_limits;
+               ipv6.mask.flow_label =
+                       rte_cpu_to_be_32((vtc_flow_mask & IPV6_HDR_FL_MASK) >>
+                                        IPV6_HDR_FL_SHIFT);
+               ipv6.mask.traffic_class = (vtc_flow_mask & IPV6_HDR_TC_MASK) >>
+                                         IPV6_HDR_TC_SHIFT;
                ipv6.mask.next_hdr = mask->hdr.proto;
                ipv6.mask.hop_limit = mask->hdr.hop_limits;
                /* Remove unwanted bits from values. */
@@ -1463,6 +1446,7 @@ mlx5_flow_create_ipv6(const struct rte_flow_item *item,
                        ipv6.val.dst_ip[i] &= ipv6.mask.dst_ip[i];
                }
                ipv6.val.flow_label &= ipv6.mask.flow_label;
+               ipv6.val.traffic_class &= ipv6.mask.traffic_class;
                ipv6.val.next_hdr &= ipv6.mask.next_hdr;
                ipv6.val.hop_limit &= ipv6.mask.hop_limit;
        }
@@ -1701,23 +1685,25 @@ priv_flow_create_action_queue_drop(struct priv *priv,
        assert(priv->pd);
        assert(priv->ctx);
        flow->drop = 1;
-       drop = (void *)((uintptr_t)parser->drop_q.ibv_attr +
-                       parser->drop_q.offset);
+       drop = (void *)((uintptr_t)parser->queue[HASH_RXQ_ETH].ibv_attr +
+                       parser->queue[HASH_RXQ_ETH].offset);
        *drop = (struct ibv_flow_spec_action_drop){
                        .type = IBV_FLOW_SPEC_ACTION_DROP,
                        .size = size,
        };
-       ++parser->drop_q.ibv_attr->num_of_specs;
-       parser->drop_q.offset += size;
-       flow->drxq.ibv_attr = parser->drop_q.ibv_attr;
+       ++parser->queue[HASH_RXQ_ETH].ibv_attr->num_of_specs;
+       parser->queue[HASH_RXQ_ETH].offset += size;
+       flow->frxq[HASH_RXQ_ETH].ibv_attr =
+               parser->queue[HASH_RXQ_ETH].ibv_attr;
        if (parser->count)
                flow->cs = parser->cs;
        if (!priv->dev->data->dev_started)
                return 0;
-       parser->drop_q.ibv_attr = NULL;
-       flow->drxq.ibv_flow = ibv_create_flow(priv->flow_drop_queue->qp,
-                                             flow->drxq.ibv_attr);
-       if (!flow->drxq.ibv_flow) {
+       parser->queue[HASH_RXQ_ETH].ibv_attr = NULL;
+       flow->frxq[HASH_RXQ_ETH].ibv_flow =
+               ibv_create_flow(priv->flow_drop_queue->qp,
+                               flow->frxq[HASH_RXQ_ETH].ibv_attr);
+       if (!flow->frxq[HASH_RXQ_ETH].ibv_flow) {
                rte_flow_error_set(error, ENOMEM, RTE_FLOW_ERROR_TYPE_HANDLE,
                                   NULL, "flow rule creation failure");
                err = ENOMEM;
@@ -1726,13 +1712,13 @@ priv_flow_create_action_queue_drop(struct priv *priv,
        return 0;
 error:
        assert(flow);
-       if (flow->drxq.ibv_flow) {
-               claim_zero(ibv_destroy_flow(flow->drxq.ibv_flow));
-               flow->drxq.ibv_flow = NULL;
+       if (flow->frxq[HASH_RXQ_ETH].ibv_flow) {
+               claim_zero(ibv_destroy_flow(flow->frxq[HASH_RXQ_ETH].ibv_flow));
+               flow->frxq[HASH_RXQ_ETH].ibv_flow = NULL;
        }
-       if (flow->drxq.ibv_attr) {
-               rte_free(flow->drxq.ibv_attr);
-               flow->drxq.ibv_attr = NULL;
+       if (flow->frxq[HASH_RXQ_ETH].ibv_attr) {
+               rte_free(flow->frxq[HASH_RXQ_ETH].ibv_attr);
+               flow->frxq[HASH_RXQ_ETH].ibv_attr = NULL;
        }
        if (flow->cs) {
                claim_zero(ibv_destroy_counter_set(flow->cs));
@@ -1930,6 +1916,7 @@ priv_flow_create(struct priv *priv,
        flow->queues = (uint16_t (*)[])(flow + 1);
        memcpy(flow->queues, parser.queues, parser.queues_n * sizeof(uint16_t));
        flow->queues_n = parser.queues_n;
+       flow->mark = parser.mark;
        /* Copy RSS configuration. */
        flow->rss_conf = parser.rss_conf;
        flow->rss_conf.rss_key = flow->rss_key;
@@ -1946,13 +1933,9 @@ priv_flow_create(struct priv *priv,
        DEBUG("Flow created %p", (void *)flow);
        return flow;
 exit:
-       if (parser.drop) {
-               rte_free(parser.drop_q.ibv_attr);
-       } else {
-               for (i = 0; i != hash_rxq_init_n; ++i) {
-                       if (parser.queue[i].ibv_attr)
-                               rte_free(parser.queue[i].ibv_attr);
-               }
+       for (i = 0; i != hash_rxq_init_n; ++i) {
+               if (parser.queue[i].ibv_attr)
+                       rte_free(parser.queue[i].ibv_attr);
        }
        rte_free(flow);
        return NULL;
@@ -2054,9 +2037,10 @@ priv_flow_destroy(struct priv *priv,
        }
 free:
        if (flow->drop) {
-               if (flow->drxq.ibv_flow)
-                       claim_zero(ibv_destroy_flow(flow->drxq.ibv_flow));
-               rte_free(flow->drxq.ibv_attr);
+               if (flow->frxq[HASH_RXQ_ETH].ibv_flow)
+                       claim_zero(ibv_destroy_flow
+                                  (flow->frxq[HASH_RXQ_ETH].ibv_flow));
+               rte_free(flow->frxq[HASH_RXQ_ETH].ibv_attr);
        } else {
                for (i = 0; i != hash_rxq_init_n; ++i) {
                        struct mlx5_flow *frxq = &flow->frxq[i];
@@ -2225,10 +2209,11 @@ priv_flow_stop(struct priv *priv, struct mlx5_flows *list)
                unsigned int i;
 
                if (flow->drop) {
-                       if (!flow->drxq.ibv_flow)
+                       if (!flow->frxq[HASH_RXQ_ETH].ibv_flow)
                                continue;
-                       claim_zero(ibv_destroy_flow(flow->drxq.ibv_flow));
-                       flow->drxq.ibv_flow = NULL;
+                       claim_zero(ibv_destroy_flow
+                                  (flow->frxq[HASH_RXQ_ETH].ibv_flow));
+                       flow->frxq[HASH_RXQ_ETH].ibv_flow = NULL;
                        /* Next flow. */
                        continue;
                }
@@ -2276,10 +2261,11 @@ priv_flow_start(struct priv *priv, struct mlx5_flows *list)
                unsigned int i;
 
                if (flow->drop) {
-                       flow->drxq.ibv_flow =
-                               ibv_create_flow(priv->flow_drop_queue->qp,
-                                               flow->drxq.ibv_attr);
-                       if (!flow->drxq.ibv_flow) {
+                       flow->frxq[HASH_RXQ_ETH].ibv_flow =
+                               ibv_create_flow
+                               (priv->flow_drop_queue->qp,
+                                flow->frxq[HASH_RXQ_ETH].ibv_attr);
+                       if (!flow->frxq[HASH_RXQ_ETH].ibv_flow) {
                                DEBUG("Flow %p cannot be applied",
                                      (void *)flow);
                                rte_errno = EINVAL;
@@ -2874,13 +2860,13 @@ priv_fdir_filter_delete(struct priv *priv,
        if (parser.drop) {
                struct ibv_flow_spec_action_drop *drop;
 
-               drop = (void *)((uintptr_t)parser.drop_q.ibv_attr +
-                               parser.drop_q.offset);
+               drop = (void *)((uintptr_t)parser.queue[HASH_RXQ_ETH].ibv_attr +
+                               parser.queue[HASH_RXQ_ETH].offset);
                *drop = (struct ibv_flow_spec_action_drop){
                        .type = IBV_FLOW_SPEC_ACTION_DROP,
                        .size = sizeof(struct ibv_flow_spec_action_drop),
                };
-               parser.drop_q.ibv_attr->num_of_specs++;
+               parser.queue[HASH_RXQ_ETH].ibv_attr->num_of_specs++;
        }
        TAILQ_FOREACH(flow, &priv->flows, next) {
                struct ibv_flow_attr *attr;
@@ -2891,14 +2877,8 @@ priv_fdir_filter_delete(struct priv *priv,
                void *flow_spec;
                unsigned int specs_n;
 
-               if (parser.drop)
-                       attr = parser.drop_q.ibv_attr;
-               else
-                       attr = parser.queue[HASH_RXQ_ETH].ibv_attr;
-               if (flow->drop)
-                       flow_attr = flow->drxq.ibv_attr;
-               else
-                       flow_attr = flow->frxq[HASH_RXQ_ETH].ibv_attr;
+               attr = parser.queue[HASH_RXQ_ETH].ibv_attr;
+               flow_attr = flow->frxq[HASH_RXQ_ETH].ibv_attr;
                /* Compare first the attributes. */
                if (memcmp(attr, flow_attr, sizeof(struct ibv_flow_attr)))
                        continue;
@@ -2928,13 +2908,9 @@ wrong_flow:
        if (flow)
                priv_flow_destroy(priv, &priv->flows, flow);
 exit:
-       if (parser.drop) {
-               rte_free(parser.drop_q.ibv_attr);
-       } else {
-               for (i = 0; i != hash_rxq_init_n; ++i) {
-                       if (parser.queue[i].ibv_attr)
-                               rte_free(parser.queue[i].ibv_attr);
-               }
+       for (i = 0; i != hash_rxq_init_n; ++i) {
+               if (parser.queue[i].ibv_attr)
+                       rte_free(parser.queue[i].ibv_attr);
        }
        return -ret;
 }