FBSD: Use VM_CNT_INC/VM_CNT_ADD on FreeBSD 12
[openafs.git] / src / afs / FBSD / osi_vnodeops.c
index 5e98588..e314e4c 100644 (file)
@@ -47,8 +47,6 @@
 #include <afsconfig.h>
 #include <afs/param.h>
 
-RCSID
-    ("$Header$");
 
 #include <afs/sysincludes.h>   /* Standard vendor system headers */
 #include <afsincludes.h>       /* Afs-based standard headers */
@@ -56,181 +54,54 @@ RCSID
 #include <sys/malloc.h>
 #include <sys/namei.h>
 #include <sys/unistd.h>
-#ifndef AFS_FBSD50_ENV
-#include <vm/vm_zone.h>
+#if __FreeBSD_version >= 1000030
+#include <sys/rwlock.h>
 #endif
 #include <vm/vm_page.h>
 #include <vm/vm_object.h>
 #include <vm/vm_pager.h>
 #include <vm/vnode_pager.h>
+#include <sys/vmmeter.h>
 extern int afs_pbuf_freecnt;
 
-#ifdef AFS_FBSD60_ENV
-static vop_access_t    afs_vop_access;
-static vop_advlock_t   afs_vop_advlock;
-static vop_close_t     afs_vop_close;
-static vop_create_t    afs_vop_create;
-static vop_fsync_t     afs_vop_fsync;
-static vop_getattr_t   afs_vop_getattr;
-static vop_getpages_t  afs_vop_getpages;
-static vop_inactive_t  afs_vop_inactive;
-static vop_ioctl_t     afs_vop_ioctl;
-static vop_link_t      afs_vop_link;
-static vop_lookup_t    afs_vop_lookup;
-static vop_mkdir_t     afs_vop_mkdir;
-static vop_mknod_t     afs_vop_mknod;
-static vop_open_t      afs_vop_open;
-static vop_pathconf_t  afs_vop_pathconf;
-static vop_poll_t      afs_vop_poll;
-static vop_print_t     afs_vop_print;
-static vop_putpages_t  afs_vop_putpages;
-static vop_read_t      afs_vop_read;
-static vop_readdir_t   afs_vop_readdir;
-static vop_readlink_t  afs_vop_readlink;
-static vop_reclaim_t   afs_vop_reclaim;
-static vop_remove_t    afs_vop_remove;
-static vop_rename_t    afs_vop_rename;
-static vop_rmdir_t     afs_vop_rmdir;
-static vop_setattr_t   afs_vop_setattr;
-static vop_strategy_t  afs_vop_strategy;
-static vop_symlink_t   afs_vop_symlink;
-static vop_write_t     afs_vop_write;
-
-struct vop_vector afs_vnodeops = {
-       .vop_default =          &default_vnodeops,
-       .vop_access =           afs_vop_access,
-       .vop_advlock =          afs_vop_advlock,
-       .vop_close =            afs_vop_close,
-       .vop_create =           afs_vop_create,
-       .vop_fsync =            afs_vop_fsync,
-       .vop_getattr =          afs_vop_getattr,
-       .vop_getpages =         afs_vop_getpages,
-       .vop_inactive =         afs_vop_inactive,
-       .vop_ioctl =            afs_vop_ioctl,
-       .vop_lease =            VOP_NULL,
-       .vop_link =             afs_vop_link,
-       .vop_lookup =           afs_vop_lookup,
-       .vop_mkdir =            afs_vop_mkdir,
-       .vop_mknod =            afs_vop_mknod,
-       .vop_open =             afs_vop_open,
-       .vop_pathconf =         afs_vop_pathconf,
-       .vop_poll =             afs_vop_poll,
-       .vop_print =            afs_vop_print,
-       .vop_putpages =         afs_vop_putpages,
-       .vop_read =             afs_vop_read,
-       .vop_readdir =          afs_vop_readdir,
-       .vop_readlink =         afs_vop_readlink,
-       .vop_reclaim =          afs_vop_reclaim,
-       .vop_remove =           afs_vop_remove,
-       .vop_rename =           afs_vop_rename,
-       .vop_rmdir =            afs_vop_rmdir,
-       .vop_setattr =          afs_vop_setattr,
-       .vop_strategy =         afs_vop_strategy,
-       .vop_symlink =          afs_vop_symlink,
-       .vop_write =            afs_vop_write,
-};
-
-#else /* AFS_FBSD60_ENV */
-
-int afs_vop_lookup(struct vop_lookup_args *);
-int afs_vop_create(struct vop_create_args *);
-int afs_vop_mknod(struct vop_mknod_args *);
-int afs_vop_open(struct vop_open_args *);
-int afs_vop_close(struct vop_close_args *);
-int afs_vop_access(struct vop_access_args *);
-int afs_vop_getattr(struct vop_getattr_args *);
-int afs_vop_setattr(struct vop_setattr_args *);
-int afs_vop_read(struct vop_read_args *);
-int afs_vop_write(struct vop_write_args *);
-int afs_vop_getpages(struct vop_getpages_args *);
-int afs_vop_putpages(struct vop_putpages_args *);
-int afs_vop_ioctl(struct vop_ioctl_args *);
-static int afs_vop_pathconf(struct vop_pathconf_args *);
-int afs_vop_poll(struct vop_poll_args *);
-#ifndef AFS_FBSD50_ENV
-int afs_vop_mmap(struct vop_mmap_args *);
-#endif
-int afs_vop_fsync(struct vop_fsync_args *);
-int afs_vop_remove(struct vop_remove_args *);
-int afs_vop_link(struct vop_link_args *);
-int afs_vop_rename(struct vop_rename_args *);
-int afs_vop_mkdir(struct vop_mkdir_args *);
-int afs_vop_rmdir(struct vop_rmdir_args *);
-int afs_vop_symlink(struct vop_symlink_args *);
-int afs_vop_readdir(struct vop_readdir_args *);
-int afs_vop_readlink(struct vop_readlink_args *);
-int afs_vop_inactive(struct vop_inactive_args *);
-int afs_vop_reclaim(struct vop_reclaim_args *);
-int afs_vop_bmap(struct vop_bmap_args *);
-int afs_vop_strategy(struct vop_strategy_args *);
-int afs_vop_print(struct vop_print_args *);
-int afs_vop_advlock(struct vop_advlock_args *);
-
-
-
-/* Global vfs data structures for AFS. */
-vop_t **afs_vnodeop_p;
-struct vnodeopv_entry_desc afs_vnodeop_entries[] = {
-    {&vop_default_desc, (vop_t *) vop_defaultop},
-    {&vop_access_desc, (vop_t *) afs_vop_access},      /* access */
-    {&vop_advlock_desc, (vop_t *) afs_vop_advlock},    /* advlock */
-    {&vop_bmap_desc, (vop_t *) afs_vop_bmap},  /* bmap */
-#ifndef AFS_FBSD50_ENV
-    {&vop_bwrite_desc, (vop_t *) vop_stdbwrite},
-#endif
-    {&vop_close_desc, (vop_t *) afs_vop_close},        /* close */
-    {&vop_createvobject_desc, (vop_t *) vop_stdcreatevobject},
-    {&vop_destroyvobject_desc, (vop_t *) vop_stddestroyvobject},
-    {&vop_create_desc, (vop_t *) afs_vop_create},      /* create */
-    {&vop_fsync_desc, (vop_t *) afs_vop_fsync},        /* fsync */
-    {&vop_getattr_desc, (vop_t *) afs_vop_getattr},    /* getattr */
-    {&vop_getpages_desc, (vop_t *) afs_vop_getpages},  /* read */
-    {&vop_getvobject_desc, (vop_t *) vop_stdgetvobject},
-    {&vop_putpages_desc, (vop_t *) afs_vop_putpages},  /* write */
-    {&vop_inactive_desc, (vop_t *) afs_vop_inactive},  /* inactive */
-    {&vop_lease_desc, (vop_t *) vop_null},
-    {&vop_link_desc, (vop_t *) afs_vop_link},  /* link */
-    {&vop_lookup_desc, (vop_t *) afs_vop_lookup},      /* lookup */
-    {&vop_mkdir_desc, (vop_t *) afs_vop_mkdir},        /* mkdir */
-    {&vop_mknod_desc, (vop_t *) afs_vop_mknod},        /* mknod */
-#ifndef AFS_FBSD50_ENV
-    {&vop_mmap_desc, (vop_t *) afs_vop_mmap},  /* mmap */
-#endif
-    {&vop_open_desc, (vop_t *) afs_vop_open},  /* open */
-    {&vop_pathconf_desc, (vop_t *) afs_vop_pathconf},  /* pathconf */
-    {&vop_poll_desc, (vop_t *) afs_vop_poll},  /* select */
-    {&vop_print_desc, (vop_t *) afs_vop_print},        /* print */
-    {&vop_read_desc, (vop_t *) afs_vop_read},  /* read */
-    {&vop_readdir_desc, (vop_t *) afs_vop_readdir},    /* readdir */
-    {&vop_readlink_desc, (vop_t *) afs_vop_readlink},  /* readlink */
-    {&vop_reclaim_desc, (vop_t *) afs_vop_reclaim},    /* reclaim */
-    {&vop_remove_desc, (vop_t *) afs_vop_remove},      /* remove */
-    {&vop_rename_desc, (vop_t *) afs_vop_rename},      /* rename */
-    {&vop_rmdir_desc, (vop_t *) afs_vop_rmdir},        /* rmdir */
-    {&vop_setattr_desc, (vop_t *) afs_vop_setattr},    /* setattr */
-    {&vop_strategy_desc, (vop_t *) afs_vop_strategy},  /* strategy */
-    {&vop_symlink_desc, (vop_t *) afs_vop_symlink},    /* symlink */
-    {&vop_write_desc, (vop_t *) afs_vop_write},        /* write */
-    {&vop_ioctl_desc, (vop_t *) afs_vop_ioctl},        /* XXX ioctl */
-    /*{ &vop_seek_desc, afs_vop_seek }, *//* seek */
-    {NULL, NULL}
-};
-struct vnodeopv_desc afs_vnodeop_opv_desc =
-    { &afs_vnodeop_p, afs_vnodeop_entries };
-#endif /* AFS_FBSD60_ENV */
-
 #define GETNAME()       \
     struct componentname *cnp = ap->a_cnp; \
     char *name; \
