SUNRPC: New xdr_streams XDR decoder API
[firefly-linux-kernel-4.4.55.git] / fs / nfs / nfs3xdr.c
index 0f07c6d55131223e2fa7aa615013dc1256c2dfae..df30a26cc4fa6aea7ae8ca1169b1749f35a632cb 100644 (file)
 #define NFS3_path_sz           (1+(NFS3_MAXPATHLEN>>2))
 #define NFS3_fattr_sz          (21)
 #define NFS3_cookieverf_sz     (NFS3_COOKIEVERFSIZE>>2)
-#define NFS3_wcc_attr_sz               (6)
+#define NFS3_wcc_attr_sz       (6)
 #define NFS3_pre_op_attr_sz    (1+NFS3_wcc_attr_sz)
 #define NFS3_post_op_attr_sz   (1+NFS3_fattr_sz)
-#define NFS3_wcc_data_sz               (NFS3_pre_op_attr_sz+NFS3_post_op_attr_sz)
-#define NFS3_fsstat_sz         
-#define NFS3_fsinfo_sz         
-#define NFS3_pathconf_sz               
-#define NFS3_entry_sz          (NFS3_filename_sz+3)
+#define NFS3_wcc_data_sz       (NFS3_pre_op_attr_sz+NFS3_post_op_attr_sz)
 #define NFS3_diropargs_sz      (NFS3_fh_sz+NFS3_filename_sz)
 
 #define NFS3_getattrargs_sz    (NFS3_fh_sz)
@@ -66,9 +62,9 @@
 #define NFS3_readdirplusargs_sz        (NFS3_fh_sz+NFS3_cookieverf_sz+4)
 #define NFS3_commitargs_sz     (NFS3_fh_sz+3)
 
-#define NFS3_attrstat_sz       (1+NFS3_fattr_sz)
-#define NFS3_wccstat_sz                (1+NFS3_wcc_data_sz)
-#define NFS3_removeres_sz      (NFS3_wccstat_sz)
+#define NFS3_getattrres_sz     (1+NFS3_fattr_sz)
+#define NFS3_setattrres_sz     (1+NFS3_wcc_data_sz)
+#define NFS3_removeres_sz      (NFS3_setattrres_sz)
 #define NFS3_lookupres_sz      (1+NFS3_fh_sz+(2 * NFS3_post_op_attr_sz))
 #define NFS3_accessres_sz      (1+NFS3_post_op_attr_sz+1)
 #define NFS3_readlinkres_sz    (1+NFS3_post_op_attr_sz+1)
@@ -130,146 +126,6 @@ static void print_overflow_msg(const char *func, const struct xdr_stream *xdr)
 }
 
 
-/*
- * Common NFS XDR functions as inlines
- */
-static inline __be32 *
-xdr_decode_fhandle(__be32 *p, struct nfs_fh *fh)
-{
-       if ((fh->size = ntohl(*p++)) <= NFS3_FHSIZE) {
-               memcpy(fh->data, p, fh->size);
-               return p + XDR_QUADLEN(fh->size);
-       }
-       return NULL;
-}
-
-static inline __be32 *
-xdr_decode_fhandle_stream(struct xdr_stream *xdr, struct nfs_fh *fh)
-{
-       __be32 *p;
-       p = xdr_inline_decode(xdr, 4);
-       if (unlikely(!p))
-               goto out_overflow;
-       fh->size = ntohl(*p++);
-
-       if (fh->size <= NFS3_FHSIZE) {
-               p = xdr_inline_decode(xdr, fh->size);
-               if (unlikely(!p))
-                       goto out_overflow;
-               memcpy(fh->data, p, fh->size);
-               return p + XDR_QUADLEN(fh->size);
-       }
-       return NULL;
-
-out_overflow:
-       print_overflow_msg(__func__, xdr);
-       return ERR_PTR(-EIO);
-}
-
-/*
- * Encode/decode time.
- */
-static inline __be32 *
-xdr_decode_time3(__be32 *p, struct timespec *timep)
-{
-       timep->tv_sec = ntohl(*p++);
-       timep->tv_nsec = ntohl(*p++);
-       return p;
-}
-
-static __be32 *
-xdr_decode_fattr(__be32 *p, struct nfs_fattr *fattr)
-{
-       unsigned int    type, major, minor;
-       umode_t         fmode;
-
-       type = ntohl(*p++);
-       if (type > NF3FIFO)
-               type = NF3NON;
-       fmode = nfs_type2fmt[type];
-       fattr->mode = (ntohl(*p++) & ~S_IFMT) | fmode;
-       fattr->nlink = ntohl(*p++);
-       fattr->uid = ntohl(*p++);
-       fattr->gid = ntohl(*p++);
-       p = xdr_decode_hyper(p, &fattr->size);
-       p = xdr_decode_hyper(p, &fattr->du.nfs3.used);
-
-       /* Turn remote device info into Linux-specific dev_t */
-       major = ntohl(*p++);
-       minor = ntohl(*p++);
-       fattr->rdev = MKDEV(major, minor);
-       if (MAJOR(fattr->rdev) != major || MINOR(fattr->rdev) != minor)
-               fattr->rdev = 0;
-
-       p = xdr_decode_hyper(p, &fattr->fsid.major);
-       fattr->fsid.minor = 0;
-       p = xdr_decode_hyper(p, &fattr->fileid);
-       p = xdr_decode_time3(p, &fattr->atime);
-       p = xdr_decode_time3(p, &fattr->mtime);
-       p = xdr_decode_time3(p, &fattr->ctime);
-
-       /* Update the mode bits */
-       fattr->valid |= NFS_ATTR_FATTR_V3;
-       return p;
-}
-
-static inline __be32 *
-xdr_decode_wcc_attr(__be32 *p, struct nfs_fattr *fattr)
-{
-       p = xdr_decode_hyper(p, &fattr->pre_size);
-       p = xdr_decode_time3(p, &fattr->pre_mtime);
-       p = xdr_decode_time3(p, &fattr->pre_ctime);
-       fattr->valid |= NFS_ATTR_FATTR_PRESIZE
-               | NFS_ATTR_FATTR_PREMTIME
-               | NFS_ATTR_FATTR_PRECTIME;
-       return p;
-}
-
-static inline __be32 *
-xdr_decode_post_op_attr(__be32 *p, struct nfs_fattr *fattr)
-{
-       if (*p++)
-               p = xdr_decode_fattr(p, fattr);
-       return p;
-}
-
-static inline __be32 *
-xdr_decode_post_op_attr_stream(struct xdr_stream *xdr, struct nfs_fattr *fattr)
-{
-       __be32 *p;
-
-       p = xdr_inline_decode(xdr, 4);
-       if (unlikely(!p))
-               goto out_overflow;
-       if (ntohl(*p++)) {
-               p = xdr_inline_decode(xdr, 84);
-               if (unlikely(!p))
-                       goto out_overflow;
-               p = xdr_decode_fattr(p, fattr);
-       }
-       return p;
-out_overflow:
-       print_overflow_msg(__func__, xdr);
-       return ERR_PTR(-EIO);
-}
-
-static inline __be32 *
-xdr_decode_pre_op_attr(__be32 *p, struct nfs_fattr *fattr)
-{
-       if (*p++)
-               return xdr_decode_wcc_attr(p, fattr);
-       return p;
-}
-
-
-static inline __be32 *
-xdr_decode_wcc_data(__be32 *p, struct nfs_fattr *fattr)
-{
-       p = xdr_decode_pre_op_attr(p, fattr);
-       return xdr_decode_post_op_attr(p, fattr);
-}
-
-
 /*
  * Encode/decode NFSv3 basic data types
  *
@@ -320,6 +176,11 @@ out_overflow:
  *
  *     typedef uint64 fileid3;
  */
+static __be32 *xdr_decode_fileid3(__be32 *p, u64 *fileid)
+{
+       return xdr_decode_hyper(p, fileid);
+}
+
 static int decode_fileid3(struct xdr_stream *xdr, u64 *fileid)
 {
        return decode_uint64(xdr, fileid);
@@ -533,6 +394,17 @@ static void encode_ftype3(struct xdr_stream *xdr, const u32 type)
        encode_uint32(xdr, type);
 }
 
