diff options
author | Andy Adamson <andros@netapp.com> | 2010-01-14 17:45:05 -0500 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2010-02-10 08:30:56 -0500 |
commit | b92b30190093377828efcde5fc4cf7598fa1ee46 (patch) | |
tree | ad13e599621c7e2c1d908738370ae827a393a98b | |
parent | 31d2b4356b054537c35f4f8a7533e0b4a494dcc6 (diff) |
nfs41: directly encode back channel error
Skip all other processing when error is encountered.
Signed-off-by: Andy Adamson <andros@netapp.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
-rw-r--r-- | fs/nfs/callback_xdr.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/fs/nfs/callback_xdr.c b/fs/nfs/callback_xdr.c index 6ae327871b86..d3e07f469949 100644 --- a/fs/nfs/callback_xdr.c +++ b/fs/nfs/callback_xdr.c | |||
@@ -625,16 +625,19 @@ static __be32 process_op(uint32_t minorversion, int nop, | |||
625 | preprocess_nfs4_op(op_nr, &op); | 625 | preprocess_nfs4_op(op_nr, &op); |
626 | if (status == htonl(NFS4ERR_OP_ILLEGAL)) | 626 | if (status == htonl(NFS4ERR_OP_ILLEGAL)) |
627 | op_nr = OP_CB_ILLEGAL; | 627 | op_nr = OP_CB_ILLEGAL; |
628 | if (status) | ||
629 | goto encode_hdr; | ||
628 | 630 | ||
629 | maxlen = xdr_out->end - xdr_out->p; | 631 | maxlen = xdr_out->end - xdr_out->p; |
630 | if (maxlen > 0 && maxlen < PAGE_SIZE) { | 632 | if (maxlen > 0 && maxlen < PAGE_SIZE) { |
631 | if (likely(status == 0 && op->decode_args != NULL)) | 633 | if (likely(op->decode_args != NULL)) |
632 | status = op->decode_args(rqstp, xdr_in, argp); | 634 | status = op->decode_args(rqstp, xdr_in, argp); |
633 | if (likely(status == 0 && op->process_op != NULL)) | 635 | if (likely(status == 0 && op->process_op != NULL)) |
634 | status = op->process_op(argp, resp); | 636 | status = op->process_op(argp, resp); |
635 | } else | 637 | } else |
636 | status = htonl(NFS4ERR_RESOURCE); | 638 | status = htonl(NFS4ERR_RESOURCE); |
637 | 639 | ||
640 | encode_hdr: | ||
638 | res = encode_op_hdr(xdr_out, op_nr, status); | 641 | res = encode_op_hdr(xdr_out, op_nr, status); |
639 | if (unlikely(res)) | 642 | if (unlikely(res)) |
640 | return res; | 643 | return res; |