8:07 a.m.
Madhu Thorat has uploaded this change for review. (
https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/499493 )
Change subject: Print error code for ERR_FSAL_IO in
nfs4_Errno_verbose()/nfs3_Errno_verbose()
......................................................................
Print error code for ERR_FSAL_IO in nfs4_Errno_verbose()/nfs3_Errno_verbose()
Currently we log ERR_FSAL_IO in nfs4_Errno_verbose()/ nfs3_Errno_verbose()
with LogCrit(..). But different errors like EMFILE, ENFILE etc are mapped
to ERR_FSAL_IO. The information about the specific error code is usually
saved in status.minor. It will be useful to log the specific error
code to know which error was mapped to ERR_FSAL_IO.
With this change we print the status.minor value in the same LogCrit()
log in nfs4_Errno_verbose()/ nfs3_Errno_verbose().
Also removed nfs4_Errno() and nfs3_Errno() as they provided only
status.major value to nfs4_Errno_verbose() and nfs3_Errno_verbose(),
but with this patch we need to provide status (both major, minor) to
nfs4_Errno_verbose()/ nfs3_Errno_verbose(). So now we use
nfs4_Errno_status()/ nfs3_Errno_status() in place of
nfs4_Errno()/ nfs3_Errno() in code.
Change-Id: I5e7e38f7f266588b07ba677ba8052ea2cc2dc83e
Signed-off-by: Madhu Thorat <madhu.punjabi(a)in.ibm.com>
---
M src/Protocols/NFS/nfs3_access.c
M src/Protocols/NFS/nfs4_op_access.c
M src/include/nfs_convert.h
M src/include/nfs_creds.h
M src/support/nfs_convert.c
M src/support/nfs_creds.c
6 files changed, 54 insertions(+), 42 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/93/499493/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/499493
To unsubscribe, or for help writing mail filters, visit
https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: I5e7e38f7f266588b07ba677ba8052ea2cc2dc83e
Gerrit-Change-Number: 499493
Gerrit-PatchSet: 1
Gerrit-Owner: Madhu Thorat <madhu.punjabi(a)in.ibm.com>
Gerrit-MessageType: newchange