RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_resp.c
authorBob Pearson <rpearsonhpe@gmail.com>
Thu, 3 Nov 2022 17:10:07 +0000 (12:10 -0500)
committerJason Gunthorpe <jgg@nvidia.com>
Thu, 10 Nov 2022 19:33:05 +0000 (15:33 -0400)
Replace calls to pr_xxx() in rxe_resp.c with rxe_dbg_xxx().

Link: https://lore.kernel.org/r/20221103171013.20659-10-rpearsonhpe@gmail.com
Signed-off-by: Bob Pearson <rpearsonhpe@gmail.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
drivers/infiniband/sw/rxe/rxe_resp.c

index 382d205..6761bcd 100644 (file)
@@ -317,7 +317,7 @@ static enum resp_states get_srq_wqe(struct rxe_qp *qp)
        /* don't trust user space data */
        if (unlikely(wqe->dma.num_sge > srq->rq.max_sge)) {
                spin_unlock_irqrestore(&srq->rq.consumer_lock, flags);
-               pr_warn("%s: invalid num_sge in SRQ entry\n", __func__);
+               rxe_dbg_qp(qp, "invalid num_sge in SRQ entry\n");
                return RESPST_ERR_MALFORMED_WQE;
        }
        size = sizeof(*wqe) + wqe->dma.num_sge*sizeof(struct rxe_sge);
@@ -473,15 +473,14 @@ static enum resp_states check_rkey(struct rxe_qp *qp,
        if (rkey_is_mw(rkey)) {
                mw = rxe_lookup_mw(qp, access, rkey);
                if (!mw) {
-                       pr_debug("%s: no MW matches rkey %#x\n",
-                                       __func__, rkey);
+                       rxe_dbg_qp(qp, "no MW matches rkey %#x\n", rkey);
                        state = RESPST_ERR_RKEY_VIOLATION;
                        goto err;
                }
 
                mr = mw->mr;
                if (!mr) {
-                       pr_err("%s: MW doesn't have an MR\n", __func__);
+                       rxe_dbg_qp(qp, "MW doesn't have an MR\n");
                        state = RESPST_ERR_RKEY_VIOLATION;
                        goto err;
                }
@@ -494,8 +493,7 @@ static enum resp_states check_rkey(struct rxe_qp *qp,
        } else {
                mr = lookup_mr(qp->pd, access, rkey, RXE_LOOKUP_REMOTE);
                if (!mr) {
-                       pr_debug("%s: no MR matches rkey %#x\n",
-                                       __func__, rkey);
+                       rxe_dbg_qp(qp, "no MR matches rkey %#x\n", rkey);
                        state = RESPST_ERR_RKEY_VIOLATION;
                        goto err;
                }
@@ -1064,7 +1062,7 @@ static int send_common_ack(struct rxe_qp *qp, u8 syndrome, u32 psn,
 
        err = rxe_xmit_packet(qp, &ack_pkt, skb);
        if (err)
-               pr_err_ratelimited("Failed sending %s\n", msg);
+               rxe_dbg_qp(qp, "Failed sending %s\n", msg);
 
        return err;
 }
@@ -1310,8 +1308,7 @@ int rxe_responder(void *arg)
        }
 
        while (1) {
-               pr_debug("qp#%d state = %s\n", qp_num(qp),
-                        resp_state_name[state]);
+               rxe_dbg_qp(qp, "state = %s\n", resp_state_name[state]);
                switch (state) {
                case RESPST_GET_REQ:
                        state = get_req(qp, &pkt);
@@ -1468,7 +1465,7 @@ int rxe_responder(void *arg)
 
                case RESPST_ERROR:
                        qp->resp.goto_error = 0;
-                       pr_debug("qp#%d moved to error state\n", qp_num(qp));
+                       rxe_dbg_qp(qp, "moved to error state\n");
                        rxe_qp_error(qp);
                        goto exit;