OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-604-gb6576d8

Gerrit Code Review gerrit@openafs.org
Wed, 29 Dec 2010 11:51:15 -0800 (PST)


The following commit has been merged in the master branch:
commit b6576d8cb554af50dfe7b6d9b668bd35ff202a83
Author: Jeffrey Altman <jaltman@your-file-system.com>
Date:   Wed Dec 29 11:35:17 2010 -0500

    Windows: buf_CleanAsync scp->fid == bp->fid
    
    If buf_CleanAsync or buf_CleanAsyncLocked are called
    with a non-NULL cm_scache_t parameter, that status object's
    fid must be the same as the associated cm_buf_t object.
    If not, the wrong locks will be held.
    
    If the cm_scache_t parameter is NULL and cm_FindSCache()
    returns NULL, it means that the cm_scache_t object associated
    with the bp->fid has been flushed from the cache.  cm_GetSCache()
    must therefore be called to allocate a new status object for the
    FID.  If the status object cannot be allocated, then any dirty
    data stored in the buffer will be discarded.
    
    Change-Id: Ie5d4eb8a1090d4b3c0753b7ddee2de0799485a2e
    Reviewed-on: http://gerrit.openafs.org/3604
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Tested-by: Jeffrey Altman <jaltman@openafs.org>
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>

 src/WINNT/afsd/cm_buf.c |   64 ++++++++++++++++++++++++++++++++--------------
 1 files changed, 44 insertions(+), 20 deletions(-)

-- 
OpenAFS Master Repository