diff options
| author |   Chuck Lever <chuck.lever@oracle.com> | 2022-09-08 18:13:42 -0400 |
|---|---|---|
| committer | Chuck Lever <chuck.lever@oracle.com> | 2022-09-26 14:02:31 -0400 |
| commit | 948755efc951de75c87d4fa916d9d36b58299295 (patch) | |
| tree | 1d26a9578c26593518ff8b8f0ec72c74abe1789a /fs/nfsd/nfsfh.c | |
| parent | nfsd: remove nfsd4_prepare_cb_recall() declaration (diff) | |
| download | linux-dev-948755efc951de75c87d4fa916d9d36b58299295.tar.xz linux-dev-948755efc951de75c87d4fa916d9d36b58299295.zip | |
NFSD: Replace dprintk() call site in fh_verify()
Record permission errors in the trace log. Note that the new trace
event is conditional, so it will only record non-zero return values
from nfsd_permission().
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Reviewed-by: Jeff Layton <jlayton@kernel.org>
Diffstat (limited to 'fs/nfsd/nfsfh.c')
| -rw-r--r-- | fs/nfsd/nfsfh.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/fs/nfsd/nfsfh.c b/fs/nfsd/nfsfh.c index a5b71526cee0..d73434200df9 100644 --- a/fs/nfsd/nfsfh.c +++ b/fs/nfsd/nfsfh.c @@ -392,13 +392,7 @@ fh_verify(struct svc_rqst *rqstp, struct svc_fh *fhp, umode_t type, int access) skip_pseudoflavor_check: /* Finally, check access permissions. */ error = nfsd_permission(rqstp, exp, dentry, access); - - if (error) { - dprintk("fh_verify: %pd2 permission failure, " - "acc=%x, error=%d\n", - dentry, - access, ntohl(error)); - } + trace_nfsd_fh_verify_err(rqstp, fhp, type, access, error); out: if (error == nfserr_stale) nfsd_stats_fh_stale_inc(exp); |