revert-afs-indextimes-in-dcache-20050125
[openafs.git] / src / afs / afs_dcache.c
index ed9b20a..f231db2 100644 (file)
 #include <afsconfig.h>
 #include "afs/param.h"
 
-RCSID("$Header$");
+RCSID
+    ("$Header$");
 
-#include "afs/sysincludes.h" /*Standard vendor system headers*/
-#include "afsincludes.h" /*AFS-based standard headers*/
-#include "afs/afs_stats.h"  /* statistics */
+#include "afs/sysincludes.h"   /*Standard vendor system headers */
+#include "afsincludes.h"       /*AFS-based standard headers */
+#include "afs/afs_stats.h"     /* statistics */
 #include "afs/afs_cbqueue.h"
 #include "afs/afs_osidnlc.h"
 
@@ -30,45 +31,45 @@ static void afs_FreeDCache(struct dcache *);
 /*
  * --------------------- Exported definitions ---------------------
  */
-afs_lock_t afs_xdcache;                        /*Lock: alloc new disk cache entries*/
-afs_int32 afs_freeDCList;              /*Free list for disk cache entries*/
-afs_int32 afs_freeDCCount;             /*Count of elts in freeDCList*/
-afs_int32 afs_discardDCList;           /*Discarded disk cache entries*/
-afs_int32 afs_discardDCCount;          /*Count of elts in discardDCList*/
-struct dcache *afs_freeDSList;         /*Free list for disk slots */
-struct dcache *afs_Initial_freeDSList; /*Initial list for above*/
-ino_t cacheInode;                      /*Inode for CacheItems file*/
+afs_lock_t afs_xdcache;                /*Lock: alloc new disk cache entries */
+afs_int32 afs_freeDCList;      /*Free list for disk cache entries */
+afs_int32 afs_freeDCCount;     /*Count of elts in freeDCList */
+afs_int32 afs_discardDCList;   /*Discarded disk cache entries */
+afs_int32 afs_discardDCCount;  /*Count of elts in discardDCList */
+struct dcache *afs_freeDSList; /*Free list for disk slots */
+struct dcache *afs_Initial_freeDSList; /*Initial list for above */
+ino_t cacheInode;              /*Inode for CacheItems file */
 struct osi_file *afs_cacheInodep = 0;  /* file for CacheItems inode */
-struct afs_q afs_DLRU;                 /*dcache LRU*/
+struct afs_q afs_DLRU;         /*dcache LRU */
 afs_int32 afs_dhashsize = 1024;
-afs_int32 *afs_dvhashTbl;              /*Data cache hash table*/
-afs_int32 *afs_dchashTbl;              /*Data cache hash table*/
-afs_int32 *afs_dvnextTbl;               /*Dcache hash table links */
-afs_int32 *afs_dcnextTbl;               /*Dcache hash table links */
-struct dcache **afs_indexTable;                /*Pointers to dcache entries*/
-afs_hyper_t *afs_indexTimes;           /*Dcache entry Access times*/
-afs_int32 *afs_indexUnique;             /*dcache entry Fid.Unique */
-unsigned char *afs_indexFlags;         /*(only one) Is there data there?*/
-afs_hyper_t afs_indexCounter;          /*Fake time for marking index
-                                         entries*/
-afs_int32 afs_cacheFiles =0;           /*Size of afs_indexTable*/
-afs_int32 afs_cacheBlocks;             /*1K blocks in cache*/
-afs_int32 afs_cacheStats;              /*Stat entries in cache*/
-afs_int32 afs_blocksUsed;              /*Number of blocks in use*/
-afs_int32 afs_blocksDiscarded;         /*Blocks freed but not truncated */
-afs_int32 afs_fsfragsize = 1023;        /*Underlying Filesystem minimum unit 
+afs_int32 *afs_dvhashTbl;      /*Data cache hash table */
+afs_int32 *afs_dchashTbl;      /*Data cache hash table */
+afs_int32 *afs_dvnextTbl;      /*Dcache hash table links */
+afs_int32 *afs_dcnextTbl;      /*Dcache hash table links */
+struct dcache **afs_indexTable;        /*Pointers to dcache entries */
+afs_hyper_t *afs_indexTimes;   /*Dcache entry Access times */
+afs_int32 *afs_indexUnique;    /*dcache entry Fid.Unique */
+unsigned char *afs_indexFlags; /*(only one) Is there data there? */
+afs_hyper_t afs_indexCounter;  /*Fake time for marking index
+                                * entries */
+afs_int32 afs_cacheFiles = 0;  /*Size of afs_indexTable */
+afs_int32 afs_cacheBlocks;     /*1K blocks in cache */
+afs_int32 afs_cacheStats;      /*Stat entries in cache */
+afs_int32 afs_blocksUsed;      /*Number of blocks in use */
+afs_int32 afs_blocksDiscarded; /*Blocks freed but not truncated */
+afs_int32 afs_fsfragsize = 1023;       /*Underlying Filesystem minimum unit 
                                         *of disk allocation usually 1K
                                         *this value is (truefrag -1 ) to
                                         *save a bunch of subtracts... */
 #ifdef AFS_64BIT_CLIENT
 #ifdef AFS_VM_RDWR_ENV
-afs_size_t afs_vmMappingEnd;           /* for large files (>= 2GB) the VM
-                                        * mapping an 32bit addressing machines
-                                        * can only be used below the 2 GB
-                                        * line. From this point upwards we
-                                        * must do direct I/O into the cache
-                                        * files. The value should be on a
-                                        * chunk boundary. */
+afs_size_t afs_vmMappingEnd;   /* for large files (>= 2GB) the VM
+                                * mapping an 32bit addressing machines
+                                * can only be used below the 2 GB
+                                * line. From this point upwards we
+                                * must do direct I/O into the cache
+                                * files. The value should be on a
+                                * chunk boundary. */
 #endif /* AFS_VM_RDWR_ENV */
 #endif /* AFS_64BIT_CLIENT */
 
@@ -79,16 +80,11 @@ int afs_WaitForCacheDrain = 0;
 int afs_TruncateDaemonRunning = 0;
 int afs_CacheTooFull = 0;
 
-afs_int32  afs_dcentries;              /* In-memory dcache entries */
+afs_int32 afs_dcentries;       /* In-memory dcache entries */
 
 
 int dcacheDisabled = 0;
 
-extern struct dcache *afs_UFSGetDSlot();
-extern struct volume *afs_UFSGetVolSlot();
-extern int osi_UFSTruncate(), afs_osi_Read(), afs_osi_Write(), osi_UFSClose();
-extern int afs_UFSRead(), afs_UFSWrite();
-extern int afs_UFSHandleLink();
 static int afs_UFSCacheFetchProc(), afs_UFSCacheStoreProc();
 struct afs_cacheOps afs_UfsCacheOps = {
     osi_UFSOpen,
@@ -120,7 +116,7 @@ struct afs_cacheOps afs_MemCacheOps = {
     afs_MemHandleLink,
 };
 
-int cacheDiskType;                     /*Type of backing disk for cache*/
+int cacheDiskType;             /*Type of backing disk for cache */
 struct afs_cacheOps *afs_cacheType;
 
 
@@ -143,70 +139,70 @@ struct afs_cacheOps *afs_cacheType;
  *     Call this from close call when vnodeops is RCS unlocked.
  */
 
-void afs_StoreWarn(register afs_int32 acode, afs_int32 avolume, register afs_int32 aflags)
+void
+afs_StoreWarn(register afs_int32 acode, afs_int32 avolume,
+             register afs_int32 aflags)
 {
     static char problem_fmt[] =
        "afs: failed to store file in volume %d (%s)\n";
     static char problem_fmt_w_error[] =
        "afs: failed to store file in volume %d (error %d)\n";
     static char netproblems[] = "network problems";
-    static char partfull[]    = "partition full";
-    static char overquota[]   = "over quota";
+    static char partfull[] = "partition full";
+    static char overquota[] = "over quota";
 
     AFS_STATCNT(afs_StoreWarn);
     if (acode < 0) {
-        /*
+       /*
         * Network problems
         */
-        if (aflags & 1)
+       if (aflags & 1)
            afs_warn(problem_fmt, avolume, netproblems);
-        if (aflags & 2)
+       if (aflags & 2)
            afs_warnuser(problem_fmt, avolume, netproblems);
-    }
-    else
-       if (acode == ENOSPC) {
-           /*
-            * Partition full
-            */
-           if (aflags & 1)
-               afs_warn(problem_fmt, avolume, partfull);
-           if (aflags & 2)
-               afs_warnuser(problem_fmt, avolume, partfull);
-       }
-       else
-#ifndef        AFS_SUN5_ENV
-           /* EDQUOT doesn't exist on solaris and won't be sent by the server.
-            * Instead ENOSPC will be sent...
-            */
-           if (acode == EDQUOT) {
-               /*
-                * Quota exceeded
-                */
-               if (aflags & 1)
-                   afs_warn(problem_fmt, avolume, overquota);
-               if (aflags & 2)
-                   afs_warnuser(problem_fmt, avolume, overquota);
-           } else
+    } else if (acode == ENOSPC) {
+       /*
+        * Partition full
+        */
+       if (aflags & 1)
+           afs_warn(problem_fmt, avolume, partfull);
+       if (aflags & 2)
+           afs_warnuser(problem_fmt, avolume, partfull);
+    } else
+#ifdef EDQUOT
+       /* EDQUOT doesn't exist on solaris and won't be sent by the server.
+        * Instead ENOSPC will be sent...
+        */
+    if (acode == EDQUOT) {
+       /*
+        * Quota exceeded
+        */
+       if (aflags & 1)
+           afs_warn(problem_fmt, avolume, overquota);
+       if (aflags & 2)
+           afs_warnuser(problem_fmt, avolume, overquota);
+    } else
 #endif
-           {
-               /*
-                * Unknown error
-                */
-               if (aflags & 1)
-                   afs_warn(problem_fmt_w_error, avolume, acode);
-               if (aflags & 2)
-                   afs_warnuser(problem_fmt_w_error, avolume, acode);
-           }
-} /*afs_StoreWarn*/
+    {
+       /*
+        * Unknown error
+        */
+       if (aflags & 1)
+           afs_warn(problem_fmt_w_error, avolume, acode);
+       if (aflags & 2)
+           afs_warnuser(problem_fmt_w_error, avolume, acode);
+    }
+}                              /*afs_StoreWarn */
 
-void afs_MaybeWakeupTruncateDaemon(void)
+void
+afs_MaybeWakeupTruncateDaemon(void)
 {
     if (!afs_CacheTooFull && afs_CacheIsTooFull()) {
        afs_CacheTooFull = 1;
        if (!afs_TruncateDaemonRunning)
            afs_osi_Wakeup((int *)afs_CacheTruncateDaemon);
-    } else if (!afs_TruncateDaemonRunning &&
-               afs_blocksDiscarded > CM_MAXDISCARDEDCHUNKS) {
+    } else if (!afs_TruncateDaemonRunning
+              && afs_blocksDiscarded > CM_MAXDISCARDEDCHUNKS) {
        afs_osi_Wakeup((int *)afs_CacheTruncateDaemon);
     }
 }
@@ -223,27 +219,33 @@ static struct CTD_stats {
 } CTD_stats;
 
 u_int afs_min_cache = 0;
