OpenAFS Master Repository branch, master, updated. BP-openafs-stable-1_8_x-262-g0835d7c
Gerrit Code Review
gerrit@openafs.org
Sat, 6 Oct 2018 16:06:52 -0400
The following commit has been merged in the master branch:
commit 0835d7c2a183f896096684df06258aefd297f080
Author: Michael Meffie <mmeffie@sinenomine.net>
Date: Fri Mar 16 09:25:18 2018 -0500
afs: make sure to call afs_Analyze after afs_Conn
The afs_Conn function is used to pick a connection for a given RPC. The
RPC is normally wrapped within a do-while loop which calls afs_Analyze
to handle the RPC code and manage the server connection references.
Among other things, afs_Analyze can mark the server as down, blacklist
idle servers, etc.
There are some special cases in which we break out of this do-while loop
early, by putting the connection reference given by afs_Conn and then
jumping out of the loop.
In these cases, be sure to call afs_Analyze to put the server connection
we got from afs_Conn, and to handle the RPC return code, possibly
marking the server as down or blacklisted.
Change-Id: Ic2c43f20d153376b93d79bbb5145914f8e478957
Reviewed-on: https://gerrit.openafs.org/13288
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
src/afs/afs_bypasscache.c | 7 ++++++-
src/afs/afs_dcache.c | 17 +++++++++++------
2 files changed, 17 insertions(+), 7 deletions(-)
--
OpenAFS Master Repository