net/mlx4: add Rx bypassing Verbs
[dpdk.git] / drivers / net / mlx4 / mlx4_rxq.c
index 30b0654..39c83bc 100644 (file)
@@ -51,6 +51,7 @@
 #pragma GCC diagnostic error "-Wpedantic"
 #endif
 
+#include <rte_byteorder.h>
 #include <rte_common.h>
 #include <rte_errno.h>
 #include <rte_ethdev.h>
 #include "mlx4_rxtx.h"
 #include "mlx4_utils.h"
 
+/**
+ * Historical RSS hash key.
+ *
+ * This used to be the default for mlx4 in Linux before v3.19 switched to
+ * generating random hash keys through netdev_rss_key_fill().
+ *
+ * It is used in this PMD for consistency with past DPDK releases but can
+ * now be overridden through user configuration.
+ *
+ * Note: this is not const to work around API quirks.
+ */
+uint8_t
+mlx4_rss_hash_key_default[MLX4_RSS_HASH_KEY_SIZE] = {
+       0x2c, 0xc6, 0x81, 0xd1,
+       0x5b, 0xdb, 0xf4, 0xf7,
+       0xfc, 0xa2, 0x83, 0x19,
+       0xdb, 0x1a, 0x3e, 0x94,
+       0x6b, 0x9e, 0x38, 0xd9,
+       0x2c, 0x9c, 0x03, 0xd1,
+       0xad, 0x99, 0x44, 0xa7,
+       0xd9, 0x56, 0x3d, 0x59,
+       0x06, 0x3c, 0x25, 0xf3,
+       0xfc, 0x1f, 0xdc, 0x2a,
+};
+
+/**
+ * Obtain a RSS context with specified properties.
+ *
+ * Used when creating a flow rule targeting one or several Rx queues.
+ *
+ * If a matching RSS context already exists, it is returned with its
+ * reference count incremented.
+ *
+ * @param priv
+ *   Pointer to private structure.
+ * @param fields
+ *   Fields for RSS processing (Verbs format).
+ * @param[in] key
+ *   Hash key to use (whose size is exactly MLX4_RSS_HASH_KEY_SIZE).
+ * @param queues
+ *   Number of target queues.
+ * @param[in] queue_id
+ *   Target queues.
+ *
+ * @return
+ *   Pointer to RSS context on success, NULL otherwise and rte_errno is set.
+ */
+struct mlx4_rss *
+mlx4_rss_get(struct priv *priv, uint64_t fields,
+            uint8_t key[MLX4_RSS_HASH_KEY_SIZE],
+            uint16_t queues, const uint16_t queue_id[])
+{
+       struct mlx4_rss *rss;
+       size_t queue_id_size = sizeof(queue_id[0]) * queues;
+
+       LIST_FOREACH(rss, &priv->rss, next)
+               if (fields == rss->fields &&
+                   queues == rss->queues &&
+                   !memcmp(key, rss->key, MLX4_RSS_HASH_KEY_SIZE) &&
+                   !memcmp(queue_id, rss->queue_id, queue_id_size)) {
+                       ++rss->refcnt;
+                       return rss;
+               }
+       rss = rte_malloc(__func__, offsetof(struct mlx4_rss, queue_id) +
+                        queue_id_size, 0);
+       if (!rss)
+               goto error;
+       *rss = (struct mlx4_rss){
+               .priv = priv,
+               .refcnt = 1,
+               .usecnt = 0,
+               .qp = NULL,
+               .ind = NULL,
+               .fields = fields,
+               .queues = queues,
+       };
+       memcpy(rss->key, key, MLX4_RSS_HASH_KEY_SIZE);
+       memcpy(rss->queue_id, queue_id, queue_id_size);
+       LIST_INSERT_HEAD(&priv->rss, rss, next);
+       return rss;
+error:
+       rte_errno = ENOMEM;
+       return NULL;
+}
+
+/**
+ * Release a RSS context instance.
+ *
+ * Used when destroying a flow rule targeting one or several Rx queues.
+ *
+ * This function decrements the reference count of the context and destroys
+ * it after reaching 0. The context must have no users at this point; all
+ * prior calls to mlx4_rss_attach() must have been followed by matching
+ * calls to mlx4_rss_detach().
+ *
+ * @param rss
+ *   RSS context to release.
+ */
+void mlx4_rss_put(struct mlx4_rss *rss)
+{
+       assert(rss->refcnt);
+       if (--rss->refcnt)
+               return;
+       assert(!rss->usecnt);
+       assert(!rss->qp);
+       assert(!rss->ind);
+       LIST_REMOVE(rss, next);
+       rte_free(rss);
+}
+
+/**
+ * Attach a user to a RSS context instance.
+ *
+ * Used when the RSS QP and indirection table objects must be instantiated,
+ * that is, when a flow rule must be enabled.
+ *
+ * This function increments the usage count of the context.
+ *
+ * @param rss
+ *   RSS context to attach to.
+ */
+int mlx4_rss_attach(struct mlx4_rss *rss)
+{
+       assert(rss->refcnt);
+       if (rss->usecnt++) {
+               assert(rss->qp);
+               assert(rss->ind);
+               return 0;
+       }
+
+       struct ibv_wq *ind_tbl[rss->queues];
+       struct priv *priv = rss->priv;
+       const char *msg;
+       unsigned int i;
+       int ret;
+
+       if (!rte_is_power_of_2(RTE_DIM(ind_tbl))) {
+               msg = "number of RSS queues must be a power of two";
+               goto error;
+       }
+       for (i = 0; i != RTE_DIM(ind_tbl); ++i) {
+               uint16_t id = rss->queue_id[i];
+               struct rxq *rxq = NULL;
+
+               if (id < priv->dev->data->nb_rx_queues)
+                       rxq = priv->dev->data->rx_queues[id];
+               if (!rxq) {
+                       msg = "RSS target queue is not configured";
+                       goto error;
+               }
+               ind_tbl[i] = rxq->wq;
+       }
+       rss->ind = ibv_create_rwq_ind_table
+               (priv->ctx,
+                &(struct ibv_rwq_ind_table_init_attr){
+                       .log_ind_tbl_size = rte_log2_u32(RTE_DIM(ind_tbl)),
+                       .ind_tbl = ind_tbl,
+                       .comp_mask = 0,
+                });
+       if (!rss->ind) {
+               msg = "RSS indirection table creation failure";
+               goto error;
+       }
+       rss->qp = ibv_create_qp_ex
+               (priv->ctx,
+                &(struct ibv_qp_init_attr_ex){
+                       .comp_mask = (IBV_QP_INIT_ATTR_PD |
+                                     IBV_QP_INIT_ATTR_RX_HASH |
+                                     IBV_QP_INIT_ATTR_IND_TABLE),
+                       .qp_type = IBV_QPT_RAW_PACKET,
+                       .pd = priv->pd,
+                       .rwq_ind_tbl = rss->ind,
+                       .rx_hash_conf = {
+                               .rx_hash_function = IBV_RX_HASH_FUNC_TOEPLITZ,
+                               .rx_hash_key_len = MLX4_RSS_HASH_KEY_SIZE,
+                               .rx_hash_key = rss->key,
+                               .rx_hash_fields_mask = rss->fields,
+                       },
+                });
+       if (!rss->qp) {
+               msg = "RSS hash QP creation failure";
+               goto error;
+       }
+       ret = ibv_modify_qp
+               (rss->qp,
+                &(struct ibv_qp_attr){
+                       .qp_state = IBV_QPS_INIT,
+                       .port_num = priv->port,
+                },
+                IBV_QP_STATE | IBV_QP_PORT);
+       if (ret) {
+               msg = "failed to switch RSS hash QP to INIT state";
+               goto error;
+       }
+       ret = ibv_modify_qp
+               (rss->qp,
+                &(struct ibv_qp_attr){
+                       .qp_state = IBV_QPS_RTR,
+                },
+                IBV_QP_STATE);
+       if (ret) {
+               msg = "failed to switch RSS hash QP to RTR state";
+               goto error;
+       }
+       return 0;
+error:
+       ERROR("mlx4: %s", msg);
+       --rss->usecnt;
+       rte_errno = EINVAL;
+       return -rte_errno;
+}
+
+/**
+ * Detach a user from a RSS context instance.
+ *
+ * Used when disabling (not destroying) a flow rule.
+ *
+ * This function decrements the usage count of the context and destroys
+ * usage resources after reaching 0.
+ *
+ * @param rss
+ *   RSS context to detach from.
+ */
+void mlx4_rss_detach(struct mlx4_rss *rss)
+{
+       assert(rss->refcnt);
+       assert(rss->qp);
+       assert(rss->ind);
+       if (--rss->usecnt)
+               return;
+       claim_zero(ibv_destroy_qp(rss->qp));
+       rss->qp = NULL;
+       claim_zero(ibv_destroy_rwq_ind_table(rss->ind));
+       rss->ind = NULL;
+}
+
 /**
  * Allocate Rx queue elements.
  *
  * @param rxq
  *   Pointer to Rx queue structure.
- * @param elts_n
- *   Number of elements to allocate.
  *
  * @return
  *   0 on success, negative errno value otherwise and rte_errno is set.
  */
 static int
