OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-stable-1_6_0b-199-ga9ee7f8

Gerrit Code Review gerrit@openafs.org
Fri, 16 Dec 2011 02:59:09 -0800 (PST)


The following commit has been merged in the openafs-stable-1_6_x branch:
commit a9ee7f8f821f73ba0a1dd6a8fd8ed4abc8ef9040
Author: Marc Dionne <marc.c.dionne@gmail.com>
Date:   Tue Apr 5 21:30:20 2011 -0400

    ubik: don't rely on timeout value after select()
    
    The value of timeout after a select() call should be considered
    undefined; relying on its value is not portable.
    Since IOMGR_Select doesn't modify the timeout it is given, the
    intention of the code seems to be to wait for gradually increasing
    timeout values, starting at 50ms.  At least under Linux, the
    timeout gets set to 0 by select() if it waited for the full specified
    time, resulting in a much shorter maximum possible wait period.
    
    Initialize the timeout value for each loop according to the existing
    logic, to get consistent behaviour between the lwp and pthreaded code.
    
    Reviewed-on: http://gerrit.openafs.org/4441
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    (cherry picked from commit 0b510fe30afb34202342364e96bd9030052e1567)
    
    Change-Id: I24eb4d4b1f758f33e3517671cb576ff23e641fb3
    Reviewed-on: http://gerrit.openafs.org/6282
    Tested-by: Derrick Brashear <shadow@dementix.org>
    Reviewed-by: Derrick Brashear <shadow@dementix.org>

 src/ubik/recovery.c |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

-- 
OpenAFS Master Repository