RDMA/rxe: Convert pr_warn/err to pr_debug in pyverbs
authorBob Pearson <rpearsonhpe@gmail.com>
Thu, 30 Jun 2022 19:04:19 +0000 (14:04 -0500)
committerJason Gunthorpe <jgg@nvidia.com>
Thu, 30 Jun 2022 23:45:00 +0000 (20:45 -0300)
The pyverbs test suite generates a few dmesg traces from intentional error
tests. This patch replaces those messages with pr_debug() calls which
improves the usefullness of the tests.

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

index 642b525..b1a0ab3 100644 (file)
@@ -19,16 +19,16 @@ int rxe_cq_chk_attr(struct rxe_dev *rxe, struct rxe_cq *cq,
        }
 
        if (cqe > rxe->attr.max_cqe) {
-               pr_warn("cqe(%d) > max_cqe(%d)\n",
-                       cqe, rxe->attr.max_cqe);
+               pr_debug("cqe(%d) > max_cqe(%d)\n",
+                               cqe, rxe->attr.max_cqe);
                goto err1;
        }
 
        if (cq) {
                count = queue_count(cq->queue, QUEUE_TYPE_TO_CLIENT);
                if (cqe < count) {
-                       pr_warn("cqe(%d) < current # elements in queue (%d)",
-                               cqe, count);
+                       pr_debug("cqe(%d) < current # elements in queue (%d)",
+                                       cqe, count);
                        goto err1;
                }
        }
index 1212bec..ccdfc1a 100644 (file)
@@ -450,7 +450,8 @@ 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_err("%s: no MW matches rkey %#x\n", __func__, rkey);
+                       pr_debug("%s: no MW matches rkey %#x\n",
+                                       __func__, rkey);
                        state = RESPST_ERR_RKEY_VIOLATION;
                        goto err;
                }
@@ -470,7 +471,8 @@ static enum resp_states check_rkey(struct rxe_qp *qp,
        } else {
                mr = lookup_mr(qp->pd, access, rkey, RXE_LOOKUP_REMOTE);
                if (!mr) {
-                       pr_err("%s: no MR matches rkey %#x\n", __func__, rkey);
+                       pr_debug("%s: no MR matches rkey %#x\n",
+                                       __func__, rkey);
                        state = RESPST_ERR_RKEY_VIOLATION;
                        goto err;
                }
@@ -1462,7 +1464,7 @@ int rxe_responder(void *arg)
 
                case RESPST_ERROR:
                        qp->resp.goto_error = 0;
-                       pr_warn("qp#%d moved to error state\n", qp_num(qp));
+                       pr_debug("qp#%d moved to error state\n", qp_num(qp));
                        rxe_qp_error(qp);
                        goto exit;