sunrpc: centralise most calls to svc_xprt_received
authorNeil Brown <neilb@suse.de>
Mon, 1 Mar 2010 05:51:14 +0000 (16:51 +1100)
committerJ. Bruce Fields <bfields@citi.umich.edu>
Mon, 3 May 2010 12:33:00 +0000 (08:33 -0400)
commitb48fa6b99100dc7772af3cd276035fcec9719ceb
tree415dba87c6c5de0ab85a8493f864838f4f3cc005
parent26c0c75e69265961e891ed80b38fb62a548ab371
sunrpc: centralise most calls to svc_xprt_received

svc_xprt_received must be called when ->xpo_recvfrom has finished
receiving a message, so that the XPT_BUSY flag will be cleared and
if necessary, requeued for further work.

This call is currently made in each ->xpo_recvfrom function, often
from multiple different points.  In each case it is the earliest point
on a particular path where it is known that the protection provided by
XPT_BUSY is no longer needed.

However there are (still) some error paths which do not call
svc_xprt_received, and requiring each ->xpo_recvfrom to make the call
does not encourage robustness.

So: move the svc_xprt_received call to be made just after the
call to ->xpo_recvfrom(), and move it of the various ->xpo_recvfrom
methods.

This means that it may not be called at the earliest possible instant,
but this is unlikely to be a measurable performance issue.

Note that there are still other calls to svc_xprt_received as it is
also needed when an xprt is newly created.

Signed-off-by: NeilBrown <neilb@suse.de>
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
net/sunrpc/svc_xprt.c
net/sunrpc/svcsock.c
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c