deepakarumugam.s@nutanix.com has uploaded this change for review.

View Change

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

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