nfs: nfs4xdr: simplify decode_exchange_id by reusing decode_opaque_inline
[safe/jmp/linux-2.6] / fs / nfs / nfs4xdr.c
index 9a03b24..3f49da0 100644 (file)
@@ -702,28 +702,12 @@ struct compound_hdr {
        u32             minorversion;
 };
 
-/*
- * START OF "GENERIC" ENCODE ROUTINES.
- *   These may look a little ugly since they are imported from a "generic"
- * set of XDR encode/decode routines which are intended to be shared by
- * all of our NFSv4 implementations (OpenBSD, MacOS X...).
- *
- * If the pain of reading these is too great, it should be a straightforward
- * task to translate them into Linux-specific versions which are more
- * consistent with the style used in NFSv2/v3...
- */
-#define WRITE64(n)               do {                          \
-       *p++ = htonl((uint32_t)((n) >> 32));                            \
-       *p++ = htonl((uint32_t)(n));                                    \
-} while (0)
-#define WRITEMEM(ptr,nbytes)     do {                          \
-       p = xdr_encode_opaque_fixed(p, ptr, nbytes);            \
-} while (0)
-
-#define RESERVE_SPACE(nbytes)  do {                            \
-       p = xdr_reserve_space(xdr, nbytes);                     \
-       BUG_ON(!p);                                             \
-} while (0)
+static __be32 *reserve_space(struct xdr_stream *xdr, size_t nbytes)
+{
+       __be32 *p = xdr_reserve_space(xdr, nbytes);
+       BUG_ON(!p);
+       return p;
+}
 
 static void encode_string(struct xdr_stream *xdr, unsigned int len, const char *str)
 {
@@ -748,12 +732,11 @@ static void encode_compound_hdr(struct xdr_stream *xdr,
 
        dprintk("encode_compound: tag=%.*s\n", (int)hdr->taglen, hdr->tag);
        BUG_ON(hdr->taglen > NFS4_MAXTAGLEN);
-       RESERVE_SPACE(12+(XDR_QUADLEN(hdr->taglen)<<2));
-       *p++ = cpu_to_be32(hdr->taglen);
-       WRITEMEM(hdr->tag, hdr->taglen);
+       p = reserve_space(xdr, 4 + hdr->taglen + 8);
+       p = xdr_encode_opaque(p, hdr->tag, hdr->taglen);
        *p++ = cpu_to_be32(hdr->minorversion);
        hdr->nops_p = p;
-       *p++ = cpu_to_be32(hdr->nops);
+       *p = cpu_to_be32(hdr->nops);
 }
 
 static void encode_nops(struct compound_hdr *hdr)
@@ -828,7 +811,7 @@ static void encode_attrs(struct xdr_stream *xdr, const struct iattr *iap, const
                len += 16;
        else if (iap->ia_valid & ATTR_MTIME)
                len += 4;
-       RESERVE_SPACE(len);
+       p = reserve_space(xdr, len);
 
        /*
         * We write the bitmap length now, but leave the bitmap and the attribute
@@ -840,7 +823,7 @@ static void encode_attrs(struct xdr_stream *xdr, const struct iattr *iap, const
 
        if (iap->ia_valid & ATTR_SIZE) {
                bmval0 |= FATTR4_WORD0_SIZE;
-               WRITE64(iap->ia_size);
+               p = xdr_encode_hyper(p, iap->ia_size);
        }
        if (iap->ia_valid & ATTR_MODE) {
                bmval1 |= FATTR4_WORD1_MODE;
@@ -848,13 +831,11 @@ static void encode_attrs(struct xdr_stream *xdr, const struct iattr *iap, const
        }
        if (iap->ia_valid & ATTR_UID) {
                bmval1 |= FATTR4_WORD1_OWNER;
-               *p++ = cpu_to_be32(owner_namelen);
-               WRITEMEM(owner_name, owner_namelen);
+               p = xdr_encode_opaque(p, owner_name, owner_namelen);
        }
        if (iap->ia_valid & ATTR_GID) {
                bmval1 |= FATTR4_WORD1_OWNER_GROUP;
-               *p++ = cpu_to_be32(owner_grouplen);
-               WRITEMEM(owner_group, owner_grouplen);
+               p = xdr_encode_opaque(p, owner_group, owner_grouplen);
        }
        if (iap->ia_valid & ATTR_ATIME_SET) {
                bmval1 |= FATTR4_WORD1_TIME_ACCESS_SET;
@@ -890,7 +871,7 @@ static void encode_attrs(struct xdr_stream *xdr, const struct iattr *iap, const
        len = (char *)p - (char *)q - 12;
        *q++ = htonl(bmval0);
        *q++ = htonl(bmval1);
-       *q++ = htonl(len);
+       *q = htonl(len);
 
 /* out: */
 }
@@ -899,9 +880,9 @@ static void encode_access(struct xdr_stream *xdr, u32 access, struct compound_hd
 {
        __be32 *p;
 
-       RESERVE_SPACE(8);
+       p = reserve_space(xdr, 8);
        *p++ = cpu_to_be32(OP_ACCESS);
-       *p++ = cpu_to_be32(access);
+       *p = cpu_to_be32(access);
        hdr->nops++;
        hdr->replen += decode_access_maxsz;
 }
@@ -910,10 +891,10 @@ static void encode_close(struct xdr_stream *xdr, const struct nfs_closeargs *arg
 {
        __be32 *p;
 
-       RESERVE_SPACE(8+NFS4_STATEID_SIZE);
+       p = reserve_space(xdr, 8+NFS4_STATEID_SIZE);
        *p++ = cpu_to_be32(OP_CLOSE);
        *p++ = cpu_to_be32(arg->seqid->sequence->counter);
-       WRITEMEM(arg->stateid->data, NFS4_STATEID_SIZE);
+       xdr_encode_opaque_fixed(p, arg->stateid->data, NFS4_STATEID_SIZE);
        hdr->nops++;
        hdr->replen += decode_close_maxsz;
 }
@@ -922,10 +903,10 @@ static void encode_commit(struct xdr_stream *xdr, const struct nfs_writeargs *ar
 {
        __be32 *p;
 
-       RESERVE_SPACE(16);
+       p = reserve_space(xdr, 16);
        *p++ = cpu_to_be32(OP_COMMIT);
-       WRITE64(args->offset);
-       *p++ = cpu_to_be32(args->count);
+       p = xdr_encode_hyper(p, args->offset);
+       *p = cpu_to_be32(args->count);
        hdr->nops++;
        hdr->replen += decode_commit_maxsz;
 }
@@ -934,30 +915,28 @@ static void encode_create(struct xdr_stream *xdr, const struct nfs4_create_arg *
 {
        __be32 *p;
 
-       RESERVE_SPACE(8);
+       p = reserve_space(xdr, 8);
        *p++ = cpu_to_be32(OP_CREATE);
-       *p++ = cpu_to_be32(create->ftype);
+       *p = cpu_to_be32(create->ftype);
 
        switch (create->ftype) {
        case NF4LNK:
-               RESERVE_SPACE(4);
-               *p++ = cpu_to_be32(create->u.symlink.len);
+               p = reserve_space(xdr, 4);
+               *p = cpu_to_be32(create->u.symlink.len);
                xdr_write_pages(xdr, create->u.symlink.pages, 0, create->u.symlink.len);
                break;
 
        case NF4BLK: case NF4CHR:
-               RESERVE_SPACE(8);
+               p = reserve_space(xdr, 8);
                *p++ = cpu_to_be32(create->u.device.specdata1);
-               *p++ = cpu_to_be32(create->u.device.specdata2);
+               *p = cpu_to_be32(create->u.device.specdata2);
                break;
 
        default:
                break;
        }
 
-       RESERVE_SPACE(4 + create->name->len);
-       *p++ = cpu_to_be32(create->name->len);
-       WRITEMEM(create->name->name, create->name->len);
+       encode_string(xdr, create->name->len, create->name->name);
        hdr->nops++;
        hdr->replen += decode_create_maxsz;
 
@@ -968,10 +947,10 @@ static void encode_getattr_one(struct xdr_stream *xdr, uint32_t bitmap, struct c
 {
        __be32 *p;
 
-       RESERVE_SPACE(12);
+       p = reserve_space(xdr, 12);
        *p++ = cpu_to_be32(OP_GETATTR);
        *p++ = cpu_to_be32(1);
-       *p++ = cpu_to_be32(bitmap);
+       *p = cpu_to_be32(bitmap);
        hdr->nops++;
        hdr->replen += decode_getattr_maxsz;
 }
@@ -980,11 +959,11 @@ static void encode_getattr_two(struct xdr_stream *xdr, uint32_t bm0, uint32_t bm
 {
        __be32 *p;
 
-       RESERVE_SPACE(16);
+       p = reserve_space(xdr, 16);
        *p++ = cpu_to_be32(OP_GETATTR);
        *p++ = cpu_to_be32(2);
        *p++ = cpu_to_be32(bm0);
-       *p++ = cpu_to_be32(bm1);
+       *p = cpu_to_be32(bm1);
        hdr->nops++;
        hdr->replen += decode_getattr_maxsz;
 }
@@ -1011,8 +990,8 @@ static void encode_getfh(struct xdr_stream *xdr, struct compound_hdr *hdr)
 {
        __be32 *p;
 
-       RESERVE_SPACE(4);
-       *p++ = cpu_to_be32(OP_GETFH);
+       p = reserve_space(xdr, 4);
+       *p = cpu_to_be32(OP_GETFH);
        hdr->nops++;
        hdr->replen += decode_getfh_maxsz;
 }
@@ -1021,10 +1000,9 @@ static void encode_link(struct xdr_stream *xdr, const struct qstr *name, struct
 {
        __be32 *p;
 
-       RESERVE_SPACE(8 + name->len);
+       p = reserve_space(xdr, 8 + name->len);
        *p++ = cpu_to_be32(OP_LINK);
-       *p++ = cpu_to_be32(name->len);
-       WRITEMEM(name->name, name->len);
+       xdr_encode_opaque(p, name->name, name->len);
        hdr->nops++;
        hdr->replen += decode_link_maxsz;
 }
@@ -1051,27 +1029,27 @@ static void encode_lock(struct xdr_stream *xdr, const struct nfs_lock_args *args
 {
        __be32 *p;
 
-       RESERVE_SPACE(32);
+       p = reserve_space(xdr, 32);
        *p++ = cpu_to_be32(OP_LOCK);
        *p++ = cpu_to_be32(nfs4_lock_type(args->fl, args->block));
        *p++ = cpu_to_be32(args->reclaim);
-       WRITE64(args->fl->fl_start);
-       WRITE64(nfs4_lock_length(args->fl));
-       *p++ = cpu_to_be32(args->new_lock_owner);
+       p = xdr_encode_hyper(p, args->fl->fl_start);
+       p = xdr_encode_hyper(p, nfs4_lock_length(args->fl));
+       *p = cpu_to_be32(args->new_lock_owner);
        if (args->new_lock_owner){
-               RESERVE_SPACE(4+NFS4_STATEID_SIZE+32);
+               p = reserve_space(xdr, 4+NFS4_STATEID_SIZE+32);
                *p++ = cpu_to_be32(args->open_seqid->sequence->counter);
-               WRITEMEM(args->open_stateid->data, NFS4_STATEID_SIZE);
+               p = xdr_encode_opaque_fixed(p, args->open_stateid->data, NFS4_STATEID_SIZE);
                *p++ = cpu_to_be32(args->lock_seqid->sequence->counter);
-               WRITE64(args->lock_owner.clientid);
+               p = xdr_encode_hyper(p, args->lock_owner.clientid);
                *p++ = cpu_to_be32(16);
-               WRITEMEM("lock id:", 8);
-               WRITE64(args->lock_owner.id);
+               p = xdr_encode_opaque_fixed(p, "lock id:", 8);
+               xdr_encode_hyper(p, args->lock_owner.id);
        }
        else {
-               RESERVE_SPACE(NFS4_STATEID_SIZE+4);
-               WRITEMEM(args->lock_stateid->data, NFS4_STATEID_SIZE);
-               *p++ = cpu_to_be32(args->lock_seqid->sequence->counter);
+               p = reserve_space(xdr, NFS4_STATEID_SIZE+4);
+               p = xdr_encode_opaque_fixed(p, args->lock_stateid->data, NFS4_STATEID_SIZE);
+               *p = cpu_to_be32(args->lock_seqid->sequence->counter);
        }
        hdr->nops++;
        hdr->replen += decode_lock_maxsz;
@@ -1081,15 +1059,15 @@ static void encode_lockt(struct xdr_stream *xdr, const struct nfs_lockt_args *ar
 {
        __be32 *p;
 
-       RESERVE_SPACE(52);
+       p = reserve_space(xdr, 52);
        *p++ = cpu_to_be32(OP_LOCKT);
        *p++ = cpu_to_be32(nfs4_lock_type(args->fl, 0));
-       WRITE64(args->fl->fl_start);
-       WRITE64(nfs4_lock_length(args->fl));
-       WRITE64(args->lock_owner.clientid);
+       p = xdr_encode_hyper(p, args->fl->fl_start);
+       p = xdr_encode_hyper(p, nfs4_lock_length(args->fl));
+       p = xdr_encode_hyper(p, args->lock_owner.clientid);
        *p++ = cpu_to_be32(16);
-       WRITEMEM("lock id:", 8);
-       WRITE64(args->lock_owner.id);
+       p = xdr_encode_opaque_fixed(p, "lock id:", 8);
+       xdr_encode_hyper(p, args->lock_owner.id);
        hdr->nops++;
        hdr->replen += decode_lockt_maxsz;
 }
@@ -1098,13 +1076,13 @@ static void encode_locku(struct xdr_stream *xdr, const struct nfs_locku_args *ar
 {
        __be32 *p;
 
-       RESERVE_SPACE(12+NFS4_STATEID_SIZE+16);
+       p = reserve_space(xdr, 12+NFS4_STATEID_SIZE+16);
        *p++ = cpu_to_be32(OP_LOCKU);
        *p++ = cpu_to_be32(nfs4_lock_type(args->fl, 0));
        *p++ = cpu_to_be32(args->seqid->sequence->counter);
-       WRITEMEM(args->stateid->data, NFS4_STATEID_SIZE);
-       WRITE64(args->fl->fl_start);
-       WRITE64(nfs4_lock_length(args->fl));
+       p = xdr_encode_opaque_fixed(p, args->stateid->data, NFS4_STATEID_SIZE);
+       p = xdr_encode_hyper(p, args->fl->fl_start);
+       xdr_encode_hyper(p, nfs4_lock_length(args->fl));
        hdr->nops++;
        hdr->replen += decode_locku_maxsz;
 }
@@ -1114,10 +1092,9 @@ static void encode_lookup(struct xdr_stream *xdr, const struct qstr *name, struc
        int len = name->len;
        __be32 *p;
 
-       RESERVE_SPACE(8 + len);
+       p = reserve_space(xdr, 8 + len);
        *p++ = cpu_to_be32(OP_LOOKUP);
-       *p++ = cpu_to_be32(len);
-       WRITEMEM(name->name, len);
+       xdr_encode_opaque(p, name->name, len);
        hdr->nops++;
        hdr->replen += decode_lookup_maxsz;
 }
@@ -1126,7 +1103,7 @@ static void encode_share_access(struct xdr_stream *xdr, fmode_t fmode)
 {
        __be32 *p;
 
-       RESERVE_SPACE(8);
+       p = reserve_space(xdr, 8);
        switch (fmode & (FMODE_READ|FMODE_WRITE)) {
        case FMODE_READ:
                *p++ = cpu_to_be32(NFS4_SHARE_ACCESS_READ);
@@ -1140,7 +1117,7 @@ static void encode_share_access(struct xdr_stream *xdr, fmode_t fmode)
        default:
                *p++ = cpu_to_be32(0);
        }
-       *p++ = cpu_to_be32(0);          /* for linux, share_deny = 0 always */
+       *p = cpu_to_be32(0);            /* for linux, share_deny = 0 always */
 }
 
 static inline void encode_openhdr(struct xdr_stream *xdr, const struct nfs_openargs *arg)
@@ -1150,29 +1127,29 @@ static inline void encode_openhdr(struct xdr_stream *xdr, const struct nfs_opena
  * opcode 4, seqid 4, share_access 4, share_deny 4, clientid 8, ownerlen 4,
  * owner 4 = 32
  */
-       RESERVE_SPACE(8);
+       p = reserve_space(xdr, 8);
        *p++ = cpu_to_be32(OP_OPEN);
-       *p++ = cpu_to_be32(arg->seqid->sequence->counter);
+       *p = cpu_to_be32(arg->seqid->sequence->counter);
        encode_share_access(xdr, arg->fmode);
-       RESERVE_SPACE(28);
-       WRITE64(arg->clientid);
+       p = reserve_space(xdr, 28);
+       p = xdr_encode_hyper(p, arg->clientid);
        *p++ = cpu_to_be32(16);
-       WRITEMEM("open id:", 8);
-       WRITE64(arg->id);
+       p = xdr_encode_opaque_fixed(p, "open id:", 8);
+       xdr_encode_hyper(p, arg->id);
 }
 
 static inline void encode_createmode(struct xdr_stream *xdr, const struct nfs_openargs *arg)
 {
        __be32 *p;
 
-       RESERVE_SPACE(4);
+       p = reserve_space(xdr, 4);
        switch(arg->open_flags & O_EXCL) {
        case 0:
-               *p++ = cpu_to_be32(NFS4_CREATE_UNCHECKED);
+               *p = cpu_to_be32(NFS4_CREATE_UNCHECKED);
                encode_attrs(xdr, arg->u.attrs, arg->server);
                break;
        default:
-               *p++ = cpu_to_be32(NFS4_CREATE_EXCLUSIVE);
+               *p = cpu_to_be32(NFS4_CREATE_EXCLUSIVE);
                encode_nfs4_verifier(xdr, &arg->u.verifier);
        }
 }
@@ -1181,14 +1158,14 @@ static void encode_opentype(struct xdr_stream *xdr, const struct nfs_openargs *a
 {
        __be32 *p;
 
-       RESERVE_SPACE(4);
+       p = reserve_space(xdr, 4);
        switch (arg->open_flags & O_CREAT) {
        case 0:
-               *p++ = cpu_to_be32(NFS4_OPEN_NOCREATE);
+               *p = cpu_to_be32(NFS4_OPEN_NOCREATE);
                break;
        default:
                BUG_ON(arg->claim != NFS4_OPEN_CLAIM_NULL);
-               *p++ = cpu_to_be32(NFS4_OPEN_CREATE);
+               *p = cpu_to_be32(NFS4_OPEN_CREATE);
                encode_createmode(xdr, arg);
        }
 }
@@ -1197,16 +1174,16 @@ static inline void encode_delegation_type(struct xdr_stream *xdr, fmode_t delega
 {
        __be32 *p;
 
-       RESERVE_SPACE(4);
+       p = reserve_space(xdr, 4);
        switch (delegation_type) {
        case 0:
-               *p++ = cpu_to_be32(NFS4_OPEN_DELEGATE_NONE);
+               *p = cpu_to_be32(NFS4_OPEN_DELEGATE_NONE);
                break;
        case FMODE_READ:
-               *p++ = cpu_to_be32(NFS4_OPEN_DELEGATE_READ);
+               *p = cpu_to_be32(NFS4_OPEN_DELEGATE_READ);
                break;
        case FMODE_WRITE|FMODE_READ:
-               *p++ = cpu_to_be32(NFS4_OPEN_DELEGATE_WRITE);
+               *p = cpu_to_be32(NFS4_OPEN_DELEGATE_WRITE);
                break;
        default:
                BUG();
@@ -1217,8 +1194,8 @@ static inline void encode_claim_null(struct xdr_stream *xdr, const struct qstr *
 {
        __be32 *p;
 
-       RESERVE_SPACE(4);
-       *p++ = cpu_to_be32(NFS4_OPEN_CLAIM_NULL);
+       p = reserve_space(xdr, 4);
+       *p = cpu_to_be32(NFS4_OPEN_CLAIM_NULL);
        encode_string(xdr, name->len, name->name);
 }
 
@@ -1226,8 +1203,8 @@ static inline void encode_claim_previous(struct xdr_stream *xdr, fmode_t type)
 {
        __be32 *p;
 
-       RESERVE_SPACE(4);
-       *p++ = cpu_to_be32(NFS4_OPEN_CLAIM_PREVIOUS);
+       p = reserve_space(xdr, 4);
+       *p = cpu_to_be32(NFS4_OPEN_CLAIM_PREVIOUS);
        encode_delegation_type(xdr, type);
 }
 
@@ -1235,9 +1212,9 @@ static inline void encode_claim_delegate_cur(struct xdr_stream *xdr, const struc
 {
        __be32 *p;
 
-       RESERVE_SPACE(4+NFS4_STATEID_SIZE);
+       p = reserve_space(xdr, 4+NFS4_STATEID_SIZE);
        *p++ = cpu_to_be32(NFS4_OPEN_CLAIM_DELEGATE_CUR);
-       WRITEMEM(stateid->data, NFS4_STATEID_SIZE);
+       xdr_encode_opaque_fixed(p, stateid->data, NFS4_STATEID_SIZE);
        encode_string(xdr, name->len, name->name);
 }
 
@@ -1266,10 +1243,10 @@ static void encode_open_confirm(struct xdr_stream *xdr, const struct nfs_open_co
 {
        __be32 *p;
 
-       RESERVE_SPACE(4+NFS4_STATEID_SIZE+4);
+       p = reserve_space(xdr, 4+NFS4_STATEID_SIZE+4);
        *p++ = cpu_to_be32(OP_OPEN_CONFIRM);
-       WRITEMEM(arg->stateid->data, NFS4_STATEID_SIZE);
-       *p++ = cpu_to_be32(arg->seqid->sequence->counter);
+       p = xdr_encode_opaque_fixed(p, arg->stateid->data, NFS4_STATEID_SIZE);
+       *p = cpu_to_be32(arg->seqid->sequence->counter);
        hdr->nops++;
        hdr->replen += decode_open_confirm_maxsz;
 }
@@ -1278,10 +1255,10 @@ static void encode_open_downgrade(struct xdr_stream *xdr, const struct nfs_close
 {
        __be32 *p;
 
-       RESERVE_SPACE(4+NFS4_STATEID_SIZE+4);
+       p = reserve_space(xdr, 4+NFS4_STATEID_SIZE+4);
        *p++ = cpu_to_be32(OP_OPEN_DOWNGRADE);
-       WRITEMEM(arg->stateid->data, NFS4_STATEID_SIZE);
-       *p++ = cpu_to_be32(arg->seqid->sequence->counter);
+       p = xdr_encode_opaque_fixed(p, arg->stateid->data, NFS4_STATEID_SIZE);
+       *p = cpu_to_be32(arg->seqid->sequence->counter);
        encode_share_access(xdr, arg->fmode);
        hdr->nops++;
        hdr->replen += decode_open_downgrade_maxsz;
@@ -1293,10 +1270,9 @@ encode_putfh(struct xdr_stream *xdr, const struct nfs_fh *fh, struct compound_hd
        int len = fh->size;
        __be32 *p;
 
-       RESERVE_SPACE(8 + len);
+       p = reserve_space(xdr, 8 + len);
        *p++ = cpu_to_be32(OP_PUTFH);
-       *p++ = cpu_to_be32(len);
-       WRITEMEM(fh->data, len);
+       xdr_encode_opaque(p, fh->data, len);
        hdr->nops++;
        hdr->replen += decode_putfh_maxsz;
 }
@@ -1305,8 +1281,8 @@ static void encode_putrootfh(struct xdr_stream *xdr, struct compound_hdr *hdr)
 {
        __be32 *p;
 
-       RESERVE_SPACE(4);
-       *p++ = cpu_to_be32(OP_PUTROOTFH);
+       p = reserve_space(xdr, 4);
+       *p = cpu_to_be32(OP_PUTROOTFH);
        hdr->nops++;
        hdr->replen += decode_putrootfh_maxsz;
 }
@@ -1316,26 +1292,26 @@ static void encode_stateid(struct xdr_stream *xdr, const struct nfs_open_context
        nfs4_stateid stateid;
        __be32 *p;
 
-       RESERVE_SPACE(NFS4_STATEID_SIZE);
+       p = reserve_space(xdr, NFS4_STATEID_SIZE);
        if (ctx->state != NULL) {
                nfs4_copy_stateid(&stateid, ctx->state, ctx->lockowner);
-               WRITEMEM(stateid.data, NFS4_STATEID_SIZE);
+               xdr_encode_opaque_fixed(p, stateid.data, NFS4_STATEID_SIZE);
        } else
-               WRITEMEM(zero_stateid.data, NFS4_STATEID_SIZE);
+               xdr_encode_opaque_fixed(p, zero_stateid.data, NFS4_STATEID_SIZE);
 }
 
 static void encode_read(struct xdr_stream *xdr, const struct nfs_readargs *args, struct compound_hdr *hdr)
 {
        __be32 *p;
 
-       RESERVE_SPACE(4);
-       *p++ = cpu_to_be32(OP_READ);
+       p = reserve_space(xdr, 4);
+       *p = cpu_to_be32(OP_READ);
 
        encode_stateid(xdr, args->context);
 
-       RESERVE_SPACE(12);
-       WRITE64(args->offset);
-       *p++ = cpu_to_be32(args->count);
+       p = reserve_space(xdr, 12);
+       p = xdr_encode_hyper(p, args->offset);
+       *p = cpu_to_be32(args->count);
        hdr->nops++;
        hdr->replen += decode_read_maxsz;
 }
@@ -1348,10 +1324,10 @@ static void encode_readdir(struct xdr_stream *xdr, const struct nfs4_readdir_arg
        };
        __be32 *p;
 
-       RESERVE_SPACE(12+NFS4_VERIFIER_SIZE+20);
+       p = reserve_space(xdr, 12+NFS4_VERIFIER_SIZE+20);
        *p++ = cpu_to_be32(OP_READDIR);
-       WRITE64(readdir->cookie);
-       WRITEMEM(readdir->verifier.data, NFS4_VERIFIER_SIZE);
+       p = xdr_encode_hyper(p, readdir->cookie);
+       p = xdr_encode_opaque_fixed(p, readdir->verifier.data, NFS4_VERIFIER_SIZE);
        *p++ = cpu_to_be32(readdir->count >> 1);  /* We're not doing readdirplus */
        *p++ = cpu_to_be32(readdir->count);
        *p++ = cpu_to_be32(2);
@@ -1361,7 +1337,7 @@ static void encode_readdir(struct xdr_stream *xdr, const struct nfs4_readdir_arg
        else
                attrs[1] &= ~FATTR4_WORD1_MOUNTED_ON_FILEID;
        *p++ = cpu_to_be32(attrs[0] & readdir->bitmask[0]);
-       *p++ = cpu_to_be32(attrs[1] & readdir->bitmask[1]);
+       *p = cpu_to_be32(attrs[1] & readdir->bitmask[1]);
        hdr->nops++;
        hdr->replen += decode_readdir_maxsz;
        dprintk("%s: cookie = %Lu, verifier = %08x:%08x, bitmap = %08x:%08x\n",
@@ -1377,8 +1353,8 @@ static void encode_readlink(struct xdr_stream *xdr, const struct nfs4_readlink *
 {
        __be32 *p;
 
-       RESERVE_SPACE(4);
-       *p++ = cpu_to_be32(OP_READLINK);
+       p = reserve_space(xdr, 4);
+       *p = cpu_to_be32(OP_READLINK);
        hdr->nops++;
        hdr->replen += decode_readlink_maxsz;
 }
@@ -1387,10 +1363,9 @@ static void encode_remove(struct xdr_stream *xdr, const struct qstr *name, struc
 {
        __be32 *p;
 
-       RESERVE_SPACE(8 + name->len);
+       p = reserve_space(xdr, 8 + name->len);
        *p++ = cpu_to_be32(OP_REMOVE);
-       *p++ = cpu_to_be32(name->len);
-       WRITEMEM(name->name, name->len);
+       xdr_encode_opaque(p, name->name, name->len);
        hdr->nops++;
        hdr->replen += decode_remove_maxsz;
 }
@@ -1399,14 +1374,10 @@ static void encode_rename(struct xdr_stream *xdr, const struct qstr *oldname, co
 {
        __be32 *p;
 
-       RESERVE_SPACE(8 + oldname->len);
-       *p++ = cpu_to_be32(OP_RENAME);
-       *p++ = cpu_to_be32(oldname->len);
-       WRITEMEM(oldname->name, oldname->len);
-
-       RESERVE_SPACE(4 + newname->len);
-       *p++ = cpu_to_be32(newname->len);
-       WRITEMEM(newname->name, newname->len);
+       p = reserve_space(xdr, 4);
+       *p = cpu_to_be32(OP_RENAME);
+       encode_string(xdr, oldname->len, oldname->name);
+       encode_string(xdr, newname->len, newname->name);
        hdr->nops++;
        hdr->replen += decode_rename_maxsz;
 }
@@ -1415,9 +1386,9 @@ static void encode_renew(struct xdr_stream *xdr, const struct nfs_client *client
 {
        __be32 *p;
 
-       RESERVE_SPACE(12);
+       p = reserve_space(xdr, 12);
        *p++ = cpu_to_be32(OP_RENEW);
-       WRITE64(client_stateid->cl_clientid);
+       xdr_encode_hyper(p, client_stateid->cl_clientid);
        hdr->nops++;
        hdr->replen += decode_renew_maxsz;
 }
@@ -1427,8 +1398,8 @@ encode_restorefh(struct xdr_stream *xdr, struct compound_hdr *hdr)
 {
        __be32 *p;
 
-       RESERVE_SPACE(4);
-       *p++ = cpu_to_be32(OP_RESTOREFH);
+       p = reserve_space(xdr, 4);
+       *p = cpu_to_be32(OP_RESTOREFH);
        hdr->nops++;
        hdr->replen += decode_restorefh_maxsz;
 }
@@ -1438,16 +1409,16 @@ encode_setacl(struct xdr_stream *xdr, struct nfs_setaclargs *arg, struct compoun
 {
        __be32 *p;
 
-       RESERVE_SPACE(4+NFS4_STATEID_SIZE);
+       p = reserve_space(xdr, 4+NFS4_STATEID_SIZE);
        *p++ = cpu_to_be32(OP_SETATTR);
-       WRITEMEM(zero_stateid.data, NFS4_STATEID_SIZE);
-       RESERVE_SPACE(2*4);
+       xdr_encode_opaque_fixed(p, zero_stateid.data, NFS4_STATEID_SIZE);
+       p = reserve_space(xdr, 2*4);
        *p++ = cpu_to_be32(1);
-       *p++ = cpu_to_be32(FATTR4_WORD0_ACL);
+       *p = cpu_to_be32(FATTR4_WORD0_ACL);
        if (arg->acl_len % 4)
                return -EINVAL;
-       RESERVE_SPACE(4);
-       *p++ = cpu_to_be32(arg->acl_len);
+       p = reserve_space(xdr, 4);
+       *p = cpu_to_be32(arg->acl_len);
        xdr_write_pages(xdr, arg->acl_pages, arg->acl_pgbase, arg->acl_len);
        hdr->nops++;
        hdr->replen += decode_setacl_maxsz;
@@ -1459,8 +1430,8 @@ encode_savefh(struct xdr_stream *xdr, struct compound_hdr *hdr)
 {
        __be32 *p;
 
-       RESERVE_SPACE(4);
-       *p++ = cpu_to_be32(OP_SAVEFH);
+       p = reserve_space(xdr, 4);
+       *p = cpu_to_be32(OP_SAVEFH);
        hdr->nops++;
        hdr->replen += decode_savefh_maxsz;
 }
@@ -1469,9 +1440,9 @@ static void encode_setattr(struct xdr_stream *xdr, const struct nfs_setattrargs
 {
        __be32 *p;
 
-       RESERVE_SPACE(4+NFS4_STATEID_SIZE);
+       p = reserve_space(xdr, 4+NFS4_STATEID_SIZE);
        *p++ = cpu_to_be32(OP_SETATTR);
-       WRITEMEM(arg->stateid.data, NFS4_STATEID_SIZE);
+       xdr_encode_opaque_fixed(p, arg->stateid.data, NFS4_STATEID_SIZE);
        hdr->nops++;
        hdr->replen += decode_setattr_maxsz;
        encode_attrs(xdr, arg->iap, server);
@@ -1481,17 +1452,17 @@ static void encode_setclientid(struct xdr_stream *xdr, const struct nfs4_setclie
 {
        __be32 *p;
 
-       RESERVE_SPACE(4 + NFS4_VERIFIER_SIZE);
+       p = reserve_space(xdr, 4 + NFS4_VERIFIER_SIZE);
        *p++ = cpu_to_be32(OP_SETCLIENTID);
-       WRITEMEM(setclientid->sc_verifier->data, NFS4_VERIFIER_SIZE);
+       xdr_encode_opaque_fixed(p, setclientid->sc_verifier->data, NFS4_VERIFIER_SIZE);
 
        encode_string(xdr, setclientid->sc_name_len, setclientid->sc_name);
-       RESERVE_SPACE(4);
-       *p++ = cpu_to_be32(setclientid->sc_prog);
+       p = reserve_space(xdr, 4);
+       *p = cpu_to_be32(setclientid->sc_prog);
        encode_string(xdr, setclientid->sc_netid_len, setclientid->sc_netid);
        encode_string(xdr, setclientid->sc_uaddr_len, setclientid->sc_uaddr);
-       RESERVE_SPACE(4);
-       *p++ = cpu_to_be32(setclientid->sc_cb_ident);
+       p = reserve_space(xdr, 4);
+       *p = cpu_to_be32(setclientid->sc_cb_ident);
        hdr->nops++;
        hdr->replen += decode_setclientid_maxsz;
 }
@@ -1500,10 +1471,10 @@ static void encode_setclientid_confirm(struct xdr_stream *xdr, const struct nfs_
 {
        __be32 *p;
 
-       RESERVE_SPACE(12 + NFS4_VERIFIER_SIZE);
+       p = reserve_space(xdr, 12 + NFS4_VERIFIER_SIZE);
        *p++ = cpu_to_be32(OP_SETCLIENTID_CONFIRM);
-       WRITE64(client_state->cl_clientid);
-       WRITEMEM(client_state->cl_confirm.data, NFS4_VERIFIER_SIZE);
+       p = xdr_encode_hyper(p, client_state->cl_clientid);
+       xdr_encode_opaque_fixed(p, client_state->cl_confirm.data, NFS4_VERIFIER_SIZE);
        hdr->nops++;
        hdr->replen += decode_setclientid_confirm_maxsz;
 }
@@ -1512,15 +1483,15 @@ static void encode_write(struct xdr_stream *xdr, const struct nfs_writeargs *arg
 {
        __be32 *p;
 
-       RESERVE_SPACE(4);
-       *p++ = cpu_to_be32(OP_WRITE);
+       p = reserve_space(xdr, 4);
+       *p = cpu_to_be32(OP_WRITE);
 
        encode_stateid(xdr, args->context);
 
-       RESERVE_SPACE(16);
-       WRITE64(args->offset);
+       p = reserve_space(xdr, 16);
+       p = xdr_encode_hyper(p, args->offset);
        *p++ = cpu_to_be32(args->stable);
-       *p++ = cpu_to_be32(args->count);
+       *p = cpu_to_be32(args->count);
 
        xdr_write_pages(xdr, args->pages, args->pgbase, args->count);
        hdr->nops++;
@@ -1531,10 +1502,10 @@ static void encode_delegreturn(struct xdr_stream *xdr, const nfs4_stateid *state
 {
        __be32 *p;
 
-       RESERVE_SPACE(4+NFS4_STATEID_SIZE);
+       p = reserve_space(xdr, 4+NFS4_STATEID_SIZE);
 
        *p++ = cpu_to_be32(OP_DELEGRETURN);
-       WRITEMEM(stateid->data, NFS4_STATEID_SIZE);
+       xdr_encode_opaque_fixed(p, stateid->data, NFS4_STATEID_SIZE);
        hdr->nops++;
        hdr->replen += decode_delegreturn_maxsz;
 }
@@ -1547,16 +1518,16 @@ static void encode_exchange_id(struct xdr_stream *xdr,
 {
        __be32 *p;
 
-       RESERVE_SPACE(4 + sizeof(args->verifier->data));
+       p = reserve_space(xdr, 4 + sizeof(args->verifier->data));
        *p++ = cpu_to_be32(OP_EXCHANGE_ID);
-       WRITEMEM(args->verifier->data, sizeof(args->verifier->data));
+       xdr_encode_opaque_fixed(p, args->verifier->data, sizeof(args->verifier->data));
 
        encode_string(xdr, args->id_len, args->id);
 
-       RESERVE_SPACE(12);
+       p = reserve_space(xdr, 12);
        *p++ = cpu_to_be32(args->flags);
        *p++ = cpu_to_be32(0);  /* zero length state_protect4_a */
-       *p++ = cpu_to_be32(0);  /* zero length implementation id array */
+       *p = cpu_to_be32(0);    /* zero length implementation id array */
        hdr->nops++;
        hdr->replen += decode_exchange_id_maxsz;
 }
@@ -1570,17 +1541,15 @@ static void encode_create_session(struct xdr_stream *xdr,
        uint32_t len;
        struct nfs_client *clp = args->client;
 
-       RESERVE_SPACE(4);
-       *p++ = cpu_to_be32(OP_CREATE_SESSION);
-
-       RESERVE_SPACE(8);
-       WRITE64(clp->cl_ex_clid);
+       len = scnprintf(machine_name, sizeof(machine_name), "%s",
+                       clp->cl_ipaddr);
 
-       RESERVE_SPACE(8);
+       p = reserve_space(xdr, 20 + 2*28 + 20 + len + 12);
+       *p++ = cpu_to_be32(OP_CREATE_SESSION);
+       p = xdr_encode_hyper(p, clp->cl_ex_clid);
        *p++ = cpu_to_be32(clp->cl_seqid);                      /*Sequence id */
        *p++ = cpu_to_be32(args->flags);                        /*flags */
 
-       RESERVE_SPACE(2*28);                    /* 2 channel_attrs */
        /* Fore Channel */
        *p++ = cpu_to_be32(args->fc_attrs.headerpadsz); /* header padding size */
        *p++ = cpu_to_be32(args->fc_attrs.max_rqst_sz); /* max req size */
@@ -1599,26 +1568,16 @@ static void encode_create_session(struct xdr_stream *xdr,
        *p++ = cpu_to_be32(args->bc_attrs.max_reqs);    /* max requests */
        *p++ = cpu_to_be32(0);                          /* rdmachannel_attrs */
 
-       RESERVE_SPACE(4);
        *p++ = cpu_to_be32(args->cb_program);           /* cb_program */
-
-       RESERVE_SPACE(4);                       /* # of security flavors */
        *p++ = cpu_to_be32(1);
-
-       RESERVE_SPACE(4);
        *p++ = cpu_to_be32(RPC_AUTH_UNIX);                      /* auth_sys */
 
        /* authsys_parms rfc1831 */
-       RESERVE_SPACE(4);
        *p++ = cpu_to_be32((u32)clp->cl_boot_time.tv_nsec);     /* stamp */
-       len = scnprintf(machine_name, sizeof(machine_name), "%s",
-                       clp->cl_ipaddr);
-       RESERVE_SPACE(16 + len);
-       *p++ = cpu_to_be32(len);
-       WRITEMEM(machine_name, len);
+       p = xdr_encode_opaque(p, machine_name, len);
        *p++ = cpu_to_be32(0);                          /* UID */
        *p++ = cpu_to_be32(0);                          /* GID */
-       *p++ = cpu_to_be32(0);                          /* No more gids */
+       *p = cpu_to_be32(0);                            /* No more gids */
        hdr->nops++;
        hdr->replen += decode_create_session_maxsz;
 }
@@ -1628,9 +1587,9 @@ static void encode_destroy_session(struct xdr_stream *xdr,
                                   struct compound_hdr *hdr)
 {
        __be32 *p;
-       RESERVE_SPACE(4 + NFS4_MAX_SESSIONID_LEN);
+       p = reserve_space(xdr, 4 + NFS4_MAX_SESSIONID_LEN);
        *p++ = cpu_to_be32(OP_DESTROY_SESSION);
-       WRITEMEM(session->sess_id.data, NFS4_MAX_SESSIONID_LEN);
+       xdr_encode_opaque_fixed(p, session->sess_id.data, NFS4_MAX_SESSIONID_LEN);
        hdr->nops++;
        hdr->replen += decode_destroy_session_maxsz;
 }
@@ -1654,7 +1613,7 @@ static void encode_sequence(struct xdr_stream *xdr,
        WARN_ON(args->sa_slotid == NFS4_MAX_SLOT_TABLE);
        slot = tp->slots + args->sa_slotid;
 
-       RESERVE_SPACE(4);
+       p = reserve_space(xdr, 4 + NFS4_MAX_SESSIONID_LEN + 16);
        *p++ = cpu_to_be32(OP_SEQUENCE);
 
        /*
@@ -1669,12 +1628,11 @@ static void encode_sequence(struct xdr_stream *xdr,
                ((u32 *)session->sess_id.data)[3],
                slot->seq_nr, args->sa_slotid,
                tp->highest_used_slotid, args->sa_cache_this);
-       RESERVE_SPACE(NFS4_MAX_SESSIONID_LEN + 16);
-       WRITEMEM(session->sess_id.data, NFS4_MAX_SESSIONID_LEN);
+       p = xdr_encode_opaque_fixed(p, session->sess_id.data, NFS4_MAX_SESSIONID_LEN);
        *p++ = cpu_to_be32(slot->seq_nr);
        *p++ = cpu_to_be32(args->sa_slotid);
        *p++ = cpu_to_be32(tp->highest_used_slotid);
-       *p++ = cpu_to_be32(args->sa_cache_this);
+       *p = cpu_to_be32(args->sa_cache_this);
        hdr->nops++;
        hdr->replen += decode_sequence_maxsz;
 #endif /* CONFIG_NFS_V4_1 */
@@ -2475,38 +2433,27 @@ static int nfs4_xdr_enc_get_lease_time(struct rpc_rqst *req, uint32_t *p,
  * task to translate them into Linux-specific versions which are more
  * consistent with the style used in NFSv2/v3...
  */
-#define READ32(x)         (x) = ntohl(*p++)
-#define READ64(x)         do {                 \
-       (x) = (u64)ntohl(*p++) << 32;           \
-       (x) |= ntohl(*p++);                     \
-} while (0)
-#define READTIME(x)       do {                 \
-       p++;                                    \
-       (x.tv_sec) = ntohl(*p++);               \
-       (x.tv_nsec) = ntohl(*p++);              \
-} while (0)
-#define COPYMEM(x,nbytes) do {                 \
-       memcpy((x), p, nbytes);                 \
-       p += XDR_QUADLEN(nbytes);               \
-} while (0)
-
 #define READ_BUF(nbytes)  do { \
        p = xdr_inline_decode(xdr, nbytes); \
        if (unlikely(!p)) { \
-               dprintk("nfs: %s: prematurely hit end of receive" \
-                               " buffer\n", __func__); \
-               dprintk("nfs: %s: xdr->p=%p, bytes=%u, xdr->end=%p\n", \
-                               __func__, xdr->p, nbytes, xdr->end); \
+               print_overflow_msg(__func__, xdr); \
                return -EIO; \
        } \
 } while (0)
 
+static void print_overflow_msg(const char *func, const struct xdr_stream *xdr)
+{
+       dprintk("nfs: %s: prematurely hit end of receive buffer. "
+               "Remaining buffer length is %tu words.\n",
+               func, xdr->end - xdr->p);
+}
+
 static int decode_opaque_inline(struct xdr_stream *xdr, unsigned int *len, char **string)
 {
        __be32 *p;
 
        READ_BUF(4);
-       READ32(*len);
+       *len = be32_to_cpup(p++);
        READ_BUF(*len);
        *string = (char *)p;
        return 0;
@@ -2517,13 +2464,13 @@ static int decode_compound_hdr(struct xdr_stream *xdr, struct compound_hdr *hdr)
        __be32 *p;
 
        READ_BUF(8);
-       READ32(hdr->status);
-       READ32(hdr->taglen);
+       hdr->status = be32_to_cpup(p++);
+       hdr->taglen = be32_to_cpup(p++);
 
        READ_BUF(hdr->taglen + 4);
        hdr->tag = (char *)p;
        p += XDR_QUADLEN(hdr->taglen);
-       READ32(hdr->nops);
+       hdr->nops = be32_to_cpup(p++);
        if (unlikely(hdr->nops < 1))
                return nfs4_stat_to_errno(hdr->status);
        return 0;
@@ -2536,14 +2483,14 @@ static int decode_op_hdr(struct xdr_stream *xdr, enum nfs_opnum4 expected)
        int32_t nfserr;
 
        READ_BUF(8);
-       READ32(opnum);
+       opnum = be32_to_cpup(p++);
        if (opnum != expected) {
                dprintk("nfs: Server returned operation"
                        " %d but we issued a request for %d\n",
                                opnum, expected);
                return -EIO;
        }
-       READ32(nfserr);
+       nfserr = be32_to_cpup(p++);
        if (nfserr != NFS_OK)
                return nfs4_stat_to_errno(nfserr);
        return 0;
@@ -2566,14 +2513,14 @@ static int decode_attr_bitmap(struct xdr_stream *xdr, uint32_t *bitmap)
        __be32 *p;
 
        READ_BUF(4);
-       READ32(bmlen);
+       bmlen = be32_to_cpup(p++);
 
        bitmap[0] = bitmap[1] = 0;
        READ_BUF((bmlen << 2));
        if (bmlen > 0) {
-               READ32(bitmap[0]);
+               bitmap[0] = be32_to_cpup(p++);
                if (bmlen > 1)
-                       READ32(bitmap[1]);
+                       bitmap[1] = be32_to_cpup(p++);
        }
        return 0;
 }
@@ -2583,7 +2530,7 @@ static inline int decode_attr_length(struct xdr_stream *xdr, uint32_t *attrlen,
        __be32 *p;
 
        READ_BUF(4);
-       READ32(*attrlen);
+       *attrlen = be32_to_cpup(p++);
        *savep = xdr->p;
        return 0;
 }
@@ -2609,7 +2556,7 @@ static int decode_attr_type(struct xdr_stream *xdr, uint32_t *bitmap, uint32_t *
                return -EIO;
        if (likely(bitmap[0] & FATTR4_WORD0_TYPE)) {
                READ_BUF(4);
-               READ32(*type);
+               *type = be32_to_cpup(p++);
                if (*type < NF4REG || *type > NF4NAMEDATTR) {
                        dprintk("%s: bad type %d\n", __func__, *type);
                        return -EIO;
@@ -2631,7 +2578,7 @@ static int decode_attr_change(struct xdr_stream *xdr, uint32_t *bitmap, uint64_t
                return -EIO;
        if (likely(bitmap[0] & FATTR4_WORD0_CHANGE)) {
                READ_BUF(8);
-               READ64(*change);
+               p = xdr_decode_hyper(p, change);
                bitmap[0] &= ~FATTR4_WORD0_CHANGE;
                ret = NFS_ATTR_FATTR_CHANGE;
        }
@@ -2650,7 +2597,7 @@ static int decode_attr_size(struct xdr_stream *xdr, uint32_t *bitmap, uint64_t *
                return -EIO;
        if (likely(bitmap[0] & FATTR4_WORD0_SIZE)) {
                READ_BUF(8);
-               READ64(*size);
+               p = xdr_decode_hyper(p, size);
                bitmap[0] &= ~FATTR4_WORD0_SIZE;
                ret = NFS_ATTR_FATTR_SIZE;
        }
@@ -2667,7 +2614,7 @@ static int decode_attr_link_support(struct xdr_stream *xdr, uint32_t *bitmap, ui
                return -EIO;
        if (likely(bitmap[0] & FATTR4_WORD0_LINK_SUPPORT)) {
                READ_BUF(4);
-               READ32(*res);
+               *res = be32_to_cpup(p++);
                bitmap[0] &= ~FATTR4_WORD0_LINK_SUPPORT;
        }
        dprintk("%s: link support=%s\n", __func__, *res == 0 ? "false" : "true");
@@ -2683,7 +2630,7 @@ static int decode_attr_symlink_support(struct xdr_stream *xdr, uint32_t *bitmap,
                return -EIO;
        if (likely(bitmap[0] & FATTR4_WORD0_SYMLINK_SUPPORT)) {
                READ_BUF(4);
-               READ32(*res);
+               *res = be32_to_cpup(p++);
                bitmap[0] &= ~FATTR4_WORD0_SYMLINK_SUPPORT;
        }
        dprintk("%s: symlink support=%s\n", __func__, *res == 0 ? "false" : "true");
@@ -2701,8 +2648,8 @@ static int decode_attr_fsid(struct xdr_stream *xdr, uint32_t *bitmap, struct nfs
                return -EIO;
        if (likely(bitmap[0] & FATTR4_WORD0_FSID)) {
                READ_BUF(16);
-               READ64(fsid->major);
-               READ64(fsid->minor);
+               p = xdr_decode_hyper(p, &fsid->major);
+               p = xdr_decode_hyper(p, &fsid->minor);
                bitmap[0] &= ~FATTR4_WORD0_FSID;
                ret = NFS_ATTR_FATTR_FSID;
        }
@@ -2721,7 +2668,7 @@ static int decode_attr_lease_time(struct xdr_stream *xdr, uint32_t *bitmap, uint
                return -EIO;
        if (likely(bitmap[0] & FATTR4_WORD0_LEASE_TIME)) {
                READ_BUF(4);
-               READ32(*res);
+               *res = be32_to_cpup(p++);
                bitmap[0] &= ~FATTR4_WORD0_LEASE_TIME;
        }
        dprintk("%s: file size=%u\n", __func__, (unsigned int)*res);
@@ -2737,7 +2684,7 @@ static int decode_attr_aclsupport(struct xdr_stream *xdr, uint32_t *bitmap, uint
                return -EIO;
        if (likely(bitmap[0] & FATTR4_WORD0_ACLSUPPORT)) {
                READ_BUF(4);
-               READ32(*res);
+               *res = be32_to_cpup(p++);
                bitmap[0] &= ~FATTR4_WORD0_ACLSUPPORT;
        }
        dprintk("%s: ACLs supported=%u\n", __func__, (unsigned int)*res);
@@ -2754,7 +2701,7 @@ static int decode_attr_fileid(struct xdr_stream *xdr, uint32_t *bitmap, uint64_t
                return -EIO;
        if (likely(bitmap[0] & FATTR4_WORD0_FILEID)) {
                READ_BUF(8);
-               READ64(*fileid);
+               p = xdr_decode_hyper(p, fileid);
                bitmap[0] &= ~FATTR4_WORD0_FILEID;
                ret = NFS_ATTR_FATTR_FILEID;
        }
@@ -2772,7 +2719,7 @@ static int decode_attr_mounted_on_fileid(struct xdr_stream *xdr, uint32_t *bitma
                return -EIO;
        if (likely(bitmap[1] & FATTR4_WORD1_MOUNTED_ON_FILEID)) {
                READ_BUF(8);
-               READ64(*fileid);
+               p = xdr_decode_hyper(p, fileid);
                bitmap[1] &= ~FATTR4_WORD1_MOUNTED_ON_FILEID;
                ret = NFS_ATTR_FATTR_FILEID;
        }
@@ -2790,7 +2737,7 @@ static int decode_attr_files_avail(struct xdr_stream *xdr, uint32_t *bitmap, uin
                return -EIO;
        if (likely(bitmap[0] & FATTR4_WORD0_FILES_AVAIL)) {
                READ_BUF(8);
-               READ64(*res);
+               p = xdr_decode_hyper(p, res);
                bitmap[0] &= ~FATTR4_WORD0_FILES_AVAIL;
        }
        dprintk("%s: files avail=%Lu\n", __func__, (unsigned long long)*res);
@@ -2807,7 +2754,7 @@ static int decode_attr_files_free(struct xdr_stream *xdr, uint32_t *bitmap, uint
                return -EIO;
        if (likely(bitmap[0] & FATTR4_WORD0_FILES_FREE)) {
                READ_BUF(8);
-               READ64(*res);
+               p = xdr_decode_hyper(p, res);
                bitmap[0] &= ~FATTR4_WORD0_FILES_FREE;
        }
        dprintk("%s: files free=%Lu\n", __func__, (unsigned long long)*res);
@@ -2824,7 +2771,7 @@ static int decode_attr_files_total(struct xdr_stream *xdr, uint32_t *bitmap, uin
                return -EIO;
        if (likely(bitmap[0] & FATTR4_WORD0_FILES_TOTAL)) {
                READ_BUF(8);
-               READ64(*res);
+               p = xdr_decode_hyper(p, res);
                bitmap[0] &= ~FATTR4_WORD0_FILES_TOTAL;
        }
        dprintk("%s: files total=%Lu\n", __func__, (unsigned long long)*res);
@@ -2838,7 +2785,7 @@ static int decode_pathname(struct xdr_stream *xdr, struct nfs4_pathname *path)
        int status = 0;
 
        READ_BUF(4);
-       READ32(n);
+       n = be32_to_cpup(p++);
        if (n == 0)
                goto root_path;
        dprintk("path ");
@@ -2890,7 +2837,7 @@ static int decode_attr_fs_locations(struct xdr_stream *xdr, uint32_t *bitmap, st
        if (unlikely(status != 0))
                goto out;
        READ_BUF(4);
-       READ32(n);
+       n = be32_to_cpup(p++);
        if (n <= 0)
                goto out_eio;
        res->nlocations = 0;
@@ -2899,7 +2846,7 @@ static int decode_attr_fs_locations(struct xdr_stream *xdr, uint32_t *bitmap, st
                struct nfs4_fs_location *loc = &res->locations[res->nlocations];
 
                READ_BUF(4);
-               READ32(m);
+               m = be32_to_cpup(p++);
 
                loc->nservers = 0;
                dprintk("%s: servers ", __func__);
@@ -2953,7 +2900,7 @@ static int decode_attr_maxfilesize(struct xdr_stream *xdr, uint32_t *bitmap, uin
                return -EIO;
        if (likely(bitmap[0] & FATTR4_WORD0_MAXFILESIZE)) {
                READ_BUF(8);
-               READ64(*res);
+               p = xdr_decode_hyper(p, res);
                bitmap[0] &= ~FATTR4_WORD0_MAXFILESIZE;
        }
        dprintk("%s: maxfilesize=%Lu\n", __func__, (unsigned long long)*res);
@@ -2970,7 +2917,7 @@ static int decode_attr_maxlink(struct xdr_stream *xdr, uint32_t *bitmap, uint32_
                return -EIO;
        if (likely(bitmap[0] & FATTR4_WORD0_MAXLINK)) {
                READ_BUF(4);
-               READ32(*maxlink);
+               *maxlink = be32_to_cpup(p++);
                bitmap[0] &= ~FATTR4_WORD0_MAXLINK;
        }
        dprintk("%s: maxlink=%u\n", __func__, *maxlink);
@@ -2987,7 +2934,7 @@ static int decode_attr_maxname(struct xdr_stream *xdr, uint32_t *bitmap, uint32_
                return -EIO;
        if (likely(bitmap[0] & FATTR4_WORD0_MAXNAME)) {
                READ_BUF(4);
-               READ32(*maxname);
+               *maxname = be32_to_cpup(p++);
                bitmap[0] &= ~FATTR4_WORD0_MAXNAME;
        }
        dprintk("%s: maxname=%u\n", __func__, *maxname);
@@ -3005,7 +2952,7 @@ static int decode_attr_maxread(struct xdr_stream *xdr, uint32_t *bitmap, uint32_
        if (likely(bitmap[0] & FATTR4_WORD0_MAXREAD)) {
                uint64_t maxread;
                READ_BUF(8);
-               READ64(maxread);
+               p = xdr_decode_hyper(p, &maxread);
                if (maxread > 0x7FFFFFFF)
                        maxread = 0x7FFFFFFF;
                *res = (uint32_t)maxread;
@@ -3026,7 +2973,7 @@ static int decode_attr_maxwrite(struct xdr_stream *xdr, uint32_t *bitmap, uint32
        if (likely(bitmap[0] & FATTR4_WORD0_MAXWRITE)) {
                uint64_t maxwrite;
                READ_BUF(8);
-               READ64(maxwrite);
+               p = xdr_decode_hyper(p, &maxwrite);
                if (maxwrite > 0x7FFFFFFF)
                        maxwrite = 0x7FFFFFFF;
                *res = (uint32_t)maxwrite;
@@ -3047,7 +2994,7 @@ static int decode_attr_mode(struct xdr_stream *xdr, uint32_t *bitmap, umode_t *m
                return -EIO;
        if (likely(bitmap[1] & FATTR4_WORD1_MODE)) {
                READ_BUF(4);
-               READ32(tmp);
+               tmp = be32_to_cpup(p++);
                *mode = tmp & ~S_IFMT;
                bitmap[1] &= ~FATTR4_WORD1_MODE;
                ret = NFS_ATTR_FATTR_MODE;
@@ -3066,7 +3013,7 @@ static int decode_attr_nlink(struct xdr_stream *xdr, uint32_t *bitmap, uint32_t
                return -EIO;
        if (likely(bitmap[1] & FATTR4_WORD1_NUMLINKS)) {
                READ_BUF(4);
-               READ32(*nlink);
+               *nlink = be32_to_cpup(p++);
                bitmap[1] &= ~FATTR4_WORD1_NUMLINKS;
                ret = NFS_ATTR_FATTR_NLINK;
        }
@@ -3085,7 +3032,7 @@ static int decode_attr_owner(struct xdr_stream *xdr, uint32_t *bitmap, struct nf
                return -EIO;
        if (likely(bitmap[1] & FATTR4_WORD1_OWNER)) {
                READ_BUF(4);
-               READ32(len);
+               len = be32_to_cpup(p++);
                READ_BUF(len);
                if (len < XDR_MAX_NETOBJ) {
                        if (nfs_map_name_to_uid(clp, (char *)p, len, uid) == 0)
@@ -3113,7 +3060,7 @@ static int decode_attr_group(struct xdr_stream *xdr, uint32_t *bitmap, struct nf
                return -EIO;
        if (likely(bitmap[1] & FATTR4_WORD1_OWNER_GROUP)) {
                READ_BUF(4);
-               READ32(len);
+               len = be32_to_cpup(p++);
                READ_BUF(len);
                if (len < XDR_MAX_NETOBJ) {
                        if (nfs_map_group_to_gid(clp, (char *)p, len, gid) == 0)
@@ -3143,8 +3090,8 @@ static int decode_attr_rdev(struct xdr_stream *xdr, uint32_t *bitmap, dev_t *rde
                dev_t tmp;
 
                READ_BUF(8);
-               READ32(major);
-               READ32(minor);
+               major = be32_to_cpup(p++);
+               minor = be32_to_cpup(p++);
                tmp = MKDEV(major, minor);
                if (MAJOR(tmp) == major && MINOR(tmp) == minor)
                        *rdev = tmp;
@@ -3165,7 +3112,7 @@ static int decode_attr_space_avail(struct xdr_stream *xdr, uint32_t *bitmap, uin
                return -EIO;
        if (likely(bitmap[1] & FATTR4_WORD1_SPACE_AVAIL)) {
                READ_BUF(8);
-               READ64(*res);
+               p = xdr_decode_hyper(p, res);
                bitmap[1] &= ~FATTR4_WORD1_SPACE_AVAIL;
        }
        dprintk("%s: space avail=%Lu\n", __func__, (unsigned long long)*res);
@@ -3182,7 +3129,7 @@ static int decode_attr_space_free(struct xdr_stream *xdr, uint32_t *bitmap, uint
                return -EIO;
        if (likely(bitmap[1] & FATTR4_WORD1_SPACE_FREE)) {
                READ_BUF(8);
-               READ64(*res);
+               p = xdr_decode_hyper(p, res);
                bitmap[1] &= ~FATTR4_WORD1_SPACE_FREE;
        }
        dprintk("%s: space free=%Lu\n", __func__, (unsigned long long)*res);
@@ -3199,7 +3146,7 @@ static int decode_attr_space_total(struct xdr_stream *xdr, uint32_t *bitmap, uin
                return -EIO;
        if (likely(bitmap[1] & FATTR4_WORD1_SPACE_TOTAL)) {
                READ_BUF(8);
-               READ64(*res);
+               p = xdr_decode_hyper(p, res);
                bitmap[1] &= ~FATTR4_WORD1_SPACE_TOTAL;
        }
        dprintk("%s: space total=%Lu\n", __func__, (unsigned long long)*res);
@@ -3216,7 +3163,7 @@ static int decode_attr_space_used(struct xdr_stream *xdr, uint32_t *bitmap, uint
                return -EIO;
        if (likely(bitmap[1] & FATTR4_WORD1_SPACE_USED)) {
                READ_BUF(8);
-               READ64(*used);
+               p = xdr_decode_hyper(p, used);
                bitmap[1] &= ~FATTR4_WORD1_SPACE_USED;
                ret = NFS_ATTR_FATTR_SPACE_USED;
        }
@@ -3232,8 +3179,8 @@ static int decode_attr_time(struct xdr_stream *xdr, struct timespec *time)
        uint32_t nsec;
 
        READ_BUF(12);
-       READ64(sec);
-       READ32(nsec);
+       p = xdr_decode_hyper(p, &sec);
+       nsec = be32_to_cpup(p++);
        time->tv_sec = (time_t)sec;
        time->tv_nsec = (long)nsec;
        return 0;
@@ -3315,9 +3262,9 @@ static int decode_change_info(struct xdr_stream *xdr, struct nfs4_change_info *c
        __be32 *p;
 
        READ_BUF(20);
-       READ32(cinfo->atomic);
-       READ64(cinfo->before);
-       READ64(cinfo->after);
+       cinfo->atomic = be32_to_cpup(p++);
+       p = xdr_decode_hyper(p, &cinfo->before);
+       p = xdr_decode_hyper(p, &cinfo->after);
        return 0;
 }
 
@@ -3331,39 +3278,56 @@ static int decode_access(struct xdr_stream *xdr, struct nfs4_accessres *access)
        if (status)
                return status;
        READ_BUF(8);
-       READ32(supp);
-       READ32(acc);
+       supp = be32_to_cpup(p++);
+       acc = be32_to_cpup(p++);
        access->supported = supp;
        access->access = acc;
        return 0;
 }
 
-static int decode_close(struct xdr_stream *xdr, struct nfs_closeres *res)
+static int decode_opaque_fixed(struct xdr_stream *xdr, void *buf, size_t len)
 {
        __be32 *p;
+
+       p = xdr_inline_decode(xdr, len);
+       if (likely(p)) {
+               memcpy(buf, p, len);
+               return 0;
+       }
+       print_overflow_msg(__func__, xdr);
+       return -EIO;
+}
+
+static int decode_stateid(struct xdr_stream *xdr, nfs4_stateid *stateid)
+{
+       return decode_opaque_fixed(xdr, stateid->data, NFS4_STATEID_SIZE);
+}
+
+static int decode_close(struct xdr_stream *xdr, struct nfs_closeres *res)
+{
        int status;
 
        status = decode_op_hdr(xdr, OP_CLOSE);
        if (status != -EIO)
                nfs_increment_open_seqid(status, res->seqid);
-       if (status)
-               return status;
-       READ_BUF(NFS4_STATEID_SIZE);
-       COPYMEM(res->stateid.data, NFS4_STATEID_SIZE);
-       return 0;
+       if (!status)
+               status = decode_stateid(xdr, &res->stateid);
+       return status;
+}
+
+static int decode_verifier(struct xdr_stream *xdr, void *verifier)
+{
+       return decode_opaque_fixed(xdr, verifier, 8);
 }
 
 static int decode_commit(struct xdr_stream *xdr, struct nfs_writeres *res)
 {
-       __be32 *p;
        int status;
 
        status = decode_op_hdr(xdr, OP_COMMIT);
-       if (status)
-               return status;
-       READ_BUF(8);
-       COPYMEM(res->verf->verifier, 8);
-       return 0;
+       if (!status)
+               status = decode_verifier(xdr, res->verf->verifier);
+       return status;
 }
 
 static int decode_create(struct xdr_stream *xdr, struct nfs4_change_info *cinfo)
@@ -3378,7 +3342,7 @@ static int decode_create(struct xdr_stream *xdr, struct nfs4_change_info *cinfo)
        if ((status = decode_change_info(xdr, cinfo)))
                return status;
        READ_BUF(4);
-       READ32(bmlen);
+       bmlen = be32_to_cpup(p++);
        READ_BUF(bmlen << 2);
        return 0;
 }
@@ -3633,12 +3597,12 @@ static int decode_getfh(struct xdr_stream *xdr, struct nfs_fh *fh)
                return status;
 
        READ_BUF(4);
-       READ32(len);
+       len = be32_to_cpup(p++);
        if (len > NFS4_FHSIZE)
                return -EIO;
        fh->size = len;
        READ_BUF(len);
-       COPYMEM(fh->data, len);
+       memcpy(fh->data, p, len);
        return 0;
 }
 
@@ -3662,9 +3626,9 @@ static int decode_lock_denied (struct xdr_stream *xdr, struct file_lock *fl)
        uint32_t namelen, type;
 
        READ_BUF(32);
-       READ64(offset);
-       READ64(length);
-       READ32(type);
+       p = xdr_decode_hyper(p, &offset);
+       p = xdr_decode_hyper(p, &length);
+       type = be32_to_cpup(p++);
        if (fl != NULL) {
                fl->fl_start = (loff_t)offset;
                fl->fl_end = fl->fl_start + (loff_t)length - 1;
@@ -3675,23 +3639,23 @@ static int decode_lock_denied (struct xdr_stream *xdr, struct file_lock *fl)
                        fl->fl_type = F_RDLCK;
                fl->fl_pid = 0;
        }
-       READ64(clientid);
-       READ32(namelen);
+       p = xdr_decode_hyper(p, &clientid);
+       namelen = be32_to_cpup(p++);
        READ_BUF(namelen);
        return -NFS4ERR_DENIED;
 }
 
 static int decode_lock(struct xdr_stream *xdr, struct nfs_lock_res *res)
 {
-       __be32 *p;
        int status;
 
        status = decode_op_hdr(xdr, OP_LOCK);
        if (status == -EIO)
                goto out;
        if (status == 0) {
-               READ_BUF(NFS4_STATEID_SIZE);
-               COPYMEM(res->stateid.data, NFS4_STATEID_SIZE);
+               status = decode_stateid(xdr, &res->stateid);
+               if (unlikely(status))
+                       goto out;
        } else if (status == -NFS4ERR_DENIED)
                status = decode_lock_denied(xdr, NULL);
        if (res->open_seqid != NULL)
@@ -3712,16 +3676,13 @@ static int decode_lockt(struct xdr_stream *xdr, struct nfs_lockt_res *res)
 
 static int decode_locku(struct xdr_stream *xdr, struct nfs_locku_res *res)
 {
-       __be32 *p;
        int status;
 
        status = decode_op_hdr(xdr, OP_LOCKU);
        if (status != -EIO)
                nfs_increment_lock_seqid(status, res->seqid);
-       if (status == 0) {
-               READ_BUF(NFS4_STATEID_SIZE);
-               COPYMEM(res->stateid.data, NFS4_STATEID_SIZE);
-       }
+       if (status == 0)
+               status = decode_stateid(xdr, &res->stateid);
        return status;
 }
 
@@ -3737,14 +3698,14 @@ static int decode_space_limit(struct xdr_stream *xdr, u64 *maxsize)
        uint32_t limit_type, nblocks, blocksize;
 
        READ_BUF(12);
-       READ32(limit_type);
+       limit_type = be32_to_cpup(p++);
        switch (limit_type) {
        case 1:
-               READ64(*maxsize);
+               p = xdr_decode_hyper(p, maxsize);
                break;
        case 2:
-               READ32(nblocks);
-               READ32(blocksize);
+               nblocks = be32_to_cpup(p++);
+               blocksize = be32_to_cpup(p++);
                *maxsize = (uint64_t)nblocks * (uint64_t)blocksize;
        }
        return 0;
@@ -3754,16 +3715,19 @@ static int decode_delegation(struct xdr_stream *xdr, struct nfs_openres *res)
 {
        __be32 *p;
        uint32_t delegation_type;
+       int status;
 
        READ_BUF(4);
-       READ32(delegation_type);
+       delegation_type = be32_to_cpup(p++);
        if (delegation_type == NFS4_OPEN_DELEGATE_NONE) {
                res->delegation_type = 0;
                return 0;
        }
-       READ_BUF(NFS4_STATEID_SIZE+4);
-       COPYMEM(res->delegation.data, NFS4_STATEID_SIZE);
-       READ32(res->do_recall);
+       status = decode_stateid(xdr, &res->delegation);
+       if (unlikely(status))
+               return status;
+       READ_BUF(4);
+       res->do_recall = be32_to_cpup(p++);
 
        switch (delegation_type) {
        case NFS4_OPEN_DELEGATE_READ:
@@ -3786,23 +3750,23 @@ static int decode_open(struct xdr_stream *xdr, struct nfs_openres *res)
        status = decode_op_hdr(xdr, OP_OPEN);
        if (status != -EIO)
                nfs_increment_open_seqid(status, res->seqid);
-       if (status)
+       if (!status)
+               status = decode_stateid(xdr, &res->stateid);
+       if (unlikely(status))
                return status;
-       READ_BUF(NFS4_STATEID_SIZE);
-       COPYMEM(res->stateid.data, NFS4_STATEID_SIZE);
 
        decode_change_info(xdr, &res->cinfo);
 
        READ_BUF(8);
-       READ32(res->rflags);
-       READ32(bmlen);
+       res->rflags = be32_to_cpup(p++);
+       bmlen = be32_to_cpup(p++);
        if (bmlen > 10)
                goto xdr_error;
 
        READ_BUF(bmlen << 2);
        savewords = min_t(uint32_t, bmlen, NFS4_BITMAP_SIZE);
        for (i = 0; i < savewords; ++i)
-               READ32(res->attrset[i]);
+               res->attrset[i] = be32_to_cpup(p++);
        for (; i < NFS4_BITMAP_SIZE; i++)
                res->attrset[i] = 0;
 
@@ -3814,32 +3778,26 @@ xdr_error:
 
 static int decode_open_confirm(struct xdr_stream *xdr, struct nfs_open_confirmres *res)
 {
-       __be32 *p;
        int status;
 
        status = decode_op_hdr(xdr, OP_OPEN_CONFIRM);
        if (status != -EIO)
                nfs_increment_open_seqid(status, res->seqid);
-       if (status)
-               return status;
-       READ_BUF(NFS4_STATEID_SIZE);
-       COPYMEM(res->stateid.data, NFS4_STATEID_SIZE);
-       return 0;
+       if (!status)
+               status = decode_stateid(xdr, &res->stateid);
+       return status;
 }
 
 static int decode_open_downgrade(struct xdr_stream *xdr, struct nfs_closeres *res)
 {
-       __be32 *p;
        int status;
 
        status = decode_op_hdr(xdr, OP_OPEN_DOWNGRADE);
        if (status != -EIO)
                nfs_increment_open_seqid(status, res->seqid);
-       if (status)
-               return status;
-       READ_BUF(NFS4_STATEID_SIZE);
-       COPYMEM(res->stateid.data, NFS4_STATEID_SIZE);
-       return 0;
+       if (!status)
+               status = decode_stateid(xdr, &res->stateid);
+       return status;
 }
 
 static int decode_putfh(struct xdr_stream *xdr)
@@ -3863,8 +3821,8 @@ static int decode_read(struct xdr_stream *xdr, struct rpc_rqst *req, struct nfs_
        if (status)
                return status;
        READ_BUF(8);
-       READ32(eof);
-       READ32(count);
+       eof = be32_to_cpup(p++);
+       count = be32_to_cpup(p++);
        hdrlen = (u8 *) p - (u8 *) iov->iov_base;
        recvd = req->rq_rcv_buf.len - hdrlen;
        if (count > recvd) {
@@ -3891,17 +3849,17 @@ static int decode_readdir(struct xdr_stream *xdr, struct rpc_rqst *req, struct n
        int             status;
 
        status = decode_op_hdr(xdr, OP_READDIR);
-       if (status)
+       if (!status)
+               status = decode_verifier(xdr, readdir->verifier.data);
+       if (unlikely(status))
                return status;
-       READ_BUF(8);
-       COPYMEM(readdir->verifier.data, 8);
        dprintk("%s: verifier = %08x:%08x\n",
                        __func__,
                        ((u32 *)readdir->verifier.data)[0],
                        ((u32 *)readdir->verifier.data)[1]);
 
 
-       hdrlen = (char *) p - (char *) iov->iov_base;
+       hdrlen = (char *) xdr->p - (char *) iov->iov_base;
        recvd = rcvbuf->len - hdrlen;
        if (pglen > recvd)
                pglen = recvd;
@@ -3990,7 +3948,7 @@ static int decode_readlink(struct xdr_stream *xdr, struct rpc_rqst *req)
 
        /* Convert length of symlink */
        READ_BUF(4);
-       READ32(len);
+       len = be32_to_cpup(p++);
        if (len >= rcvbuf->page_len || len <= 0) {
                dprintk("nfs: server returned giant symlink!\n");
                return -ENAMETOOLONG;
@@ -4112,7 +4070,7 @@ static int decode_setattr(struct xdr_stream *xdr)
        if (status)
                return status;
        READ_BUF(4);
-       READ32(bmlen);
+       bmlen = be32_to_cpup(p++);
        READ_BUF(bmlen << 2);
        return 0;
 }
@@ -4124,28 +4082,28 @@ static int decode_setclientid(struct xdr_stream *xdr, struct nfs_client *clp)
        int32_t nfserr;
 
        READ_BUF(8);
-       READ32(opnum);
+       opnum = be32_to_cpup(p++);
        if (opnum != OP_SETCLIENTID) {
                dprintk("nfs: decode_setclientid: Server returned operation"
                        " %d\n", opnum);
                return -EIO;
        }
-       READ32(nfserr);
+       nfserr = be32_to_cpup(p++);
        if (nfserr == NFS_OK) {
                READ_BUF(8 + NFS4_VERIFIER_SIZE);
-               READ64(clp->cl_clientid);
-               COPYMEM(clp->cl_confirm.data, NFS4_VERIFIER_SIZE);
+               p = xdr_decode_hyper(p, &clp->cl_clientid);
+               memcpy(clp->cl_confirm.data, p, NFS4_VERIFIER_SIZE);
        } else if (nfserr == NFSERR_CLID_INUSE) {
                uint32_t len;
 
                /* skip netid string */
                READ_BUF(4);
-               READ32(len);
+               len = be32_to_cpup(p++);
                READ_BUF(len);
 
                /* skip uaddr string */
                READ_BUF(4);
-               READ32(len);
+               len = be32_to_cpup(p++);
                READ_BUF(len);
                return -NFSERR_CLID_INUSE;
        } else
@@ -4169,9 +4127,9 @@ static int decode_write(struct xdr_stream *xdr, struct nfs_writeres *res)
                return status;
 
        READ_BUF(16);
-       READ32(res->count);
-       READ32(res->verf->committed);
-       COPYMEM(res->verf->verifier, 8);
+       res->count = be32_to_cpup(p++);
+       res->verf->committed = be32_to_cpup(p++);
+       memcpy(res->verf->verifier, p, 8);
        return 0;
 }
 
@@ -4186,6 +4144,7 @@ static int decode_exchange_id(struct xdr_stream *xdr,
 {
        __be32 *p;
        uint32_t dummy;
+       char *dummy_str;
        int status;
        struct nfs_client *clp = res->client;
 
@@ -4194,13 +4153,13 @@ static int decode_exchange_id(struct xdr_stream *xdr,
                return status;
 
        READ_BUF(8);
-       READ64(clp->cl_ex_clid);
+       p = xdr_decode_hyper(p, &clp->cl_ex_clid);
        READ_BUF(12);
-       READ32(clp->cl_seqid);
-       READ32(clp->cl_exchange_flags);
+       clp->cl_seqid = be32_to_cpup(p++);
+       clp->cl_exchange_flags = be32_to_cpup(p++);
 
        /* We ask for SP4_NONE */
-       READ32(dummy);
+       dummy = be32_to_cpup(p++);
        if (dummy != SP4_NONE)
                return -EIO;
 
@@ -4208,19 +4167,19 @@ static int decode_exchange_id(struct xdr_stream *xdr,
        READ_BUF(8);
 
        /* Throw away Major id */
-       READ_BUF(4);
-       READ32(dummy);
-       READ_BUF(dummy);
+       status = decode_opaque_inline(xdr, &dummy, &dummy_str);
+       if (unlikely(status))
+               return status;
 
        /* Throw away server_scope */
-       READ_BUF(4);
-       READ32(dummy);
-       READ_BUF(dummy);
+       status = decode_opaque_inline(xdr, &dummy, &dummy_str);
+       if (unlikely(status))
+               return status;
 
        /* Throw away Implementation id array */
-       READ_BUF(4);
-       READ32(dummy);
-       READ_BUF(dummy);
+       status = decode_opaque_inline(xdr, &dummy, &dummy_str);
+       if (unlikely(status))
+               return status;
 
        return 0;
 }
@@ -4232,13 +4191,13 @@ static int decode_chan_attrs(struct xdr_stream *xdr,
        u32 nr_attrs;
 
        READ_BUF(28);
-       READ32(attrs->headerpadsz);
-       READ32(attrs->max_rqst_sz);
-       READ32(attrs->max_resp_sz);
-       READ32(attrs->max_resp_sz_cached);
-       READ32(attrs->max_ops);
-       READ32(attrs->max_reqs);
-       READ32(nr_attrs);
+       attrs->headerpadsz = be32_to_cpup(p++);
+       attrs->max_rqst_sz = be32_to_cpup(p++);
+       attrs->max_resp_sz = be32_to_cpup(p++);
+       attrs->max_resp_sz_cached = be32_to_cpup(p++);
+       attrs->max_ops = be32_to_cpup(p++);
+       attrs->max_reqs = be32_to_cpup(p++);
+       nr_attrs = be32_to_cpup(p++);
        if (unlikely(nr_attrs > 1)) {
                printk(KERN_WARNING "%s: Invalid rdma channel attrs count %u\n",
                        __func__, nr_attrs);
@@ -4249,6 +4208,11 @@ static int decode_chan_attrs(struct xdr_stream *xdr,
        return 0;
 }
 
+static int decode_sessionid(struct xdr_stream *xdr, struct nfs4_sessionid *sid)
+{
+       return decode_opaque_fixed(xdr, sid->data, NFS4_MAX_SESSIONID_LEN);
+}
+
 static int decode_create_session(struct xdr_stream *xdr,
                                 struct nfs41_create_session_res *res)
 {
@@ -4258,18 +4222,15 @@ static int decode_create_session(struct xdr_stream *xdr,
        struct nfs4_session *session = clp->cl_session;
 
        status = decode_op_hdr(xdr, OP_CREATE_SESSION);
-
-       if (status)
+       if (!status)
+               status = decode_sessionid(xdr, &session->sess_id);
+       if (unlikely(status))
                return status;
 
-       /* sessionid */
-       READ_BUF(NFS4_MAX_SESSIONID_LEN);
-       COPYMEM(&session->sess_id, NFS4_MAX_SESSIONID_LEN);
-
        /* seqid, flags */
        READ_BUF(8);
-       READ32(clp->cl_seqid);
-       READ32(session->flags);
+       clp->cl_seqid = be32_to_cpup(p++);
+       session->flags = be32_to_cpup(p++);
 
        /* Channel attributes */
        status = decode_chan_attrs(xdr, &session->fc_attrs);
@@ -4299,7 +4260,9 @@ static int decode_sequence(struct xdr_stream *xdr,
                return 0;
 
        status = decode_op_hdr(xdr, OP_SEQUENCE);
-       if (status)
+       if (!status)
+               status = decode_sessionid(xdr, &id);
+       if (unlikely(status))
                goto out_err;
 
        /*
@@ -4308,32 +4271,33 @@ static int decode_sequence(struct xdr_stream *xdr,
         */
        status = -ESERVERFAULT;
 
-       slot = &res->sr_session->fc_slot_table.slots[res->sr_slotid];
-       READ_BUF(NFS4_MAX_SESSIONID_LEN + 20);
-       COPYMEM(id.data, NFS4_MAX_SESSIONID_LEN);
        if (memcmp(id.data, res->sr_session->sess_id.data,
                   NFS4_MAX_SESSIONID_LEN)) {
                dprintk("%s Invalid session id\n", __func__);
                goto out_err;
        }
+
+       READ_BUF(20);
+
        /* seqid */
-       READ32(dummy);
+       slot = &res->sr_session->fc_slot_table.slots[res->sr_slotid];
+       dummy = be32_to_cpup(p++);
        if (dummy != slot->seq_nr) {
                dprintk("%s Invalid sequence number\n", __func__);
                goto out_err;
        }
        /* slot id */
-       READ32(dummy);
+       dummy = be32_to_cpup(p++);
        if (dummy != res->sr_slotid) {
                dprintk("%s Invalid slot id\n", __func__);
                goto out_err;
        }
        /* highest slot id - currently not processed */
-       READ32(dummy);
+       dummy = be32_to_cpup(p++);
        /* target highest slot id - currently not processed */
-       READ32(dummy);
+       dummy = be32_to_cpup(p++);
        /* result flags - currently not processed */
-       READ32(dummy);
+       dummy = be32_to_cpup(p++);
        status = 0;
 out_err:
        res->sr_status = status;