RDMA/siw: No need to check term_info.valid before call siw_send_terminate
authorGuoqing Jiang <guoqing.jiang@linux.dev>
Mon, 13 Nov 2023 11:57:15 +0000 (19:57 +0800)
committerLeon Romanovsky <leon@kernel.org>
Wed, 15 Nov 2023 13:58:13 +0000 (15:58 +0200)
Remove the redundate checking since siw_send_terminate check it inside.

Acked-by: Bernard Metzler <bmt@zurich.ibm.com>
Signed-off-by: Guoqing Jiang <guoqing.jiang@linux.dev>
Link: https://lore.kernel.org/r/20231113115726.12762-7-guoqing.jiang@linux.dev
Signed-off-by: Leon Romanovsky <leon@kernel.org>
drivers/infiniband/sw/siw/siw_cm.c

index 7de651cb44e8b5acfa47fed116e2d1bc9d3dd5f9..5e6e571536111c90c11a14f3eb3105a1c7572bba 100644 (file)
@@ -393,8 +393,7 @@ void siw_qp_cm_drop(struct siw_qp *qp, int schedule)
                }
                siw_dbg_cep(cep, "immediate close, state %d\n", cep->state);
 
-               if (qp->term_info.valid)
-                       siw_send_terminate(qp);
+               siw_send_terminate(qp);
 
                if (cep->cm_id) {
                        switch (cep->state) {
@@ -1061,7 +1060,7 @@ static void siw_cm_work_handler(struct work_struct *w)
                /*
                 * QP scheduled LLP close
                 */
-               if (cep->qp && cep->qp->term_info.valid)
+               if (cep->qp)
                        siw_send_terminate(cep->qp);
 
                if (cep->cm_id)