Commit f5fc3c50 authored by Chuck Lever's avatar Chuck Lever Committed by Trond Myklebust

NFS: Switch in new NFSv3 decoder functions

The naming scheme of the new decoder functions, which follows the
NFSv4 XDR decoder functions, is slightly different than the scheme
used for the old functions.  Rename the functions as a separate
step to keep the patches clean.
Signed-off-by: default avatarChuck Lever <chuck.lever@oracle.com>
Tested-by: default avatarJ. Bruce Fields <bfields@redhat.com>
Signed-off-by: default avatarTrond Myklebust <Trond.Myklebust@netapp.com>
parent e4f93234
...@@ -38,14 +38,10 @@ ...@@ -38,14 +38,10 @@
#define NFS3_path_sz (1+(NFS3_MAXPATHLEN>>2)) #define NFS3_path_sz (1+(NFS3_MAXPATHLEN>>2))
#define NFS3_fattr_sz (21) #define NFS3_fattr_sz (21)
#define NFS3_cookieverf_sz (NFS3_COOKIEVERFSIZE>>2) #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_pre_op_attr_sz (1+NFS3_wcc_attr_sz)
#define NFS3_post_op_attr_sz (1+NFS3_fattr_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_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_diropargs_sz (NFS3_fh_sz+NFS3_filename_sz) #define NFS3_diropargs_sz (NFS3_fh_sz+NFS3_filename_sz)
#define NFS3_getattrargs_sz (NFS3_fh_sz) #define NFS3_getattrargs_sz (NFS3_fh_sz)
...@@ -66,9 +62,9 @@ ...@@ -66,9 +62,9 @@
#define NFS3_readdirplusargs_sz (NFS3_fh_sz+NFS3_cookieverf_sz+4) #define NFS3_readdirplusargs_sz (NFS3_fh_sz+NFS3_cookieverf_sz+4)
#define NFS3_commitargs_sz (NFS3_fh_sz+3) #define NFS3_commitargs_sz (NFS3_fh_sz+3)
#define NFS3_attrstat_sz (1+NFS3_fattr_sz) #define NFS3_getattrres_sz (1+NFS3_fattr_sz)
#define NFS3_wccstat_sz (1+NFS3_wcc_data_sz) #define NFS3_setattrres_sz (1+NFS3_wcc_data_sz)
#define NFS3_removeres_sz (NFS3_wccstat_sz) #define NFS3_removeres_sz (NFS3_setattrres_sz)
#define NFS3_lookupres_sz (1+NFS3_fh_sz+(2 * NFS3_post_op_attr_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_accessres_sz (1+NFS3_post_op_attr_sz+1)
#define NFS3_readlinkres_sz (1+NFS3_post_op_attr_sz+1) #define NFS3_readlinkres_sz (1+NFS3_post_op_attr_sz+1)
...@@ -3017,36 +3013,36 @@ static int nfs3_xdr_dec_setacl3res(struct rpc_rqst *req, __be32 *p, ...@@ -3017,36 +3013,36 @@ static int nfs3_xdr_dec_setacl3res(struct rpc_rqst *req, __be32 *p,
[NFS3PROC_##proc] = { \ [NFS3PROC_##proc] = { \
.p_proc = NFS3PROC_##proc, \ .p_proc = NFS3PROC_##proc, \
.p_encode = (kxdrproc_t)nfs3_xdr_enc_##argtype##3args, \ .p_encode = (kxdrproc_t)nfs3_xdr_enc_##argtype##3args, \
.p_decode = (kxdrproc_t) nfs3_xdr_##restype, \ .p_decode = (kxdrproc_t)nfs3_xdr_dec_##restype##3res, \
.p_arglen = NFS3_##argtype##args_sz, \ .p_arglen = NFS3_##argtype##args_sz, \
.p_replen = NFS3_##restype##_sz, \ .p_replen = NFS3_##restype##res_sz, \
.p_timer = timer, \ .p_timer = timer, \
.p_statidx = NFS3PROC_##proc, \ .p_statidx = NFS3PROC_##proc, \
.p_name = #proc, \ .p_name = #proc, \
} }
struct rpc_procinfo nfs3_procedures[] = { struct rpc_procinfo nfs3_procedures[] = {
PROC(GETATTR, getattr, attrstat, 1), PROC(GETATTR, getattr, getattr, 1),
PROC(SETATTR, setattr, wccstat, 0), PROC(SETATTR, setattr, setattr, 0),
PROC(LOOKUP, lookup, lookupres, 2), PROC(LOOKUP, lookup, lookup, 2),
PROC(ACCESS, access, accessres, 1), PROC(ACCESS, access, access, 1),
PROC(READLINK, readlink, readlinkres, 3), PROC(READLINK, readlink, readlink, 3),
PROC(READ, read, readres, 3), PROC(READ, read, read, 3),
PROC(WRITE, write, writeres, 4), PROC(WRITE, write, write, 4),
PROC(CREATE, create, createres, 0), PROC(CREATE, create, create, 0),
PROC(MKDIR, mkdir, createres, 0), PROC(MKDIR, mkdir, create, 0),
PROC(SYMLINK, symlink, createres, 0), PROC(SYMLINK, symlink, create, 0),
PROC(MKNOD, mknod, createres, 0), PROC(MKNOD, mknod, create, 0),
PROC(REMOVE, remove, removeres, 0), PROC(REMOVE, remove, remove, 0),
PROC(RMDIR, lookup, wccstat, 0), PROC(RMDIR, lookup, setattr, 0),
PROC(RENAME, rename, renameres, 0), PROC(RENAME, rename, rename, 0),
PROC(LINK, link, linkres, 0), PROC(LINK, link, link, 0),
PROC(READDIR, readdir, readdirres, 3), PROC(READDIR, readdir, readdir, 3),
PROC(READDIRPLUS, readdirplus, readdirres, 3), PROC(READDIRPLUS, readdirplus, readdir, 3),
PROC(FSSTAT, getattr, fsstatres, 0), PROC(FSSTAT, getattr, fsstat, 0),
PROC(FSINFO, getattr, fsinfores, 0), PROC(FSINFO, getattr, fsinfo, 0),
PROC(PATHCONF, getattr, pathconfres, 0), PROC(PATHCONF, getattr, pathconf, 0),
PROC(COMMIT, commit, commitres, 5), PROC(COMMIT, commit, commit, 5),
}; };
struct rpc_version nfs_version3 = { struct rpc_version nfs_version3 = {
...@@ -3060,7 +3056,7 @@ static struct rpc_procinfo nfs3_acl_procedures[] = { ...@@ -3060,7 +3056,7 @@ static struct rpc_procinfo nfs3_acl_procedures[] = {
[ACLPROC3_GETACL] = { [ACLPROC3_GETACL] = {
.p_proc = ACLPROC3_GETACL, .p_proc = ACLPROC3_GETACL,
.p_encode = (kxdrproc_t)nfs3_xdr_enc_getacl3args, .p_encode = (kxdrproc_t)nfs3_xdr_enc_getacl3args,
.p_decode = (kxdrproc_t) nfs3_xdr_getaclres, .p_decode = (kxdrproc_t)nfs3_xdr_dec_getacl3res,
.p_arglen = ACL3_getaclargs_sz, .p_arglen = ACL3_getaclargs_sz,
.p_replen = ACL3_getaclres_sz, .p_replen = ACL3_getaclres_sz,
.p_timer = 1, .p_timer = 1,
...@@ -3069,7 +3065,7 @@ static struct rpc_procinfo nfs3_acl_procedures[] = { ...@@ -3069,7 +3065,7 @@ static struct rpc_procinfo nfs3_acl_procedures[] = {
[ACLPROC3_SETACL] = { [ACLPROC3_SETACL] = {
.p_proc = ACLPROC3_SETACL, .p_proc = ACLPROC3_SETACL,
.p_encode = (kxdrproc_t)nfs3_xdr_enc_setacl3args, .p_encode = (kxdrproc_t)nfs3_xdr_enc_setacl3args,
.p_decode = (kxdrproc_t) nfs3_xdr_setaclres, .p_decode = (kxdrproc_t)nfs3_xdr_dec_setacl3res,
.p_arglen = ACL3_setaclargs_sz, .p_arglen = ACL3_setaclargs_sz,
.p_replen = ACL3_setaclres_sz, .p_replen = ACL3_setaclres_sz,
.p_timer = 0, .p_timer = 0,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment