net/mlx5: fix Rx segmented packets on mbuf starvation
[dpdk.git] / drivers / net / mlx5 / mlx5_devx.c
index da3bb78..2cb3bd1 100644 (file)
@@ -15,6 +15,7 @@
 
 #include <mlx5_glue.h>
 #include <mlx5_devx_cmds.h>
+#include <mlx5_common_devx.h>
 #include <mlx5_malloc.h>
 
 #include "mlx5.h"
@@ -44,7 +45,7 @@ mlx5_rxq_obj_modify_rq_vlan_strip(struct mlx5_rxq_obj *rxq_obj, int on)
        rq_attr.state = MLX5_RQC_STATE_RDY;
        rq_attr.vsd = (on ? 0 : 1);
        rq_attr.modify_bitmask = MLX5_MODIFY_RQ_IN_MODIFY_BITMASK_VSD;
-       return mlx5_devx_cmd_modify_rq(rxq_obj->rq, &rq_attr);
+       return mlx5_devx_cmd_modify_rq(rxq_obj->rq_obj.rq, &rq_attr);
 }
 
 /**
@@ -84,7 +85,7 @@ mlx5_devx_modify_rq(struct mlx5_rxq_obj *rxq_obj, uint8_t type)
        default:
                break;
        }
-       return mlx5_devx_cmd_modify_rq(rxq_obj->rq, &rq_attr);
+       return mlx5_devx_cmd_modify_rq(rxq_obj->rq_obj.rq, &rq_attr);
 }
 
 /**
@@ -114,7 +115,7 @@ mlx5_devx_modify_sq(struct mlx5_txq_obj *obj, enum mlx5_txq_modify_type type,
                else
                        msq_attr.sq_state = MLX5_SQC_STATE_RDY;
                msq_attr.state = MLX5_SQC_STATE_RST;
-               ret = mlx5_devx_cmd_modify_sq(obj->sq_devx, &msq_attr);
+               ret = mlx5_devx_cmd_modify_sq(obj->sq_obj.sq, &msq_attr);
                if (ret) {
                        DRV_LOG(ERR, "Cannot change the Tx SQ state to RESET"
                                " %s", strerror(errno));
@@ -126,7 +127,7 @@ mlx5_devx_modify_sq(struct mlx5_txq_obj *obj, enum mlx5_txq_modify_type type,
                /* Change queue state to ready. */
                msq_attr.sq_state = MLX5_SQC_STATE_RST;
                msq_attr.state = MLX5_SQC_STATE_RDY;
-               ret = mlx5_devx_cmd_modify_sq(obj->sq_devx, &msq_attr);
+               ret = mlx5_devx_cmd_modify_sq(obj->sq_obj.sq, &msq_attr);
                if (ret) {
                        DRV_LOG(ERR, "Cannot change the Tx SQ state to READY"
                                " %s", strerror(errno));
@@ -144,57 +145,18 @@ mlx5_devx_modify_sq(struct mlx5_txq_obj *obj, enum mlx5_txq_modify_type type,
 }
 
 /**
- * Release the resources allocated for an RQ DevX object.
+ * Destroy the Rx queue DevX object.
  *
- * @param rxq_ctrl
- *   DevX Rx queue object.
- */
-static void
-mlx5_rxq_release_devx_rq_resources(struct mlx5_rxq_ctrl *rxq_ctrl)
-{
-       struct mlx5_devx_dbr_page *dbr_page = rxq_ctrl->rq_dbrec_page;
-
-       if (rxq_ctrl->wq_umem) {
-               mlx5_os_umem_dereg(rxq_ctrl->wq_umem);
-               rxq_ctrl->wq_umem = NULL;
-       }
-       if (rxq_ctrl->rxq.wqes) {
-               mlx5_free((void *)(uintptr_t)rxq_ctrl->rxq.wqes);
-               rxq_ctrl->rxq.wqes = NULL;
-       }
-       if (dbr_page) {
-               claim_zero(mlx5_release_dbr(&rxq_ctrl->priv->dbrpgs,
-                                           mlx5_os_get_umem_id(dbr_page->umem),
-                                           rxq_ctrl->rq_dbr_offset));
-               rxq_ctrl->rq_dbrec_page = NULL;
-       }
-}
-
-/**
- * Release the resources allocated for the Rx CQ DevX object.
- *
- * @param rxq_ctrl
- *   DevX Rx queue object.
+ * @param rxq_obj
+ *   Rxq object to destroy.
  */
 static void
-mlx5_rxq_release_devx_cq_resources(struct mlx5_rxq_ctrl *rxq_ctrl)
+mlx5_rxq_release_devx_resources(struct mlx5_rxq_obj *rxq_obj)
 {
-       struct mlx5_devx_dbr_page *dbr_page = rxq_ctrl->cq_dbrec_page;
-
-       if (rxq_ctrl->cq_umem) {
-               mlx5_os_umem_dereg(rxq_ctrl->cq_umem);
-               rxq_ctrl->cq_umem = NULL;
-       }
-       if (rxq_ctrl->rxq.cqes) {
-               rte_free((void *)(uintptr_t)rxq_ctrl->rxq.cqes);
-               rxq_ctrl->rxq.cqes = NULL;
-       }
-       if (dbr_page) {
-               claim_zero(mlx5_release_dbr(&rxq_ctrl->priv->dbrpgs,
-                                           mlx5_os_get_umem_id(dbr_page->umem),
-                                           rxq_ctrl->cq_dbr_offset));
-               rxq_ctrl->cq_dbrec_page = NULL;
-       }
+       mlx5_devx_rq_destroy(&rxq_obj->rq_obj);
+       memset(&rxq_obj->rq_obj, 0, sizeof(rxq_obj->rq_obj));
+       mlx5_devx_cq_destroy(&rxq_obj->cq_obj);
+       memset(&rxq_obj->cq_obj, 0, sizeof(rxq_obj->cq_obj));
 }
 
 /**
@@ -207,19 +169,17 @@ static void
 mlx5_rxq_devx_obj_release(struct mlx5_rxq_obj *rxq_obj)
 {
        MLX5_ASSERT(rxq_obj);
-       MLX5_ASSERT(rxq_obj->rq);
        if (rxq_obj->rxq_ctrl->type == MLX5_RXQ_TYPE_HAIRPIN) {
+               MLX5_ASSERT(rxq_obj->rq);
                mlx5_devx_modify_rq(rxq_obj, MLX5_RXQ_MOD_RDY2RST);
                claim_zero(mlx5_devx_cmd_destroy(rxq_obj->rq));
        } else {
-               MLX5_ASSERT(rxq_obj->devx_cq);
-               claim_zero(mlx5_devx_cmd_destroy(rxq_obj->rq));
-               claim_zero(mlx5_devx_cmd_destroy(rxq_obj->devx_cq));
+               MLX5_ASSERT(rxq_obj->cq_obj.cq);
+               MLX5_ASSERT(rxq_obj->rq_obj.rq);
+               mlx5_rxq_release_devx_resources(rxq_obj);
                if (rxq_obj->devx_channel)
                        mlx5_os_devx_destroy_event_channel
                                                        (rxq_obj->devx_channel);
-               mlx5_rxq_release_devx_rq_resources(rxq_obj->rxq_ctrl);
-               mlx5_rxq_release_devx_cq_resources(rxq_obj->rxq_ctrl);
        }
 }
 
@@ -248,7 +208,7 @@ mlx5_rx_devx_get_event(struct mlx5_rxq_obj *rxq_obj)
                rte_errno = errno;
                return -rte_errno;
        }
-       if (out.event_resp.cookie != (uint64_t)(uintptr_t)rxq_obj->devx_cq) {
+       if (out.event_resp.cookie != (uint64_t)(uintptr_t)rxq_obj->cq_obj.cq) {
                rte_errno = EINVAL;
                return -rte_errno;
        }
@@ -260,52 +220,6 @@ mlx5_rx_devx_get_event(struct mlx5_rxq_obj *rxq_obj)
 #endif /* HAVE_IBV_DEVX_EVENT */
 }
 
