MacOS: allow cdead vcaches to be found in FindVCache if requested
[openafs.git] / src / afs / afs_vcache.c
index a3285ff..dc6a799 100644 (file)
 #include <afsconfig.h>
 #include "afs/param.h"
 
-
-#include "afs/sysincludes.h"   /*Standard vendor system headers */
-#include "afsincludes.h"       /*AFS-based standard headers */
+#include "afs/sysincludes.h"   /*Standard vendor system headers */
+#include "afsincludes.h"       /*AFS-based standard headers */
 #include "afs/afs_stats.h"
 #include "afs/afs_cbqueue.h"
 #include "afs/afs_osidnlc.h"
 
-#if defined(AFS_OSF_ENV) || defined(AFS_LINUX22_ENV)
 afs_int32 afs_maxvcount = 0;   /* max number of vcache entries */
 afs_int32 afs_vcount = 0;      /* number of vcache in use now */
-#endif /* AFS_OSF_ENV */
 
 #ifdef AFS_SGI_ENV
 int afsvnumbers = 0;
@@ -61,9 +58,7 @@ char *makesname();
 #endif /* AFS_SGI64_ENV */
 
 /* Exported variables */
-#ifdef AFS_DISCON_ENV
 afs_rwlock_t afs_xvcdirty;     /*Lock: discon vcache dirty list mgmt */
-#endif
 afs_rwlock_t afs_xvcache;      /*Lock: alloc new stat cache entries */
 afs_rwlock_t afs_xvreclaim;    /*Lock: entries reclaimed, not on free list */
 afs_lock_t afs_xvcb;           /*Lock: fids on which there are callbacks */
@@ -80,22 +75,19 @@ struct afs_q afs_vhashTV[VCSIZE];
 static struct afs_cbr *afs_cbrHashT[CBRSIZE];
 afs_int32 afs_bulkStatsLost;
 int afs_norefpanic = 0;
-extern int afsd_dynamic_vcaches;
 
 
-/* Disk backed vcache definitions 
+/* Disk backed vcache definitions
  * Both protected by xvcache */
-#ifdef AFS_DISCON_ENV
 static int afs_nextVcacheSlot = 0;
 static struct afs_slotlist *afs_freeSlotList = NULL;
-#endif
 
 /* Forward declarations */
 static afs_int32 afs_QueueVCB(struct vcache *avc);
 
 /*!
  * Generate an index into the hash table for a given Fid.
- * \param fid 
+ * \param fid
  * \return The hash value.
  */
 static int
@@ -150,11 +142,6 @@ afs_FlushVCache(struct vcache *avc, int *slept)
     AFS_STATCNT(afs_FlushVCache);
     afs_Trace2(afs_iclSetp, CM_TRACE_FLUSHV, ICL_TYPE_POINTER, avc,
               ICL_TYPE_INT32, avc->f.states);
-#ifdef  AFS_OSF_ENV
-    AFS_GUNLOCK();
-    VN_LOCK(AFSTOV(avc));
-    AFS_GLOCK();
-#endif
 
     code = osi_VM_FlushVCache(avc, slept);
     if (code)
@@ -218,14 +205,8 @@ afs_FlushVCache(struct vcache *avc, int *slept)
     vn_reinit(AFSTOV(avc));
 #endif
     afs_FreeAllAxs(&(avc->Access));
-
-    /* we can't really give back callbacks on RO files, since the
-     * server only tracks them on a per-volume basis, and we don't
-     * know whether we still have some other files from the same
-     * volume. */
-    if ((avc->f.states & CRO) == 0 && avc->callback) {
+    if (!afs_shuttingdown)
        afs_QueueVCB(avc);
-    }
     ObtainWriteLock(&afs_xcbhash, 460);
     afs_DequeueCallback(avc);  /* remove it from queued callbacks list */
     avc->f.states &= ~(CStatd | CUnique);
@@ -245,7 +226,8 @@ afs_FlushVCache(struct vcache *avc, int *slept)
     else
        afs_evenZaps++;
 
-#if !defined(AFS_OSF_ENV) && !defined(AFS_LINUX22_ENV)
+    afs_vcount--;
+#if !defined(AFS_LINUX22_ENV)
     /* put the entry in the free list */
     avc->nextfree = freeVCList;
     freeVCList = avc;
@@ -255,46 +237,33 @@ afs_FlushVCache(struct vcache *avc, int *slept)
     avc->f.states |= CVFlushed;
 #else
     /* This should put it back on the vnode free list since usecount is 1 */
-    afs_vcount--;
     vSetType(avc, VREG);
     if (VREFCOUNT_GT(avc,0)) {
-#if defined(AFS_OSF_ENV)
-       VN_UNLOCK(AFSTOV(avc));
-#endif
        AFS_RELE(AFSTOV(avc));
        afs_stats_cmperf.vcacheXAllocs--;
     } else {
        if (afs_norefpanic) {
-           printf("flush vc refcnt < 1");
+           afs_warn("flush vc refcnt < 1");
            afs_norefpanic++;
-#if defined(AFS_OSF_ENV)
-           (void)vgone(avc, VX_NOSLEEP, NULL);
-           AFS_GLOCK();
-           VN_UNLOCK(AFSTOV(avc));
-#endif
        } else
            osi_Panic("flush vc refcnt < 1");
     }
-#endif /* AFS_OSF_ENV */
+#endif /* AFS_LINUX22_ENV */
     return 0;
 
   bad:
-#ifdef AFS_OSF_ENV
-    VN_UNLOCK(AFSTOV(avc));
-#endif
     return code;
-
 }                              /*afs_FlushVCache */
 
 #ifndef AFS_SGI_ENV
 /*!
  *  The core of the inactive vnode op for all but IRIX.
  *
- * \param avc 
+ * \param avc
  * \param acred
  */
 void