+static __be32 *xdr_decode_ftype3(__be32 *p, umode_t *mode)
+{
+       u32 type;
+
+       type = be32_to_cpup(p++);
+       if (type > NF3FIFO)
+               type = NF3NON;
+       *mode = nfs_type2fmt[type];
+       return p;
+}
+
 /*
  * specdata3
  *
@@ -550,6 +422,18 @@ static void encode_specdata3(struct xdr_stream *xdr, const dev_t rdev)
        *p = cpu_to_be32(MINOR(rdev));
 }
 
+static __be32 *xdr_decode_specdata3(__be32 *p, dev_t *rdev)
+{
+       unsigned int major, minor;
+
+       major = be32_to_cpup(p++);
+       minor = be32_to_cpup(p++);
+       *rdev = MKDEV(major, minor);
+       if (MAJOR(*rdev) != major || MINOR(*rdev) != minor)
+               *rdev = 0;
+       return p;
+}
+
 /*
  * nfs_fh3
  *
@@ -611,6 +495,13 @@ static __be32 *xdr_encode_nfstime3(__be32 *p, const struct timespec *timep)
        return p;
 }
 
+static __be32 *xdr_decode_nfstime3(__be32 *p, struct timespec *timep)
+{
+       timep->tv_sec = be32_to_cpup(p++);
+       timep->tv_nsec = be32_to_cpup(p++);
+       return p;
+}
+
 /*
  * sattr3
  *
@@ -759,12 +650,33 @@ static void encode_sattr3(struct xdr_stream *xdr, const struct iattr *attr)
  */
 static int decode_fattr3(struct xdr_stream *xdr, struct nfs_fattr *fattr)
 {
+       umode_t fmode;
        __be32 *p;
 
        p = xdr_inline_decode(xdr, NFS3_fattr_sz << 2);
        if (unlikely(p == NULL))
                goto out_overflow;
-       xdr_decode_fattr(p, fattr);
+
+       p = xdr_decode_ftype3(p, &fmode);
+
+       fattr->mode = (be32_to_cpup(p++) & ~S_IFMT) | fmode;
+       fattr->nlink = be32_to_cpup(p++);
+       fattr->uid = be32_to_cpup(p++);
+       fattr->gid = be32_to_cpup(p++);
+
+       p = xdr_decode_size3(p, &fattr->size);
+       p = xdr_decode_size3(p, &fattr->du.nfs3.used);
+       p = xdr_decode_specdata3(p, &fattr->rdev);
+
+       p = xdr_decode_hyper(p, &fattr->fsid.major);
+       fattr->fsid.minor = 0;
+
+       p = xdr_decode_fileid3(p, &fattr->fileid);
+       p = xdr_decode_nfstime3(p, &fattr->atime);
+       p = xdr_decode_nfstime3(p, &fattr->mtime);
+       xdr_decode_nfstime3(p, &fattr->ctime);
+
+       fattr->valid |= NFS_ATTR_FATTR_V3;
        return 0;
 out_overflow:
        print_overflow_msg(__func__, xdr);
@@ -811,7 +723,15 @@ static int decode_wcc_attr(struct xdr_stream *xdr, struct nfs_fattr *fattr)
        p = xdr_inline_decode(xdr, NFS3_wcc_attr_sz << 2);
        if (unlikely(p == NULL))
                goto out_overflow;
-       xdr_decode_wcc_attr(p, fattr);
+
+       fattr->valid |= NFS_ATTR_FATTR_PRESIZE
+               | NFS_ATTR_FATTR_PREMTIME
+               | NFS_ATTR_FATTR_PRECTIME;
+
+       p = xdr_decode_size3(p, &fattr->pre_size);
+       p = xdr_decode_nfstime3(p, &fattr->pre_mtime);
+       xdr_decode_nfstime3(p, &fattr->pre_ctime);
+
        return 0;
 out_overflow:
        print_overflow_msg(__func__, xdr);
@@ -915,14 +835,11 @@ static void encode_diropargs3(struct xdr_stream *xdr, const struct nfs_fh *fh,
  *             nfs_fh3  object;
  *     };
  */
-static int nfs3_xdr_enc_getattr3args(struct rpc_rqst *req, __be32 *p,
-                                    const struct nfs_fh *fh)
+static void nfs3_xdr_enc_getattr3args(struct rpc_rqst *req,
+                                     struct xdr_stream *xdr,
+                                     const struct nfs_fh *fh)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_nfs_fh3(&xdr, fh);
-       return 0;
+       encode_nfs_fh3(xdr, fh);
 }
 
 /*
@@ -956,16 +873,13 @@ static void encode_sattrguard3(struct xdr_stream *xdr,
        }
 }
 
-static int nfs3_xdr_enc_setattr3args(struct rpc_rqst *req, __be32 *p,
-                                    const struct nfs3_sattrargs *args)
+static void nfs3_xdr_enc_setattr3args(struct rpc_rqst *req,
+                                     struct xdr_stream *xdr,
+                                     const struct nfs3_sattrargs *args)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_nfs_fh3(&xdr, args->fh);
-       encode_sattr3(&xdr, args->sattr);
-       encode_sattrguard3(&xdr, args);
-       return 0;
+       encode_nfs_fh3(xdr, args->fh);
+       encode_sattr3(xdr, args->sattr);
+       encode_sattrguard3(xdr, args);
 }
 
 /*
@@ -975,14 +889,11 @@ static int nfs3_xdr_enc_setattr3args(struct rpc_rqst *req, __be32 *p,
  *             diropargs3  what;
  *     };
  */
-static int nfs3_xdr_enc_lookup3args(struct rpc_rqst *req, __be32 *p,
-                                   const struct nfs3_diropargs *args)
+static void nfs3_xdr_enc_lookup3args(struct rpc_rqst *req,
+                                    struct xdr_stream *xdr,
+                                    const struct nfs3_diropargs *args)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_diropargs3(&xdr, args->fh, args->name, args->len);
-       return 0;
+       encode_diropargs3(xdr, args->fh, args->name, args->len);
 }
 
 /*
@@ -1000,14 +911,11 @@ static void encode_access3args(struct xdr_stream *xdr,
        encode_uint32(xdr, args->access);
 }
 
-static int nfs3_xdr_enc_access3args(struct rpc_rqst *req, __be32 *p,
-                                   const struct nfs3_accessargs *args)
+static void nfs3_xdr_enc_access3args(struct rpc_rqst *req,
+                                    struct xdr_stream *xdr,
+                                    const struct nfs3_accessargs *args)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_access3args(&xdr, args);
-       return 0;
+       encode_access3args(xdr, args);
 }
 
 /*
@@ -1017,16 +925,13 @@ static int nfs3_xdr_enc_access3args(struct rpc_rqst *req, __be32 *p,
  *             nfs_fh3 symlink;
  *     };
  */
-static int nfs3_xdr_enc_readlink3args(struct rpc_rqst *req, __be32 *p,
-                                     const struct nfs3_readlinkargs *args)
+static void nfs3_xdr_enc_readlink3args(struct rpc_rqst *req,
+                                      struct xdr_stream *xdr,
+                                      const struct nfs3_readlinkargs *args)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_nfs_fh3(&xdr, args->fh);
+       encode_nfs_fh3(xdr, args->fh);
        prepare_reply_buffer(req, args->pages, args->pgbase,
                                        args->pglen, NFS3_readlinkres_sz);