-    MALLOC(name, char *, cnp->cn_namelen+1, M_TEMP, M_WAITOK); \
+    name = malloc(cnp->cn_namelen+1, M_TEMP, M_WAITOK); \
     memcpy(name, cnp->cn_nameptr, cnp->cn_namelen); \
     name[cnp->cn_namelen] = '\0'
 
-#define DROPNAME() FREE(name, M_TEMP)
+#define DROPNAME() free(name, M_TEMP)
+
+#ifdef LINK_MAX
+# define AFS_LINK_MAX LINK_MAX
+#else
+# define AFS_LINK_MAX (32767)
+#endif
+
+/*
+ * Here we define compatibility functions/macros for interfaces that
+ * have changed between different FreeBSD versions.
+ */
+static __inline void ma_vm_page_lock_queues(void) {};
+static __inline void ma_vm_page_unlock_queues(void) {};
+static __inline void ma_vm_page_lock(vm_page_t m) { vm_page_lock(m); };
+static __inline void ma_vm_page_unlock(vm_page_t m) { vm_page_unlock(m); };
+
+#if __FreeBSD_version >= 1000030
+#define AFS_VM_OBJECT_WLOCK(o) VM_OBJECT_WLOCK(o)
+#define AFS_VM_OBJECT_WUNLOCK(o)       VM_OBJECT_WUNLOCK(o)
+#else
+#define AFS_VM_OBJECT_WLOCK(o) VM_OBJECT_LOCK(o)
+#define AFS_VM_OBJECT_WUNLOCK(o)       VM_OBJECT_UNLOCK(o)
+#endif
 
-/* This is a bit of a cheat... */
-#ifdef AFS_FBSD50_ENV
-#define a_p a_td
+#ifdef VM_CNT_ADD
+# define AFS_VM_CNT_ADD(var, x) VM_CNT_ADD(var, x)
+# define AFS_VM_CNT_INC(var)    VM_CNT_INC(var)
+#else
+# define AFS_VM_CNT_ADD(var, x) PCPU_ADD(cnt.var, x)
+# define AFS_VM_CNT_INC(var)    PCPU_INC(cnt.var)
 #endif
 
 /*
@@ -247,7 +118,7 @@ afs_vop_pathconf(struct vop_pathconf_args *ap)
        error = 0;
        switch (ap->a_name) {
        case _PC_LINK_MAX:
-               *ap->a_retval = LINK_MAX;
+               *ap->a_retval = AFS_LINK_MAX;
                break;
        case _PC_NAME_MAX:
                *ap->a_retval = NAME_MAX;
@@ -326,7 +197,7 @@ afs_vop_pathconf(struct vop_pathconf_args *ap)
        return (error);
 }
 
-int
+static int
 afs_vop_lookup(ap)
      struct vop_lookup_args    /* {
                                 * struct vnodeop_desc * a_desc;
@@ -338,33 +209,50 @@ afs_vop_lookup(ap)
     int error;
     struct vcache *vcp;
     struct vnode *vp, *dvp;
-    register int flags = ap->a_cnp->cn_flags;
-    int lockparent;            /* 1 => lockparent flag is set */
-    int wantparent;            /* 1 => wantparent or lockparent flag */
-#ifdef AFS_FBSD50_ENV
-    struct thread *p = ap->a_cnp->cn_thread;
-#else
-    struct proc *p = ap->a_cnp->cn_proc;
-#endif
-    GETNAME();
+    int flags = ap->a_cnp->cn_flags;
 