-void afs_CacheTruncateDaemon(void)
+void
+afs_CacheTruncateDaemon(void)
 {
     osi_timeval_t CTD_tmpTime;
     u_int counter;
     u_int cb_lowat;
-    u_int dc_hiwat = (100-CM_DCACHECOUNTFREEPCT+CM_DCACHEEXTRAPCT)*afs_cacheFiles/100;
-    afs_min_cache = (((10 * AFS_CHUNKSIZE(0)) + afs_fsfragsize) & ~afs_fsfragsize)>>10;
+    u_int dc_hiwat =
+       (100 - CM_DCACHECOUNTFREEPCT +
+        CM_DCACHEEXTRAPCT) * afs_cacheFiles / 100;
+    afs_min_cache =
+       (((10 * AFS_CHUNKSIZE(0)) + afs_fsfragsize) & ~afs_fsfragsize) >> 10;
 
     osi_GetuTime(&CTD_stats.CTD_afterSleep);
     afs_TruncateDaemonRunning = 1;
     while (1) {
-       cb_lowat =  ((CM_DCACHESPACEFREEPCT-CM_DCACHEEXTRAPCT)
-                    * afs_cacheBlocks) / 100;
-       MObtainWriteLock(&afs_xdcache,266);
+       cb_lowat = ((CM_DCACHESPACEFREEPCT - CM_DCACHEEXTRAPCT)
+                   * afs_cacheBlocks) / 100;
+       MObtainWriteLock(&afs_xdcache, 266);
        if (afs_CacheTooFull) {
-         int space_needed, slots_needed;
+           int space_needed, slots_needed;
            /* if we get woken up, we should try to clean something out */
            for (counter = 0; counter < 10; counter++) {
-               space_needed = afs_blocksUsed - afs_blocksDiscarded - cb_lowat;
-               slots_needed = dc_hiwat - afs_freeDCCount - afs_discardDCCount;
-               afs_GetDownD(slots_needed, &space_needed);      
+               space_needed =
+                   afs_blocksUsed - afs_blocksDiscarded - cb_lowat;
+               slots_needed =
+                   dc_hiwat - afs_freeDCCount - afs_discardDCCount;
+               afs_GetDownD(slots_needed, &space_needed);
                if ((space_needed <= 0) && (slots_needed <= 0)) {
                    break;
                }
@@ -262,26 +264,24 @@ void afs_CacheTruncateDaemon(void)
         * any cache files then give up the global lock so other
         * threads get a chance to run.
         */
-       if ((afs_termState!=AFSOP_STOP_TRUNCDAEMON) && afs_CacheTooFull &&
-           (!afs_blocksDiscarded || afs_WaitForCacheDrain)) {
-           afs_osi_Wait(100, 0, 0); /* 100 milliseconds */
+       if ((afs_termState != AFSOP_STOP_TRUNCDAEMON) && afs_CacheTooFull
+           && (!afs_blocksDiscarded || afs_WaitForCacheDrain)) {
+           afs_osi_Wait(100, 0, 0);    /* 100 milliseconds */
        }
 
        /*
         * This is where we free the discarded cache elements.
         */
-       while(afs_blocksDiscarded && !afs_WaitForCacheDrain && 
-             (afs_termState!=AFSOP_STOP_TRUNCDAEMON))
-       {
+       while (afs_blocksDiscarded && !afs_WaitForCacheDrain
+              && (afs_termState != AFSOP_STOP_TRUNCDAEMON)) {
            afs_FreeDiscardedDCache();
        }
 
        /* See if we need to continue to run. Someone may have
         * signalled us while we were executing.
         */
-       if (!afs_WaitForCacheDrain && !afs_CacheTooFull &&
-                       (afs_termState!=AFSOP_STOP_TRUNCDAEMON))
-       {
+       if (!afs_WaitForCacheDrain && !afs_CacheTooFull
+           && (afs_termState != AFSOP_STOP_TRUNCDAEMON)) {
            /* Collect statistics on truncate daemon. */
            CTD_stats.CTD_nSleeps++;
            osi_GetuTime(&CTD_stats.CTD_beforeSleep);
@@ -290,7 +290,7 @@ void afs_CacheTruncateDaemon(void)
            afs_stats_AddTo(CTD_stats.CTD_runTime, CTD_tmpTime);
 
            afs_TruncateDaemonRunning = 0;
-           afs_osi_Sleep((int *)afs_CacheTruncateDaemon);  
+           afs_osi_Sleep((int *)afs_CacheTruncateDaemon);
            afs_TruncateDaemonRunning = 1;
 
            osi_GetuTime(&CTD_stats.CTD_afterSleep);
@@ -327,16 +327,19 @@ void afs_CacheTruncateDaemon(void)
  *     anewsize : New size desired.
  */
 
-void afs_AdjustSize(register struct dcache *adc, register afs_int32 newSize)
+void
+afs_AdjustSize(register struct dcache *adc, register afs_int32 newSize)
 {
     register afs_int32 oldSize;
 
     AFS_STATCNT(afs_AdjustSize);
 
     adc->dflags |= DFEntryMod;
-    oldSize = ((adc->f.chunkBytes + afs_fsfragsize)^afs_fsfragsize)>>10;/* round up */
+    oldSize = ((adc->f.chunkBytes + afs_fsfragsize) ^ afs_fsfragsize) >> 10;   /* round up */
     adc->f.chunkBytes = newSize;
-    newSize = ((newSize + afs_fsfragsize)^afs_fsfragsize)>>10;/* round up */
+    if (!newSize)
+       adc->validPos = 0;
+    newSize = ((newSize + afs_fsfragsize) ^ afs_fsfragsize) >> 10;     /* round up */
     if (newSize > oldSize) {
        /* We're growing the file, wakeup the daemon */
        afs_MaybeWakeupTruncateDaemon();
@@ -372,7 +375,8 @@ void afs_AdjustSize(register struct dcache *adc, register afs_int32 newSize)
  */
 
 #define        MAXATONCE   16          /* max we can obtain at once */
-static void afs_GetDownD(int anumber, int *aneedSpace)
+static void
+afs_GetDownD(int anumber, int *aneedSpace)
 {
 
     struct dcache *tdc;
@@ -383,10 +387,10 @@ static void afs_GetDownD(int anumber, int *aneedSpace)
     register struct vcache *tvc;
     afs_uint32 victims[MAXATONCE];
     struct dcache *victimDCs[MAXATONCE];
-    afs_hyper_t victimTimes[MAXATONCE];/* youngest (largest LRU time) first */
-    afs_uint32 victimPtr;          /* next free item in victim arrays */
-    afs_hyper_t maxVictimTime;     /* youngest (largest LRU time) victim */
-    afs_uint32 maxVictimPtr;               /* where it is */
+    afs_hyper_t victimTimes[MAXATONCE];        /* youngest (largest LRU time) first */
+    afs_uint32 victimPtr;      /* next free item in victim arrays */
+    afs_hyper_t maxVictimTime; /* youngest (largest LRU time) victim */
+    afs_uint32 maxVictimPtr;   /* where it is */
     int discard;
 
     AFS_STATCNT(afs_GetDownD);
@@ -397,11 +401,12 @@ static void afs_GetDownD(int anumber, int *aneedSpace)
      * because we should try to free space even if anumber <=0 */
     if (!aneedSpace || *aneedSpace <= 0) {
        anumber -= afs_freeDCCount;
-       if (anumber <= 0) return;       /* enough already free */
+       if (anumber <= 0)
+           return;             /* enough already free */
     }
     /* bounds check parameter */
-    if (anumber        > MAXATONCE)
-       anumber = MAXATONCE;   /* all we can do */
+    if (anumber > MAXATONCE)
+       anumber = MAXATONCE;    /* all we can do */
 
     /*
      * The phase variable manages reclaims.  Set to 0, the first pass,
@@ -413,25 +418,26 @@ static void afs_GetDownD(int anumber, int *aneedSpace)
        /* turn off all flags */
        afs_indexFlags[i] &= ~IFFlag;
 
-    while (anumber > 0 || (aneedSpace && *aneedSpace >0)) {
+    while (anumber > 0 || (aneedSpace && *aneedSpace > 0)) {
        /* find oldest entries for reclamation */
        maxVictimPtr = victimPtr = 0;
        hzero(maxVictimTime);
        /* select victims from access time array */
        for (i = 0; i < afs_cacheFiles; i++) {
            if (afs_indexFlags[i] & (IFDataMod | IFFree | IFDiscarded)) {
-             /* skip if dirty or already free */
-             continue;
+               /* skip if dirty or already free */
+               continue;
            }
            tdc = afs_indexTable[i];
            if (tdc && (tdc->refCount != 0)) {
-             /* Referenced; can't use it! */
-             continue;
+               /* Referenced; can't use it! */
+               continue;
            }
            hset(vtime, afs_indexTimes[i]);
 
            /* if we've already looked at this one, skip it */
-           if (afs_indexFlags[i] & IFFlag) continue;
+           if (afs_indexFlags[i] & IFFlag)
+               continue;
 
            if (victimPtr < MAXATONCE) {
                /* if there's at least one free victim slot left */
@@ -442,33 +448,33 @@ static void afs_GetDownD(int anumber, int *aneedSpace)
                    maxVictimPtr = victimPtr;
                }
                victimPtr++;
-           }
-           else if (hcmp(vtime, maxVictimTime) < 0) {
+           } else if (hcmp(vtime, maxVictimTime) < 0) {
                /*
                 * We're older than youngest victim, so we replace at
                 * least one victim
                 */
                /* find youngest (largest LRU) victim */
-               j = maxVictimPtr;
-               if (j == victimPtr) osi_Panic("getdownd local");
+               j = maxVictimPtr;
+               if (j == victimPtr)
+                   osi_Panic("getdownd local");
                victims[j] = i;
                hset(victimTimes[j], vtime);
                /* recompute maxVictimTime */
                hset(maxVictimTime, vtime);
-               for(j = 0; j < victimPtr; j++)
+               for (j = 0; j < victimPtr; j++)
                    if (hcmp(maxVictimTime, victimTimes[j]) < 0) {
                        hset(maxVictimTime, victimTimes[j]);
                        maxVictimPtr = j;
                    }
            }
-       } /* big for loop */
+       }                       /* big for loop */
 
        /* now really reclaim the victims */
-       j = 0;  /* flag to track if we actually got any of the victims */
+       j = 0;                  /* flag to track if we actually got any of the victims */
        /* first, hold all the victims, since we're going to release the lock
         * during the truncate operation.
         */
-       for(i=0; i < victimPtr; i++) {
+       for (i = 0; i < victimPtr; i++) {
            tdc = afs_GetDSlot(victims[i], 0);
            /* We got tdc->tlock(R) here */
            if (tdc->refCount == 1)
@@ -476,9 +482,10 @@ static void afs_GetDownD(int anumber, int *aneedSpace)
            else
                victimDCs[i] = 0;
            ReleaseReadLock(&tdc->tlock);
-           if (!victimDCs[i]) afs_PutDCache(tdc);
+           if (!victimDCs[i])
+               afs_PutDCache(tdc);
        }
-       for(i = 0; i < victimPtr; i++) {
+       for (i = 0; i < victimPtr; i++) {
            /* q is first elt in dcache entry */
            tdc = victimDCs[i];
            /* now, since we're dropping the afs_xdcache lock below, we
@@ -489,8 +496,8 @@ static void afs_GetDownD(int anumber, int *aneedSpace)
             */
            if (tdc && tdc->refCount == 1) {
                unsigned char chunkFlags;
-               afs_size_t tchunkoffset;
-                afid = &tdc->f.fid;
+               afs_size_t tchunkoffset = 0;
+               afid = &tdc->f.fid;
                /* xdcache is lower than the xvcache lock */
                MReleaseWriteLock(&afs_xdcache);
                MObtainReadLock(&afs_xvcache);
@@ -498,18 +505,22 @@ static void afs_GetDownD(int anumber, int *aneedSpace)
                MReleaseReadLock(&afs_xvcache);
                MObtainWriteLock(&afs_xdcache, 527);
                skip = 0;
-               if (tdc->refCount > 1) skip = 1;
+               if (tdc->refCount > 1)
+                   skip = 1;
                if (tvc) {
                    tchunkoffset = AFS_CHUNKTOBASE(tdc->f.chunk);
                    chunkFlags = afs_indexFlags[tdc->index];
-                   if (phase == 0 && osi_Active(tvc)) skip = 1;
-                    if (phase > 0 && osi_Active(tvc) && (tvc->states & CDCLock)
-                                && (chunkFlags & IFAnyPages)) skip = 1;
-                   if (chunkFlags & IFDataMod) skip = 1;
+                   if (phase == 0 && osi_Active(tvc))
+                       skip = 1;
+                   if (phase > 0 && osi_Active(tvc)
+                       && (tvc->states & CDCLock)
+                       && (chunkFlags & IFAnyPages))
+                       skip = 1;
+                   if (chunkFlags & IFDataMod)
+                       skip = 1;
                    afs_Trace4(afs_iclSetp, CM_TRACE_GETDOWND,
                               ICL_TYPE_POINTER, tvc, ICL_TYPE_INT32, skip,
-                              ICL_TYPE_INT32, tdc->index,
-                              ICL_TYPE_OFFSET, 
+                              ICL_TYPE_INT32, tdc->index, ICL_TYPE_OFFSET,
                               ICL_HANDLE_OFFSET(tchunkoffset));
 
 #if    defined(AFS_SUN5_ENV)
@@ -536,10 +547,9 @@ static void afs_GetDownD(int anumber, int *aneedSpace)
                        /* One last recheck */
                        MObtainWriteLock(&afs_xdcache, 333);
                        chunkFlags = afs_indexFlags[tdc->index];
-                       if (tdc->refCount > 1
-                           || (chunkFlags & IFDataMod)
+                       if (tdc->refCount > 1 || (chunkFlags & IFDataMod)
                            || (osi_Active(tvc) && (tvc->states & CDCLock)
-                                && (chunkFlags & IFAnyPages))) {
+                               && (chunkFlags & IFAnyPages))) {
                            skip = 1;
                            MReleaseWriteLock(&afs_xdcache);
                            goto endputpage;
@@ -548,16 +558,18 @@ static void afs_GetDownD(int anumber, int *aneedSpace)
 
                        code = osi_VM_GetDownD(tvc, tdc);
 
-                       MObtainWriteLock(&afs_xdcache,269);
+                       MObtainWriteLock(&afs_xdcache, 269);
                        /* we actually removed all pages, clean and dirty */
                        if (code == 0) {
-                           afs_indexFlags[tdc->index] &= ~(IFDirtyPages| IFAnyPages);
+                           afs_indexFlags[tdc->index] &=
+                               ~(IFDirtyPages | IFAnyPages);
                        } else
                            skip = 1;
                        MReleaseWriteLock(&afs_xdcache);
-endputpage:
+                     endputpage:
                        MObtainWriteLock(&tvc->vlock, 544);
-                       if (--tvc->activeV == 0 && (tvc->vstates & VRevokeWait)) {
+                       if (--tvc->activeV == 0
+                           && (tvc->vstates & VRevokeWait)) {
                            tvc->vstates &= ~VRevokeWait;
                            afs_osi_Wakeup((char *)&tvc->vstates);
 
@@ -566,10 +578,10 @@ endputpage:
                            tvc->vstates &= ~VPageCleaning;
                            afs_osi_Wakeup((char *)&tvc->vstates);
                        }
-endmultipage:
+                     endmultipage:
                        MReleaseWriteLock(&tvc->vlock);
                    } else
-#endif /* AFS_SUN5_ENV */
+#endif /* AFS_SUN5_ENV */
                    {
                        MReleaseWriteLock(&afs_xdcache);
                    }
@@ -577,8 +589,10 @@ endmultipage:
                    AFS_FAST_RELE(tvc);
                    MObtainWriteLock(&afs_xdcache, 528);
                    if (afs_indexFlags[tdc->index] &
-                        (IFDataMod | IFDirtyPages | IFAnyPages)) skip = 1;
-                   if (tdc->refCount > 1) skip = 1;
+                       (IFDataMod | IFDirtyPages | IFAnyPages))
+                       skip = 1;
+                   if (tdc->refCount > 1)
+                       skip = 1;
                }
 #if    defined(AFS_SUN5_ENV)
                else {
@@ -589,7 +603,8 @@ endmultipage:
                     * Hold vcache lock to prevent vnode from being
                     * created while we're clearing IFDirtyPages.
                     */
-                   afs_indexFlags[tdc->index] &= ~(IFDirtyPages | IFAnyPages);
+                   afs_indexFlags[tdc->index] &=
+                       ~(IFDirtyPages | IFAnyPages);
                }
 #endif
                if (skip) {
@@ -597,21 +612,18 @@ endmultipage:
                    afs_indexFlags[tdc->index] |= IFFlag;
                    afs_Trace4(afs_iclSetp, CM_TRACE_GETDOWND,
                               ICL_TYPE_POINTER, tvc, ICL_TYPE_INT32, 2,
-                              ICL_TYPE_INT32, tdc->index,
-                              ICL_TYPE_OFFSET, 
+                              ICL_TYPE_INT32, tdc->index, ICL_TYPE_OFFSET,
                               ICL_HANDLE_OFFSET(tchunkoffset));
-               }
-               else {
+               } else {
                    /* flush this dude from the data cache and reclaim;
-                     * first, make sure no one will care that we damage
-                     * it, by removing it from all hash tables.  Then,
-                     * melt it down for parts.  Note that any concurrent
-                     * (new possibility!) calls to GetDownD won't touch
-                     * this guy because his reference count is > 0. */
+                    * first, make sure no one will care that we damage
+                    * it, by removing it from all hash tables.  Then,
+                    * melt it down for parts.  Note that any concurrent
+                    * (new possibility!) calls to GetDownD won't touch
+                    * this guy because his reference count is > 0. */
                    afs_Trace4(afs_iclSetp, CM_TRACE_GETDOWND,
                               ICL_TYPE_POINTER, tvc, ICL_TYPE_INT32, 3,
-                              ICL_TYPE_INT32, tdc->index,
-                              ICL_TYPE_OFFSET, 
+                              ICL_TYPE_INT32, tdc->index, ICL_TYPE_OFFSET,
                               ICL_HANDLE_OFFSET(tchunkoffset));
 #ifndef        AFS_DEC_ENV
                    AFS_STATCNT(afs_gget);
@@ -620,7 +632,8 @@ endmultipage:
                    if (tdc->f.chunkBytes != 0) {
                        discard = 1;
                        if (aneedSpace)
-                         *aneedSpace -= (tdc->f.chunkBytes + afs_fsfragsize) >> 10;
+                           *aneedSpace -=
+                               (tdc->f.chunkBytes + afs_fsfragsize) >> 10;
                    } else {
                        discard = 0;
                    }
@@ -635,7 +648,7 @@ endmultipage:
            }
            afs_PutDCache(tdc);
        }
-       
+
        if (phase == 0) {
            /* Phase is 0 and no one was found, so try phase 1 (ignore
             * osi_Active flag) */
@@ -645,15 +658,15 @@ endmultipage:
                    /* turn off all flags */
                    afs_indexFlags[i] &= ~IFFlag;
            }
-       }
-       else {
+       } else {
            /* found no one in phase 1, we're hosed */
-           if (victimPtr == 0) break;
+           if (victimPtr == 0)
+               break;
        }
-    } /* big while loop */
+    }                          /* big while loop */
     return;
 
-} /*afs_GetDownD*/
+}                              /*afs_GetDownD */
 
 
 /*
@@ -664,7 +677,8 @@ endmultipage:
  *
  * Locks: Must have the afs_xdcache lock write-locked to call this function.
  */
-int afs_HashOutDCache(struct dcache *adc)
+int
+afs_HashOutDCache(struct dcache *adc)
 {
     int i, us;
 
@@ -672,7 +686,7 @@ int afs_HashOutDCache(struct dcache *adc)
     AFS_STATCNT(afs_glink);
 #endif
     /* we know this guy's in the LRUQ.  We'll move dude into DCQ below */
-    DZap(&adc->f.inode);
+    DZap(adc);
     /* if this guy is in the hash table, pull him out */
     if (adc->f.fid.Fid.Volume != 0) {
        /* remove entry from first hash chains */
@@ -681,8 +695,7 @@ int afs_HashOutDCache(struct dcache *adc)
        if (us == adc->index) {
            /* first dude in the list */
            afs_dchashTbl[i] = afs_dcnextTbl[adc->index];
-       }
-       else {
+       } else {
            /* somewhere on the chain */
            while (us != NULLIDX) {
                if (afs_dcnextTbl[us] == adc->index) {
@@ -692,7 +705,8 @@ int afs_HashOutDCache(struct dcache *adc)
                }
                us = afs_dcnextTbl[us];
            }
-           if (us == NULLIDX) osi_Panic("dcache hc");
+           if (us == NULLIDX)
+               osi_Panic("dcache hc");
        }
        /* remove entry from *other* hash chain */
        i = DVHash(&adc->f.fid);
@@ -700,8 +714,7 @@ int afs_HashOutDCache(struct dcache *adc)
        if (us == adc->index) {
            /* first dude in the list */
            afs_dvhashTbl[i] = afs_dvnextTbl[adc->index];
-       }
-       else {
+       } else {
            /* somewhere on the chain */
            while (us != NULLIDX) {
                if (afs_dvnextTbl[us] == adc->index) {
@@ -711,7 +724,8 @@ int afs_HashOutDCache(struct dcache *adc)
                }
                us = afs_dvnextTbl[us];
            }
-           if (us == NULLIDX) osi_Panic("dcache hv");
+           if (us == NULLIDX)
+               osi_Panic("dcache hv");
        }
     }
 
@@ -726,7 +740,7 @@ int afs_HashOutDCache(struct dcache *adc)
 
     /* all done */
     return 0;
-} /*afs_HashOutDCache */
+}                              /*afs_HashOutDCache */
 
 
 /*
@@ -744,7 +758,8 @@ int afs_HashOutDCache(struct dcache *adc)
  *     (in write mode)
  */
 
-void afs_FlushDCache(register struct dcache *adc)
+void
+afs_FlushDCache(register struct dcache *adc)
 {
     AFS_STATCNT(afs_FlushDCache);
     /*
@@ -769,12 +784,12 @@ void afs_FlushDCache(register struct dcache *adc)
 
     if (afs_WaitForCacheDrain) {
        if (afs_blocksUsed <=
-           (CM_CACHESIZEDRAINEDPCT*afs_cacheBlocks)/100) {
+           (CM_CACHESIZEDRAINEDPCT * afs_cacheBlocks) / 100) {
            afs_WaitForCacheDrain = 0;
            afs_osi_Wakeup(&afs_WaitForCacheDrain);
        }
     }
-} /*afs_FlushDCache*/
+}                              /*afs_FlushDCache */
 
 
 /*
@@ -786,7 +801,8 @@ void afs_FlushDCache(register struct dcache *adc)
  *
  * Environment: called with afs_xdcache lock write-locked.
  */
-static void afs_FreeDCache(register struct dcache *adc)
+static void
+afs_FreeDCache(register struct dcache *adc)
 {
     /* Thread on free list, update free list count and mark entry as
      * freed in its indexFlags element.  Also, ensure DCache entry gets
@@ -801,7 +817,7 @@ static void afs_FreeDCache(register struct dcache *adc)
 
     if (afs_WaitForCacheDrain) {
        if ((afs_blocksUsed - afs_blocksDiscarded) <=
-           (CM_CACHESIZEDRAINEDPCT*afs_cacheBlocks)/100) {
+           (CM_CACHESIZEDRAINEDPCT * afs_cacheBlocks) / 100) {
            afs_WaitForCacheDrain = 0;
            afs_osi_Wakeup(&afs_WaitForCacheDrain);
        }
@@ -828,7 +844,8 @@ static void afs_FreeDCache(register struct dcache *adc)
  *     Must be called with afs_xdcache write-locked.
  */
 
-static void afs_DiscardDCache(register struct dcache *adc)
+static void
+afs_DiscardDCache(register struct dcache *adc)
 {
     register afs_int32 size;
 
@@ -836,7 +853,7 @@ static void afs_DiscardDCache(register struct dcache *adc)
 
     osi_Assert(adc->refCount == 1);
 
-    size = ((adc->f.chunkBytes + afs_fsfragsize)^afs_fsfragsize)>>10;/* round up */
+    size = ((adc->f.chunkBytes + afs_fsfragsize) ^ afs_fsfragsize) >> 10;      /* round up */
     afs_blocksDiscarded += size;
     afs_stats_cmperf.cacheBlocksDiscarded = afs_blocksDiscarded;
 
@@ -850,13 +867,13 @@ static void afs_DiscardDCache(register struct dcache *adc)
 
     if (afs_WaitForCacheDrain) {
        if ((afs_blocksUsed - afs_blocksDiscarded) <=
-           (CM_CACHESIZEDRAINEDPCT*afs_cacheBlocks)/100) {
+           (CM_CACHESIZEDRAINEDPCT * afs_cacheBlocks) / 100) {
            afs_WaitForCacheDrain = 0;
            afs_osi_Wakeup(&afs_WaitForCacheDrain);
        }
     }
 
-} /*afs_DiscardDCache*/
+}                              /*afs_DiscardDCache */
 
 /*
  * afs_FreeDiscardedDCache
@@ -864,15 +881,16 @@ static void afs_DiscardDCache(register struct dcache *adc)
  * Description:
  *     Free the next element on the list of discarded cache elements.
  */
-static void afs_FreeDiscardedDCache(void)
+static void
+afs_FreeDiscardedDCache(void)
 {
     register struct dcache *tdc;
-    register struct osi_file *tfile; 
+    register struct osi_file *tfile;
     register afs_int32 size;
 
     AFS_STATCNT(afs_FreeDiscardedDCache);
 
-    MObtainWriteLock(&afs_xdcache,510);
+    MObtainWriteLock(&afs_xdcache, 510);
     if (!afs_blocksDiscarded) {
        MReleaseWriteLock(&afs_xdcache);
        return;
@@ -888,7 +906,7 @@ static void afs_FreeDiscardedDCache(void)
     afs_discardDCList = afs_dvnextTbl[tdc->index];
     afs_dvnextTbl[tdc->index] = NULLIDX;
     afs_discardDCCount--;
-    size = ((tdc->f.chunkBytes + afs_fsfragsize)^afs_fsfragsize)>>10;/* round up */
+    size = ((tdc->f.chunkBytes + afs_fsfragsize) ^ afs_fsfragsize) >> 10;      /* round up */
     afs_blocksDiscarded -= size;
     afs_stats_cmperf.cacheBlocksDiscarded = afs_blocksDiscarded;
     /* We can lock because we just took it off the free list */
@@ -906,7 +924,7 @@ static void afs_FreeDiscardedDCache(void)
     /*
      * Free the element we just truncated
      */
-    MObtainWriteLock(&afs_xdcache,511);
+    MObtainWriteLock(&afs_xdcache, 511);
     afs_indexFlags[tdc->index] &= ~IFDiscarded;
     afs_FreeDCache(tdc);
     ReleaseWriteLock(&tdc->lock);
@@ -924,13 +942,15 @@ static void afs_FreeDiscardedDCache(void)
  * Parameters:
  *      None
  */
-int afs_MaybeFreeDiscardedDCache(void)
+int
+afs_MaybeFreeDiscardedDCache(void)
 {
 
     AFS_STATCNT(afs_MaybeFreeDiscardedDCache);
 
-    while (afs_blocksDiscarded &&
-          (afs_blocksUsed > (CM_WAITFORDRAINPCT*afs_cacheBlocks)/100)) {
+    while (afs_blocksDiscarded
+          && (afs_blocksUsed >
+              (CM_WAITFORDRAINPCT * afs_cacheBlocks) / 100)) {
        afs_FreeDiscardedDCache();
     }
     return 0;
@@ -948,7 +968,8 @@ int afs_MaybeFreeDiscardedDCache(void)
  * Environment:
  *     Must be called with afs_xdcache write-locked.
  */
-static void afs_GetDownDSlot(int anumber)
+static void
+afs_GetDownDSlot(int anumber)
 {
     struct afs_q *tq, *nq;
     struct dcache *tdc;
@@ -963,17 +984,18 @@ static void afs_GetDownDSlot(int anumber)
        osi_Panic("getdowndslot nolock");
 
     /* decrement anumber first for all dudes in free list */
-    for(tdc = afs_freeDSList; tdc; tdc = (struct dcache *)tdc->lruq.next)
+    for (tdc = afs_freeDSList; tdc; tdc = (struct dcache *)tdc->lruq.next)
        anumber--;
-    if (anumber        <= 0)
-       return;                         /* enough already free */
+    if (anumber <= 0)
+       return;                 /* enough already free */
 
-    for(cnt=0, tq = afs_DLRU.prev; tq != &afs_DLRU && anumber > 0;
-       tq = nq, cnt++) {
-       tdc = (struct dcache *) tq;     /* q is first elt in dcache entry */
-       nq = QPrev(tq); /* in case we remove it */
+    for (cnt = 0, tq = afs_DLRU.prev; tq != &afs_DLRU && anumber > 0;
+        tq = nq, cnt++) {
+       tdc = (struct dcache *)tq;      /* q is first elt in dcache entry */
+       nq = QPrev(tq);         /* in case we remove it */
        if (tdc->refCount == 0) {
-           if ((ix=tdc->index) == NULLIDX) osi_Panic("getdowndslot");
+           if ((ix = tdc->index) == NULLIDX)
+               osi_Panic("getdowndslot");
            /* pull the entry out of the lruq and put it on the free list */
            QRemove(&tdc->lruq);
 
@@ -983,7 +1005,7 @@ static void afs_GetDownDSlot(int anumber)
                /*
                 * ask proxy to do this for us - we don't have the stack space
                 */
-               while (tdc->dflags & DFEntryMod) {
+               while (tdc->dflags & DFEntryMod) {
                    int s;
                    AFS_GUNLOCK();
                    s = SPLOCK(afs_sgibklock);
@@ -1004,12 +1026,12 @@ static void afs_GetDownDSlot(int anumber)
 
            tdc->stamp = 0;
 #ifdef IHINT
-             if (tdc->ihint) {
-                 struct osi_file * f = (struct osi_file *)tdc->ihint;
-                 tdc->ihint = 0;
-                 afs_UFSClose(f);
-                nihints--;
-             }
+           if (tdc->ihint) {
+               struct osi_file *f = (struct osi_file *)tdc->ihint;
+               tdc->ihint = 0;
+               afs_UFSClose(f);
+               nihints--;
+           }
 #endif /* IHINT */
 
 
@@ -1017,12 +1039,12 @@ static void afs_GetDownDSlot(int anumber)
            afs_indexTable[ix] = NULL;
            afs_indexFlags[ix] &= ~IFEverUsed;
            tdc->index = NULLIDX;
-           tdc->lruq.next = (struct afs_q *) afs_freeDSList;
+           tdc->lruq.next = (struct afs_q *)afs_freeDSList;
            afs_freeDSList = tdc;
            anumber--;
        }
     }
-} /*afs_GetDownDSlot*/
+}                              /*afs_GetDownDSlot */
 
 
 /*
@@ -1040,7 +1062,8 @@ static void afs_GetDownDSlot(int anumber)
  * Environment:
  *     Nothing interesting.
  */
-int afs_RefDCache(struct dcache *adc)
+int
+afs_RefDCache(struct dcache *adc)
 {
     ObtainWriteLock(&adc->tlock, 627);
     if (adc->refCount < 0)
@@ -1063,7 +1086,8 @@ int afs_RefDCache(struct dcache *adc)
  * Environment:
  *     Nothing interesting.
  */
-int afs_PutDCache(register struct dcache *adc)
+int
+afs_PutDCache(register struct dcache *adc)
 {
     AFS_STATCNT(afs_PutDCache);
     ObtainWriteLock(&adc->tlock, 276);
@@ -1088,8 +1112,8 @@ int afs_PutDCache(register struct dcache *adc)
  * Environment:
  *     Both pvnLock and lock are write held.
  */
-void afs_TryToSmush(register struct vcache *avc, struct AFS_UCRED *acred, 
-       int sync)
+void
+afs_TryToSmush(register struct vcache *avc, struct AFS_UCRED *acred, int sync)
 {
     register struct dcache *tdc;
     register int index;
@@ -1097,11 +1121,11 @@ void afs_TryToSmush(register struct vcache *avc, struct AFS_UCRED *acred,
     AFS_STATCNT(afs_TryToSmush);
     afs_Trace2(afs_iclSetp, CM_TRACE_TRYTOSMUSH, ICL_TYPE_POINTER, avc,
               ICL_TYPE_OFFSET, ICL_HANDLE_OFFSET(avc->m.Length));
-    sync = 1;                          /* XX Temp testing XX*/
+    sync = 1;                  /* XX Temp testing XX */
 
 #if     defined(AFS_SUN5_ENV)
     ObtainWriteLock(&avc->vlock, 573);
-    avc->activeV++;     /* block new getpages */
+    avc->activeV++;            /* block new getpages */
     ReleaseWriteLock(&avc->vlock);
 #endif
 
@@ -1111,27 +1135,28 @@ void afs_TryToSmush(register struct vcache *avc, struct AFS_UCRED *acred,
     /*
      * Get the hash chain containing all dce's for this fid
      */
-    i =        DVHash(&avc->fid);
-    MObtainWriteLock(&afs_xdcache,277);
-    for(index = afs_dvhashTbl[i]; index != NULLIDX; index=i) {
-      i = afs_dvnextTbl[index];        /* next pointer this hash table */
-      if (afs_indexUnique[index] == avc->fid.Fid.Unique) {
-       int releaseTlock = 1;
-       tdc = afs_GetDSlot(index, NULL);
-       if (!FidCmp(&tdc->f.fid, &avc->fid)) {
-           if (sync) {
-               if ((afs_indexFlags[index] & IFDataMod) == 0 &&
-                   tdc->refCount == 1) {
-                   ReleaseReadLock(&tdc->tlock);
-                   releaseTlock = 0;
-                   afs_FlushDCache(tdc);
-               }
-           } else
-               afs_indexTable[index] = 0;
+    i = DVHash(&avc->fid);
+    MObtainWriteLock(&afs_xdcache, 277);
+    for (index = afs_dvhashTbl[i]; index != NULLIDX; index = i) {
+       i = afs_dvnextTbl[index];       /* next pointer this hash table */
+       if (afs_indexUnique[index] == avc->fid.Fid.Unique) {
+           int releaseTlock = 1;
+           tdc = afs_GetDSlot(index, NULL);
+           if (!FidCmp(&tdc->f.fid, &avc->fid)) {
+               if (sync) {
+                   if ((afs_indexFlags[index] & IFDataMod) == 0
+                       && tdc->refCount == 1) {
+                       ReleaseReadLock(&tdc->tlock);
+                       releaseTlock = 0;
+                       afs_FlushDCache(tdc);
+                   }
+               } else
+                   afs_indexTable[index] = 0;
+           }
+           if (releaseTlock)
+               ReleaseReadLock(&tdc->tlock);
+           afs_PutDCache(tdc);
        }
-       if (releaseTlock) ReleaseReadLock(&tdc->tlock);
-       afs_PutDCache(tdc);
-      }
     }
 #if    defined(AFS_SUN5_ENV)
     ObtainWriteLock(&avc->vlock, 545);
@@ -1169,11 +1194,12 @@ void afs_TryToSmush(register struct vcache *avc, struct AFS_UCRED *acred,
  *     The vcache entry is held upon entry.
  */
 
-struct dcache *afs_FindDCache(register struct vcache *avc, afs_size_t abyte)
+struct dcache *
+afs_FindDCache(register struct vcache *avc, afs_size_t abyte)
 {
     afs_int32 chunk;
     register afs_int32 i, index;
-    register struct dcache *tdc;
+    register struct dcache *tdc = NULL;
 
     AFS_STATCNT(afs_FindDCache);
     chunk = AFS_CHUNK(abyte);
@@ -1183,8 +1209,8 @@ struct dcache *afs_FindDCache(register struct vcache *avc, afs_size_t abyte)
      * after write-locking the dcache.
      */
     i = DCHash(&avc->fid, chunk);
-    MObtainWriteLock(&afs_xdcache,278);
-    for(index = afs_dchashTbl[i]; index != NULLIDX;) {
+    MObtainWriteLock(&afs_xdcache, 278);
+    for (index = afs_dchashTbl[i]; index != NULLIDX;) {
        if (afs_indexUnique[index] == avc->fid.Fid.Unique) {
            tdc = afs_GetDSlot(index, NULL);
            ReleaseReadLock(&tdc->tlock);
@@ -1200,11 +1226,10 @@ struct dcache *afs_FindDCache(register struct vcache *avc, afs_size_t abyte)
        hset(afs_indexTimes[tdc->index], afs_indexCounter);
        hadd32(afs_indexCounter, 1);
        return tdc;
-    }
-    else
+    } else
        return NULL;
 
-} /*afs_FindDCache*/
+}                              /*afs_FindDCache */
 
 
 /*
@@ -1229,9 +1254,11 @@ struct dcache *afs_FindDCache(register struct vcache *avc, afs_size_t abyte)
  * Environment:
  *     Nothing interesting.
  */
-static int afs_UFSCacheStoreProc(register struct rx_call *acall, 
-       struct osi_file *afile, register afs_int32 alen, struct vcache *avc, 
-       int *shouldWake, afs_size_t *abytesToXferP, afs_size_t *abytesXferredP)
+static int
+afs_UFSCacheStoreProc(register struct rx_call *acall, struct osi_file *afile,
+                     register afs_int32 alen, struct vcache *avc,
+                     int *shouldWake, afs_size_t * abytesToXferP,
+                     afs_size_t * abytesXferredP)
 {
     afs_int32 code, got;
     register char *tbuffer;
@@ -1244,19 +1271,18 @@ static int afs_UFSCacheStoreProc(register struct rx_call *acall,
      * In this case, alen is *always* the amount of data we'll be trying
      * to ship here.
      */
-    (*abytesToXferP)  = alen;
+    (*abytesToXferP) = alen;
     (*abytesXferredP) = 0;
 #endif /* AFS_NOSTATS */
 
     afs_Trace4(afs_iclSetp, CM_TRACE_STOREPROC, ICL_TYPE_POINTER, avc,
-                       ICL_TYPE_FID, &(avc->fid),
-                       ICL_TYPE_OFFSET, ICL_HANDLE_OFFSET(avc->m.Length), 
-                       ICL_TYPE_INT32, alen);
+              ICL_TYPE_FID, &(avc->fid), ICL_TYPE_OFFSET,
+              ICL_HANDLE_OFFSET(avc->m.Length), ICL_TYPE_INT32, alen);
     tbuffer = osi_AllocLargeSpace(AFS_LRALLOCSIZ);
     while (alen > 0) {
        tlen = (alen > AFS_LRALLOCSIZ ? AFS_LRALLOCSIZ : alen);
        got = afs_osi_Read(afile, -1, tbuffer, tlen);
-       if ((got < 0) 
+       if ((got < 0)
 #if defined(KERNEL_HAVE_UERROR)
            || (got != tlen && getuerror())
 #endif
@@ -1264,13 +1290,12 @@ static int afs_UFSCacheStoreProc(register struct rx_call *acall,
            osi_FreeLargeSpace(tbuffer);
            return EIO;
        }
-       afs_Trace2(afs_iclSetp, CM_TRACE_STOREPROC2, 
-                       ICL_TYPE_OFFSET, ICL_HANDLE_OFFSET(*tbuffer),
-                       ICL_TYPE_INT32, got);
+       afs_Trace2(afs_iclSetp, CM_TRACE_STOREPROC2, ICL_TYPE_OFFSET,
+                  ICL_HANDLE_OFFSET(*tbuffer), ICL_TYPE_INT32, got);
        RX_AFS_GUNLOCK();
-       code = rx_Write(acall, tbuffer, got);  /* writing 0 bytes will
-       * push a short packet.  Is that really what we want, just because the
-        * data didn't come back from the disk yet?  Let's try it and see. */
+       code = rx_Write(acall, tbuffer, got);   /* writing 0 bytes will
+                                                * push a short packet.  Is that really what we want, just because the
+                                                * data didn't come back from the disk yet?  Let's try it and see. */
        RX_AFS_GLOCK();
 #ifndef AFS_NOSTATS
        (*abytesXferredP) += code;
@@ -1285,18 +1310,17 @@ static int afs_UFSCacheStoreProc(register struct rx_call *acall,
         * to continue.
         */
        if (shouldWake && *shouldWake && (rx_GetRemoteStatus(acall) & 1)) {
-           *shouldWake = 0;  /* only do this once */
+           *shouldWake = 0;    /* only do this once */
            afs_wakeup(avc);
        }
     }
     afs_Trace4(afs_iclSetp, CM_TRACE_STOREPROC, ICL_TYPE_POINTER, avc,
-                       ICL_TYPE_FID, &(avc->fid),
-                       ICL_TYPE_OFFSET, ICL_HANDLE_OFFSET(avc->m.Length), 
-                       ICL_TYPE_INT32, alen);
+              ICL_TYPE_FID, &(avc->fid), ICL_TYPE_OFFSET,
+              ICL_HANDLE_OFFSET(avc->m.Length), ICL_TYPE_INT32, alen);
     osi_FreeLargeSpace(tbuffer);
     return 0;
 
-} /* afs_UFSCacheStoreProc*/
+}                              /* afs_UFSCacheStoreProc */
 
 
 /*
@@ -1323,10 +1347,11 @@ static int afs_UFSCacheStoreProc(register struct rx_call *acall,
  *     Nothing interesting.
  */
 
-static int afs_UFSCacheFetchProc(register struct rx_call *acall, 
-       struct osi_file *afile, afs_size_t abase, struct dcache *adc, 
-       struct vcache *avc, afs_size_t *abytesToXferP, 
-       afs_size_t *abytesXferredP, afs_int32 lengthFound)
+static int
+afs_UFSCacheFetchProc(register struct rx_call *acall, struct osi_file *afile,
+                     afs_size_t abase, struct dcache *adc,
+                     struct vcache *avc, afs_size_t * abytesToXferP,
+                     afs_size_t * abytesXferredP, afs_int32 lengthFound)
 {
     afs_int32 length;
     register afs_int32 code;
@@ -1336,10 +1361,10 @@ static int afs_UFSCacheFetchProc(register struct rx_call *acall,
 
     AFS_STATCNT(UFS_CacheFetchProc);
     osi_Assert(WriteLocked(&adc->lock));
-    afile->offset = 0; /* Each time start from the beginning */
+    afile->offset = 0;         /* Each time start from the beginning */
     length = lengthFound;
 #ifndef AFS_NOSTATS
-    (*abytesToXferP)  = 0;
+    (*abytesToXferP) = 0;
     (*abytesXferredP) = 0;
 #endif /* AFS_NOSTATS */
     tbuffer = osi_AllocLargeSpace(AFS_LRALLOCSIZ);
@@ -1347,14 +1372,14 @@ static int afs_UFSCacheFetchProc(register struct rx_call *acall,
     do {
        if (moredata) {
            RX_AFS_GUNLOCK();
-            code = rx_Read(acall, (char *)&length, sizeof(afs_int32));
+           code = rx_Read(acall, (char *)&length, sizeof(afs_int32));
            RX_AFS_GLOCK();
            length = ntohl(length);
-            if (code != sizeof(afs_int32)) {
-                osi_FreeLargeSpace(tbuffer);
-                code = rx_Error(acall);
-                return (code?code:-1);      /* try to return code, not -1 */
-            }    
+           if (code != sizeof(afs_int32)) {
+               osi_FreeLargeSpace(tbuffer);
+               code = rx_Error(acall);
+               return (code ? code : -1);      /* try to return code, not -1 */
+           }
        }
        /*
         * The fetch protocol is extended for the AFS/DFS translator
@@ -1373,28 +1398,28 @@ static int afs_UFSCacheFetchProc(register struct rx_call *acall,
        }
 #ifndef AFS_NOSTATS
        (*abytesToXferP) += length;
-#endif                         /* AFS_NOSTATS */
+#endif /* AFS_NOSTATS */
        while (length > 0) {
            tlen = (length > AFS_LRALLOCSIZ ? AFS_LRALLOCSIZ : length);
 #ifdef RX_KERNEL_TRACE
-           afs_Trace1(afs_iclSetp, CM_TRACE_TIMESTAMP,
-                       ICL_TYPE_STRING, "before rx_Read");
+           afs_Trace1(afs_iclSetp, CM_TRACE_TIMESTAMP, ICL_TYPE_STRING,
+                      "before rx_Read");
 #endif
            RX_AFS_GUNLOCK();
            code = rx_Read(acall, tbuffer, tlen);
            RX_AFS_GLOCK();
 #ifdef RX_KERNEL_TRACE
-           afs_Trace1(afs_iclSetp, CM_TRACE_TIMESTAMP,
-                       ICL_TYPE_STRING, "after rx_Read");
+           afs_Trace1(afs_iclSetp, CM_TRACE_TIMESTAMP, ICL_TYPE_STRING,
+                      "after rx_Read");
 #endif
 #ifndef AFS_NOSTATS
            (*abytesXferredP) += code;
-#endif                         /* AFS_NOSTATS */
+#endif /* AFS_NOSTATS */
            if (code != tlen) {
                osi_FreeLargeSpace(tbuffer);
                afs_Trace3(afs_iclSetp, CM_TRACE_FETCH64READ,
-                       ICL_TYPE_POINTER, avc, ICL_TYPE_INT32, code,
-                       ICL_TYPE_INT32, length);
+                          ICL_TYPE_POINTER, avc, ICL_TYPE_INT32, code,
+                          ICL_TYPE_INT32, length);
                return -34;
            }
            code = afs_osi_Write(afile, -1, tbuffer, tlen);
@@ -1406,17 +1431,16 @@ static int afs_UFSCacheFetchProc(register struct rx_call *acall,
            length -= tlen;
            adc->validPos = abase;
            if (afs_osi_Wakeup(&adc->validPos) == 0)
-               afs_Trace4(afs_iclSetp, CM_TRACE_DCACHEWAKE,
-                          ICL_TYPE_STRING, __FILE__,
-                          ICL_TYPE_INT32, __LINE__,
-                          ICL_TYPE_POINTER, adc,
-                          ICL_TYPE_INT32, adc->dflags);
+               afs_Trace4(afs_iclSetp, CM_TRACE_DCACHEWAKE, ICL_TYPE_STRING,
+                          __FILE__, ICL_TYPE_INT32, __LINE__,
+                          ICL_TYPE_POINTER, adc, ICL_TYPE_INT32,
+                          adc->dflags);
        }
     } while (moredata);
     osi_FreeLargeSpace(tbuffer);
     return 0;
 
-} /* afs_UFSCacheFetchProc*/
+}                              /* afs_UFSCacheFetchProc */
 
 /*
  * afs_GetDCache
@@ -1458,9 +1482,10 @@ struct tlocal1 {
  * Update the vnode-to-dcache hint if we can get the vnode lock
  * right away.  Assumes dcache entry is at least read-locked.
  */
-void updateV2DC(int lockVc, struct vcache *v, struct dcache *d, int src)
+void
+updateV2DC(int lockVc, struct vcache *v, struct dcache *d, int src)
 {
-    if (!lockVc || 0 == NBObtainWriteLock(&v->lock,src)) {
+    if (!lockVc || 0 == NBObtainWriteLock(&v->lock, src)) {
        if (hsame(v->m.DataVersion, d->f.versionNo) && v->callback) {
            v->quick.dc = d;
            v->quick.stamp = d->stamp = MakeStamp();
@@ -1469,16 +1494,18 @@ void updateV2DC(int lockVc, struct vcache *v, struct dcache *d, int src)
            v->quick.len = d->f.chunkBytes;
            v->h1.dchint = d;
        }
-       if (lockVc) ReleaseWriteLock(&v->lock);
+       if (lockVc)
+           ReleaseWriteLock(&v->lock);
     }
 }
 
 /* avc - Write-locked unless aflags & 1 */
-struct dcache *afs_GetDCache(register struct vcache *avc, afs_size_t abyte, 
-       register struct vrequest *areq, afs_size_t *aoffset, afs_size_t *alen, 
-       int aflags)
+struct dcache *
+afs_GetDCache(register struct vcache *avc, afs_size_t abyte,
+             register struct vrequest *areq, afs_size_t * aoffset,
+             afs_size_t * alen, int aflags)
 {
-    register afs_int32 i, code, code1=0, shortcut;
+    register afs_int32 i, code, code1 = 0, shortcut;
 #if    defined(AFS_AIX32_ENV) || defined(AFS_SGI_ENV)
     register afs_int32 adjustsize = 0;
 #endif
@@ -1486,20 +1513,20 @@ struct dcache *afs_GetDCache(register struct vcache *avc, afs_size_t abyte,
     afs_int32 index;
     afs_int32 us;
     afs_int32 chunk;
-    afs_size_t maxGoodLength;          /* amount of good data at server */
+    afs_size_t maxGoodLength;  /* amount of good data at server */
     struct rx_call *tcall;
     afs_size_t Position = 0;
 #ifdef AFS_64BIT_CLIENT
     afs_size_t tsize;
-    afs_size_t lengthFound;            /* as returned from server */
+    afs_size_t lengthFound;    /* as returned from server */
 #endif /* AFS_64BIT_CLIENT */
-    afs_int32 size, tlen;              /* size of segment to transfer */
+    afs_int32 size, tlen;      /* size of segment to transfer */
     struct tlocal1 *tsmall = 0;
     register struct dcache *tdc;
     register struct osi_file *file;
     register struct conn *tc;
     int downDCount = 0;
-    struct server *newCallback;
+    struct server *newCallback = NULL;
     char setNewCallback;
     char setVcacheStatus;
     char doVcacheUpdate;
@@ -1508,16 +1535,16 @@ struct dcache *afs_GetDCache(register struct vcache *avc, afs_size_t abyte,
     int doReallyAdjustSize = 0;
     int overWriteWholeChunk = 0;
 
-    XSTATS_DECLS
+    XSTATS_DECLS;
 #ifndef AFS_NOSTATS
     struct afs_stats_xferData *xferP;  /* Ptr to this op's xfer struct */
-    osi_timeval_t  xferStartTime,      /*FS xfer start time*/
-                   xferStopTime;       /*FS xfer stop time*/
-    afs_size_t bytesToXfer;                    /* # bytes to xfer*/
-    afs_size_t bytesXferred;                   /* # bytes actually xferred*/
-    struct afs_stats_AccessInfo *accP; /*Ptr to access record in stats*/
-    int fromReplica;                   /*Are we reading from a replica?*/
-    int numFetchLoops;                 /*# times around the fetch/analyze loop*/
+    osi_timeval_t xferStartTime,       /*FS xfer start time */
+      xferStopTime;            /*FS xfer stop time */
+    afs_size_t bytesToXfer;    /* # bytes to xfer */
+    afs_size_t bytesXferred;   /* # bytes actually xferred */
+    struct afs_stats_AccessInfo *accP; /*Ptr to access record in stats */
+    int fromReplica;           /*Are we reading from a replica? */
+    int numFetchLoops;         /*# times around the fetch/analyze loop */
 #endif /* AFS_NOSTATS */
 
     AFS_STATCNT(afs_GetDCache);
@@ -1531,15 +1558,14 @@ struct dcache *afs_GetDCache(register struct vcache *avc, afs_size_t abyte,
      * Determine the chunk number and offset within the chunk corresponding
      * to the desired byte.
      */
-    if (avc->fid.Fid.Vnode & 1) { /* if (vType(avc) == VDIR) */
+    if (avc->fid.Fid.Vnode & 1) {      /* if (vType(avc) == VDIR) */
        chunk = 0;
-    }
-    else {
+    } else {
        chunk = AFS_CHUNK(abyte);
     }
 
     /* come back to here if we waited for the cache to drain. */
-RetryGetDCache:
+  RetryGetDCache:
 
     setNewCallback = setVcacheStatus = 0;
 
@@ -1572,10 +1598,9 @@ RetryGetDCache:
        MObtainReadLock(&afs_xdcache);
        dcLocked = (0 == NBObtainSharedLock(&tdc->lock, 601));
 
-       if (dcLocked &&
-           (tdc->index != NULLIDX) && !FidCmp(&tdc->f.fid, &avc->fid) &&
-           chunk == tdc->f.chunk &&
-           !(afs_indexFlags[tdc->index] & (IFFree|IFDiscarded))) {
+       if (dcLocked && (tdc->index != NULLIDX)
+           && !FidCmp(&tdc->f.fid, &avc->fid) && chunk == tdc->f.chunk
+           && !(afs_indexFlags[tdc->index] & (IFFree | IFDiscarded))) {
            /* got the right one.  It might not be the right version, and it 
             * might be fetching, but it's the right dcache entry.
             */
@@ -1589,8 +1614,8 @@ RetryGetDCache:
            MReleaseReadLock(&afs_xdcache);
            shortcut = 1;
 
-           if (hsame(tdc->f.versionNo, avc->m.DataVersion) &&
-               !(tdc->dflags & DFFetching)) {
+           if (hsame(tdc->f.versionNo, avc->m.DataVersion)
+               && !(tdc->dflags & DFFetching)) {
 
                afs_stats_cmperf.dcacheHits++;
                MObtainWriteLock(&afs_xdcache, 559);
@@ -1606,7 +1631,8 @@ RetryGetDCache:
                goto done;
            }
        } else {
-           if (dcLocked) ReleaseSharedLock(&tdc->lock);
+           if (dcLocked)
+               ReleaseSharedLock(&tdc->lock);
            MReleaseReadLock(&afs_xdcache);
        }
 
@@ -1620,12 +1646,12 @@ RetryGetDCache:
      * tdc->lock(S) if tdc
      */
 
-    if (!tdc) {        /* If the hint wasn't the right dcache entry */
+    if (!tdc) {                        /* If the hint wasn't the right dcache entry */
        /*
         * Hash on the [fid, chunk] and get the corresponding dcache index
         * after write-locking the dcache.
         */
-RetryLookup:
+      RetryLookup:
 
        /* Locks held:
         * avc->lock(R) if setLocks && !slowPass
@@ -1636,9 +1662,9 @@ RetryLookup:
        /* check to make sure our space is fine */
        afs_MaybeWakeupTruncateDaemon();
 
-       MObtainWriteLock(&afs_xdcache,280);
+       MObtainWriteLock(&afs_xdcache, 280);
        us = NULLIDX;
-       for (index = afs_dchashTbl[i]; index != NULLIDX; ) {
+       for (index = afs_dchashTbl[i]; index != NULLIDX;) {
            if (afs_indexUnique[index] == avc->fid.Fid.Unique) {
                tdc = afs_GetDSlot(index, NULL);
                ReleaseReadLock(&tdc->tlock);
@@ -1650,14 +1676,14 @@ RetryLookup:
                 */
                if (!FidCmp(&tdc->f.fid, &avc->fid) && chunk == tdc->f.chunk) {
                    /* Move it up in the beginning of the list */
-                   if (afs_dchashTbl[i] != index)  {
+                   if (afs_dchashTbl[i] != index) {
                        afs_dcnextTbl[us] = afs_dcnextTbl[index];
                        afs_dcnextTbl[index] = afs_dchashTbl[i];
                        afs_dchashTbl[i] = index;
                    }
                    MReleaseWriteLock(&afs_xdcache);
                    ObtainSharedLock(&tdc->lock, 606);
-                   break;  /* leaving refCount high for caller */
+                   break;      /* leaving refCount high for caller */
                }
                afs_PutDCache(tdc);
                tdc = 0;
@@ -1682,14 +1708,17 @@ RetryLookup:
            /* Make sure there is a free dcache entry for us to use */
            if (afs_discardDCList == NULLIDX && afs_freeDCList == NULLIDX) {
                while (1) {
-                   if (!setLocks) avc->states |= CDCLock;
-                   afs_GetDownD(5, (int*)0);   /* just need slots */
-                   if (!setLocks) avc->states &= ~CDCLock;
-                   if (afs_discardDCList != NULLIDX || afs_freeDCList != NULLIDX)
+                   if (!setLocks)
+                       avc->states |= CDCLock;
+                   afs_GetDownD(5, (int *)0);  /* just need slots */
+                   if (!setLocks)
+                       avc->states &= ~CDCLock;
+                   if (afs_discardDCList != NULLIDX
+                       || afs_freeDCList != NULLIDX)
                        break;
                    /* If we can't get space for 5 mins we give up and panic */
                    if (++downDCount > 300)
-                       osi_Panic("getdcache"); 
+                       osi_Panic("getdcache");
                    MReleaseWriteLock(&afs_xdcache);
                    /*
                     * Locks held:
@@ -1701,8 +1730,8 @@ RetryLookup:
                }
            }
 
-           if (afs_discardDCList == NULLIDX ||
-               ((aflags & 2) && afs_freeDCList != NULLIDX)) {
+           if (afs_discardDCList == NULLIDX
+               || ((aflags & 2) && afs_freeDCList != NULLIDX)) {
 
                afs_indexFlags[afs_freeDCList] &= ~IFFree;
                tdc = afs_GetDSlot(afs_freeDCList, 0);
@@ -1719,7 +1748,9 @@ RetryLookup:
                ObtainWriteLock(&tdc->lock, 605);
                afs_discardDCList = afs_dvnextTbl[tdc->index];
                afs_discardDCCount--;
-               size = ((tdc->f.chunkBytes + afs_fsfragsize)^afs_fsfragsize)>>10;
+               size =
+                   ((tdc->f.chunkBytes +
+                     afs_fsfragsize) ^ afs_fsfragsize) >> 10;
                afs_blocksDiscarded -= size;
                afs_stats_cmperf.cacheBlocksDiscarded = afs_blocksDiscarded;
                if (aflags & 2) {
@@ -1745,11 +1776,12 @@ RetryLookup:
            afs_indexFlags[tdc->index] &= ~(IFDirtyPages | IFAnyPages);
            tdc->f.fid = avc->fid;
            afs_indexUnique[tdc->index] = tdc->f.fid.Fid.Unique;
-           hones(tdc->f.versionNo);        /* invalid value */
+           hones(tdc->f.versionNo);    /* invalid value */
            tdc->f.chunk = chunk;
            tdc->validPos = AFS_CHUNKTOBASE(chunk);
            /* XXX */
-           if (tdc->lruq.prev == &tdc->lruq) osi_Panic("lruq 1");
+           if (tdc->lruq.prev == &tdc->lruq)
+               osi_Panic("lruq 1");
 
            /*
             * Now add to the two hash chains - note that i is still set
@@ -1767,19 +1799,20 @@ RetryLookup:
            MReleaseWriteLock(&afs_xdcache);
            ConvertWToSLock(&tdc->lock);
        }
-    } /* vcache->dcache hint failed */
+    }
 
+
+    /* vcache->dcache hint failed */
     /*
      * Locks held:
      * avc->lock(R) if setLocks && !slowPass
      * avc->lock(W) if !setLocks || slowPass
      * tdc->lock(S)
      */
-
     afs_Trace4(afs_iclSetp, CM_TRACE_GETDCACHE2, ICL_TYPE_POINTER, avc,
-              ICL_TYPE_POINTER, tdc,
-              ICL_TYPE_INT32, hgetlo(tdc->f.versionNo),
-              ICL_TYPE_INT32, hgetlo(avc->m.DataVersion));
+              ICL_TYPE_POINTER, tdc, ICL_TYPE_INT32,
+              hgetlo(tdc->f.versionNo), ICL_TYPE_INT32,
+              hgetlo(avc->m.DataVersion));
     /*
      * Here we have the entry in tdc, with its refCount incremented.
      * Note: we don't use the S-lock on avc; it costs concurrency when
@@ -1794,16 +1827,15 @@ RetryLookup:
      * that this chunk's data hasn't been filled by another client.
      */
     size = AFS_CHUNKSIZE(abyte);
-    if (aflags & 4)    /* called from write */
+    if (aflags & 4)            /* called from write */
        tlen = *alen;
-    else               /* called from read */
+    else                       /* called from read */
        tlen = tdc->validPos - abyte;
     Position = AFS_CHUNKTOBASE(chunk);
-    afs_Trace4(afs_iclSetp, CM_TRACE_GETDCACHE3, 
-              ICL_TYPE_INT32, tlen,
-              ICL_TYPE_INT32, aflags,
-              ICL_TYPE_OFFSET, ICL_HANDLE_OFFSET(abyte),
-              ICL_TYPE_OFFSET, ICL_HANDLE_OFFSET(Position));
+    afs_Trace4(afs_iclSetp, CM_TRACE_GETDCACHE3, ICL_TYPE_INT32, tlen,
+              ICL_TYPE_INT32, aflags, ICL_TYPE_OFFSET,
+              ICL_HANDLE_OFFSET(abyte), ICL_TYPE_OFFSET,
+              ICL_HANDLE_OFFSET(Position));
     if ((aflags & 4) && (hiszero(avc->m.DataVersion)))
        doAdjustSize = 1;
     if ((aflags & 4) && (abyte == Position) && (tlen >= size))
@@ -1812,22 +1844,25 @@ RetryLookup:
 #if    defined(AFS_AIX32_ENV) || defined(AFS_SGI_ENV)
 #ifdef AFS_SGI_ENV
 #ifdef AFS_SGI64_ENV
-        if (doAdjustSize) adjustsize = NBPP;
+       if (doAdjustSize)
+           adjustsize = NBPP;
 #else /* AFS_SGI64_ENV */
-       if (doAdjustSize) adjustsize = 8192;
+       if (doAdjustSize)
+           adjustsize = 8192;
 #endif /* AFS_SGI64_ENV */
 #else /* AFS_SGI_ENV */
-        if (doAdjustSize) adjustsize = 4096;
+       if (doAdjustSize)
+           adjustsize = 4096;
 #endif /* AFS_SGI_ENV */
-        if (AFS_CHUNKTOBASE(chunk)+adjustsize >= avc->m.Length &&
+       if (AFS_CHUNKTOBASE(chunk) + adjustsize >= avc->m.Length &&
 #else /* defined(AFS_AIX32_ENV) || defined(AFS_SGI_ENV) */
 #if    defined(AFS_SUN_ENV)  || defined(AFS_OSF_ENV)
-        if ((doAdjustSize || (AFS_CHUNKTOBASE(chunk) >= avc->m.Length)) &&
+       if ((doAdjustSize || (AFS_CHUNKTOBASE(chunk) >= avc->m.Length)) &&
 #else
-        if (AFS_CHUNKTOBASE(chunk) >= avc->m.Length &&
+       if (AFS_CHUNKTOBASE(chunk) >= avc->m.Length &&
 #endif
 #endif /* defined(AFS_AIX32_ENV) || defined(AFS_SGI_ENV) */
-       !hsame(avc->m.DataVersion, tdc->f.versionNo))
+           !hsame(avc->m.DataVersion, tdc->f.versionNo))
            doReallyAdjustSize = 1;
 
        if (doReallyAdjustSize || overWriteWholeChunk) {
@@ -1872,7 +1907,7 @@ RetryLookup:
            else
                ReleaseReadLock(&avc->lock);
        }
-       return tdc;     /* check if we're done */
+       return tdc;             /* check if we're done */
     }
 
     /*
@@ -1945,24 +1980,25 @@ RetryLookup:
 
        /* Watch for standard race condition around osi_FlushText */
        if (hsame(avc->m.DataVersion, tdc->f.versionNo)) {
-           updateV2DC(setLocks, avc, tdc, 569); /* set hint */
-           afs_stats_cmperf.dcacheHits++;      
+           updateV2DC(setLocks, avc, tdc, 569);        /* set hint */
+           afs_stats_cmperf.dcacheHits++;
            ConvertWToSLock(&tdc->lock);
            goto done;
        }
 
        /* Sleep here when cache needs to be drained. */
-       if (setLocks && !slowPass &&
-           (afs_blocksUsed > (CM_WAITFORDRAINPCT*afs_cacheBlocks)/100)) {
+       if (setLocks && !slowPass
+           && (afs_blocksUsed >
+               (CM_WAITFORDRAINPCT * afs_cacheBlocks) / 100)) {
            /* Make sure truncate daemon is running */
            afs_MaybeWakeupTruncateDaemon();
            ObtainWriteLock(&tdc->tlock, 614);
-           tdc->refCount--; /* we'll re-obtain the dcache when we re-try. */
+           tdc->refCount--;    /* we'll re-obtain the dcache when we re-try. */
            ReleaseWriteLock(&tdc->tlock);
            ReleaseWriteLock(&tdc->lock);
            ReleaseReadLock(&avc->lock);
-           while ((afs_blocksUsed-afs_blocksDiscarded) >
-                  (CM_WAITFORDRAINPCT*afs_cacheBlocks)/100) {
+           while ((afs_blocksUsed - afs_blocksDiscarded) >
+                  (CM_WAITFORDRAINPCT * afs_cacheBlocks) / 100) {
                afs_WaitForCacheDrain = 1;
                afs_osi_Sleep(&afs_WaitForCacheDrain);
            }
@@ -1973,7 +2009,8 @@ RetryLookup:
 
        /* Do not fetch data beyond truncPos. */
        maxGoodLength = avc->m.Length;
-       if (avc->truncPos < maxGoodLength) maxGoodLength = avc->truncPos;
+       if (avc->truncPos < maxGoodLength)
+           maxGoodLength = avc->truncPos;
        Position = AFS_CHUNKBASE(abyte);
        if (vType(avc) == VDIR) {
            size = avc->m.Length;
@@ -1981,42 +2018,41 @@ RetryLookup:
                /* pre-reserve space for file */
                afs_AdjustSize(tdc, size);
            }
-           size = 999999999;       /* max size for transfer */
-       }
-       else {
+           size = 999999999;   /* max size for transfer */
+       } else {
            size = AFS_CHUNKSIZE(abyte);        /* expected max size */
            /* don't read past end of good data on server */
            if (Position + size > maxGoodLength)
                size = maxGoodLength - Position;
-           if (size < 0) size = 0; /* Handle random races */
+           if (size < 0)
+               size = 0;       /* Handle random races */
            if (size > tdc->f.chunkBytes) {
                /* pre-reserve space for file */
                afs_AdjustSize(tdc, size);      /* changes chunkBytes */
                /* max size for transfer still in size */
            }
        }
-       if (afs_mariner && !tdc->f.chunk) 
-           afs_MarinerLog("fetch$Fetching", avc); /* , Position, size, afs_indexCounter );*/
+       if (afs_mariner && !tdc->f.chunk)
+           afs_MarinerLog("fetch$Fetching", avc);      /* , Position, size, afs_indexCounter ); */
        /*
         * Right now, we only have one tool, and it's a hammer.  So, we
         * fetch the whole file.
         */
-       DZap(&tdc->f.inode);    /* pages in cache may be old */
+       DZap(tdc);      /* pages in cache may be old */
 #ifdef  IHINT
-        if (file = tdc->ihint) {
-         if (tdc->f.inode == file->inum ) 
-           usedihint++;
-         else {
-           tdc->ihint = 0;
-           afs_UFSClose(file);
-           file = 0;
-           nihints--;
-           file = osi_UFSOpen(tdc->f.inode);
-         }
-       }
-        else
+       if (file = tdc->ihint) {
+           if (tdc->f.inode == file->inum)
+               usedihint++;
+           else {
+               tdc->ihint = 0;
+               afs_UFSClose(file);
+               file = 0;
+               nihints--;
+               file = osi_UFSOpen(tdc->f.inode);
+           }
+       } else
 #endif /* IHINT */
-       file = afs_CFileOpen(tdc->f.inode);
+           file = afs_CFileOpen(tdc->f.inode);
        afs_RemoveVCB(&avc->fid);
        tdc->f.states |= DWriting;
        tdc->dflags |= DFFetching;
@@ -2024,13 +2060,13 @@ RetryLookup:
        if (tdc->mflags & DFFetchReq) {
            tdc->mflags &= ~DFFetchReq;
            if (afs_osi_Wakeup(&tdc->validPos) == 0)
-               afs_Trace4(afs_iclSetp, CM_TRACE_DCACHEWAKE,
-                      ICL_TYPE_STRING, __FILE__,
-                      ICL_TYPE_INT32, __LINE__,
-                      ICL_TYPE_POINTER, tdc,
-                      ICL_TYPE_INT32, tdc->dflags);
+               afs_Trace4(afs_iclSetp, CM_TRACE_DCACHEWAKE, ICL_TYPE_STRING,
+                          __FILE__, ICL_TYPE_INT32, __LINE__,
+                          ICL_TYPE_POINTER, tdc, ICL_TYPE_INT32,
+                          tdc->dflags);
        }
-       tsmall = (struct tlocal1 *) osi_AllocLargeSpace(sizeof(struct tlocal1));
+       tsmall =
+           (struct tlocal1 *)osi_AllocLargeSpace(sizeof(struct tlocal1));
        setVcacheStatus = 0;
 #ifndef AFS_NOSTATS
        /*
@@ -2047,11 +2083,11 @@ RetryLookup:
 #endif /* AFS_NOSTATS */
        /* this is a cache miss */
        afs_Trace4(afs_iclSetp, CM_TRACE_FETCHPROC, ICL_TYPE_POINTER, avc,
-                       ICL_TYPE_FID, &(avc->fid),
-                       ICL_TYPE_OFFSET, ICL_HANDLE_OFFSET(Position), 
-                       ICL_TYPE_INT32, size);
+                  ICL_TYPE_FID, &(avc->fid), ICL_TYPE_OFFSET,
+                  ICL_HANDLE_OFFSET(Position), ICL_TYPE_INT32, size);
 
-        if (size) afs_stats_cmperf.dcacheMisses++; 
+       if (size)
+           afs_stats_cmperf.dcacheMisses++;
        code = 0;
        /*
         * Dynamic root support:  fetch data from local memory.
@@ -2066,7 +2102,8 @@ RetryLookup:
            dynrootLen -= Position;
            if (size > dynrootLen)
                size = dynrootLen;
-           if (size < 0) size = 0;
+           if (size < 0)
+               size = 0;
            code = afs_CFileWrite(file, 0, dynrootDir, size);
            afs_PutDynroot();
 
@@ -2076,242 +2113,269 @@ RetryLookup:
                code = -1;
 
            tdc->validPos = Position + size;
-           afs_CFileTruncate(file, size); /* prune it */
+           afs_CFileTruncate(file, size);      /* prune it */
        } else
-       /*
-        * Not a dynamic vnode:  do the real fetch.
-        */
-       do {
            /*
-            * Locks held:
-            * avc->lock(R) if setLocks && !slowPass
-            * avc->lock(W) if !setLocks || slowPass
-            * tdc->lock(W)
+            * Not a dynamic vnode:  do the real fetch.
             */
+           do {
+               /*
+                * Locks held:
+                * avc->lock(R) if setLocks && !slowPass
+                * avc->lock(W) if !setLocks || slowPass
+                * tdc->lock(W)
+                */
 
-           tc = afs_Conn(&avc->fid, areq, SHARED_LOCK);
-           if (tc) {
-               afs_int32 length_hi, length, bytes;
+               tc = afs_Conn(&avc->fid, areq, SHARED_LOCK);
+               if (tc) {
+                   afs_int32 length_hi, length, bytes;
 #ifndef AFS_NOSTATS
-               numFetchLoops++;
-               if (fromReplica)
-                   (accP->numReplicasAccessed)++;
-               
+                   numFetchLoops++;
+                   if (fromReplica)
+                       (accP->numReplicasAccessed)++;
+
 #endif /* AFS_NOSTATS */
-               if (!setLocks || slowPass) {
-                   avc->callback = tc->srvr->server;
-               } else {
-                   newCallback = tc->srvr->server;
-                   setNewCallback = 1;
-               }
-               i = osi_Time();
-               RX_AFS_GUNLOCK();
-               tcall = rx_NewCall(tc->id);
-               RX_AFS_GLOCK();
+                   if (!setLocks || slowPass) {
+                       avc->callback = tc->srvr->server;
+                   } else {
+                       newCallback = tc->srvr->server;
+                       setNewCallback = 1;
+                   }
+                   i = osi_Time();
+                   RX_AFS_GUNLOCK();
+                   tcall = rx_NewCall(tc->id);
+                   RX_AFS_GLOCK();
 
-               XSTATS_START_TIME(AFS_STATS_FS_RPCIDX_FETCHDATA);
+                   XSTATS_START_TIME(AFS_STATS_FS_RPCIDX_FETCHDATA);
 #ifdef AFS_64BIT_CLIENT
-               length_hi = code = 0;
-               if (!afs_serverHasNo64Bit(tc)) {
-                   tsize = size;
-                   RX_AFS_GUNLOCK();
-                   code = StartRXAFS_FetchData64(tcall,
-                                           (struct AFSFid *) &avc->fid.Fid,
-                                           Position, tsize);
-                   if (code != 0) {
-                       RX_AFS_GLOCK();
-                       afs_Trace2(afs_iclSetp, CM_TRACE_FETCH64CODE,
-                                  ICL_TYPE_POINTER, avc, ICL_TYPE_INT32, code);
-                   } else {
-                       bytes = rx_Read(tcall, (char *)&length_hi, sizeof(afs_int32));
-                       RX_AFS_GLOCK();
-                       if (bytes == sizeof(afs_int32)) {
-                           length_hi = ntohl(length_hi);
-                       } else {
-                           length_hi = 0;
-                           code = rx_Error(tcall); 
+                   length_hi = code = 0;
+                   if (!afs_serverHasNo64Bit(tc)) {
+                       tsize = size;
+                       RX_AFS_GUNLOCK();
+                       code =
+                           StartRXAFS_FetchData64(tcall,
+                                                  (struct AFSFid *)&avc->fid.
+                                                  Fid, Position, tsize);
+                       if (code != 0) {
+                           RX_AFS_GLOCK();
+                           afs_Trace2(afs_iclSetp, CM_TRACE_FETCH64CODE,
+                                      ICL_TYPE_POINTER, avc, ICL_TYPE_INT32,
+                                      code);
+                       } else {
+                           bytes =
+                               rx_Read(tcall, (char *)&length_hi,
+                                       sizeof(afs_int32));
+                           RX_AFS_GLOCK();
+                           if (bytes == sizeof(afs_int32)) {
+                               length_hi = ntohl(length_hi);
+                           } else {
+                               length_hi = 0;
+                               code = rx_Error(tcall);
+                               RX_AFS_GUNLOCK();
+                               code1 = rx_EndCall(tcall, code);
+                               RX_AFS_GLOCK();
+                               tcall = (struct rx_call *)0;
+                           }
+                       }
+                   }
+                   if (code == RXGEN_OPCODE || afs_serverHasNo64Bit(tc)) {
+                       if (Position > 0x7FFFFFFF) {
+                           code = EFBIG;
+                       } else {
+                           afs_int32 pos;
+                           pos = Position;
                            RX_AFS_GUNLOCK();
-                           code1 = rx_EndCall(tcall, code);
-                            RX_AFS_GLOCK();
-                           tcall = (struct rx_call *) 0;
+                           if (!tcall)
+                               tcall = rx_NewCall(tc->id);
+                           code =
+                               StartRXAFS_FetchData(tcall, (struct AFSFid *)
+                                                    &avc->fid.Fid, pos,
+                                                    size);
+                           RX_AFS_GLOCK();
                        }
+                       afs_serverSetNo64Bit(tc);
                    }
-               }
-               if (code == RXGEN_OPCODE || afs_serverHasNo64Bit(tc)) {
-                   if (Position > 0x7FFFFFFF) {
-                       code = EFBIG;
-                   } else {
-                       afs_int32 pos;
-                       pos = Position;
+                   if (code == 0) {
                        RX_AFS_GUNLOCK();
-                       if (!tcall)
-                           tcall = rx_NewCall(tc->id);
-                       code = StartRXAFS_FetchData(tcall,
-                                   (struct AFSFid *) &avc->fid.Fid, pos, size);
-                       RX_AFS_GLOCK();
-                   }
-                   afs_serverSetNo64Bit(tc);
-               }
-               if (code == 0) {                
-                   RX_AFS_GUNLOCK();
-                   bytes = rx_Read(tcall, (char *)&length, sizeof(afs_int32));
-                    RX_AFS_GLOCK();
-                   if (bytes == sizeof(afs_int32)) {
-                       length = ntohl(length);
-                   } else {
-                       code = rx_Error(tcall); 
+                       bytes =
+                           rx_Read(tcall, (char *)&length,
+                                   sizeof(afs_int32));
+                       RX_AFS_GLOCK();
+                       if (bytes == sizeof(afs_int32)) {
+                           length = ntohl(length);
+                       } else {
+                           code = rx_Error(tcall);
+                       }
                    }
-               }
-               FillInt64(lengthFound, length_hi, length);
-               afs_Trace3(afs_iclSetp, CM_TRACE_FETCH64LENG,
-                       ICL_TYPE_POINTER, avc, ICL_TYPE_INT32, code,
-                       ICL_TYPE_OFFSET, ICL_HANDLE_OFFSET(lengthFound));
+                   FillInt64(lengthFound, length_hi, length);
+                   afs_Trace3(afs_iclSetp, CM_TRACE_FETCH64LENG,
+                              ICL_TYPE_POINTER, avc, ICL_TYPE_INT32, code,
+                              ICL_TYPE_OFFSET,
+                              ICL_HANDLE_OFFSET(lengthFound));
 #else /* AFS_64BIT_CLIENT */
-               RX_AFS_GUNLOCK();
-               code = StartRXAFS_FetchData(tcall,
-                                           (struct AFSFid *) &avc->fid.Fid,
-                                           Position, size);
-               RX_AFS_GLOCK();
-               if (code == 0) {                
-                   RX_AFS_GUNLOCK();
-                   bytes = rx_Read(tcall, (char *)&length, sizeof(afs_int32));
+                   RX_AFS_GUNLOCK();
+                   code =
+                       StartRXAFS_FetchData(tcall,
+                                            (struct AFSFid *)&avc->fid.Fid,
+                                            Position, size);
                    RX_AFS_GLOCK();
-                   if (bytes == sizeof(afs_int32)) {
-                       length = ntohl(length);
-                   } else {
-                       code = rx_Error(tcall); 
+                   if (code == 0) {
+                       RX_AFS_GUNLOCK();
+                       bytes =
+                           rx_Read(tcall, (char *)&length,
+                                   sizeof(afs_int32));
+                       RX_AFS_GLOCK();
+                       if (bytes == sizeof(afs_int32)) {
+                           length = ntohl(length);
+                       } else {
+                           code = rx_Error(tcall);
+                       }
                    }
-               }
 #endif /* AFS_64BIT_CLIENT */
-               if (code == 0) {
+                   if (code == 0) {
 
 #ifndef AFS_NOSTATS
-                   xferP = &(afs_stats_cmfullperf.rpc.fsXferTimes[AFS_STATS_FS_XFERIDX_FETCHDATA]);
-                   osi_GetuTime(&xferStartTime);
-
-                   code = afs_CacheFetchProc(tcall, file, 
-                               (afs_size_t) Position, tdc, avc,
-                               &bytesToXfer, &bytesXferred, length);
-
-                   osi_GetuTime(&xferStopTime);
-                   (xferP->numXfers)++;
-                   if (!code) {
-                       (xferP->numSuccesses)++;
-                       afs_stats_XferSumBytes[AFS_STATS_FS_XFERIDX_FETCHDATA] += bytesXferred;
-                       (xferP->sumBytes) += (afs_stats_XferSumBytes[AFS_STATS_FS_XFERIDX_FETCHDATA] >> 10);
-                       afs_stats_XferSumBytes[AFS_STATS_FS_XFERIDX_FETCHDATA] &= 0x3FF;
-                       if (bytesXferred < xferP->minBytes)
-                          xferP->minBytes = bytesXferred;
-                       if (bytesXferred > xferP->maxBytes)
-                          xferP->maxBytes = bytesXferred;
-                   
-                       /*
-                        * Tally the size of the object.  Note: we tally the actual size,
-                        * NOT the number of bytes that made it out over the wire.
-                        */
-                       if (bytesToXfer <= AFS_STATS_MAXBYTES_BUCKET0)
-                           (xferP->count[0])++;
-                       else
-                           if (bytesToXfer <= AFS_STATS_MAXBYTES_BUCKET1)
-                               (xferP->count[1])++;
-                       else
-                           if (bytesToXfer <= AFS_STATS_MAXBYTES_BUCKET2)
-                               (xferP->count[2])++;
-                       else
-                           if (bytesToXfer <= AFS_STATS_MAXBYTES_BUCKET3)
-                               (xferP->count[3])++;
-                       else
-                           if (bytesToXfer <= AFS_STATS_MAXBYTES_BUCKET4)
-                               (xferP->count[4])++;
-                       else
-                           if (bytesToXfer <= AFS_STATS_MAXBYTES_BUCKET5)
-                                (xferP->count[5])++;
-                       else
-                           if (bytesToXfer <= AFS_STATS_MAXBYTES_BUCKET6)
-                               (xferP->count[6])++;
-                       else
-                           if (bytesToXfer <= AFS_STATS_MAXBYTES_BUCKET7)
-                               (xferP->count[7])++;
-                       else
-                           (xferP->count[8])++;
-
-                       afs_stats_GetDiff(elapsedTime, xferStartTime, xferStopTime);
-                       afs_stats_AddTo((xferP->sumTime), elapsedTime);
-                        afs_stats_SquareAddTo((xferP->sqrTime), elapsedTime);
-                       if (afs_stats_TimeLessThan(elapsedTime, (xferP->minTime))) {
-                          afs_stats_TimeAssign((xferP->minTime), elapsedTime);
-                       }
-                       if (afs_stats_TimeGreaterThan(elapsedTime, (xferP->maxTime))) {
-                          afs_stats_TimeAssign((xferP->maxTime), elapsedTime);
-                       }
-                   }
+                       xferP =
+                           &(afs_stats_cmfullperf.rpc.
+                             fsXferTimes[AFS_STATS_FS_XFERIDX_FETCHDATA]);
+                       osi_GetuTime(&xferStartTime);
+
+                       code =
+                           afs_CacheFetchProc(tcall, file,
+                                              (afs_size_t) Position, tdc,
+                                              avc, &bytesToXfer,
+                                              &bytesXferred, length);
+
+                       osi_GetuTime(&xferStopTime);
+                       (xferP->numXfers)++;
+                       if (!code) {
+                           (xferP->numSuccesses)++;
+                           afs_stats_XferSumBytes
+                               [AFS_STATS_FS_XFERIDX_FETCHDATA] +=
+                               bytesXferred;
+                           (xferP->sumBytes) +=
+                               (afs_stats_XferSumBytes
+                                [AFS_STATS_FS_XFERIDX_FETCHDATA] >> 10);
+                           afs_stats_XferSumBytes
+                               [AFS_STATS_FS_XFERIDX_FETCHDATA] &= 0x3FF;
+                           if (bytesXferred < xferP->minBytes)
+                               xferP->minBytes = bytesXferred;
+                           if (bytesXferred > xferP->maxBytes)
+                               xferP->maxBytes = bytesXferred;
+
+                           /*
+                            * Tally the size of the object.  Note: we tally the actual size,
+                            * NOT the number of bytes that made it out over the wire.
+                            */
+                           if (bytesToXfer <= AFS_STATS_MAXBYTES_BUCKET0)
+                               (xferP->count[0])++;
+                           else if (bytesToXfer <=
+                                    AFS_STATS_MAXBYTES_BUCKET1)
+                               (xferP->count[1])++;
+                           else if (bytesToXfer <=
+                                    AFS_STATS_MAXBYTES_BUCKET2)
+                               (xferP->count[2])++;
+                           else if (bytesToXfer <=
+                                    AFS_STATS_MAXBYTES_BUCKET3)
+                               (xferP->count[3])++;
+                           else if (bytesToXfer <=
+                                    AFS_STATS_MAXBYTES_BUCKET4)
+                               (xferP->count[4])++;
+                           else if (bytesToXfer <=
+                                    AFS_STATS_MAXBYTES_BUCKET5)
+                               (xferP->count[5])++;
+                           else if (bytesToXfer <=
+                                    AFS_STATS_MAXBYTES_BUCKET6)
+                               (xferP->count[6])++;
+                           else if (bytesToXfer <=
+                                    AFS_STATS_MAXBYTES_BUCKET7)
+                               (xferP->count[7])++;
+                           else
+                               (xferP->count[8])++;
+
+                           afs_stats_GetDiff(elapsedTime, xferStartTime,
+                                             xferStopTime);
+                           afs_stats_AddTo((xferP->sumTime), elapsedTime);
+                           afs_stats_SquareAddTo((xferP->sqrTime),
+                                                 elapsedTime);
+                           if (afs_stats_TimeLessThan
+                               (elapsedTime, (xferP->minTime))) {
+                               afs_stats_TimeAssign((xferP->minTime),
+                                                    elapsedTime);
+                           }
+                           if (afs_stats_TimeGreaterThan
+                               (elapsedTime, (xferP->maxTime))) {
+                               afs_stats_TimeAssign((xferP->maxTime),
+                                                    elapsedTime);
+                           }
+                       }
 #else
-                   code = afs_CacheFetchProc(tcall, file, Position, tdc, avc, 0, 0, length);
+                       code =
+                           afs_CacheFetchProc(tcall, file, Position, tdc,
+                                              avc, 0, 0, length);
 #endif /* AFS_NOSTATS */
-               }
-               if (code == 0) {
+                   }
+                   if (code == 0) {
+                       RX_AFS_GUNLOCK();
+                       code =
+                           EndRXAFS_FetchData(tcall, &tsmall->OutStatus,
+                                              &tsmall->CallBack,
+                                              &tsmall->tsync);
+                       RX_AFS_GLOCK();
+                   }
+                   XSTATS_END_TIME;
                    RX_AFS_GUNLOCK();
-                   code = EndRXAFS_FetchData(tcall,
-                                             &tsmall->OutStatus,
-                                             &tsmall->CallBack,
-                                             &tsmall->tsync);
+                   if (tcall)
+                       code1 = rx_EndCall(tcall, code);
                    RX_AFS_GLOCK();
-               }
-               XSTATS_END_TIME;
-               RX_AFS_GUNLOCK();
-               if (tcall)
-                   code1 = rx_EndCall(tcall, code);
-               RX_AFS_GLOCK();
-           }
-           else {
-               code = -1;
-           }
-           if ( !code && code1 )
-               code = code1;
-
-           if (code == 0) {
-               /* callback could have been broken (or expired) in a race here, 
-                * but we return the data anyway.  It's as good as we knew about
-                * when we started. */
-               /* 
-                * validPos is updated by CacheFetchProc, and can only be 
-                * modifed under a dcache write lock, which we've blocked out 
-                */
-               size = tdc->validPos - Position;  /* actual segment size */
-               if (size < 0) size = 0;
-               afs_CFileTruncate(file, size); /* prune it */
-           }
-           else {
-               if (!setLocks || slowPass) {
-                   ObtainWriteLock(&afs_xcbhash, 453);
-                   afs_DequeueCallback(avc);
-                   avc->states &= ~(CStatd | CUnique);   
-                   avc->callback = NULL;
-                   ReleaseWriteLock(&afs_xcbhash);
-                   if (avc->fid.Fid.Vnode & 1 || (vType(avc) == VDIR))
-                       osi_dnlc_purgedp(avc);
                } else {
-                   /* Something lost.  Forget about performance, and go
-                    * back with a vcache write lock.
+                   code = -1;
+               }
+               if (!code && code1)
+                   code = code1;
+
+               if (code == 0) {
+                   /* callback could have been broken (or expired) in a race here, 
+                    * but we return the data anyway.  It's as good as we knew about
+                    * when we started. */
+                   /* 
+                    * validPos is updated by CacheFetchProc, and can only be 
+                    * modifed under a dcache write lock, which we've blocked out 
                     */
-                   afs_CFileTruncate(file, 0);
-                   afs_AdjustSize(tdc, 0);
-                   afs_CFileClose(file);
-                   osi_FreeLargeSpace(tsmall);
-                   tsmall = 0;
-                   ReleaseWriteLock(&tdc->lock);
-                   afs_PutDCache(tdc);
-                   tdc = 0;
-                   ReleaseReadLock(&avc->lock);
-                   slowPass = 1;
-                   goto RetryGetDCache;
+                   size = tdc->validPos - Position;    /* actual segment size */
+                   if (size < 0)
+                       size = 0;
+                   afs_CFileTruncate(file, size);      /* prune it */
+               } else {
+                   if (!setLocks || slowPass) {
+                       ObtainWriteLock(&afs_xcbhash, 453);
+                       afs_DequeueCallback(avc);
+                       avc->states &= ~(CStatd | CUnique);
+                       avc->callback = NULL;
+                       ReleaseWriteLock(&afs_xcbhash);
+                       if (avc->fid.Fid.Vnode & 1 || (vType(avc) == VDIR))
+                           osi_dnlc_purgedp(avc);
+                   } else {
+                       /* Something lost.  Forget about performance, and go
+                        * back with a vcache write lock.
+                        */
+                       afs_CFileTruncate(file, 0);
+                       afs_AdjustSize(tdc, 0);
+                       afs_CFileClose(file);
+                       osi_FreeLargeSpace(tsmall);
+                       tsmall = 0;
+                       ReleaseWriteLock(&tdc->lock);
+                       afs_PutDCache(tdc);
+                       tdc = 0;
+                       ReleaseReadLock(&avc->lock);
+                       slowPass = 1;
+                       goto RetryGetDCache;
+                   }
                }
-           }
 
-       } while
-           (afs_Analyze(tc, code, &avc->fid, areq,
-                        AFS_STATS_FS_RPCIDX_FETCHDATA,
-                        SHARED_LOCK, NULL));
+           } while (afs_Analyze
+                    (tc, code, &avc->fid, areq,
+                     AFS_STATS_FS_RPCIDX_FETCHDATA, SHARED_LOCK, NULL));
 
        /*
         * Locks held:
@@ -2335,12 +2399,11 @@ RetryLookup:
 
        tdc->dflags &= ~DFFetching;
        if (afs_osi_Wakeup(&tdc->validPos) == 0)
-           afs_Trace4(afs_iclSetp, CM_TRACE_DCACHEWAKE,
-                      ICL_TYPE_STRING, __FILE__,
-                      ICL_TYPE_INT32, __LINE__,
-                      ICL_TYPE_POINTER, tdc,
-                      ICL_TYPE_INT32, tdc->dflags);
-       if (avc->execsOrWriters == 0) tdc->f.states &= ~DWriting;
+           afs_Trace4(afs_iclSetp, CM_TRACE_DCACHEWAKE, ICL_TYPE_STRING,
+                      __FILE__, ICL_TYPE_INT32, __LINE__, ICL_TYPE_POINTER,
+                      tdc, ICL_TYPE_INT32, tdc->dflags);
+       if (avc->execsOrWriters == 0)
+           tdc->f.states &= ~DWriting;
 
        /* now, if code != 0, we have an error and should punt.
         * note that we have the vcache write lock, either because
@@ -2352,13 +2415,13 @@ RetryLookup:
            afs_CFileClose(file);
            ZapDCE(tdc);        /* sets DFEntryMod */
            if (vType(avc) == VDIR) {
-               DZap(&tdc->f.inode);
+               DZap(tdc);
            }
            ReleaseWriteLock(&tdc->lock);
            afs_PutDCache(tdc);
            ObtainWriteLock(&afs_xcbhash, 454);
            afs_DequeueCallback(avc);
-           avc->states &= ~( CStatd | CUnique );
+           avc->states &= ~(CStatd | CUnique);
            ReleaseWriteLock(&afs_xcbhash);
            if (avc->fid.Fid.Vnode & 1 || (vType(avc) == VDIR))
                osi_dnlc_purgedp(avc);
@@ -2368,12 +2431,12 @@ RetryLookup:
             */
            osi_Assert(!setLocks || slowPass);
            tdc = NULL;
-            goto done;
+           goto done;
        }
 
        /* otherwise we copy in the just-fetched info */
        afs_CFileClose(file);
-       afs_AdjustSize(tdc, size);  /* new size */
+       afs_AdjustSize(tdc, size);      /* new size */
        /*
         * Copy appropriate fields into vcache.  Status is
         * copied later where we selectively acquire the
@@ -2383,20 +2446,21 @@ RetryLookup:
            afs_ProcessFS(avc, &tsmall->OutStatus, areq);
        else
            setVcacheStatus = 1;
-       hset64(tdc->f.versionNo, tsmall->OutStatus.dataVersionHigh, tsmall->OutStatus.DataVersion);
+       hset64(tdc->f.versionNo, tsmall->OutStatus.dataVersionHigh,
+              tsmall->OutStatus.DataVersion);
        tdc->dflags |= DFEntryMod;
        afs_indexFlags[tdc->index] |= IFEverUsed;
        ConvertWToSLock(&tdc->lock);
-    } /*Data version numbers don't match*/
+    } /*Data version numbers don't match */
     else {
        /*
         * Data version numbers match.
         */
        afs_stats_cmperf.dcacheHits++;
-    }  /*Data version numbers match*/
+    }                          /*Data version numbers match */
 
-    updateV2DC(setLocks, avc, tdc, 335); /* set hint */
-done:
+    updateV2DC(setLocks, avc, tdc, 335);       /* set hint */
+  done:
     /*
      * Locks held:
      * avc->lock(R) if setLocks && !slowPass
@@ -2458,7 +2522,7 @@ done:
 
        if (tsmall) {
            hset64(statusDV, tsmall->OutStatus.dataVersionHigh,
-                            tsmall->OutStatus.DataVersion);
+                  tsmall->OutStatus.DataVersion);
 
            if (setVcacheStatus && avc->m.Length != tsmall->OutStatus.Length)
                doVcacheUpdate = 1;
@@ -2485,15 +2549,19 @@ done:
 
     /* Check if we need to perform any last-minute fixes with a write-lock */
     if (!setLocks || doVcacheUpdate) {
-       if (setNewCallback) avc->callback = newCallback;
-       if (tsmall && setVcacheStatus) afs_ProcessFS(avc, &tsmall->OutStatus, areq);
-       if (setLocks) ReleaseWriteLock(&avc->lock);
+       if (setNewCallback)
+           avc->callback = newCallback;
+       if (tsmall && setVcacheStatus)
+           afs_ProcessFS(avc, &tsmall->OutStatus, areq);
+       if (setLocks)
+           ReleaseWriteLock(&avc->lock);
     }
 
-    if (tsmall) osi_FreeLargeSpace(tsmall);
+    if (tsmall)
+       osi_FreeLargeSpace(tsmall);
 
     return tdc;
-} /*afs_GetDCache*/
+}                              /*afs_GetDCache */
 
 
 /*
@@ -2509,12 +2577,13 @@ done:
  * Environment:
  *     The afs_xdcache is write-locked through this whole affair.
  */
-void afs_WriteThroughDSlots(void)
+void
+afs_WriteThroughDSlots(void)
 {
     register struct dcache *tdc;
-    register afs_int32 i, touchedit=0;
-    struct dcache **ents;
-    int entmax, entcount;
+    register afs_int32 i, touchedit = 0;
+
+    struct afs_q DirtyQ, *tq;
 
     AFS_STATCNT(afs_WriteThroughDSlots);
 
@@ -2523,11 +2592,9 @@ void afs_WriteThroughDSlots(void)
      * holding afs_xdcache.  So we enter xdcache, get a reference
      * for every dcache entry, and exit xdcache.
      */
-    MObtainWriteLock(&afs_xdcache,283);
-    entmax = afs_cacheFiles;
-    ents = afs_osi_Alloc(entmax * sizeof(struct dcache *));
-    entcount = 0;
-    for(i = 0; i < afs_cacheFiles; i++) {
+    MObtainWriteLock(&afs_xdcache, 283);
+    QInit(&DirtyQ);
+    for (i = 0; i < afs_cacheFiles; i++) {
        tdc = afs_indexTable[i];
 
        /* Grab tlock in case the existing refcount isn't zero */
@@ -2536,7 +2603,7 @@ void afs_WriteThroughDSlots(void)
            tdc->refCount++;
            ReleaseWriteLock(&tdc->tlock);
 
-           ents[entcount++] = tdc;
+           QAdd(&DirtyQ, &tdc->dirty);
        }
     }
     MReleaseWriteLock(&afs_xdcache);
@@ -2547,9 +2614,11 @@ void afs_WriteThroughDSlots(void)
      * afs_cacheInodep, and flush it.  Don't forget to put back
      * the refcounts.
      */
-    for (i = 0; i < entcount; i++) {
-       tdc = ents[i];
 
+#define DQTODC(q)      ((struct dcache *)(((char *) (q)) - sizeof(struct afs_q)))
+
+    for (tq = DirtyQ.prev; tq != &DirtyQ; tq = QPrev(tq)) {
+       tdc = DQTODC(tq);
        if (tdc->dflags & DFEntryMod) {
            int wrLock;
 
@@ -2563,12 +2632,12 @@ void afs_WriteThroughDSlots(void)
                MReleaseWriteLock(&afs_xdcache);
                touchedit = 1;
            }
-           if (wrLock) ReleaseWriteLock(&tdc->lock);
+           if (wrLock)
+               ReleaseWriteLock(&tdc->lock);
        }
 
        afs_PutDCache(tdc);
     }
-    afs_osi_Free(ents, entmax * sizeof(struct dcache *));
 
     MObtainWriteLock(&afs_xdcache, 617);
     if (!touchedit && (cacheDiskType != AFS_FCACHE_TYPE_MEM)) {
@@ -2602,17 +2671,20 @@ void afs_WriteThroughDSlots(void)
  *     Must be called with afs_xdcache write-locked.
  */
 
-struct dcache *afs_MemGetDSlot(register afs_int32 aslot, register struct dcache *tmpdc)
+struct dcache *
+afs_MemGetDSlot(register afs_int32 aslot, register struct dcache *tmpdc)
 {
     register struct dcache *tdc;
     int existing = 0;
 
     AFS_STATCNT(afs_MemGetDSlot);
-    if (CheckLock(&afs_xdcache) != -1) osi_Panic("getdslot nolock");
-    if (aslot < 0 || aslot >= afs_cacheFiles) osi_Panic("getdslot slot");
+    if (CheckLock(&afs_xdcache) != -1)
+       osi_Panic("getdslot nolock");
+    if (aslot < 0 || aslot >= afs_cacheFiles)
+       osi_Panic("getdslot slot");
     tdc = afs_indexTable[aslot];
     if (tdc) {
-       QRemove(&tdc->lruq);        /* move to queue head */
+       QRemove(&tdc->lruq);    /* move to queue head */
        QAdd(&afs_DLRU, &tdc->lruq);
        /* We're holding afs_xdcache, but get tlock in case refCount != 0 */
        ObtainWriteLock(&tdc->tlock, 624);
@@ -2621,29 +2693,30 @@ struct dcache *afs_MemGetDSlot(register afs_int32 aslot, register struct dcache
        return tdc;
     }
     if (tmpdc == NULL) {
-       if (!afs_freeDSList) afs_GetDownDSlot(4); 
+       if (!afs_freeDSList)
+           afs_GetDownDSlot(4);
        if (!afs_freeDSList) {
            /* none free, making one is better than a panic */
            afs_stats_cmperf.dcacheXAllocs++;   /* count in case we have a leak */
-           tdc = (struct dcache *) afs_osi_Alloc(sizeof (struct dcache));
+           tdc = (struct dcache *)afs_osi_Alloc(sizeof(struct dcache));
 #ifdef KERNEL_HAVE_PIN
            pin((char *)tdc, sizeof(struct dcache));    /* XXX */
 #endif
        } else {
            tdc = afs_freeDSList;
-           afs_freeDSList = (struct dcache *) tdc->lruq.next;
+           afs_freeDSList = (struct dcache *)tdc->lruq.next;
            existing = 1;
        }
        tdc->dflags = 0;        /* up-to-date, not in free q */
        tdc->mflags = 0;
        QAdd(&afs_DLRU, &tdc->lruq);
-       if (tdc->lruq.prev == &tdc->lruq) osi_Panic("lruq 3");
-    }
-    else {
+       if (tdc->lruq.prev == &tdc->lruq)
+           osi_Panic("lruq 3");
+    } else {
        tdc = tmpdc;
        tdc->f.states = 0;
     }
-    
+
     /* initialize entry */
     tdc->f.fid.Cell = 0;
     tdc->f.fid.Fid.Volume = 0;
@@ -2665,12 +2738,12 @@ struct dcache *afs_MemGetDSlot(register afs_int32 aslot, register struct dcache
     RWLOCK_INIT(&tdc->tlock, "dcache tlock");
     RWLOCK_INIT(&tdc->mflock, "dcache flock");
     ObtainReadLock(&tdc->tlock);
-    
+
     if (tmpdc == NULL)
        afs_indexTable[aslot] = tdc;
     return tdc;
 
-} /*afs_MemGetDSlot*/
+}                              /*afs_MemGetDSlot */
 
 unsigned int last_error = 0, lasterrtime = 0;
 
@@ -2688,7 +2761,8 @@ unsigned int last_error = 0, lasterrtime = 0;
  * Environment:
  *     afs_xdcache lock write-locked.
  */
-struct dcache *afs_UFSGetDSlot(register afs_int32 aslot, register struct dcache *tmpdc)
+struct dcache *
+afs_UFSGetDSlot(register afs_int32 aslot, register struct dcache *tmpdc)
 {
     register afs_int32 code;
     register struct dcache *tdc;
@@ -2696,11 +2770,13 @@ struct dcache *afs_UFSGetDSlot(register afs_int32 aslot, register struct dcache
     int entryok;
 
     AFS_STATCNT(afs_UFSGetDSlot);
-    if (CheckLock(&afs_xdcache) != -1) osi_Panic("getdslot nolock");
-    if (aslot < 0 || aslot >= afs_cacheFiles) osi_Panic("getdslot slot");
+    if (CheckLock(&afs_xdcache) != -1)
+       osi_Panic("getdslot nolock");
+    if (aslot < 0 || aslot >= afs_cacheFiles)
+       osi_Panic("getdslot slot");
     tdc = afs_indexTable[aslot];
     if (tdc) {
-       QRemove(&tdc->lruq);        /* move to queue head */
+       QRemove(&tdc->lruq);    /* move to queue head */
        QAdd(&afs_DLRU, &tdc->lruq);
        /* Grab tlock in case refCount != 0 */
        ObtainWriteLock(&tdc->tlock, 625);
@@ -2714,36 +2790,39 @@ struct dcache *afs_UFSGetDSlot(register afs_int32 aslot, register struct dcache
      * we have to allocate one.
      */
     if (tmpdc == NULL) {
-       if (!afs_freeDSList) afs_GetDownDSlot(4);
+       if (!afs_freeDSList)
+           afs_GetDownDSlot(4);
        if (!afs_freeDSList) {
            /* none free, making one is better than a panic */
            afs_stats_cmperf.dcacheXAllocs++;   /* count in case we have a leak */
-           tdc = (struct dcache *) afs_osi_Alloc(sizeof (struct dcache));
+           tdc = (struct dcache *)afs_osi_Alloc(sizeof(struct dcache));
 #ifdef KERNEL_HAVE_PIN
            pin((char *)tdc, sizeof(struct dcache));    /* XXX */
 #endif
        } else {
            tdc = afs_freeDSList;
-           afs_freeDSList = (struct dcache *) tdc->lruq.next;
+           afs_freeDSList = (struct dcache *)tdc->lruq.next;
            existing = 1;
        }
        tdc->dflags = 0;        /* up-to-date, not in free q */
        tdc->mflags = 0;
        QAdd(&afs_DLRU, &tdc->lruq);
-       if (tdc->lruq.prev == &tdc->lruq) osi_Panic("lruq 3");
-    }
-    else {
+       if (tdc->lruq.prev == &tdc->lruq)
+           osi_Panic("lruq 3");
+    } else {
        tdc = tmpdc;
        tdc->f.states = 0;
        tdc->ihint = 0;
     }
 
     /*
-      * Seek to the aslot'th entry and read it in.
-      */
-    code = afs_osi_Read(afs_cacheInodep, sizeof(struct fcache) * aslot +
-                                        sizeof(struct afs_fheader),
-                       (char *)(&tdc->f), sizeof(struct fcache));
+     * Seek to the aslot'th entry and read it in.
+     */
+    code =
+       afs_osi_Read(afs_cacheInodep,
+                    sizeof(struct fcache) * aslot +
+                    sizeof(struct afs_fheader), (char *)(&tdc->f),
+                    sizeof(struct fcache));
     entryok = 1;
     if (code != sizeof(struct fcache))
        entryok = 0;
@@ -2764,6 +2843,10 @@ struct dcache *afs_UFSGetDSlot(register afs_int32 aslot, register struct dcache
     }
     tdc->refCount = 1;
     tdc->index = aslot;
+    if (tdc->f.chunk >= 0)
+       tdc->validPos = AFS_CHUNKTOBASE(tdc->f.chunk) + tdc->f.chunkBytes;
+    else
+       tdc->validPos = 0;
 
     if (existing) {
        osi_Assert(0 == NBObtainWriteLock(&tdc->lock, 674));
@@ -2784,7 +2867,7 @@ struct dcache *afs_UFSGetDSlot(register afs_int32 aslot, register struct dcache
        afs_indexTable[aslot] = tdc;
     return tdc;
 
-}  /*afs_UFSGetDSlot*/
+}                              /*afs_UFSGetDSlot */
 
 
 
@@ -2805,11 +2888,13 @@ struct dcache *afs_UFSGetDSlot(register afs_int32 aslot, register struct dcache
  *     The reference count is not changed.
  */
 
-int afs_WriteDCache(register struct dcache *adc, int atime)
+int
+afs_WriteDCache(register struct dcache *adc, int atime)
 {
     register afs_int32 code;
 
-    if (cacheDiskType == AFS_FCACHE_TYPE_MEM) return 0;
+    if (cacheDiskType == AFS_FCACHE_TYPE_MEM)
+       return 0;
     AFS_STATCNT(afs_WriteDCache);
     if (atime)
        adc->f.modTime = osi_Time();
@@ -2817,10 +2902,13 @@ int afs_WriteDCache(register struct dcache *adc, int atime)
      * Seek to the right dcache slot and write the in-memory image out to disk.
      */
     afs_cellname_write();
-    code = afs_osi_Write(afs_cacheInodep, sizeof(struct fcache) * adc->index +
-                                         sizeof(struct afs_fheader), 
-                        (char *)(&adc->f), sizeof(struct fcache));
-    if (code != sizeof(struct fcache)) return EIO;
+    code =
+       afs_osi_Write(afs_cacheInodep,
+                     sizeof(struct fcache) * adc->index +
+                     sizeof(struct afs_fheader), (char *)(&adc->f),
+                     sizeof(struct fcache));
+    if (code != sizeof(struct fcache))
+       return EIO;
     return 0;
 }
 
@@ -2840,7 +2928,8 @@ int afs_WriteDCache(register struct dcache *adc, int atime)
  *     Nothing interesting.
  */
 
-int afs_wakeup(register struct vcache *avc)
+int
+afs_wakeup(register struct vcache *avc)
 {
     register int i;
     register struct brequest *tb;
@@ -2848,7 +2937,7 @@ int afs_wakeup(register struct vcache *avc)
     AFS_STATCNT(afs_wakeup);
     for (i = 0; i < NBRS; i++, tb++) {
        /* if request is valid and for this file, we've found it */
-       if (tb->refCount > 0 && avc == tb->vnode) {
+       if (tb->refCount > 0 && avc == tb->vc) {
 
            /*
             * If CSafeStore is on, then we don't awaken the guy
@@ -2890,7 +2979,8 @@ int afs_wakeup(register struct vcache *avc)
  *     This function is called only during initialization.
  */
 
-int afs_InitCacheFile(char *afile, ino_t ainode)
+int
+afs_InitCacheFile(char *afile, ino_t ainode)
 {
     register afs_int32 code;
 #if defined(AFS_LINUX22_ENV)
@@ -2906,9 +2996,10 @@ int afs_InitCacheFile(char *afile, ino_t ainode)
 
     AFS_STATCNT(afs_InitCacheFile);
     index = afs_stats_cmperf.cacheNumEntries;
-    if (index >= afs_cacheFiles) return EINVAL;
+    if (index >= afs_cacheFiles)
+       return EINVAL;
 
-    MObtainWriteLock(&afs_xdcache,282);
+    MObtainWriteLock(&afs_xdcache, 282);
     tdc = afs_GetDSlot(index, NULL);
     ReleaseReadLock(&tdc->tlock);
     MReleaseWriteLock(&afs_xdcache);
@@ -2916,11 +3007,7 @@ int afs_InitCacheFile(char *afile, ino_t ainode)
     ObtainWriteLock(&tdc->lock, 621);
     MObtainWriteLock(&afs_xdcache, 622);
     if (afile) {
-       code = gop_lookupname(afile,
-                             AFS_UIOSYS,
-                             0,
-                             NULL,
-                             &filevp);
+       code = gop_lookupname(afile, AFS_UIOSYS, 0, NULL, &filevp);
        if (code) {
            ReleaseWriteLock(&afs_xdcache);
            ReleaseWriteLock(&tdc->lock);
@@ -2940,24 +3027,25 @@ int afs_InitCacheFile(char *afile, ino_t ainode)
 #ifdef AFS_DEC_ENV
        grele(filevp);
 #else
-       AFS_RELE((struct vnode *)filevp);
+       AFS_RELE(filevp);
 #endif
 #endif /* AFS_LINUX22_ENV */
-    }
-    else {
+    } else {
        tdc->f.inode = ainode;
     }
     fileIsBad = 0;
-    if ((tdc->f.states & DWriting) ||
-       tdc->f.fid.Fid.Volume == 0) fileIsBad = 1;
+    if ((tdc->f.states & DWriting) || tdc->f.fid.Fid.Volume == 0)
+       fileIsBad = 1;
     tfile = osi_UFSOpen(tdc->f.inode);
     code = afs_osi_Stat(tfile, &tstat);
-    if (code) osi_Panic("initcachefile stat");
+    if (code)
+       osi_Panic("initcachefile stat");
 
     /*
      * If file size doesn't match the cache info file, it's probably bad.
      */
-    if (tdc->f.chunkBytes != tstat.size) fileIsBad = 1;
+    if (tdc->f.chunkBytes != tstat.size)
+       fileIsBad = 1;
     tdc->f.chunkBytes = 0;
 
     /*
@@ -2966,14 +3054,17 @@ int afs_InitCacheFile(char *afile, ino_t ainode)
      * the cache info file may be incorrectly identified, and so slot
      * may be bad.
      */
-    if (cacheInfoModTime < tstat.mtime + 120) fileIsBad = 1;
-    if (cacheInfoModTime < tdc->f.modTime + 120) fileIsBad = 1;
+    if (cacheInfoModTime < tstat.mtime + 120)
+       fileIsBad = 1;
+    if (cacheInfoModTime < tdc->f.modTime + 120)
+       fileIsBad = 1;
     /* In case write through is behind, make sure cache items entry is
      * at least as new as the chunk.
      */
-    if (tdc->f.modTime < tstat.mtime) fileIsBad = 1;
+    if (tdc->f.modTime < tstat.mtime)
+       fileIsBad = 1;
     if (fileIsBad) {
-       tdc->f.fid.Fid.Volume = 0;  /* not in the hash table */
+       tdc->f.fid.Fid.Volume = 0;      /* not in the hash table */
        if (tstat.size != 0)
            osi_UFSTruncate(tfile, 0);
        /* put entry in free cache slot list */
@@ -2982,19 +3073,18 @@ int afs_InitCacheFile(char *afile, ino_t ainode)
        afs_freeDCCount++;
        afs_indexFlags[index] |= IFFree;
        afs_indexUnique[index] = 0;
-    }
-    else {
+    } else {
        /*
         * We must put this entry in the appropriate hash tables.
         * Note that i is still set from the above DCHash call
         */
        code = DCHash(&tdc->f.fid, tdc->f.chunk);
-       afs_dcnextTbl[tdc->index] = afs_dchashTbl[code];        
+       afs_dcnextTbl[tdc->index] = afs_dchashTbl[code];
        afs_dchashTbl[code] = tdc->index;
        code = DVHash(&tdc->f.fid);
        afs_dvnextTbl[tdc->index] = afs_dvhashTbl[code];
        afs_dvhashTbl[code] = tdc->index;
-       afs_AdjustSize(tdc, tstat.size);    /* adjust to new size */
+       afs_AdjustSize(tdc, tstat.size);        /* adjust to new size */
        if (tstat.size > 0)
            /* has nontrivial amt of data */
            afs_indexFlags[index] |= IFEverUsed;
@@ -3008,7 +3098,7 @@ int afs_InitCacheFile(char *afile, ino_t ainode)
            hset32(afs_indexCounter, tstat.atime);
        }
        afs_indexUnique[index] = tdc->f.fid.Fid.Unique;
-    } /*File is not bad*/
+    }                          /*File is not bad */
 
     osi_UFSClose(tfile);
     tdc->f.states &= ~DWriting;
@@ -3036,8 +3126,8 @@ int afs_InitCacheFile(char *afile, ino_t ainode)
  * Description:
  *     Initialize dcache related variables.
  */
-void afs_dcacheInit(int afiles, int ablocks, int aDentries, int achunk,
-                   int aflags)
+void
+afs_dcacheInit(int afiles, int ablocks, int aDentries, int achunk, int aflags)
 {
     register struct dcache *tdp;
     int i;
@@ -3050,92 +3140,97 @@ void afs_dcacheInit(int afiles, int ablocks, int aDentries, int achunk,
     hzero(afs_indexCounter);
 
     LOCK_INIT(&afs_xdcache, "afs_xdcache");
-   
+
     /*
      * Set chunk size
      */
     if (achunk) {
        if (achunk < 0 || achunk > 30)
-           achunk = 13;                /* Use default */
+           achunk = 13;        /* Use default */
        AFS_SETCHUNKSIZE(achunk);
     }
-    
-    if(!aDentries)
+
+    if (!aDentries)
        aDentries = DDSIZE;
-    
-    if(aflags & AFSCALL_INIT_MEMCACHE) {
+
+    if (aflags & AFSCALL_INIT_MEMCACHE) {
        /*
         * Use a memory cache instead of a disk cache
         */
        cacheDiskType = AFS_FCACHE_TYPE_MEM;
        afs_cacheType = &afs_MemCacheOps;
-       afiles = (afiles < aDentries) ? afiles : aDentries; /* min */
-       ablocks = afiles * (AFS_FIRSTCSIZE/1024);
+       afiles = (afiles < aDentries) ? afiles : aDentries;     /* min */
+       ablocks = afiles * (AFS_FIRSTCSIZE / 1024);
        /* ablocks is reported in 1K blocks */
-       code = afs_InitMemCache(afiles * AFS_FIRSTCSIZE, AFS_FIRSTCSIZE, aflags);
+       code = afs_InitMemCache(afiles, AFS_FIRSTCSIZE, aflags);
        if (code != 0) {
            printf("afsd: memory cache too large for available memory.\n");
            printf("afsd: AFS files cannot be accessed.\n\n");
            dcacheDisabled = 1;
            afiles = ablocks = 0;
-       }
-       else
-           printf("Memory cache: Allocating %d dcache entries...", aDentries);
+       } else
+           printf("Memory cache: Allocating %d dcache entries...",
+                  aDentries);
     } else {
        cacheDiskType = AFS_FCACHE_TYPE_UFS;
        afs_cacheType = &afs_UfsCacheOps;
     }
 
-    if (aDentries > 512) 
+    if (aDentries > 512)
        afs_dhashsize = 2048;
     /* initialize hash tables */
-    afs_dvhashTbl = (afs_int32 *) afs_osi_Alloc(afs_dhashsize * sizeof(afs_int32));
-    afs_dchashTbl = (afs_int32 *) afs_osi_Alloc(afs_dhashsize * sizeof(afs_int32));
-    for(i=0;i< afs_dhashsize;i++) {
+    afs_dvhashTbl =
+       (afs_int32 *) afs_osi_Alloc(afs_dhashsize * sizeof(afs_int32));
+    afs_dchashTbl =
+       (afs_int32 *) afs_osi_Alloc(afs_dhashsize * sizeof(afs_int32));
+    for (i = 0; i < afs_dhashsize; i++) {
        afs_dvhashTbl[i] = NULLIDX;
        afs_dchashTbl[i] = NULLIDX;
     }
     afs_dvnextTbl = (afs_int32 *) afs_osi_Alloc(afiles * sizeof(afs_int32));
     afs_dcnextTbl = (afs_int32 *) afs_osi_Alloc(afiles * sizeof(afs_int32));
-    for(i=0;i< afiles;i++) {
+    for (i = 0; i < afiles; i++) {
        afs_dvnextTbl[i] = NULLIDX;
        afs_dcnextTbl[i] = NULLIDX;
     }
-    
+
     /* Allocate and zero the pointer array to the dcache entries */
     afs_indexTable = (struct dcache **)
        afs_osi_Alloc(sizeof(struct dcache *) * afiles);
     memset((char *)afs_indexTable, 0, sizeof(struct dcache *) * afiles);
-    afs_indexTimes = (afs_hyper_t *) afs_osi_Alloc(afiles * sizeof(afs_hyper_t));
+    afs_indexTimes =
+       (afs_hyper_t *) afs_osi_Alloc(afiles * sizeof(afs_hyper_t));
     memset((char *)afs_indexTimes, 0, afiles * sizeof(afs_hyper_t));
-    afs_indexUnique = (afs_int32 *) afs_osi_Alloc(afiles * sizeof(afs_uint32));
+    afs_indexUnique =
+       (afs_int32 *) afs_osi_Alloc(afiles * sizeof(afs_uint32));
     memset((char *)afs_indexUnique, 0, afiles * sizeof(afs_uint32));
     afs_indexFlags = (u_char *) afs_osi_Alloc(afiles * sizeof(u_char));
     memset((char *)afs_indexFlags, 0, afiles * sizeof(char));
-    
+
     /* Allocate and thread the struct dcache entries themselves */
     tdp = afs_Initial_freeDSList =
-       (struct dcache *) afs_osi_Alloc(aDentries * sizeof(struct dcache));
+       (struct dcache *)afs_osi_Alloc(aDentries * sizeof(struct dcache));
     memset((char *)tdp, 0, aDentries * sizeof(struct dcache));
 #ifdef KERNEL_HAVE_PIN
-    pin((char *)afs_indexTable, sizeof(struct dcache *) * afiles);/* XXX */    
-    pin((char *)afs_indexTimes, sizeof(afs_hyper_t) * afiles); /* XXX */    
-    pin((char *)afs_indexFlags, sizeof(char) * afiles);                /* XXX */    
-    pin((char *)afs_indexUnique, sizeof(afs_int32) * afiles);  /* XXX */    
-    pin((char *)tdp, aDentries * sizeof(struct dcache));       /* XXX */    
-    pin((char *)afs_dvhashTbl, sizeof(afs_int32) * afs_dhashsize);     /* XXX */    
-    pin((char *)afs_dchashTbl, sizeof(afs_int32) * afs_dhashsize);     /* XXX */    
-    pin((char *)afs_dcnextTbl, sizeof(afs_int32) * afiles);            /* XXX */    
-    pin((char *)afs_dvnextTbl, sizeof(afs_int32) * afiles);            /* XXX */    
+    pin((char *)afs_indexTable, sizeof(struct dcache *) * afiles);     /* XXX */
+    pin((char *)afs_indexTimes, sizeof(afs_hyper_t) * afiles); /* XXX */
+    pin((char *)afs_indexFlags, sizeof(char) * afiles);        /* XXX */
+    pin((char *)afs_indexUnique, sizeof(afs_int32) * afiles);  /* XXX */
+    pin((char *)tdp, aDentries * sizeof(struct dcache));       /* XXX */
+    pin((char *)afs_dvhashTbl, sizeof(afs_int32) * afs_dhashsize);     /* XXX */
+    pin((char *)afs_dchashTbl, sizeof(afs_int32) * afs_dhashsize);     /* XXX */
+    pin((char *)afs_dcnextTbl, sizeof(afs_int32) * afiles);    /* XXX */
+    pin((char *)afs_dvnextTbl, sizeof(afs_int32) * afiles);    /* XXX */
 #endif
 
     afs_freeDSList = &tdp[0];
-    for(i=0; i < aDentries-1; i++) {
-       tdp[i].lruq.next = (struct afs_q *) (&tdp[i+1]);
+    for (i = 0; i < aDentries - 1; i++) {
+       tdp[i].lruq.next = (struct afs_q *)(&tdp[i + 1]);
     }
-    tdp[aDentries-1].lruq.next = (struct afs_q *) 0;
+    tdp[aDentries - 1].lruq.next = (struct afs_q *)0;
 
-    afs_stats_cmperf.cacheBlocksOrig = afs_stats_cmperf.cacheBlocksTotal = afs_cacheBlocks = ablocks;
+    afs_stats_cmperf.cacheBlocksOrig = afs_stats_cmperf.cacheBlocksTotal =
+       afs_cacheBlocks = ablocks;
     afs_ComputeCacheParms();   /* compute parms based on cache size */
 
     afs_dcentries = aDentries;
@@ -3147,7 +3242,8 @@ void afs_dcacheInit(int afiles, int ablocks, int aDentries, int achunk,
  * shutdown_dcache
  *
  */
-void shutdown_dcache(void)
+void
+shutdown_dcache(void)
 {
     int i;
 
@@ -3157,28 +3253,31 @@ void shutdown_dcache(void)
     afs_osi_Free(afs_indexTimes, afs_cacheFiles * sizeof(afs_hyper_t));
     afs_osi_Free(afs_indexUnique, afs_cacheFiles * sizeof(afs_uint32));
     afs_osi_Free(afs_indexFlags, afs_cacheFiles * sizeof(u_char));
-    afs_osi_Free(afs_Initial_freeDSList, afs_dcentries * sizeof(struct dcache));
+    afs_osi_Free(afs_Initial_freeDSList,
+                afs_dcentries * sizeof(struct dcache));
 #ifdef KERNEL_HAVE_PIN
     unpin((char *)afs_dcnextTbl, afs_cacheFiles * sizeof(afs_int32));
     unpin((char *)afs_dvnextTbl, afs_cacheFiles * sizeof(afs_int32));
     unpin((char *)afs_indexTable, afs_cacheFiles * sizeof(struct dcache *));
     unpin((char *)afs_indexTimes, afs_cacheFiles * sizeof(afs_hyper_t));
     unpin((char *)afs_indexUnique, afs_cacheFiles * sizeof(afs_uint32));
-    unpin((u_char *)afs_indexFlags, afs_cacheFiles * sizeof(u_char));
+    unpin((u_char *) afs_indexFlags, afs_cacheFiles * sizeof(u_char));
     unpin(afs_Initial_freeDSList, afs_dcentries * sizeof(struct dcache));
 #endif
 
 
-    for(i=0;i< afs_dhashsize;i++) {
+    for (i = 0; i < afs_dhashsize; i++) {
        afs_dvhashTbl[i] = NULLIDX;
        afs_dchashTbl[i] = NULLIDX;
     }
 
+    afs_osi_Free(afs_dvhashTbl, afs_dhashsize * sizeof(afs_int32));
+    afs_osi_Free(afs_dchashTbl, afs_dhashsize * sizeof(afs_int32));
 
     afs_blocksUsed = afs_dcentries = 0;
     hzero(afs_indexCounter);
 
-    afs_freeDCCount =  0;
+    afs_freeDCCount = 0;
     afs_freeDCList = NULLIDX;
     afs_discardDCList = NULLIDX;
     afs_freeDSList = afs_Initial_freeDSList = 0;