afs: Refactor osi_vnhold/AFS_FAST_HOLD 71/13971/6
authorAndrew Deason <adeason@dson.org>
Tue, 19 Nov 2019 05:17:12 +0000 (23:17 -0600)
committerBenjamin Kaduk <kaduk@mit.edu>
Fri, 22 May 2020 20:27:18 +0000 (16:27 -0400)
commit13e44b2b200cd99d0df4e03cf6413d3a6915783f
tree3105a17256b38a54c58b844abad364dd05cbb404
parentd01398731550b8a93b293800642c3e1592099114
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.

Change-Id: Id2f3717be6c305d06305685247ac789815e1ebf7
Reviewed-on: https://gerrit.openafs.org/13971
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
34 files changed:
src/afs/AIX/osi_machdep.h
src/afs/AIX/osi_vcache.c
src/afs/DARWIN/osi_machdep.h
src/afs/DARWIN/osi_vcache.c
src/afs/DARWIN/osi_vfsops.c
src/afs/DARWIN/osi_vm.c
src/afs/FBSD/osi_machdep.h
src/afs/FBSD/osi_vcache.c
src/afs/HPUX/osi_machdep.h
src/afs/HPUX/osi_vcache.c
src/afs/IRIX/osi_machdep.h
src/afs/IRIX/osi_vcache.c
src/afs/LINUX/osi_machdep.h
src/afs/LINUX/osi_vcache.c
src/afs/NBSD/osi_machdep.h
src/afs/NBSD/osi_vcache.c
src/afs/OBSD/osi_machdep.h
src/afs/OBSD/osi_vcache.c
src/afs/SOLARIS/osi_machdep.h
src/afs/SOLARIS/osi_vcache.c
src/afs/SOLARIS/osi_vfsops.c
src/afs/UKERNEL/osi_machdep.h
src/afs/UKERNEL/osi_vcache.c
src/afs/VNOPS/afs_vnop_fid.c
src/afs/VNOPS/afs_vnop_lookup.c
src/afs/VNOPS/afs_vnop_remove.c
src/afs/afs.h
src/afs/afs_callback.c
src/afs/afs_daemons.c
src/afs/afs_osi.h
src/afs/afs_osidnlc.c
src/afs/afs_pioctl.c
src/afs/afs_vcache.c
src/afs/afs_volume.c