-       return 0;
 }
 
 /*
@@ -1050,17 +955,14 @@ static void encode_read3args(struct xdr_stream *xdr,
        *p = cpu_to_be32(args->count);
 }
 
-static int nfs3_xdr_enc_read3args(struct rpc_rqst *req, __be32 *p,
-                                 const struct nfs_readargs *args)
+static void nfs3_xdr_enc_read3args(struct rpc_rqst *req,
+                                  struct xdr_stream *xdr,
+                                  const struct nfs_readargs *args)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_read3args(&xdr, args);
+       encode_read3args(xdr, args);
        prepare_reply_buffer(req, args->pages, args->pgbase,
                                        args->count, NFS3_readres_sz);
        req->rq_rcv_buf.flags |= XDRBUF_READ;
-       return 0;
 }
 
 /*
@@ -1090,23 +992,17 @@ static void encode_write3args(struct xdr_stream *xdr,
        p = xdr_reserve_space(xdr, 8 + 4 + 4 + 4);
        p = xdr_encode_hyper(p, args->offset);
        *p++ = cpu_to_be32(args->count);
-
-       BUG_ON(args->stable > NFS_FILE_SYNC);
        *p++ = cpu_to_be32(args->stable);
-
        *p = cpu_to_be32(args->count);
        xdr_write_pages(xdr, args->pages, args->pgbase, args->count);
 }
 
-static int nfs3_xdr_enc_write3args(struct rpc_rqst *req, __be32 *p,
-                                  const struct nfs_writeargs *args)
+static void nfs3_xdr_enc_write3args(struct rpc_rqst *req,
+                                   struct xdr_stream *xdr,
+                                   const struct nfs_writeargs *args)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_write3args(&xdr, args);
-       xdr.buf->flags |= XDRBUF_WRITE;
-       return 0;
+       encode_write3args(xdr, args);
+       xdr->buf->flags |= XDRBUF_WRITE;
 }
 
 /*
@@ -1148,15 +1044,12 @@ static void encode_createhow3(struct xdr_stream *xdr,
        }
 }
 
-static int nfs3_xdr_enc_create3args(struct rpc_rqst *req, __be32 *p,
-                                   const struct nfs3_createargs *args)
+static void nfs3_xdr_enc_create3args(struct rpc_rqst *req,
+                                    struct xdr_stream *xdr,
+                                    const struct nfs3_createargs *args)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_diropargs3(&xdr, args->fh, args->name, args->len);
-       encode_createhow3(&xdr, args);
-       return 0;
+       encode_diropargs3(xdr, args->fh, args->name, args->len);
+       encode_createhow3(xdr, args);
 }
 
 /*
@@ -1167,15 +1060,12 @@ static int nfs3_xdr_enc_create3args(struct rpc_rqst *req, __be32 *p,
  *             sattr3          attributes;
  *     };
  */
-static int nfs3_xdr_enc_mkdir3args(struct rpc_rqst *req, __be32 *p,
-                                  const struct nfs3_mkdirargs *args)
+static void nfs3_xdr_enc_mkdir3args(struct rpc_rqst *req,
+                                   struct xdr_stream *xdr,
+                                   const struct nfs3_mkdirargs *args)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_diropargs3(&xdr, args->fh, args->name, args->len);
-       encode_sattr3(&xdr, args->sattr);
-       return 0;
+       encode_diropargs3(xdr, args->fh, args->name, args->len);
+       encode_sattr3(xdr, args->sattr);
 }
 
 /*
@@ -1198,15 +1088,12 @@ static void encode_symlinkdata3(struct xdr_stream *xdr,
        encode_nfspath3(xdr, args->pages, args->pathlen);
 }
 
-static int nfs3_xdr_enc_symlink3args(struct rpc_rqst *req, __be32 *p,
-                                    const struct nfs3_symlinkargs *args)
+static void nfs3_xdr_enc_symlink3args(struct rpc_rqst *req,
+                                     struct xdr_stream *xdr,
+                                     const struct nfs3_symlinkargs *args)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_diropargs3(&xdr, args->fromfh, args->fromname, args->fromlen);
-       encode_symlinkdata3(&xdr, args);
-       return 0;
+       encode_diropargs3(xdr, args->fromfh, args->fromname, args->fromlen);
+       encode_symlinkdata3(xdr, args);
 }
 
 /*
@@ -1261,15 +1148,12 @@ static void encode_mknoddata3(struct xdr_stream *xdr,
        }
 }
 
-static int nfs3_xdr_enc_mknod3args(struct rpc_rqst *req, __be32 *p,
-                                  const struct nfs3_mknodargs *args)
+static void nfs3_xdr_enc_mknod3args(struct rpc_rqst *req,
+                                   struct xdr_stream *xdr,
+                                   const struct nfs3_mknodargs *args)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_diropargs3(&xdr, args->fh, args->name, args->len);
-       encode_mknoddata3(&xdr, args);
-       return 0;
+       encode_diropargs3(xdr, args->fh, args->name, args->len);
+       encode_mknoddata3(xdr, args);
 }
 
 /*
@@ -1279,14 +1163,11 @@ static int nfs3_xdr_enc_mknod3args(struct rpc_rqst *req, __be32 *p,
  *             diropargs3  object;
  *     };
  */
-static int nfs3_xdr_enc_remove3args(struct rpc_rqst *req, __be32 *p,
-                                   const struct nfs_removeargs *args)
+static void nfs3_xdr_enc_remove3args(struct rpc_rqst *req,
+                                    struct xdr_stream *xdr,
+                                    const struct nfs_removeargs *args)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_diropargs3(&xdr, args->fh, args->name.name, args->name.len);
-       return 0;
+       encode_diropargs3(xdr, args->fh, args->name.name, args->name.len);
 }
 
 /*
@@ -1297,17 +1178,15 @@ static int nfs3_xdr_enc_remove3args(struct rpc_rqst *req, __be32 *p,
  *             diropargs3      to;
  *     };
  */
-static int nfs3_xdr_enc_rename3args(struct rpc_rqst *req, __be32 *p,
-                                   const struct nfs_renameargs *args)
+static void nfs3_xdr_enc_rename3args(struct rpc_rqst *req,
+                                    struct xdr_stream *xdr,
+                                    const struct nfs_renameargs *args)
 {
        const struct qstr *old = args->old_name;
        const struct qstr *new = args->new_name;
-       struct xdr_stream xdr;
 
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_diropargs3(&xdr, args->old_dir, old->name, old->len);
-       encode_diropargs3(&xdr, args->new_dir, new->name, new->len);
-       return 0;
+       encode_diropargs3(xdr, args->old_dir, old->name, old->len);
+       encode_diropargs3(xdr, args->new_dir, new->name, new->len);
 }
 
 /*
@@ -1318,15 +1197,12 @@ static int nfs3_xdr_enc_rename3args(struct rpc_rqst *req, __be32 *p,
  *             diropargs3      link;
  *     };
  */
