OpenAFS Master Repository branch, master, updated. BP-openafs-stable-1_8_x-599-g13e44b2

Gerrit Code Review gerrit@openafs.org
Fri, 22 May 2020 16:28:57 -0400


The following commit has been merged in the master branch:
commit 13e44b2b200cd99d0df4e03cf6413d3a6915783f
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.
    
    Change-Id: Id2f3717be6c305d06305685247ac789815e1ebf7
    Reviewed-on: https://gerrit.openafs.org/13971
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

 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       |    7 ++++
 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       |    7 ++++
 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, 156 insertions(+), 83 deletions(-)

-- 
OpenAFS Master Repository