summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorLeon Romanovsky <leonro@mellanox.com>2019-05-20 09:54:19 +0300
committerJason Gunthorpe <jgg@mellanox.com>2019-05-21 15:50:53 -0300
commiteaa1ca9cf99224a57852f339bfbf058fbeadcb79 (patch)
tree745763d7fb071cd988cc8570afaa975088379503 /net
parentdeee3c7e499108b0575ee0d71b786da627a7cdee (diff)
downloadlinux-eaa1ca9cf99224a57852f339bfbf058fbeadcb79.tar.gz
linux-eaa1ca9cf99224a57852f339bfbf058fbeadcb79.tar.bz2
linux-eaa1ca9cf99224a57852f339bfbf058fbeadcb79.zip
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>
Diffstat (limited to 'net')
-rw-r--r--net/rds/ib_cm.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/net/rds/ib_cm.c b/net/rds/ib_cm.c
index 66c6eb56072b..5a42ebb892cd 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: