diff options
| author |   Leon Romanovsky <leonro@mellanox.com> | 2019-05-20 09:54:19 +0300 |
|---|---|---|
| committer |   Jason Gunthorpe <jgg@mellanox.com> | 2019-05-21 15:50:53 -0300 |
| commit | eaa1ca9cf99224a57852f339bfbf058fbeadcb79 (patch) | |
| tree | 745763d7fb071cd988cc8570afaa975088379503 /net | |
| parent | RDMA/core: Return void from ib_device_check_mandatory() (diff) | |
| download | linux-dev-eaa1ca9cf99224a57852f339bfbf058fbeadcb79.tar.xz linux-dev-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.c | 8 |
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: |