OpenAFS Master Repository branch, openafs-devel-1_7_x, updated. openafs-devel-1_7_6-11-gde60151

Gerrit Code Review gerrit@openafs.org
Fri, 17 Feb 2012 16:10:44 -0800 (PST)


The following commit has been merged in the openafs-devel-1_7_x branch:
commit de60151ebdccf92dce4d982303c2dcd6c53250d2
Author: Jeffrey Altman <jaltman@your-file-system.com>
Date:   Thu Feb 16 23:50:18 2012 -0500

    Windows: VolumeCB->ObjectInfoTree.TreeLock Deadlock
    
    AFSPrimaryVolumeWorkerThread held the VolumeCB->ObjectInfoTree.TreeLock
    exclusively across calls to AFSCleanupFcb() which in turn triggers
    a file extent release to the service which can in turn result in
    an object invalidation.  Processing the invalidation requires shared
    access to VolumeCB->ObjectInfoTree.TreeLock which results in a deadlock.
    
    This patch alters the processing of AFSPrimaryVolumeWorkerThread
    so that the VolumeCB->ObjectInfoTree.TreeLock is not held across
    the AFSCleanupFcb() calls.
    
    FIXES 130431
    
    Reviewed-on: http://gerrit.openafs.org/6724
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Peter Scott <pscott@kerneldrivers.com>
    Reviewed-by: Jeffrey Altman <jaltman@secure-endpoints.com>
    Tested-by: Jeffrey Altman <jaltman@secure-endpoints.com>
    (cherry picked from commit 780e497b32a927e008474a63b0427eca5d5a8877)
    
    Change-Id: I55198e24138984fdae8cc87cbf9b0dac49aa9ed8
    Reviewed-on: http://gerrit.openafs.org/6731
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Jeffrey Altman <jaltman@secure-endpoints.com>
    Tested-by: Jeffrey Altman <jaltman@secure-endpoints.com>

 src/WINNT/afsrdr/kernel/lib/AFSWorker.cpp |  122 +++++++++++------------------
 1 files changed, 45 insertions(+), 77 deletions(-)

-- 
OpenAFS Master Repository