diff mbox

NFSD: allow OP_DESTROY_CLIENTID to be only op in COMPOUND

Message ID 1308238750-9822-1-git-send-email-benny@tonian.com (mailing list archive)
State New, archived
Headers show

Commit Message

Benny Halevy June 16, 2011, 3:39 p.m. UTC
From RFC5661, section 18.50:
   DESTROY_CLIENTID MAY be preceded with a SEQUENCE operation as long as
   the client ID derived from the session ID of SEQUENCE is not the same
   as the client ID to be destroyed.  If the client IDs are the same,
   then the server MUST return NFS4ERR_CLIENTID_BUSY.

(that's not implemented yet)

   If DESTROY_CLIENTID is not prefixed by SEQUENCE, it MUST be the only
   operation in the COMPOUND request (otherwise, the server MUST return
   NFS4ERR_NOT_ONLY_OP).

Signed-off-by: Benny Halevy <benny@tonian.com>
---
 fs/nfsd/nfs4proc.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

Comments

Bruce Fields June 20, 2011, 7:31 p.m. UTC | #1
On Thu, Jun 16, 2011 at 11:39:10AM -0400, Benny Halevy wrote:
> >From RFC5661, section 18.50:
>    DESTROY_CLIENTID MAY be preceded with a SEQUENCE operation as long as
>    the client ID derived from the session ID of SEQUENCE is not the same
>    as the client ID to be destroyed.  If the client IDs are the same,
>    then the server MUST return NFS4ERR_CLIENTID_BUSY.
> 
> (that's not implemented yet)
> 
>    If DESTROY_CLIENTID is not prefixed by SEQUENCE, it MUST be the only
>    operation in the COMPOUND request (otherwise, the server MUST return
>    NFS4ERR_NOT_ONLY_OP).

Applying for 3.1, with a note that:

	"This fixes the error return; before, we returned
	NFS4ERR_OP_NOT_IN_SESSION; after this patch, we return
	NFS4ERR_NOTSUPP."

(I think that's what you saw?)

--b.

> 
> Signed-off-by: Benny Halevy <benny@tonian.com>
> ---
>  fs/nfsd/nfs4proc.c |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c
> index 730b6e7..02adcc3 100644
> --- a/fs/nfsd/nfs4proc.c
> +++ b/fs/nfsd/nfs4proc.c
> @@ -1773,6 +1773,11 @@ static struct nfsd4_operation nfsd4_ops[] = {
>  		.op_flags = ALLOWED_WITHOUT_FH | ALLOWED_AS_FIRST_OP,
>  		.op_name = "OP_SEQUENCE",
>  	},
> +	[OP_DESTROY_CLIENTID] = {
> +		.op_func = NULL,
> +		.op_flags = ALLOWED_WITHOUT_FH | ALLOWED_AS_FIRST_OP,
> +		.op_name = "OP_DESTROY_CLIENTID",
> +	},
>  	[OP_RECLAIM_COMPLETE] = {
>  		.op_func = (nfsd4op_func)nfsd4_reclaim_complete,
>  		.op_flags = ALLOWED_WITHOUT_FH,
> -- 
> 1.7.4.4
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c
index 730b6e7..02adcc3 100644
--- a/fs/nfsd/nfs4proc.c
+++ b/fs/nfsd/nfs4proc.c
@@ -1773,6 +1773,11 @@  static struct nfsd4_operation nfsd4_ops[] = {
 		.op_flags = ALLOWED_WITHOUT_FH | ALLOWED_AS_FIRST_OP,
 		.op_name = "OP_SEQUENCE",
 	},
+	[OP_DESTROY_CLIENTID] = {
+		.op_func = NULL,
+		.op_flags = ALLOWED_WITHOUT_FH | ALLOWED_AS_FIRST_OP,
+		.op_name = "OP_DESTROY_CLIENTID",
+	},
 	[OP_RECLAIM_COMPLETE] = {
 		.op_func = (nfsd4op_func)nfsd4_reclaim_complete,
 		.op_flags = ALLOWED_WITHOUT_FH,