On Mon, Dec 07, 2009 at 11:30:51AM +0200, Benny Halevy wrote: > From: Dean Hildebrand <seattleplus@xxxxxxxxx> > > [was pnfsd: Add use of pnfs exchange flags] > Should this code be surrounded by CONFIG_PNFSD? Might be nice, but I suppose it's not important. > > Signed-off-by: Dean Hildebrand <dhildeb@xxxxxxxxxx> > Signed-off-by: Benny Halevy <bhalevy@xxxxxxxxxxx> > --- > fs/nfsd/nfs4state.c | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c > index 9ecbc25..dc9d553 100644 > --- a/fs/nfsd/nfs4state.c > +++ b/fs/nfsd/nfs4state.c > @@ -1349,6 +1349,10 @@ nfsd4_create_session(struct svc_rqst *rqstp, > cr_ses->flags &= ~SESSION4_PERSIST; > cr_ses->flags &= ~SESSION4_RDMA; > > + if (!(unconf->cl_exchange_flags & EXCHGID4_FLAG_USE_PNFS_MDS) && > + (unconf->cl_exchange_flags & EXCHGID4_FLAG_USE_PNFS_DS)) > + cr_ses->flags &= ~SESSION4_BACK_CHAN; > + Might be easier to read as: if (is_ds_only_session(unconf)) cr_ses->flags &= ~SESSION4_BACK_CHAN; with is_ds_only_session() defined in the obvious way. --b. > if (cr_ses->flags & SESSION4_BACK_CHAN) { > unconf->cl_cb_xprt = rqstp->rq_xprt; > svc_xprt_get(unconf->cl_cb_xprt); > -- > 1.6.5.1 > -- 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