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@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@in.ibm.com> Gerrit-MessageType: newchange