-static int nfs3_xdr_enc_link3args(struct rpc_rqst *req, __be32 *p,
-                                 const struct nfs3_linkargs *args)
+static void nfs3_xdr_enc_link3args(struct rpc_rqst *req,
+                                  struct xdr_stream *xdr,
+                                  const struct nfs3_linkargs *args)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_nfs_fh3(&xdr, args->fromfh);
-       encode_diropargs3(&xdr, args->tofh, args->toname, args->tolen);
-       return 0;
+       encode_nfs_fh3(xdr, args->fromfh);
+       encode_diropargs3(xdr, args->tofh, args->toname, args->tolen);
 }
 
 /*
@@ -1352,16 +1228,13 @@ static void encode_readdir3args(struct xdr_stream *xdr,
        *p = cpu_to_be32(args->count);
 }
 
-static int nfs3_xdr_enc_readdir3args(struct rpc_rqst *req, __be32 *p,
-                                    const struct nfs3_readdirargs *args)
+static void nfs3_xdr_enc_readdir3args(struct rpc_rqst *req,
+                                     struct xdr_stream *xdr,
+                                     const struct nfs3_readdirargs *args)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_readdir3args(&xdr, args);
+       encode_readdir3args(xdr, args);
        prepare_reply_buffer(req, args->pages, 0,
                                args->count, NFS3_readdirres_sz);
-       return 0;
 }
 
 /*
@@ -1395,62 +1268,13 @@ static void encode_readdirplus3args(struct xdr_stream *xdr,
        *p = cpu_to_be32(args->count);
 }
 
-static int nfs3_xdr_enc_readdirplus3args(struct rpc_rqst *req, __be32 *p,
-                                        const struct nfs3_readdirargs *args)
+static void nfs3_xdr_enc_readdirplus3args(struct rpc_rqst *req,
+                                         struct xdr_stream *xdr,
+                                         const struct nfs3_readdirargs *args)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_readdirplus3args(&xdr, args);
+       encode_readdirplus3args(xdr, args);
        prepare_reply_buffer(req, args->pages, 0,
                                args->count, NFS3_readdirres_sz);
-       return 0;
-}
-
-/*
- * Decode the result of a readdir call.
- * We just check for syntactical correctness.
- */
-static int
-nfs3_xdr_readdirres(struct rpc_rqst *req, __be32 *p, struct nfs3_readdirres *res)
-{
-       struct xdr_buf *rcvbuf = &req->rq_rcv_buf;
-       struct kvec *iov = rcvbuf->head;
-       struct page **page;
-       size_t hdrlen;
-       u32 recvd, pglen;
-       int status;
-
-       status = ntohl(*p++);
-       /* Decode post_op_attrs */
-       p = xdr_decode_post_op_attr(p, res->dir_attr);
-       if (status)
-               return nfs_stat_to_errno(status);
-       /* Decode verifier cookie */
-       if (res->verf) {
-               res->verf[0] = *p++;
-               res->verf[1] = *p++;
-       } else {
-               p += 2;
-       }
-
-       hdrlen = (u8 *) p - (u8 *) iov->iov_base;
-       if (iov->iov_len < hdrlen) {
-               dprintk("NFS: READDIR reply header overflowed:"
-                               "length %Zu > %Zu\n", hdrlen, iov->iov_len);
-               return -errno_NFSERR_IO;
-       } else if (iov->iov_len != hdrlen) {
-               dprintk("NFS: READDIR header is short. iovec will be shifted.\n");
-               xdr_shift_buf(rcvbuf, iov->iov_len - hdrlen);
-       }
-
-       pglen = rcvbuf->page_len;
-       recvd = rcvbuf->len - hdrlen;
-       if (pglen > recvd)
-               pglen = recvd;
-       page = rcvbuf->pages;
-
-       return pglen;
 }
 
 /*
@@ -1474,78 +1298,59 @@ static void encode_commit3args(struct xdr_stream *xdr,
        *p = cpu_to_be32(args->count);
 }
 
-static int nfs3_xdr_enc_commit3args(struct rpc_rqst *req, __be32 *p,
-                                   const struct nfs_writeargs *args)
+static void nfs3_xdr_enc_commit3args(struct rpc_rqst *req,
+                                    struct xdr_stream *xdr,
+                                    const struct nfs_writeargs *args)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_commit3args(&xdr, args);
-       return 0;
+       encode_commit3args(xdr, args);
 }
 
 #ifdef CONFIG_NFS_V3_ACL
 
-static int nfs3_xdr_enc_getacl3args(struct rpc_rqst *req, __be32 *p,
-                                   const struct nfs3_getaclargs *args)
+static void nfs3_xdr_enc_getacl3args(struct rpc_rqst *req,
+                                    struct xdr_stream *xdr,
+                                    const struct nfs3_getaclargs *args)
 {
-       struct xdr_stream xdr;
-
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_nfs_fh3(&xdr, args->fh);
-       encode_uint32(&xdr, args->mask);
+       encode_nfs_fh3(xdr, args->fh);
+       encode_uint32(xdr, args->mask);
        if (args->mask & (NFS_ACL | NFS_DFACL))
                prepare_reply_buffer(req, args->pages, 0,
                                        NFSACL_MAXPAGES << PAGE_SHIFT,
                                        ACL3_getaclres_sz);
-       return 0;
 }
 
-static int nfs3_xdr_enc_setacl3args(struct rpc_rqst *req, __be32 *p,
-                                   const struct nfs3_setaclargs *args)
+static void nfs3_xdr_enc_setacl3args(struct rpc_rqst *req,
+                                    struct xdr_stream *xdr,
+                                    const struct nfs3_setaclargs *args)
 {
-       struct xdr_stream xdr;
        unsigned int base;
        int error;
 
-       xdr_init_encode(&xdr, &req->rq_snd_buf, p);
-       encode_nfs_fh3(&xdr, NFS_FH(args->inode));
-       encode_uint32(&xdr, args->mask);
+       encode_nfs_fh3(xdr, NFS_FH(args->inode));
+       encode_uint32(xdr, args->mask);
        if (args->npages != 0)
-               xdr_write_pages(&xdr, args->pages, 0, args->len);
+               xdr_write_pages(xdr, args->pages, 0, args->len);
 
        base = req->rq_slen;
-       error = nfsacl_encode(xdr.buf, base, args->inode,
+       error = nfsacl_encode(xdr->buf, base, args->inode,
                            (args->mask & NFS_ACL) ?
                            args->acl_access : NULL, 1, 0);
        BUG_ON(error < 0);
-       error = nfsacl_encode(xdr.buf, base + error, args->inode,
+       error = nfsacl_encode(xdr->buf, base + error, args->inode,
                            (args->mask & NFS_DFACL) ?
                            args->acl_default : NULL, 1,
                            NFS_ACL_DEFAULT);
        BUG_ON(error < 0);
-       return 0;
 }
 
 #endif  /* CONFIG_NFS_V3_ACL */
 
 /*
- * NFS XDR decode functions
- */
-
-/*
- * Decode attrstat reply.
+ * NFSv3 XDR decode functions
+ *
+ * NFSv3 result types are defined in section 3.3 of RFC 1813:
+ * "NFS Version 3 Protocol Specification".
  */
-static int
-nfs3_xdr_attrstat(struct rpc_rqst *req, __be32 *p, struct nfs_fattr *fattr)
-{
-       int     status;
-
-       if ((status = ntohl(*p++)))
-               return nfs_stat_to_errno(status);
-       xdr_decode_fattr(p, fattr);
-       return 0;
-}
 
 /*
  * 3.3.1  GETATTR3res
@@ -1561,41 +1366,25 @@ nfs3_xdr_attrstat(struct rpc_rqst *req, __be32 *p, struct nfs_fattr *fattr)
  *             void;
  *     };
  */
-static int nfs3_xdr_dec_getattr3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_getattr3res(struct rpc_rqst *req,
+                                   struct xdr_stream *xdr,
                                    struct nfs_fattr *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
                goto out_default;
-       error = decode_fattr3(&xdr, result);
+       error = decode_fattr3(xdr, result);
 out:
        return error;
 out_default:
        return nfs_stat_to_errno(status);
 }
 
-/*
- * Decode status+wcc_data reply
- * SATTR, REMOVE, RMDIR
- */
-static int
-nfs3_xdr_wccstat(struct rpc_rqst *req, __be32 *p, struct nfs_fattr *fattr)
-{
-       int     status;
-
-       if ((status = ntohl(*p++)))
-               status = nfs_stat_to_errno(status);
-       xdr_decode_wcc_data(p, fattr);
-       return status;
-}
-
 /*
  * 3.3.2  SETATTR3res
  *
@@ -1614,18 +1403,17 @@ nfs3_xdr_wccstat(struct rpc_rqst *req, __be32 *p, struct nfs_fattr *fattr)
  *             SETATTR3resfail resfail;
  *     };
  */
-static int nfs3_xdr_dec_setattr3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_setattr3res(struct rpc_rqst *req,
+                                   struct xdr_stream *xdr,
                                    struct nfs_fattr *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
-       error = decode_wcc_data(&xdr, result);
+       error = decode_wcc_data(xdr, result);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
@@ -1636,31 +1424,6 @@ out_status:
        return nfs_stat_to_errno(status);
 }
 
-static int
-nfs3_xdr_removeres(struct rpc_rqst *req, __be32 *p, struct nfs_removeres *res)
-{
-       return nfs3_xdr_wccstat(req, p, res->dir_attr);
-}
-
-/*
- * Decode LOOKUP reply
- */
-static int
-nfs3_xdr_lookupres(struct rpc_rqst *req, __be32 *p, struct nfs3_diropres *res)
-{
-       int     status;
-
-       if ((status = ntohl(*p++))) {
-               status = nfs_stat_to_errno(status);
-       } else {
-               if (!(p = xdr_decode_fhandle(p, res->fh)))
-                       return -errno_NFSERR_IO;
-               p = xdr_decode_post_op_attr(p, res->fattr);
-       }
-       xdr_decode_post_op_attr(p, res->dir_attr);
-       return status;
-}
-
 /*
  * 3.3.3  LOOKUP3res
  *
@@ -1681,50 +1444,34 @@ nfs3_xdr_lookupres(struct rpc_rqst *req, __be32 *p, struct nfs3_diropres *res)
  *             LOOKUP3resfail  resfail;
  *     };
  */
