OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-stable-1_6_1pre2-27-ga498e1f

Gerrit Code Review gerrit@openafs.org
Thu, 23 Feb 2012 09:36:37 -0800 (PST)


The following commit has been merged in the openafs-stable-1_6_x branch:
commit a498e1ff26fe01cc0a60474f76ff8fa32e0086ba
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Fri Feb 17 17:12:46 2012 -0600

    viced: Relax "h_TossStuff_r failed" warnings
    
    Currently, h_TossStuff_r bails out and logs a message if we detect
    that somebody grabbed a reference or locked the host while we tried to
    h_NBLock_r. The reasoning for this is that it is not legal for anyone
    to h_Hold_r a host that has HOSTDELETED set (but the error is
    detectable and recoverable); callers are supposed to check for
    HOSTDELETED and not hold a host in that case.
    
    However, HOSTDELETED may not be set when h_TossStuff_r is called,
    since we call it if either HOSTDELETED _or_ CLIENTDELETED are set. If
    CLIENTDELETED is set and HOSTDELETED is not, it's perfectly fine (and
    necessary) for callers to grab a reference to the host. So, if that's
    what is going on, don't log a message, since that's normal behavior.
    
    Check for HOSTDELETED before we h_NBLock_r, since it is technically
    possible (and legal) for someone to grab a reference to the host and
    somehow set HOSTDELETED while we wait for h_NBLock_r to return. Also
    log the flags when we see this message.
    
    Reviewed-on: http://gerrit.openafs.org/6733
    Reviewed-by: Alistair Ferguson <alistair.ferguson@mac.com>
    Reviewed-by: Derrick Brashear <shadow@dementix.org>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    (cherry picked from commit fe4e52655ce7e5a8e5f6c23cde678fc66c3db490)
    
    Change-Id: Ic1b72c808aec158d99f088a3144e86adf969efcc
    Reviewed-on: http://gerrit.openafs.org/6770
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Alistair Ferguson <alistair.ferguson@mac.com>
    Reviewed-by: Derrick Brashear <shadow@dementix.org>

 src/viced/host.c |   25 +++++++++++++++++++------
 1 files changed, 19 insertions(+), 6 deletions(-)

-- 
OpenAFS Master Repository