doc: fix naming of Mellanox devices
[dpdk.git] / drivers / net / mlx5 / mlx5_txq.c
index 2db1120..57bc116 100644 (file)
@@ -4,7 +4,6 @@
  */
 
 #include <stddef.h>
-#include <assert.h>
 #include <errno.h>
 #include <string.h>
 #include <stdint.h>
 #include <rte_ethdev_driver.h>
 #include <rte_common.h>
 
-#include "mlx5_utils.h"
+#include <mlx5_glue.h>
+#include <mlx5_devx_cmds.h>
+#include <mlx5_common.h>
+
 #include "mlx5_defs.h"
+#include "mlx5_utils.h"
 #include "mlx5.h"
 #include "mlx5_rxtx.h"
 #include "mlx5_autoconf.h"
-#include "mlx5_glue.h"
 
 /**
  * Allocate TX queue elements.
@@ -80,9 +82,9 @@ txq_free_elts(struct mlx5_txq_ctrl *txq_ctrl)
        while (elts_tail != elts_head) {
                struct rte_mbuf *elt = (*elts)[elts_tail & elts_m];
 
-               assert(elt != NULL);
+               MLX5_ASSERT(elt != NULL);
                rte_pktmbuf_free_seg(elt);
-#ifndef NDEBUG
+#ifdef RTE_LIBRTE_MLX5_DEBUG
                /* Poisoning. */
                memset(&(*elts)[elts_tail & elts_m],
                       0x77,
@@ -344,8 +346,8 @@ txq_uar_init(struct mlx5_txq_ctrl *txq_ctrl)
 
        if (txq_ctrl->type != MLX5_TXQ_TYPE_STANDARD)
                return;
-       assert(rte_eal_process_type() == RTE_PROC_PRIMARY);
-       assert(ppriv);
+       MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_PRIMARY);
+       MLX5_ASSERT(ppriv);
        ppriv->uar_table[txq_ctrl->txq.idx] = txq_ctrl->bf_reg;
        txq_uar_ncattr_init(txq_ctrl, page_size);
 #ifndef RTE_ARCH_64
@@ -383,7 +385,7 @@ txq_uar_init_secondary(struct mlx5_txq_ctrl *txq_ctrl, int fd)
 
        if (txq_ctrl->type != MLX5_TXQ_TYPE_STANDARD)
                return 0;
-       assert(ppriv);
+       MLX5_ASSERT(ppriv);
        /*
         * As rdma-core, UARs are mapped in size of OS page
         * size. Ref to libmlx5 function: mlx5_init_context()
@@ -444,7 +446,7 @@ mlx5_tx_uar_init_secondary(struct rte_eth_dev *dev, int fd)
        unsigned int i;
        int ret;
 
-       assert(rte_eal_process_type() == RTE_PROC_SECONDARY);
+       MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_SECONDARY);
        for (i = 0; i != priv->txqs_n; ++i) {
                if (!(*priv->txqs)[i])
                        continue;
@@ -452,7 +454,7 @@ mlx5_tx_uar_init_secondary(struct rte_eth_dev *dev, int fd)
                txq_ctrl = container_of(txq, struct mlx5_txq_ctrl, txq);
                if (txq_ctrl->type != MLX5_TXQ_TYPE_STANDARD)
                        continue;
-               assert(txq->idx == (uint16_t)i);
+               MLX5_ASSERT(txq->idx == (uint16_t)i);
                ret = txq_uar_init_secondary(txq_ctrl, fd);
                if (ret)
                        goto error;
@@ -491,9 +493,10 @@ mlx5_txq_obj_hairpin_new(struct rte_eth_dev *dev, uint16_t idx)
        struct mlx5_devx_create_sq_attr attr = { 0 };
        struct mlx5_txq_obj *tmpl = NULL;
        int ret = 0;
+       uint32_t max_wq_data;
 
-       assert(txq_data);
-       assert(!txq_ctrl->obj);
+       MLX5_ASSERT(txq_data);
+       MLX5_ASSERT(!txq_ctrl->obj);
        tmpl = rte_calloc_socket(__func__, 1, sizeof(*tmpl), 0,
                                 txq_ctrl->socket);
        if (!tmpl) {
@@ -507,11 +510,15 @@ mlx5_txq_obj_hairpin_new(struct rte_eth_dev *dev, uint16_t idx)
        tmpl->txq_ctrl = txq_ctrl;
        attr.hairpin = 1;
        attr.tis_lst_sz = 1;
-       /* Workaround for hairpin startup */
-       attr.wq_attr.log_hairpin_num_packets = log2above(32);
-       /* Workaround for packets larger than 1KB */
+       max_wq_data = priv->config.hca_attr.log_max_hairpin_wq_data_sz;
+       /* Jumbo frames > 9KB should be supported, and more packets. */
        attr.wq_attr.log_hairpin_data_sz =
-                       priv->config.hca_attr.log_max_hairpin_wq_data_sz;
+                       (max_wq_data < MLX5_HAIRPIN_JUMBO_LOG_SIZE) ?
+                       max_wq_data : MLX5_HAIRPIN_JUMBO_LOG_SIZE;
+       /* Set the packets number to the maximum value for performance. */
+       attr.wq_attr.log_hairpin_num_packets =
+                       attr.wq_attr.log_hairpin_data_sz -
+                       MLX5_HAIRPIN_QUEUE_STRIDE;
        attr.tis_num = priv->sh->tis->id;
        tmpl->sq = mlx5_devx_cmd_create_sq(priv->sh->ctx, &attr);
        if (!tmpl->sq) {
@@ -578,7 +585,7 @@ mlx5_txq_obj_new(struct rte_eth_dev *dev, uint16_t idx,
        if (priv->config.devx && !priv->sh->tdn)
                qp.comp_mask |= MLX5DV_QP_MASK_RAW_QP_HANDLES;
 #endif
-       assert(txq_data);
+       MLX5_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")) {
@@ -717,9 +724,7 @@ mlx5_txq_obj_new(struct rte_eth_dev *dev, uint16_t idx,
        txq_data->cq_db = cq_info.dbrec;
        txq_data->cqes = (volatile struct mlx5_cqe *)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_data->wqe_comp = 0;
@@ -831,7 +836,7 @@ mlx5_txq_obj_get(struct rte_eth_dev *dev, uint16_t idx)
 int
 mlx5_txq_obj_release(struct mlx5_txq_obj *txq_obj)
 {
-       assert(txq_obj);
+       MLX5_ASSERT(txq_obj);
        if (rte_atomic32_dec_and_test(&txq_obj->refcnt)) {
                if (txq_obj->type == MLX5_TXQ_OBJ_TYPE_DEVX_HAIRPIN) {
                        if (txq_obj->tis)
@@ -978,7 +983,7 @@ txq_set_params(struct mlx5_txq_ctrl *txq_ctrl)
         * If there is requested minimal amount of data to inline
         * we MUST enable inlining. This is a case for ConnectX-4
         * which usually requires L2 inlined for correct operating
-        * and ConnectX-4LX which requires L2-L4 inlined to
+        * and ConnectX-4 Lx which requires L2-L4 inlined to
         * support E-Switch Flows.
         */
        if (inlen_mode) {
@@ -1049,12 +1054,12 @@ txq_set_params(struct mlx5_txq_ctrl *txq_ctrl)
                 * beginning of inlining buffer in Ethernet
                 * Segment.
                 */
-               assert(inlen_send >= MLX5_ESEG_MIN_INLINE_SIZE);
-               assert(inlen_send <= MLX5_WQE_SIZE_MAX +
-                                    MLX5_ESEG_MIN_INLINE_SIZE -
-                                    MLX5_WQE_CSEG_SIZE -
-                                    MLX5_WQE_ESEG_SIZE -
-                                    MLX5_WQE_DSEG_SIZE * 2);
+               MLX5_ASSERT(inlen_send >= MLX5_ESEG_MIN_INLINE_SIZE);
+               MLX5_ASSERT(inlen_send <= MLX5_WQE_SIZE_MAX +
+                                         MLX5_ESEG_MIN_INLINE_SIZE -
+                                         MLX5_WQE_CSEG_SIZE -
+                                         MLX5_WQE_ESEG_SIZE -
+                                         MLX5_WQE_DSEG_SIZE * 2);
        } else if (inlen_mode) {
                /*
                 * If minimal inlining is requested we must
@@ -1104,12 +1109,12 @@ txq_set_params(struct mlx5_txq_ctrl *txq_ctrl)
                                PORT_ID(priv), inlen_empw, temp);
                        inlen_empw = temp;
                }
-               assert(inlen_empw >= MLX5_ESEG_MIN_INLINE_SIZE);
-               assert(inlen_empw <= MLX5_WQE_SIZE_MAX +
-                                    MLX5_DSEG_MIN_INLINE_SIZE -
-                                    MLX5_WQE_CSEG_SIZE -
-                                    MLX5_WQE_ESEG_SIZE -
-                                    MLX5_WQE_DSEG_SIZE);
+               MLX5_ASSERT(inlen_empw >= MLX5_ESEG_MIN_INLINE_SIZE);
+               MLX5_ASSERT(inlen_empw <= MLX5_WQE_SIZE_MAX +
+                                         MLX5_DSEG_MIN_INLINE_SIZE -
+                                         MLX5_WQE_CSEG_SIZE -
+                                         MLX5_WQE_ESEG_SIZE -
+                                         MLX5_WQE_DSEG_SIZE);
                txq_ctrl->txq.inlen_empw = inlen_empw;
        }
        txq_ctrl->max_inline_data = RTE_MAX(inlen_send, inlen_empw);
@@ -1224,11 +1229,11 @@ txq_adjust_params(struct mlx5_txq_ctrl *txq_ctrl)
        }
        txq_ctrl->max_inline_data = RTE_MAX(txq_ctrl->txq.inlen_send,
                                            txq_ctrl->txq.inlen_empw);
-       assert(txq_ctrl->max_inline_data <= max_inline);
-       assert(txq_ctrl->txq.inlen_mode <= max_inline);
-       assert(txq_ctrl->txq.inlen_mode <= txq_ctrl->txq.inlen_send);
-       assert(txq_ctrl->txq.inlen_mode <= txq_ctrl->txq.inlen_empw ||
-              !txq_ctrl->txq.inlen_empw);
+       MLX5_ASSERT(txq_ctrl->max_inline_data <= max_inline);
+       MLX5_ASSERT(txq_ctrl->txq.inlen_mode <= max_inline);
+       MLX5_ASSERT(txq_ctrl->txq.inlen_mode <= txq_ctrl->txq.inlen_send);
+       MLX5_ASSERT(txq_ctrl->txq.inlen_mode <= txq_ctrl->txq.inlen_empw ||
+                   !txq_ctrl->txq.inlen_empw);
        return 0;
 error:
        rte_errno = ENOMEM;
@@ -1274,7 +1279,7 @@ mlx5_txq_new(struct rte_eth_dev *dev, uint16_t idx, uint16_t desc,
        }
        /* Save pointer of global generation number to check memory event. */
        tmpl->txq.mr_ctrl.dev_gen_ptr = &priv->sh->mr.dev_gen;
-       assert(desc > MLX5_TX_COMP_THRESH);
+       MLX5_ASSERT(desc > MLX5_TX_COMP_THRESH);
        tmpl->txq.offloads = conf->offloads |
                             dev->data->dev_conf.txmode.offloads;
        tmpl->priv = priv;