On 25/11/2021 14:24, Tony Lu wrote: > @@ -2398,7 +2400,12 @@ static int smc_shutdown(struct socket *sock, int how) > } > switch (how) { > case SHUT_RDWR: /* shutdown in both directions */ > + old_state = sk->sk_state; > rc = smc_close_active(smc); > + if (old_state == SMC_ACTIVE && > + sk->sk_state == SMC_PEERCLOSEWAIT1) > + do_shutdown = false; > + > break; Please send a v3 without the extra empty line before the break statement, and then the patch is fine with me. Thank you!