>From Jeff Layton <jlay...@redhat.com>:
Jeff Layton has uploaded this change for review. (
https://review.gerrithub.io/377400
Change subject: Take a reference to the session over a v4.1+ callback
......................................................................
Take a reference to the session over a v4.1+ callback
Overhaul the loop in nfs_rpc_v41_single, such that we hold the cid_mutex
while walking the list of sessions.
Once we have a session and it looks like its bc is up, take a reference
to it by searching for it in the session hashtable. Hold that reference
over the lifetime of the callback, and put it when the RPC completes.
Note that we must drop the cid_mutex in order to clean up the session
reference, so we do so just before submitting the RPC, to make it
possible if it fails.
We also want to walk all of the sessions without waiting for the first
pass, and then do a pass with waiting. That was done before with a
for loop, but this changes that to use a "wait" bool instead.
Change-Id: Ic0c0eca52182009289634a985c2e4d1e34dcb895
Signed-off-by: Jeff Layton <jlay...@redhat.com>
---
M src/MainNFSD/nfs_rpc_callback.c
1 file changed, 66 insertions(+), 37 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha
refs/changes/00/377400/1
--
To view, visit https://review.gerrithub.io/377400
To unsubscribe, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic0c0eca52182009289634a985c2e4d1e34dcb895
Gerrit-Change-Number: 377400
Gerrit-PatchSet: 1
Gerrit-Owner: Jeff Layton <jlay...@redhat.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
