RDMA/ocrdma: For ERX2 irrespective of Qid, num_posted offset is 24
authorNaresh Gottumukkala <bgottumukkala@emulex.com>
Mon, 26 Aug 2013 09:57:42 +0000 (15:27 +0530)
committerRoland Dreier <roland@purestorage.com>
Tue, 3 Sep 2013 04:17:55 +0000 (21:17 -0700)
1) All RQ doorbells are handled by ERX2 and doorbell->num_posted
   offset is constant to bit offset 24 for ERX2 irrspective of Q id.

2) Fixed RESET to INIT state change (from ERR->RST->INIT->RTR case).

Signed-off-by: Naresh Gottumukkala <bgottumukkala@emulex.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
drivers/infiniband/hw/ocrdma/ocrdma_hw.c
drivers/infiniband/hw/ocrdma/ocrdma_sli.h
drivers/infiniband/hw/ocrdma/ocrdma_verbs.c
drivers/infiniband/hw/ocrdma/ocrdma_verbs.h

index 1b14ef811b361707443439dfa1c83893189c4d28..2c2991b7dae09c49b5a7bf69ee238655f558503b 100644 (file)
@@ -256,11 +256,11 @@ static int ocrdma_get_mbx_cqe_errno(u16 cqe_status)
                break;
        case OCRDMA_MBX_CQE_STATUS_INSUFFICIENT_RESOURCES:
        case OCRDMA_MBX_CQE_STATUS_QUEUE_FLUSHING:
-               err_num = -EAGAIN;
+               err_num = -EINVAL;
                break;
        case OCRDMA_MBX_CQE_STATUS_DMA_FAILED:
        default:
-               err_num = -EIO;
+               err_num = -EINVAL;
                break;
        }
        return err_num;
@@ -1654,6 +1654,14 @@ void ocrdma_flush_qp(struct ocrdma_qp *qp)
        spin_unlock_irqrestore(&qp->dev->flush_q_lock, flags);
 }
 
