deepakarumugam.s@nutanix.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1220340?usp=email ) Change subject: Remove unnecessary lookup from cih_remove_checked ...................................................................... Remove unnecessary lookup from cih_remove_checked If the entry is not in the avl theres no need to do a lookup of the entry in the avl. In my company, we try to release more entries during reaping. From flamegraph, the reaper thread spends a lot of time reaping a non existent entry from the cache. This becomes a big bottleneck when we have to reap large number of entries. With this change we got 30% improvement in reaping latencies. Change-Id: Ied9b02789c5736b80dbce4dc226dd585ce6d1161 Signed-off-by: Deepak Arumugam Sankara Subramanian <deepakarumugam.s@nutanix.com> --- M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_hash.h 1 file changed, 18 insertions(+), 13 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/40/1220340/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1220340?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: Ied9b02789c5736b80dbce4dc226dd585ce6d1161 Gerrit-Change-Number: 1220340 Gerrit-PatchSet: 1 Gerrit-Owner: deepakarumugam.s@nutanix.com