From: Leon Romanovsky <leonro@mellanox.com>
Date: Mon, 20 May 2019 06:54:19 +0000 (+0300)
Subject: rds: Don't check return value from destroy CQ
X-Git-Tag: v5.15~5930^2~176
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=eaa1ca9cf99224a57852f339bfbf058fbeadcb79;p=platform%2Fkernel%2Flinux-starfive.git

rds: Don't check return value from destroy CQ

There is no value in checking ib_destroy_cq() result and skipping to clear
struct ic fields. This connection needs to be reinitialized anyway.

Signed-off-by: Leon Romanovsky <leonro@mellanox.com>
Acked-by: Santosh Shilimkar <santosh.shilimkar@oracle.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
---

diff --git a/net/rds/ib_cm.c b/net/rds/ib_cm.c
index 66c6eb5..5a42ebb 100644
--- a/net/rds/ib_cm.c
+++ b/net/rds/ib_cm.c
@@ -611,11 +611,11 @@ send_hdrs_dma_out:
 qp_out:
 	rdma_destroy_qp(ic->i_cm_id);
 recv_cq_out:
-	if (!ib_destroy_cq(ic->i_recv_cq))
-		ic->i_recv_cq = NULL;
+	ib_destroy_cq(ic->i_recv_cq);
+	ic->i_recv_cq = NULL;
 send_cq_out:
-	if (!ib_destroy_cq(ic->i_send_cq))
-		ic->i_send_cq = NULL;
+	ib_destroy_cq(ic->i_send_cq);
+	ic->i_send_cq = NULL;
 rds_ibdev_out:
 	rds_ib_remove_conn(rds_ibdev, conn);
 out: