nfsd4: move more write parameters into xdr argument
authorJ. Bruce Fields <bfields@redhat.com>
Fri, 16 Nov 2012 19:16:46 +0000 (14:16 -0500)
committerJ. Bruce Fields <bfields@redhat.com>
Mon, 26 Nov 2012 14:08:14 +0000 (09:08 -0500)
In preparation for moving some of this elsewhere.

Signed-off-by: J. Bruce Fields <bfields@redhat.com>
fs/nfsd/nfs4xdr.c
fs/nfsd/xdr4.h

index 579dc707bad9ef924ea52a709d782812c3761904..cb9f9017af8ff35c322ea0273e97c219d3ddc5e2 100644 (file)
@@ -1139,16 +1139,17 @@ nfsd4_decode_verify(struct nfsd4_compoundargs *argp, struct nfsd4_verify *verify
        DECODE_TAIL;
 }
 
-static int fill_in_write_vector(struct kvec *vec, struct kvec *head, struct page **pagelist, int buflen)
+static int fill_in_write_vector(struct kvec *vec, struct nfsd4_write *write)
 {
        int i = 1;
+       int buflen = write->wr_buflen;
 
-       vec[0].iov_base = head->iov_base;
-       vec[0].iov_len = min_t(int, buflen, head->iov_len);
+       vec[0].iov_base = write->wr_head.iov_base;
+       vec[0].iov_len = min_t(int, buflen, write->wr_head.iov_len);
        buflen -= vec[0].iov_len;
 
        while (buflen) {
-               vec[i].iov_base = page_address(pagelist[i - 1]);
+               vec[i].iov_base = page_address(write->wr_pagelist[i - 1]);
                vec[i].iov_len = min_t(int, PAGE_SIZE, buflen);
                buflen -= vec[i].iov_len;
                i++;
@@ -1161,8 +1162,6 @@ nfsd4_decode_write(struct nfsd4_compoundargs *argp, struct nfsd4_write *write)
 {
        int avail;
        int len;
-       struct page **pagelist;
-       struct kvec head;
        DECODE_HEAD;
 
        status = nfsd4_decode_stateid(argp, &write->wr_stateid);
@@ -1185,10 +1184,10 @@ nfsd4_decode_write(struct nfsd4_compoundargs *argp, struct nfsd4_write *write)
                                __FILE__, __LINE__);
                goto xdr_error;
        }
-       head.iov_base = p;
-       head.iov_len = avail;
+       write->wr_head.iov_base = p;
+       write->wr_head.iov_len = avail;
        WARN_ON(avail != (XDR_QUADLEN(avail) << 2));
-       pagelist = argp->pagelist;
+       write->wr_pagelist = argp->pagelist;
 
        len = XDR_QUADLEN(write->wr_buflen) << 2;
        if (len >= avail) {
@@ -1205,8 +1204,7 @@ nfsd4_decode_write(struct nfsd4_compoundargs *argp, struct nfsd4_write *write)
                argp->end = argp->p + XDR_QUADLEN(PAGE_SIZE);
        }
        argp->p += XDR_QUADLEN(len);
-       write->wr_vlen = fill_in_write_vector(argp->rqstp->rq_vec,
-               &head, pagelist, write->wr_buflen);
+       write->wr_vlen = fill_in_write_vector(argp->rqstp->rq_vec, write);
        WARN_ON_ONCE(write->wr_vlen > ARRAY_SIZE(argp->rqstp->rq_vec));
 
        DECODE_TAIL;
index 3c414c1be295023e4cf779ed614cde8342a28ccd..152867b8125dd6fbf03009be5bc9a42aadca9dd4 100644 (file)
@@ -386,6 +386,8 @@ struct nfsd4_write {
        u32             wr_stable_how;      /* request */
        u32             wr_buflen;          /* request */
        int             wr_vlen;
+       struct kvec     wr_head;
+       struct page **  wr_pagelist;        /* request */
 
        u32             wr_bytes_written;   /* response */
        u32             wr_how_written;     /* response */