net/mlx5: use flow to enable unicast traffic
[dpdk.git] / drivers / net / mlx5 / mlx5_rxmode.c
index 7794608..0ef2cdf 100644 (file)
 /* Verbs header. */
 /* ISO C doesn't support unnamed structs/unions, disabling -pedantic. */
 #ifdef PEDANTIC
-#pragma GCC diagnostic ignored "-pedantic"
+#pragma GCC diagnostic ignored "-Wpedantic"
 #endif
 #include <infiniband/verbs.h>
 #ifdef PEDANTIC
-#pragma GCC diagnostic error "-pedantic"
+#pragma GCC diagnostic error "-Wpedantic"
 #endif
 
-/* DPDK headers don't like -pedantic. */
-#ifdef PEDANTIC
-#pragma GCC diagnostic ignored "-pedantic"
-#endif
 #include <rte_ethdev.h>
-#ifdef PEDANTIC
-#pragma GCC diagnostic error "-pedantic"
-#endif
 
 #include "mlx5.h"
 #include "mlx5_rxtx.h"
 #include "mlx5_utils.h"
 
-static void hash_rxq_promiscuous_disable(struct hash_rxq *);
-static void hash_rxq_allmulticast_disable(struct hash_rxq *);
-
-/**
- * Enable promiscuous mode in a hash RX queue.
- *
- * @param hash_rxq
- *   Pointer to hash RX queue structure.
- *
- * @return
- *   0 on success, errno value on failure.
- */
-static int
-hash_rxq_promiscuous_enable(struct hash_rxq *hash_rxq)
-{
-       struct ibv_flow *flow;
-       FLOW_ATTR_SPEC_ETH(data, hash_rxq_flow_attr(hash_rxq, NULL, 0));
-       struct ibv_flow_attr *attr = &data->attr;
-
-       if (hash_rxq->promisc_flow != NULL)
-               return 0;
-       DEBUG("%p: enabling promiscuous mode", (void *)hash_rxq);
-       /* Promiscuous flows only differ from normal flows by not filtering
-        * on specific MAC addresses. */
-       hash_rxq_flow_attr(hash_rxq, attr, sizeof(data));
-       errno = 0;
-       flow = ibv_create_flow(hash_rxq->qp, attr);
-       if (flow == NULL) {
-               /* It's not clear whether errno is always set in this case. */
-               ERROR("%p: flow configuration failed, errno=%d: %s",
-                     (void *)hash_rxq, errno,
-                     (errno ? strerror(errno) : "Unknown error"));
-               if (errno)
-                       return errno;
-               return EINVAL;
-       }
-       hash_rxq->promisc_flow = flow;
-       DEBUG("%p: promiscuous mode enabled", (void *)hash_rxq);
-       return 0;
-}
-
-/**
- * Enable promiscuous mode in all hash RX queues.
- *
- * @param priv
- *   Private structure.
- *
- * @return
- *   0 on success, errno value on failure.
- */
-int
-priv_promiscuous_enable(struct priv *priv)
-{
-       unsigned int i;
-
-       for (i = 0; (i != priv->hash_rxqs_n); ++i) {
-               struct hash_rxq *hash_rxq = &(*priv->hash_rxqs)[i];
-               int ret;
-
-               ret = hash_rxq_promiscuous_enable(hash_rxq);
-               if (!ret)
-                       continue;
-               /* Failure, rollback. */
-               while (i != 0) {
-                       hash_rxq = &(*priv->hash_rxqs)[--i];
-                       hash_rxq_promiscuous_disable(hash_rxq);
-               }
-               return ret;
-       }
-       return 0;
-}
-
 /**
  * DPDK callback to enable promiscuous mode.
  *
@@ -139,47 +60,10 @@ priv_promiscuous_enable(struct priv *priv)
 void
 mlx5_promiscuous_enable(struct rte_eth_dev *dev)
 {
-       struct priv *priv = dev->data->dev_private;
-       int ret;
-
-       priv_lock(priv);
-       priv->promisc_req = 1;
-       ret = priv_promiscuous_enable(priv);
-       if (ret)
-               ERROR("cannot enable promiscuous mode: %s", strerror(ret));
-       priv_unlock(priv);
-}
-
-/**
- * Disable promiscuous mode in a hash RX queue.
- *
- * @param hash_rxq
- *   Pointer to hash RX queue structure.
- */
-static void
-hash_rxq_promiscuous_disable(struct hash_rxq *hash_rxq)
-{
-       if (hash_rxq->promisc_flow == NULL)
+       if (mlx5_is_secondary())
                return;
-       DEBUG("%p: disabling promiscuous mode", (void *)hash_rxq);
-       claim_zero(ibv_destroy_flow(hash_rxq->promisc_flow));
-       hash_rxq->promisc_flow = NULL;
-       DEBUG("%p: promiscuous mode disabled", (void *)hash_rxq);
-}
-
-/**
- * Disable promiscuous mode in all hash RX queues.
- *
- * @param priv
- *   Private structure.
- */
-void
-priv_promiscuous_disable(struct priv *priv)
-{
-       unsigned int i;
-
-       for (i = 0; (i != priv->hash_rxqs_n); ++i)
-               hash_rxq_promiscuous_disable(&(*priv->hash_rxqs)[i]);
+       dev->data->promiscuous = 1;
+       mlx5_traffic_restart(dev);
 }
 
 /**
@@ -191,99 +75,10 @@ priv_promiscuous_disable(struct priv *priv)
 void
 mlx5_promiscuous_disable(struct rte_eth_dev *dev)
 {
-       struct priv *priv = dev->data->dev_private;
-
-       priv_lock(priv);
-       priv->promisc_req = 0;
-       priv_promiscuous_disable(priv);
-       priv_unlock(priv);
-}
-
-/**
- * Enable allmulti mode in a hash RX queue.
- *
- * @param hash_rxq
- *   Pointer to hash RX queue structure.
- *
- * @return
- *   0 on success, errno value on failure.
- */
-static int
-hash_rxq_allmulticast_enable(struct hash_rxq *hash_rxq)
-{
-       struct ibv_flow *flow;
-       FLOW_ATTR_SPEC_ETH(data, hash_rxq_flow_attr(hash_rxq, NULL, 0));
-       struct ibv_flow_attr *attr = &data->attr;
-       struct ibv_flow_spec_eth *spec = &data->spec;
-
-       if (hash_rxq->allmulti_flow != NULL)
-               return 0;
-       DEBUG("%p: enabling allmulticast mode", (void *)hash_rxq);
-       /*
-        * No padding must be inserted by the compiler between attr and spec.
-        * This layout is expected by libibverbs.
-        */
-       assert(((uint8_t *)attr + sizeof(*attr)) == (uint8_t *)spec);
-       hash_rxq_flow_attr(hash_rxq, attr, sizeof(data));
-       *spec = (struct ibv_flow_spec_eth){
-               .type = IBV_FLOW_SPEC_ETH,
-               .size = sizeof(*spec),
-               .val = {
-                       .dst_mac = "\x01\x00\x00\x00\x00\x00",
-               },
-               .mask = {
-                       .dst_mac = "\x01\x00\x00\x00\x00\x00",
-               },
-       };
-       errno = 0;
-       flow = ibv_create_flow(hash_rxq->qp, attr);
-       if (flow == NULL) {
-               /* It's not clear whether errno is always set in this case. */
-               ERROR("%p: flow configuration failed, errno=%d: %s",
-                     (void *)hash_rxq, errno,
-                     (errno ? strerror(errno) : "Unknown error"));
-               if (errno)
-                       return errno;
-               return EINVAL;
-       }
-       hash_rxq->allmulti_flow = flow;
-       DEBUG("%p: allmulticast mode enabled", (void *)hash_rxq);
-       return 0;
-}
-
-/**
- * Enable allmulti mode in most hash RX queues.
- * TCP queues are exempted to save resources.
- *
- * @param priv
- *   Private structure.
- *
- * @return
- *   0 on success, errno value on failure.
- */
-int
-priv_allmulticast_enable(struct priv *priv)
-{
-       unsigned int i;
-
-       for (i = 0; (i != priv->hash_rxqs_n); ++i) {
-               struct hash_rxq *hash_rxq = &(*priv->hash_rxqs)[i];
-               int ret;
-
-               /* allmulticast not relevant for TCP. */
-               if (hash_rxq->type == HASH_RXQ_TCPV4)
-                       continue;
-               ret = hash_rxq_allmulticast_enable(hash_rxq);
-               if (!ret)
-                       continue;
-               /* Failure, rollback. */
-               while (i != 0) {
-                       hash_rxq = &(*priv->hash_rxqs)[--i];
-                       hash_rxq_allmulticast_disable(hash_rxq);
-               }
-               return ret;
-       }
-       return 0;
+       if (mlx5_is_secondary())
+               return;
+       dev->data->promiscuous = 0;
+       mlx5_traffic_restart(dev);
 }
 
 /**
@@ -295,47 +90,10 @@ priv_allmulticast_enable(struct priv *priv)
 void
 mlx5_allmulticast_enable(struct rte_eth_dev *dev)
 {
-       struct priv *priv = dev->data->dev_private;
-       int ret;
-
-       priv_lock(priv);
-       priv->allmulti_req = 1;
-       ret = priv_allmulticast_enable(priv);
-       if (ret)
-               ERROR("cannot enable allmulticast mode: %s", strerror(ret));
-       priv_unlock(priv);
-}
-
-/**
- * Disable allmulti mode in a hash RX queue.
- *
- * @param hash_rxq
- *   Pointer to hash RX queue structure.
- */
-static void
-hash_rxq_allmulticast_disable(struct hash_rxq *hash_rxq)
-{
-       if (hash_rxq->allmulti_flow == NULL)
+       if (mlx5_is_secondary())
                return;
-       DEBUG("%p: disabling allmulticast mode", (void *)hash_rxq);
-       claim_zero(ibv_destroy_flow(hash_rxq->allmulti_flow));
-       hash_rxq->allmulti_flow = NULL;
-       DEBUG("%p: allmulticast mode disabled", (void *)hash_rxq);
-}
-
-/**
- * Disable allmulti mode in all hash RX queues.
- *
- * @param priv
- *   Private structure.
- */
-void
-priv_allmulticast_disable(struct priv *priv)
-{
-       unsigned int i;
-
-       for (i = 0; (i != priv->hash_rxqs_n); ++i)
-               hash_rxq_allmulticast_disable(&(*priv->hash_rxqs)[i]);
+       dev->data->all_multicast = 1;
+       mlx5_traffic_restart(dev);
 }
 
 /**
@@ -347,10 +105,8 @@ priv_allmulticast_disable(struct priv *priv)
 void
 mlx5_allmulticast_disable(struct rte_eth_dev *dev)
 {
-       struct priv *priv = dev->data->dev_private;
-
-       priv_lock(priv);
-       priv->allmulti_req = 0;
-       priv_allmulticast_disable(priv);
-       priv_unlock(priv);
+       if (mlx5_is_secondary())
+               return;
+       dev->data->all_multicast = 0;
+       mlx5_traffic_restart(dev);
 }