>From Matt Benjamin <mbenja...@redhat.com>:
Matt Benjamin has uploaded this change for review. (
https://review.gerrithub.io/359858
Change subject: dispatch: call SVC_STAT before svcerr_*
......................................................................
dispatch: call SVC_STAT before svcerr_*
Ancient is_rpc_call_valid() predicate had to release and take
locks to avoid deadlocks, and this got missed in recent changes
to svc_dg. In particular, svc_dg_recv() takes and holds the
transport lock until a following svc_dg_stat().
Follow the new intent by pulling the result of the predicate into
thr_decode_rpc_request() and calling the RPC error routines after
SVC_STAT(), if needed.
Change-Id: Ibeb2995fbb784e33b10ca110495641c15ba98baa
Signed-off-by: Matt Benjamin <mbenja...@redhat.com>
---
M src/MainNFSD/nfs_rpc_dispatcher_thread.c
1 file changed, 79 insertions(+), 57 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha
refs/changes/58/359858/1
--
To view, visit https://review.gerrithub.io/359858
To unsubscribe, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibeb2995fbb784e33b10ca110495641c15ba98baa
Gerrit-Change-Number: 359858
Gerrit-PatchSet: 1
Gerrit-Owner: Matt Benjamin <mbenja...@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
