net/mlx5: fix allocation when no memory on device NUMA node
[dpdk.git] / drivers / net / mlx5 / mlx5_txq.c
index 84d37be..18321c3 100644 (file)
@@ -51,7 +51,7 @@
 
 #include <rte_mbuf.h>
 #include <rte_malloc.h>
-#include <rte_ethdev.h>
+#include <rte_ethdev_driver.h>
 #include <rte_common.h>
 
 #include "mlx5_utils.h"
@@ -115,6 +115,63 @@ txq_free_elts(struct mlx5_txq_ctrl *txq_ctrl)
        }
 }
 
+/**
+ * Returns the per-port supported offloads.
+ *
+ * @param priv
+ *   Pointer to private structure.
+ *
+ * @return
+ *   Supported Tx offloads.
+ */
+uint64_t
+mlx5_priv_get_tx_port_offloads(struct priv *priv)
+{
+       uint64_t offloads = (DEV_TX_OFFLOAD_MULTI_SEGS |
+                            DEV_TX_OFFLOAD_VLAN_INSERT);
+       struct mlx5_dev_config *config = &priv->config;
+
+       if (config->hw_csum)
+               offloads |= (DEV_TX_OFFLOAD_IPV4_CKSUM |
+                            DEV_TX_OFFLOAD_UDP_CKSUM |
+                            DEV_TX_OFFLOAD_TCP_CKSUM);
+       if (config->tso)
+               offloads |= DEV_TX_OFFLOAD_TCP_TSO;
+       if (config->tunnel_en) {
+               if (config->hw_csum)
+                       offloads |= DEV_TX_OFFLOAD_OUTER_IPV4_CKSUM;
+               if (config->tso)
+                       offloads |= (DEV_TX_OFFLOAD_VXLAN_TNL_TSO |
+                                    DEV_TX_OFFLOAD_GRE_TNL_TSO);
+       }
+       return offloads;
+}
+
+/**
+ * Checks if the per-queue offload configuration is valid.
+ *
+ * @param priv
+ *   Pointer to private structure.
+ * @param offloads
+ *   Per-queue offloads configuration.
+ *
+ * @return
+ *   1 if the configuration is valid, 0 otherwise.
+ */
+static int
+priv_is_tx_queue_offloads_allowed(struct priv *priv, uint64_t offloads)
+{
+       uint64_t port_offloads = priv->dev->data->dev_conf.txmode.offloads;
+       uint64_t port_supp_offloads = mlx5_priv_get_tx_port_offloads(priv);
+
+       /* There are no Tx offloads which are per queue. */
+       if ((offloads & port_supp_offloads) != offloads)
+               return 0;
+       if ((port_offloads ^ offloads) & port_supp_offloads)
+               return 0;
+       return 1;
+}
+
 /**
  * DPDK callback to configure a TX queue.
  *
@@ -143,6 +200,20 @@ mlx5_tx_queue_setup(struct rte_eth_dev *dev, uint16_t idx, uint16_t desc,
        int ret = 0;
 
        priv_lock(priv);
+       /*
+        * Don't verify port offloads for application which
+        * use the old API.
+        */
+       if (!!(conf->txq_flags & ETH_TXQ_FLAGS_IGNORE) &&
+           !priv_is_tx_queue_offloads_allowed(priv, conf->offloads)) {
+               ret = ENOTSUP;
+               ERROR("%p: Tx queue offloads 0x%" PRIx64 " don't match port "
+                     "offloads 0x%" PRIx64 " or supported offloads 0x%" PRIx64,
+                     (void *)dev, conf->offloads,
+                     dev->data->dev_conf.txmode.offloads,
+                     mlx5_priv_get_tx_port_offloads(priv));
+               goto out;
+       }
        if (desc <= MLX5_TX_COMP_THRESH) {
                WARN("%p: number of descriptors requested for TX queue %u"
                     " must be higher than MLX5_TX_COMP_THRESH, using"
@@ -272,6 +343,25 @@ priv_tx_uar_remap(struct priv *priv, int fd)
        return 0;
 }
 
+/**
+ * Check if the burst function is using eMPW.
+ *
+ * @param tx_pkt_burst
+ *   Tx burst function pointer.
+ *
+ * @return
+ *   1 if the burst function is using eMPW, 0 otherwise.
+ */
+static int
+is_empw_burst_func(eth_tx_burst_t tx_pkt_burst)
+{
+       if (tx_pkt_burst == mlx5_tx_burst_raw_vec ||
+           tx_pkt_burst == mlx5_tx_burst_vec ||
+           tx_pkt_burst == mlx5_tx_burst_empw)
+               return 1;
+       return 0;
+}
+
 /**
  * Create the Tx queue Verbs object.
  *
@@ -302,9 +392,12 @@ mlx5_priv_txq_ibv_new(struct priv *priv, uint16_t idx)
        struct mlx5dv_cq cq_info;
        struct mlx5dv_obj obj;
        const int desc = 1 << txq_data->elts_n;
+       eth_tx_burst_t tx_pkt_burst = priv_select_tx_function(priv, priv->dev);
        int ret = 0;
 
        assert(txq_data);
+       priv->verbs_alloc_ctx.type = MLX5_VERBS_ALLOC_TYPE_TX_QUEUE;
+       priv->verbs_alloc_ctx.obj = txq_ctrl;
        if (mlx5_getenv_int("MLX5_ENABLE_CQE_COMPRESSION")) {
                ERROR("MLX5_ENABLE_CQE_COMPRESSION must never be set");
                goto error;
@@ -316,7 +409,7 @@ mlx5_priv_txq_ibv_new(struct priv *priv, uint16_t idx)
        };
        cqe_n = ((desc / MLX5_TX_COMP_THRESH) - 1) ?
                ((desc / MLX5_TX_COMP_THRESH) - 1) : 1;
-       if (priv->mps == MLX5_MPW_ENHANCED)
+       if (is_empw_burst_func(tx_pkt_burst))
                cqe_n += MLX5_TX_COMP_THRESH_INLINE_DIV;
        tmpl.cq = ibv_create_cq(priv->ctx, cqe_n, NULL, NULL, 0);
        if (tmpl.cq == NULL) {
@@ -353,7 +446,7 @@ mlx5_priv_txq_ibv_new(struct priv *priv, uint16_t idx)
                .pd = priv->pd,
                .comp_mask = IBV_QP_INIT_ATTR_PD,
        };
-       if (txq_data->inline_en)
+       if (txq_data->max_inline)
                attr.init.cap.max_inline_data = txq_ctrl->max_inline_data;
        if (txq_data->tso_en) {
                attr.init.max_tso_header = txq_ctrl->max_tso_header;
@@ -418,7 +511,9 @@ mlx5_priv_txq_ibv_new(struct priv *priv, uint16_t idx)
                (volatile struct mlx5_cqe (*)[])
                (uintptr_t)cq_info.buf;
        txq_data->cq_ci = 0;
+#ifndef NDEBUG
        txq_data->cq_pi = 0;
+#endif
        txq_data->wqe_ci = 0;
        txq_data->wqe_pi = 0;
        txq_ibv->qp = tmpl.qp;
@@ -433,12 +528,14 @@ mlx5_priv_txq_ibv_new(struct priv *priv, uint16_t idx)
        DEBUG("%p: Verbs Tx queue %p: refcnt %d", (void *)priv,
              (void *)txq_ibv, rte_atomic32_read(&txq_ibv->refcnt));
        LIST_INSERT_HEAD(&priv->txqsibv, txq_ibv, next);
+       priv->verbs_alloc_ctx.type = MLX5_VERBS_ALLOC_TYPE_NONE;
        return txq_ibv;
 error:
        if (tmpl.cq)
                claim_zero(ibv_destroy_cq(tmpl.cq));
        if (tmpl.qp)
                claim_zero(ibv_destroy_qp(tmpl.qp));
+       priv->verbs_alloc_ctx.type = MLX5_VERBS_ALLOC_TYPE_NONE;
        return NULL;
 }
 
@@ -539,84 +636,73 @@ mlx5_priv_txq_ibv_verify(struct priv *priv)
 }
 
 /**
- * Create a DPDK Tx queue.
+ * Set Tx queue parameters from device configuration.
  *
- * @param priv
- *   Pointer to private structure.
- * @param idx
- *   TX queue index.
- * @param desc
- *   Number of descriptors to configure in queue.
- * @param socket
- *   NUMA socket on which memory must be allocated.
- * @param[in] conf
- *  Thresholds parameters.
- *
- * @return
- *   A DPDK queue object on success.
+ * @param txq_ctrl
+ *   Pointer to Tx queue control structure.
  */
-struct mlx5_txq_ctrl*
-mlx5_priv_txq_new(struct priv *priv, uint16_t idx, uint16_t desc,
-                 unsigned int socket,
-                 const struct rte_eth_txconf *conf)
+static void
+txq_set_params(struct mlx5_txq_ctrl *txq_ctrl)
 {
+       struct priv *priv = txq_ctrl->priv;
+       struct mlx5_dev_config *config = &priv->config;
        const unsigned int max_tso_inline =
                ((MLX5_MAX_TSO_HEADER + (RTE_CACHE_LINE_SIZE - 1)) /
                 RTE_CACHE_LINE_SIZE);
-       struct mlx5_txq_ctrl *tmpl;
-
-       tmpl = rte_calloc_socket("TXQ", 1,
-                                sizeof(*tmpl) +
-                                desc * sizeof(struct rte_mbuf *),
-                                0, socket);
-       if (!tmpl)
-               return NULL;
-       assert(desc > MLX5_TX_COMP_THRESH);
-       tmpl->txq.flags = conf->txq_flags;
-       tmpl->priv = priv;
-       tmpl->socket = socket;
-       tmpl->txq.elts_n = log2above(desc);
-       if (priv->mps == MLX5_MPW_ENHANCED)
-               tmpl->txq.mpw_hdr_dseg = priv->mpw_hdr_dseg;
-       /* MRs will be registered in mp2mr[] later. */
-       DEBUG("priv->device_attr.max_qp_wr is %d",
-             priv->device_attr.orig_attr.max_qp_wr);
-       DEBUG("priv->device_attr.max_sge is %d",
-             priv->device_attr.orig_attr.max_sge);
-       if (priv->txq_inline && (priv->txqs_n >= priv->txqs_inline)) {
+       unsigned int txq_inline;
+       unsigned int txqs_inline;
+       unsigned int inline_max_packet_sz;
+       eth_tx_burst_t tx_pkt_burst = priv_select_tx_function(priv, priv->dev);
+       int is_empw_func = is_empw_burst_func(tx_pkt_burst);
+       int tso = !!(txq_ctrl->txq.offloads & DEV_TX_OFFLOAD_TCP_TSO);
+
+       txq_inline = (config->txq_inline == MLX5_ARG_UNSET) ?
+               0 : config->txq_inline;
+       txqs_inline = (config->txqs_inline == MLX5_ARG_UNSET) ?
+               0 : config->txqs_inline;
+       inline_max_packet_sz =
+               (config->inline_max_packet_sz == MLX5_ARG_UNSET) ?
+               0 : config->inline_max_packet_sz;
+       if (is_empw_func) {
+               if (config->txq_inline == MLX5_ARG_UNSET)
+                       txq_inline = MLX5_WQE_SIZE_MAX - MLX5_WQE_SIZE;
+               if (config->txqs_inline == MLX5_ARG_UNSET)
+                       txqs_inline = MLX5_EMPW_MIN_TXQS;
+               if (config->inline_max_packet_sz == MLX5_ARG_UNSET)
+                       inline_max_packet_sz = MLX5_EMPW_MAX_INLINE_LEN;
+               txq_ctrl->txq.mpw_hdr_dseg = config->mpw_hdr_dseg;
+               txq_ctrl->txq.inline_max_packet_sz = inline_max_packet_sz;
+       }
+       if (txq_inline && priv->txqs_n >= txqs_inline) {
                unsigned int ds_cnt;
 
-               tmpl->txq.max_inline =
-                       ((priv->txq_inline + (RTE_CACHE_LINE_SIZE - 1)) /
+               txq_ctrl->txq.max_inline =
+                       ((txq_inline + (RTE_CACHE_LINE_SIZE - 1)) /
                         RTE_CACHE_LINE_SIZE);
-               tmpl->txq.inline_en = 1;
-               /* TSO and MPS can't be enabled concurrently. */
-               assert(!priv->tso || !priv->mps);
-               if (priv->mps == MLX5_MPW_ENHANCED) {
-                       tmpl->txq.inline_max_packet_sz =
-                               priv->inline_max_packet_sz;
+               if (is_empw_func) {
                        /* To minimize the size of data set, avoid requesting
                         * too large WQ.
                         */
-                       tmpl->max_inline_data =
-                               ((RTE_MIN(priv->txq_inline,
-                                         priv->inline_max_packet_sz) +
+                       txq_ctrl->max_inline_data =
+                               ((RTE_MIN(txq_inline,
+                                         inline_max_packet_sz) +
                                  (RTE_CACHE_LINE_SIZE - 1)) /
                                 RTE_CACHE_LINE_SIZE) * RTE_CACHE_LINE_SIZE;
-               } else if (priv->tso) {
-                       int inline_diff = tmpl->txq.max_inline - max_tso_inline;
+               } else if (tso) {
+                       int inline_diff = txq_ctrl->txq.max_inline -
+                                         max_tso_inline;
 
                        /*
                         * Adjust inline value as Verbs aggregates
                         * tso_inline and txq_inline fields.
                         */
-                       tmpl->max_inline_data = inline_diff > 0 ?
+                       txq_ctrl->max_inline_data = inline_diff > 0 ?
                                               inline_diff *
                                               RTE_CACHE_LINE_SIZE :
                                               0;
                } else {
-                       tmpl->max_inline_data =
-                               tmpl->txq.max_inline * RTE_CACHE_LINE_SIZE;
+                       txq_ctrl->max_inline_data =
+                               txq_ctrl->txq.max_inline * RTE_CACHE_LINE_SIZE;
                }
                /*
                 * Check if the inline size is too large in a way which
@@ -626,7 +712,7 @@ mlx5_priv_txq_new(struct priv *priv, uint16_t idx, uint16_t desc,
                 *      WQE ETH  (1 DS)
                 *      Inline part (N DS)
                 */
-               ds_cnt = 2 + (tmpl->txq.max_inline / MLX5_WQE_DWORD_SIZE);
+               ds_cnt = 2 + (txq_ctrl->txq.max_inline / MLX5_WQE_DWORD_SIZE);
                if (ds_cnt > MLX5_DSEG_MAX) {
                        unsigned int max_inline = (MLX5_DSEG_MAX - 2) *
                                                  MLX5_WQE_DWORD_SIZE;
@@ -635,18 +721,61 @@ mlx5_priv_txq_new(struct priv *priv, uint16_t idx, uint16_t desc,
                                                   RTE_CACHE_LINE_SIZE);
                        WARN("txq inline is too large (%d) setting it to "
                             "the maximum possible: %d\n",
-                            priv->txq_inline, max_inline);
-                       tmpl->txq.max_inline = max_inline / RTE_CACHE_LINE_SIZE;
+                            txq_inline, max_inline);
+                       txq_ctrl->txq.max_inline = max_inline /
+                                                  RTE_CACHE_LINE_SIZE;
                }
        }
-       if (priv->tso) {
-               tmpl->max_tso_header = max_tso_inline * RTE_CACHE_LINE_SIZE;
-               tmpl->txq.max_inline = RTE_MAX(tmpl->txq.max_inline,
-                                              max_tso_inline);
-               tmpl->txq.tso_en = 1;
+       if (tso) {
+               txq_ctrl->max_tso_header = max_tso_inline * RTE_CACHE_LINE_SIZE;
+               txq_ctrl->txq.max_inline = RTE_MAX(txq_ctrl->txq.max_inline,
+                                                  max_tso_inline);
+               txq_ctrl->txq.tso_en = 1;
        }
-       if (priv->tunnel_en)
-               tmpl->txq.tunnel_en = 1;
+       txq_ctrl->txq.tunnel_en = config->tunnel_en;
+}
+
+/**
+ * Create a DPDK Tx queue.
+ *
+ * @param priv
+ *   Pointer to private structure.
+ * @param idx
+ *   TX queue index.
+ * @param desc
+ *   Number of descriptors to configure in queue.
+ * @param socket
+ *   NUMA socket on which memory must be allocated.
+ * @param[in] conf
+ *  Thresholds parameters.
+ *
+ * @return
+ *   A DPDK queue object on success.
+ */
+struct mlx5_txq_ctrl*
+mlx5_priv_txq_new(struct priv *priv, uint16_t idx, uint16_t desc,
+                 unsigned int socket,
+                 const struct rte_eth_txconf *conf)
+{
+       struct mlx5_txq_ctrl *tmpl;
+
+       tmpl = rte_calloc_socket("TXQ", 1,
+                                sizeof(*tmpl) +
+                                desc * sizeof(struct rte_mbuf *),
+                                0, socket);
+       if (!tmpl)
+               return NULL;
+       assert(desc > MLX5_TX_COMP_THRESH);
+       tmpl->txq.offloads = conf->offloads;
+       tmpl->priv = priv;
+       tmpl->socket = socket;
+       tmpl->txq.elts_n = log2above(desc);
+       txq_set_params(tmpl);
+       /* MRs will be registered in mp2mr[] later. */
+       DEBUG("priv->device_attr.max_qp_wr is %d",
+             priv->device_attr.orig_attr.max_qp_wr);
+       DEBUG("priv->device_attr.max_sge is %d",
+             priv->device_attr.orig_attr.max_sge);
        tmpl->txq.elts =
                (struct rte_mbuf *(*)[1 << tmpl->txq.elts_n])(tmpl + 1);
        tmpl->txq.stats.idx = idx;