Commit 065186d2 authored by Guoqing Jiang's avatar Guoqing Jiang Committed by Leon Romanovsky

RDMA/siw: No need to check term_info.valid before call siw_send_terminate

Remove the redundate checking since siw_send_terminate check it inside.
Acked-by: default avatarBernard Metzler <bmt@zurich.ibm.com>
Signed-off-by: default avatarGuoqing Jiang <guoqing.jiang@linux.dev>
Link: https://lore.kernel.org/r/20231113115726.12762-7-guoqing.jiang@linux.devSigned-off-by: default avatarLeon Romanovsky <leon@kernel.org>
parent 659da08e
...@@ -393,8 +393,7 @@ void siw_qp_cm_drop(struct siw_qp *qp, int schedule) ...@@ -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); 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) { if (cep->cm_id) {
switch (cep->state) { switch (cep->state) {
...@@ -1061,7 +1060,7 @@ static void siw_cm_work_handler(struct work_struct *w) ...@@ -1061,7 +1060,7 @@ static void siw_cm_work_handler(struct work_struct *w)
/* /*
* QP scheduled LLP close * QP scheduled LLP close
*/ */
if (cep->qp && cep->qp->term_info.valid) if (cep->qp)
siw_send_terminate(cep->qp); siw_send_terminate(cep->qp);
if (cep->cm_id) if (cep->cm_id)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment