From: Boaz Harrosh <bharrosh@panasas.com>
To: Benny Halevy <bhalevy@panasas.com>,
open-osd <osd-dev@open-osd.org>,
NFS list <linux-nfs@vger.kernel.org>
Subject: [PATCH 05/10] SQUASHME: pnfsd-exofs: osd_xdr_srv: Convert XDR API to return enum nfsstat4
Date: Tue, 14 Jun 2011 17:36:21 -0400 [thread overview]
Message-ID: <1308087381-13179-1-git-send-email-bharrosh@panasas.com> (raw)
In-Reply-To: <4DF7D36F.9070602@panasas.com>
* Change pnfs_osd_xdr_srv API to return nfs4 errors
Signed-off-by: Boaz Harrosh <bharrosh@panasas.com>
---
fs/exportfs/pnfs_osd_xdr_srv.c | 30 +++++++++++++++---------------
include/linux/nfsd/pnfs_osd_xdr_srv.h | 6 +++---
2 files changed, 18 insertions(+), 18 deletions(-)
diff --git a/fs/exportfs/pnfs_osd_xdr_srv.c b/fs/exportfs/pnfs_osd_xdr_srv.c
index ec814c1..4d92f1c 100644
--- a/fs/exportfs/pnfs_osd_xdr_srv.c
+++ b/fs/exportfs/pnfs_osd_xdr_srv.c
@@ -51,14 +51,14 @@
* u32 odm_raid_algorithm;
* };
*/
-static int pnfs_osd_xdr_encode_data_map(
+static enum nfsstat4 pnfs_osd_xdr_encode_data_map(
struct exp_xdr_stream *xdr,
struct pnfs_osd_data_map *data_map)
{
__be32 *p = exp_xdr_reserve_qwords(xdr, 1+2+1+1+1+1);
if (!p)
- return -E2BIG;
+ return NFS4ERR_TOOSMALL;
p = exp_xdr_encode_u32(p, data_map->odm_num_comps);
p = exp_xdr_encode_u64(p, data_map->odm_stripe_unit);
@@ -77,7 +77,7 @@ static int pnfs_osd_xdr_encode_data_map(
* u64 oid_object_id;
* };
*/
-static inline int pnfs_osd_xdr_encode_objid(
+static inline enum nfsstat4 pnfs_osd_xdr_encode_objid(
struct exp_xdr_stream *xdr,
struct pnfs_osd_objid *object_id)
{
@@ -86,7 +86,7 @@ static inline int pnfs_osd_xdr_encode_objid(
(struct nfsd4_pnfs_deviceid *)&object_id->oid_device_id;
if (!p)
- return -E2BIG;
+ return NFS4ERR_TOOSMALL;
p = exp_xdr_encode_u64(p, dev_id->fsid);
p = exp_xdr_encode_u64(p, dev_id->devid);
@@ -110,12 +110,12 @@ static inline int pnfs_osd_xdr_encode_objid(
* struct pnfs_osd_opaque_cred oc_cap;
* };
*/
-int pnfs_osd_xdr_encode_layout_cred(
+enum nfsstat4 pnfs_osd_xdr_encode_layout_cred(
struct exp_xdr_stream *xdr,
struct pnfs_osd_object_cred *olo_comp)
{
__be32 *p;
- int err;
+ enum nfsstat4 err;
err = pnfs_osd_xdr_encode_objid(xdr, &olo_comp->oc_object_id);
if (err)
@@ -123,7 +123,7 @@ int pnfs_osd_xdr_encode_layout_cred(
p = exp_xdr_reserve_space(xdr, 3*4 + 4+olo_comp->oc_cap.cred_len);
if (!p)
- return -E2BIG;
+ return NFS4ERR_TOOSMALL;
p = exp_xdr_encode_u32(p, olo_comp->oc_osd_version);
@@ -146,12 +146,12 @@ EXPORT_SYMBOL(pnfs_osd_xdr_encode_layout_cred);
* struct pnfs_osd_object_cred *olo_comps;
* };
*/
-int pnfs_osd_xdr_encode_layout_hdr(
+enum nfsstat4 pnfs_osd_xdr_encode_layout_hdr(
struct exp_xdr_stream *xdr,
struct pnfs_osd_layout *pol)
{
__be32 *p;
- int err;
+ enum nfsstat4 err;
err = pnfs_osd_xdr_encode_data_map(xdr, &pol->olo_map);
if (err)
@@ -159,7 +159,7 @@ int pnfs_osd_xdr_encode_layout_hdr(
p = exp_xdr_reserve_qwords(xdr, 2);
if (!p)
- return -E2BIG;
+ return NFS4ERR_TOOSMALL;
p = exp_xdr_encode_u32(p, pol->olo_comps_index);
p = exp_xdr_encode_u32(p, pol->olo_num_comps);
@@ -168,13 +168,13 @@ int pnfs_osd_xdr_encode_layout_hdr(
}
EXPORT_SYMBOL(pnfs_osd_xdr_encode_layout_hdr);
-static int _encode_string(struct exp_xdr_stream *xdr,
+static enum nfsstat4 _encode_string(struct exp_xdr_stream *xdr,
const struct nfs4_string *str)
{
__be32 *p = exp_xdr_reserve_space(xdr, 4 + str->len);
if (!p)
- return -E2BIG;
+ return NFS4ERR_TOOSMALL;
exp_xdr_encode_opaque(p, str->data, str->len);
return 0;
}
@@ -188,15 +188,15 @@ static int _encode_string(struct exp_xdr_stream *xdr,
* struct nfs4_string oda_osdname;
* };
*/
-int pnfs_osd_xdr_encode_deviceaddr(
+enum nfsstat4 pnfs_osd_xdr_encode_deviceaddr(
struct exp_xdr_stream *xdr, struct pnfs_osd_deviceaddr *devaddr)
{
__be32 *p;
- int err;
+ enum nfsstat4 err;
p = exp_xdr_reserve_space(xdr, 4 + 4 + sizeof(devaddr->oda_lun));
if (!p)
- return -E2BIG;
+ return NFS4ERR_TOOSMALL;
/* Empty oda_targetid */
p = exp_xdr_encode_u32(p, OBJ_TARGET_ANON);
diff --git a/include/linux/nfsd/pnfs_osd_xdr_srv.h b/include/linux/nfsd/pnfs_osd_xdr_srv.h
index e9a2187..2909fbe 100644
--- a/include/linux/nfsd/pnfs_osd_xdr_srv.h
+++ b/include/linux/nfsd/pnfs_osd_xdr_srv.h
@@ -17,16 +17,16 @@
#include <linux/exp_xdr.h>
/* Layout encoding */
-int pnfs_osd_xdr_encode_layout_hdr(
+enum nfsstat4 pnfs_osd_xdr_encode_layout_hdr(
struct exp_xdr_stream *xdr,
struct pnfs_osd_layout *layout);
-int pnfs_osd_xdr_encode_layout_cred(
+enum nfsstat4 pnfs_osd_xdr_encode_layout_cred(
struct exp_xdr_stream *xdr,
struct pnfs_osd_object_cred *cred);
/* deviceaddr encoding */
-int pnfs_osd_xdr_encode_deviceaddr(
+enum nfsstat4 pnfs_osd_xdr_encode_deviceaddr(
struct exp_xdr_stream *xdr, struct pnfs_osd_deviceaddr *devaddr);
/* layout_commit decoding */
--
1.7.3.4
next prev parent reply other threads:[~2011-06-14 21:36 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-14 20:53 [PATCHSET 0/8] pnfsd-exofs: New set of exofs pnfs export patches Boaz Harrosh
2011-06-14 21:00 ` Benny Halevy
2011-06-14 22:24 ` Benny Halevy
2011-06-14 21:02 ` [PATCH 1/8] nfs_xdr: Move nfs4_string definition out of #ifdef CONFIG_NFS_V4 Boaz Harrosh
2011-06-14 21:02 ` [PATCH 2/8] exofs: pnfs-tree: Remove pnfs-osd private definitions Boaz Harrosh
2011-06-14 21:03 ` [PATCH 4/8] pnfsd-exofs: pnfs objects server XDR functions Boaz Harrosh
2011-06-14 21:03 ` [PATCH 5/8] pnfsd-exofs: Add pNFS export support Boaz Harrosh
2011-06-14 21:03 ` [PATCH 6/8] pnfsd-exofs: get_device_info implementation Boaz Harrosh
2011-06-14 21:03 ` [PATCH 8/8] pnfsd-exofs: layoutreturn pnfs-obj information decoding Boaz Harrosh
2011-06-14 21:20 ` [PATCH 3/8] pnfs-obj: pnfs_osd_xdr: Remove dead code and cleanup Boaz Harrosh
2011-06-14 21:20 ` [PATCH 7/8] pnfsd-exofs: Recall layouts on truncate Boaz Harrosh
2011-06-14 21:32 ` [PATCHSET 0/10] pnfsd-exofs: SQUASHMES that went into last changes Boaz Harrosh
2011-06-14 21:35 ` [PATCH 01/10] SQUASHME: pnfsd-exofs xdr_srv: Wrong type in comments Boaz Harrosh
2011-06-14 21:35 ` [PATCH 02/10] SQUASHME: pnfsd-exofs: Move all Server xdr definisions to separate header Boaz Harrosh
2011-06-14 21:36 ` [PATCH 03/10] SQUASHME: pnfsd-exofs: Prettify pnfs_osd_xdr_srv.h Boaz Harrosh
2011-06-14 21:36 ` [PATCH 04/10] SQUASHME: pnfsd-exofs: Change layout encoding to two pass Boaz Harrosh
2011-06-14 21:36 ` Boaz Harrosh [this message]
2011-06-14 21:36 ` [PATCH 06/10] SQUASHME: pnfsd-exofs: Server xdr definisions were moved to a separate header Boaz Harrosh
2011-06-14 21:36 ` [PATCH 07/10] SQUASHME: pnfsd-exofs: osd_xdr_srv header moved Boaz Harrosh
2011-06-14 21:36 ` [PATCH 08/10] SQUASHME: pnfsd-exofs: layout encoding was Changed to two pass Boaz Harrosh
2011-06-14 21:36 ` [PATCH 09/10] SQUASHME: pnfsd-exofs: Convert XDR API to return enum nfsstat4 Boaz Harrosh
2011-06-14 21:38 ` [PATCH 10/10] SQUASHME: pnfsd-exofs: checkpatch love and a dprint Boaz Harrosh
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1308087381-13179-1-git-send-email-bharrosh@panasas.com \
--to=bharrosh@panasas.com \
--cc=bhalevy@panasas.com \
--cc=linux-nfs@vger.kernel.org \
--cc=osd-dev@open-osd.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).