-static int nfs3_xdr_dec_lookup3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_lookup3res(struct rpc_rqst *req,
+                                  struct xdr_stream *xdr,
                                   struct nfs3_diropres *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
                goto out_default;
-       error = decode_nfs_fh3(&xdr, result->fh);
+       error = decode_nfs_fh3(xdr, result->fh);
        if (unlikely(error))
                goto out;
-       error = decode_post_op_attr(&xdr, result->fattr);
+       error = decode_post_op_attr(xdr, result->fattr);
        if (unlikely(error))
                goto out;
-       error = decode_post_op_attr(&xdr, result->dir_attr);
+       error = decode_post_op_attr(xdr, result->dir_attr);
 out:
        return error;
 out_default:
-       error = decode_post_op_attr(&xdr, result->dir_attr);
+       error = decode_post_op_attr(xdr, result->dir_attr);
        if (unlikely(error))
                goto out;
        return nfs_stat_to_errno(status);
 }
 
-/*
- * Decode ACCESS reply
- */
-static int
-nfs3_xdr_accessres(struct rpc_rqst *req, __be32 *p, struct nfs3_accessres *res)
-{
-       int     status = ntohl(*p++);
-
-       p = xdr_decode_post_op_attr(p, res->fattr);
-       if (status)
-               return nfs_stat_to_errno(status);
-       res->access = ntohl(*p++);
-       return 0;
-}
-
 /*
  * 3.3.4  ACCESS3res
  *
@@ -1744,75 +1491,28 @@ nfs3_xdr_accessres(struct rpc_rqst *req, __be32 *p, struct nfs3_accessres *res)
  *             ACCESS3resfail  resfail;
  *     };
  */
-static int nfs3_xdr_dec_access3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_access3res(struct rpc_rqst *req,
+                                  struct xdr_stream *xdr,
                                   struct nfs3_accessres *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
-       error = decode_post_op_attr(&xdr, result->fattr);
+       error = decode_post_op_attr(xdr, result->fattr);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
                goto out_default;
-       error = decode_uint32(&xdr, &result->access);
+       error = decode_uint32(xdr, &result->access);
 out:
        return error;
 out_default:
        return nfs_stat_to_errno(status);
 }
 
-/*
- * Decode READLINK reply
- */
-static int
-nfs3_xdr_readlinkres(struct rpc_rqst *req, __be32 *p, struct nfs_fattr *fattr)
-{
-       struct xdr_buf *rcvbuf = &req->rq_rcv_buf;
-       struct kvec *iov = rcvbuf->head;
-       size_t hdrlen;
-       u32 len, recvd;
-       int     status;
-
-       status = ntohl(*p++);
-       p = xdr_decode_post_op_attr(p, fattr);
-
-       if (status != 0)
-               return nfs_stat_to_errno(status);
-
-       /* Convert length of symlink */
-       len = ntohl(*p++);
-       if (len >= rcvbuf->page_len) {
-               dprintk("nfs: server returned giant symlink!\n");
-               return -ENAMETOOLONG;
-       }
-
-       hdrlen = (u8 *) p - (u8 *) iov->iov_base;
-       if (iov->iov_len < hdrlen) {
-               dprintk("NFS: READLINK reply header overflowed:"
-                               "length %Zu > %Zu\n", hdrlen, iov->iov_len);
-               return -errno_NFSERR_IO;
-       } else if (iov->iov_len != hdrlen) {
-               dprintk("NFS: READLINK header is short. "
-                       "iovec will be shifted.\n");
-               xdr_shift_buf(rcvbuf, iov->iov_len - hdrlen);
-       }
-       recvd = req->rq_rcv_buf.len - hdrlen;
-       if (recvd < len) {
-               dprintk("NFS: server cheating in readlink reply: "
-                               "count %u > recvd %u\n", len, recvd);
-               return -EIO;
-       }
-
-       xdr_terminate_string(rcvbuf, len);
-       return 0;
-}
-
 /*
  * 3.3.5  READLINK3res
  *
@@ -1832,82 +1532,28 @@ nfs3_xdr_readlinkres(struct rpc_rqst *req, __be32 *p, struct nfs_fattr *fattr)
  *             READLINK3resfail resfail;
  *     };
  */
-static int nfs3_xdr_dec_readlink3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_readlink3res(struct rpc_rqst *req,
+                                    struct xdr_stream *xdr,
                                     struct nfs_fattr *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
-       error = decode_post_op_attr(&xdr, result);
+       error = decode_post_op_attr(xdr, result);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
                goto out_default;
-       error = decode_nfspath3(&xdr);
+       error = decode_nfspath3(xdr);
 out:
        return error;
 out_default:
        return nfs_stat_to_errno(status);
 }
 
-/*
- * Decode READ reply
- */
-static int
-nfs3_xdr_readres(struct rpc_rqst *req, __be32 *p, struct nfs_readres *res)
-{
-       struct kvec *iov = req->rq_rcv_buf.head;
-       size_t hdrlen;
-       u32 count, ocount, recvd;
-       int status;
-
-       status = ntohl(*p++);
-       p = xdr_decode_post_op_attr(p, res->fattr);
-
-       if (status != 0)
-               return nfs_stat_to_errno(status);
-
-       /* Decode reply count and EOF flag. NFSv3 is somewhat redundant
-        * in that it puts the count both in the res struct and in the
-        * opaque data count. */
-       count    = ntohl(*p++);
-       res->eof = ntohl(*p++);
-       ocount   = ntohl(*p++);
-
-       if (ocount != count) {
-               dprintk("NFS: READ count doesn't match RPC opaque count.\n");
-               return -errno_NFSERR_IO;
-       }
-
-       hdrlen = (u8 *) p - (u8 *) iov->iov_base;
-       if (iov->iov_len < hdrlen) {
-               dprintk("NFS: READ reply header overflowed:"
-                               "length %Zu > %Zu\n", hdrlen, iov->iov_len);
-                       return -errno_NFSERR_IO;
-       } else if (iov->iov_len != hdrlen) {
-               dprintk("NFS: READ header is short. iovec will be shifted.\n");
-               xdr_shift_buf(&req->rq_rcv_buf, iov->iov_len - hdrlen);
-       }
-
-       recvd = req->rq_rcv_buf.len - hdrlen;
-       if (count > recvd) {
-               dprintk("NFS: server cheating in read reply: "
-                       "count %u > recvd %u\n", count, recvd);
-               count = recvd;
-               res->eof = 0;
-       }
-
-       if (count < res->count)
-               res->count = count;
-
-       return count;
-}
-
 /*
  * 3.3.6  READ3res
  *
@@ -1969,51 +1615,27 @@ out_overflow:
        return -EIO;
 }
 
-static int nfs3_xdr_dec_read3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_read3res(struct rpc_rqst *req, struct xdr_stream *xdr,
                                 struct nfs_readres *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
-       error = decode_post_op_attr(&xdr, result->fattr);
+       error = decode_post_op_attr(xdr, result->fattr);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
                goto out_status;
-       error = decode_read3resok(&xdr, result);
+       error = decode_read3resok(xdr, result);
 out:
        return error;
 out_status:
        return nfs_stat_to_errno(status);
 }
 
-/*
- * Decode WRITE response
- */
-static int
-nfs3_xdr_writeres(struct rpc_rqst *req, __be32 *p, struct nfs_writeres *res)
-{
-       int     status;
-
-       status = ntohl(*p++);
-       p = xdr_decode_wcc_data(p, res->fattr);
-
-       if (status != 0)
-               return nfs_stat_to_errno(status);
-
-       res->count = ntohl(*p++);
-       res->verf->committed = (enum nfs3_stable_how)ntohl(*p++);
-       res->verf->verifier[0] = *p++;
-       res->verf->verifier[1] = *p++;
-
-       return res->count;
-}
-
 /*
  * 3.3.7  WRITE3res
  *
@@ -2063,56 +1685,27 @@ out_overflow:
        return -EIO;
 }
 
-static int nfs3_xdr_dec_write3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_write3res(struct rpc_rqst *req, struct xdr_stream *xdr,
                                  struct nfs_writeres *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
-       error = decode_wcc_data(&xdr, result->fattr);
+       error = decode_wcc_data(xdr, result->fattr);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
                goto out_status;
-       error = decode_write3resok(&xdr, result);
+       error = decode_write3resok(xdr, result);
 out:
        return error;
 out_status:
        return nfs_stat_to_errno(status);
 }
 
-/*
- * Decode a CREATE response
- */
-static int
-nfs3_xdr_createres(struct rpc_rqst *req, __be32 *p, struct nfs3_diropres *res)
-{
-       int     status;
-
-       status = ntohl(*p++);
-       if (status == 0) {
-               if (*p++) {
-                       if (!(p = xdr_decode_fhandle(p, res->fh)))
-                               return -errno_NFSERR_IO;
-                       p = xdr_decode_post_op_attr(p, res->fattr);
-               } else {
-                       memset(res->fh, 0, sizeof(*res->fh));
-                       /* Do decode post_op_attr but set it to NULL */
-                       p = xdr_decode_post_op_attr(p, res->fattr);
-                       res->fattr->valid = 0;
-               }
-       } else {
-               status = nfs_stat_to_errno(status);
-       }
-       p = xdr_decode_wcc_data(p, res->dir_attr);
-       return status;
-}
-
 /*
  * 3.3.8  CREATE3res
  *
@@ -2155,24 +1748,23 @@ out:
        return error;
 }
 
-static int nfs3_xdr_dec_create3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_create3res(struct rpc_rqst *req,
+                                  struct xdr_stream *xdr,
                                   struct nfs3_diropres *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
                goto out_default;
-       error = decode_create3resok(&xdr, result);
+       error = decode_create3resok(xdr, result);
 out:
        return error;
 out_default:
-       error = decode_wcc_data(&xdr, result->dir_attr);
+       error = decode_wcc_data(xdr, result->dir_attr);
        if (unlikely(error))
                goto out;
        return nfs_stat_to_errno(status);
@@ -2196,18 +1788,17 @@ out_default:
  *             REMOVE3resfail resfail;
  *     };
  */
