Re: Forwarded: Re: [syzbot ci] Re: net/smc: fix clcsock and lgr/lnk races in smc_diag dump path

> For archival purposes, forwarding an incoming command email to
> [email protected].
>
> ***
>
> Subject: Re: [syzbot ci] Re: net/smc: fix clcsock and lgr/lnk races in smc_diag dump path
> Author: [email protected]
>
> #syz test:

syzbot-ci does not support `#syz test` with arguments.

> git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
>
> v4: Move hash(sk) to the end of smc_sk_init() so the socket is published
> only after it is fully constructed.
>
> diff --git a/net/smc/af_smc.c b/net/smc/af_smc.c
> index 00403175b740..5e7560417ec7 100644
> --- a/net/smc/af_smc.c
> +++ b/net/smc/af_smc.c
> @@ -409,13 +409,13 @@ void smc_sk_init(struct net *net, struct sock *sk,
> int protocol)
>  				      "sk_lock-AF_SMC", &smc_key);
>  	spin_lock_init(&smc->accept_q_lock);
>  	spin_lock_init(&smc->conn.send_lock);
> -	sk->sk_prot->hash(sk);
>  	mutex_init(&smc->clcsock_release_lock);
>  	smc_init_saved_callbacks(smc);
>  	smc->limit_smc_hs = net->smc.limit_smc_hs;
>  	smc->use_fallback = false; /* assume rdma capability first */
>  	smc->fallback_rsn = 0;
>  	smc_close_init(smc);
> +	sk->sk_prot->hash(sk);
>  }
>
>  static struct sock *smc_sock_alloc(struct net *net, struct socket *sock,