-mlx4_rxq_alloc_elts(struct rxq *rxq, unsigned int elts_n)
+mlx4_rxq_alloc_elts(struct rxq *rxq)
 {
+       const uint32_t elts_n = 1 << rxq->elts_n;
+       const uint32_t sges_n = 1 << rxq->sges_n;
+       struct rte_mbuf *(*elts)[elts_n] = rxq->elts;
        unsigned int i;
-       struct rxq_elt (*elts)[elts_n] =
-               rte_calloc_socket("RXQ elements", 1, sizeof(*elts), 0,
-                                 rxq->socket);
 
-       if (elts == NULL) {
-               rte_errno = ENOMEM;
-               ERROR("%p: can't allocate packets array", (void *)rxq);
-               goto error;
-       }
-       /* For each WR (packet). */
-       for (i = 0; (i != elts_n); ++i) {
-               struct rxq_elt *elt = &(*elts)[i];
-               struct ibv_recv_wr *wr = &elt->wr;
-               struct ibv_sge *sge = &(*elts)[i].sge;
+       assert(rte_is_power_of_2(elts_n));
+       for (i = 0; i != RTE_DIM(*elts); ++i) {
+               volatile struct mlx4_wqe_data_seg *scat = &(*rxq->wqes)[i];
                struct rte_mbuf *buf = rte_pktmbuf_alloc(rxq->mp);
 
                if (buf == NULL) {
+                       while (i--) {
+                               rte_pktmbuf_free_seg((*elts)[i]);
+                               (*elts)[i] = NULL;
+                       }
                        rte_errno = ENOMEM;
-                       ERROR("%p: empty mbuf pool", (void *)rxq);
-                       goto error;
+                       return -rte_errno;
                }
-               elt->buf = buf;
-               wr->next = &(*elts)[(i + 1)].wr;
-               wr->sg_list = sge;
-               wr->num_sge = 1;
                /* Headroom is reserved by rte_pktmbuf_alloc(). */
                assert(buf->data_off == RTE_PKTMBUF_HEADROOM);
                /* Buffer is supposed to be empty. */
                assert(rte_pktmbuf_data_len(buf) == 0);
                assert(rte_pktmbuf_pkt_len(buf) == 0);
-               /* sge->addr must be able to store a pointer. */
-               assert(sizeof(sge->addr) >= sizeof(uintptr_t));
-               /* SGE keeps its headroom. */
-               sge->addr = (uintptr_t)
-                       ((uint8_t *)buf->buf_addr + RTE_PKTMBUF_HEADROOM);
-               sge->length = (buf->buf_len - RTE_PKTMBUF_HEADROOM);
-               sge->lkey = rxq->mr->lkey;
-               /* Redundant check for tailroom. */
-               assert(sge->length == rte_pktmbuf_tailroom(buf));
+               /* Only the first segment keeps headroom. */
+               if (i % sges_n)
+                       buf->data_off = 0;
+               buf->port = rxq->port_id;
+               buf->data_len = rte_pktmbuf_tailroom(buf);
+               buf->pkt_len = rte_pktmbuf_tailroom(buf);
+               buf->nb_segs = 1;
+               *scat = (struct mlx4_wqe_data_seg){
+                       .addr = rte_cpu_to_be_64(rte_pktmbuf_mtod(buf,
+                                                                 uintptr_t)),
+                       .byte_count = rte_cpu_to_be_32(buf->data_len),
+                       .lkey = rte_cpu_to_be_32(rxq->mr->lkey),
+               };
+               (*elts)[i] = buf;
        }
-       /* The last WR pointer must be NULL. */
-       (*elts)[(i - 1)].wr.next = NULL;
-       DEBUG("%p: allocated and configured %u single-segment WRs",
-             (void *)rxq, elts_n);
-       rxq->elts_n = elts_n;
-       rxq->elts_head = 0;
-       rxq->elts = elts;
+       DEBUG("%p: allocated and configured %u segments (max %u packets)",
+             (void *)rxq, elts_n, elts_n / sges_n);
        return 0;
-error:
-       if (elts != NULL) {
-               for (i = 0; (i != RTE_DIM(*elts)); ++i)
-                       rte_pktmbuf_free_seg((*elts)[i].buf);
-               rte_free(elts);
-       }
-       DEBUG("%p: failed, freed everything", (void *)rxq);
-       assert(rte_errno > 0);
-       return -rte_errno;
 }
 
 /**
@@ -148,17 +366,15 @@ static void
 mlx4_rxq_free_elts(struct rxq *rxq)
 {
        unsigned int i;
-       unsigned int elts_n = rxq->elts_n;
-       struct rxq_elt (*elts)[elts_n] = rxq->elts;
+       struct rte_mbuf *(*elts)[1 << rxq->elts_n] = rxq->elts;
 
-       DEBUG("%p: freeing WRs", (void *)rxq);
-       rxq->elts_n = 0;
-       rxq->elts = NULL;
-       if (elts == NULL)
-               return;
-       for (i = 0; (i != RTE_DIM(*elts)); ++i)
-               rte_pktmbuf_free_seg((*elts)[i].buf);
-       rte_free(elts);
+       DEBUG("%p: freeing Rx queue elements", (void *)rxq);
+       for (i = 0; (i != RTE_DIM(*elts)); ++i) {
+               if (!(*elts)[i])
+                       continue;
+               rte_pktmbuf_free_seg((*elts)[i]);
+               (*elts)[i] = NULL;
+       }
 }
 
 /**
@@ -186,9 +402,25 @@ mlx4_rx_queue_setup(struct rte_eth_dev *dev, uint16_t idx, uint16_t desc,
                    struct rte_mempool *mp)
 {
        struct priv *priv = dev->data->dev_private;
+       struct mlx4dv_obj mlxdv;
+       struct mlx4dv_rwq dv_rwq;
+       struct mlx4dv_cq dv_cq;
        uint32_t mb_len = rte_pktmbuf_data_room_size(mp);
+       struct rte_mbuf *(*elts)[rte_align32pow2(desc)];
        struct rte_flow_error error;
        struct rxq *rxq;
+       struct mlx4_malloc_vec vec[] = {
+               {
+                       .align = RTE_CACHE_LINE_SIZE,
+                       .size = sizeof(*rxq),
+                       .addr = (void **)&rxq,
+               },
+               {
+                       .align = RTE_CACHE_LINE_SIZE,
+                       .size = sizeof(*elts),
+                       .addr = (void **)&elts,
+               },
+       };
        int ret;
 
        (void)conf; /* Thresholds configuration (ignored). */
@@ -212,10 +444,15 @@ mlx4_rx_queue_setup(struct rte_eth_dev *dev, uint16_t idx, uint16_t desc,
                ERROR("%p: invalid number of Rx descriptors", (void *)dev);
                return -rte_errno;
        }
+       if (desc != RTE_DIM(*elts)) {
+               desc = RTE_DIM(*elts);
+               WARN("%p: increased number of descriptors in Rx queue %u"
+                    " to the next power of two (%u)",
+                    (void *)dev, idx, desc);
+       }
        /* Allocate and initialize Rx queue. */
-       rxq = rte_calloc_socket("RXQ", 1, sizeof(*rxq), 0, socket);
+       mlx4_zmallocv_socket("RXQ", vec, RTE_DIM(vec), socket);
        if (!rxq) {
-               rte_errno = ENOMEM;
                ERROR("%p: unable to allocate queue index %u",
                      (void *)dev, idx);
                return -rte_errno;
@@ -224,6 +461,9 @@ mlx4_rx_queue_setup(struct rte_eth_dev *dev, uint16_t idx, uint16_t desc,
                .priv = priv,
                .mp = mp,
                .port_id = dev->data->port_id,
+               .sges_n = 0,
+               .elts_n = rte_log2_u32(desc),
+               .elts = elts,
                .stats.idx = idx,
                .socket = socket,
        };
@@ -233,9 +473,29 @@ mlx4_rx_queue_setup(struct rte_eth_dev *dev, uint16_t idx, uint16_t desc,
            (mb_len - RTE_PKTMBUF_HEADROOM)) {
                ;
        } else if (dev->data->dev_conf.rxmode.enable_scatter) {
-               WARN("%p: scattered mode has been requested but is"
-                    " not supported, this may lead to packet loss",
-                    (void *)dev);
+               uint32_t size =
+                       RTE_PKTMBUF_HEADROOM +
+                       dev->data->dev_conf.rxmode.max_rx_pkt_len;
+               uint32_t sges_n;
+
+               /*
+                * Determine the number of SGEs needed for a full packet
+                * and round it to the next power of two.
+                */
+               sges_n = rte_log2_u32((size / mb_len) + !!(size % mb_len));
+               rxq->sges_n = sges_n;
+               /* Make sure sges_n did not overflow. */
+               size = mb_len * (1 << rxq->sges_n);
+               size -= RTE_PKTMBUF_HEADROOM;
+               if (size < dev->data->dev_conf.rxmode.max_rx_pkt_len) {
+                       rte_errno = EOVERFLOW;
+                       ERROR("%p: too many SGEs (%u) needed to handle"
+                             " requested maximum packet size %u",
+                             (void *)dev,
+                             1 << sges_n,
+                             dev->data->dev_conf.rxmode.max_rx_pkt_len);
+                       goto error;
+               }
        } else {
                WARN("%p: the requested maximum Rx packet size (%u) is"
                     " larger than a single mbuf (%u) and scattered"
@@ -244,6 +504,17 @@ mlx4_rx_queue_setup(struct rte_eth_dev *dev, uint16_t idx, uint16_t desc,
                     dev->data->dev_conf.rxmode.max_rx_pkt_len,
                     mb_len - RTE_PKTMBUF_HEADROOM);
        }
+       DEBUG("%p: maximum number of segments per packet: %u",
+             (void *)dev, 1 << rxq->sges_n);
+       if (desc % (1 << rxq->sges_n)) {
+               rte_errno = EINVAL;
+               ERROR("%p: number of Rx queue descriptors (%u) is not a"
+                     " multiple of maximum segments per packet (%u)",
+                     (void *)dev,
+                     desc,
+                     1 << rxq->sges_n);
+               goto error;
+       }
        /* Use the entire Rx mempool as the memory region. */
        rxq->mr = mlx4_mp2mr(priv->pd, mp);
        if (!rxq->mr) {
@@ -268,69 +539,64 @@ mlx4_rx_queue_setup(struct rte_eth_dev *dev, uint16_t idx, uint16_t desc,
                        goto error;
                }
        }
-       rxq->cq = ibv_create_cq(priv->ctx, desc, NULL, rxq->channel, 0);
+       rxq->cq = ibv_create_cq(priv->ctx, desc >> rxq->sges_n, NULL,
+                               rxq->channel, 0);
        if (!rxq->cq) {
                rte_errno = ENOMEM;
                ERROR("%p: CQ creation failure: %s",
                      (void *)dev, strerror(rte_errno));
                goto error;
        }
-       rxq->qp = ibv_create_qp
-               (priv->pd,
-                &(struct ibv_qp_init_attr){
-                       .send_cq = rxq->cq,
-                       .recv_cq = rxq->cq,
-                       .cap = {
-                               .max_recv_wr =
-                                       RTE_MIN(priv->device_attr.max_qp_wr,
-                                               desc),
-                               .max_recv_sge = 1,
-                       },
-                       .qp_type = IBV_QPT_RAW_PACKET,
+       rxq->wq = ibv_create_wq
+               (priv->ctx,
+                &(struct ibv_wq_init_attr){
+                       .wq_type = IBV_WQT_RQ,
+                       .max_wr = desc >> rxq->sges_n,
+                       .max_sge = 1 << rxq->sges_n,
+                       .pd = priv->pd,
+                       .cq = rxq->cq,
                 });
-       if (!rxq->qp) {
+       if (!rxq->wq) {
                rte_errno = errno ? errno : EINVAL;
-               ERROR("%p: QP creation failure: %s",
+               ERROR("%p: WQ creation failure: %s",
                      (void *)dev, strerror(rte_errno));
                goto error;
        }
-       ret = ibv_modify_qp
-               (rxq->qp,
-                &(struct ibv_qp_attr){
-                       .qp_state = IBV_QPS_INIT,
-                       .port_num = priv->port,
-                },
-                IBV_QP_STATE | IBV_QP_PORT);
+       ret = ibv_modify_wq
+               (rxq->wq,
+                &(struct ibv_wq_attr){
+                       .attr_mask = IBV_WQ_ATTR_STATE,
+                       .wq_state = IBV_WQS_RDY,
+                });
        if (ret) {
                rte_errno = ret;
-               ERROR("%p: QP state to IBV_QPS_INIT failed: %s",
+               ERROR("%p: WQ state to IBV_WPS_RDY failed: %s",
                      (void *)dev, strerror(rte_errno));
                goto error;
        }
-       ret = mlx4_rxq_alloc_elts(rxq, desc);
+       /* Retrieve device queue information. */
+       mlxdv.cq.in = rxq->cq;
+       mlxdv.cq.out = &dv_cq;
+       mlxdv.rwq.in = rxq->wq;
+       mlxdv.rwq.out = &dv_rwq;
+       ret = mlx4dv_init_obj(&mlxdv, MLX4DV_OBJ_RWQ | MLX4DV_OBJ_CQ);
        if (ret) {
-               ERROR("%p: RXQ allocation failed: %s",
-                     (void *)dev, strerror(rte_errno));
-               goto error;
-       }
-       ret = ibv_post_recv(rxq->qp, &(*rxq->elts)[0].wr,
-                           &(struct ibv_recv_wr *){ NULL });
-       if (ret) {
-               rte_errno = ret;
-               ERROR("%p: ibv_post_recv() failed: %s",
-                     (void *)dev,
-                     strerror(rte_errno));
+               rte_errno = EINVAL;
+               ERROR("%p: failed to obtain device information", (void *)dev);
                goto error;
        }
-       ret = ibv_modify_qp
-               (rxq->qp,
-                &(struct ibv_qp_attr){
-                       .qp_state = IBV_QPS_RTR,
-                },
-                IBV_QP_STATE);
+       rxq->wqes =
+               (volatile struct mlx4_wqe_data_seg (*)[])
+               ((uintptr_t)dv_rwq.buf.buf + dv_rwq.rq.offset);
+       rxq->rq_db = dv_rwq.rdb;
+       rxq->rq_ci = 0;
+       rxq->mcq.buf = dv_cq.buf.buf;
+       rxq->mcq.cqe_cnt = dv_cq.cqe_cnt;
+       rxq->mcq.set_ci_db = dv_cq.set_ci_db;
+       rxq->mcq.cqe_64 = (dv_cq.cqe_size & 64) ? 1 : 0;
+       ret = mlx4_rxq_alloc_elts(rxq);
        if (ret) {
-               rte_errno = ret;
-               ERROR("%p: QP state to IBV_QPS_RTR failed: %s",
+               ERROR("%p: RXQ allocation failed: %s",
                      (void *)dev, strerror(rte_errno));
                goto error;
        }
@@ -338,8 +604,13 @@ mlx4_rx_queue_setup(struct rte_eth_dev *dev, uint16_t idx, uint16_t desc,
        dev->data->rx_queues[idx] = rxq;
        /* Enable associated flows. */
        ret = mlx4_flow_sync(priv, &error);
-       if (!ret)
+       if (!ret) {
+               /* Update doorbell counter. */
+               rxq->rq_ci = desc >> rxq->sges_n;
+               rte_wmb();
+               *rxq->rq_db = rte_cpu_to_be_32(rxq->rq_ci);
                return 0;
+       }
        ERROR("cannot re-attach flow rules to queue %u"
              " (code %d, \"%s\"), flow error type %d, cause %p, message: %s",
              idx, -ret, strerror(-ret), error.type, error.cause,
@@ -378,8 +649,8 @@ mlx4_rx_queue_release(void *dpdk_rxq)
                }
        mlx4_flow_sync(priv, NULL);
        mlx4_rxq_free_elts(rxq);
-       if (rxq->qp)
-               claim_zero(ibv_destroy_qp(rxq->qp));
+       if (rxq->wq)
+               claim_zero(ibv_destroy_wq(rxq->wq));
        if (rxq->cq)
                claim_zero(ibv_destroy_cq(rxq->cq));
        if (rxq->channel)