On Fri, Aug 22, 2014 at 03:10:59PM -0400, Chuck Lever wrote: > A client may not want to use the back channel on a transport it sent > CREATE_SESSION on, in which case it clears SESSION4_BACK_CHAN. > > However, cl_cb_addr should be populated anyway, to be used if the > client binds other connections to this session. If cl_cb_addr is > not initialized, rpc_create() fails when the server attempts to > set up a back channel on such secondary transports. OK, applying.--b. > > Signed-off-by: Chuck Lever <chuck.lever@xxxxxxxxxx> > --- > > fs/nfsd/nfs4state.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c > index 2e80a59..a45f244 100644 > --- a/fs/nfsd/nfs4state.c > +++ b/fs/nfsd/nfs4state.c > @@ -1429,7 +1429,7 @@ static void init_session(struct svc_rqst *rqstp, struct nfsd4_session *new, stru > list_add(&new->se_perclnt, &clp->cl_sessions); > spin_unlock(&clp->cl_lock); > > - if (cses->flags & SESSION4_BACK_CHAN) { > + { > struct sockaddr *sa = svc_addr(rqstp); > /* > * This is a little silly; with sessions there's no real > -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html