Re: [PATCH v2 05/35] pnfsd: don't set up back channel on create_session for ds

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux