Description of problem: - `is_mdc_key_satisfied()` is returning 0 when it has not checked any of the keys - This causes the cache'd value for the root inode to always be invalid (`mdc_xattr_satisfied()` returns 0, which causes us to jump to `uncached'). - In this diff we add a new option called "strict-xattrs", when enabled winds getxattr calls for those keys not present in our cache. - This allows "special" getxattr commands (quota cli commands for example) to work when md-cache is enabled. Version-Release number of selected component (if applicable): Found in FB development branaches How reproducible: Always reproducible Committed in release-3.8-fb @ http://review.gluster.org/#/c/16060/
REVIEW: http://review.gluster.org/16125 (performance/md-cache: Fix caching for root inode) posted (#1) for review on master by Shreyas Siravara (sshreyas)
REVIEW: http://review.gluster.org/16125 (performance/md-cache: Fix caching for root inode) posted (#2) for review on master by Shreyas Siravara (sshreyas)
REVIEW: http://review.gluster.org/16125 (performance/md-cache: Fix caching for root inode) posted (#3) for review on master by Shreyas Siravara (sshreyas)
REVIEW: http://review.gluster.org/16126 (performance/md-cache: Add an option to cache all xattrs for an inode) posted (#1) for review on release-3.8-fb by Shreyas Siravara (sshreyas)
REVIEW: http://review.gluster.org/16126 (performance/md-cache: Add an option to cache all xattrs for an inode) posted (#2) for review on release-3.8-fb by Shreyas Siravara (sshreyas)
REVIEW: http://review.gluster.org/16126 (performance/md-cache: Add an option to cache all xattrs for an inode) posted (#3) for review on release-3.8-fb by Shreyas Siravara (sshreyas)
REVIEW: http://review.gluster.org/16134 (performance/md-cache: Add an option to cache all xattrs for an inode) posted (#1) for review on master by Shreyas Siravara (sshreyas)
REVIEW: http://review.gluster.org/16134 (performance/md-cache: Add an option to cache all xattrs for an inode) posted (#2) for review on master by Shreyas Siravara (sshreyas)
REVIEW: http://review.gluster.org/16134 (performance/md-cache: Add an option to cache all xattrs for an inode) posted (#3) for review on master by Shreyas Siravara (sshreyas)
REVIEW: https://review.gluster.org/16134 (performance/md-cache: Add an option to cache all xattrs for an inode) posted (#4) for review on master by Jeff Darcy (jdarcy)
*** Bug 1428050 has been marked as a duplicate of this bug. ***
This update is done in bulk based on the state of the patch and the time since last activity. If the issue is still seen, please reopen the bug.