From: Santosh Shilimkar Date: Thu, 15 Sep 2016 04:56:40 +0000 (-0700) Subject: ipoib: supress the retry related completion errors X-Git-Tag: v4.1.12-92~67^2~8 X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=b36ed73a4513404b10ed414baf117e9fa4391bbe;p=users%2Fjedix%2Flinux-maple.git ipoib: supress the retry related completion errors IPoIB doesn't support transport/rnr retry schemes as per RFC so those errors are expected. No need to flood the log files with them. It was tempting make it as upstream(debug mode only) but since we have this erros in Oracle IB stack for while, leaving rest of them to be logged. Orabug: 24663803, 22347191 Tested-by: Michael Nowak Tested-by: Rafael Alejandro Peralez Tested-by: Liwen Huang Tested-by: Hong Liu Reviewed-by: Mukesh Kacker Reported-by: Rajiv Raja Signed-off-by: Santosh Shilimkar --- diff --git a/drivers/infiniband/ulp/ipoib/ipoib_cm.c b/drivers/infiniband/ulp/ipoib/ipoib_cm.c index 1888f2c078303..7cecea9c1a5a5 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_cm.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_cm.c @@ -918,13 +918,16 @@ void ipoib_cm_handle_tx_wc(struct net_device *dev, struct ib_wc *wc) wc->status != IB_WC_WR_FLUSH_ERR) { struct ipoib_neigh *neigh; - /*IB_WC_RNR_RETRY_EXC_ERR error is part of the life cycle, so don't make waves.*/ - if (IB_WC_RNR_RETRY_EXC_ERR != wc->status) - ipoib_warn(priv, "%s: failed cm send event " + /* IB_WC[_RNR]_RETRY_EXC_ERR error is part of the life cycle, + * so don't make waves. + */ + if (IB_WC_RNR_RETRY_EXC_ERR == wc->status || + IB_WC_RETRY_EXC_ERR == wc->status) + ipoib_dbg(priv, "%s: failed cm send event " "(status=%d, wrid=%d vend_err 0x%x)\n", __func__, wc->status, wr_id, wc->vendor_err); else - ipoib_dbg(priv, "%s: failed cm send event " + ipoib_warn(priv, "%s: failed cm send event " "(status=%d, wrid=%d vend_err 0x%x)\n", __func__, wc->status, wr_id, wc->vendor_err);