projects
/
linux.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
659da08
)
RDMA/siw: No need to check term_info.valid before call siw_send_terminate
author
Guoqing Jiang
<guoqing.jiang@linux.dev>
Mon, 13 Nov 2023 11:57:15 +0000
(19:57 +0800)
committer
Leon 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
patch
|
blob
|
history
diff --git
a/drivers/infiniband/sw/siw/siw_cm.c
b/drivers/infiniband/sw/siw/siw_cm.c
index 7de651cb44e8b5acfa47fed116e2d1bc9d3dd5f9..5e6e571536111c90c11a14f3eb3105a1c7572bba 100644
(file)
--- a/
drivers/infiniband/sw/siw/siw_cm.c
+++ b/
drivers/infiniband/sw/siw/siw_cm.c
@@
-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)