-static int nfs3_xdr_dec_remove3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_remove3res(struct rpc_rqst *req,
+                                  struct xdr_stream *xdr,
                                   struct nfs_removeres *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
-       error = decode_wcc_data(&xdr, result->dir_attr);
+       error = decode_wcc_data(xdr, result->dir_attr);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
@@ -2218,21 +1809,6 @@ out_status:
        return nfs_stat_to_errno(status);
 }
 
-/*
- * Decode RENAME reply
- */
-static int
-nfs3_xdr_renameres(struct rpc_rqst *req, __be32 *p, struct nfs_renameres *res)
-{
-       int     status;
-
-       if ((status = ntohl(*p++)) != 0)
-               status = nfs_stat_to_errno(status);
-       p = xdr_decode_wcc_data(p, res->old_fattr);
-       p = xdr_decode_wcc_data(p, res->new_fattr);
-       return status;
-}
-
 /*
  * 3.3.14  RENAME3res
  *
@@ -2253,21 +1829,20 @@ nfs3_xdr_renameres(struct rpc_rqst *req, __be32 *p, struct nfs_renameres *res)
  *             RENAME3resfail resfail;
  *     };
  */
-static int nfs3_xdr_dec_rename3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_rename3res(struct rpc_rqst *req,
+                                  struct xdr_stream *xdr,
                                   struct nfs_renameres *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
-       error = decode_wcc_data(&xdr, result->old_fattr);
+       error = decode_wcc_data(xdr, result->old_fattr);
        if (unlikely(error))
                goto out;
-       error = decode_wcc_data(&xdr, result->new_fattr);
+       error = decode_wcc_data(xdr, result->new_fattr);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
@@ -2278,21 +1853,6 @@ out_status:
        return nfs_stat_to_errno(status);
 }
 
-/*
- * Decode LINK reply
- */
-static int
-nfs3_xdr_linkres(struct rpc_rqst *req, __be32 *p, struct nfs3_linkres *res)
-{
-       int     status;
-
-       if ((status = ntohl(*p++)) != 0)
-               status = nfs_stat_to_errno(status);
-       p = xdr_decode_post_op_attr(p, res->fattr);
-       p = xdr_decode_wcc_data(p, res->dir_attr);
-       return status;
-}
-
 /*
  * 3.3.15  LINK3res
  *
@@ -2313,21 +1873,19 @@ nfs3_xdr_linkres(struct rpc_rqst *req, __be32 *p, struct nfs3_linkres *res)
  *             LINK3resfail    resfail;
  *     };
  */
-static int nfs3_xdr_dec_link3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_link3res(struct rpc_rqst *req, struct xdr_stream *xdr,
                                 struct nfs3_linkres *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
-       error = decode_post_op_attr(&xdr, result->fattr);
+       error = decode_post_op_attr(xdr, result->fattr);
        if (unlikely(error))
                goto out;
-       error = decode_wcc_data(&xdr, result->dir_attr);
+       error = decode_wcc_data(xdr, result->dir_attr);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
@@ -2343,10 +1901,10 @@ out_status:
  *                     the local page cache
  * @xdr: XDR stream where entry resides
  * @entry: buffer to fill in with entry data
- * @server: nfs_server data for this directory
  * @plus: boolean indicating whether this should be a readdirplus entry
  *
- * Returns the position of the next item in the buffer, or an ERR_PTR.
+ * Returns zero if successful, otherwise a negative errno value is
+ * returned.
  *
  * This function is not invoked during READDIR reply decoding, but
  * rather whenever an application invokes the getdents(2) system call
@@ -2373,8 +1931,8 @@ out_status:
  *             entryplus3      *nextentry;
  *     };
  */
-__be32 *nfs3_decode_dirent(struct xdr_stream *xdr, struct nfs_entry *entry,
-                          struct nfs_server *server, int plus)
+int nfs3_decode_dirent(struct xdr_stream *xdr, struct nfs_entry *entry,
+                      int plus)
 {
        struct nfs_entry old = *entry;
        __be32 *p;
@@ -2388,23 +1946,23 @@ __be32 *nfs3_decode_dirent(struct xdr_stream *xdr, struct nfs_entry *entry,
                if (unlikely(p == NULL))
                        goto out_overflow;
                if (*p == xdr_zero)
-                       return ERR_PTR(-EAGAIN);
+                       return -EAGAIN;
                entry->eof = 1;
-               return ERR_PTR(-EBADCOOKIE);
+               return -EBADCOOKIE;
        }
 
        error = decode_fileid3(xdr, &entry->ino);
        if (unlikely(error))
-               return ERR_PTR(error);
+               return error;
 
        error = decode_inline_filename3(xdr, &entry->name, &entry->len);
        if (unlikely(error))
-               return ERR_PTR(error);
+               return error;
 
        entry->prev_cookie = entry->cookie;
        error = decode_cookie3(xdr, &entry->cookie);
        if (unlikely(error))
-               return ERR_PTR(error);
+               return error;
 
        entry->d_type = DT_UNKNOWN;
 
@@ -2412,7 +1970,7 @@ __be32 *nfs3_decode_dirent(struct xdr_stream *xdr, struct nfs_entry *entry,
                entry->fattr->valid = 0;
                error = decode_post_op_attr(xdr, entry->fattr);
                if (unlikely(error))
-                       return ERR_PTR(error);
+                       return error;
                if (entry->fattr->valid & NFS_ATTR_FATTR_V3)
                        entry->d_type = nfs_umode_to_dtype(entry->fattr->mode);
 
@@ -2425,7 +1983,7 @@ __be32 *nfs3_decode_dirent(struct xdr_stream *xdr, struct nfs_entry *entry,
                        if (unlikely(error)) {
                                if (error == -E2BIG)
                                        goto out_truncated;
-                               return ERR_PTR(error);
+                               return error;
                        }
                } else
                        zero_nfs_fh3(entry->fh);
@@ -2436,15 +1994,15 @@ __be32 *nfs3_decode_dirent(struct xdr_stream *xdr, struct nfs_entry *entry,
        entry->eof = 0;
        if (p != NULL)
                entry->eof = (p[0] == xdr_zero) && (p[1] != xdr_zero);
-       return p;
+       return 0;
 
 out_overflow:
        print_overflow_msg(__func__, xdr);
-       return ERR_PTR(-EAGAIN);
+       return -EAGAIN;
 out_truncated:
        dprintk("NFS: directory entry contains invalid file handle\n");
        *entry = old;
-       return ERR_PTR(-EAGAIN);
+       return -EAGAIN;
 }
 
 /*
@@ -2513,54 +2071,28 @@ out:
        return error;
 }
 
-static int nfs3_xdr_dec_readdir3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_readdir3res(struct rpc_rqst *req,
+                                   struct xdr_stream *xdr,
                                    struct nfs3_readdirres *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
                goto out_default;
-       error = decode_readdir3resok(&xdr, result);
+       error = decode_readdir3resok(xdr, result);
 out:
        return error;
 out_default:
-       error = decode_post_op_attr(&xdr, result->dir_attr);
+       error = decode_post_op_attr(xdr, result->dir_attr);
        if (unlikely(error))
                goto out;
        return nfs_stat_to_errno(status);
 }
 
-/*
- * Decode FSSTAT reply
- */
-static int
-nfs3_xdr_fsstatres(struct rpc_rqst *req, __be32 *p, struct nfs_fsstat *res)
-{
-       int             status;
-
-       status = ntohl(*p++);
-
-       p = xdr_decode_post_op_attr(p, res->fattr);
-       if (status != 0)
-               return nfs_stat_to_errno(status);
-
-       p = xdr_decode_hyper(p, &res->tbytes);
-       p = xdr_decode_hyper(p, &res->fbytes);
-       p = xdr_decode_hyper(p, &res->abytes);
-       p = xdr_decode_hyper(p, &res->tfiles);
-       p = xdr_decode_hyper(p, &res->ffiles);
-       p = xdr_decode_hyper(p, &res->afiles);
-
-       /* ignore invarsec */
-       return 0;
-}
-
 /*
  * 3.3.18  FSSTAT3res
  *
@@ -2607,58 +2139,28 @@ out_overflow:
        return -EIO;
 }
 
-static int nfs3_xdr_dec_fsstat3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_fsstat3res(struct rpc_rqst *req,
+                                  struct xdr_stream *xdr,
                                   struct nfs_fsstat *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
-       error = decode_post_op_attr(&xdr, result->fattr);
+       error = decode_post_op_attr(xdr, result->fattr);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
                goto out_status;
-       error = decode_fsstat3resok(&xdr, result);
+       error = decode_fsstat3resok(xdr, result);
 out:
        return error;
 out_status:
        return nfs_stat_to_errno(status);
 }
 
-/*
- * Decode FSINFO reply
- */
-static int
-nfs3_xdr_fsinfores(struct rpc_rqst *req, __be32 *p, struct nfs_fsinfo *res)
-{
-       int             status;
-
-       status = ntohl(*p++);
-
-       p = xdr_decode_post_op_attr(p, res->fattr);
-       if (status != 0)
-               return nfs_stat_to_errno(status);
-
-       res->rtmax  = ntohl(*p++);
-       res->rtpref = ntohl(*p++);
-       res->rtmult = ntohl(*p++);
-       res->wtmax  = ntohl(*p++);
-       res->wtpref = ntohl(*p++);
-       res->wtmult = ntohl(*p++);
-       res->dtpref = ntohl(*p++);
-       p = xdr_decode_hyper(p, &res->maxfilesize);
-       p = xdr_decode_time3(p, &res->time_delta);
-
-       /* ignore properties */
-       res->lease_time = 0;
-       return 0;
-}
-
 /*
  * 3.3.19  FSINFO3res
  *
@@ -2703,7 +2205,7 @@ static int decode_fsinfo3resok(struct xdr_stream *xdr,
        result->wtmult = be32_to_cpup(p++);
        result->dtpref = be32_to_cpup(p++);
        p = xdr_decode_size3(p, &result->maxfilesize);
-       xdr_decode_time3(p, &result->time_delta);
+       xdr_decode_nfstime3(p, &result->time_delta);
 
        /* ignore properties */
        result->lease_time = 0;
@@ -2713,49 +2215,28 @@ out_overflow:
        return -EIO;
 }
 
-static int nfs3_xdr_dec_fsinfo3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_fsinfo3res(struct rpc_rqst *req,
+                                  struct xdr_stream *xdr,
                                   struct nfs_fsinfo *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
-       error = decode_post_op_attr(&xdr, result->fattr);
+       error = decode_post_op_attr(xdr, result->fattr);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
                goto out_status;
-       error = decode_fsinfo3resok(&xdr, result);
+       error = decode_fsinfo3resok(xdr, result);
 out:
        return error;
 out_status:
        return nfs_stat_to_errno(status);
 }
 
-/*
- * Decode PATHCONF reply
- */
-static int
-nfs3_xdr_pathconfres(struct rpc_rqst *req, __be32 *p, struct nfs_pathconf *res)
-{
-       int             status;
-
-       status = ntohl(*p++);
-
-       p = xdr_decode_post_op_attr(p, res->fattr);
-       if (status != 0)
-               return nfs_stat_to_errno(status);
-       res->max_link = ntohl(*p++);
-       res->max_namelen = ntohl(*p++);
-
-       /* ignore remaining fields */
-       return 0;
-}
-
 /*
  * 3.3.20  PATHCONF3res
  *
@@ -2797,47 +2278,28 @@ out_overflow:
        return -EIO;
 }
 
-static int nfs3_xdr_dec_pathconf3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_pathconf3res(struct rpc_rqst *req,
+                                    struct xdr_stream *xdr,
                                     struct nfs_pathconf *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
-       error = decode_post_op_attr(&xdr, result->fattr);
+       error = decode_post_op_attr(xdr, result->fattr);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
                goto out_status;
-       error = decode_pathconf3resok(&xdr, result);
+       error = decode_pathconf3resok(xdr, result);
 out:
        return error;
 out_status:
        return nfs_stat_to_errno(status);
 }
 
-/*
- * Decode COMMIT reply
- */
-static int
-nfs3_xdr_commitres(struct rpc_rqst *req, __be32 *p, struct nfs_writeres *res)
-{
-       int             status;
-
-       status = ntohl(*p++);
-       p = xdr_decode_wcc_data(p, res->fattr);
-       if (status != 0)
-               return nfs_stat_to_errno(status);
-
-       res->verf->verifier[0] = *p++;
-       res->verf->verifier[1] = *p++;
-       return 0;
-}
-
 /*
  * 3.3.21  COMMIT3res
  *
@@ -2857,23 +2319,22 @@ nfs3_xdr_commitres(struct rpc_rqst *req, __be32 *p, struct nfs_writeres *res)
  *             COMMIT3resfail  resfail;
  *     };
  */
-static int nfs3_xdr_dec_commit3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_commit3res(struct rpc_rqst *req,
+                                  struct xdr_stream *xdr,
                                   struct nfs_writeres *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
-       error = decode_wcc_data(&xdr, result->fattr);
+       error = decode_wcc_data(xdr, result->fattr);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
                goto out_status;
-       error = decode_writeverf3(&xdr, result->verf->verifier);
+       error = decode_writeverf3(xdr, result->verf->verifier);
 out:
        return error;
 out_status:
@@ -2881,37 +2342,6 @@ out_status:
 }
 
 #ifdef CONFIG_NFS_V3_ACL
-/*
- * Decode GETACL reply
- */
-static int
-nfs3_xdr_getaclres(struct rpc_rqst *req, __be32 *p,
-                  struct nfs3_getaclres *res)
-{
-       struct xdr_buf *buf = &req->rq_rcv_buf;
-       int status = ntohl(*p++);
-       struct posix_acl **acl;
-       unsigned int *aclcnt;
-       int err, base;
-
-       if (status != 0)
-               return nfs_stat_to_errno(status);
-       p = xdr_decode_post_op_attr(p, res->fattr);
-       res->mask = ntohl(*p++);
-       if (res->mask & ~(NFS_ACL|NFS_ACLCNT|NFS_DFACL|NFS_DFACLCNT))
-               return -EINVAL;
-       base = (char *)p - (char *)req->rq_rcv_buf.head->iov_base;
-
-       acl = (res->mask & NFS_ACL) ? &res->acl_access : NULL;
-       aclcnt = (res->mask & NFS_ACLCNT) ? &res->acl_access_count : NULL;
-       err = nfsacl_decode(buf, base, aclcnt, acl);
-
-       acl = (res->mask & NFS_DFACL) ? &res->acl_default : NULL;
-       aclcnt = (res->mask & NFS_DFACLCNT) ? &res->acl_default_count : NULL;
-       if (err > 0)
-               err = nfsacl_decode(buf, base + err, aclcnt, acl);
-       return (err > 0) ? 0 : err;
-}
 
 static inline int decode_getacl3resok(struct xdr_stream *xdr,
                                      struct nfs3_getaclres *result)
@@ -2957,54 +2387,38 @@ out:
        return error;
 }
 
-static int nfs3_xdr_dec_getacl3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_getacl3res(struct rpc_rqst *req,
+                                  struct xdr_stream *xdr,
                                   struct nfs3_getaclres *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
                goto out_default;
-       error = decode_getacl3resok(&xdr, result);
+       error = decode_getacl3resok(xdr, result);
 out:
        return error;
 out_default:
        return nfs_stat_to_errno(status);
 }
 
-/*
- * Decode setacl reply.
- */
-static int
-nfs3_xdr_setaclres(struct rpc_rqst *req, __be32 *p, struct nfs_fattr *fattr)
-{
-       int status = ntohl(*p++);
-
-       if (status)
-               return nfs_stat_to_errno(status);
-       xdr_decode_post_op_attr(p, fattr);
-       return 0;
-}
-
-static int nfs3_xdr_dec_setacl3res(struct rpc_rqst *req, __be32 *p,
+static int nfs3_xdr_dec_setacl3res(struct rpc_rqst *req,
+                                  struct xdr_stream *xdr,
                                   struct nfs_fattr *result)
 {
-       struct xdr_stream xdr;
        enum nfs_stat status;
        int error;
 
-       xdr_init_decode(&xdr, &req->rq_rcv_buf, p);
-       error = decode_nfsstat3(&xdr, &status);
+       error = decode_nfsstat3(xdr, &status);
        if (unlikely(error))
                goto out;
        if (status != NFS3_OK)
                goto out_default;
-       error = decode_post_op_attr(&xdr, result);
+       error = decode_post_op_attr(xdr, result);
 out:
        return error;
 out_default:
@@ -3016,37 +2430,37 @@ out_default:
 #define PROC(proc, argtype, restype, timer)                            \
 [NFS3PROC_##proc] = {                                                  \
        .p_proc      = NFS3PROC_##proc,                                 \
-       .p_encode    = (kxdrproc_t)nfs3_xdr_enc_##argtype##3args,       \
-       .p_decode    = (kxdrproc_t) nfs3_xdr_##restype,                 \
+       .p_encode    = (kxdreproc_t)nfs3_xdr_enc_##argtype##3args,      \
+       .p_decode    = (kxdrdproc_t)nfs3_xdr_dec_##restype##3res,       \
        .p_arglen    = NFS3_##argtype##args_sz,                         \
-       .p_replen    = NFS3_##restype##_sz,                             \
+       .p_replen    = NFS3_##restype##res_sz,                          \
        .p_timer     = timer,                                           \
        .p_statidx   = NFS3PROC_##proc,                                 \
        .p_name      = #proc,                                           \
        }
 
 struct rpc_procinfo    nfs3_procedures[] = {
-       PROC(GETATTR,           getattr,        attrstat,       1),
-       PROC(SETATTR,           setattr,        wccstat,        0),
-       PROC(LOOKUP,            lookup,         lookupres,      2),
-       PROC(ACCESS,            access,         accessres,      1),
-       PROC(READLINK,          readlink,       readlinkres,    3),
-       PROC(READ,              read,           readres,        3),
-       PROC(WRITE,             write,          writeres,       4),
-       PROC(CREATE,            create,         createres,      0),
-       PROC(MKDIR,             mkdir,          createres,      0),
-       PROC(SYMLINK,           symlink,        createres,      0),
-       PROC(MKNOD,             mknod,          createres,      0),
-       PROC(REMOVE,            remove,         removeres,      0),
-       PROC(RMDIR,             lookup,         wccstat,        0),
-       PROC(RENAME,            rename,         renameres,      0),
-       PROC(LINK,              link,           linkres,        0),
-       PROC(READDIR,           readdir,        readdirres,     3),
-       PROC(READDIRPLUS,       readdirplus,    readdirres,     3),
-       PROC(FSSTAT,            getattr,        fsstatres,      0),
-       PROC(FSINFO,            getattr,        fsinfores,      0),
-       PROC(PATHCONF,          getattr,        pathconfres,    0),
-       PROC(COMMIT,            commit,         commitres,      5),
+       PROC(GETATTR,           getattr,        getattr,        1),
+       PROC(SETATTR,           setattr,        setattr,        0),
+       PROC(LOOKUP,            lookup,         lookup,         2),
+       PROC(ACCESS,            access,         access,         1),
+       PROC(READLINK,          readlink,       readlink,       3),
+       PROC(READ,              read,           read,           3),
+       PROC(WRITE,             write,          write,          4),
+       PROC(CREATE,            create,         create,         0),
+       PROC(MKDIR,             mkdir,          create,         0),
+       PROC(SYMLINK,           symlink,        create,         0),
+       PROC(MKNOD,             mknod,          create,         0),
+       PROC(REMOVE,            remove,         remove,         0),
+       PROC(RMDIR,             lookup,         setattr,        0),
+       PROC(RENAME,            rename,         rename,         0),
+       PROC(LINK,              link,           link,           0),
+       PROC(READDIR,           readdir,        readdir,        3),
+       PROC(READDIRPLUS,       readdirplus,    readdir,        3),
+       PROC(FSSTAT,            getattr,        fsstat,         0),
+       PROC(FSINFO,            getattr,        fsinfo,         0),
+       PROC(PATHCONF,          getattr,        pathconf,       0),
+       PROC(COMMIT,            commit,         commit,         5),
 };
 
 struct rpc_version             nfs_version3 = {
@@ -3059,8 +2473,8 @@ struct rpc_version                nfs_version3 = {
 static struct rpc_procinfo     nfs3_acl_procedures[] = {
        [ACLPROC3_GETACL] = {
                .p_proc = ACLPROC3_GETACL,
-               .p_encode = (kxdrproc_t)nfs3_xdr_enc_getacl3args,
-               .p_decode = (kxdrproc_t) nfs3_xdr_getaclres,
+               .p_encode = (kxdreproc_t)nfs3_xdr_enc_getacl3args,
+               .p_decode = (kxdrdproc_t)nfs3_xdr_dec_getacl3res,
                .p_arglen = ACL3_getaclargs_sz,
                .p_replen = ACL3_getaclres_sz,
                .p_timer = 1,
@@ -3068,8 +2482,8 @@ static struct rpc_procinfo        nfs3_acl_procedures[] = {
        },
        [ACLPROC3_SETACL] = {
                .p_proc = ACLPROC3_SETACL,
-               .p_encode = (kxdrproc_t)nfs3_xdr_enc_setacl3args,
-               .p_decode = (kxdrproc_t) nfs3_xdr_setaclres,
+               .p_encode = (kxdreproc_t)nfs3_xdr_enc_setacl3args,
+               .p_decode = (kxdrdproc_t)nfs3_xdr_dec_setacl3res,
                .p_arglen = ACL3_setaclargs_sz,
                .p_replen = ACL3_setaclres_sz,
                .p_timer = 0,