OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-722-g296cf7d

[Gerrit Code Review]

The following commit has been merged in the master branch:
commit 296cf7d1630c57874b870e161bac5db9fe4dab48
Author: Marc Dionne <marc.c.dionne@gmail.com>
Date:   Thu Jan 27 19:07:32 2011 -0500

    ubik: always prefer a dirty cache page for write transactions
    
    If a write transaction is running concurrently with a read transaction,
    a DRead in the write transaction may return a clean cache page brought
    in by the read transaction, instead of a previously written dirty page
    from the same transaction.  This can result in loss of the written data.
    
    Fix by making sure there is not a dirty version of the requested page
    before returning a clean one.
    
    Bug spotted by Jeffrey Hutzelman.
    
    Change-Id: I20543693c98218d8ec1f791508d9404043819376
    Reviewed-on: http://gerrit.openafs.org/3764
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>

 src/ubik/disk.c |   22 +++++++++++++++++-----
 1 files changed, 17 insertions(+), 5 deletions(-)

-- 
OpenAFS Master Repository