OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-1311-ge8c4fc4

Gerrit Code Review gerrit@openafs.org
Tue, 21 Jun 2011 18:59:18 -0700 (PDT)


The following commit has been merged in the master branch:
commit 784babbea1247660f39463403233589a74c6e73b
Author: Simon Wilkinson <sxw@your-file-system.com>
Date:   Sat Jun 18 11:48:45 2011 +0100

    rx: Make clock_Add correctly add to itself
    
    With the existing clock_Add code, the following:
        struct clock a = {2, 800000};
        clock_Add(&a, &a);
    gives a clock value of {6, 600000}, rather than the expected {5, 60000}.
    
    This is because the ordering of instructions leads it to double count
    the carry on the seconds field. Reorder the instructions so that the
    carry is correctly applied.
    
    Change-Id: Ia71b387ce521a11e4caf9ec200907efe1d2be8ff
    Reviewed-on: http://gerrit.openafs.org/4864
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    Tested-by: Jeffrey Altman <jaltman@openafs.org>

 src/rx/rx_clock.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

-- 
OpenAFS Master Repository