net/mlx5: use flow to enable promiscuous mode
[dpdk.git] / drivers / net / mlx5 / mlx5_trigger.c
index dced025..085abcc 100644 (file)
  *   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  *   OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
+#include <unistd.h>
 
-/* DPDK headers don't like -pedantic. */
-#ifdef PEDANTIC
-#pragma GCC diagnostic ignored "-pedantic"
-#endif
 #include <rte_ether.h>
 #include <rte_ethdev.h>
-#ifdef PEDANTIC
-#pragma GCC diagnostic error "-pedantic"
-#endif
+#include <rte_interrupts.h>
+#include <rte_alarm.h>
 
 #include "mlx5.h"
 #include "mlx5_rxtx.h"
 #include "mlx5_utils.h"
 
+static void
+priv_txq_stop(struct priv *priv)
+{
+       unsigned int i;
+
+       for (i = 0; i != priv->txqs_n; ++i)
+               mlx5_priv_txq_release(priv, i);
+}
+
+static int
+priv_txq_start(struct priv *priv)
+{
+       unsigned int i;
+       int ret = 0;
+
+       /* Add memory regions to Tx queues. */
+       for (i = 0; i != priv->txqs_n; ++i) {
+               unsigned int idx = 0;
+               struct mlx5_mr *mr;
+               struct mlx5_txq_ctrl *txq_ctrl = mlx5_priv_txq_get(priv, i);
+
+               if (!txq_ctrl)
+                       continue;
+               LIST_FOREACH(mr, &priv->mr, next)
+                       priv_txq_mp2mr_reg(priv, &txq_ctrl->txq, mr->mp, idx++);
+               txq_alloc_elts(txq_ctrl);
+               txq_ctrl->ibv = mlx5_priv_txq_ibv_new(priv, i);
+               if (!txq_ctrl->ibv) {
+                       ret = ENOMEM;
+                       goto error;
+               }
+       }
+       return -ret;
+error:
+       priv_txq_stop(priv);
+       return -ret;
+}
+
+static void
+priv_rxq_stop(struct priv *priv)
+{
+       unsigned int i;
+
+       for (i = 0; i != priv->rxqs_n; ++i)
+               mlx5_priv_rxq_release(priv, i);
+}
+
+static int
+priv_rxq_start(struct priv *priv)
+{
+       unsigned int i;
+       int ret = 0;
+
+       for (i = 0; i != priv->rxqs_n; ++i) {
+               struct mlx5_rxq_ctrl *rxq_ctrl = mlx5_priv_rxq_get(priv, i);
+
+               if (!rxq_ctrl)
+                       continue;
+               ret = rxq_alloc_elts(rxq_ctrl);
+               if (ret)
+                       goto error;
+               rxq_ctrl->ibv = mlx5_priv_rxq_ibv_new(priv, i);
+               if (!rxq_ctrl->ibv) {
+                       ret = ENOMEM;
+                       goto error;
+               }
+       }
+       return -ret;
+error:
+       priv_rxq_stop(priv);
+       return -ret;
+}
+
 /**
  * DPDK callback to start the device.
  *
@@ -60,55 +129,79 @@ int
 mlx5_dev_start(struct rte_eth_dev *dev)
 {
        struct priv *priv = dev->data->dev_private;
-       unsigned int i = 0;
-       unsigned int r;
-       struct rxq *rxq;
+       struct mlx5_mr *mr = NULL;
+       int err;
+
+       if (mlx5_is_secondary())
+               return -E_RTE_SECONDARY;
 
        priv_lock(priv);
-       if (priv->started) {
-               priv_unlock(priv);
-               return 0;
+       DEBUG("%p: allocating and configuring hash RX queues", (void *)dev);
+       rte_mempool_walk(mlx5_mp2mr_iter, priv);
+       err = priv_txq_start(priv);
+       if (err) {
+               ERROR("%p: TXQ allocation failed: %s",
+                     (void *)dev, strerror(err));
+               goto error;
        }
-       DEBUG("%p: attaching configured flows to all RX queues", (void *)dev);
-       priv->started = 1;
-       if (priv->rss) {
-               rxq = &priv->rxq_parent;
-               r = 1;
-       } else {
-               rxq = (*priv->rxqs)[0];
-               r = priv->rxqs_n;
+       /* Update send callback. */
+       priv_dev_select_tx_function(priv, dev);
+       err = priv_rxq_start(priv);
+       if (err) {
+               ERROR("%p: RXQ allocation failed: %s",
+                     (void *)dev, strerror(err));
+               goto error;
        }
-       /* Iterate only once when RSS is enabled. */
-       do {
-               int ret;
-
-               /* Ignore nonexistent RX queues. */
-               if (rxq == NULL)
-                       continue;
-               ret = rxq_mac_addrs_add(rxq);
-               if (!ret && priv->promisc_req)
-                       ret = rxq_promiscuous_enable(rxq);
-               if (!ret && priv->allmulti_req)
-                       ret = rxq_allmulticast_enable(rxq);
-               if (!ret)
-                       continue;
-               WARN("%p: QP flow attachment failed: %s",
-                    (void *)dev, strerror(ret));
-               /* Rollback. */
-               while (i != 0) {
-                       rxq = (*priv->rxqs)[--i];
-                       if (rxq != NULL) {
-                               rxq_allmulticast_disable(rxq);
-                               rxq_promiscuous_disable(rxq);
-                               rxq_mac_addrs_del(rxq);
-                       }
-               }
-               priv->started = 0;
-               priv_unlock(priv);
-               return -ret;
-       } while ((--r) && ((rxq = (*priv->rxqs)[++i]), i));
+       /* Update receive callback. */
+       priv_dev_select_rx_function(priv, dev);
+       err = priv_create_hash_rxqs(priv);
+       if (!err)
+               err = priv_rehash_flows(priv);
+       else {
+               ERROR("%p: an error occurred while configuring hash RX queues:"
+                     " %s",
+                     (void *)priv, strerror(err));
+               goto error;
+       }
+       if (dev->data->promiscuous)
+               mlx5_promiscuous_enable(dev);
+       err = priv_flow_start(priv, &priv->ctrl_flows);
+       if (err) {
+               ERROR("%p: an error occurred while configuring control flows:"
+                     " %s",
+                     (void *)priv, strerror(err));
+               goto error;
+       }
+       err = priv_flow_start(priv, &priv->flows);
+       if (err) {
+               ERROR("%p: an error occurred while configuring flows:"
+                     " %s",
+                     (void *)priv, strerror(err));
+               goto error;
+       }
+       err = priv_rx_intr_vec_enable(priv);
+       if (err) {
+               ERROR("%p: RX interrupt vector creation failed",
+                     (void *)priv);
+               goto error;
+       }
+       priv_dev_interrupt_handler_install(priv, dev);
+       priv_xstats_init(priv);
        priv_unlock(priv);
        return 0;
+error:
+       /* Rollback. */
+       LIST_FOREACH(mr, &priv->mr, next)
+               priv_mr_release(priv, mr);
+       priv_special_flow_disable_all(priv);
+       priv_mac_addrs_disable(priv);
+       priv_destroy_hash_rxqs(priv);
+       priv_flow_stop(priv, &priv->flows);
+       priv_flow_flush(priv, &priv->ctrl_flows);
+       priv_rxq_stop(priv);
+       priv_txq_stop(priv);
+       priv_unlock(priv);
+       return -err;
 }
 
 /**
@@ -123,32 +216,30 @@ void
 mlx5_dev_stop(struct rte_eth_dev *dev)
 {
        struct priv *priv = dev->data->dev_private;
-       unsigned int i = 0;
-       unsigned int r;
-       struct rxq *rxq;
+       struct mlx5_mr *mr;
 
-       priv_lock(priv);
-       if (!priv->started) {
-               priv_unlock(priv);
+       if (mlx5_is_secondary())
                return;
+
+       priv_lock(priv);
+       dev->data->dev_started = 0;
+       /* Prevent crashes when queues are still in use. */
+       dev->rx_pkt_burst = removed_rx_burst;
+       dev->tx_pkt_burst = removed_tx_burst;
+       rte_wmb();
+       usleep(1000 * priv->rxqs_n);
+       DEBUG("%p: cleaning up and destroying hash RX queues", (void *)dev);
+       priv_special_flow_disable_all(priv);
+       priv_mac_addrs_disable(priv);
+       priv_destroy_hash_rxqs(priv);
+       priv_flow_stop(priv, &priv->flows);
+       priv_flow_flush(priv, &priv->ctrl_flows);
+       priv_rx_intr_vec_disable(priv);
+       priv_dev_interrupt_handler_uninstall(priv, dev);
+       priv_txq_stop(priv);
+       priv_rxq_stop(priv);
+       LIST_FOREACH(mr, &priv->mr, next) {
+               priv_mr_release(priv, mr);
        }
-       DEBUG("%p: detaching flows from all RX queues", (void *)dev);
-       priv->started = 0;
-       if (priv->rss) {
-               rxq = &priv->rxq_parent;
-               r = 1;
-       } else {
-               rxq = (*priv->rxqs)[0];
-               r = priv->rxqs_n;
-       }
-       /* Iterate only once when RSS is enabled. */
-       do {
-               /* Ignore nonexistent RX queues. */
-               if (rxq == NULL)
-                       continue;
-               rxq_allmulticast_disable(rxq);
-               rxq_promiscuous_disable(rxq);
-               rxq_mac_addrs_del(rxq);
-       } while ((--r) && ((rxq = (*priv->rxqs)[++i]), i));
        priv_unlock(priv);
 }