-    lockparent = flags & LOCKPARENT;
-    wantparent = flags & (LOCKPARENT | WANTPARENT);
-
-    if (ap->a_dvp->v_type != VDIR) {
-       *ap->a_vpp = 0;
-       DROPNAME();
+    dvp = ap->a_dvp;
+    if (dvp->v_type != VDIR) {
        return ENOTDIR;
     }
-    dvp = ap->a_dvp;
+
+    if ((flags & ISDOTDOT) && (dvp->v_vflag & VV_ROOT))
+       return EIO;
+
+    GETNAME();
+
+#if __FreeBSD_version < 1000021
+    cnp->cn_flags |= MPSAFE; /* steel */
+#endif
+
+    /*
+     * Locking rules:
+     *
+     * - 'dvp' is locked by our caller. We must return it locked, whether we
+     * return success or error.
+     *
+     * - If the lookup succeeds, 'vp' must be locked before we return.
+     *
+     * - If we lock multiple vnodes, parent vnodes must be locked before
+     * children vnodes.
+     *
+     * As a result, looking up the parent directory (if 'flags' has ISDOTDOT
+     * set) is a bit of a special case. In that case, we must unlock 'dvp'
+     * before performing the lookup, since the lookup operation may lock the
+     * target vnode, and the target vnode is the parent of 'dvp' (so we must
+     * lock 'dvp' after locking the target vnode).
+     */
+
     if (flags & ISDOTDOT)
-       VOP_UNLOCK(dvp, 0, p);
+       VOP_UNLOCK(dvp, 0);
+
     AFS_GLOCK();
     error = afs_lookup(VTOAFS(dvp), name, &vcp, cnp->cn_cred);
     AFS_GUNLOCK();
+
     if (error) {
        if (flags & ISDOTDOT)
-           VOP_LOCK(dvp, LK_EXCLUSIVE | LK_RETRY, p);
+           vn_lock(dvp, LK_EXCLUSIVE | LK_RETRY);
        if ((cnp->cn_nameiop == CREATE || cnp->cn_nameiop == RENAME)
            && (flags & ISLASTCN) && error == ENOENT)
            error = EJUSTRETURN;
@@ -376,39 +264,26 @@ afs_vop_lookup(ap)
     }
     vp = AFSTOV(vcp);          /* always get a node if no error */
 
-    /* The parent directory comes in locked.  We unlock it on return
-     * unless the caller wants it left locked.
-     * we also always return the vnode locked. */
-
     if (flags & ISDOTDOT) {
-       vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, p);
-       /* always return the child locked */
-       if (lockparent && (flags & ISLASTCN)
-           && (error = vn_lock(dvp, LK_EXCLUSIVE, p))) {
-           vput(vp);
-           DROPNAME();
-           return (error);
-       }
+       /* Must lock 'vp' before 'dvp', since 'vp' is the parent vnode. */
+       vn_lock(vp, LK_EXCLUSIVE | LK_RETRY);
+       vn_lock(dvp, LK_EXCLUSIVE | LK_RETRY);
     } else if (vp == dvp) {
        /* they're the same; afs_lookup() already ref'ed the leaf.
         * It came in locked, so we don't need to ref OR lock it */
     } else {
-       if (!lockparent || !(flags & ISLASTCN))
-           VOP_UNLOCK(dvp, 0, p);      /* done with parent. */
-       vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, p);
-       /* always return the child locked */
+       vn_lock(vp, LK_EXCLUSIVE | LK_CANRECURSE | LK_RETRY);
     }
     *ap->a_vpp = vp;
 
-    if ((cnp->cn_nameiop == RENAME && wantparent && (flags & ISLASTCN))
-       || (cnp->cn_nameiop != LOOKUP && (flags & ISLASTCN)))
+    if (cnp->cn_nameiop != LOOKUP && (flags & ISLASTCN))
        cnp->cn_flags |= SAVENAME;
 
     DROPNAME();
     return error;
 }
 
-int
+static int
 afs_vop_create(ap)
      struct vop_create_args    /* {
                                 * struct vnode *a_dvp;
@@ -419,12 +294,7 @@ afs_vop_create(ap)
 {
     int error = 0;
     struct vcache *vcp;
-    register struct vnode *dvp = ap->a_dvp;
-#ifdef AFS_FBSD50_ENV
-    struct thread *p = ap->a_cnp->cn_thread;
-#else
-    struct proc *p = ap->a_cnp->cn_proc;
-#endif
+    struct vnode *dvp = ap->a_dvp;
     GETNAME();
 
     AFS_GLOCK();
@@ -440,7 +310,7 @@ afs_vop_create(ap)
 
     if (vcp) {
        *ap->a_vpp = AFSTOV(vcp);
-       vn_lock(AFSTOV(vcp), LK_EXCLUSIVE | LK_RETRY, p);
+       vn_lock(AFSTOV(vcp), LK_EXCLUSIVE | LK_RETRY);
     } else
        *ap->a_vpp = 0;
 
@@ -448,7 +318,7 @@ afs_vop_create(ap)
     return error;
 }
 
-int
+static int
 afs_vop_mknod(ap)
      struct vop_mknod_args     /* {
                                 * struct vnode *a_dvp;
@@ -460,34 +330,14 @@ afs_vop_mknod(ap)
     return (ENODEV);
 }
 
-#if 0
 static int
-validate_vops(struct vnode *vp, int after)
-{
-    int ret = 0;
-    struct vnodeopv_entry_desc *this;
-    for (this = afs_vnodeop_entries; this->opve_op; this++) {
-       if (vp->v_op[this->opve_op->vdesc_offset] != this->opve_impl) {
-           if (!ret) {
-               printf("v_op %d ", after);
-               vprint("check", vp);
-           }
-           ret = 1;
-           printf("For oper %d (%s), func is %p, not %p",
-                  this->opve_op->vdesc_offset, this->opve_op->vdesc_name,
-                  vp->v_op[this->opve_op->vdesc_offset], this->opve_impl);
-       }
-    }
-    return ret;
-}
-#endif
-int
 afs_vop_open(ap)
      struct vop_open_args      /* {
                                 * struct vnode *a_vp;
                                 * int  a_mode;
                                 * struct ucred *a_cred;
-                                * struct proc *a_p;
+                                * struct thread *a_td;
+                                * struct file *a_fp;
                                 * } */ *ap;
 {
     int error;
@@ -500,73 +350,85 @@ afs_vop_open(ap)
        panic("AFS open changed vnode!");
 #endif
     AFS_GUNLOCK();
-#ifdef AFS_FBSD60_ENV
-    vnode_create_vobject(ap->a_vp, vc->m.Length, ap->a_td);
-#endif
+    vnode_create_vobject(ap->a_vp, vc->f.m.Length, ap->a_td);
     osi_FlushPages(vc, ap->a_cred);
     return error;
 }
 
-int
+static int
 afs_vop_close(ap)
      struct vop_close_args     /* {
                                 * struct vnode *a_vp;
                                 * int  a_fflag;
                                 * struct ucred *a_cred;
-                                * struct proc *a_p;
+                                * struct thread *a_td;
                                 * } */ *ap;
 {
-    int code;
-    struct vcache *avc = VTOAFS(ap->a_vp);
+    int code, iflag;
+    struct vnode *vp = ap->a_vp;
+    struct vcache *avc = VTOAFS(vp);
+
+    VI_LOCK(vp);
+    iflag = vp->v_iflag & VI_DOOMED;
+    VI_UNLOCK(vp);
+    if (iflag & VI_DOOMED) {
+        /* osi_FlushVCache (correctly) calls vgone() on recycled vnodes, we don't
+         * have an afs_close to process, in that case */
+        if (avc->opens != 0)
+            panic("afs_vop_close: doomed vnode %p has vcache %p with non-zero opens %d\n",
+                  vp, avc, avc->opens);
+        return 0;
+    }
+
     AFS_GLOCK();
     if (ap->a_cred)
        code = afs_close(avc, ap->a_fflag, ap->a_cred);
     else
        code = afs_close(avc, ap->a_fflag, afs_osi_credp);
-    osi_FlushPages(avc, ap->a_cred);   /* hold bozon lock, but not basic vnode lock */
+    osi_FlushPages(avc, ap->a_cred);   /* hold GLOCK, but not basic vnode lock */
     AFS_GUNLOCK();
     return code;
 }
 
-int
+static int
 afs_vop_access(ap)
      struct vop_access_args    /* {
                                 * struct vnode *a_vp;
-                                * int  a_mode;
+                                * accmode_t a_accmode;
                                 * struct ucred *a_cred;
-                                * struct proc *a_p;
+                                * struct thread *a_td;
                                 * } */ *ap;
 {
     int code;
     AFS_GLOCK();
-    code = afs_access(VTOAFS(ap->a_vp), ap->a_mode, ap->a_cred);
+    code = afs_access(VTOAFS(ap->a_vp), ap->a_accmode, ap->a_cred);
     AFS_GUNLOCK();
     return code;
 }
 
-int
+static int
 afs_vop_getattr(ap)
      struct vop_getattr_args   /* {
                                 * struct vnode *a_vp;
                                 * struct vattr *a_vap;
                                 * struct ucred *a_cred;
-                                * struct proc *a_p;
                                 * } */ *ap;
 {
     int code;
+
     AFS_GLOCK();
     code = afs_getattr(VTOAFS(ap->a_vp), ap->a_vap, ap->a_cred);
     AFS_GUNLOCK();
+
     return code;
 }
 
-int
+static int
 afs_vop_setattr(ap)
      struct vop_setattr_args   /* {
                                 * struct vnode *a_vp;
                                 * struct vattr *a_vap;
                                 * struct ucred *a_cred;
-                                * struct proc *a_p;
                                 * } */ *ap;
 {
     int code;
@@ -576,7 +438,7 @@ afs_vop_setattr(ap)
     return code;
 }
 
-int
+static int
 afs_vop_read(ap)
      struct vop_read_args      /* {
                                 * struct vnode *a_vp;
@@ -589,8 +451,8 @@ afs_vop_read(ap)
     int code;
     struct vcache *avc = VTOAFS(ap->a_vp);
     AFS_GLOCK();
-    osi_FlushPages(avc, ap->a_cred);   /* hold bozon lock, but not basic vnode lock */
-    code = afs_read(avc, ap->a_uio, ap->a_cred, 0, 0, 0);
+    osi_FlushPages(avc, ap->a_cred);   /* hold GLOCK, but not basic vnode lock */
+    code = afs_read(avc, ap->a_uio, ap->a_cred, 0);
     AFS_GUNLOCK();
     return code;
 }
@@ -599,121 +461,136 @@ afs_vop_read(ap)
  *     struct vnode *a_vp;
  *     vm_page_t *a_m;
  *     int a_count;
- *     int a_reqpage;
- *     vm_oofset_t a_offset;
+ *     int *a_rbehind;
+ *     int *a_rahead;
  * };
  */
