On 03/04/2014 03:41 PM, Hariprasad Shenai wrote:

From: Steve Wise <[email protected]>

Based on original work by Anand Priyadarshee <[email protected]>.

Signed-off-by: Steve Wise <[email protected]>
[...]

diff --git a/drivers/infiniband/hw/cxgb4/cm.c b/drivers/infiniband/hw/cxgb4/cm.c
index 74a2250..c63dfe5 100644
--- a/drivers/infiniband/hw/cxgb4/cm.c
+++ b/drivers/infiniband/hw/cxgb4/cm.c
@@ -1648,6 +1648,15 @@ static inline int act_open_has_tid(int status)
               status != CPL_ERR_ARP_MISS;
  }

+/* Returns whether a CPL status conveys negative advice.
+ */

Hm, why not just make this one-line comment? Sorry that I haven't said that before.

+static int is_neg_adv(unsigned int status)
+{
+       return status == CPL_ERR_RTX_NEG_ADVICE ||
+              status == CPL_ERR_PERSIST_NEG_ADVICE ||
+              status == CPL_ERR_KEEPALV_NEG_ADVICE;
+}
+
  #define ACT_OPEN_RETRY_COUNT 2

  static int import_ep(struct c4iw_ep *ep, int iptype, __u8 *peer_ip,

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to