Solomon Boulos has uploaded this change for review. (
https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/487461 )
Change subject: Fix close/close2: needed to return "not opened".
......................................................................
Fix close/close2: needed to return "not opened".
Ganesha's metadata cache tries to keep track of how many open "fds" there
are,
but if you tell it you did a close() rather than "nothing was open" it
decrements open_fd_count. Once that goes negative, all hell breaks loose.
I also improved the logging while debugging, as well as left a comment related
to REMOVE vs RMDIR (the spec suggests you may use REMOVE, but in reality nfsd's
want RMDIR, so I'm keeping it as is).
Change-Id: I71e0cd64557bcd9c9d694fd187983886b910f4c9
Signed-off-by: Solomon Boulos <boulos(a)google.com>
---
M src/FSAL/FSAL_PROXY_V3/main.c
1 file changed, 23 insertions(+), 9 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/61/487461/1
--
To view, visit
https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/487461
To unsubscribe, or for help writing mail filters, visit
https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: I71e0cd64557bcd9c9d694fd187983886b910f4c9
Gerrit-Change-Number: 487461
Gerrit-PatchSet: 1
Gerrit-Owner: Solomon Boulos <boulos(a)google.com>
Gerrit-MessageType: newchange