| Message ID | d98dbfcf9e5979e053c624acc88d73e7ed14eb05.1742941932.git.trond.myklebust@hammerspace.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | Ensure that ENETUNREACH terminates state recovery | expand |
On 25 Mar 2025, at 18:35, trondmy@kernel.org wrote: > From: Trond Myklebust <trond.myklebust@hammerspace.com> > > The autobind setting was supposed to be determined in rpc_create(), > since commit c2866763b402 ("SUNRPC: use sockaddr + size when creating > remote transport endpoints"). > > Reviewed-by: Jeff Layton <jlayton@kernel.org> > Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com> > --- > net/sunrpc/clnt.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/net/sunrpc/clnt.c b/net/sunrpc/clnt.c > index 1cfaf93ceec1..6f75862d9782 100644 > --- a/net/sunrpc/clnt.c > +++ b/net/sunrpc/clnt.c > @@ -270,9 +270,6 @@ static struct rpc_xprt *rpc_clnt_set_transport(struct rpc_clnt *clnt, > old = rcu_dereference_protected(clnt->cl_xprt, > lockdep_is_held(&clnt->cl_lock)); > > - if (!xprt_bound(xprt)) > - clnt->cl_autobind = 1; > - > clnt->cl_timeout = timeout; > rcu_assign_pointer(clnt->cl_xprt, xprt); > spin_unlock(&clnt->cl_lock); > -- > 2.49.0 Reviewed-by: Benjamin Coddington <bcodding@redhat.com> Ben
diff --git a/net/sunrpc/clnt.c b/net/sunrpc/clnt.c index 1cfaf93ceec1..6f75862d9782 100644 --- a/net/sunrpc/clnt.c +++ b/net/sunrpc/clnt.c @@ -270,9 +270,6 @@ static struct rpc_xprt *rpc_clnt_set_transport(struct rpc_clnt *clnt, old = rcu_dereference_protected(clnt->cl_xprt, lockdep_is_held(&clnt->cl_lock)); - if (!xprt_bound(xprt)) - clnt->cl_autobind = 1; - clnt->cl_timeout = timeout; rcu_assign_pointer(clnt->cl_xprt, xprt); spin_unlock(&clnt->cl_lock);