From mboxrd@z Thu Jan 1 00:00:00 1970 From: "bfields-uC3wQj2KruNg9hUCZPvPmw@public.gmane.org" Subject: Re: [PATCH] nfsd: consider CLAIM_FH when handing out delegation Date: Mon, 27 Jan 2014 18:01:31 -0500 Message-ID: <20140127230131.GB8534@fieldses.org> References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "trond.myklebust-7I+n7zu2hftEKMMhf/gKZA@public.gmane.org" , Erez Zadok , "dhildeb-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org" , "geoff-6hRICwe+Y+k3uPMLIKxrzw@public.gmane.org" To: Ming Chen Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-nfs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-fsdevel.vger.kernel.org On Thu, Jan 09, 2014 at 09:26:10PM +0000, Ming Chen wrote: > CLAIM_FH was added by NFSv4.1. It is the same as CLAIM_NULL except that it > uses only current FH to identify the file to be opened. > > The NFS client is using CLAIM_FH if the FH is available when opening a file. > Currently, we cannot get any delegation if we stat a file before open it > because the server delegation code does not recognize CLAIM_FH. > > We tested this patch and found delegation can be handed out now when claim is > CLAIM_FH. Looks reasonable, thanks; applying.--b. > > See http://marc.info/?l=linux-nfs&m=136369847801388&w=2 and > http://www.linux-nfs.org/wiki/index.php/Server_4.0_and_4.1_issues#New_open_claim_types > > Signed-off-by: Ming Chen > --- > fs/nfsd/nfs4state.c | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c > index 105d6fa..514d9c2 100644 > --- a/fs/nfsd/nfs4state.c > +++ b/fs/nfsd/nfs4state.c > @@ -3125,6 +3125,7 @@ nfs4_open_delegation(struct net *net, struct svc_fh *fh, > goto out_no_deleg; > break; > case NFS4_OPEN_CLAIM_NULL: > + case NFS4_OPEN_CLAIM_FH: > /* > * Let's not give out any delegations till everyone's > * had the chance to reclaim theirs.... > -- > 1.7.1 -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html