>From Daniel Gryniewicz <d...@redhat.com>:

Daniel Gryniewicz has uploaded a new change for review.

  https://review.gerrithub.io/295524

Change subject: Re-add owner validation cache optimization
......................................................................

Re-add owner validation cache optimization

2.3 checked the owner against the cached owner unconditionally
(regardless of cache validity) when doing permission checks for
read/write.  2.4-dev removed this, causing a getattr() call on each
write (since write invalidates attributes).

Re-add this optimization, as it is a 30%-80% performance hit on small
writes, and owner changes infrequently.  In the future, finer grained
cache validity is likely needed.

Change-Id: Ida91b03cb5791bf767161beb7829d51122d0a83b
Signed-off-by: Daniel Gryniewicz <d...@redhat.com>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_handle.c
1 file changed, 7 insertions(+), 10 deletions(-)


  git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha 
refs/changes/24/295524/1
-- 
To view, visit https://review.gerrithub.io/295524
To unsubscribe, visit https://review.gerrithub.io/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ida91b03cb5791bf767161beb7829d51122d0a83b
Gerrit-PatchSet: 1
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Owner: Daniel Gryniewicz <d...@redhat.com>

------------------------------------------------------------------------------
_______________________________________________
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel

Reply via email to