RDS: IB: suppress log prints for FLUSH_ERR/RETRY_EXC

Flush errors are normal while draining the QP and retry exceeded
errors are normal for RC connections with finite transport retry.
No need so flood the log file. RDS in-memory trace already logs it.

Orabug: 24663803

Signed-off-by: Santosh Shilimkar <[email protected]>

Reviewed-by: Mukesh Kacker <[email protected]>

Tested-by: Michael Nowak <[email protected]>
Tested-by: Rafael Alejandro Peralez <[email protected]>
Tested-by: Liwen Huang <[email protected]>
Tested-by: Hong Liu <[email protected]>

Orabug: 27364391

(cherry picked from commit 1695087)
cherry-pick-repo=linux-uek.git

Signed-off-by: Gerd Rausch <[email protected]>
Signed-off-by: Somasundaram Krishnasamy <[email protected]>

Author: SantoshShilimkar

net/rds/ib_recv.c

@@ -1336,9 +1336,11 @@ void rds_ib_recv_cqe_handler(struct rds_ib_connection *ic,
 	} else {
 		/* We expect errors as the qp is drained during shutdown */
 		if (rds_conn_up(conn) || rds_conn_connecting(conn)) {
-			pr_warn("RDS/IB: recv completion <%pI4,%pI4,%d> had status %u vendor_err 0x%x, disconnecting and reconnecting\n",
-				&conn->c_laddr, &conn->c_faddr, conn->c_tos,
-				wc->status, wc->vendor_err);
+			/* Flush errors are normal while draining the QP */
+			if (wc->status != IB_WC_WR_FLUSH_ERR)
+				pr_warn("RDS/IB: recv completion <%pI4,%pI4,%d> had status %u vendor_err 0x%x, disconnecting and reconnecting\n",
+					&conn->c_laddr, &conn->c_faddr, conn->c_tos,
+					wc->status, wc->vendor_err);
 			rds_conn_drop(conn, DR_IB_RECV_COMP_ERR);
 			rds_rtd(RDS_RTD_ERR, "status %u => %s\n", wc->status,
 				rds_ib_wc_status_str(wc->status));

net/rds/ib_send.c

@@ -341,9 +341,12 @@ void rds_ib_send_cqe_handler(struct rds_ib_connection *ic, struct ib_wc *wc)
 
 	/* We expect errors as the qp is drained during shutdown */
 	if (wc->status != IB_WC_SUCCESS && rds_conn_up(conn)) {
-		pr_warn("RDS/IB: send completion <%pI4,%pI4,%d> status %u vendor_err 0x%x, disconnecting and reconnecting\n",
-			&conn->c_laddr, &conn->c_faddr, conn->c_tos,
-			wc->status, wc->vendor_err);
+		/* Flush errors are normal while draining the QP */
+		if (!(wc->status == IB_WC_WR_FLUSH_ERR ||
+		    wc->status == IB_WC_RETRY_EXC_ERR))
+			pr_warn("RDS/IB: send completion <%pI4,%pI4,%d> status %u vendor_err 0x%x, disconnecting and reconnecting\n",
+				&conn->c_laddr, &conn->c_faddr, conn->c_tos,
+				wc->status, wc->vendor_err);
 		rds_conn_drop(conn, DR_IB_SEND_COMP_ERR);
 		rds_rtd(RDS_RTD_ERR, "status %u => %s\n", wc->status,
 			rds_ib_wc_status_str(wc->status));