Malahal has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/498187 ) Change subject: Fix mdcache_test_attrs_trust to check for required attrs ...................................................................... Fix mdcache_test_attrs_trust to check for required attrs pynfs st_read.testLargeCount test (RD4) creates a file with 10MB size with OPEN request and reads it. When pynfs reads the file, nfs4_read_cb() sets EOF bit as the file size attribute happens to be zero in the attribute cache then. This happens after commit 72d4e632d3c81ff1db0988459bd5e45be7b18949. The pynfs test open request compound has OPEN and GETFH in addition to others, and nfs4_op_getfh() is using is_referral() to cache all attributes. Commit 72d4e632d3c81ff1db0988459bd5e45be7b18949 made the mdcache_is_referral() ignore the passed in attrs leading to attribute cache not having all attributes in the cache. mdcache_test_attrs_trust() currently checks if we can trust attributes but doesn't check if all the requested attributes are valid. Change-Id: Idbdb5a22809cd62b991fbe3aab81e62cfef58c48 Signed-off-by: Malahal Naineni <malahal@us.ibm.com> --- M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_int.h 1 file changed, 3 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/87/498187/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/498187 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: Idbdb5a22809cd62b991fbe3aab81e62cfef58c48 Gerrit-Change-Number: 498187 Gerrit-PatchSet: 1 Gerrit-Owner: Malahal <malahal@gmail.com> Gerrit-MessageType: newchange