+static void ocrdma_init_hwq_ptr(struct ocrdma_qp *qp)
+{
+       qp->sq.head = 0;
+       qp->sq.tail = 0;
+       qp->rq.head = 0;
+       qp->rq.tail = 0;
+}
+
 int ocrdma_qp_state_change(struct ocrdma_qp *qp, enum ib_qp_state new_ib_state,
                           enum ib_qp_state *old_ib_state)
 {
@@ -1673,8 +1681,12 @@ int ocrdma_qp_state_change(struct ocrdma_qp *qp, enum ib_qp_state new_ib_state,
        }
 
 
-       if (new_state == OCRDMA_QPS_ERR)
+       if (new_state == OCRDMA_QPS_INIT) {
+               ocrdma_init_hwq_ptr(qp);
+               ocrdma_del_flush_qp(qp);
+       } else if (new_state == OCRDMA_QPS_ERR) {
                ocrdma_flush_qp(qp);
+       }
 
        qp->state = new_state;
 
@@ -2317,7 +2329,8 @@ int ocrdma_mbx_modify_srq(struct ocrdma_srq *srq, struct ib_srq_attr *srq_attr)
 {
        int status = -ENOMEM;
        struct ocrdma_modify_srq *cmd;
-       struct ocrdma_dev *dev = get_ocrdma_dev(srq->ibsrq.device);
+       struct ocrdma_pd *pd = srq->pd;
+       struct ocrdma_dev *dev = get_ocrdma_dev(pd->ibpd.device);
 
        cmd = ocrdma_init_emb_mqe(OCRDMA_CMD_CREATE_SRQ, sizeof(*cmd));
        if (!cmd)
index 6cf5a96f38c6f49b37fab5ecb3f32757a5c68b96..bfd0acb5659d38ae6a613ee9802c9c2aaad35272 100644 (file)
@@ -95,12 +95,11 @@ enum {
 
 enum {
        OCRDMA_DB_RQ_OFFSET             = 0xE0,
-       OCRDMA_DB_GEN2_RQ1_OFFSET       = 0x100,
-       OCRDMA_DB_GEN2_RQ2_OFFSET       = 0xC0,
+       OCRDMA_DB_GEN2_RQ_OFFSET        = 0x100,
        OCRDMA_DB_SQ_OFFSET             = 0x60,
        OCRDMA_DB_GEN2_SQ_OFFSET        = 0x1C0,
        OCRDMA_DB_SRQ_OFFSET            = OCRDMA_DB_RQ_OFFSET,
-       OCRDMA_DB_GEN2_SRQ_OFFSET       = OCRDMA_DB_GEN2_RQ1_OFFSET,
+       OCRDMA_DB_GEN2_SRQ_OFFSET       = OCRDMA_DB_GEN2_RQ_OFFSET,
        OCRDMA_DB_CQ_OFFSET             = 0x120,
        OCRDMA_DB_EQ_OFFSET             = OCRDMA_DB_CQ_OFFSET,
        OCRDMA_DB_MQ_OFFSET             = 0x140
index 7698572b79fba011a9f17011308f6e35b305e32b..9e1d8c6bde52b51e7610540078b2f02cf49cb5e3 100644 (file)
@@ -930,9 +930,8 @@ static int ocrdma_copy_qp_uresp(struct ocrdma_qp *qp,
        uresp.db_page_size = dev->nic_info.db_page_size;
        if (dev->nic_info.dev_family == OCRDMA_GEN2_FAMILY) {
                uresp.db_sq_offset = OCRDMA_DB_GEN2_SQ_OFFSET;
-               uresp.db_rq_offset = ((qp->id & 0xFFFF) < 128) ?
-                       OCRDMA_DB_GEN2_RQ1_OFFSET : OCRDMA_DB_GEN2_RQ2_OFFSET;
-               uresp.db_shift = (qp->id < 128) ? 24 : 16;
+               uresp.db_rq_offset = OCRDMA_DB_GEN2_RQ_OFFSET;
+               uresp.db_shift = 24;
        } else {
                uresp.db_sq_offset = OCRDMA_DB_SQ_OFFSET;
                uresp.db_rq_offset = OCRDMA_DB_RQ_OFFSET;
@@ -975,8 +974,7 @@ static void ocrdma_set_qp_db(struct ocrdma_dev *dev, struct ocrdma_qp *qp,
                        OCRDMA_DB_GEN2_SQ_OFFSET;
                qp->rq_db = dev->nic_info.db +
                        (pd->id * dev->nic_info.db_page_size) +
-                       ((qp->id < 128) ?
-                       OCRDMA_DB_GEN2_RQ1_OFFSET : OCRDMA_DB_GEN2_RQ2_OFFSET);
+                       OCRDMA_DB_GEN2_RQ_OFFSET;
        } else {
                qp->sq_db = dev->nic_info.db +
                        (pd->id * dev->nic_info.db_page_size) +
@@ -1399,7 +1397,7 @@ skip_cqe:
        spin_unlock_irqrestore(&cq->cq_lock, cq_flags);
 }
 
-static void ocrdma_del_flush_qp(struct ocrdma_qp *qp)
+void ocrdma_del_flush_qp(struct ocrdma_qp *qp)
 {
        int found = false;
        unsigned long flags;
@@ -1496,7 +1494,7 @@ static int ocrdma_copy_srq_uresp(struct ocrdma_dev *dev, struct ocrdma_srq *srq,
        uresp.db_page_size = dev->nic_info.db_page_size;
        uresp.num_rqe_allocated = srq->rq.max_cnt;
        if (dev->nic_info.dev_family == OCRDMA_GEN2_FAMILY) {
-               uresp.db_rq_offset = OCRDMA_DB_GEN2_RQ1_OFFSET;
+               uresp.db_rq_offset = OCRDMA_DB_GEN2_RQ_OFFSET;
                uresp.db_shift = 24;
        } else {
                uresp.db_rq_offset = OCRDMA_DB_RQ_OFFSET;
index 7f3056731af1f0fbfec4df09f1a72781ffc1e959..b8f7853fd36ce7a61e53004105c2b9082af14ebb 100644 (file)
@@ -72,6 +72,7 @@ int ocrdma_query_qp(struct ib_qp *,
                    struct ib_qp_attr *qp_attr,
                    int qp_attr_mask, struct ib_qp_init_attr *);
 int ocrdma_destroy_qp(struct ib_qp *);
+void ocrdma_del_flush_qp(struct ocrdma_qp *qp);
 
 struct ib_srq *ocrdma_create_srq(struct ib_pd *, struct ib_srq_init_attr *,
                                 struct ib_udata *);