Malahal has uploaded this change for review.

View Change

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 change 498187. To unsubscribe, or for help writing mail filters, visit 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