OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-3458-g5acb1a0

Gerrit Code Review gerrit@openafs.org
Sun, 24 Feb 2013 05:23:39 -0800 (PST)


The following commit has been merged in the master branch:
commit 5acb1a0f323a6692cca66ee3aa4c7e5f1a0f24f2
Author: Simon Wilkinson <sxw@your-file-system.com>
Date:   Fri Feb 22 16:54:17 2013 +0000

    Unix CM: Fix byte accounting for storebehind
    
    In the current version of CacheStoreDCaches, the stored variable is
    maintained within the for loop that iterates over the chunk list. This
    means that it is reset to 0 each time we handle a new chunk.
    
    However, this means that our progress is no longer accurately tracked,
    as (bytes - stored) no longer gives the number of bytes which remain to
    be transfered. In fact, as stored is zeroed with each loop iteration,
    (bytes - stored) == bytes. This means that store behind is no longer
    activated according to the users settings.
    
    Prior to commit 334114ac58b0039ae90d7e29fa2f019fe068bd79, the
    stored variable was maintained within the outer, function, scope.
    Just move it back there to restore the previous behaviour.
    
    Caught by clang-analyzer
    
    Change-Id: I248ed81f1331f4d3d4ed2e1206acebb0a8200762
    Reviewed-on: http://gerrit.openafs.org/9237
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@your-file-system.com>

 src/afs/afs_fetchstore.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

-- 
OpenAFS Master Repository