Commit cae3fa54 authored by Bob Pearson's avatar Bob Pearson Committed by Jason Gunthorpe
Browse files

RDMA/rxe: Convert pr_warn/err to pr_debug in pyverbs 

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: default avatarBob Pearson <rpearsonhpe@gmail.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@nvidia.com>
parent 7cb33d1b

drivers/infiniband/sw/rxe/rxe_cq.c

+4 −4
Original line number Diff line number Diff line
@@ -19,7 +19,7 @@ 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",

		pr_debug("cqe(%d) > max_cqe(%d)\n",

				cqe, rxe->attr.max_cqe);

		goto err1;

	}
@@ -27,7 +27,7 @@ int rxe_cq_chk_attr(struct rxe_dev *rxe, struct rxe_cq *cq, 
	if (cq) {

		count = queue_count(cq->queue, QUEUE_TYPE_TO_CLIENT);

		if (cqe < count) {

			pr_warn("cqe(%d) < current # elements in queue (%d)",

			pr_debug("cqe(%d) < current # elements in queue (%d)",

					cqe, count);

			goto err1;

		}

drivers/infiniband/sw/rxe/rxe_resp.c

+5 −3
Original line number Diff line number Diff line
@@ -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;