Hey Peng, Can you add this to NFS v2 as well? Then we'll be covered if we ever want to re-use this variable somewhere else! :) Thanks, Anna On 12/24/2014 02:13 AM, Tom Haynes wrote: > From: Peng Tao <tao.peng@xxxxxxxxxxxxxxx> > > Flexfiles layout would want to use them to report DS IO status. > > Signed-off-by: Peng Tao <tao.peng@xxxxxxxxxxxxxxx> > Signed-off-by: Tom Haynes <Thomas.Haynes@xxxxxxxxxxxxxxx> > --- > fs/nfs/nfs3xdr.c | 3 +++ > fs/nfs/nfs4xdr.c | 3 +++ > include/linux/nfs_xdr.h | 2 ++ > 3 files changed, 8 insertions(+) > > diff --git a/fs/nfs/nfs3xdr.c b/fs/nfs/nfs3xdr.c > index 8f4cbe7..2a932fd 100644 > --- a/fs/nfs/nfs3xdr.c > +++ b/fs/nfs/nfs3xdr.c > @@ -1636,6 +1636,7 @@ static int nfs3_xdr_dec_read3res(struct rpc_rqst *req, struct xdr_stream *xdr, > error = decode_post_op_attr(xdr, result->fattr); > if (unlikely(error)) > goto out; > + result->op_status = status; > if (status != NFS3_OK) > goto out_status; > error = decode_read3resok(xdr, result); > @@ -1708,6 +1709,7 @@ static int nfs3_xdr_dec_write3res(struct rpc_rqst *req, struct xdr_stream *xdr, > error = decode_wcc_data(xdr, result->fattr); > if (unlikely(error)) > goto out; > + result->op_status = status; > if (status != NFS3_OK) > goto out_status; > error = decode_write3resok(xdr, result); > @@ -2323,6 +2325,7 @@ static int nfs3_xdr_dec_commit3res(struct rpc_rqst *req, > error = decode_wcc_data(xdr, result->fattr); > if (unlikely(error)) > goto out; > + result->op_status = status; > if (status != NFS3_OK) > goto out_status; > error = decode_writeverf3(xdr, &result->verf->verifier); > diff --git a/fs/nfs/nfs4xdr.c b/fs/nfs/nfs4xdr.c > index cb4376b..7d8d7a4 100644 > --- a/fs/nfs/nfs4xdr.c > +++ b/fs/nfs/nfs4xdr.c > @@ -6567,6 +6567,7 @@ static int nfs4_xdr_dec_read(struct rpc_rqst *rqstp, struct xdr_stream *xdr, > int status; > > status = decode_compound_hdr(xdr, &hdr); > + res->op_status = hdr.status; > if (status) > goto out; > status = decode_sequence(xdr, &res->seq_res, rqstp); > @@ -6592,6 +6593,7 @@ static int nfs4_xdr_dec_write(struct rpc_rqst *rqstp, struct xdr_stream *xdr, > int status; > > status = decode_compound_hdr(xdr, &hdr); > + res->op_status = hdr.status; > if (status) > goto out; > status = decode_sequence(xdr, &res->seq_res, rqstp); > @@ -6621,6 +6623,7 @@ static int nfs4_xdr_dec_commit(struct rpc_rqst *rqstp, struct xdr_stream *xdr, > int status; > > status = decode_compound_hdr(xdr, &hdr); > + res->op_status = hdr.status; > if (status) > goto out; > status = decode_sequence(xdr, &res->seq_res, rqstp); > diff --git a/include/linux/nfs_xdr.h b/include/linux/nfs_xdr.h > index 467c84e..962f461 100644 > --- a/include/linux/nfs_xdr.h > +++ b/include/linux/nfs_xdr.h > @@ -513,6 +513,7 @@ struct nfs_pgio_res { > struct nfs4_sequence_res seq_res; > struct nfs_fattr * fattr; > __u32 count; > + __u32 op_status; > int eof; /* used by read */ > struct nfs_writeverf * verf; /* used by write */ > const struct nfs_server *server; /* used by write */ > @@ -532,6 +533,7 @@ struct nfs_commitargs { > > struct nfs_commitres { > struct nfs4_sequence_res seq_res; > + __u32 op_status; > struct nfs_fattr *fattr; > struct nfs_writeverf *verf; > const struct nfs_server *server; > -- 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