-/**
- * Fill common fields of create RQ attributes structure.
- *
- * @param rxq_data
- *   Pointer to Rx queue data.
- * @param cqn
- *   CQ number to use with this RQ.
- * @param rq_attr
- *   RQ attributes structure to fill..
- */
-static void
-mlx5_devx_create_rq_attr_fill(struct mlx5_rxq_data *rxq_data, uint32_t cqn,
-                             struct mlx5_devx_create_rq_attr *rq_attr)
-{
-       rq_attr->state = MLX5_RQC_STATE_RST;
-       rq_attr->vsd = (rxq_data->vlan_strip) ? 0 : 1;
-       rq_attr->cqn = cqn;
-       rq_attr->scatter_fcs = (rxq_data->crc_present) ? 1 : 0;
-}
-
-/**
- * Fill common fields of DevX WQ attributes structure.
- *
- * @param priv
- *   Pointer to device private data.
- * @param rxq_ctrl
- *   Pointer to Rx queue control structure.
- * @param wq_attr
- *   WQ attributes structure to fill..
- */
-static void
-mlx5_devx_wq_attr_fill(struct mlx5_priv *priv, struct mlx5_rxq_ctrl *rxq_ctrl,
-                      struct mlx5_devx_wq_attr *wq_attr)
-{
-       wq_attr->end_padding_mode = priv->config.hw_padding ?
-                                       MLX5_WQ_END_PAD_MODE_ALIGN :
-                                       MLX5_WQ_END_PAD_MODE_NONE;
-       wq_attr->pd = priv->sh->pdn;
-       wq_attr->dbr_addr = rxq_ctrl->rq_dbr_offset;
-       wq_attr->dbr_umem_id =
-                       mlx5_os_get_umem_id(rxq_ctrl->rq_dbrec_page->umem);
-       wq_attr->dbr_umem_valid = 1;
-       wq_attr->wq_umem_id = mlx5_os_get_umem_id(rxq_ctrl->wq_umem);
-       wq_attr->wq_umem_valid = 1;
-}
-
 /**
  * Create a RQ object using DevX.
  *
@@ -315,9 +229,9 @@ mlx5_devx_wq_attr_fill(struct mlx5_priv *priv, struct mlx5_rxq_ctrl *rxq_ctrl,
  *   Queue index in DPDK Rx queue array.
  *
  * @return
- *   The DevX RQ object initialized, NULL otherwise and rte_errno is set.
+ *   0 on success, a negative errno value otherwise and rte_errno is set.
  */
-static struct mlx5_devx_obj *
+static int
 mlx5_rxq_create_devx_rq_resources(struct rte_eth_dev *dev, uint16_t idx)
 {
        struct mlx5_priv *priv = dev->data->dev_private;
@@ -325,20 +239,15 @@ mlx5_rxq_create_devx_rq_resources(struct rte_eth_dev *dev, uint16_t idx)
        struct mlx5_rxq_ctrl *rxq_ctrl =
                container_of(rxq_data, struct mlx5_rxq_ctrl, rxq);
        struct mlx5_devx_create_rq_attr rq_attr = { 0 };
-       uint32_t wqe_n = 1 << (rxq_data->elts_n - rxq_data->sges_n);
-       uint32_t cqn = rxq_ctrl->obj->devx_cq->id;
-       struct mlx5_devx_dbr_page *dbr_page;
-       int64_t dbr_offset;
-       uint32_t wq_size = 0;
-       uint32_t wqe_size = 0;
-       uint32_t log_wqe_size = 0;
-       void *buf = NULL;
-       struct mlx5_devx_obj *rq;
+       uint16_t log_desc_n = rxq_data->elts_n - rxq_data->sges_n;
+       uint32_t wqe_size, log_wqe_size;
 
        /* Fill RQ attributes. */
        rq_attr.mem_rq_type = MLX5_RQC_MEM_RQ_TYPE_MEMORY_RQ_INLINE;
        rq_attr.flush_in_error_en = 1;
-       mlx5_devx_create_rq_attr_fill(rxq_data, cqn, &rq_attr);
+       rq_attr.vsd = (rxq_data->vlan_strip) ? 0 : 1;
+       rq_attr.cqn = rxq_ctrl->obj->cq_obj.cq->id;
+       rq_attr.scatter_fcs = (rxq_data->crc_present) ? 1 : 0;
        /* Fill WQ attributes for this RQ. */
        if (mlx5_rxq_mprq_enabled(rxq_data)) {
                rq_attr.wq_attr.wq_type = MLX5_WQ_TYPE_CYCLIC_STRIDING_RQ;
@@ -359,45 +268,18 @@ mlx5_rxq_create_devx_rq_resources(struct rte_eth_dev *dev, uint16_t idx)
                wqe_size = sizeof(struct mlx5_wqe_data_seg);
        }
        log_wqe_size = log2above(wqe_size) + rxq_data->sges_n;
-       rq_attr.wq_attr.log_wq_stride = log_wqe_size;
-       rq_attr.wq_attr.log_wq_sz = rxq_data->elts_n - rxq_data->sges_n;
-       /* Calculate and allocate WQ memory space. */
        wqe_size = 1 << log_wqe_size; /* round up power of two.*/
-       wq_size = wqe_n * wqe_size;
-       size_t alignment = MLX5_WQE_BUF_ALIGNMENT;
-       if (alignment == (size_t)-1) {
-               DRV_LOG(ERR, "Failed to get mem page size");
-               rte_errno = ENOMEM;
-               return NULL;
-       }
-       buf = mlx5_malloc(MLX5_MEM_RTE | MLX5_MEM_ZERO, wq_size,
-                         alignment, rxq_ctrl->socket);
-       if (!buf)
-               return NULL;
-       rxq_data->wqes = buf;
-       rxq_ctrl->wq_umem = mlx5_os_umem_reg(priv->sh->ctx,
-                                                    buf, wq_size, 0);
-       if (!rxq_ctrl->wq_umem)
-               goto error;
-       /* Allocate RQ door-bell. */
-       dbr_offset = mlx5_get_dbr(priv->sh->ctx, &priv->dbrpgs, &dbr_page);
-       if (dbr_offset < 0) {
-               DRV_LOG(ERR, "Failed to allocate RQ door-bell.");
-               goto error;
-       }
-       rxq_ctrl->rq_dbr_offset = dbr_offset;
-       rxq_ctrl->rq_dbrec_page = dbr_page;
-       rxq_data->rq_db = (uint32_t *)((uintptr_t)dbr_page->dbrs +
-                         (uintptr_t)rxq_ctrl->rq_dbr_offset);
+       rq_attr.wq_attr.log_wq_stride = log_wqe_size;
+       rq_attr.wq_attr.log_wq_sz = log_desc_n;
+       rq_attr.wq_attr.end_padding_mode = priv->config.hw_padding ?
+                                               MLX5_WQ_END_PAD_MODE_ALIGN :
+                                               MLX5_WQ_END_PAD_MODE_NONE;
+       rq_attr.wq_attr.pd = priv->sh->pdn;
+       rq_attr.counter_set_id = priv->counter_set_id;
        /* Create RQ using DevX API. */
-       mlx5_devx_wq_attr_fill(priv, rxq_ctrl, &rq_attr.wq_attr);
-       rq = mlx5_devx_cmd_create_rq(priv->sh->ctx, &rq_attr, rxq_ctrl->socket);
-       if (!rq)
-               goto error;
-       return rq;
-error:
-       mlx5_rxq_release_devx_rq_resources(rxq_ctrl);
-       return NULL;
+       return mlx5_devx_rq_create(priv->sh->ctx, &rxq_ctrl->obj->rq_obj,
+                                  wqe_size, log_desc_n, &rq_attr,
+                                  rxq_ctrl->socket);
 }
 
 /**
@@ -409,31 +291,23 @@ error:
  *   Queue index in DPDK Rx queue array.
  *
  * @return
- *   The DevX CQ object initialized, NULL otherwise and rte_errno is set.
+ *   0 on success, a negative errno value otherwise and rte_errno is set.
  */
-static struct mlx5_devx_obj *
+static int
 mlx5_rxq_create_devx_cq_resources(struct rte_eth_dev *dev, uint16_t idx)
 {
-       struct mlx5_devx_obj *cq_obj = 0;
+       struct mlx5_devx_cq *cq_obj = 0;
        struct mlx5_devx_cq_attr cq_attr = { 0 };
        struct mlx5_priv *priv = dev->data->dev_private;
+       struct mlx5_dev_ctx_shared *sh = priv->sh;
        struct mlx5_rxq_data *rxq_data = (*priv->rxqs)[idx];
        struct mlx5_rxq_ctrl *rxq_ctrl =
                container_of(rxq_data, struct mlx5_rxq_ctrl, rxq);
-       size_t page_size = rte_mem_page_size();
        unsigned int cqe_n = mlx5_rxq_cqe_num(rxq_data);
-       struct mlx5_devx_dbr_page *dbr_page;
-       int64_t dbr_offset;
-       void *buf = NULL;
-       uint16_t event_nums[1] = {0};
        uint32_t log_cqe_n;
-       uint32_t cq_size;
+       uint16_t event_nums[1] = { 0 };
        int ret = 0;
 
-       if (page_size == (size_t)-1) {
-               DRV_LOG(ERR, "Failed to get page_size.");
-               goto error;
-       }
        if (priv->config.cqe_comp && !rxq_data->hw_timestamp &&
            !rxq_data->lro) {
                cq_attr.cqe_comp_en = 1u;
@@ -486,73 +360,37 @@ mlx5_rxq_create_devx_cq_resources(struct rte_eth_dev *dev, uint16_t idx)
                        "Port %u Rx CQE compression is disabled for LRO.",
                        dev->data->port_id);
        }
-       if (priv->config.cqe_pad)
-               cq_attr.cqe_size = MLX5_CQE_SIZE_128B;
+       cq_attr.uar_page_id = mlx5_os_get_devx_uar_page_id(sh->devx_rx_uar);
        log_cqe_n = log2above(cqe_n);
-       cq_size = sizeof(struct mlx5_cqe) * (1 << log_cqe_n);
-       buf = rte_calloc_socket(__func__, 1, cq_size, page_size,
-                               rxq_ctrl->socket);
-       if (!buf) {
-               DRV_LOG(ERR, "Failed to allocate memory for CQ.");
-               goto error;
-       }
-       rxq_data->cqes = (volatile struct mlx5_cqe (*)[])(uintptr_t)buf;
-       rxq_ctrl->cq_umem = mlx5_os_umem_reg(priv->sh->ctx, buf,
-                                                    cq_size,
-                                                    IBV_ACCESS_LOCAL_WRITE);
-       if (!rxq_ctrl->cq_umem) {
-               DRV_LOG(ERR, "Failed to register umem for CQ.");
-               goto error;
-       }
-       /* Allocate CQ door-bell. */
-       dbr_offset = mlx5_get_dbr(priv->sh->ctx, &priv->dbrpgs, &dbr_page);
-       if (dbr_offset < 0) {
-               DRV_LOG(ERR, "Failed to allocate CQ door-bell.");
-               goto error;
-       }
-       rxq_ctrl->cq_dbr_offset = dbr_offset;
-       rxq_ctrl->cq_dbrec_page = dbr_page;
-       rxq_data->cq_db = (uint32_t *)((uintptr_t)dbr_page->dbrs +
-                         (uintptr_t)rxq_ctrl->cq_dbr_offset);
-       rxq_data->cq_uar =
-                       mlx5_os_get_devx_uar_base_addr(priv->sh->devx_rx_uar);
        /* Create CQ using DevX API. */
-       cq_attr.eqn = priv->sh->eqn;
-       cq_attr.uar_page_id =
-                       mlx5_os_get_devx_uar_page_id(priv->sh->devx_rx_uar);
-       cq_attr.q_umem_id = mlx5_os_get_umem_id(rxq_ctrl->cq_umem);
-       cq_attr.q_umem_valid = 1;
-       cq_attr.log_cq_size = log_cqe_n;
-       cq_attr.log_page_size = rte_log2_u32(page_size);
-       cq_attr.db_umem_offset = rxq_ctrl->cq_dbr_offset;
-       cq_attr.db_umem_id = mlx5_os_get_umem_id(dbr_page->umem);
-       cq_attr.db_umem_valid = 1;
-       cq_obj = mlx5_devx_cmd_create_cq(priv->sh->ctx, &cq_attr);
-       if (!cq_obj)
-               goto error;
+       ret = mlx5_devx_cq_create(sh->ctx, &rxq_ctrl->obj->cq_obj, log_cqe_n,
+                                 &cq_attr, sh->numa_node);
+       if (ret)
+               return ret;
+       cq_obj = &rxq_ctrl->obj->cq_obj;
+       rxq_data->cqes = (volatile struct mlx5_cqe (*)[])
+                                                       (uintptr_t)cq_obj->cqes;
+       rxq_data->cq_db = cq_obj->db_rec;
+       rxq_data->cq_uar = mlx5_os_get_devx_uar_base_addr(sh->devx_rx_uar);
        rxq_data->cqe_n = log_cqe_n;
-       rxq_data->cqn = cq_obj->id;
+       rxq_data->cqn = cq_obj->cq->id;
        if (rxq_ctrl->obj->devx_channel) {
                ret = mlx5_os_devx_subscribe_devx_event
-                                               (rxq_ctrl->obj->devx_channel,
-                                                cq_obj->obj,
-                                                sizeof(event_nums),
-                                                event_nums,
-                                                (uint64_t)(uintptr_t)cq_obj);
+                                             (rxq_ctrl->obj->devx_channel,
+                                              cq_obj->cq->obj,
+                                              sizeof(event_nums),
+                                              event_nums,
+                                              (uint64_t)(uintptr_t)cq_obj->cq);
                if (ret) {
                        DRV_LOG(ERR, "Fail to subscribe CQ to event channel.");
-                       rte_errno = errno;
-                       goto error;
+                       ret = errno;
+                       mlx5_devx_cq_destroy(cq_obj);
+                       memset(cq_obj, 0, sizeof(*cq_obj));
+                       rte_errno = ret;
+                       return -ret;
                }
        }
-       /* Initialise CQ to 1's to mark HW ownership for all CQEs. */
-       memset((void *)(uintptr_t)rxq_data->cqes, 0xFF, cq_size);
-       return cq_obj;
-error:
-       if (cq_obj)
-               mlx5_devx_cmd_destroy(cq_obj);
-       mlx5_rxq_release_devx_cq_resources(rxq_ctrl);
-       return NULL;
+       return 0;
 }
 
 /**
@@ -601,6 +439,7 @@ mlx5_rxq_obj_hairpin_new(struct rte_eth_dev *dev, uint16_t idx)
        attr.wq_attr.log_hairpin_num_packets =
                        attr.wq_attr.log_hairpin_data_sz -
                        MLX5_HAIRPIN_QUEUE_STRIDE;
+       attr.counter_set_id = priv->counter_set_id;
        tmpl->rq = mlx5_devx_cmd_create_rq(priv->sh->ctx, &attr,
                                           rxq_ctrl->socket);
        if (!tmpl->rq) {
@@ -656,14 +495,14 @@ mlx5_rxq_devx_obj_new(struct rte_eth_dev *dev, uint16_t idx)
                tmpl->fd = mlx5_os_get_devx_channel_fd(tmpl->devx_channel);
        }
        /* Create CQ using DevX API. */
