>From <madhu.punj...@in.ibm.com>:
madhu.punj...@in.ibm.com has uploaded this change for review. (
https://review.gerrithub.io/371209
Change subject: Decrement FD count in fsal_close even if obj_ops.close() fails
......................................................................
Decrement FD count in fsal_close even if obj_ops.close() fails
Currently in fsal_close() we decrement the "open_fd_count" only
when "obj_ops.close(obj_hdl)" completes successfully. But the FSAL
specific "close" operation sets "openflags = FSAL_O_CLOSED" even if
close was not successful. So if lru_run_lane() calls fsal_close()
which fails then we won't decrement the open_fd_count. And next time
when lru_run_lane calls fsal_close() for the same FD, the FSAL specific
"close" operation would return "ERR_FSAL_NOT_OPENED" and "open_fd_count"
won't decrement again.
To avoid this we now decrement the "open_fd_count" in fsal_close() for
errors other than "ERR_FSAL_NOT_OPENED".
Change-Id: I1f86deda12dc0756f7904d414592cb75498760b0
Signed-off-by: Madhu Thorat <madhu.punj...@in.ibm.com>
---
M src/include/fsal.h
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha
refs/changes/09/371209/1
--
To view, visit https://review.gerrithub.io/371209
To unsubscribe, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: I1f86deda12dc0756f7904d414592cb75498760b0
Gerrit-Change-Number: 371209
Gerrit-PatchSet: 1
Gerrit-Owner: madhu.punj...@in.ibm.com
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
