inneek(a)gmail.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245714?usp=email ) Change subject: Reorder operations in fsal_test_access to eliminate redundant getattrs calls. ...................................................................... Reorder operations in fsal_test_access to eliminate redundant getattrs calls. When access_type is 0 ("nothing requested"), fsal_test_access previously called getattrs on the object handle before checking access_type. Moving the access_type check to the beginning of fsal_test_access avoids executing an unnecessary getattrs RPC query when access_type == 0 Change-Id: I791a175e3a92f2fb449f856a9c3f7918258dffdd Signed-off-by: Inna Batenkov <inneek(a)gmail.com&gt; --- M src/FSAL/access_check.c 1 file changed, 17 insertions(+), 8 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/14/1245714/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245714?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I791a175e3a92f2fb449f856a9c3f7918258dffdd Gerrit-Change-Number: 1245714 Gerrit-PatchSet: 1 Gerrit-Owner: inneek(a)gmail.com