lisa/android_kernel_xiaomi_sm8350
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

RDMA/siw: Fix connection failure handling

Browse Source 
commit 53a3f777049771496f791504e7dc8ef017cba590 upstream.

In case immediate MPA request processing fails, the newly
created endpoint unlinks the listening endpoint and is
ready to be dropped. This special case was not handled
correctly by the code handling the later TCP socket close,
causing a NULL dereference crash in siw_cm_work_handler()
when dereferencing a NULL listener. We now also cancel
the useless MPA timeout, if immediate MPA request
processing fails.

This patch furthermore simplifies MPA processing in general:
Scheduling a useless TCP socket read in sk_data_ready() upcall
is now surpressed, if the socket is already moved out of
TCP_ESTABLISHED state.

Fixes: 6c52fdc244 ("rdma/siw: connection management")
Signed-off-by: Bernard Metzler <bmt@zurich.ibm.com>
Link: https://lore.kernel.org/r/20230905145822.446263-1-bmt@zurich.ibm.com
Signed-off-by: Leon Romanovsky <leon@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Bernard Metzler 2023-09-05 16:58:22 +02:00 committed by Greg Kroah-Hartman
parent 8ab1fb16dc
commit 6e26812e28
1 changed files with 12 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
drivers/infiniband/sw/siw/siw_cm.c
View File

@ -981,6 +981,7 @@ static void siw_accept_newconn(struct siw_cep *cep)
siw_cep_put(cep); siw_cep_put(cep);
new_cep->listen_cep = NULL; new_cep->listen_cep = NULL;
if (rv) { if (rv) {
siw_cancel_mpatimer(new_cep);
siw_cep_set_free(new_cep); siw_cep_set_free(new_cep);
goto error; goto error;
} }
@ -1105,11 +1106,14 @@ static void siw_cm_work_handler(struct work_struct *w)
/* /*
* Socket close before MPA request received. * Socket close before MPA request received.
*/ */
siw_dbg_cep(cep, "no mpareq: drop listener\n"); if (cep->listen_cep) {
siw_dbg_cep(cep,
"no mpareq: drop listener\n");
siw_cep_put(cep->listen_cep); siw_cep_put(cep->listen_cep);
cep->listen_cep = NULL; cep->listen_cep = NULL;
} }
} }
}
release_cep = 1; release_cep = 1;
break; break;
@ -1230,7 +1234,11 @@ static void siw_cm_llp_data_ready(struct sock *sk)
if (!cep) if (!cep)
goto out; goto out;
siw_dbg_cep(cep, "state: %d\n", cep->state); siw_dbg_cep(cep, "cep state: %d, socket state %d\n",
cep->state, sk->sk_state);
if (sk->sk_state != TCP_ESTABLISHED)
goto out;
switch (cep->state) { switch (cep->state) {
case SIW_EPSTATE_RDMA_MODE: case SIW_EPSTATE_RDMA_MODE: