NFS: Clean up decode_getattr_args()
Removing the dprintk() lets us return the status value directly, rather than jumping to a label if an error occurs. Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com> Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
This commit is contained in:
parent
be55f1bca7
commit
56938bb77a
@ -192,11 +192,8 @@ static __be32 decode_getattr_args(struct svc_rqst *rqstp, struct xdr_stream *xdr
|
|||||||
|
|
||||||
status = decode_fh(xdr, &args->fh);
|
status = decode_fh(xdr, &args->fh);
|
||||||
if (unlikely(status != 0))
|
if (unlikely(status != 0))
|
||||||
goto out;
|
return status;
|
||||||
status = decode_bitmap(xdr, args->bitmap);
|
return decode_bitmap(xdr, args->bitmap);
|
||||||
out:
|
|
||||||
dprintk("%s: exit with status = %d\n", __func__, ntohl(status));
|
|
||||||
return status;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static __be32 decode_recall_args(struct svc_rqst *rqstp, struct xdr_stream *xdr, struct cb_recallargs *args)
|
static __be32 decode_recall_args(struct svc_rqst *rqstp, struct xdr_stream *xdr, struct cb_recallargs *args)
|
||||||
|
Loading…
Reference in New Issue
Block a user