-int
+static int
 afs_vop_getpages(struct vop_getpages_args *ap)
 {
     int code;
-    int i, nextoff, size, toff, npages;
+    int i, nextoff, size, toff, npages, count;
     struct uio uio;
     struct iovec iov;
     struct buf *bp;
     vm_offset_t kva;
     vm_object_t object;
+    vm_page_t *pages;
     struct vnode *vp;
     struct vcache *avc;
 
-#ifdef AFS_FBSD50_ENV
-    GIANT_REQUIRED;
-#endif
+    memset(&uio, 0, sizeof(uio));
+    memset(&iov, 0, sizeof(iov));
+
     vp = ap->a_vp;
     avc = VTOAFS(vp);
+    pages = ap->a_m;
+#ifdef FBSD_VOP_GETPAGES_BUSIED
+    npages = ap->a_count;
+    if (ap->a_rbehind)
+        *ap->a_rbehind = 0;
+    if (ap->a_rahead)
+        *ap->a_rahead = 0;
+#else
+    npages = btoc(ap->a_count);
+#endif
+
     if ((object = vp->v_object) == NULL) {
        printf("afs_getpages: called with non-merged cache vnode??\n");
        return VM_PAGER_ERROR;
     }
-    npages = btoc(ap->a_count);
+
     /*
      * If the requested page is partially valid, just return it and
      * allow the pager to zero-out the blanks.  Partially valid pages
      * can only occur at the file EOF.
      */
-
     {
-       vm_page_t m = ap->a_m[ap->a_reqpage];
-
-#ifdef AFS_FBSD50_ENV
-       VM_OBJECT_LOCK(object);
-       vm_page_lock_queues();
-#endif
+#ifdef FBSD_VOP_GETPAGES_BUSIED
+       AFS_VM_OBJECT_WLOCK(object);
+       ma_vm_page_lock_queues();
+       if(pages[npages - 1]->valid != 0) {
+           if (--npages == 0) {
+               ma_vm_page_unlock_queues();
+               AFS_VM_OBJECT_WUNLOCK(object);
+               return (VM_PAGER_OK);
+           }
+       }
+#else
+       vm_page_t m = pages[ap->a_reqpage];
+       AFS_VM_OBJECT_WLOCK(object);
+       ma_vm_page_lock_queues();
        if (m->valid != 0) {
            /* handled by vm_fault now        */
            /* vm_page_zero_invalid(m, TRUE); */
            for (i = 0; i < npages; ++i) {
-               if (i != ap->a_reqpage)
-                   vm_page_free(ap->a_m[i]);
+               if (i != ap->a_reqpage) {
+                   ma_vm_page_lock(pages[i]);
+                   vm_page_free(pages[i]);
+                   ma_vm_page_unlock(pages[i]);
+               }
            }
-#ifdef AFS_FBSD50_ENV
-           vm_page_unlock_queues();
-           VM_OBJECT_UNLOCK(object);
-#endif
+           ma_vm_page_unlock_queues();
+           AFS_VM_OBJECT_WUNLOCK(object);
            return (0);
        }
-#ifdef AFS_FBSD50_ENV
-       vm_page_unlock_queues();
-       VM_OBJECT_UNLOCK(object);
 #endif
+       ma_vm_page_unlock_queues();
+       AFS_VM_OBJECT_WUNLOCK(object);
     }
     bp = getpbuf(&afs_pbuf_freecnt);
 
     kva = (vm_offset_t) bp->b_data;
-    pmap_qenter(kva, ap->a_m, npages);
-#ifdef AFS_FBSD50_ENV
-    cnt.v_vnodein++;
-    cnt.v_vnodepgsin += npages;
-#endif
+    pmap_qenter(kva, pages, npages);
+    AFS_VM_CNT_INC(v_vnodein);
+    AFS_VM_CNT_ADD(v_vnodepgsin, npages);
 
+#ifdef FBSD_VOP_GETPAGES_BUSIED
+    count = ctob(npages);
+#else
+    count = ap->a_count;
+#endif
     iov.iov_base = (caddr_t) kva;
-    iov.iov_len = ap->a_count;
+    iov.iov_len = count;
     uio.uio_iov = &iov;
     uio.uio_iovcnt = 1;
-    uio.uio_offset = IDX_TO_OFF(ap->a_m[0]->pindex);
-    uio.uio_resid = ap->a_count;
+    uio.uio_offset = IDX_TO_OFF(pages[0]->pindex);
+    uio.uio_resid = count;
     uio.uio_segflg = UIO_SYSSPACE;
     uio.uio_rw = UIO_READ;
-#ifdef AFS_FBSD50_ENV
     uio.uio_td = curthread;
-#else
-    uio.uio_procp = curproc;
-#endif
 
     AFS_GLOCK();
-    osi_FlushPages(avc, osi_curcred());        /* hold bozon lock, but not basic vnode lock */
-    code = afs_read(avc, &uio, osi_curcred(), 0, 0, 0);
+    osi_FlushPages(avc, osi_curcred());        /* hold GLOCK, but not basic vnode lock */
+    code = afs_read(avc, &uio, osi_curcred(), 0);
     AFS_GUNLOCK();
     pmap_qremove(kva, npages);
 
     relpbuf(bp, &afs_pbuf_freecnt);
 
-    if (code && (uio.uio_resid == ap->a_count)) {
-#ifdef AFS_FBSD50_ENV
-       VM_OBJECT_LOCK(object);
-       vm_page_lock_queues();
-#endif
+    if (code && (uio.uio_resid == count)) {
+#ifndef FBSD_VOP_GETPAGES_BUSIED
+       AFS_VM_OBJECT_WLOCK(object);
+       ma_vm_page_lock_queues();
        for (i = 0; i < npages; ++i) {
            if (i != ap->a_reqpage)
-               vm_page_free(ap->a_m[i]);
+               vm_page_free(pages[i]);
        }
-#ifdef AFS_FBSD50_ENV
-       vm_page_unlock_queues();
-       VM_OBJECT_UNLOCK(object);
+       ma_vm_page_unlock_queues();
+       AFS_VM_OBJECT_WUNLOCK(object);
 #endif
        return VM_PAGER_ERROR;
     }
 
-    size = ap->a_count - uio.uio_resid;
-#ifdef AFS_FBSD50_ENV
-    VM_OBJECT_LOCK(object);
-    vm_page_lock_queues();
-#endif
+    size = count - uio.uio_resid;
+    AFS_VM_OBJECT_WLOCK(object);
+    ma_vm_page_lock_queues();
     for (i = 0, toff = 0; i < npages; i++, toff = nextoff) {
        vm_page_t m;
        nextoff = toff + PAGE_SIZE;
-       m = ap->a_m[i];
+       m = pages[i];
 
+       /* XXX not in nfsclient? */
        m->flags &= ~PG_ZERO;
 
        if (nextoff <= size) {
@@ -721,18 +598,21 @@ afs_vop_getpages(struct vop_getpages_args *ap)
             * Read operation filled an entire page
             */
            m->valid = VM_PAGE_BITS_ALL;
-           vm_page_undirty(m);
+           KASSERT(m->dirty == 0, ("afs_getpages: page %p is dirty", m));
        } else if (size > toff) {
            /*
             * Read operation filled a partial page.
             */
            m->valid = 0;
            vm_page_set_validclean(m, 0, size - toff);
-           /* handled by vm_fault now        */
-           /* vm_page_zero_invalid(m, TRUE); */
+           KASSERT(m->dirty == 0, ("afs_getpages: page %p is dirty", m));
        }
 
+#ifndef FBSD_VOP_GETPAGES_BUSIED
        if (i != ap->a_reqpage) {
+#if __FreeBSD_version >= 1000042
+           vm_page_readahead_finish(m);
+#else
            /*
             * Whether or not to leave the page activated is up in
             * the air, but we should put the page on a page queue
@@ -746,24 +626,32 @@ afs_vop_getpages(struct vop_getpages_args *ap)
             * now tell them that it is ok to use.
             */
            if (!code) {
-               if (m->flags & PG_WANTED)
+               if (m->oflags & VPO_WANTED) {
+                   ma_vm_page_lock(m);
                    vm_page_activate(m);
-               else
+                   ma_vm_page_unlock(m);
+               }
+               else {
+                   ma_vm_page_lock(m);
                    vm_page_deactivate(m);
+                   ma_vm_page_unlock(m);
+               }
                vm_page_wakeup(m);
            } else {
+               ma_vm_page_lock(m);
                vm_page_free(m);
+               ma_vm_page_unlock(m);
            }
+#endif /* __FreeBSD_version 1000042 */
        }
+#endif   /* ndef FBSD_VOP_GETPAGES_BUSIED */
     }
-#ifdef AFS_FBSD50_ENV
-    vm_page_unlock_queues();
-    VM_OBJECT_UNLOCK(object);
-#endif
-    return 0;
+    ma_vm_page_unlock_queues();
+    AFS_VM_OBJECT_WUNLOCK(object);
+    return VM_PAGER_OK;
 }
 
-int
+static int
 afs_vop_write(ap)
      struct vop_write_args     /* {
                                 * struct vnode *a_vp;
@@ -775,7 +663,7 @@ afs_vop_write(ap)
     int code;
     struct vcache *avc = VTOAFS(ap->a_vp);
     AFS_GLOCK();
-    osi_FlushPages(avc, ap->a_cred);   /* hold bozon lock, but not basic vnode lock */
+    osi_FlushPages(avc, ap->a_cred);   /* hold GLOCK, but not basic vnode lock */
     code =
        afs_write(VTOAFS(ap->a_vp), ap->a_uio, ap->a_ioflag, ap->a_cred, 0);
     AFS_GUNLOCK();
@@ -796,7 +684,7 @@ afs_vop_write(ap)
  * All of the pages passed to us in ap->a_m[] are already marked as busy,
  * so there is no additional locking required to set their flags.  -GAW
  */
-int
+static int
 afs_vop_putpages(struct vop_putpages_args *ap)
 {
     int code;
@@ -808,9 +696,8 @@ afs_vop_putpages(struct vop_putpages_args *ap)
     struct vnode *vp;
     struct vcache *avc;
 
-#ifdef AFS_FBSD50_ENV
-    GIANT_REQUIRED;
-#endif
+    memset(&uio, 0, sizeof(uio));
+    memset(&iov, 0, sizeof(iov));
 
     vp = ap->a_vp;
     avc = VTOAFS(vp);
@@ -830,10 +717,8 @@ afs_vop_putpages(struct vop_putpages_args *ap)
 
     kva = (vm_offset_t) bp->b_data;
     pmap_qenter(kva, ap->a_m, npages);
-#ifdef AFS_FBSD50_ENV
-    cnt.v_vnodeout++;
-    cnt.v_vnodepgsout += ap->a_count;
-#endif
+    AFS_VM_CNT_INC(v_vnodeout);
+    AFS_VM_CNT_ADD(v_vnodepgsout, ap->a_count);
 
     iov.iov_base = (caddr_t) kva;
     iov.iov_len = ap->a_count;
@@ -843,11 +728,7 @@ afs_vop_putpages(struct vop_putpages_args *ap)
     uio.uio_resid = ap->a_count;
     uio.uio_segflg = UIO_SYSSPACE;
     uio.uio_rw = UIO_WRITE;
-#ifdef AFS_FBSD50_ENV
     uio.uio_td = curthread;
-#else
-    uio.uio_procp = curproc;
-#endif
     sync = IO_VMIO;
     if (ap->a_sync & VM_PAGER_PUT_SYNC)
        sync |= IO_SYNC;
@@ -871,15 +752,15 @@ afs_vop_putpages(struct vop_putpages_args *ap)
     return ap->a_rtvals[0];
 }
 
-int
+static int
 afs_vop_ioctl(ap)
      struct vop_ioctl_args     /* {
                                 * struct vnode *a_vp;
-                                * int  a_command;
-                                * caddr_t  a_data;
+                                * u_long a_command;
+                                * void *a_data;
                                 * int  a_fflag;
                                 * struct ucred *a_cred;
-                                * struct proc *a_p;
+                                * struct thread *a_td;
                                 * } */ *ap;
 {
     struct vcache *tvc = VTOAFS(ap->a_vp);
@@ -891,8 +772,7 @@ afs_vop_ioctl(ap)
     if (((ap->a_command >> 8) & 0xff) == 'V') {
        /* This is a VICEIOCTL call */
        AFS_GLOCK();
-       error = HandleIoctl(tvc, NULL /*Not used */ ,
-                           ap->a_command, ap->a_data);
+       error = HandleIoctl(tvc, ap->a_command, ap->a_data);
        AFS_GUNLOCK();
        return (error);
     } else {
@@ -901,67 +781,25 @@ afs_vop_ioctl(ap)
     }
 }
 
-/* ARGSUSED */
-int
-afs_vop_poll(ap)
-     struct vop_poll_args      /* {
-                                * struct vnode *a_vp;
-                                * int  a_events;
-                                * struct ucred *a_cred;
-                                * struct proc *a_p;
-                                * } */ *ap;
-{
-    /*
-     * We should really check to see if I/O is possible.
-     */
-    return (1);
-}
-
-/*
- * Mmap a file
- *
- * NB Currently unsupported.
- */
-/* ARGSUSED */
-int
-afs_vop_mmap(ap)
-     struct vop_mmap_args      /* {
-                                * struct vnode *a_vp;
-                                * int  a_fflags;
-                                * struct ucred *a_cred;
-                                * struct proc *a_p;
-                                * } */ *ap;
-{
-    return (EINVAL);
-}
-
-int
+static int
 afs_vop_fsync(ap)
      struct vop_fsync_args     /* {
                                 * struct vnode *a_vp;
-                                * struct ucred *a_cred;
                                 * int a_waitfor;
-                                * struct proc *a_p;
+                                * struct thread *td;
                                 * } */ *ap;
 {
     int error;
-    register struct vnode *vp = ap->a_vp;
+    struct vnode *vp = ap->a_vp;
 
     AFS_GLOCK();
     /*vflushbuf(vp, wait); */
-#ifdef AFS_FBSD60_ENV
     error = afs_fsync(VTOAFS(vp), ap->a_td->td_ucred);
-#else
-    if (ap->a_cred)
-       error = afs_fsync(VTOAFS(vp), ap->a_cred);
-    else
-       error = afs_fsync(VTOAFS(vp), afs_osi_credp);
-#endif
     AFS_GUNLOCK();
     return error;
 }
 
-int
+static int
 afs_vop_remove(ap)
      struct vop_remove_args    /* {
                                 * struct vnode *a_dvp;
@@ -970,8 +808,8 @@ afs_vop_remove(ap)
                                 * } */ *ap;
 {
     int error = 0;
-    register struct vnode *vp = ap->a_vp;
-    register struct vnode *dvp = ap->a_dvp;
+    struct vnode *vp = ap->a_vp;
+    struct vnode *dvp = ap->a_dvp;
 
     GETNAME();
     AFS_GLOCK();
@@ -982,7 +820,7 @@ afs_vop_remove(ap)
     return error;
 }
 
-int
+static int
 afs_vop_link(ap)
      struct vop_link_args      /* {
                                 * struct vnode *a_vp;
@@ -991,13 +829,8 @@ afs_vop_link(ap)
                                 * } */ *ap;
 {
     int error = 0;
-    register struct vnode *dvp = ap->a_tdvp;
-    register struct vnode *vp = ap->a_vp;
-#ifdef AFS_FBSD50_ENV
-    struct thread *p = ap->a_cnp->cn_thread;
-#else
-    struct proc *p = ap->a_cnp->cn_proc;
-#endif
+    struct vnode *dvp = ap->a_tdvp;
+    struct vnode *vp = ap->a_vp;
 
     GETNAME();
     if (dvp->v_mount != vp->v_mount) {
@@ -1008,20 +841,20 @@ afs_vop_link(ap)
        error = EISDIR;
        goto out;
     }
-    if ((error = vn_lock(vp, LK_EXCLUSIVE, p)) != 0) {
+    if ((error = vn_lock(vp, LK_CANRECURSE | LK_EXCLUSIVE)) != 0) {
        goto out;
     }
     AFS_GLOCK();
     error = afs_link(VTOAFS(vp), VTOAFS(dvp), name, cnp->cn_cred);
     AFS_GUNLOCK();
     if (dvp != vp)
-       VOP_UNLOCK(vp, 0, p);
+       VOP_UNLOCK(vp, 0);
   out:
     DROPNAME();
     return error;
 }
 
-int
+static int
 afs_vop_rename(ap)
      struct vop_rename_args    /* {
                                 * struct vnode *a_fdvp;
@@ -1038,14 +871,9 @@ afs_vop_rename(ap)
     struct componentname *tcnp = ap->a_tcnp;
     char *tname;
     struct vnode *tvp = ap->a_tvp;
-    register struct vnode *tdvp = ap->a_tdvp;
+    struct vnode *tdvp = ap->a_tdvp;
     struct vnode *fvp = ap->a_fvp;
-    register struct vnode *fdvp = ap->a_fdvp;
-#ifdef AFS_FBSD50_ENV
-    struct thread *p = fcnp->cn_thread;
-#else
-    struct proc *p = fcnp->cn_proc;
-#endif
+    struct vnode *fdvp = ap->a_fdvp;
 
     /*
      * Check for cross-device rename.
@@ -1104,13 +932,13 @@ afs_vop_rename(ap)
        vput(fvp);
        return (error);
     }
-    if ((error = vn_lock(fvp, LK_EXCLUSIVE, p)) != 0)
+    if ((error = vn_lock(fvp, LK_EXCLUSIVE)) != 0)
        goto abortit;
 
-    MALLOC(fname, char *, fcnp->cn_namelen + 1, M_TEMP, M_WAITOK);
+    fname = malloc(fcnp->cn_namelen + 1, M_TEMP, M_WAITOK);
     memcpy(fname, fcnp->cn_nameptr, fcnp->cn_namelen);
     fname[fcnp->cn_namelen] = '\0';
-    MALLOC(tname, char *, tcnp->cn_namelen + 1, M_TEMP, M_WAITOK);
+    tname = malloc(tcnp->cn_namelen + 1, M_TEMP, M_WAITOK);
     memcpy(tname, tcnp->cn_nameptr, tcnp->cn_namelen);
     tname[tcnp->cn_namelen] = '\0';
 
@@ -1121,8 +949,8 @@ afs_vop_rename(ap)
        afs_rename(VTOAFS(fdvp), fname, VTOAFS(tdvp), tname, tcnp->cn_cred);
     AFS_GUNLOCK();
 
-    FREE(fname, M_TEMP);
-    FREE(tname, M_TEMP);
+    free(fname, M_TEMP);
+    free(tname, M_TEMP);
     if (tdvp == tvp)
        vrele(tdvp);
     else
@@ -1134,7 +962,7 @@ afs_vop_rename(ap)
     return error;
 }
 
-int
+static int
 afs_vop_mkdir(ap)
      struct vop_mkdir_args     /* {
                                 * struct vnode *a_dvp;
@@ -1143,15 +971,10 @@ afs_vop_mkdir(ap)
                                 * struct vattr *a_vap;
                                 * } */ *ap;
 {
-    register struct vnode *dvp = ap->a_dvp;
-    register struct vattr *vap = ap->a_vap;
+    struct vnode *dvp = ap->a_dvp;
+    struct vattr *vap = ap->a_vap;
     int error = 0;
     struct vcache *vcp;
-#ifdef AFS_FBSD50_ENV
-    struct thread *p = ap->a_cnp->cn_thread;
-#else
-    struct proc *p = ap->a_cnp->cn_proc;
-#endif
 
     GETNAME();
 #ifdef DIAGNOSTIC
@@ -1167,14 +990,14 @@ afs_vop_mkdir(ap)
     }
     if (vcp) {
        *ap->a_vpp = AFSTOV(vcp);
-       vn_lock(AFSTOV(vcp), LK_EXCLUSIVE | LK_RETRY, p);
+       vn_lock(AFSTOV(vcp), LK_EXCLUSIVE | LK_RETRY);
     } else
        *ap->a_vpp = 0;
     DROPNAME();
     return error;
 }
 
-int
+static int
 afs_vop_rmdir(ap)
      struct vop_rmdir_args     /* {
                                 * struct vnode *a_dvp;
@@ -1183,7 +1006,7 @@ afs_vop_rmdir(ap)
                                 * } */ *ap;
 {
     int error = 0;
-    register struct vnode *dvp = ap->a_dvp;
+    struct vnode *dvp = ap->a_dvp;
 
     GETNAME();
     AFS_GLOCK();
@@ -1201,7 +1024,7 @@ afs_vop_rmdir(ap)
  *     char *a_target;
  * };
  */
-int
+static int
 afs_vop_symlink(struct vop_symlink_args *ap)
 {
     struct vnode *dvp;
@@ -1216,16 +1039,13 @@ afs_vop_symlink(struct vop_symlink_args *ap)
     newvp = NULL;
 
     error =
-       afs_symlink(VTOAFS(dvp), name, ap->a_vap, ap->a_target, cnp->cn_cred);
+       afs_symlink(VTOAFS(dvp), name, ap->a_vap, ap->a_target, NULL,
+                   cnp->cn_cred);
     if (error == 0) {
        error = afs_lookup(VTOAFS(dvp), name, &vcp, cnp->cn_cred);
        if (error == 0) {
            newvp = AFSTOV(vcp);
-#ifdef AFS_FBSD50_ENV
-           vn_lock(newvp, LK_EXCLUSIVE | LK_RETRY, cnp->cn_thread);
-#else
-           vn_lock(newvp, LK_EXCLUSIVE | LK_RETRY, cnp->cn_proc);
-#endif
+           vn_lock(newvp, LK_EXCLUSIVE | LK_RETRY);
        }
     }
     AFS_GUNLOCK();
@@ -1234,7 +1054,7 @@ afs_vop_symlink(struct vop_symlink_args *ap)
     return error;
 }
 
-int
+static int
 afs_vop_readdir(ap)
      struct vop_readdir_args   /* {
                                 * struct vnode *a_vp;
@@ -1270,7 +1090,7 @@ afs_vop_readdir(ap)
             dp = (const struct dirent *)((const char *)dp + dp->d_reclen))
            ncookies++;
 
-       MALLOC(cookies, u_long *, ncookies * sizeof(u_long), M_TEMP,
+       cookies = malloc(ncookies * sizeof(u_long), M_TEMP,
               M_WAITOK);
        for (dp = dp_start, cookiep = cookies; dp < dp_end;
             dp = (const struct dirent *)((const char *)dp + dp->d_reclen)) {
@@ -1284,7 +1104,7 @@ afs_vop_readdir(ap)
     return error;
 }
 
-int
+static int
 afs_vop_readlink(ap)
      struct vop_readlink_args  /* {
                                 * struct vnode *a_vp;
@@ -1300,24 +1120,18 @@ afs_vop_readlink(ap)
     return error;
 }
 
-extern int prtactive;
-
-int
+static int
 afs_vop_inactive(ap)
      struct vop_inactive_args  /* {
                                 * struct vnode *a_vp;
-                                * struct proc *a_p;
+                                * struct thread *td;
                                 * } */ *ap;
 {
-    register struct vnode *vp = ap->a_vp;
-
-    if (prtactive && vp->v_usecount != 0)
-       vprint("afs_vop_inactive(): pushing active", vp);
+    struct vnode *vp = ap->a_vp;
 
     AFS_GLOCK();
     afs_InactiveVCache(VTOAFS(vp), 0); /* decrs ref counts */
     AFS_GUNLOCK();
-    VOP_UNLOCK(vp, 0, ap->a_p);
     return 0;
 }
 
@@ -1326,72 +1140,58 @@ afs_vop_inactive(ap)
  *     struct vnode *a_vp;
  * };
  */
-int
+static int
 afs_vop_reclaim(struct vop_reclaim_args *ap)
 {
-    /* copied from ../OBSD/osi_vnodeops.c:afs_nbsd_reclaim() */
     int code, slept;
     struct vnode *vp = ap->a_vp;
     struct vcache *avc = VTOAFS(vp);
     int haveGlock = ISAFS_GLOCK();
-    int haveVlock = CheckLock(&afs_xvcache);
+
+    /*
+     * In other code paths, we acquire the vnode lock while afs_xvcache is
+     * already held (e.g. afs_PutVCache() -> vrele()). Here, we already have
+     * the vnode lock, and we need afs_xvcache. So drop the vnode lock in order
+     * to hold afs_xvcache.
+     */
+    VOP_UNLOCK(vp, 0);
 
     if (!haveGlock)
        AFS_GLOCK();
-    if (!haveVlock)
-       ObtainWriteLock(&afs_xvcache, 901);
-#ifndef AFS_DISCON_ENV
-    code = afs_FlushVCache(avc, &slept);       /* tosses our stuff from vnode */
-#else
-    /* reclaim the vnode and the in-memory vcache, but keep the on-disk vcache */
-    code = afs_FlushVS(avc);
-#endif
-    if (!haveVlock)
-       ReleaseWriteLock(&afs_xvcache);
-    if (!haveGlock)
-       AFS_GUNLOCK();
+    ObtainWriteLock(&afs_xvcache, 901);
 
     /*
-     * XXX Pretend it worked, to prevent panic on shutdown
-     * Garrett, please fix - Jim Rees
+     * Note that we deliberately call VOP_LOCK() instead of vn_lock() here.
+     * vn_lock() will return an error for VI_DOOMED vnodes, but we know this
+     * vnode is already VI_DOOMED. We just want to lock it again, and skip the
+     * VI_DOOMED check.
      */
-    if (code)
-       printf("afs_vop_reclaim: afs_FlushVCache failed code %d\n", code);
-#ifdef AFS_FBSD60_ENV
-    else
-       vnode_destroy_vobject(vp);
-#endif
-    return 0;
-}
+    VOP_LOCK(vp, LK_EXCLUSIVE);
 
