OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_1-48-ge358807

[Gerrit Code Review]

The following commit has been merged in the master branch:
commit e358807347b9ca2cbdd75bc7b0bdb201395c0d26
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Fri Feb 21 10:49:09 2020 -0500

    opr: Assert opr_cv_timedwait return codes
    
    Almost all of our opr locking primitives assert that they did not
    receive an error from pthreads. opr_cv_timedwait is the exception to
    this, since it needs to return two possible codes that aren't
    internal failures: 0 and ETIMEDOUT. As a result, most callers assert
    that our returned code is one of these two values.
    
    To make opr_cv_timedwait act more like our other locking primitives,
    removing the need to add opr_Assert()s everywhere, change
    opr_cv_timedwait to check that the returned code is 0 or ETIMEDOUT,
    and remove the relevant asserts from its callers.
    
    Change-Id: Ie9a62f2edb23969d66e4ed821af37077bc6400c4
    Reviewed-on: https://gerrit.openafs.org/14079
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>

 src/fsprobe/fsprobe.c |    1 -
 src/opr/opr_lock.h    |   11 +++++++++--
 src/viced/viced.c     |    2 --
 src/xstat/xstat_cm.c  |    1 -
 src/xstat/xstat_fs.c  |    1 -
 5 files changed, 9 insertions(+), 7 deletions(-)

-- 
OpenAFS Master Repository