Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit eaa1ca9c authored by Leon Romanovsky's avatar Leon Romanovsky Committed by Jason Gunthorpe
Browse files

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: default avatarLeon Romanovsky <leonro@mellanox.com>
Acked-by: default avatarSantosh Shilimkar <santosh.shilimkar@oracle.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@mellanox.com>
parent deee3c7e

net/rds/ib_cm.c

+4 −4
Original line number Original line Diff line number Diff line
@@ -611,10 +611,10 @@ static int rds_ib_setup_qp(struct rds_connection *conn) 
qp_out:

qp_out:

	rdma_destroy_qp(ic->i_cm_id);

	rdma_destroy_qp(ic->i_cm_id);

recv_cq_out:

recv_cq_out:

	if (!ib_destroy_cq(ic->i_recv_cq))

	ib_destroy_cq(ic->i_recv_cq);

	ic->i_recv_cq = NULL;

	ic->i_recv_cq = NULL;

send_cq_out:

send_cq_out:

	if (!ib_destroy_cq(ic->i_send_cq))

	ib_destroy_cq(ic->i_send_cq);

	ic->i_send_cq = NULL;

	ic->i_send_cq = NULL;

rds_ibdev_out:

rds_ibdev_out:

	rds_ib_remove_conn(rds_ibdev, conn);

	rds_ib_remove_conn(rds_ibdev, conn);