RDS: avoid duplicate connection drop for self loopback

For self-IB loopback is special mode and the c_passive conn is just a
place holder to stick the the second QP.

Orabug: 22347191

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 61dce74)
cherry-pick-repo=linux-uek.git

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

Author: SantoshShilimkar

net/rds/rdma_transport.c

@@ -307,7 +307,8 @@ int rds_rdma_cm_event_handler(struct rdma_cm_id *cm_id,
 				"ADDR_CHANGE: calling rds_conn_drop <%pI4,%pI4,%d>\n",
 				&conn->c_laddr, &conn->c_faddr,
 				conn->c_tos);
-			rds_conn_drop(conn, DR_IB_ADDR_CHANGE);
+			if (!rds_conn_self_loopback_passive(conn))
+				rds_conn_drop(conn, DR_IB_ADDR_CHANGE);
 		}
 		break;
 

net/rds/rds.h

@@ -888,6 +888,15 @@ rds_conn_connecting(struct rds_connection *conn)
 	return atomic_read(&conn->c_state) == RDS_CONN_CONNECTING;
 }
 
+static inline bool
+rds_conn_self_loopback_passive(struct rds_connection *conn)
+{
+	if (conn->c_laddr == conn->c_faddr && !conn->c_passive)
+		return true;
+	else
+		return false;
+}
+
 /* message.c */
 struct rds_message *rds_message_alloc(unsigned int nents, gfp_t gfp);
 struct scatterlist *rds_message_alloc_sgs(struct rds_message *rm, int nents);