-afs_InactiveVCache(struct vcache *avc, struct AFS_UCRED *acred)
+afs_InactiveVCache(struct vcache *avc, afs_ucred_t *acred)
 {
     AFS_STATCNT(afs_inactive);
     if (avc->f.states & CDirty) {
@@ -325,34 +294,40 @@ afs_InactiveVCache(struct vcache *avc, struct AFS_UCRED *acred)
  */
 static struct afs_cbr *afs_cbrSpace = 0;
 /* if alloc limit below changes, fix me! */
-static struct afs_cbr *afs_cbrHeads[2];
+static struct afs_cbr *afs_cbrHeads[16];
 struct afs_cbr *
 afs_AllocCBR(void)
 {
-    register struct afs_cbr *tsp;
+    struct afs_cbr *tsp;
     int i;
 
-    while (!afs_cbrSpace) {
-       if (afs_stats_cmperf.CallBackAlloced >= 2) {
-           /* don't allocate more than 2 * AFS_NCBRS for now */
-           afs_FlushVCBs(0);
+    if (!afs_cbrSpace) {
+       afs_osi_CancelWait(&AFS_WaitHandler);   /* trigger FlushVCBs asap */
+
+       if (afs_stats_cmperf.CallBackAlloced >= sizeof(afs_cbrHeads)/sizeof(afs_cbrHeads[0])) {
+           /* don't allocate more than 16 * AFS_NCBRS for now */
+           tsp = (struct afs_cbr *)osi_AllocSmallSpace(sizeof(*tsp));
+           tsp->dynalloc = 1;
+           tsp->next = NULL;
            afs_stats_cmperf.CallBackFlushes++;
        } else {
            /* try allocating */
-           tsp =
-               (struct afs_cbr *)afs_osi_Alloc(AFS_NCBRS *
-                                               sizeof(struct afs_cbr));
+           tsp = afs_osi_Alloc(AFS_NCBRS * sizeof(struct afs_cbr));
+           osi_Assert(tsp != NULL);
            for (i = 0; i < AFS_NCBRS - 1; i++) {
                tsp[i].next = &tsp[i + 1];
+               tsp[i].dynalloc = 0;
            }
            tsp[AFS_NCBRS - 1].next = 0;
-           afs_cbrSpace = tsp;
+           tsp[AFS_NCBRS - 1].dynalloc = 0;
+           afs_cbrSpace = tsp->next;
            afs_cbrHeads[afs_stats_cmperf.CallBackAlloced] = tsp;
            afs_stats_cmperf.CallBackAlloced++;
        }
+    } else {
+       tsp = afs_cbrSpace;
+       afs_cbrSpace = tsp->next;
     }
-    tsp = afs_cbrSpace;
-    afs_cbrSpace = tsp->next;
     return tsp;
 }
 
@@ -366,7 +341,7 @@ afs_AllocCBR(void)
  * \rerurn 0
  */
 int
-afs_FreeCBR(register struct afs_cbr *asp)
+afs_FreeCBR(struct afs_cbr *asp)
 {
     *(asp->pprev) = asp->next;
     if (asp->next)
@@ -376,11 +351,48 @@ afs_FreeCBR(register struct afs_cbr *asp)
     if (asp->hash_next)
        asp->hash_next->hash_pprev = asp->hash_pprev;
 
-    asp->next = afs_cbrSpace;
-    afs_cbrSpace = asp;
+    if (asp->dynalloc) {
+       osi_FreeSmallSpace(asp);
+    } else {
+       asp->next = afs_cbrSpace;
+       afs_cbrSpace = asp;
+    }
     return 0;
 }
 
+static void
+FlushAllVCBs(struct rx_connection **rxconns, int nconns, int nservers,
+            struct afs_conn **conns, struct srvAddr **addrs)
+{
+    afs_int32 *results;
+    afs_int32 i;
+
+    results = afs_osi_Alloc(nservers * sizeof (afs_int32));
+    osi_Assert(results != NULL);
+
+    AFS_GUNLOCK();
+    multi_Rx(rxconns,nconns)
+    {
+        multi_RXAFS_GiveUpAllCallBacks();
+        results[multi_i] = multi_error;
+    } multi_End;
+    AFS_GLOCK();
+
+    /*
+     * Freeing the CBR will unlink it from the server's CBR list
+     * do it here, not in the loop, because a dynamic CBR will call
+     * into the memory management routines.
+     */
+    for ( i = 0 ; i < nconns ; i++ ) {
+       if (results[i] == 0) {
+           /* Unchain all of them */
+           while (addrs[i]->server->cbrs)
+               afs_FreeCBR(addrs[i]->server->cbrs);
+       }
+    }
+    afs_osi_Free(results, nservers * sizeof(afs_int32));
+}
+
 /*!
  *   Flush all queued callbacks to all servers.
  *
@@ -405,13 +417,27 @@ afs_FlushVCBs(afs_int32 lockit)
     struct afs_conn *tc;
     int safety1, safety2, safety3;
     XSTATS_DECLS;
+
+    if (AFS_IS_DISCONNECTED)
+       return ENETDOWN;
+
     if ((code = afs_InitReq(&treq, afs_osi_credp)))
        return code;
     treq.flags |= O_NONBLOCK;
     tfids = afs_osi_Alloc(sizeof(struct AFSFid) * AFS_MAXCBRSCALL);
+    osi_Assert(tfids != NULL);
 
     if (lockit)
-       MObtainWriteLock(&afs_xvcb, 273);
+       ObtainWriteLock(&afs_xvcb, 273);
+    /*
+     * Shutting down.
+     * First, attempt a multi across everything, all addresses
+     * for all servers we know of.
+     */
+
+    if (lockit == 2)
+       afs_LoopServers(2, NULL, 0, FlushAllVCBs, NULL);
+
     ObtainReadLock(&afs_xserver);
     for (i = 0; i < NSERVERS; i++) {
        for (safety1 = 0, tsp = afs_servers[i];
@@ -437,7 +463,7 @@ afs_FlushVCBs(afs_int32 lockit)
                    cbArray.AFSCBs_val = callBacks;
                    memset(&callBacks[0], 0, sizeof(callBacks[0]));
                    callBacks[0].CallBackType = CB_EXCLUSIVE;
-                   for (safety3 = 0; safety3 < MAXHOSTS * 2; safety3++) {
+                   for (safety3 = 0; safety3 < AFS_MAXHOSTS * 2; safety3++) {
                        tc = afs_ConnByHost(tsp, tsp->cell->fsport,
                                            tsp->cell->cellNum, &treq, 0,
                                            SHARED_LOCK);
@@ -496,7 +522,7 @@ afs_FlushVCBs(afs_int32 lockit)
 
     ReleaseReadLock(&afs_xserver);
     if (lockit)
-       MReleaseWriteLock(&afs_xvcb);
+       ReleaseWriteLock(&afs_xvcb);
     afs_osi_Free(tfids, sizeof(struct AFSFid) * AFS_MAXCBRSCALL);
     return 0;
 }
@@ -509,23 +535,34 @@ afs_FlushVCBs(afs_int32 lockit)
  *     Called when the xvcache lock is already held.
  *
  * \param avc vcache entry
- * \return 0 for success < 0 otherwise.
+ * \return 1 if queued, 0 otherwise
  */
 
 static afs_int32
 afs_QueueVCB(struct vcache *avc)
 {
+    int queued = 0;
     struct server *tsp;
     struct afs_cbr *tcbp;
 
     AFS_STATCNT(afs_QueueVCB);
+
+    ObtainWriteLock(&afs_xvcb, 274);
+
+    /* we can't really give back callbacks on RO files, since the
+     * server only tracks them on a per-volume basis, and we don't
+     * know whether we still have some other files from the same
+     * volume. */
+    if (!((avc->f.states & CRO) == 0 && avc->callback)) {
+        goto done;
+    }
+
     /* The callback is really just a struct server ptr. */
     tsp = (struct server *)(avc->callback);
 
     /* we now have a pointer to the server, so we just allocate
      * a queue entry and queue it.
      */
-    MObtainWriteLock(&afs_xvcb, 274);
     tcbp = afs_AllocCBR();
     tcbp->fid = avc->f.fid.Fid;
 
@@ -537,10 +574,12 @@ afs_QueueVCB(struct vcache *avc)
     tcbp->pprev = &tsp->cbrs;
 
     afs_InsertHashCBR(tcbp);
+    queued = 1;
 
+ done:
     /* now release locks and return */
-    MReleaseWriteLock(&afs_xvcb);
-    return 0;
+    ReleaseWriteLock(&afs_xvcb);
+    return queued;
 }
 
 
@@ -563,7 +602,7 @@ afs_RemoveVCB(struct VenusFid *afid)
     struct afs_cbr *cbr, *ncbr;
 
     AFS_STATCNT(afs_RemoveVCB);
-    MObtainWriteLock(&afs_xvcb, 275);
+    ObtainWriteLock(&afs_xvcb, 275);
 
     slot = afs_HashCBRFid(&afid->Fid);
     ncbr = afs_cbrHashT[slot];
@@ -579,16 +618,16 @@ afs_RemoveVCB(struct VenusFid *afid)
        }
     }
 
-    MReleaseWriteLock(&afs_xvcb);
+    ReleaseWriteLock(&afs_xvcb);
 }
 
-void 
+void
 afs_FlushReclaimedVcaches(void)
 {
 #if !defined(AFS_LINUX22_ENV)
     struct vcache *tvc;
     int code, fv_slept;
-    struct vcache *tmpReclaimedVCList = NULL;  
+    struct vcache *tmpReclaimedVCList = NULL;
 
     ObtainWriteLock(&afs_xvreclaim, 76);
     while (ReclaimedVCList) {
@@ -604,7 +643,7 @@ afs_FlushReclaimedVcaches(void)
               We probably need a way to be smarter about this. */
            tvc->nextfree = tmpReclaimedVCList;
            tmpReclaimedVCList = tvc;
-           printf("Reclaim list flush %lx failed: %d\n", (unsigned long) tvc, code);
+           /* printf("Reclaim list flush %lx failed: %d\n", (unsigned long) tvc, code); */
        }
         if (tvc->f.states & (CVInit
 #ifdef AFS_DARWIN80_ENV
@@ -619,209 +658,118 @@ afs_FlushReclaimedVcaches(void)
           afs_osi_Wakeup(&tvc->f.states);
        }
     }
-    if (tmpReclaimedVCList) 
+    if (tmpReclaimedVCList)
        ReclaimedVCList = tmpReclaimedVCList;
 
     ReleaseWriteLock(&afs_xvreclaim);
 #endif
 }
 
+void
+afs_PostPopulateVCache(struct vcache *avc, struct VenusFid *afid, int seq)
+{
+    /*
+     * The proper value for mvstat (for root fids) is setup by the caller.
+     */
+    avc->mvstat = 0;
+    if (afid->Fid.Vnode == 1 && afid->Fid.Unique == 1)
+       avc->mvstat = 2;
+
+    if (afs_globalVFS == 0)
+       osi_Panic("afs globalvfs");
+
+    osi_PostPopulateVCache(avc);
+
+    avc->dchint = NULL;
+    osi_dnlc_purgedp(avc);     /* this may be overkill */
+    memset(&(avc->callsort), 0, sizeof(struct afs_q));
+    avc->slocks = NULL;
+    avc->f.states &=~ CVInit;
+    if (seq) {
+       avc->f.states |= CBulkFetching;
+       avc->f.m.Length = seq;
+    }
+    afs_osi_Wakeup(&avc->f.states);
+}
+
 int
 afs_ShakeLooseVCaches(afs_int32 anumber)
 {
-#if defined(AFS_OSF_ENV) || defined(AFS_LINUX22_ENV)
-    afs_int32 i;
+    afs_int32 i, loop;
     struct vcache *tvc;
     struct afs_q *tq, *uq;
-    int code, fv_slept;
+    int fv_slept, defersleep = 0;
     afs_int32 target = anumber;
-    int haveGlock = 1;
 
-    /* Should probably deal better */
-    if (!ISAFS_GLOCK()) {
-       haveGlock = 0;
-       AFS_GLOCK();
-    }
-
-    if (
-#ifdef AFS_MAXVCOUNT_ENV
-       afsd_dynamic_vcaches || /* Always run if dynamic vcaches are enabled. */
-#endif
-       afs_vcount >= afs_maxvcount
-       ) {
-       i = 0;
-       for (tq = VLRU.prev; tq != &VLRU && anumber > 0; tq = uq) {
-           tvc = QTOV(tq);
-           uq = QPrev(tq);
-           if (tvc->f.states & CVFlushed) {
-               refpanic("CVFlushed on VLRU");
-           } else if (
-#ifdef AFS_MAXVCOUNT_ENV
-           ! afsd_dynamic_vcaches && 
-#endif
-           i++ > afs_maxvcount) {
-               refpanic("Exceeded pool of AFS vnodes(VLRU cycle?)");
-           } else if (QNext(uq) != tq) {
-               refpanic("VLRU inconsistent");
-           } else if (!VREFCOUNT_GT(tvc,0)) {
-               refpanic("refcnt 0 on VLRU");
-           }
-
-#if defined(AFS_LINUX22_ENV)
-           if (tvc != afs_globalVp && VREFCOUNT(tvc) > 1 && tvc->opens == 0) {
-                struct dentry *dentry;
-                struct list_head *cur, *head;
-                AFS_GUNLOCK();
-#if defined(AFS_LINUX24_ENV)
-                spin_lock(&dcache_lock);
-#endif /* AFS_LINUX24_ENV */
-               head = &(AFSTOV(tvc))->i_dentry;
-
-restart:
-                cur = head;
-                while ((cur = cur->next) != head) {
-                    dentry = list_entry(cur, struct dentry, d_alias);
-
-                   if (d_unhashed(dentry))
-                       continue;
+    i = 0;
+    loop = 0;
+    for (tq = VLRU.prev; tq != &VLRU && anumber > 0; tq = uq) {
+       tvc = QTOV(tq);
+       uq = QPrev(tq);
+       if (tvc->f.states & CVFlushed) {
+           refpanic("CVFlushed on VLRU");
+           /* In the other path, this was 2 * afs_cacheStats */
+       } else if (!afsd_dynamic_vcaches && i++ > afs_maxvcount) {
+           refpanic("Exceeded pool of AFS vnodes(VLRU cycle?)");
+       } else if (QNext(uq) != tq) {
+           refpanic("VLRU inconsistent");
+       } else if (tvc->f.states & CVInit) {
+           continue;
+       }
 
-                   dget_locked(dentry);
+       fv_slept = 0;
+       if (osi_TryEvictVCache(tvc, &fv_slept, defersleep))
+           anumber--;
 
-#if defined(AFS_LINUX24_ENV)
-                   spin_unlock(&dcache_lock);
-#endif /* AFS_LINUX24_ENV */
-                   if (d_invalidate(dentry) == -EBUSY) {
-                       dput(dentry);
-                       /* perhaps lock and try to continue? (use cur as head?) */
-                       goto inuse;
-                   }
-                   dput(dentry);
-#if defined(AFS_LINUX24_ENV)
-                   spin_lock(&dcache_lock);
-#endif /* AFS_LINUX24_ENV */
-                   goto restart;
-               }                   
-#if defined(AFS_LINUX24_ENV)
-               spin_unlock(&dcache_lock);
-#endif /* AFS_LINUX24_ENV */
-           inuse:
-               AFS_GLOCK();
-           }
-#endif /* AFS_LINUX22_ENV */
-
-           if (VREFCOUNT_GT(tvc,0) && !VREFCOUNT_GT(tvc,1) &&
-               tvc->opens == 0
-               && (tvc->f.states & CUnlinkedDel) == 0) {
-               code = afs_FlushVCache(tvc, &fv_slept);
-               if (code == 0) {
-                   anumber--;
-               }
-               if (fv_slept) {
-                   uq = VLRU.prev;
-                   i = 0;
-                   continue;   /* start over - may have raced. */
-               }
-           }
-           if (tq == uq)
+       if (fv_slept) {
+           if (loop++ > 100)
                break;
+           uq = VLRU.prev;
+           i = 0;
+           continue;   /* start over - may have raced. */
        }
-       if (
-#ifdef AFS_MAXVCOUNT_ENV
-        !afsd_dynamic_vcaches &&
-#endif
-        anumber == target) {
-           printf("afs_ShakeLooseVCaches: warning none freed, using %d of %d\n",
-                  afs_vcount, afs_maxvcount);
+       if (tq == uq) {
+           if (anumber && !defersleep) {
+               defersleep = 1;
+               tq = VLRU.prev;
+               continue;
+           }
+           break;
        }
-    } /* finished freeing up space */
-/*
-    printf("recycled %d entries\n", target-anumber);
-*/
-    if (!haveGlock)
-       AFS_GUNLOCK();
-#endif
+    }
+    if (!afsd_dynamic_vcaches && anumber == target) {
+       afs_warn("afs_ShakeLooseVCaches: warning none freed, using %d of %d\n",
+              afs_vcount, afs_maxvcount);
+    }
+
     return 0;
 }
 
 /* Alloc new vnode. */
 
 static struct vcache *
-afs_AllocVCache(void) 
+afs_AllocVCache(void)
 {
     struct vcache *tvc;
-#if defined(AFS_OSF30_ENV)
-    struct vcache *nvc;
-    AFS_GUNLOCK();
-    if (getnewvnode(MOUNT_AFS, &Afs_vnodeops, &nvc)) {
-       /* What should we do ???? */
-       osi_Panic("afs_AllocVCache: no more vnodes");
-    }
-    AFS_GLOCK();
 
-    tvc = nvc;
-    tvc->nextfree = NULL;
-    afs_vcount++;
-#elif defined(AFS_LINUX22_ENV)
-    struct inode *ip;
-
-    AFS_GUNLOCK();
-    ip = new_inode(afs_globalVFS);
-    if (!ip)
-       osi_Panic("afs_AllocVCache: no more inodes");
-    AFS_GLOCK();
-#if defined(STRUCT_SUPER_HAS_ALLOC_INODE)
-    tvc = VTOAFS(ip);
-#else
-    tvc = afs_osi_Alloc(sizeof(struct vcache));
-    ip->u.generic_ip = tvc;
-    tvc->v = ip;
-#endif
+    tvc = osi_NewVnode();
 
     afs_vcount++;
-#ifdef AFS_MAXVCOUNT_ENV
+
     /* track the peak */
     if (afsd_dynamic_vcaches && afs_maxvcount < afs_vcount) {
        afs_maxvcount = afs_vcount;
        /*printf("peak vnodes: %d\n", afs_maxvcount);*/
     }
-#endif
-    afs_stats_cmperf.vcacheXAllocs++;  /* count in case we have a leak */
-#else
-    /* none free, making one is better than a panic */
+
     afs_stats_cmperf.vcacheXAllocs++;  /* count in case we have a leak */
-    tvc = (struct vcache *)afs_osi_Alloc(sizeof(struct vcache));
-#if defined(AFS_DARWIN_ENV) && !defined(UKERNEL)
-    tvc->v = NULL; /* important to clean this, or use memset 0 */
-#endif
-#ifdef KERNEL_HAVE_PIN
-    pin((char *)tvc, sizeof(struct vcache));   /* XXX */
-#endif
-#if defined(AFS_SGI_ENV)
-    {
-       char name[METER_NAMSZ];
-       memset(tvc, 0, sizeof(struct vcache));
-       tvc->v.v_number = ++afsvnumbers;
-       tvc->vc_rwlockid = OSI_NO_LOCKID;
-       initnsema(&tvc->vc_rwlock, 1,
-                 makesname(name, "vrw", tvc->v.v_number));
-#ifndef        AFS_SGI53_ENV
-       initnsema(&tvc->v.v_sync, 0,
-                 makesname(name, "vsy", tvc->v.v_number));
-#endif
-#ifndef AFS_SGI62_ENV
-       initnlock(&tvc->v.v_lock,
-                 makesname(name, "vlk", tvc->v.v_number));
-#endif
-    }
-#endif /* AFS_SGI_ENV */
-#endif
-#ifdef AFS_DISCON_ENV
+
     /* If we create a new inode, we either give it a new slot number,
      * or if one's available, use a slot number from the slot free list
      */
     if (afs_freeSlotList != NULL) {
        struct afs_slotlist *tmp;
-   
+
        tvc->diskSlot = afs_freeSlotList->slot;
        tmp = afs_freeSlotList;
        afs_freeSlotList = tmp->next;
@@ -829,11 +777,53 @@ afs_AllocVCache(void)
     }  else {
        tvc->diskSlot = afs_nextVcacheSlot++;
     }
-#endif
 
     return tvc;
 }
 
+/* Pre populate a newly allocated vcache. On platforms where the actual
+ * vnode is attached to the vcache, this function is called before attachment,
+ * therefore it cannot perform any actions on the vnode itself */
+
+static void
+afs_PrePopulateVCache(struct vcache *avc, struct VenusFid *afid,
+                     struct server *serverp) {
+
+    afs_uint32 slot;
+    slot = avc->diskSlot;
+
+    osi_PrePopulateVCache(avc);
+
+    avc->diskSlot = slot;
+    QZero(&avc->metadirty);
+
+    AFS_RWLOCK_INIT(&avc->lock, "vcache lock");
+
+    avc->mvid = NULL;
+    avc->linkData = NULL;
+    avc->cbExpires = 0;
+    avc->opens = 0;
+    avc->execsOrWriters = 0;
+    avc->flockCount = 0;
+    avc->f.states = CVInit;
+    avc->last_looker = 0;
+    avc->f.fid = *afid;
+    avc->asynchrony = -1;
+    avc->vc_error = 0;
+
+    hzero(avc->mapDV);
+    avc->f.truncPos = AFS_NOTRUNC;   /* don't truncate until we need to */
+    hzero(avc->f.m.DataVersion);     /* in case we copy it into flushDV */
+    avc->Access = NULL;
+    avc->callback = serverp;         /* to minimize chance that clear
+                                     * request is lost */
+
+#if defined(AFS_CACHE_BYPASS)
+    avc->cachingStates = 0;
+    avc->cachingTransitions = 0;
+#endif
+}
+
 /*!
  *   This routine is responsible for allocating a new cache entry
  * from the free list.  It formats the cache entry and inserts it
@@ -847,123 +837,32 @@ afs_AllocVCache(void)
  *
  * \return The new vcache struct.
  */
-struct vcache *
-afs_NewVCache(struct VenusFid *afid, struct server *serverp)
+
+static_inline struct vcache *
+afs_NewVCache_int(struct VenusFid *afid, struct server *serverp, int seq)
 {
     struct vcache *tvc;
     afs_int32 i, j;
     afs_int32 anumber = VCACHE_FREE;
-#ifdef AFS_AIX_ENV
-    struct gnode *gnodepnt;
-#endif
-#if !defined(AFS_OSF_ENV) && !defined(AFS_LINUX22_ENV)
-    struct afs_q *tq, *uq;
-    int code, fv_slept;
-#endif
 
     AFS_STATCNT(afs_NewVCache);
 
     afs_FlushReclaimedVcaches();
 
-#if defined(AFS_OSF_ENV) || defined(AFS_LINUX22_ENV)
-#ifdef AFS_MAXVCOUNT_ENV
-    if(!afsd_dynamic_vcaches) {
-#endif
+#if defined(AFS_LINUX22_ENV)
+    if(!afsd_dynamic_vcaches && afs_vcount >= afs_maxvcount) {
        afs_ShakeLooseVCaches(anumber);
        if (afs_vcount >= afs_maxvcount) {
-           printf("afs_NewVCache - none freed\n");
+           afs_warn("afs_NewVCache - none freed\n");
            return NULL;
        }
-#ifdef AFS_MAXVCOUNT_ENV
     }
-#endif
     tvc = afs_AllocVCache();
-#else /* AFS_OSF_ENV */
+#else /* AFS_LINUX22_ENV */
     /* pull out a free cache entry */
     if (!freeVCList) {
-        int loop = 0;
-       i = 0;
-       for (tq = VLRU.prev; (anumber > 0) && (tq != &VLRU); tq = uq) {
-           tvc = QTOV(tq);
-           uq = QPrev(tq);
-
-           if (tvc->f.states & CVFlushed) {
-               refpanic("CVFlushed on VLRU");
-           } else if (i++ > 2 * afs_cacheStats) {      /* even allowing for a few xallocs... */
-               refpanic("Increase -stat parameter of afsd(VLRU cycle?)");
-           } else if (QNext(uq) != tq) {
-               refpanic("VLRU inconsistent");
-           } else if (tvc->f.states & CVInit) {
-               continue;
-            }
-
-           if (!VREFCOUNT_GT(tvc,0)
-#if defined(AFS_DARWIN_ENV) && !defined(UKERNEL) && !defined(AFS_DARWIN80_ENV)
-               || ((VREFCOUNT(tvc) == 1) && 
-                   (UBCINFOEXISTS(AFSTOV(tvc))))
-#endif
-               && tvc->opens == 0 && (tvc->f.states & CUnlinkedDel) == 0) {
-#if defined (AFS_DARWIN_ENV) || defined(AFS_XBSD_ENV)
-#ifdef AFS_DARWIN80_ENV
-               vnode_t tvp = AFSTOV(tvc);
-               /* VREFCOUNT_GT only sees usecounts, not iocounts */
-               /* so this may fail to actually recycle the vnode now */
-               /* must call vnode_get to avoid races. */
-                fv_slept = 0;
-               if (vnode_get(tvp) == 0) {
-                   fv_slept=1;
-                   /* must release lock, since vnode_put will immediately
-                      reclaim if there are no other users */
-                   ReleaseWriteLock(&afs_xvcache);
-                   AFS_GUNLOCK();
-                   vnode_recycle(tvp);
-                   vnode_put(tvp);
-                   AFS_GLOCK();
-                   ObtainWriteLock(&afs_xvcache, 336);
-               }
-               /* we can't use the vnode_recycle return value to figure
-                * this out, since the iocount we have to hold makes it
-                * always "fail" */
-               if (AFSTOV(tvc) == tvp) {
-                    if (anumber > 0 && fv_slept) {
-                       QRemove(&tvc->vlruq);
-                       QAdd(&VLRU, &tvc->vlruq);
-                    }
-                   code = EBUSY;
-               } else
-                   code = 0;
-#else /* AFS_DARWIN80_ENV */
-                /*
-                 * vgone() reclaims the vnode, which calls afs_FlushVCache(),
-                 * then it puts the vnode on the free list.
-                 * If we don't do this we end up with a cleaned vnode that's
-                 * not on the free list.
-                 * XXX assume FreeBSD is the same for now.
-                 */
-                AFS_GUNLOCK();
-                vgone(AFSTOV(tvc));
-                fv_slept = 0;
-                code = 0;
-                AFS_GLOCK();
-#endif
-#else /* AFS_DARWIN80_ENV || AFS_XBSD_ENV */
-                code = afs_FlushVCache(tvc, &fv_slept);
-#endif /* AFS_DARWIN80_ENV || AFS_XBSD_ENV */
-               if (code == 0) {
-                   anumber--;
-               }
-               if (fv_slept) {
-                    if (loop++ > 100)
-                       break;
-                   uq = VLRU.prev;
-                   i = 0;
-                   continue;   /* start over - may have raced. */
-               }
-           }
-           if (tq == uq)
-               break;
-       }
-    } /* end of if (!freeVCList) */
+       afs_ShakeLooseVCaches(anumber);
+    }
 
     if (!freeVCList) {
        tvc = afs_AllocVCache();
@@ -971,63 +870,20 @@ afs_NewVCache(struct VenusFid *afid, struct server *serverp)
        tvc = freeVCList;       /* take from free list */
        freeVCList = tvc->nextfree;
        tvc->nextfree = NULL;
+       afs_vcount++; /* balanced by FlushVCache */
     } /* end of if (!freeVCList) */
 
-#endif /* AFS_OSF_ENV */
+#endif /* AFS_LINUX22_ENV */
 
 #if defined(AFS_XBSD_ENV) || defined(AFS_DARWIN_ENV)
     if (tvc->v)
        panic("afs_NewVCache(): free vcache with vnode attached");
 #endif
 
-#if !defined(AFS_SGI_ENV) && !defined(AFS_OSF_ENV) && !defined(AFS_LINUX22_ENV)
+    /* Populate the vcache with as much as we can. */
+    afs_PrePopulateVCache(tvc, afid, serverp);
 
-#if defined(AFS_DISCON_ENV)
-    /* We need to preserve the slot that we're being stored into on
-     * disk */
-    { 
-       afs_uint32 slot;
-        slot = tvc->diskSlot;
-       memset((char *)tvc, 0, sizeof(struct vcache));
-       tvc->diskSlot = slot;
-    }
-#else
-    memset((char *)tvc, 0, sizeof(struct vcache));
-#endif
-
-#else
-    tvc->uncred = 0;
-    memset(&(tvc->f), 0, sizeof(struct fvcache));
-#endif
-
-    AFS_RWLOCK_INIT(&tvc->lock, "vcache lock");
-#if    defined(AFS_SUN5_ENV)
-    AFS_RWLOCK_INIT(&tvc->vlock, "vcache vlock");
-#endif /* defined(AFS_SUN5_ENV) */
-
-    tvc->mvid = NULL;
-    tvc->linkData = NULL;
-    tvc->cbExpires = 0;
-    tvc->opens = 0;
-    tvc->execsOrWriters = 0;
-    tvc->flockCount = 0;
-    tvc->f.states = CVInit;
-    tvc->last_looker = 0;
-    tvc->f.fid = *afid;
-    tvc->asynchrony = -1;
-    tvc->vc_error = 0;
-#ifdef AFS_TEXT_ENV
-    tvc->flushDV.low = tvc->flushDV.high = AFS_MAXDV;
-#endif
-    hzero(tvc->mapDV);
-    tvc->f.truncPos = AFS_NOTRUNC;        /* don't truncate until we need to */
-    hzero(tvc->f.m.DataVersion);  /* in case we copy it into flushDV */
-    tvc->Access = NULL;
-    tvc->callback = serverp;    /* to minimize chance that clear
-                                * request is lost */
-#if defined(AFS_DISCON_ENV)
-    QZero(&tvc->metadirty);
-#endif
+    /* Thread the vcache onto the VLRU */
 
     i = VCHash(afid);
     j = VCHashV(afid);
@@ -1035,7 +891,7 @@ afs_NewVCache(struct VenusFid *afid, struct server *serverp)
     tvc->hnext = afs_vhashT[i];
     afs_vhashT[i] = tvc;
     QAdd(&afs_vhashTV[j], &tvc->vhashq);
-    
+
     if ((VLRU.next->prev != &VLRU) || (VLRU.prev->next != &VLRU)) {
         refpanic("NewVCache VLRU inconsistent");
     }
@@ -1050,206 +906,43 @@ afs_NewVCache(struct VenusFid *afid, struct server *serverp)
         refpanic("NewVCache VLRU inconsistent4");
     }
     vcachegen++;
-    /* it should now be safe to drop the xvcache lock */
-#ifdef AFS_OBSD_ENV
-    ReleaseWriteLock(&afs_xvcache);
-    AFS_GUNLOCK();
-    afs_nbsd_getnewvnode(tvc); /* includes one refcount */
-    AFS_GLOCK();
-    ObtainWriteLock(&afs_xvcache,337);
-    lockinit(&tvc->rwlock, PINOD, "vcache", 0, 0);
-#endif
-#ifdef AFS_DARWIN_ENV
-    ReleaseWriteLock(&afs_xvcache);
-    AFS_GUNLOCK();
-    afs_darwin_getnewvnode(tvc);       /* includes one refcount */
-    AFS_GLOCK();
-    ObtainWriteLock(&afs_xvcache,338);
-#ifdef AFS_DARWIN80_ENV
-    LOCKINIT(tvc->rwlock);
-#else
-    lockinit(&tvc->rwlock, PINOD, "vcache", 0, 0);
-#endif
-#endif
-#ifdef AFS_FBSD_ENV
-    {
-       struct vnode *vp;
 
-       ReleaseWriteLock(&afs_xvcache);
-       AFS_GUNLOCK();
-#if defined(AFS_FBSD60_ENV)
-       if (getnewvnode(MOUNT_AFS, afs_globalVFS, &afs_vnodeops, &vp))
-#elif defined(AFS_FBSD50_ENV)
-       if (getnewvnode(MOUNT_AFS, afs_globalVFS, afs_vnodeop_p, &vp))
-#else
-       if (getnewvnode(VT_AFS, afs_globalVFS, afs_vnodeop_p, &vp))
-#endif
-           panic("afs getnewvnode");   /* can't happen */
-       AFS_GLOCK();
-       ObtainWriteLock(&afs_xvcache,339);
-       if (tvc->v != NULL) {
-           /* I'd like to know if this ever happens...
-            * We don't drop global for the rest of this function,
-            * so if we do lose the race, the other thread should
-            * have found the same vnode and finished initializing
-            * the vcache entry.  Is it conceivable that this vcache
-            * entry could be recycled during this interval?  If so,
-            * then there probably needs to be some sort of additional
-            * mutual exclusion (an Embryonic flag would suffice).
-            * -GAW */
-           printf("afs_NewVCache: lost the race\n");
-           return (tvc);
-       }
-       tvc->v = vp;
-       tvc->v->v_data = tvc;
-       lockinit(&tvc->rwlock, PINOD, "vcache", 0, 0);
-    }
-#endif
-
-#if defined(AFS_OSF_ENV) || defined(AFS_LINUX22_ENV)
+    /* it should now be safe to drop the xvcache lock - so attach an inode
+     * to this vcache, where necessary */
+    osi_AttachVnode(tvc, seq);
+
+    /* Get a reference count to hold this vcache for the VLRUQ. Note that
+     * we have to do this after attaching the vnode, because the reference
+     * count may be held in the vnode itself */
+
+#if defined(AFS_LINUX22_ENV)
     /* Hold it for the LRU (should make count 2) */
-    VN_HOLD(AFSTOV(tvc));
-#else /* AFS_OSF_ENV */
-#if !(defined (AFS_DARWIN_ENV) || defined(AFS_XBSD_ENV))
+    AFS_FAST_HOLD(tvc);
+#elif !(defined (AFS_DARWIN_ENV) || defined(AFS_XBSD_ENV))
     VREFCOUNT_SET(tvc, 1);     /* us */
-#endif /* AFS_XBSD_ENV */
-#endif /* AFS_OSF_ENV */
-#ifdef AFS_AIX32_ENV
-    LOCK_INIT(&tvc->pvmlock, "vcache pvmlock");
-    tvc->vmh = tvc->segid = NULL;
-    tvc->credp = NULL;
-#endif
-
-#if defined(AFS_CACHE_BYPASS)
-    tvc->cachingStates = 0;
-    tvc->cachingTransitions = 0;
 #endif
 
-#ifdef AFS_BOZONLOCK_ENV
-#if    defined(AFS_SUN5_ENV)
-    rw_init(&tvc->rwlock, "vcache rwlock", RW_DEFAULT, NULL);
-
-#if    defined(AFS_SUN55_ENV)
-    /* This is required if the kaio (kernel aynchronous io)
-     ** module is installed. Inside the kernel, the function
-     ** check_vp( common/os/aio.c) checks to see if the kernel has
-     ** to provide asynchronous io for this vnode. This
-     ** function extracts the device number by following the
-     ** v_data field of the vnode. If we do not set this field
-     ** then the system panics. The  value of the v_data field
-     ** is not really important for AFS vnodes because the kernel
-     ** does not do asynchronous io for regular files. Hence,
-     ** for the time being, we fill up the v_data field with the
-     ** vnode pointer itself. */
-    tvc->v.v_data = (char *)tvc;
-#endif /* AFS_SUN55_ENV */
-#endif
-    afs_BozonInit(&tvc->pvnLock, tvc);
-#endif
-
-    /* initialize vnode data, note vrefCount is v.v_count */
-#ifdef AFS_AIX_ENV
-    /* Don't forget to free the gnode space */
-    tvc->v.v_gnode = gnodepnt =
-       (struct gnode *)osi_AllocSmallSpace(sizeof(struct gnode));
-    memset((char *)gnodepnt, 0, sizeof(struct gnode));
-#endif
-#ifdef AFS_SGI64_ENV
-    memset((void *)&(tvc->vc_bhv_desc), 0, sizeof(tvc->vc_bhv_desc));
-    bhv_desc_init(&(tvc->vc_bhv_desc), tvc, tvc, &Afs_vnodeops);
-#ifdef AFS_SGI65_ENV
-    vn_bhv_head_init(&(tvc->v.v_bh), "afsvp");
-    vn_bhv_insert_initial(&(tvc->v.v_bh), &(tvc->vc_bhv_desc));
-#else
-    bhv_head_init(&(tvc->v.v_bh));
-    bhv_insert_initial(&(tvc->v.v_bh), &(tvc->vc_bhv_desc));
+#if defined (AFS_FBSD_ENV)
+    if (tvc->f.states & CVInit)
 #endif
-#ifdef AFS_SGI65_ENV
-    tvc->v.v_mreg = tvc->v.v_mregb = (struct pregion *)tvc;
-#ifdef VNODE_TRACING
-    tvc->v.v_trace = ktrace_alloc(VNODE_TRACE_SIZE, 0);
-#endif
-    init_bitlock(&tvc->v.v_pcacheflag, VNODE_PCACHE_LOCKBIT, "afs_pcache",
-                tvc->v.v_number);
-    init_mutex(&tvc->v.v_filocksem, MUTEX_DEFAULT, "afsvfl", (long)tvc);
-    init_mutex(&tvc->v.v_buf_lock, MUTEX_DEFAULT, "afsvnbuf", (long)tvc);
-#endif
-    vnode_pcache_init(&tvc->v);
-#if defined(DEBUG) && defined(VNODE_INIT_BITLOCK)
-    /* Above define is never true execpt in SGI test kernels. */
-    init_bitlock(&(tvc->v.v_flag, VLOCK, "vnode", tvc->v.v_number);
-#endif
-#ifdef INTR_KTHREADS
-                AFS_VN_INIT_BUF_LOCK(&(tvc->v));
-#endif
-#else
-    SetAfsVnode(AFSTOV(tvc));
-#endif /* AFS_SGI64_ENV */
-    /*
-     * The proper value for mvstat (for root fids) is setup by the caller.
-     */
-    tvc->mvstat = 0;
-    if (afid->Fid.Vnode == 1 && afid->Fid.Unique == 1)
-       tvc->mvstat = 2;
-    if (afs_globalVFS == 0)
-       osi_Panic("afs globalvfs");
-#if !defined(AFS_LINUX22_ENV)
-    vSetVfsp(tvc, afs_globalVFS);
-#endif
-    vSetType(tvc, VREG);
-#ifdef AFS_AIX_ENV
-    tvc->v.v_vfsnext = afs_globalVFS->vfs_vnodes;      /* link off vfs */
-    tvc->v.v_vfsprev = NULL;
-    afs_globalVFS->vfs_vnodes = &tvc->v;
-    if (tvc->v.v_vfsnext != NULL)
-       tvc->v.v_vfsnext->v_vfsprev = &tvc->v;
-    tvc->v.v_next = gnodepnt->gn_vnode;        /*Single vnode per gnode for us! */
-    gnodepnt->gn_vnode = &tvc->v;
-#endif
-#if    defined(AFS_DUX40_ENV)
-    insmntque(tvc, afs_globalVFS, &afs_ubcops);
-#else
-#ifdef  AFS_OSF_ENV
-    /* Is this needed??? */
-    insmntque(tvc, afs_globalVFS);
-#endif /* AFS_OSF_ENV */
-#endif /* AFS_DUX40_ENV */
-#ifdef AFS_FBSD70_ENV
-#ifndef AFS_FBSD80_ENV /* yup.  they put it back. */
-    insmntque(AFSTOV(tvc), afs_globalVFS);
-#endif
-#endif
-#if defined(AFS_SGI_ENV)
-    VN_SET_DPAGES(&(tvc->v), (struct pfdat *)NULL);
-    osi_Assert((tvc->v.v_flag & VINACT) == 0);
-    tvc->v.v_flag = 0;
-    osi_Assert(VN_GET_PGCNT(&(tvc->v)) == 0);
-    osi_Assert(tvc->mapcnt == 0 && tvc->vc_locktrips == 0);
-    osi_Assert(tvc->vc_rwlockid == OSI_NO_LOCKID);
-    osi_Assert(tvc->v.v_filocks == NULL);
-#if !defined(AFS_SGI65_ENV)
-    osi_Assert(tvc->v.v_filocksem == NULL);
-#endif
-    osi_Assert(tvc->cred == NULL);
-#ifdef AFS_SGI64_ENV
-    vnode_pcache_reinit(&tvc->v);
-    tvc->v.v_rdev = NODEV;
-#endif
-    vn_initlist((struct vnlist *)&tvc->v);
-    tvc->lastr = 0;
-#endif /* AFS_SGI_ENV */
-    tvc->dchint = NULL;
-    osi_dnlc_purgedp(tvc);     /* this may be overkill */
-    memset((char *)&(tvc->callsort), 0, sizeof(struct afs_q));
-    tvc->slocks = NULL;
-    tvc->f.states &=~ CVInit;
-    afs_osi_Wakeup(&tvc->f.states);
+    afs_PostPopulateVCache(tvc, afid, seq);
 
     return tvc;
-
 }                              /*afs_NewVCache */
 
 
+struct vcache *
+afs_NewVCache(struct VenusFid *afid, struct server *serverp)
+{
+    return afs_NewVCache_int(afid, serverp, 0);
+}
+
+struct vcache *
+afs_NewBulkVCache(struct VenusFid *afid, struct server *serverp, int seq)
+{
+    return afs_NewVCache_int(afid, serverp, seq);
+}
+
 /*!
  * ???
  *
@@ -1258,13 +951,13 @@ afs_NewVCache(struct VenusFid *afid, struct server *serverp)
  * \param doflocks : Do we handle flocks?
  */
 void
-afs_FlushActiveVcaches(register afs_int32 doflocks)
+afs_FlushActiveVcaches(afs_int32 doflocks)
 {
-    register struct vcache *tvc;
-    register int i;
-    register struct afs_conn *tc;
-    register afs_int32 code;
-    register struct AFS_UCRED *cred = NULL;
+    struct vcache *tvc;
+    int i;
+    struct afs_conn *tc;
+    afs_int32 code;
+    afs_ucred_t *cred = NULL;
     struct vrequest treq, ureq;
     struct AFSVolSync tsync;
     int didCore;
@@ -1336,7 +1029,7 @@ afs_FlushActiveVcaches(register afs_int32 doflocks)
                if (tvc->f.states & CCore) {
                    tvc->f.states &= ~CCore;
                    /* XXXX Find better place-holder for cred XXXX */
-                   cred = (struct AFS_UCRED *)tvc->linkData;
+                   cred = (afs_ucred_t *)tvc->linkData;
                    tvc->linkData = NULL;       /* XXX */
                    afs_InitReq(&ureq, cred);
                    afs_Trace2(afs_iclSetp, CM_TRACE_ACTCCORE,
@@ -1419,9 +1112,9 @@ afs_FlushActiveVcaches(register afs_int32 doflocks)
  */
 
 /*!
- * 
+ *
  *   Make sure a cache entry is up-to-date status-wise.
- *   
+ *
  *   NOTE: everywhere that calls this can potentially be sped up
  *       by checking CStatd first, and avoiding doing the InitReq
  *       if this is up-to-date.
@@ -1437,18 +1130,10 @@ afs_FlushActiveVcaches(register afs_int32 doflocks)
 int
 afs_VerifyVCache2(struct vcache *avc, struct vrequest *areq)
 {
-    register struct vcache *tvc;
+    struct vcache *tvc;
 
     AFS_STATCNT(afs_VerifyVCache);
 
-#if defined(AFS_OSF_ENV)
-    ObtainReadLock(&avc->lock);
-    if (afs_IsWired(avc)) {
-       ReleaseReadLock(&avc->lock);
-       return 0;
-    }
-    ReleaseReadLock(&avc->lock);
-#endif /* AFS_OSF_ENV */
     /* otherwise we must fetch the status info */
 
     ObtainWriteLock(&avc->lock, 53);
@@ -1493,24 +1178,21 @@ afs_VerifyVCache2(struct vcache *avc, struct vrequest *areq)
  *
  */
 static void
-afs_SimpleVStat(register struct vcache *avc,
-               register struct AFSFetchStatus *astat, struct vrequest *areq)
+afs_SimpleVStat(struct vcache *avc,
+               struct AFSFetchStatus *astat, struct vrequest *areq)
 {
     afs_size_t length;
     AFS_STATCNT(afs_SimpleVStat);
 
-#ifdef AFS_SGI_ENV
-    if ((avc->execsOrWriters <= 0) && !afs_DirtyPages(avc)
-       && !AFS_VN_MAPPED((vnode_t *) avc)) {
-#else
-    if ((avc->execsOrWriters <= 0) && !afs_DirtyPages(avc)) {
-#endif
 #ifdef AFS_64BIT_CLIENT
        FillInt64(length, astat->Length_hi, astat->Length);
 #else /* AFS_64BIT_CLIENT */
        length = astat->Length;
 #endif /* AFS_64BIT_CLIENT */
+
 #if defined(AFS_SGI_ENV)
+    if ((avc->execsOrWriters <= 0) && !afs_DirtyPages(avc)
+       && !AFS_VN_MAPPED((vnode_t *) avc)) {
        osi_Assert((valusema(&avc->vc_rwlock) <= 0)
                   && (OSI_GET_LOCKID() == avc->vc_rwlockid));
        if (length < avc->f.m.Length) {
@@ -1523,8 +1205,11 @@ afs_SimpleVStat(register struct vcache *avc,
            AFS_GLOCK();
            ObtainWriteLock(&avc->lock, 67);
        }
+    }
 #endif
-       /* if writing the file, don't fetch over this value */
+
+    if (!afs_DirtyPages(avc)) {
+       /* if actively writing the file, don't fetch over this value */
        afs_Trace3(afs_iclSetp, CM_TRACE_SIMPLEVSTAT, ICL_TYPE_POINTER, avc,
                   ICL_TYPE_OFFSET, ICL_HANDLE_OFFSET(avc->f.m.Length),
                   ICL_TYPE_OFFSET, ICL_HANDLE_OFFSET(length));
@@ -1586,8 +1271,8 @@ afs_SimpleVStat(register struct vcache *avc,
  */
 
 int
-afs_WriteVCache(register struct vcache *avc,
-               register struct AFSStoreStatus *astatus,
+afs_WriteVCache(struct vcache *avc,
+               struct AFSStoreStatus *astatus,
                struct vrequest *areq)
 {
     afs_int32 code;
@@ -1637,7 +1322,6 @@ afs_WriteVCache(register struct vcache *avc,
     return code;
 
 }                              /*afs_WriteVCache */
-#if defined(AFS_DISCON_ENV)
 
 /*!
  * Store status info only locally, set the proper disconnection flags
@@ -1649,9 +1333,10 @@ afs_WriteVCache(register struct vcache *avc,
  *
  * \note Must be called with a shared lock on the vnode
  */
-int afs_WriteVCacheDiscon(register struct vcache *avc,
-                               register struct AFSStoreStatus *astatus,
-                               struct vattr *attrs)
+int
+afs_WriteVCacheDiscon(struct vcache *avc,
+                     struct AFSStoreStatus *astatus,
+                     struct vattr *attrs)
 {
     afs_int32 code = 0;
     afs_int32 flags = 0;
@@ -1671,13 +1356,13 @@ int afs_WriteVCacheDiscon(register struct vcache *avc,
        }
 
        if (astatus->Mask & AFS_SETOWNER) {
-               printf("Not allowed yet. \n");
-               /*avc->f.m.Owner = astatus->Owner;*/
+           /* printf("Not allowed yet. \n"); */
+           /*avc->f.m.Owner = astatus->Owner;*/
        }
 
        if (astatus->Mask & AFS_SETGROUP) {
-               printf("Not allowed yet. \n");
-               /*avc->f.m.Group =  astatus->Group;*/
+           /* printf("Not allowed yet. \n"); */
+           /*avc->f.m.Group =  astatus->Group;*/
        }
 
        if (astatus->Mask & AFS_SETMODE) {
@@ -1716,13 +1401,11 @@ int afs_WriteVCacheDiscon(register struct vcache *avc,
     return code;
 }
 
-#endif
-
 /*!
  * Copy astat block into vcache info
  *
  * \note This code may get dataversion and length out of sync if the file has
- * been modified.  This is less than ideal.  I haven't thought about it sufficiently 
+ * been modified.  This is less than ideal.  I haven't thought about it sufficiently
  * to be certain that it is adequate.
  *
  * \note Environment: Must be called under a write lock
@@ -1732,8 +1415,8 @@ int afs_WriteVCacheDiscon(register struct vcache *avc,
  * \param areq Ptr to associated request.
  */
 void
-afs_ProcessFS(register struct vcache *avc,
-             register struct AFSFetchStatus *astat, struct vrequest *areq)
+afs_ProcessFS(struct vcache *avc,
+             struct AFSFetchStatus *astat, struct vrequest *areq)
 {
     afs_size_t length;
     AFS_STATCNT(afs_ProcessFS);
@@ -1818,18 +1501,18 @@ afs_ProcessFS(register struct vcache *avc,
 /*!
  * Get fid from server.
  *
- * \param afid 
+ * \param afid
  * \param areq Request to be passed on.
  * \param name Name of ?? to lookup.
  * \param OutStatus Fetch status.
- * \param CallBackp 
+ * \param CallBackp
  * \param serverp
  * \param tsyncp
  *
  * \return Success status of operation.
  */
 int
-afs_RemoteLookup(register struct VenusFid *afid, struct vrequest *areq,
+afs_RemoteLookup(struct VenusFid *afid, struct vrequest *areq,
                 char *name, struct VenusFid *nfid,
                 struct AFSFetchStatus *OutStatusp,
                 struct AFSCallBack *CallBackp, struct server **serverp,
@@ -1837,16 +1520,16 @@ afs_RemoteLookup(register struct VenusFid *afid, struct vrequest *areq,
 {
     afs_int32 code;
     afs_uint32 start;
-    register struct afs_conn *tc;
+    struct afs_conn *tc;
     struct AFSFetchStatus OutDirStatus;
     XSTATS_DECLS;
     if (!name)
-       name = "";              /* XXX */    
+       name = "";              /* XXX */
     do {
        tc = afs_Conn(afid, areq, SHARED_LOCK);
        if (tc) {
            if (serverp)
-               *serverp = tc->srvr->server;
+               *serverp = tc->parent->srvr->server;
            start = osi_Time();
            XSTATS_START_TIME(AFS_STATS_FS_RPCIDX_XLOOKUP);
            RX_AFS_GUNLOCK();
@@ -1894,19 +1577,19 @@ afs_RemoteLookup(register struct VenusFid *afid, struct vrequest *areq,
  *     of a parent dir cache entry, given a file (to check its access
  *     control list).  It also allows renames to be handled easily by
  *     locking directories in a constant order.
- * 
+ *
  * \note NB.  NewVCache -> FlushVCache presently (4/10/95) drops the xvcache lock.
  *
  * \note Might have a vcache structure already, which must
- *  already be held by the caller 
+ *  already be held by the caller
  */
 struct vcache *
-afs_GetVCache(register struct VenusFid *afid, struct vrequest *areq,
+afs_GetVCache(struct VenusFid *afid, struct vrequest *areq,
              afs_int32 * cached, struct vcache *avc)
 {
 
     afs_int32 code, newvcache = 0;
-    register struct vcache *tvc;
+    struct vcache *tvc;
     struct volume *tvp;
     afs_int32 retry;
 
@@ -1929,7 +1612,6 @@ afs_GetVCache(register struct VenusFid *afid, struct vrequest *areq,
        goto loop;
 #endif
     }
-
     if (tvc) {
        if (cached)
            *cached = 1;
@@ -1964,12 +1646,6 @@ afs_GetVCache(register struct VenusFid *afid, struct vrequest *areq,
        ReleaseWriteLock(&tvc->lock);
        return tvc;
     }
-#if defined(AFS_OSF_ENV)
-    if (afs_IsWired(tvc)) {
-       ReleaseWriteLock(&tvc->lock);
-       return tvc;
-    }
-#endif /* AFS_OSF_ENV */
 #ifdef AFS_DARWIN80_ENV
 /* Darwin 8.0 only has bufs in nfs, so we shouldn't have to worry about them.
    What about ubc? */
@@ -1995,12 +1671,12 @@ afs_GetVCache(register struct VenusFid *afid, struct vrequest *areq,
        if (!iheldthelock)
            vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, current_proc());
        /* this is messy. we can call fsync which will try to reobtain this */
-       if (VTOAFS(vp) == tvc) 
+       if (VTOAFS(vp) == tvc)
          ReleaseWriteLock(&tvc->lock);
        if (UBCINFOEXISTS(vp)) {
          vinvalbuf(vp, V_SAVE, &afs_osi_cred, current_proc(), PINOD, 0);
        }
-       if (VTOAFS(vp) == tvc) 
+       if (VTOAFS(vp) == tvc)
          ObtainWriteLock(&tvc->lock, 954);
        if (!iheldthelock)
            VOP_UNLOCK(vp, LK_EXCLUSIVE, current_proc());
@@ -2015,30 +1691,25 @@ afs_GetVCache(register struct VenusFid *afid, struct vrequest *areq,
            if (glocked)
                AFS_GLOCK();
        }
-       vinvalbuf(vp, V_SAVE, curthread, PINOD, 0);
+       vinvalbuf(vp, V_SAVE, PINOD, 0); /* changed late in 8.0-CURRENT */
        if (!iheldthelock)
            VOP_UNLOCK(vp, 0);
 #elif defined(AFS_FBSD60_ENV)
        iheldthelock = VOP_ISLOCKED(vp, curthread);
        if (!iheldthelock)
            vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, curthread);
+       AFS_GUNLOCK();
        vinvalbuf(vp, V_SAVE, curthread, PINOD, 0);
+       AFS_GLOCK();
        if (!iheldthelock)
            VOP_UNLOCK(vp, LK_EXCLUSIVE, curthread);
-#elif defined(AFS_FBSD50_ENV)
+#elif defined(AFS_FBSD_ENV)
        iheldthelock = VOP_ISLOCKED(vp, curthread);
        if (!iheldthelock)
            vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, curthread);
        vinvalbuf(vp, V_SAVE, osi_curcred(), curthread, PINOD, 0);
        if (!iheldthelock)
            VOP_UNLOCK(vp, LK_EXCLUSIVE, curthread);
-#elif defined(AFS_FBSD40_ENV)
-       iheldthelock = VOP_ISLOCKED(vp, curproc);
-       if (!iheldthelock)
-           vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, curproc);
-       vinvalbuf(vp, V_SAVE, osi_curcred(), curproc, PINOD, 0);
-       if (!iheldthelock)
-           VOP_UNLOCK(vp, LK_EXCLUSIVE, curproc);
 #elif defined(AFS_OBSD_ENV)
        iheldthelock = VOP_ISLOCKED(vp, curproc);
        if (!iheldthelock)
@@ -2046,6 +1717,14 @@ afs_GetVCache(register struct VenusFid *afid, struct vrequest *areq,
        uvm_vnp_uncache(vp);
        if (!iheldthelock)
            VOP_UNLOCK(vp, 0, curproc);
+#elif defined(AFS_NBSD40_ENV)
+       iheldthelock = VOP_ISLOCKED(vp);
+       if (!iheldthelock) {
+           VOP_LOCK(vp, LK_EXCLUSIVE | LK_RETRY);
+       }
+       uvm_vnp_uncache(vp);
+       if (!iheldthelock)
+           VOP_UNLOCK(vp, 0);
 #endif
     }
 #endif
@@ -2099,7 +1778,7 @@ afs_GetVCache(register struct VenusFid *afid, struct vrequest *areq,
            if (AFS_IS_DISCONNECTED) {
                /* Nothing to do otherwise...*/
                code = ENETDOWN;
-               printf("Network is down in afs_GetCache");
+               /* printf("Network is down in afs_GetCache"); */
            } else
                code = afs_FetchStatus(tvc, afid, areq, &OutStatus);
 
@@ -2141,9 +1820,9 @@ afs_GetVCache(register struct VenusFid *afid, struct vrequest *areq,
  * Lookup a vcache by fid. Look inside the cache first, if not
  * there, lookup the file on the server, and then get it's fresh
  * cache entry.
- * 
+ *
  * \param afid
- * \param areq 
+ * \param areq
  * \param cached Is element cached? If NULL, don't answer.
  * \param adp
  * \param aname
@@ -2156,7 +1835,7 @@ afs_LookupVCache(struct VenusFid *afid, struct vrequest *areq,
 {
     afs_int32 code, now, newvcache = 0;
     struct VenusFid nfid;
-    register struct vcache *tvc;
+    struct vcache *tvc;
     struct volume *tvp;
     struct AFSFetchStatus OutStatus;
     struct AFSCallBack CallBack;
@@ -2206,13 +1885,13 @@ afs_LookupVCache(struct VenusFid *afid, struct vrequest *areq,
     nfid = *afid;
     now = osi_Time();
     origCBs = afs_allCBs;      /* if anything changes, we don't have a cb */
-    
+
     if (AFS_IS_DISCONNECTED) {
-       printf("Network is down in afs_LookupVcache\n");
+       /* printf("Network is down in afs_LookupVcache\n"); */
         code = ENETDOWN;
-    } else 
+    } else
         code =
-           afs_RemoteLookup(&adp->f.fid, areq, aname, &nfid, &OutStatus, 
+           afs_RemoteLookup(&adp->f.fid, areq, aname, &nfid, &OutStatus,
                             &CallBack, &serverp, &tsync);
 
 #if    defined(AFS_SGI_ENV) && !defined(AFS_SGI53_ENV)
@@ -2330,15 +2009,12 @@ afs_GetRootVCache(struct VenusFid *afid, struct vrequest *areq,
     afs_int32 getNewFid = 0;
     afs_uint32 start;
     struct VenusFid nfid;
-    register struct vcache *tvc;
+    struct vcache *tvc;
     struct server *serverp = 0;
     struct AFSFetchStatus OutStatus;
     struct AFSCallBack CallBack;
     struct AFSVolSync tsync;
     int origCBs = 0;
-#ifdef AFS_OSF_ENV
-    int vg;
-#endif
 #ifdef AFS_DARWIN80_ENV
     vnode_t tvp;
 #endif
@@ -2380,23 +2056,14 @@ afs_GetRootVCache(struct VenusFid *afid, struct vrequest *areq,
                afs_osi_Sleep(&tvc->f.states);
                goto rootvc_loop;
             }
-#ifdef AFS_OSF_ENV
-           /* Grab this vnode, possibly reactivating from the free list */
-           /* for the present (95.05.25) everything on the hash table is
-            * definitively NOT in the free list -- at least until afs_reclaim
-            * can be safely implemented */
-           AFS_GUNLOCK();
-           vg = vget(AFSTOV(tvc));     /* this bumps ref count */
-           AFS_GLOCK();
-           if (vg)
-               continue;
-#endif /* AFS_OSF_ENV */
 #ifdef AFS_DARWIN80_ENV
-            if (tvc->f.states & CDeadVnode) {
-               ReleaseSharedLock(&afs_xvcache);
-               afs_osi_Sleep(&tvc->f.states);
-               goto rootvc_loop;
-            }
+           if (tvc->f.states & CDeadVnode) {
+               if (!(tvc->f.states & CBulkFetching)) {
+                   ReleaseSharedLock(&afs_xvcache);
+                   afs_osi_Sleep(&tvc->f.states);
+                   goto rootvc_loop;
+               }
+           }
            tvp = AFSTOV(tvc);
            if (vnode_get(tvp))       /* this bumps ref count */
                continue;
@@ -2407,6 +2074,11 @@ afs_GetRootVCache(struct VenusFid *afid, struct vrequest *areq,
                AFS_GLOCK();
                continue;
            }
+           if (tvc->f.states & (CBulkFetching|CDeadVnode)) {
+               AFS_GUNLOCK();
+               vnode_recycle(AFSTOV(tvc));
+               AFS_GLOCK();
+           }
 #endif
            break;
        }
@@ -2414,10 +2086,6 @@ afs_GetRootVCache(struct VenusFid *afid, struct vrequest *areq,
 
     if (!haveStatus && (!tvc || !(tvc->f.states & CStatd))) {
        /* Mount point no longer stat'd or unknown. FID may have changed. */
-#ifdef AFS_OSF_ENV
-       if (tvc)
-           AFS_RELE(AFSTOV(tvc));
-#endif
        getNewFid = 1;
        ReleaseSharedLock(&afs_xvcache);
 #ifdef AFS_DARWIN80_ENV
@@ -2447,9 +2115,9 @@ afs_GetRootVCache(struct VenusFid *afid, struct vrequest *areq,
        if (cached)
            *cached = 1;
        afs_stats_cmperf.vcacheHits++;
-#if    defined(AFS_OSF_ENV) || defined(AFS_DARWIN80_ENV)
+#if    defined(AFS_DARWIN80_ENV)
        /* we already bumped the ref count in the for loop above */
-#else /* AFS_OSF_ENV */
+#else /* AFS_DARWIN80_ENV */
        osi_vnhold(tvc, 0);
 #endif
        UpgradeSToWLock(&afs_xvcache, 24);
@@ -2583,12 +2251,9 @@ afs_GetRootVCache(struct VenusFid *afid, struct vrequest *areq,
  * \note The vcache must be write locked.
  */
 void
-afs_UpdateStatus(struct vcache *avc,
-                       struct VenusFid *afid,
-                       struct vrequest *areq,
-                       struct AFSFetchStatus *Outsp,
-                       struct AFSCallBack *acb,
-                       afs_uint32 start)
+afs_UpdateStatus(struct vcache *avc, struct VenusFid *afid,
+                struct vrequest *areq, struct AFSFetchStatus *Outsp,
+                struct AFSCallBack *acb, afs_uint32 start)
 {
     struct volume *volp;
 
@@ -2641,7 +2306,7 @@ afs_FetchStatus(struct vcache * avc, struct VenusFid * afid,
 {
     int code;
     afs_uint32 start = 0;
-    register struct afs_conn *tc;
+    struct afs_conn *tc;
     struct AFSCallBack CallBack;
     struct AFSVolSync tsync;
     XSTATS_DECLS;
@@ -2649,7 +2314,7 @@ afs_FetchStatus(struct vcache * avc, struct VenusFid * afid,
        tc = afs_Conn(afid, areq, SHARED_LOCK);
        avc->dchint = NULL;     /* invalidate hints */
        if (tc) {
-           avc->callback = tc->srvr->server;
+           avc->callback = tc->parent->srvr->server;
            start = osi_Time();
            XSTATS_START_TIME(AFS_STATS_FS_RPCIDX_FETCHSTATUS);
            RX_AFS_GUNLOCK();
@@ -2704,13 +2369,13 @@ afs_FetchStatus(struct vcache * avc, struct VenusFid * afid,
  *     Nothing interesting.
  */
 void
-afs_StuffVcache(register struct VenusFid *afid,
+afs_StuffVcache(struct VenusFid *afid,
                struct AFSFetchStatus *OutStatus,
-               struct AFSCallBack *CallBack, register struct afs_conn *tc,
+               struct AFSCallBack *CallBack, struct afs_conn *tc,
                struct vrequest *areq)
 {
-    register afs_int32 code, i, newvcache = 0;
-    register struct vcache *tvc;
+    afs_int32 code, i, newvcache = 0;
+    struct vcache *tvc;
     struct AFSVolSync tsync;
     struct volume *tvp;
     struct axscache *ac;
@@ -2838,7 +2503,7 @@ afs_StuffVcache(register struct VenusFid *afid,
  * \note Environment: Nothing interesting.
  */
 void
-afs_PutVCache(register struct vcache *avc)
+afs_PutVCache(struct vcache *avc)
 {
     AFS_STATCNT(afs_PutVCache);
 #ifdef AFS_DARWIN80_ENV
@@ -2858,14 +2523,15 @@ afs_PutVCache(register struct vcache *avc)
 /*!
  * Reset a vcache entry, so local contents are ignored, and the
  * server will be reconsulted next time the vcache is used
- * 
+ *
  * \param avc Pointer to the cache entry to reset
- * \param acred 
+ * \param acred
  *
  * \note avc must be write locked on entry
  */
 void
-afs_ResetVCache(struct vcache *avc, struct AFS_UCRED *acred) {
+afs_ResetVCache(struct vcache *avc, afs_ucred_t *acred)
+{
     ObtainWriteLock(&afs_xcbhash, 456);
     afs_DequeueCallback(avc);
     avc->f.states &= ~(CStatd | CDirty);    /* next reference will re-stat */
@@ -2886,9 +2552,12 @@ afs_ResetVCache(struct vcache *avc, struct AFS_UCRED *acred) {
  * \param vcache Enter sleep state.
  * \param flag Determines what locks to use.
  *
- * \return 
+ * \return
  */
-static void findvc_sleep(struct vcache *avc, int flag) {
+static void
+findvc_sleep(struct vcache *avc, int flag)
+{
+    int fstates = avc->f.states;
     if (flag & IS_SLOCK) {
            ReleaseSharedLock(&afs_xvcache);
     } else {
@@ -2898,7 +2567,16 @@ static void findvc_sleep(struct vcache *avc, int flag) {
            ReleaseReadLock(&afs_xvcache);
        }
     }
-    afs_osi_Sleep(&avc->f.states);
+    if (flag & FIND_CDEAD) {
+       ObtainWriteLock(&afs_xvcache, 342);
+       afs_FlushReclaimedVcaches();
+       if (fstates == avc->f.states) {
+           ReleaseWriteLock(&afs_xvcache);
+           afs_osi_Sleep(&avc->f.states);
+       } else
+           ReleaseWriteLock(&afs_xvcache);
+    } else
+       afs_osi_Sleep(&avc->f.states);
     if (flag & IS_SLOCK) {
            ObtainSharedLock(&afs_xvcache, 341);
     } else {
@@ -2909,6 +2587,45 @@ static void findvc_sleep(struct vcache *avc, int flag) {
        }
     }
 }
+
+/*!
+ * Add a reference on an existing vcache entry.
+ *
+ * \param tvc Pointer to the vcache.
+ *
+ * \note Environment: Must be called with at least one reference from
+ * elsewhere on the vcache, even if that reference will be dropped.
+ * The global lock is required.
+ *
+ * \return 0 on success, -1 on failure.
+ */
+
+int
+afs_RefVCache(struct vcache *tvc)
+{
+#ifdef AFS_DARWIN80_ENV
+    vnode_t tvp;
+#endif
+
+    /* AFS_STATCNT(afs_RefVCache); */
+
+#ifdef  AFS_DARWIN80_ENV
+    tvp = AFSTOV(tvc);
+    if (vnode_get(tvp))
+       return -1;
+    if (vnode_ref(tvp)) {
+       AFS_GUNLOCK();
+       /* AFSTOV(tvc) may be NULL */
+       vnode_put(tvp);
+       AFS_GLOCK();
+       return -1;
+    }
+#else
+       osi_vnhold(tvc, 0);
+#endif
+    return 0;
+}                              /*afs_RefVCache */
+
 /*!
  * Find a vcache entry given a fid.
  *
@@ -2927,12 +2644,10 @@ struct vcache *
 afs_FindVCache(struct VenusFid *afid, afs_int32 * retry, afs_int32 flag)
 {
 
-    register struct vcache *tvc;
+    struct vcache *tvc;
     afs_int32 i;
-#if defined( AFS_OSF_ENV)
-    int vg;
-#endif
 #ifdef AFS_DARWIN80_ENV
+    struct vcache *deadvc = NULL, *livevc = NULL;
     vnode_t tvp;
 #endif
 
@@ -2942,48 +2657,78 @@ afs_FindVCache(struct VenusFid *afid, afs_int32 * retry, afs_int32 flag)
     i = VCHash(afid);
     for (tvc = afs_vhashT[i]; tvc; tvc = tvc->hnext) {
        if (FidMatches(afid, tvc)) {
+#ifdef  AFS_DARWIN80_ENV
+           if (flag & FIND_CDEAD) {
+               if (tvc->f.states & (CDeadVnode|CBulkFetching)) {
+                   deadvc = tvc;
+                   continue;
+               }
+           } else {
+               if (tvc->f.states & CDeadVnode)
+                   if ((tvc->f.states & CBulkFetching) &&
+                       !(flag & FIND_BULKDEAD))
+                       continue;
+           }
+#endif
             if (tvc->f.states & CVInit) {
                findvc_sleep(tvc, flag);
                goto findloop;
-            }
-#ifdef  AFS_OSF_ENV
-           /* Grab this vnode, possibly reactivating from the free list */
-           AFS_GUNLOCK();
-           vg = vget(AFSTOV(tvc));
-           AFS_GLOCK();
-           if (vg)
-               continue;
-#endif /* AFS_OSF_ENV */
+           }
 #ifdef  AFS_DARWIN80_ENV
             if (tvc->f.states & CDeadVnode) {
-                findvc_sleep(tvc, flag);
+               findvc_sleep(tvc, flag);
                goto findloop;
             }
-           tvp = AFSTOV(tvc);
-           if (vnode_get(tvp))
-               continue;
-           if (vnode_ref(tvp)) {
-               AFS_GUNLOCK();
-               /* AFSTOV(tvc) may be NULL */
-               vnode_put(tvp);
-               AFS_GLOCK();
+           if (flag & FIND_CDEAD) {
+               livevc = tvc;
                continue;
            }
 #endif
            break;
        }
     }
+#ifdef  AFS_DARWIN80_ENV
+       if (flag & FIND_CDEAD) {
+           if (livevc && deadvc) {
+               /* discard deadvc */
+               AFS_GUNLOCK();
+               vnode_recycle(AFSTOV(deadvc));
+               vnode_put(AFSTOV(deadvc));
+               vnode_rele(AFSTOV(deadvc));
+               AFS_GLOCK();
+               deadvc = NULL;
+           }
+
+           /* return what's left */
+           tvc = livevc ? livevc : deadvc;
+       }
+#endif
 
     /* should I have a read lock on the vnode here? */
     if (tvc) {
        if (retry)
            *retry = 0;
-#if !defined(AFS_OSF_ENV) && !defined(AFS_DARWIN80_ENV)
-       osi_vnhold(tvc, retry); /* already held, above */
-       if (retry && *retry)
-           return 0;
-#endif
-#if defined(AFS_DARWIN_ENV) && !defined(AFS_DARWIN80_ENV)
+#if defined(AFS_DARWIN80_ENV)
+       tvp = AFSTOV(tvc);
+       if (vnode_get(tvp))
+           tvp = NULL;
+       if (tvp && vnode_ref(tvp)) {
+           AFS_GUNLOCK();
+           /* AFSTOV(tvc) may be NULL */
+           vnode_put(tvp);
+           AFS_GLOCK();
+           tvp = NULL;
+       }
+       if (tvp && (tvc->f.states & (CBulkFetching|CDeadVnode))) {
+           AFS_GUNLOCK();
+           vnode_recycle(AFSTOV(tvc));
+           AFS_GLOCK();
+       }
+       if (!tvp) {
+           tvc = NULL;
+           return tvc;
+       }
+#elif defined(AFS_DARWIN_ENV)
        tvc->f.states |= CUBCinit;
        AFS_GUNLOCK();
        if (UBCINFOMISSING(AFSTOV(tvc)) ||
@@ -2992,6 +2737,10 @@ afs_FindVCache(struct VenusFid *afid, afs_int32 * retry, afs_int32 flag)
        }
        AFS_GLOCK();
        tvc->f.states &= ~CUBCinit;
+#else
+       osi_vnhold(tvc, retry); /* already held, above */
+       if (retry && *retry)
+           return 0;
 #endif
        /*
         * only move to front of vlru if we have proper vcache locking)
@@ -3059,13 +2808,10 @@ int afs_duplicate_nfs_fids = 0;
 afs_int32
 afs_NFSFindVCache(struct vcache **avcp, struct VenusFid *afid)
 {
-    register struct vcache *tvc;
+    struct vcache *tvc;
     afs_int32 i;
     afs_int32 count = 0;
     struct vcache *found_tvc = NULL;
-#ifdef  AFS_OSF_ENV
-    int vg;
-#endif
 #ifdef AFS_DARWIN80_ENV
     vnode_t tvp;
 #endif
@@ -3088,22 +2834,14 @@ afs_NFSFindVCache(struct vcache **avcp, struct VenusFid *afid)
                afs_osi_Sleep(&tvc->f.states);
                goto loop;
             }
-#ifdef  AFS_OSF_ENV
-           /* Grab this vnode, possibly reactivating from the free list */
-           AFS_GUNLOCK();
-           vg = vget(AFSTOV(tvc));
-           AFS_GLOCK();
-           if (vg) {
-               /* This vnode no longer exists. */
-               continue;
-           }
-#endif /* AFS_OSF_ENV */
 #ifdef  AFS_DARWIN80_ENV
-            if (tvc->f.states & CDeadVnode) {
-               ReleaseSharedLock(&afs_xvcache);
-               afs_osi_Sleep(&tvc->f.states);
-               goto loop;
-            }
+           if (tvc->f.states & CDeadVnode) {
+               if (!(tvc->f.states & CBulkFetching)) {
+                   ReleaseSharedLock(&afs_xvcache);
+                   afs_osi_Sleep(&tvc->f.states);
+                   goto loop;
+               }
+           }
            tvp = AFSTOV(tvc);
            if (vnode_get(tvp)) {
                /* This vnode no longer exists. */
@@ -3117,15 +2855,15 @@ afs_NFSFindVCache(struct vcache **avcp, struct VenusFid *afid)
                AFS_GLOCK();
                continue;
            }
+           if (tvc->f.states & (CBulkFetching|CDeadVnode)) {
+               AFS_GUNLOCK();
+               vnode_recycle(AFSTOV(tvc));
+               AFS_GLOCK();
+           }
 #endif /* AFS_DARWIN80_ENV */
            count++;
            if (found_tvc) {
                /* Duplicates */
-#ifdef AFS_OSF_ENV
-               /* Drop our reference counts. */
-               vrele(AFSTOV(tvc));
-               vrele(AFSTOV(found_tvc));
-#endif
                afs_duplicate_nfs_fids++;
                ReleaseSharedLock(&afs_xvcache);
 #ifdef AFS_DARWIN80_ENV
@@ -3142,6 +2880,7 @@ afs_NFSFindVCache(struct vcache **avcp, struct VenusFid *afid)
     tvc = found_tvc;
     /* should I have a read lock on the vnode here? */
     if (tvc) {
+#ifndef AFS_DARWIN80_ENV
 #if defined(AFS_SGI_ENV) && !defined(AFS_SGI53_ENV)
        afs_int32 retry = 0;
        osi_vnhold(tvc, &retry);
@@ -3153,7 +2892,6 @@ afs_NFSFindVCache(struct vcache **avcp, struct VenusFid *afid)
            goto loop;
        }
 #else
-#if !defined(AFS_OSF_ENV)
        osi_vnhold(tvc, (int *)0);      /* already held, above */
 #endif
 #endif
@@ -3212,34 +2950,25 @@ afs_NFSFindVCache(struct vcache **avcp, struct VenusFid *afid)
 void
 afs_vcacheInit(int astatSize)
 {
-#if (!defined(AFS_OSF_ENV) && !defined(AFS_LINUX22_ENV)) || defined(AFS_SGI_ENV)
-    register struct vcache *tvp;
+#if !defined(AFS_LINUX22_ENV)
+    struct vcache *tvp;
 #endif
     int i;
-#if defined(AFS_OSF_ENV) || defined(AFS_LINUX22_ENV)
     if (!afs_maxvcount) {
-#if defined(AFS_LINUX22_ENV)
        afs_maxvcount = astatSize;      /* no particular limit on linux? */
-#elif defined(AFS_OSF30_ENV)
-       afs_maxvcount = max_vnodes / 2; /* limit ourselves to half the total */
-#else
-       afs_maxvcount = nvnode / 2;     /* limit ourselves to half the total */
-#endif
-       if (astatSize < afs_maxvcount) {
-           afs_maxvcount = astatSize;
-       }
     }
-#else /* AFS_OSF_ENV */
+#if !defined(AFS_LINUX22_ENV)
     freeVCList = NULL;
 #endif
 
     AFS_RWLOCK_INIT(&afs_xvcache, "afs_xvcache");
     LOCK_INIT(&afs_xvcb, "afs_xvcb");
 
-#if !defined(AFS_OSF_ENV) && !defined(AFS_LINUX22_ENV)
+#if !defined(AFS_LINUX22_ENV)
     /* Allocate and thread the struct vcache entries */
-    tvp = (struct vcache *)afs_osi_Alloc(astatSize * sizeof(struct vcache));
-    memset((char *)tvp, 0, sizeof(struct vcache) * astatSize);
+    tvp = afs_osi_Alloc(astatSize * sizeof(struct vcache));
+    osi_Assert(tvp != NULL);
+    memset(tvp, 0, sizeof(struct vcache) * astatSize);
 
     Initial_freeVCList = tvp;
     freeVCList = &(tvp[0]);
@@ -3247,9 +2976,9 @@ afs_vcacheInit(int astatSize)
        tvp[i].nextfree = &(tvp[i + 1]);
     }
     tvp[astatSize - 1].nextfree = NULL;
-#ifdef  KERNEL_HAVE_PIN
+# ifdef  KERNEL_HAVE_PIN
     pin((char *)tvp, astatSize * sizeof(struct vcache));       /* XXX */
-#endif
+# endif
 #endif
 
 #if defined(AFS_SGI_ENV)
@@ -3290,8 +3019,8 @@ shutdown_vcache(void)
      */
 
     {
-       register struct afs_q *tq, *uq = NULL;
-       register struct vcache *tvc;
+       struct afs_q *tq, *uq = NULL;
+       struct vcache *tvc;
        for (tq = VLRU.prev; tq != &VLRU; tq = uq) {
            tvc = QTOV(tq);
            uq = QPrev(tq);
@@ -3361,16 +3090,16 @@ shutdown_vcache(void)
     }
     afs_cbrSpace = 0;
 
-#if !defined(AFS_OSF_ENV) && !defined(AFS_LINUX22_ENV)
+#if !defined(AFS_LINUX22_ENV)
     afs_osi_Free(Initial_freeVCList, afs_cacheStats * sizeof(struct vcache));
-#endif
-#ifdef  KERNEL_HAVE_PIN
+
+# ifdef  KERNEL_HAVE_PIN
     unpin(Initial_freeVCList, afs_cacheStats * sizeof(struct vcache));
-#endif
+# endif
 
-#if !defined(AFS_OSF_ENV) && !defined(AFS_LINUX22_ENV)
     freeVCList = Initial_freeVCList = 0;
 #endif
+
     AFS_RWLOCK_INIT(&afs_xvcache, "afs_xvcache");
     LOCK_INIT(&afs_xvcb, "afs_xvcb");
     QInit(&VLRU);
@@ -3378,42 +3107,44 @@ shutdown_vcache(void)
        QInit(&afs_vhashTV[i]);
 }
 
-void afs_DisconGiveUpCallbacks(void) {
+void
+afs_DisconGiveUpCallbacks(void)
+{
     int i;
     struct vcache *tvc;
     int nq=0;
-            
+
     ObtainWriteLock(&afs_xvcache, 1002); /* XXX - should be a unique number */
-    
+
     /* Somehow, walk the set of vcaches, with each one coming out as tvc */
     for (i = 0; i < VCSIZE; i++) {
         for (tvc = afs_vhashT[i]; tvc; tvc = tvc->hnext) {
-            if ((tvc->f.states & CRO) == 0 && tvc->callback) {
-                afs_QueueVCB(tvc);
+            if (afs_QueueVCB(tvc)) {
                 tvc->callback = NULL;
                 nq++;
             }
         }
     }
-    /*printf("%d callbacks to be discarded. queued ... ", nq);*/
-    afs_FlushVCBs(0);
-    
+
     ReleaseWriteLock(&afs_xvcache);
-    /*printf("gone\n");*/
+
+    afs_FlushVCBs(2);
 }
 
 /*!
  *
  * Clear the Statd flag from all vcaches
  *
- * This function removes the Statd flag from all vcaches. It's used by 
+ * This function removes the Statd flag from all vcaches. It's used by
  * disconnected mode to tidy up during reconnection
  *
  */
-void afs_ClearAllStatdFlag(void) {
+void
+afs_ClearAllStatdFlag(void)
+{
     int i;
     struct vcache *tvc;
-   
+
     ObtainWriteLock(&afs_xvcache, 715);
 
     for (i = 0; i < VCSIZE; i++) {