The following commit has been merged in the master branch:
commit e358807347b9ca2cbdd75bc7b0bdb201395c0d26
Author: Andrew Deason <[email protected]>
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 <[email protected]>
Tested-by: BuildBot <[email protected]>
Reviewed-by: Michael Meffie <[email protected]>
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
_______________________________________________
OpenAFS-cvs mailing list
[email protected]
https://lists.openafs.org/mailman/listinfo/openafs-cvs