-#ifndef AFS_FBSD60_ENV
-int
-afs_vop_bmap(ap)
-     struct vop_bmap_args      /* {
-                                * struct vnode *a_vp;
-                                * daddr_t  a_bn;
-                                * struct vnode **a_vpp;
-                                * daddr_t *a_bnp;
-                                * int *a_runp;
-                                * int *a_runb;
-                                * } */ *ap;
-{
-    if (ap->a_bnp) {
-       *ap->a_bnp = ap->a_bn * (PAGE_SIZE / DEV_BSIZE);
+    code = afs_FlushVCache(avc, &slept);
+
+    if (avc->f.states & CVInit) {
+       avc->f.states &= ~CVInit;
+       afs_osi_Wakeup(&avc->f.states);
     }
-    if (ap->a_vpp) {
-       *ap->a_vpp = ap->a_vp;
+
+    ReleaseWriteLock(&afs_xvcache);
+    if (!haveGlock)
+       AFS_GUNLOCK();
+
+    if (code) {
+       afs_warn("afs_vop_reclaim: afs_FlushVCache failed code %d vnode\n", code);
+       VOP_PRINT(vp);
+       panic("afs: afs_FlushVCache failed during reclaim");
     }
-    if (ap->a_runp != NULL)
-       *ap->a_runp = 0;
-    if (ap->a_runb != NULL)
-       *ap->a_runb = 0;
+
+    vnode_destroy_vobject(vp);
+    vp->v_data = 0;
 
     return 0;
 }
-#endif
 
-int
+static int
 afs_vop_strategy(ap)
      struct vop_strategy_args  /* {
                                 * struct buf *a_bp;
@@ -1399,32 +1199,25 @@ afs_vop_strategy(ap)
 {
     int error;
     AFS_GLOCK();
-    error = afs_ustrategy(ap->a_bp);
+    error = afs_ustrategy(ap->a_bp, osi_curcred());
     AFS_GUNLOCK();
     return error;
 }
 
-int
+static int
 afs_vop_print(ap)
      struct vop_print_args     /* {
                                 * struct vnode *a_vp;
                                 * } */ *ap;
 {
-    register struct vnode *vp = ap->a_vp;
-    register struct vcache *vc = VTOAFS(ap->a_vp);
-    int s = vc->states;
-
-#ifdef AFS_FBSD50_ENV
-    printf("tag %s, fid: %d.%x.%x.%x, opens %d, writers %d", vp->v_tag,
-          (int)vc->fid.Cell, (u_int) vc->fid.Fid.Volume,
-          (u_int) vc->fid.Fid.Vnode, (u_int) vc->fid.Fid.Unique, vc->opens,
-          vc->execsOrWriters);
-#else
-    printf("tag %d, fid: %ld.%x.%x.%x, opens %d, writers %d", vp->v_tag,
-          vc->fid.Cell, (u_int) vc->fid.Fid.Volume,
-          (u_int) vc->fid.Fid.Vnode, (u_int) vc->fid.Fid.Unique, vc->opens,
+    struct vnode *vp = ap->a_vp;
+    struct vcache *vc = VTOAFS(ap->a_vp);
+    int s = vc->f.states;
+
+    printf("vc %p vp %p tag %s, fid: %d.%d.%d.%d, opens %d, writers %d", vc, vp, vp->v_tag,
+          (int)vc->f.fid.Cell, (u_int) vc->f.fid.Fid.Volume,
+          (u_int) vc->f.fid.Fid.Vnode, (u_int) vc->f.fid.Fid.Unique, vc->opens,
           vc->execsOrWriters);
-#endif
     printf("\n  states%s%s%s%s%s", (s & CStatd) ? " statd" : "",
           (s & CRO) ? " readonly" : "", (s & CDirty) ? " dirty" : "",
           (s & CMAPPED) ? " mapped" : "",
@@ -1436,7 +1229,7 @@ afs_vop_print(ap)
 /*
  * Advisory record locking support (fcntl() POSIX style)
  */
-int
+static int
 afs_vop_advlock(ap)
      struct vop_advlock_args   /* {
                                 * struct vnode *a_vp;
@@ -1446,12 +1239,59 @@ afs_vop_advlock(ap)
                                 * int  a_flags;
                                 * } */ *ap;
 {
-    int error;
+    int error, a_op;
     struct ucred cr = *osi_curcred();
 
+    a_op = ap->a_op;
+    if (a_op == F_UNLCK) {
+       /*
+        * When a_fl->type is F_UNLCK, FreeBSD passes in an a_op of F_UNLCK.
+        * This is (confusingly) different than how you actually release a lock
+        * with fcntl(), which is done with an a_op of F_SETLK and an l_type of
+        * F_UNLCK. Pretend we were given an a_op of F_SETLK in this case,
+        * since this is what afs_lockctl expects.
+        */
+       a_op = F_SETLK;
+    }
+
     AFS_GLOCK();
     error =
-       afs_lockctl(VTOAFS(ap->a_vp), ap->a_fl, ap->a_op, &cr, (int)ap->a_id);
+       afs_lockctl(VTOAFS(ap->a_vp),
+               ap->a_fl,
+               a_op, &cr,
+               (int)(intptr_t)ap->a_id);       /* XXX: no longer unique! */
     AFS_GUNLOCK();
     return error;
 }
+
+struct vop_vector afs_vnodeops = {
+       .vop_default =          &default_vnodeops,
+       .vop_access =           afs_vop_access,
+       .vop_advlock =          afs_vop_advlock,
+       .vop_close =            afs_vop_close,
+       .vop_create =           afs_vop_create,
+       .vop_fsync =            afs_vop_fsync,
+       .vop_getattr =          afs_vop_getattr,
+       .vop_getpages =         afs_vop_getpages,
+       .vop_inactive =         afs_vop_inactive,
+       .vop_ioctl =            afs_vop_ioctl,
+       .vop_link =             afs_vop_link,
+       .vop_lookup =           afs_vop_lookup,
+       .vop_mkdir =            afs_vop_mkdir,
+       .vop_mknod =            afs_vop_mknod,
+       .vop_open =             afs_vop_open,
+       .vop_pathconf =         afs_vop_pathconf,
+       .vop_print =            afs_vop_print,
+       .vop_putpages =         afs_vop_putpages,
+       .vop_read =             afs_vop_read,
+       .vop_readdir =          afs_vop_readdir,
+       .vop_readlink =         afs_vop_readlink,
+       .vop_reclaim =          afs_vop_reclaim,
+       .vop_remove =           afs_vop_remove,
+       .vop_rename =           afs_vop_rename,
+       .vop_rmdir =            afs_vop_rmdir,
+       .vop_setattr =          afs_vop_setattr,
+       .vop_strategy =         afs_vop_strategy,
+       .vop_symlink =          afs_vop_symlink,
+       .vop_write =            afs_vop_write,
+};