OpenAFS Master Repository branch, openafs-stable-1_8_x, updated. openafs-stable-1_8_8-13-g33b6729
Gerrit Code Review
gerrit@openafs.org
Thu, 23 Sep 2021 16:16:20 -0400
The following commit has been merged in the openafs-stable-1_8_x branch:
commit 33b672918f1c282c4d74934e132e41357c9a4fdd
Author: Andrew Deason <adeason@dson.org>
Date: Mon Nov 18 23:17:12 2019 -0600
afs: Refactor osi_vnhold/AFS_FAST_HOLD
Make a few changes to osi_vnhold and AFS_FAST_HOLD:
- Currently, the second argument of osi_vnhold ("retry") is never used
by any implementation. Get rid of it.
- AFS_FAST_HOLD() is the same as osi_vnhold(). Get rid of
AFS_FAST_HOLD, and just have all callers use osi_vnhold instead.
- Allow osi_vnhold to return an error, and adjust callers to handle
it.
- Change osi_vnhold to be a real function, instead of a macro, to make
nontrivial implementations less cumbersome.
Most platforms never return an error from osi_vnhold(), so the added
code paths to check the return value of osi_vnhold() will not trigger.
However, this lets us add future commits that do make osi_vnhold()
return an error.
Reviewed-on: https://gerrit.openafs.org/13971
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
(cherry picked from commit 13e44b2b200cd99d0df4e03cf6413d3a6915783f)
Change-Id: Ia3e1c4e45a870cd095f90adc66880eb524a10c1f
Reviewed-on: https://gerrit.openafs.org/14793
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Tim Creech <tcreech@tcreech.com>
Reviewed-by: Cheyenne Wills <cwills@sinenomine.net>
Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>
src/afs/AIX/osi_machdep.h | 2 --
src/afs/AIX/osi_vcache.c | 6 ++++
src/afs/DARWIN/osi_machdep.h | 1 -
src/afs/DARWIN/osi_vcache.c | 7 +++++
src/afs/DARWIN/osi_vfsops.c | 2 +-
src/afs/DARWIN/osi_vm.c | 2 +-
src/afs/FBSD/osi_machdep.h | 2 --
src/afs/FBSD/osi_vcache.c | 6 ++++
src/afs/HPUX/osi_machdep.h | 1 -
src/afs/HPUX/osi_vcache.c | 6 ++++
src/afs/IRIX/osi_machdep.h | 2 --
src/afs/IRIX/osi_vcache.c | 6 ++++
src/afs/LINUX/osi_machdep.h | 1 -
src/afs/LINUX/osi_vcache.c | 12 +++++++-
src/afs/NBSD/osi_machdep.h | 1 -
src/afs/NBSD/osi_vcache.c | 6 ++++
src/afs/OBSD/osi_machdep.h | 1 -
src/afs/OBSD/osi_vcache.c | 5 ++++
src/afs/SOLARIS/osi_machdep.h | 13 ---------
src/afs/SOLARIS/osi_vcache.c | 16 +++++++++++
src/afs/SOLARIS/osi_vfsops.c | 2 +-
src/afs/UKERNEL/osi_machdep.h | 2 --
src/afs/UKERNEL/osi_vcache.c | 6 ++++
src/afs/VNOPS/afs_vnop_fid.c | 2 +-
src/afs/VNOPS/afs_vnop_lookup.c | 16 +++++++++--
src/afs/VNOPS/afs_vnop_remove.c | 2 +-
src/afs/afs.h | 2 +-
src/afs/afs_callback.c | 16 ++++-------
src/afs/afs_daemons.c | 2 +-
src/afs/afs_osi.h | 12 ++++++--
src/afs/afs_osidnlc.c | 6 +++-
src/afs/afs_pioctl.c | 4 ++-
src/afs/afs_vcache.c | 63 +++++++++++++++++++++--------------------
src/afs/afs_volume.c | 4 ++-
34 files changed, 154 insertions(+), 83 deletions(-)
--
OpenAFS Master Repository