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: 22347191, 24663803

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]>
Reviewed-by: Mukesh Kacker <[email protected]>
Signed-off-by: Santosh Shilimkar <[email protected]>

Author: SantoshShilimkar

net/rds/ib_recv.c

@@ -1341,9 +1341,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));