On Thu, Oct 20, 2011 at 12:36:25PM +0000, Benny Halevy wrote:
> OK, I read you. Just please take note that WANT_NO_DELEG flag is important to simplified clients that do not support delegations, like the one I was testing against in the BAT, and it is the only one in use that I could tell.
If you can suggest an implementation that meets the minimum required by
the spec, I'd happily take it....
It might not be much harder: it's probably just a matter of returrning
the right OPEN_DELEGATE_NONE_EXT?
Though unfortunately there's not a "why" for "I just didn't feel like
giving you one", so we probably would have to make an effort to detect
the different case.
Also, I wonder if it's wise for a client to depend on this--it really is
an optional feature as far as I can tell. It would be safer to teach
the client just to handle delegations in the simplest way possible
(possibly just return them immediately?).
--b.
>
> Benny
> -----Original Message-----
> From: "J. Bruce Fields" <bfields@xxxxxxxxxxxx>Date: Thu, 20 Oct 2011 07:52:24
> To: Benny Halevy<benny@xxxxxxxxxx>
> Cc: J. Bruce Fields<bfields@xxxxxxxxxx>; <linux-nfs@xxxxxxxxxxxxxxx>; Benny Halevy<bhalevy@xxxxxxxxxx>
> Subject: Re: [PATCH 5/7] nfsd4: implement NFS4_SHARE_WANT_NO_DELEG,
> NFS4_OPEN_DELEGATE_NONE_EXT
>
> On Wed, Oct 19, 2011 at 07:13:21PM -0700, Benny Halevy wrote:
> > From: Benny Halevy <bhalevy@xxxxxxxxxx>
> >
> > Respect client request for not getting a delegation in NFSv4.1
> > Appropriately return delegation "type" NFS4_OPEN_DELEGATE_NONE_EXT
> > and WND4_NOT_WANTED reason.
>
> As I read it, if we're going to implement part of this, then we have to
> implement all of it. ("If the server supports the new _WANT_ flags and
> the client sends one or more of the new flags, then in the event the
> server does not return a delegation, it MUST return a delegation type of
> OPEN_DELEGATE_NONE_EXT. The field ond_why in the reply indicates why no
> delegation was returned and will be one of....")
>
> As long as we don't implement any want flags, we escape that MUST.
>
> So I'd rather put this aside till it's more complete.
>
> --b.
>
> >
> > Signed-off-by: Benny Halevy <bhalevy@xxxxxxxxxx>
> > ---
> > fs/nfsd/nfs4state.c | 10 ++++++++--
> > fs/nfsd/nfs4xdr.c | 3 +++
> > include/linux/nfs4.h | 15 ++++++++++++++-
> > 3 files changed, 25 insertions(+), 3 deletions(-)
> >
> > diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c
> > index ec361bb..5c3377c 100644
> > --- a/fs/nfsd/nfs4state.c
> > +++ b/fs/nfsd/nfs4state.c
> > @@ -2931,15 +2931,21 @@ static int nfs4_set_delegation(struct nfs4_delegation *dp, int flag)
> > update_stateid(&stp->st_stid.sc_stateid);
> > memcpy(&open->op_stateid, &stp->st_stid.sc_stateid, sizeof(stateid_t));
> >
> > - if (nfsd4_has_session(&resp->cstate))
> > + if (nfsd4_has_session(&resp->cstate)) {
> > open->op_openowner->oo_flags |= NFS4_OO_CONFIRMED;
> >
> > + if (open->op_share_access & NFS4_SHARE_WANT_NO_DELEG) {
> > + open->op_delegate_type = NFS4_OPEN_DELEGATE_NONE_EXT;
> > + goto nodeleg;
> > + }
> > + }
> > +
> > /*
> > * Attempt to hand out a delegation. No error return, because the
> > * OPEN succeeds even if we fail.
> > */
> > nfs4_open_delegation(current_fh, open, stp);
> > -
> > +nodeleg:
> > status = nfs_ok;
> >
> > dprintk("%s: stateid=" STATEID_FMT "\n", __func__,
> > diff --git a/fs/nfsd/nfs4xdr.c b/fs/nfsd/nfs4xdr.c
> > index 706ada1..1a419c0 100644
> > --- a/fs/nfsd/nfs4xdr.c
> > +++ b/fs/nfsd/nfs4xdr.c
> > @@ -2842,6 +2842,9 @@ static __be32 nfsd4_encode_bind_conn_to_session(struct nfsd4_compoundres *resp,
> > WRITE32(0); /* XXX: is NULL principal ok? */
> > ADJUST_ARGS();
> > break;
> > + case NFS4_OPEN_DELEGATE_NONE_EXT: /* 4.1 */
> > + WRITE32(WND4_NOT_WANTED); /* only reason for now */
> > + break;
> > default:
> > BUG();
> > }
> > diff --git a/include/linux/nfs4.h b/include/linux/nfs4.h
> > index 32345c2..8cdde4d 100644
> > --- a/include/linux/nfs4.h
> > +++ b/include/linux/nfs4.h
> > @@ -441,7 +441,20 @@ enum limit_by4 {
> > enum open_delegation_type4 {
> > NFS4_OPEN_DELEGATE_NONE = 0,
> > NFS4_OPEN_DELEGATE_READ = 1,
> > - NFS4_OPEN_DELEGATE_WRITE = 2
> > + NFS4_OPEN_DELEGATE_WRITE = 2,
> > + NFS4_OPEN_DELEGATE_NONE_EXT = 3, /* 4.1 */
> > +};
> > +
> > +enum why_no_delegation4 { /* new to v4.1 */
> > + WND4_NOT_WANTED = 0,
> > + WND4_CONTENTION = 1,
> > + WND4_RESOURCE = 2,
> > + WND4_NOT_SUPP_FTYPE = 3,
> > + WND4_WRITE_DELEG_NOT_SUPP_FTYPE = 4,
> > + WND4_NOT_SUPP_UPGRADE = 5,
> > + WND4_NOT_SUPP_DOWNGRADE = 6,
> > + WND4_CANCELLED = 7,
> > + WND4_IS_DIR = 8,
> > };
> >
> > enum lock_type4 {
> > --
> > 1.7.6
> >
--
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
