On Feb. 05, 2010, 19:10 +0200, andros@xxxxxxxxxx wrote:
> From: Andy Adamson <andros@xxxxxxxxxx>
>
> Signed-off-by: Andy Adamson <andros@xxxxxxxxxx>
> ---
> fs/nfsd/nfs4pnfsd.c | 6 ++----
> 1 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/fs/nfsd/nfs4pnfsd.c b/fs/nfsd/nfs4pnfsd.c
> index 816e2f0..3951e02 100644
> --- a/fs/nfsd/nfs4pnfsd.c
> +++ b/fs/nfsd/nfs4pnfsd.c
> @@ -870,6 +870,7 @@ nfs4_pnfs_get_layout(struct nfsd4_pnfs_layoutget *lgp,
> if (status) {
> switch (status) {
> case -ETOOSMALL:
> + case -E2BIG:
Should we allow the filesystem to return nfs errors?
Or even require it to do so?
This can be done by adding cases for the
valid error values for LAYOUTGET in this switch statement.
Benny
> status = nfserr_toosmall;
> break;
> case -ENOMEM:
> @@ -878,10 +879,7 @@ nfs4_pnfs_get_layout(struct nfsd4_pnfs_layoutget *lgp,
case nfserr_layouttrylater:
> status = nfserr_layouttrylater;
> break;
> case -ENOENT:
> - status = nfserr_badlayout;
> - break;
> - case -E2BIG:
> - status = nfserr_toosmall;
> + status = nfserr_stale;
> break;
> default:
> status = nfserr_layoutunavailable;
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