-       tmpl->devx_cq = mlx5_rxq_create_devx_cq_resources(dev, idx);
-       if (!tmpl->devx_cq) {
+       ret = mlx5_rxq_create_devx_cq_resources(dev, idx);
+       if (ret) {
                DRV_LOG(ERR, "Failed to create CQ.");
                goto error;
        }
        /* Create RQ using DevX API. */
-       tmpl->rq = mlx5_rxq_create_devx_rq_resources(dev, idx);
-       if (!tmpl->rq) {
+       ret = mlx5_rxq_create_devx_rq_resources(dev, idx);
+       if (ret) {
                DRV_LOG(ERR, "Port %u Rx queue %u RQ creation failure.",
                        dev->data->port_id, idx);
                rte_errno = ENOMEM;
@@ -673,22 +512,17 @@ mlx5_rxq_devx_obj_new(struct rte_eth_dev *dev, uint16_t idx)
        ret = mlx5_devx_modify_rq(tmpl, MLX5_RXQ_MOD_RST2RDY);
        if (ret)
                goto error;
+       rxq_data->wqes = (void *)(uintptr_t)tmpl->rq_obj.umem_buf;
+       rxq_data->rq_db = (uint32_t *)(uintptr_t)tmpl->rq_obj.db_rec;
        rxq_data->cq_arm_sn = 0;
-       mlx5_rxq_initialize(rxq_data);
        rxq_data->cq_ci = 0;
+       mlx5_rxq_initialize(rxq_data);
        dev->data->rx_queue_state[idx] = RTE_ETH_QUEUE_STATE_STARTED;
-       rxq_ctrl->wqn = tmpl->rq->id;
+       rxq_ctrl->wqn = tmpl->rq_obj.rq->id;
        return 0;
 error:
        ret = rte_errno; /* Save rte_errno before cleanup. */
-       if (tmpl->rq)
-               claim_zero(mlx5_devx_cmd_destroy(tmpl->rq));
-       if (tmpl->devx_cq)
-               claim_zero(mlx5_devx_cmd_destroy(tmpl->devx_cq));
-       if (tmpl->devx_channel)
-               mlx5_os_devx_destroy_event_channel(tmpl->devx_channel);
-       mlx5_rxq_release_devx_rq_resources(rxq_ctrl);
-       mlx5_rxq_release_devx_cq_resources(rxq_ctrl);
+       mlx5_rxq_devx_obj_release(tmpl);
        rte_errno = ret; /* Restore rte_errno. */
        return -rte_errno;
 }
@@ -732,7 +566,7 @@ mlx5_devx_ind_table_create_rqt_attr(struct rte_eth_dev *dev,
                struct mlx5_rxq_ctrl *rxq_ctrl =
                                container_of(rxq, struct mlx5_rxq_ctrl, rxq);
 
-               rqt_attr->rq_list[i] = rxq_ctrl->obj->rq->id;
+               rqt_attr->rq_list[i] = rxq_ctrl->obj->rq_obj.rq->id;
        }
        MLX5_ASSERT(i > 0);
        for (j = 0; i != rqt_n; ++j, ++i)
@@ -1112,58 +946,6 @@ mlx5_txq_obj_hairpin_new(struct rte_eth_dev *dev, uint16_t idx)
 }
 
 #if defined(HAVE_MLX5DV_DEVX_UAR_OFFSET) || !defined(HAVE_INFINIBAND_VERBS_H)
-/**
- * Release DevX SQ resources.
- *
- * @param txq_obj
- *   DevX Tx queue object.
- */
-static void
-mlx5_txq_release_devx_sq_resources(struct mlx5_txq_obj *txq_obj)
-{
-       if (txq_obj->sq_devx) {
-               claim_zero(mlx5_devx_cmd_destroy(txq_obj->sq_devx));
-               txq_obj->sq_devx = NULL;
-       }
-       if (txq_obj->sq_umem) {
-               claim_zero(mlx5_os_umem_dereg(txq_obj->sq_umem));
-               txq_obj->sq_umem = NULL;
-       }
-       if (txq_obj->sq_buf) {
-               mlx5_free(txq_obj->sq_buf);
-               txq_obj->sq_buf = NULL;
-       }
-       if (txq_obj->sq_dbrec_page) {
-               claim_zero(mlx5_release_dbr(&txq_obj->txq_ctrl->priv->dbrpgs,
-                                           mlx5_os_get_umem_id
-                                                (txq_obj->sq_dbrec_page->umem),
-                                           txq_obj->sq_dbrec_offset));
-               txq_obj->sq_dbrec_page = NULL;
-       }
-}
-
-/**
- * Release DevX Tx CQ resources.
- *
- * @param txq_obj
- *   DevX Tx queue object.
- */
-static void
-mlx5_txq_release_devx_cq_resources(struct mlx5_txq_obj *txq_obj)
-{
-       if (txq_obj->cq_devx)
-               claim_zero(mlx5_devx_cmd_destroy(txq_obj->cq_devx));
-       if (txq_obj->cq_umem)
-               claim_zero(mlx5_os_umem_dereg(txq_obj->cq_umem));
-       if (txq_obj->cq_buf)
-               mlx5_free(txq_obj->cq_buf);
-       if (txq_obj->cq_dbrec_page)
-               claim_zero(mlx5_release_dbr(&txq_obj->txq_ctrl->priv->dbrpgs,
-                                           mlx5_os_get_umem_id
-                                                (txq_obj->cq_dbrec_page->umem),
-                                           txq_obj->cq_dbrec_offset));
-}
-
 /**
  * Destroy the Tx queue DevX object.
  *
@@ -1173,127 +955,10 @@ mlx5_txq_release_devx_cq_resources(struct mlx5_txq_obj *txq_obj)
 static void
 mlx5_txq_release_devx_resources(struct mlx5_txq_obj *txq_obj)
 {
-       mlx5_txq_release_devx_sq_resources(txq_obj);
-       mlx5_txq_release_devx_cq_resources(txq_obj);
-}
-
-/**
- * Create a DevX CQ object and its resources for an Tx queue.
- *
- * @param dev
- *   Pointer to Ethernet device.
- * @param idx
- *   Queue index in DPDK Tx queue array.
- *
- * @return
- *   Number of CQEs in CQ, 0 otherwise and rte_errno is set.
- */
-static uint32_t
-mlx5_txq_create_devx_cq_resources(struct rte_eth_dev *dev, uint16_t idx)
-{
-       struct mlx5_priv *priv = dev->data->dev_private;
-       struct mlx5_txq_data *txq_data = (*priv->txqs)[idx];
-       struct mlx5_txq_ctrl *txq_ctrl =
-                       container_of(txq_data, struct mlx5_txq_ctrl, txq);
-       struct mlx5_txq_obj *txq_obj = txq_ctrl->obj;
-       struct mlx5_devx_cq_attr cq_attr = { 0 };
-       struct mlx5_cqe *cqe;
-       size_t page_size;
-       size_t alignment;
-       uint32_t cqe_n;
-       uint32_t i;
-       int ret;
-
-       MLX5_ASSERT(txq_data);
-       MLX5_ASSERT(txq_obj);
-       page_size = rte_mem_page_size();
-       if (page_size == (size_t)-1) {
-               DRV_LOG(ERR, "Failed to get mem page size.");
-               rte_errno = ENOMEM;
-               return 0;
-       }
-       /* Allocate memory buffer for CQEs. */
-       alignment = MLX5_CQE_BUF_ALIGNMENT;
-       if (alignment == (size_t)-1) {
-               DRV_LOG(ERR, "Failed to get CQE buf alignment.");
-               rte_errno = ENOMEM;
-               return 0;
-       }
-       /* Create the Completion Queue. */
-       cqe_n = (1UL << txq_data->elts_n) / MLX5_TX_COMP_THRESH +
-               1 + MLX5_TX_COMP_THRESH_INLINE_DIV;
-       cqe_n = 1UL << log2above(cqe_n);
-       if (cqe_n > UINT16_MAX) {
-               DRV_LOG(ERR,
-                       "Port %u Tx queue %u requests to many CQEs %u.",
-                       dev->data->port_id, txq_data->idx, cqe_n);
-               rte_errno = EINVAL;
-               return 0;
-       }
-       txq_obj->cq_buf = mlx5_malloc(MLX5_MEM_RTE | MLX5_MEM_ZERO,
-                                     cqe_n * sizeof(struct mlx5_cqe),
-                                     alignment,
-                                     priv->sh->numa_node);
-       if (!txq_obj->cq_buf) {
-               DRV_LOG(ERR,
-                       "Port %u Tx queue %u cannot allocate memory (CQ).",
-                       dev->data->port_id, txq_data->idx);
-               rte_errno = ENOMEM;
-               return 0;
-       }
-       /* Register allocated buffer in user space with DevX. */
-       txq_obj->cq_umem = mlx5_os_umem_reg(priv->sh->ctx,
-                                               (void *)txq_obj->cq_buf,
-                                               cqe_n * sizeof(struct mlx5_cqe),
-                                               IBV_ACCESS_LOCAL_WRITE);
-       if (!txq_obj->cq_umem) {
-               rte_errno = errno;
-               DRV_LOG(ERR,
-                       "Port %u Tx queue %u cannot register memory (CQ).",
-                       dev->data->port_id, txq_data->idx);
-               goto error;
-       }
-       /* Allocate doorbell record for completion queue. */
-       txq_obj->cq_dbrec_offset = mlx5_get_dbr(priv->sh->ctx,
-                                               &priv->dbrpgs,
-                                               &txq_obj->cq_dbrec_page);
-       if (txq_obj->cq_dbrec_offset < 0) {
-               rte_errno = errno;
-               DRV_LOG(ERR, "Failed to allocate CQ door-bell.");
-               goto error;
-       }
-       cq_attr.cqe_size = (sizeof(struct mlx5_cqe) == 128) ?
-                           MLX5_CQE_SIZE_128B : MLX5_CQE_SIZE_64B;
-       cq_attr.uar_page_id = mlx5_os_get_devx_uar_page_id(priv->sh->tx_uar);
-       cq_attr.eqn = priv->sh->eqn;
-       cq_attr.q_umem_valid = 1;
-       cq_attr.q_umem_offset = (uintptr_t)txq_obj->cq_buf % page_size;
-       cq_attr.q_umem_id = mlx5_os_get_umem_id(txq_obj->cq_umem);
-       cq_attr.db_umem_valid = 1;
-       cq_attr.db_umem_offset = txq_obj->cq_dbrec_offset;
-       cq_attr.db_umem_id = mlx5_os_get_umem_id(txq_obj->cq_dbrec_page->umem);
-       cq_attr.log_cq_size = rte_log2_u32(cqe_n);
-       cq_attr.log_page_size = rte_log2_u32(page_size);
-       /* Create completion queue object with DevX. */
-       txq_obj->cq_devx = mlx5_devx_cmd_create_cq(priv->sh->ctx, &cq_attr);
-       if (!txq_obj->cq_devx) {
-               rte_errno = errno;
-               DRV_LOG(ERR, "Port %u Tx queue %u CQ creation failure.",
-                       dev->data->port_id, idx);
-               goto error;
-       }
-       /* Initial fill CQ buffer with invalid CQE opcode. */
-       cqe = (struct mlx5_cqe *)txq_obj->cq_buf;
-       for (i = 0; i < cqe_n; i++) {
-               cqe->op_own = (MLX5_CQE_INVALID << 4) | MLX5_CQE_OWNER_MASK;
-               ++cqe;
-       }
-       return cqe_n;
-error:
-       ret = rte_errno;
-       mlx5_txq_release_devx_cq_resources(txq_obj);
-       rte_errno = ret;
-       return 0;
+       mlx5_devx_sq_destroy(&txq_obj->sq_obj);
+       memset(&txq_obj->sq_obj, 0, sizeof(txq_obj->sq_obj));
+       mlx5_devx_cq_destroy(&txq_obj->cq_obj);
+       memset(&txq_obj->cq_obj, 0, sizeof(txq_obj->cq_obj));
 }
 
 /**
@@ -1303,100 +968,39 @@ error:
  *   Pointer to Ethernet device.
  * @param idx
  *   Queue index in DPDK Tx queue array.
+ * @param[in] log_desc_n
+ *   Log of number of descriptors in queue.
  *
  * @return
- *   Number of WQEs in SQ, 0 otherwise and rte_errno is set.
+ *   0 on success, a negative errno value otherwise and rte_errno is set.
  */
-static uint32_t
-mlx5_txq_create_devx_sq_resources(struct rte_eth_dev *dev, uint16_t idx)
+static int
+mlx5_txq_create_devx_sq_resources(struct rte_eth_dev *dev, uint16_t idx,
+                                 uint16_t log_desc_n)
 {
        struct mlx5_priv *priv = dev->data->dev_private;
        struct mlx5_txq_data *txq_data = (*priv->txqs)[idx];
        struct mlx5_txq_ctrl *txq_ctrl =
                        container_of(txq_data, struct mlx5_txq_ctrl, txq);
        struct mlx5_txq_obj *txq_obj = txq_ctrl->obj;
-       struct mlx5_devx_create_sq_attr sq_attr = { 0 };
-       size_t page_size;
-       uint32_t wqe_n;
-       int ret;
+       struct mlx5_devx_create_sq_attr sq_attr = {
+               .flush_in_error_en = 1,
+               .allow_multi_pkt_send_wqe = !!priv->config.mps,
+               .min_wqe_inline_mode = priv->config.hca_attr.vport_inline_mode,
+               .allow_swp = !!priv->config.swp,
+               .cqn = txq_obj->cq_obj.cq->id,
+               .tis_lst_sz = 1,
+               .tis_num = priv->sh->tis->id,
+               .wq_attr = (struct mlx5_devx_wq_attr){
+                       .pd = priv->sh->pdn,
+                       .uar_page =
+                                mlx5_os_get_devx_uar_page_id(priv->sh->tx_uar),
+               },
+       };
 
-       MLX5_ASSERT(txq_data);
-       MLX5_ASSERT(txq_obj);
-       page_size = rte_mem_page_size();
-       if (page_size == (size_t)-1) {
-               DRV_LOG(ERR, "Failed to get mem page size.");
-               rte_errno = ENOMEM;
-               return 0;
-       }
-       wqe_n = RTE_MIN(1UL << txq_data->elts_n,
-                       (uint32_t)priv->sh->device_attr.max_qp_wr);
-       txq_obj->sq_buf = mlx5_malloc(MLX5_MEM_RTE | MLX5_MEM_ZERO,
-                                     wqe_n * sizeof(struct mlx5_wqe),
-                                     page_size, priv->sh->numa_node);
-       if (!txq_obj->sq_buf) {
-               DRV_LOG(ERR,
-                       "Port %u Tx queue %u cannot allocate memory (SQ).",
-                       dev->data->port_id, txq_data->idx);
-               rte_errno = ENOMEM;
-               goto error;
-       }
-       /* Register allocated buffer in user space with DevX. */
-       txq_obj->sq_umem = mlx5_os_umem_reg
-                                       (priv->sh->ctx,
-                                        (void *)txq_obj->sq_buf,
-                                        wqe_n * sizeof(struct mlx5_wqe),
-                                        IBV_ACCESS_LOCAL_WRITE);
-       if (!txq_obj->sq_umem) {
-               rte_errno = errno;
-               DRV_LOG(ERR,
-                       "Port %u Tx queue %u cannot register memory (SQ).",
-                       dev->data->port_id, txq_data->idx);
-               goto error;
-       }
-       /* Allocate doorbell record for send queue. */
-       txq_obj->sq_dbrec_offset = mlx5_get_dbr(priv->sh->ctx,
-                                               &priv->dbrpgs,
-                                               &txq_obj->sq_dbrec_page);
-       if (txq_obj->sq_dbrec_offset < 0) {
-               rte_errno = errno;
-               DRV_LOG(ERR, "Failed to allocate SQ door-bell.");
-               goto error;
-       }
-       sq_attr.tis_lst_sz = 1;
-       sq_attr.tis_num = priv->sh->tis->id;
-       sq_attr.state = MLX5_SQC_STATE_RST;
-       sq_attr.cqn = txq_obj->cq_devx->id;
-       sq_attr.flush_in_error_en = 1;
-       sq_attr.allow_multi_pkt_send_wqe = !!priv->config.mps;
-       sq_attr.allow_swp = !!priv->config.swp;
-       sq_attr.min_wqe_inline_mode = priv->config.hca_attr.vport_inline_mode;
-       sq_attr.wq_attr.uar_page =
-                               mlx5_os_get_devx_uar_page_id(priv->sh->tx_uar);
-       sq_attr.wq_attr.wq_type = MLX5_WQ_TYPE_CYCLIC;
-       sq_attr.wq_attr.pd = priv->sh->pdn;
-       sq_attr.wq_attr.log_wq_stride = rte_log2_u32(MLX5_WQE_SIZE);
-       sq_attr.wq_attr.log_wq_sz = log2above(wqe_n);
-       sq_attr.wq_attr.dbr_umem_valid = 1;
-       sq_attr.wq_attr.dbr_addr = txq_obj->sq_dbrec_offset;
-       sq_attr.wq_attr.dbr_umem_id =
-                       mlx5_os_get_umem_id(txq_obj->sq_dbrec_page->umem);
-       sq_attr.wq_attr.wq_umem_valid = 1;
-       sq_attr.wq_attr.wq_umem_id = mlx5_os_get_umem_id(txq_obj->sq_umem);
-       sq_attr.wq_attr.wq_umem_offset = (uintptr_t)txq_obj->sq_buf % page_size;
        /* Create Send Queue object with DevX. */
-       txq_obj->sq_devx = mlx5_devx_cmd_create_sq(priv->sh->ctx, &sq_attr);
-       if (!txq_obj->sq_devx) {
-               rte_errno = errno;
-               DRV_LOG(ERR, "Port %u Tx queue %u SQ creation failure.",
-                       dev->data->port_id, idx);
-               goto error;
-       }
-       return wqe_n;
-error:
-       ret = rte_errno;
-       mlx5_txq_release_devx_sq_resources(txq_obj);
-       rte_errno = ret;
-       return 0;
+       return mlx5_devx_sq_create(priv->sh->ctx, &txq_obj->sq_obj, log_desc_n,
+                                  &sq_attr, priv->sh->numa_node);
 }
 #endif
 
@@ -1429,51 +1033,88 @@ mlx5_txq_devx_obj_new(struct rte_eth_dev *dev, uint16_t idx)
 #else
        struct mlx5_dev_ctx_shared *sh = priv->sh;
        struct mlx5_txq_obj *txq_obj = txq_ctrl->obj;
+       struct mlx5_devx_cq_attr cq_attr = {
+               .uar_page_id = mlx5_os_get_devx_uar_page_id(sh->tx_uar),
+       };
        void *reg_addr;
-       uint32_t cqe_n;
-       uint32_t wqe_n;
+       uint32_t cqe_n, log_desc_n;
+       uint32_t wqe_n, wqe_size;
        int ret = 0;
 
        MLX5_ASSERT(txq_data);
        MLX5_ASSERT(txq_obj);
        txq_obj->txq_ctrl = txq_ctrl;
        txq_obj->dev = dev;
-       cqe_n = mlx5_txq_create_devx_cq_resources(dev, idx);
-       if (!cqe_n) {
-               rte_errno = errno;
+       cqe_n = (1UL << txq_data->elts_n) / MLX5_TX_COMP_THRESH +
+               1 + MLX5_TX_COMP_THRESH_INLINE_DIV;
+       log_desc_n = log2above(cqe_n);
+       cqe_n = 1UL << log_desc_n;
+       if (cqe_n > UINT16_MAX) {
+               DRV_LOG(ERR, "Port %u Tx queue %u requests to many CQEs %u.",
+                       dev->data->port_id, txq_data->idx, cqe_n);
+               rte_errno = EINVAL;
+               return 0;
+       }
+       /* Create completion queue object with DevX. */
+       ret = mlx5_devx_cq_create(sh->ctx, &txq_obj->cq_obj, log_desc_n,
+                                 &cq_attr, priv->sh->numa_node);
+       if (ret) {
+               DRV_LOG(ERR, "Port %u Tx queue %u CQ creation failure.",
+                       dev->data->port_id, idx);
                goto error;
        }
-       txq_data->cqe_n = log2above(cqe_n);
-       txq_data->cqe_s = 1 << txq_data->cqe_n;
+       txq_data->cqe_n = log_desc_n;
+       txq_data->cqe_s = cqe_n;
        txq_data->cqe_m = txq_data->cqe_s - 1;
-       txq_data->cqes = (volatile struct mlx5_cqe *)txq_obj->cq_buf;
+       txq_data->cqes = txq_obj->cq_obj.cqes;
        txq_data->cq_ci = 0;
        txq_data->cq_pi = 0;
-       txq_data->cq_db = (volatile uint32_t *)(txq_obj->cq_dbrec_page->dbrs +
-                                               txq_obj->cq_dbrec_offset);
+       txq_data->cq_db = txq_obj->cq_obj.db_rec;
        *txq_data->cq_db = 0;
+       /*
+        * Adjust the amount of WQEs depending on inline settings.
+        * The number of descriptors should be enough to handle
+        * the specified number of packets. If queue is being created
+        * with Verbs the rdma-core does queue size adjustment
+        * internally in the mlx5_calc_sq_size(), we do the same
+        * for the queue being created with DevX at this point.
+        */
+       wqe_size = txq_data->tso_en ?
+                  RTE_ALIGN(txq_ctrl->max_tso_header, MLX5_WSEG_SIZE) : 0;
+       wqe_size += sizeof(struct mlx5_wqe_cseg) +
+                   sizeof(struct mlx5_wqe_eseg) +
+                   sizeof(struct mlx5_wqe_dseg);
+       if (txq_data->inlen_send)
+               wqe_size = RTE_MAX(wqe_size, sizeof(struct mlx5_wqe_cseg) +
+                                            sizeof(struct mlx5_wqe_eseg) +
+                                            RTE_ALIGN(txq_data->inlen_send +
+                                                      sizeof(uint32_t),
+                                                      MLX5_WSEG_SIZE));
+       wqe_size = RTE_ALIGN(wqe_size, MLX5_WQE_SIZE) / MLX5_WQE_SIZE;
        /* Create Send Queue object with DevX. */
-       wqe_n = mlx5_txq_create_devx_sq_resources(dev, idx);
-       if (!wqe_n) {
+       wqe_n = RTE_MIN((1UL << txq_data->elts_n) * wqe_size,
+                       (uint32_t)priv->sh->device_attr.max_qp_wr);
+       log_desc_n = log2above(wqe_n);
+       ret = mlx5_txq_create_devx_sq_resources(dev, idx, log_desc_n);
+       if (ret) {
+               DRV_LOG(ERR, "Port %u Tx queue %u SQ creation failure.",
+                       dev->data->port_id, idx);
                rte_errno = errno;
                goto error;
        }
        /* Create the Work Queue. */
-       txq_data->wqe_n = log2above(wqe_n);
+       txq_data->wqe_n = log_desc_n;
        txq_data->wqe_s = 1 << txq_data->wqe_n;
        txq_data->wqe_m = txq_data->wqe_s - 1;
-       txq_data->wqes = (struct mlx5_wqe *)txq_obj->sq_buf;
+       txq_data->wqes = (struct mlx5_wqe *)(uintptr_t)txq_obj->sq_obj.wqes;
        txq_data->wqes_end = txq_data->wqes + txq_data->wqe_s;
        txq_data->wqe_ci = 0;
        txq_data->wqe_pi = 0;
        txq_data->wqe_comp = 0;
        txq_data->wqe_thres = txq_data->wqe_s / MLX5_TX_COMP_THRESH_INLINE_DIV;
-       txq_data->qp_db = (volatile uint32_t *)
-                                       (txq_obj->sq_dbrec_page->dbrs +
-                                        txq_obj->sq_dbrec_offset +
-                                        MLX5_SND_DBR * sizeof(uint32_t));
+       txq_data->qp_db = txq_obj->sq_obj.db_rec;
        *txq_data->qp_db = 0;
-       txq_data->qp_num_8s = txq_obj->sq_devx->id << 8;
+       txq_data->qp_num_8s = txq_obj->sq_obj.sq->id << 8;
        /* Change Send Queue state to Ready-to-Send. */
        ret = mlx5_devx_modify_sq(txq_obj, MLX5_TXQ_MOD_RST2RDY, 0);
        if (ret) {