volinfo: refactor vnode handling
[openafs.git] / src / vol / vol-info.c
index 6a95cbd..ed60bf0 100644 (file)
 #include <roken.h>
 
 #include <ctype.h>
-#include <errno.h>
-#include <sys/stat.h>
-#include <stdio.h>
-#include <string.h>
-#ifdef AFS_NT40_ENV
-#include <fcntl.h>
-#include <time.h>
-#include <io.h>
-#else
-#include <sys/param.h>
+
+#ifdef HAVE_SYS_FILE_H
 #include <sys/file.h>
-#include <sys/time.h>
 #endif
+
 #include <afs/cmd.h>
 #include <afs/dir.h>
-
-#include <rx/xdr.h>
 #include <afs/afsint.h>
-#include "nfs.h"
 #include <afs/errors.h>
+#include <opr/queue.h>
+
+#include "nfs.h"
 #include "lock.h"
-#include "lwp.h"
-#include <afs/afssyscalls.h>
 #include "ihandle.h"
 #include "vnode.h"
 #include "volume.h"
 #include "partition.h"
-#include "viceinode.h"
-#include <afs/afssyscalls.h>
-#include <afs/afsutil.h>
 
-#ifdef _AIX
-#include <time.h>
+#ifndef AFS_NT40_ENV
+#include "AFS_component_version_number.c"
 #endif
 
-#include <dirent.h>
-
-int DumpVnodes = 0;            /* Dump everything, i.e. summary of all vnodes */
-int DumpInodeNumber = 0;       /* Dump inode numbers with vnodes */
-int DumpDate = 0;              /* Dump vnode date (server modify date) with vnode */
-int InodeTimes = 0;            /* Dump some of the dates associated with inodes */
+static const char *progname = "volinfo";
+
+/* Command line options */
+typedef enum {
+    P_ONLINE,
+    P_VNODE,
+    P_DATE,
+    P_INODE,
+    P_ITIME,
+    P_PART,
+    P_VOLUMEID,
+    P_HEADER,
+    P_SIZEONLY,
+    P_FIXHEADER,
+    P_SAVEINODES,
+    P_ORPHANED,
+    P_FILENAMES
+} volinfo_parm_t;
+
+/* Vnode details for vnode handling procedures */
+struct VnodeDetails {
+    Volume *vp;
+    VnodeClass class;
+    VnodeDiskObject *vnode;
+    VnodeId vnodeNumber;
+    afs_foff_t offset;
+    int index;
+};
+
+/* Modes */
+static int DumpInfo = 0;            /**< Dump volume information */
+static int DumpHeader = 0;          /**< Dump volume header files info */
+static int DumpVnodes = 0;          /**< Dump vnode info */
+static int DumpInodeNumber = 0;     /**< Dump inode numbers with vnodes */
+static int DumpDate = 0;            /**< Dump vnode date (server modify date) with vnode */
+static int InodeTimes = 0;          /**< Dump some of the dates associated with inodes */
 #if defined(AFS_NAMEI_ENV)
-int PrintFileNames = 0;
+static int PrintFileNames = 0;      /**< Dump vnode and special file name filenames */
 #endif
-int online = 0;
-int dheader = 0;
-int dsizeOnly = 0, totvolsize = 0, Vauxsize = 0, Vdiskused = 0, Vvnodesize =
-    0;
-int Vvnodesize_k = 0, Vauxsize_k = 0;
-int Totvolsize = 0, TVauxsize = 0, TVdiskused = 0, TVvnodesize = 0;
-int Stotvolsize = 0, SVauxsize = 0, SVdiskused = 0, SVvnodesize = 0;
-int fixheader = 0, saveinodes = 0, orphaned = 0;
-int VolumeChanged;
+static int ShowOrphaned = 0;        /**< Show "orphaned" vnodes (vnodes with parent of 0) */
+static int ShowSizes = 0;           /**< Show volume size summary */
+static int SaveInodes = 0;          /**< Save vnode data to files */
+static int FixHeader = 0;           /**< Repair header files magic and version fields. */
+
+/**
+ * Volume size running totals
+ */
+struct sizeTotals {
+    afs_uint64 diskused_k;     /**< volume size from disk data file, in kilobytes */
+    afs_uint64 auxsize;                /**< size of header files, in bytes  */
+    afs_uint64 auxsize_k;      /**< size of header files, in kilobytes */
+    afs_uint64 vnodesize;      /**< size of the large and small vnodes, in bytes */
+    afs_uint64 vnodesize_k;    /**< size of the large and small vnodes, in kilobytes */
+    afs_uint64 size_k;         /**< size of headers and vnodes, in kilobytes */
+};
+
+static struct sizeTotals volumeTotals = { 0, 0, 0, 0, 0, 0 };
+static struct sizeTotals partitionTotals = { 0, 0, 0, 0, 0, 0 };
+static struct sizeTotals serverTotals = { 0, 0, 0, 0, 0, 0 };
+static int PrintingVolumeSizes = 0;    /*print volume size lines */
+
+/**
+ * List of procedures to call when scanning vnodes.
+ */
+struct VnodeScanProc {
+    struct opr_queue link;
+    const char *heading;
+    void (*proc) (struct VnodeDetails * vdp);
+};
+static struct opr_queue VnodeScanLists[nVNODECLASSES];
 
 /* Forward Declarations */
 void PrintHeader(Volume * vp);
@@ -82,15 +121,23 @@ void HandleVolume(struct DiskPartition64 *partP, char *name);
 struct DiskPartition64 *FindCurrentPartition(void);
 Volume *AttachVolume(struct DiskPartition64 *dp, char *volname,
                     struct VolumeHeader *header);
-#if defined(AFS_NAMEI_ENV)
-void PrintVnode(afs_foff_t offset, VnodeDiskObject * vnode, VnodeId vnodeNumber,
-               Inode ino, Volume * vp);
-#else
-void PrintVnode(afs_foff_t offset, VnodeDiskObject * vnode, VnodeId vnodeNumber,
-               Inode ino);
-#endif
-void PrintVnodes(Volume * vp, VnodeClass class);
+void HandleVnodes(Volume * vp, VnodeClass class);
+static void AddVnodeToSizeTotals(struct VnodeDetails *vdp);
+static void SaveInode(struct VnodeDetails *vdp);
+static void PrintVnode(struct VnodeDetails *vdp);
 
+/**
+ * Format time as a timestamp string
+ *
+ * @param[in] date  time value to format
+ *
+ * @return timestamp string in the form YYYY/MM/DD.hh:mm:ss
+ *
+ * @note A static array of 8 strings are stored by this
+ *       function. The array slots are overwritten, so the
+ *       caller must not reference the returned string after
+ *       seven additional calls to this function.
+ */
 char *
 date(time_t date)
 {
@@ -106,14 +153,151 @@ date(time_t date)
     return results[next];
 }
 
-#ifndef AFS_NT40_ENV
-#include "AFS_component_version_number.c"
+#ifdef AFS_NT40_ENV
+/**
+ * Format file time as a timestamp string
+ *
+ * @param[in] ft  file time
+ *
+ * @return timestamp string in the form YYYY/MM/DD.hh:mm:ss
+ *
+ * @note A static array of 8 strings are stored by this
+ *       function. The array slots are overwritten, so the
+ *       caller must not reference the returned string after
+ *       seven additional calls to this function.
+ */
+char *
+NT_date(FILETIME * ft)
+{
+    static char result[8][64];
+    static int next = 0;
+    SYSTEMTIME st;
+    FILETIME lft;
+
+    if (!FileTimeToLocalFileTime(ft, &lft)
+       || !FileTimeToSystemTime(&lft, &st)) {
+       fprintf(stderr, "%s: Time conversion failed.\n", progname);
+       exit(1);
+    }
+    sprintf(result[next = ((next + 1) & 7)], "%4d/%02d/%02d.%2d:%2d:%2d",
+           st.wYear, st.wMonth, st.wDay, st.wHour, st.wMinute, st.wSecond);
+    return result[next];
+}
 #endif
 
-char name[VMAXPATHLEN];
+/**
+ * Add vnode size to the running volume totals.
+ *
+ * @param[in]  vdp   vnode details object
+ *
+ * @return none
+ */
+static void
+AddVnodeToSizeTotals(struct VnodeDetails *vdp)
+{
+    afs_fsize_t fileLength;
 
-char BU[1000];
+    VNDISK_GET_LEN(fileLength, vdp->vnode);
+    if (fileLength > 0) {
+       volumeTotals.vnodesize += fileLength;
+       volumeTotals.vnodesize_k += fileLength / 1024;
+    }
+}
 
+/**
+ * Print the volume size table heading line, if needed.
+ *
+ * @return none
+ */
+static void
+PrintVolumeSizeHeading(void)
+{
+    if (!PrintingVolumeSizes) {
+       printf
+           ("Volume-Id\t  Volsize  Auxsize Inodesize  AVolsize SizeDiff                (VolName)\n");
+       PrintingVolumeSizes = 1;
+    }
+}
+
+/**
+ * Accumulate totals.
+ *
+ * @return 0
+ */
+static void
+AddSizeTotals(struct sizeTotals *a, struct sizeTotals *b)
+{
+    a->diskused_k += b->diskused_k;
+    a->auxsize += b->auxsize;
+    a->auxsize_k += b->auxsize_k;
+    a->vnodesize += b->vnodesize;
+    a->vnodesize_k += b->vnodesize_k;
+    a->size_k += b->size_k;
+}
+
+/**
+ * Print the sizes for a volume.
+ *
+ * @return none
+ */
+static void
+PrintVolumeSizes(Volume * vp)
+{
+    afs_int64 diff_k = volumeTotals.size_k - volumeTotals.diskused_k;
+
+    PrintVolumeSizeHeading();
+    printf("%u\t%9llu%9llu%10llu%10llu%9lld\t%24s\n",
+          V_id(vp),
+          volumeTotals.diskused_k,
+          volumeTotals.auxsize_k, volumeTotals.vnodesize_k,
+          volumeTotals.size_k, diff_k, V_name(vp));
+}
+
+/**
+ * Print the size totals for the partition.
+ *
+ * @return none
+ */
+static void
+PrintPartitionTotals(afs_uint64 nvols)
+{
+    afs_int64 diff_k = partitionTotals.size_k - partitionTotals.diskused_k;
+
+    PrintVolumeSizeHeading();
+    printf("\nPart Totals  %12llu%9llu%10llu%10llu%9lld (%llu volumes)\n\n",
+          partitionTotals.diskused_k, partitionTotals.auxsize,
+          partitionTotals.vnodesize, partitionTotals.size_k, diff_k, nvols);
+    PrintingVolumeSizes = 0;
+}
+
+/**
+ * Print the size totals for all the partitions.
+ *
+ * @return none
+ */
+static void
+PrintServerTotals(void)
+{
+    afs_int64 diff_k = serverTotals.size_k - serverTotals.diskused_k;
+
+    printf("\nServer Totals%12lld%9lld%10lld%10lld%9lld\n",
+          serverTotals.diskused_k, serverTotals.auxsize,
+          serverTotals.vnodesize, serverTotals.size_k, diff_k);
+}
+
+/**
+ * Read a volume header file
+ *
+ * @param[in] ih        ihandle of the header file
+ * @param[in] to        destination
+ * @param[in] size      expected header size
+ * @param[in] magic     expected header magic number
+ * @param[in] version   expected header version number
+ *
+ * @return error code
+ *   @retval 0 success
+ *   @retval -1 failed to read file
+ */
 int
 ReadHdr1(IHandle_t * ih, char *to, int size, u_int magic, u_int version)
 {
@@ -129,35 +313,45 @@ ReadHdr1(IHandle_t * ih, char *to, int size, u_int magic, u_int version)
 
     if (vsn->magic != magic) {
        bad++;
-       printf("Inode %s: Bad magic %x (%x): IGNORED\n",
-              PrintInode(NULL, ih->ih_ino), vsn->magic, magic);
+       fprintf(stderr, "%s: Inode %s: Bad magic %x (%x): IGNORED\n",
+               progname, PrintInode(NULL, ih->ih_ino), vsn->magic, magic);
     }
 
     /* Check is conditional, in case caller wants to inspect version himself */
     if (version && vsn->version != version) {
        bad++;
-       printf("Inode %s: Bad version %x (%x): IGNORED\n",
-              PrintInode(NULL, ih->ih_ino), vsn->version, version);
+       fprintf(stderr, "%s: Inode %s: Bad version %x (%x): IGNORED\n",
+               progname,
+               PrintInode(NULL, ih->ih_ino), vsn->version, version);
     }
-    if (bad && fixheader) {
+    if (bad && FixHeader) {
        vsn->magic = magic;
        vsn->version = version;
        printf("Special index inode %s has a bad header. Reconstructing...\n",
               PrintInode(NULL, ih->ih_ino));
        code = IH_IWRITE(ih, 0, to, size);
        if (code != size) {
-           printf("Write failed; header left in damaged state\n");
-       }
-    } else {
-       if (!dsizeOnly && !saveinodes) {
-           printf("Inode %s: Good magic %x and version %x\n",
-                  PrintInode(NULL, ih->ih_ino), magic, version);
+           fprintf(stderr,
+                   "%s: Write failed for inode %s; header left in damaged state\n",
+                   progname, PrintInode(NULL, ih->ih_ino));
        }
     }
+    if (!bad && DumpInfo) {
+       printf("Inode %s: Good magic %x and version %x\n",
+              PrintInode(NULL, ih->ih_ino), magic, version);
+    }
     return 0;
 }
 
-
+/**
+ * Simplified attach volume
+ *
+ * param[in] dp       vice disk partition object
+ * param[in] volname  volume header file name
+ * param[in] header   volume header object
+ *
+ * @return volume object or null on error
+ */
 Volume *
 AttachVolume(struct DiskPartition64 * dp, char *volname,
             struct VolumeHeader * header)
@@ -166,6 +360,10 @@ AttachVolume(struct DiskPartition64 * dp, char *volname,
     afs_int32 ec = 0;
 
     vp = (Volume *) calloc(1, sizeof(Volume));
+    if (!vp) {
+       fprintf(stderr, "%s: Failed to allocate volume object.\n", progname);
+       return NULL;
+    }
     vp->specialStatus = 0;
     vp->device = dp->device;
     vp->partition = dp;
@@ -181,6 +379,11 @@ AttachVolume(struct DiskPartition64 * dp, char *volname,
     vp->goingOffline = 0;
     vp->nUsers = 1;
     vp->header = (struct volHeader *)calloc(1, sizeof(*vp->header));
+    if (!vp->header) {
+       fprintf(stderr, "%s: Failed to allocate volume header.\n", progname);
+       free(vp);
+       return NULL;
+    }
     ec = ReadHdr1(V_diskDataHandle(vp), (char *)&V_disk(vp),
                  sizeof(V_disk(vp)), VOLUMEINFOMAGIC, VOLUMEINFOVERSION);
     if (!ec) {
@@ -205,93 +408,125 @@ AttachVolume(struct DiskPartition64 * dp, char *volname,
     return vp;
 }
 
+/**
+ * Simplified detach volume
+ *
+ * param[in] vp       volume object from AttachVolume
+ *
+ * @return none
+ */
+static void
+DetachVolume(Volume * vp)
+{
+    IH_RELEASE(vp->vnodeIndex[vLarge].handle);
+    IH_RELEASE(vp->vnodeIndex[vSmall].handle);
+    IH_RELEASE(vp->diskDataHandle);
+    IH_RELEASE(V_linkHandle(vp));
+    free(vp->header);
+    free(vp);
+}
 
+/**
+ * Convert the partition device number into a partition name.
+ *
+ * @param[in]   partId      partition number, 0 to 254
+ * @param[out]  partName    buffer to hold partition name (e.g. /vicepa)
+ * @param[in]   partNameSize    size of partName buffer
+ *
+ * @return status
+ *   @retval 0 success
+ *   @retval -1 error, partId is out of range
+ *   @retval -2 error, partition name exceeds partNameSize
+ */
 static int
-handleit(struct cmd_syndesc *as, void *arock)
+GetPartitionName(afs_uint32 partId, char *partName, afs_size_t partNameSize)
+{
+    const int OLD_NUM_DEVICES = 26;    /* a..z */
+
+    if (partId < OLD_NUM_DEVICES) {
+       if (partNameSize < 8) {
+           return -2;
+       }
+       strlcpy(partName, "/vicep", partNameSize);
+       partName[6] = partId + 'a';
+       partName[7] = '\0';
+       return 0;
+    }
+    if (partId < VOLMAXPARTS) {
+       if (partNameSize < 9) {
+           return -2;
+       }
+       strlcpy(partName, "/vicep", partNameSize);
+       partId -= OLD_NUM_DEVICES;
+       partName[6] = 'a' + (partId / OLD_NUM_DEVICES);
+       partName[7] = 'a' + (partId % OLD_NUM_DEVICES);
+       partName[8] = '\0';
+       return 0;
+    }
+    return -1;
+}
+
+/**
+ * Scan the volumes in the partitions
+ *
+ * Scan the specified volume in the specified partition if both
+ * are given. Scan all the volumes in the specified partition if
+ * only the partition is given.  If neither a partition nor volume
+ * is given, scan all the volumes in all the partitions.  If only
+ * the volume is given, attempt to find it in the current working
+ * directory.
+ *
+ * @param[in] partNameOrId   partition name or id to be scannned
+ * @param[in] volumeId       volume id to be scanned
+ *
+ * @return 0 if successful
+ */
+static int
+ScanPartitions(char *partNameOrId, afs_uint32 volumeId)
 {
-    struct cmd_item *ti;
     int err = 0;
-    afs_uint32 volumeId = 0;
-    char *partName = 0;
+    char partName[64] = "";
     struct DiskPartition64 *partP = NULL;
 
-
 #ifndef AFS_NT40_ENV
     if (geteuid() != 0) {
-       printf("vol-info must be run as root; sorry\n");
-       exit(1);
+       fprintf(stderr, "%s: Must be run as root; sorry\n", progname);
+       return 1;
     }
 #endif
 
-    if (as->parms[0].items)
-       online = 1;
-    else
-       online = 0;
-    if (as->parms[1].items)
-       DumpVnodes = 1;
-    else
-       DumpVnodes = 0;
-    if (as->parms[2].items)
-       DumpDate = 1;
-    else
-       DumpDate = 0;
-    if (as->parms[3].items)
-       DumpInodeNumber = 1;
-    else
-       DumpInodeNumber = 0;
-    if (as->parms[4].items)
-       InodeTimes = 1;
-    else
-       InodeTimes = 0;
-    if ((ti = as->parms[5].items))
-       partName = ti->data;
-    if ((ti = as->parms[6].items))
-       volumeId = strtoul(ti->data, NULL, 10);
-    if (as->parms[7].items)
-       dheader = 1;
-    else
-       dheader = 0;
-    if (as->parms[8].items) {
-       dsizeOnly = 1;
-       dheader = 1;
-       DumpVnodes = 1;
-    } else
-       dsizeOnly = 0;
-    if (as->parms[9].items) {
-       fixheader = 1;
-    } else
-       fixheader = 0;
-    if (as->parms[10].items) {
-       saveinodes = 1;
-       dheader = 1;
-       DumpVnodes = 1;
-    } else
-       saveinodes = 0;
-    if (as->parms[11].items) {
-       orphaned = 1;
-       DumpVnodes = 1;
-    } else
-#if defined(AFS_NAMEI_ENV)
-    if (as->parms[12].items) {
-       PrintFileNames = 1;
-       DumpVnodes = 1;
-    } else
-#endif
-       orphaned = 0;
-
     DInit(10);
 
+    /* Allow user to specify partition by name or id. */
+    if (partNameOrId) {
+       afs_uint32 partId = volutil_GetPartitionID(partNameOrId);
+       if (partId == -1) {
+           fprintf(stderr, "%s: Could not parse '%s' as a partition name.\n",
+                   progname, partNameOrId);
+           return 1;
+       }
+       if (GetPartitionName(partId, partName, sizeof(partName))) {
+           fprintf(stderr,
+                   "%s: Could not format '%s' as a partition name.\n",
+                   progname, partNameOrId);
+           return 1;
+       }
+    }
+
     err = VAttachPartitions();
     if (err) {
-       printf("%d partitions had errors during attach.\n", err);
+       fprintf(stderr, "%s: %d partitions had errors during attach.\n",
+               progname, err);
+       return 1;
     }
 
-    if (partName) {
+    if (partName[0]) {
        partP = VGetPartition(partName, 0);
        if (!partP) {
-           printf("%s is not an AFS partition name on this server.\n",
-                  partName);
-           exit(1);
+           fprintf(stderr,
+                   "%s: %s is not an AFS partition name on this server.\n",
+                   progname, partName);
+           return 1;
        }
     }
 
@@ -307,20 +542,138 @@ handleit(struct cmd_syndesc *as, void *arock)
        if (!partP) {
            partP = FindCurrentPartition();
            if (!partP) {
-               printf("Current partition is not a vice partition.\n");
-               exit(1);
+               fprintf(stderr,
+                       "%s: Current partition is not a vice partition.\n",
+                       progname);
+               return 1;
            }
        }
        snprintf(name1, sizeof name1, VFORMAT,
                 afs_printable_uint32_lu(volumeId));
-       if (dsizeOnly && !saveinodes)
-           printf
-               ("Volume-Id\t  Volsize  Auxsize Inodesize  AVolsize SizeDiff                (VolName)\n");
        HandleVolume(partP, name1);
     }
+
     return 0;
 }
 
+/**
+ * Add a vnode scanning procedure
+ *
+ * @param[in]   class   vnode class for this handler
+ * @param[in]   proc    handler proc to be called by HandleVnodes
+ * @param[in]   heading optional string to pring before scanning vnodes
+ *
+ * @return none
+ */
+static void
+AddVnodeHandler(VnodeClass class, void (*proc) (struct VnodeDetails * vdp),
+               const char *heading)
+{
+    struct VnodeScanProc *entry = malloc(sizeof(struct VnodeScanProc));
+    entry->proc = proc;
+    entry->heading = heading;
+    opr_queue_Append(&VnodeScanLists[class], (struct opr_queue *)entry);
+}
+
+/**
+ * Process command line options and start scanning
+ *
+ * @param[in] as     command syntax object
+ * @param[in] arock  opaque object; not used
+ *
+ * @return error code
+ */
+static int
+handleit(struct cmd_syndesc *as, void *arock)
+{
+    struct cmd_item *ti;
+    afs_uint32 volumeId = 0;
+    char *partNameOrId = NULL;
+
+    DumpInfo = 1;              /* volinfo default mode */
+
+    if (as->parms[P_ONLINE].items) {
+       fprintf(stderr, "%s: -online not supported\n", progname);
+       return 1;
+    }
+    if (as->parms[P_VNODE].items) {
+       DumpVnodes = 1;
+    }
+    if (as->parms[P_DATE].items) {
+       DumpDate = 1;
+    }
+    if (as->parms[P_INODE].items) {
+       DumpInodeNumber = 1;
+    }
+    if (as->parms[P_ITIME].items) {
+       InodeTimes = 1;
+    }
+    if ((ti = as->parms[P_PART].items)) {
+       partNameOrId = ti->data;
+    }
+    if ((ti = as->parms[P_VOLUMEID].items)) {
+       volumeId = strtoul(ti->data, NULL, 10);
+    }
+    if (as->parms[P_HEADER].items) {
+       DumpHeader = 1;
+    }
+    if (as->parms[P_SIZEONLY].items) {
+       ShowSizes = 1;
+    }
+    if (as->parms[P_FIXHEADER].items) {
+       FixHeader = 1;
+    }
+    if (as->parms[P_SAVEINODES].items) {
+       SaveInodes = 1;
+    }
+    if (as->parms[P_ORPHANED].items) {
+       ShowOrphaned = 1;
+    }
+#if defined(AFS_NAMEI_ENV)
+    if (as->parms[P_FILENAMES].items) {
+       PrintFileNames = 1;
+    }
+#endif
+
+    /* -saveinodes and -sizeOnly override the default mode.
+     * For compatibility with old versions of volinfo, -orphaned
+     * and -filename options imply -vnodes */
+    if (SaveInodes || ShowSizes) {
+       DumpInfo = 0;
+       DumpHeader = 0;
+       DumpVnodes = 0;
+       InodeTimes = 0;
+       ShowOrphaned = 0;
+    } else if (ShowOrphaned) {
+       DumpVnodes = 1;         /* implied */
+#ifdef AFS_NAMEI_ENV
+    } else if (PrintFileNames) {
+       DumpVnodes = 1;         /* implied */
+#endif
+    }
+
+    if (SaveInodes) {
+       AddVnodeHandler(vSmall, SaveInode,
+                       "Saving all volume files to current directory ...\n");
+    }
+    if (ShowSizes) {
+       AddVnodeHandler(vLarge, AddVnodeToSizeTotals, NULL);
+       AddVnodeHandler(vSmall, AddVnodeToSizeTotals, NULL);
+    }
+    if (DumpVnodes) {
+       AddVnodeHandler(vLarge, PrintVnode, "\nLarge vnodes (directories)\n");
+       AddVnodeHandler(vSmall, PrintVnode,
+                       "\nSmall vnodes(files, symbolic links)\n");
+    }
+
+    return ScanPartitions(partNameOrId, volumeId);
+}
+
+/**
+ * Determine if the current directory is a vice partition
+ *
+ * @return disk partition object
+ */
 #ifdef AFS_NT40_ENV
 #include <direct.h>
 struct DiskPartition64 *
@@ -335,7 +688,8 @@ FindCurrentPartition(void)
            break;
     }
     if (!dp) {
-       printf("Current drive is not a valid vice partition.\n");
+       fprintf(stderr, "%s: Current drive is not a valid vice partition.\n",
+               progname);
     }
     return dp;
 }
@@ -349,8 +703,10 @@ FindCurrentPartition(void)
     struct DiskPartition64 *dp;
 
     if (!getcwd(partName, 1023)) {
+       fprintf(stderr, "%s: Failed to get current working directory: ",
+               progname);
        perror("pwd");
-       exit(1);
+       return NULL;
     }
     p = strchr(&partName[1], OS_DIRSEPC);
     if (p) {
@@ -360,13 +716,19 @@ FindCurrentPartition(void)
     if (!(dp = VGetPartition(partName, 0))) {
        if (tmp)
            *p = tmp;
-       printf("%s is not a valid vice partition.\n", partName);
-       exit(1);
+       fprintf(stderr, "%s: %s is not a valid vice partition.\n", progname,
+               partName);
+       return NULL;
     }
     return dp;
 }
 #endif
 
+/**
+ * Scan and handle all the partitions detected on this server
+ *
+ * @return none
+ */
 void
 HandleAllPart(void)
 {
@@ -374,25 +736,31 @@ HandleAllPart(void)
 
 
     for (partP = DiskPartitionList; partP; partP = partP->next) {
-       printf("Processing Partition %s:\n", partP->name);
+       if (DumpInfo || SaveInodes || ShowSizes) {
+           printf("Processing Partition %s:\n", partP->name);
+       }
        HandlePart(partP);
-       Stotvolsize += Totvolsize;
-       SVauxsize += TVauxsize;
-       SVvnodesize += TVvnodesize;
-       SVdiskused += TVdiskused;
+       if (ShowSizes) {
+           AddSizeTotals(&serverTotals, &partitionTotals);
+       }
     }
 
-    if (dsizeOnly) {
-       printf("\nServer Totals%12d%9d%10d%10d%9d\n", SVdiskused, SVauxsize,
-              SVvnodesize, Stotvolsize, Stotvolsize - SVdiskused);
+    if (ShowSizes) {
+       PrintServerTotals();
     }
 }
 
-
+/**
+ * Scan the partition and handle volumes
+ *
+ * @param[in] partP  disk partition to scan
+ *
+ * @return none
+ */
 void
 HandlePart(struct DiskPartition64 *partP)
 {
-    int nvols = 0;
+    afs_int64 nvols = 0;
     DIR *dirp;
     struct dirent *dp;
 #ifdef AFS_NT40_ENV
@@ -404,244 +772,287 @@ HandlePart(struct DiskPartition64 *partP)
 #endif
 
     if ((dirp = opendir(p)) == NULL) {
-       printf("Can't read directory %s; giving up\n", p);
-       exit(1);
+       fprintf(stderr, "%s: Can't read directory %s; giving up\n", progname,
+               p);
+       return;
     }
-    if (dsizeOnly && !saveinodes)
-       printf
-           ("Volume-Id\t  Volsize  Auxsize Inodesize  AVolsize SizeDiff                (VolName)\n");
     while ((dp = readdir(dirp))) {
        p = (char *)strrchr(dp->d_name, '.');
        if (p != NULL && strcmp(p, VHDREXT) == 0) {
            HandleVolume(partP, dp->d_name);
-           Totvolsize += totvolsize;
-           TVauxsize += Vauxsize;
-           TVvnodesize += Vvnodesize;
-           TVdiskused += Vdiskused;
-           nvols++;
+           if (ShowSizes) {
+               nvols++;
+               AddSizeTotals(&partitionTotals, &volumeTotals);
+           }
        }
     }
     closedir(dirp);
-    if (dsizeOnly) {
-       printf("\nPart Totals  %12d%9d%10d%10d%9d (%d volumes)\n\n",
-              TVdiskused, TVauxsize, TVvnodesize, Totvolsize,
-              Totvolsize - TVdiskused, nvols);
+    if (ShowSizes) {
+       PrintPartitionTotals(nvols);
     }
 }
 
+/**
+ * Inspect a volume header special file.
+ *
+ * @param[in]  name       descriptive name of the type of header special file
+ * @param[in]  dp         partition object for this volume
+ * @param[in]  header     header object for this volume
+ * @param[in]  inode      fileserver inode number for this header special file
+ * @param[out] psize      total of the header special file
+ *
+ * @return none
+ */
+void
+HandleSpecialFile(const char *name, struct DiskPartition64 *dp,
+                 struct VolumeHeader *header, Inode inode,
+                 afs_sfsize_t * psize)
+{
+    afs_sfsize_t size = 0;
+    IHandle_t *ih = NULL;
+    FdHandle_t *fdP = NULL;
+#ifdef AFS_NAMEI_ENV
+    namei_t filename;
+#endif /* AFS_NAMEI_ENV */
 
+    IH_INIT(ih, dp->device, header->parent, inode);
+    fdP = IH_OPEN(ih);
+    if (fdP == NULL) {
+       fprintf(stderr,
+               "%s: Error opening header file '%s' for volume %u", progname,
+               name, header->id);
+       perror("open");
+       goto error;
+    }
+    size = FDH_SIZE(fdP);
+    if (size == -1) {
+       fprintf(stderr,
+               "%s: Error getting size of header file '%s' for volume %u",
+               progname, name, header->id);
+       perror("fstat");
+       goto error;
+    }
+    if (DumpInfo) {
+       printf("\t%s inode\t= %s (size = %lld)\n",
+              name, PrintInode(NULL, inode), size);
+#ifdef AFS_NAMEI_ENV
+       namei_HandleToName(&filename, ih);
+       printf("\t%s namei\t= %s\n", name, filename.n_path);
+#endif /* AFS_NAMEI_ENV */
+    }
+    *psize += size;
+
+  error:
+    if (fdP != NULL) {
+       FDH_REALLYCLOSE(fdP);
+    }
+    if (ih != NULL) {
+       IH_RELEASE(ih);
+    }
+}
+
+/**
+ * Inspect this volume header files.
+ *
+ * @param[in]  dp         partition object for this volume
+ * @param[in]  header_fd  volume header file descriptor
+ * @param[in]  header     volume header object
+ * @param[out] psize      total of the header special file
+ *
+ * @return none
+ */
+void
+HandleHeaderFiles(struct DiskPartition64 *dp, FD_t header_fd,
+                 struct VolumeHeader *header)
+{
+    afs_sfsize_t size = 0;
+
+    if (DumpInfo) {
+       size = OS_SIZE(header_fd);
+       printf("Volume header (size = %lld):\n", size);
+       printf("\tstamp\t= 0x%x\n", header->stamp.version);
+       printf("\tVolId\t= %u\n", header->id);
+       printf("\tparent\t= %u\n", header->parent);
+    }
+
+    HandleSpecialFile("Info", dp, header, header->volumeInfo, &size);
+    HandleSpecialFile("Small", dp, header, header->smallVnodeIndex,
+                     &size);
+    HandleSpecialFile("Large", dp, header, header->largeVnodeIndex,
+                     &size);
+#ifdef AFS_NAMEI_ENV
+    HandleSpecialFile("Link", dp, header, header->linkTable, &size);
+#endif /* AFS_NAMEI_ENV */
+
+    if (DumpInfo) {
+       printf("Total aux volume size = %lld\n\n", size);
+    }
+
+    if (ShowSizes) {
+        volumeTotals.auxsize = size;
+        volumeTotals.auxsize_k = size / 1024;
+    }
+}
+
+/**
+ * Attach and scan the volume and handle the header and vnodes
+ *
+ * Print the volume header and vnode information, depending on the
+ * current modes.
+ *
+ * @param[in] dp    vice partition object for this volume
+ * @param[in] name  volume header file name
+ *
+ * @return none
+ */
 void
 HandleVolume(struct DiskPartition64 *dp, char *name)
 {
     struct VolumeHeader header;
     struct VolumeDiskHeader diskHeader;
     FD_t fd = INVALID_FD;
-    Volume *vp;
-    IHandle_t *ih;
+    Volume *vp = NULL;
     char headerName[1024];
+    afs_sfsize_t n;
 
-    if (online) {
-       printf("volinfo: -online not supported\n");
-       exit(1);
-    } else {
-       afs_sfsize_t n;
-
-       snprintf(headerName, sizeof headerName, "%s" OS_DIRSEP "%s",
-                VPartitionPath(dp), name);
-       if ((fd = OS_OPEN(headerName, O_RDONLY, 0666)) == INVALID_FD
-           || OS_SIZE(fd) < 0) {
-           printf("Volinfo: Cannot read volume header %s\n", name);
-           OS_CLOSE(fd);
-           exit(1);
-       }
-       n = OS_READ(fd, &diskHeader, sizeof(diskHeader));
+    snprintf(headerName, sizeof headerName, "%s" OS_DIRSEP "%s",
+            VPartitionPath(dp), name);
+    if ((fd = OS_OPEN(headerName, O_RDONLY, 0666)) == INVALID_FD) {
+       fprintf(stderr, "%s: Cannot open volume header %s\n", progname, name);
+       goto cleanup;
+    }
+    if (OS_SIZE(fd) < 0) {
+       fprintf(stderr, "%s: Cannot read volume header %s\n", progname, name);
+       goto cleanup;
+    }
+    n = OS_READ(fd, &diskHeader, sizeof(diskHeader));
+    if (n != sizeof(diskHeader)
+       || diskHeader.stamp.magic != VOLUMEHEADERMAGIC) {
+       fprintf(stderr, "%s: Error reading volume header %s\n", progname,
+               name);
+       goto cleanup;
+    }
+    if (diskHeader.stamp.version != VOLUMEHEADERVERSION) {
+       fprintf(stderr,
+               "%s: Volume %s, version number is incorrect; volume needs to be salvaged\n",
+               progname, name);
+       goto cleanup;
+    }
 
-       if (n != sizeof(diskHeader)
-           || diskHeader.stamp.magic != VOLUMEHEADERMAGIC) {
-           printf("Volinfo: Error reading volume header %s\n", name);
-           exit(1);
-       }
-       if (diskHeader.stamp.version != VOLUMEHEADERVERSION) {
-           printf
-               ("Volinfo: Volume %s, version number is incorrect; volume needs salvage\n",
-                name);
-           exit(1);
-       }
-       DiskToVolumeHeader(&header, &diskHeader);
-
-       if (dheader) {
-           FdHandle_t *fdP;
-           afs_sfsize_t size = 0;
-           afs_sfsize_t code;
-
-           if (!dsizeOnly && !saveinodes) {
-               size = OS_SIZE(fd);
-               printf("Volume header (size = %d):\n", (int)size);
-               printf("\tstamp\t= 0x%x\n", header.stamp.version);
-               printf("\tVolId\t= %u\n", header.id);
-           }
+    DiskToVolumeHeader(&header, &diskHeader);
+    if (DumpHeader || ShowSizes) {
+       HandleHeaderFiles(dp, fd, &header);
+    }
 
-           IH_INIT(ih, dp->device, header.parent, header.volumeInfo);
-           fdP = IH_OPEN(ih);
-           if (fdP == NULL) {
-               perror("opening volume info");
-               exit(1);
-           }
-           code = FDH_SIZE(fdP);
-           if (code == -1) {
-               perror("fstat");
-               exit(1);
-           }
-           FDH_REALLYCLOSE(fdP);
-           IH_RELEASE(ih);
-           size += code;
-           if (!dsizeOnly && !saveinodes) {
-               printf("\tparent\t= %u\n", header.parent);
-               printf("\tInfo inode\t= %s (size = %d)\n",
-                      PrintInode(NULL, header.volumeInfo), (int)code);
-           }
+    vp = AttachVolume(dp, name, &header);
+    if (!vp) {
+       fprintf(stderr, "%s: Error attaching volume header %s\n",
+               progname, name);
+       goto cleanup;
+    }
 
-           IH_INIT(ih, dp->device, header.parent, header.smallVnodeIndex);
-           fdP = IH_OPEN(ih);
-           if (fdP == NULL) {
-               perror("opening small vnode index");
-               exit(1);
-           }
-           code = FDH_SIZE(fdP);
-           if (code == -1) {
-               perror("fstat");
-               exit(1);
-           }
-           FDH_REALLYCLOSE(fdP);
-           IH_RELEASE(ih);
-           size += code;
-           if (!dsizeOnly && !saveinodes) {
-               printf("\tSmall inode\t= %s (size = %d)\n",
-                      PrintInode(NULL, header.smallVnodeIndex), (int)code);
-           }
+    if (DumpInfo) {
+       PrintHeader(vp);
+    }
 
-           IH_INIT(ih, dp->device, header.parent, header.largeVnodeIndex);
-           fdP = IH_OPEN(ih);
-           if (fdP == NULL) {
-               perror("opening large vnode index");
-               exit(1);
-           }
-           code = FDH_SIZE(fdP);
-           if (code == -1) {
-               perror("fstat");
-               exit(1);
-           }
-           FDH_REALLYCLOSE(fdP);
-           IH_RELEASE(ih);
-           size += code;
-           if (!dsizeOnly && !saveinodes) {
-               printf("\tLarge inode\t= %s (size = %d)\n",
-                      PrintInode(NULL, header.largeVnodeIndex), (int)code);
-#ifndef AFS_NT40_ENV
-               printf("Total aux volume size = %d\n\n", (int)size);
-#endif
-           }
-#ifdef AFS_NAMEI_ENV
-           IH_INIT(ih, dp->device, header.parent, header.linkTable);
-           fdP = IH_OPEN(ih);
-           if (fdP == NULL) {
-               perror("opening link table index");
-               exit(1);
-           }
-           code = FDH_SIZE(fdP);
-           if (code == -1) {
-               perror("fstat");
-               exit(1);
-           }
-           FDH_REALLYCLOSE(fdP);
-           IH_RELEASE(ih);
-           size += code;
-           if (!dsizeOnly && !saveinodes) {
-               printf("\tLink inode\t= %s (size = %d)\n",
-                      PrintInode(NULL, header.linkTable), (int)code);
-               printf("Total aux volume size = %d\n\n", (int)size);
-           }
-#endif
-           Vauxsize = size;
-           Vauxsize_k = size / 1024;
+    HandleVnodes(vp, vLarge);
+    HandleVnodes(vp, vSmall);
+
+    if (ShowSizes) {
+       volumeTotals.diskused_k = V_diskused(vp);
+       volumeTotals.size_k =
+           volumeTotals.auxsize_k + volumeTotals.vnodesize_k;
+       if (SaveInodes) {
+           PrintingVolumeSizes = 0;    /* print heading again */
        }
+       PrintVolumeSizes(vp);
+    }
+
+  cleanup:
+    if (fd != INVALID_FD) {
        OS_CLOSE(fd);
-       vp = AttachVolume(dp, name, &header);
-       if (!vp) {
-           printf("Volinfo: Error attaching volume header %s\n", name);
-           return;
-       }
     }
-    PrintHeader(vp);
-    if (DumpVnodes) {
-       if (!dsizeOnly && !saveinodes)
-           printf("\nLarge vnodes (directories)\n");
-       PrintVnodes(vp, vLarge);
-       if (!dsizeOnly && !saveinodes) {
-           printf("\nSmall vnodes(files, symbolic links)\n");
-           fflush(stdout);
-       }
-       if (saveinodes)
-           printf("Saving all volume files to current directory ...\n");
-       PrintVnodes(vp, vSmall);
-    }
-    if (dsizeOnly) {
-       totvolsize = Vauxsize_k + Vvnodesize_k;
-       if (saveinodes)
-           printf
-               ("Volume-Id\t  Volsize  Auxsize Inodesize  AVolsize SizeDiff                (VolName)\n");
-       printf("%u\t%9d%9d%10d%10d%9d\t%24s\n", V_id(vp), Vdiskused,
-              Vauxsize_k, Vvnodesize_k, totvolsize, totvolsize - Vdiskused,
-              V_name(vp));
+    if (vp) {
+       DetachVolume(vp);
     }
-    free(vp->header);
-    free(vp);
 }
 
+/**
+ * volinfo program entry
+ */
 int
 main(int argc, char **argv)
 {
     struct cmd_syndesc *ts;
     afs_int32 code;
 
-    ts = cmd_CreateSyntax(NULL, handleit, NULL, "Dump volume's internal state");
-    cmd_AddParm(ts, "-online", CMD_FLAG, CMD_OPTIONAL,
-               "Get info from running fileserver");
-    cmd_AddParm(ts, "-vnode", CMD_FLAG, CMD_OPTIONAL, "Dump vnode info");
-    cmd_AddParm(ts, "-date", CMD_FLAG, CMD_OPTIONAL,
-               "Also dump vnode's mod date");
-    cmd_AddParm(ts, "-inode", CMD_FLAG, CMD_OPTIONAL,
-               "Dump vnode's inode number");
-    cmd_AddParm(ts, "-itime", CMD_FLAG, CMD_OPTIONAL,
-               "Dump special inode's mod times");
-    cmd_AddParm(ts, "-part", CMD_LIST, CMD_OPTIONAL,
-               "AFS partition name (default current partition)");
-    cmd_AddParm(ts, "-volumeid", CMD_LIST, CMD_OPTIONAL, "Volume id");
-    cmd_AddParm(ts, "-header", CMD_FLAG, CMD_OPTIONAL,
-               "Dump volume's header info");
-    cmd_AddParm(ts, "-sizeOnly", CMD_FLAG, CMD_OPTIONAL,
-               "Dump volume's size");
-    cmd_AddParm(ts, "-fixheader", CMD_FLAG, CMD_OPTIONAL,
-               "Try to fix header");
-    cmd_AddParm(ts, "-saveinodes", CMD_FLAG, CMD_OPTIONAL,
-               "Try to save all inodes");
-    cmd_AddParm(ts, "-orphaned", CMD_FLAG, CMD_OPTIONAL,
-               "List all dir/files without a parent");
+    opr_queue_Init(&VnodeScanLists[vLarge]);
+    opr_queue_Init(&VnodeScanLists[vSmall]);
+
+    ts = cmd_CreateSyntax(NULL, handleit, NULL,
+                         "Dump volume's internal state");
+    cmd_AddParmAtOffset(ts, P_ONLINE, "-online", CMD_FLAG, CMD_OPTIONAL,
+                       "Get info from running fileserver");
+    cmd_AddParmAtOffset(ts, P_VNODE, "-vnode", CMD_FLAG, CMD_OPTIONAL,
+                       "Dump vnode info");
+    cmd_AddParmAtOffset(ts, P_DATE, "-date", CMD_FLAG, CMD_OPTIONAL,
+                       "Also dump vnode's mod date");
+    cmd_AddParmAtOffset(ts, P_INODE, "-inode", CMD_FLAG, CMD_OPTIONAL,
+                       "Also dump vnode's inode number");
+    cmd_AddParmAtOffset(ts, P_ITIME, "-itime", CMD_FLAG, CMD_OPTIONAL,
+                       "Dump special inode's mod times");
+    cmd_AddParmAtOffset(ts, P_PART, "-part", CMD_LIST, CMD_OPTIONAL,
+                       "AFS partition name or id (default current partition)");
+    cmd_AddParmAtOffset(ts, P_VOLUMEID, "-volumeid", CMD_LIST, CMD_OPTIONAL,
+                       "Volume id");
+    cmd_AddParmAtOffset(ts, P_HEADER, "-header", CMD_FLAG, CMD_OPTIONAL,
+                       "Dump volume's header info");
+    cmd_AddParmAtOffset(ts, P_SIZEONLY, "-sizeonly", CMD_FLAG, CMD_OPTIONAL,
+                       "Dump volume's size");
+    cmd_AddParmAtOffset(ts, P_FIXHEADER, "-fixheader", CMD_FLAG,
+                       CMD_OPTIONAL, "Try to fix header");
+    cmd_AddParmAtOffset(ts, P_SAVEINODES, "-saveinodes", CMD_FLAG,
+                       CMD_OPTIONAL, "Try to save all inodes");
+    cmd_AddParmAtOffset(ts, P_ORPHANED, "-orphaned", CMD_FLAG, CMD_OPTIONAL,
+                       "List all dir/files without a parent");
 #if defined(AFS_NAMEI_ENV)
-    cmd_AddParm(ts, "-filenames", CMD_FLAG, CMD_OPTIONAL, "Print filenames");
+    cmd_AddParmAtOffset(ts, P_FILENAMES, "-filenames", CMD_FLAG,
+                       CMD_OPTIONAL, "Also dump vnode's namei filename");
 #endif
+
+    /* For compatibility with older versions. */
+    cmd_AddParmAlias(ts, P_SIZEONLY, "-sizeOnly");
+
     code = cmd_Dispatch(argc, argv);
     return code;
 }
 
-#define typestring(type) (type == RWVOL? "read/write": type == ROVOL? "readonly": type == BACKVOL? "backup": "unknown")
+/**
+ * Return a display string for the volume type.
+ *
+ * @param[in]  type  volume type
+ *
+ * @return volume type description string
+ */
+static_inline char *
+volumeTypeString(int type)
+{
+    return
+       (type == RWVOL ? "read/write" :
+        (type == ROVOL ? "readonly" :
+         (type == BACKVOL ? "backup" : "unknown")));
+}
 
+/**
+ * Print the volume header information
+ *
+ * @param[in]  volume object
+ *
+ * @return none
+ */
 void
 PrintHeader(Volume * vp)
 {
-    Vdiskused = V_diskused(vp);
-    if (dsizeOnly || saveinodes)
-       return;
     printf("Volume header for volume %u (%s)\n", V_id(vp), V_name(vp));
     printf("stamp.magic = %x, stamp.version = %u\n", V_stamp(vp).magic,
           V_stamp(vp).version);
@@ -651,7 +1062,7 @@ PrintHeader(Volume * vp)
         V_dontSalvage(vp));
     printf
        ("type = %d (%s), uniquifier = %u, needsCallback = %d, destroyMe = %x\n",
-        V_type(vp), typestring(V_type(vp)), V_uniquifier(vp),
+        V_type(vp), volumeTypeString(V_type(vp)), V_uniquifier(vp),
         V_needsCallback(vp), V_destroyMe(vp));
     printf
        ("id = %u, parentId = %u, cloneId = %u, backupId = %u, restoredFromId = %u\n",
@@ -677,89 +1088,183 @@ PrintHeader(Volume * vp)
     printf("volUpdateCounter = %u\n", V_volUpCounter(vp));
 }
 
-/* GetFileInfo
- * OS independent file info. Die on failure.
+/**
+ * Get the size and times of a file
+ *
+ * @param[in]  fd     file descriptor of file to stat
+ * @param[out] size   size of the file
+ * @param[out] ctime  ctime of file as a formatted string
+ * @param[out] mtime  mtime of file as a formatted string
+ * @param[out] atime  atime of file as a formatted string
+ *
+ * @return error code
+ *   @retval 0 success
+ *   @retval -1 failed to retrieve file information
  */
-#ifdef AFS_NT40_ENV
-char *
-NT_date(FILETIME * ft)
-{
-    static char result[8][64];
-    static int next = 0;
-    SYSTEMTIME st;
-    FILETIME lft;
-
-    if (!FileTimeToLocalFileTime(ft, &lft)
-       || !FileTimeToSystemTime(&lft, &st)) {
-       printf("Time conversion failed.\n");
-       exit(1);
-    }
-    sprintf(result[next = ((next + 1) & 7)], "%4d/%02d/%02d.%2d:%2d:%2d",
-           st.wYear, st.wMonth, st.wDay, st.wHour, st.wMinute, st.wSecond);
-    return result[next];
-}
-#endif
-
-static void
-GetFileInfo(FD_t fd, int *size, char **ctime, char **mtime, char **atime)
+static int
+GetFileInfo(FD_t fd, afs_sfsize_t * size, char **ctime, char **mtime,
+           char **atime)
 {
 #ifdef AFS_NT40_ENV
     BY_HANDLE_FILE_INFORMATION fi;
+    LARGE_INTEGER fsize;
     if (!GetFileInformationByHandle(fd, &fi)) {
-       printf("GetFileInformationByHandle failed, exiting\n");
-       exit(1);
+       fprintf(stderr, "%s: GetFileInformationByHandle failed\n", progname);
+       return -1;
     }
-    *size = (int)fi.nFileSizeLow;
+    if (!GetFileSizeEx(fd, &fsize)) {
+       fprintf(stderr, "%s: GetFileSizeEx failed\n", progname);
+       return -1;
+    }
+    *size = fsize.QuadPart;
     *ctime = "N/A";
     *mtime = NT_date(&fi.ftLastWriteTime);
     *atime = NT_date(&fi.ftLastAccessTime);
 #else
     struct afs_stat_st status;
     if (afs_fstat(fd, &status) == -1) {
-       printf("fstat failed %d\n", errno);
-       exit(1);
+       fprintf(stderr, "%s: fstat failed %d\n", progname, errno);
+       return -1;
     }
-    *size = (int)status.st_size;
+    *size = status.st_size;
     *ctime = date(status.st_ctime);
     *mtime = date(status.st_mtime);
     *atime = date(status.st_atime);
 #endif
+    return 0;
+}
+
+/**
+ * Copy the inode data to a file in the current directory.
+ *
+ * @param[in] vdp     vnode details object
+ *
+ * @return none
+ */
+static void
+SaveInode(struct VnodeDetails *vdp)
+{
+    IHandle_t *ih;
+    FdHandle_t *fdP;
+    char nfile[50], buffer[256];
+    int ofd = 0;
+    afs_foff_t total;
+    ssize_t len;
+    Inode ino = VNDISK_GET_INO(vdp->vnode);
+
+    if (!VALID_INO(ino)) {
+       return;
+    }
+
+    IH_INIT(ih, V_device(vdp->vp), V_parentId(vdp->vp), ino);
+    fdP = IH_OPEN(ih);
+    if (fdP == NULL) {
+       fprintf(stderr,
+               "%s: Can't open inode %s error %d (ignored)\n",
+               progname, PrintInode(NULL, ino), errno);
+       return;
+    }
+    snprintf(nfile, sizeof nfile, "TmpInode.%s", PrintInode(NULL, ino));
+    ofd = afs_open(nfile, O_CREAT | O_RDWR | O_TRUNC, 0600);
+    if (ofd < 0) {
+       fprintf(stderr,
+               "%s: Can't create file %s; error %d (ignored)\n",
+               progname, nfile, errno);
+
+       FDH_REALLYCLOSE(fdP);
+       IH_RELEASE(ih);
+       return;
+    }
+    total = 0;
+    while (1) {
+       ssize_t nBytes;
+       len = FDH_PREAD(fdP, buffer, sizeof(buffer), total);
+       if (len < 0) {
+           FDH_REALLYCLOSE(fdP);
+           IH_RELEASE(ih);
+           close(ofd);
+           unlink(nfile);
+           fprintf(stderr,
+                   "%s: Error while reading from inode %s (%d)\n",
+                   progname, PrintInode(NULL, ino), errno);
+           return;
+       }
+       if (len == 0)
+           break;              /* No more input */
+       nBytes = write(ofd, buffer, len);
+       if (nBytes != len) {
+           FDH_REALLYCLOSE(fdP);
+           IH_RELEASE(ih);
+           close(ofd);
+           unlink(nfile);
+           fprintf(stderr,
+                   "%s: Error while writing to \"%s\" (%d - ignored)\n",
+                   progname, nfile, errno);
+           return;
+       }
+       total += len;
+    }
+
+    FDH_REALLYCLOSE(fdP);
+    IH_RELEASE(ih);
+    close(ofd);
+    printf("... Copied inode %s to file %s (%lu bytes)\n",
+          PrintInode(NULL, ino), nfile, (unsigned long)total);
 }
 
+/**
+ * Scan a volume index and handle each vnode
+ *
+ * @param[in] vp      volume object
+ * @param[in] class   which index to scan
+ *
+ * @return none
+ */
 void
-PrintVnodes(Volume * vp, VnodeClass class)
+HandleVnodes(Volume * vp, VnodeClass class)
 {
     afs_int32 diskSize =
        (class == vSmall ? SIZEOF_SMALLDISKVNODE : SIZEOF_LARGEDISKVNODE);
     char buf[SIZEOF_LARGEDISKVNODE];
     struct VnodeDiskObject *vnode = (struct VnodeDiskObject *)buf;
-    StreamHandle_t *file;
-    int vnodeIndex, nVnodes;
+    StreamHandle_t *file = NULL;
+    int vnodeIndex;
+    afs_sfsize_t nVnodes;
     afs_foff_t offset = 0;
-    Inode ino;
     IHandle_t *ih = vp->vnodeIndex[class].handle;
-    FdHandle_t *fdP;
-    int size;
+    FdHandle_t *fdP = NULL;
+    afs_sfsize_t size;
     char *ctime, *atime, *mtime;
-    char nfile[50], buffer[256];
-    int ofd, bad = 0;
-    afs_foff_t total;
-    ssize_t len;
+    struct opr_queue *scanList = &VnodeScanLists[class];
+    struct opr_queue *cursor;
+
+    if (opr_queue_IsEmpty(scanList)) {
+       return;
+    }
+
+    for (opr_queue_Scan(scanList, cursor)) {
+       struct VnodeScanProc *entry = (struct VnodeScanProc *)cursor;
+       if (entry->heading) {
+           printf(entry->heading);
+       }
+    }
 
     fdP = IH_OPEN(ih);
     if (fdP == NULL) {
-       printf("open failed\n");
-       exit(1);
+       fprintf(stderr, "%s: open failed: ", progname);
+       goto error;
     }
 
     file = FDH_FDOPEN(fdP, "r");
     if (!file) {
-       printf("fdopen failed\n");
-       exit(1);
+       fprintf(stderr, "%s: fdopen failed\n", progname);
+       goto error;
     }
 
-    GetFileInfo(fdP->fd_fd, &size, &ctime, &atime, &mtime);
-    if (InodeTimes && !dsizeOnly) {
+    if (GetFileInfo(fdP->fd_fd, &size, &ctime, &atime, &mtime) != 0) {
+       goto error;
+    }
+    if (InodeTimes) {
        printf("ichanged : %s\nimodified: %s\niaccessed: %s\n\n", ctime,
               mtime, atime);
     }
@@ -774,114 +1279,72 @@ PrintVnodes(Volume * vp, VnodeClass class)
         nVnodes && STREAM_READ(vnode, diskSize, 1, file) == 1;
         nVnodes--, vnodeIndex++, offset += diskSize) {
 
-       ino = VNDISK_GET_INO(vnode);
-       if (saveinodes) {
-           if (VALID_INO(ino) && (class == vSmall)) {
-               IHandle_t *ih1;
-               FdHandle_t *fdP1;
-               IH_INIT(ih1, V_device(vp), V_parentId(vp), ino);
-               fdP1 = IH_OPEN(ih1);
-               if (fdP1 == NULL) {
-                   printf("Can't open inode %s error %d (ignored)\n",
-                          PrintInode(NULL, ino), errno);
-                   continue;
-               }
-               snprintf(nfile, sizeof nfile, "TmpInode.%s",
-                        PrintInode(NULL, ino));
-               ofd = afs_open(nfile, O_CREAT | O_RDWR | O_TRUNC, 0600);
-               if (ofd < 0) {
-                   printf("Can't create file %s; error %d (ignored)\n",
-                          nfile, errno);
-                   continue;
-               }
-               total = bad = 0;
-               while (1) {
-                   ssize_t nBytes;
-                   len = FDH_PREAD(fdP1, buffer, sizeof(buffer), total);
-                   if (len < 0) {
-                       FDH_REALLYCLOSE(fdP1);
-                       IH_RELEASE(ih1);
-                       close(ofd);
-                       unlink(nfile);
-                       printf
-                           ("Error while reading from inode %s (%d - ignored)\n",
-                            PrintInode(NULL, ino), errno);
-                       bad = 1;
-                       break;
-                   }
-                   if (len == 0)
-                       break;  /* No more input */
-                   nBytes = write(ofd, buffer, len);
-                   if (nBytes != len) {
-                       FDH_REALLYCLOSE(fdP1);
-                       IH_RELEASE(ih1);
-                       close(ofd);
-                       unlink(nfile);
-                       printf
-                           ("Error while writing to \"%s\" (%d - ignored)\n",
-                            nfile, errno);
-                       bad = 1;
-                       break;
-                   }
-                   total += len;
-               }
-               if (bad)
-                   continue;
-               FDH_REALLYCLOSE(fdP1);
-               IH_RELEASE(ih1);
-               close(ofd);
-               printf("... Copied inode %s to file %s (%lu bytes)\n",
-                      PrintInode(NULL, ino), nfile, (unsigned long)total);
+       struct VnodeDetails vnodeDetails;
+
+       vnodeDetails.vp = vp;
+       vnodeDetails.class = class;
+       vnodeDetails.vnode = vnode;
+       vnodeDetails.vnodeNumber = bitNumberToVnodeNumber(vnodeIndex, class);
+       vnodeDetails.offset = offset;
+       vnodeDetails.index = vnodeIndex;
+
+       for (opr_queue_Scan(scanList, cursor)) {
+           struct VnodeScanProc *entry = (struct VnodeScanProc *)cursor;
+           if (entry->proc) {
+               (*entry->proc) (&vnodeDetails);
            }
-       } else {
-#if defined(AFS_NAMEI_ENV)
-           PrintVnode(offset, vnode,
-                      bitNumberToVnodeNumber(vnodeIndex, class), ino, vp);
-#else
-           PrintVnode(offset, vnode,
-                      bitNumberToVnodeNumber(vnodeIndex, class), ino);
-#endif
        }
     }
-    STREAM_CLOSE(file);
-    FDH_CLOSE(fdP);
+
+  error:
+    if (file) {
+       STREAM_CLOSE(file);
+    }
+    if (fdP) {
+       FDH_CLOSE(fdP);
+    }
 }
 
-#if defined(AFS_NAMEI_ENV)
-void
-PrintVnode(afs_foff_t offset, VnodeDiskObject * vnode, VnodeId vnodeNumber,
-          Inode ino, Volume * vp)
-#else
+/**
+ * Print vnode information
+ *
+ * @param[in] vdp          vnode details object
+ *
+ * @return none
+ */
 void
-PrintVnode(afs_foff_t offset, VnodeDiskObject * vnode, VnodeId vnodeNumber,
-          Inode ino)
-#endif
+PrintVnode(struct VnodeDetails *vdp)
 {
 #if defined(AFS_NAMEI_ENV)
     IHandle_t *ihtmpp;
     namei_t filename;
 #endif
+    afs_foff_t offset = vdp->offset;
+    VnodeDiskObject *vnode = vdp->vnode;
     afs_fsize_t fileLength;
+    Inode ino;
 
+    ino = VNDISK_GET_INO(vnode);
     VNDISK_GET_LEN(fileLength, vnode);
-    Vvnodesize += fileLength;
-    Vvnodesize_k += fileLength / 1024;
-    if (dsizeOnly)
-       return;
-    if (orphaned && (fileLength == 0 || vnode->parent || !offset))
+
+    /* The check for orphaned vnodes is currently limited to non-empty
+     * vnodes with a parent of zero (and which are not the first entry
+     * in the index). */
+    if (ShowOrphaned && (fileLength == 0 || vnode->parent || !offset))
        return;
+
     printf
        ("%10lld Vnode %u.%u.%u cloned: %u, length: %llu linkCount: %d parent: %u",
-        (long long)offset, vnodeNumber, vnode->uniquifier, vnode->dataVersion,
-        vnode->cloned, (afs_uintmax_t) fileLength, vnode->linkCount,
-        vnode->parent);
+        (long long)offset, vdp->vnodeNumber, vnode->uniquifier,
+        vnode->dataVersion, vnode->cloned, (afs_uintmax_t) fileLength,
+        vnode->linkCount, vnode->parent);
     if (DumpInodeNumber)
        printf(" inode: %s", PrintInode(NULL, ino));
     if (DumpDate)
        printf(" ServerModTime: %s", date(vnode->serverModifyTime));
 #if defined(AFS_NAMEI_ENV)
     if (PrintFileNames) {
-       IH_INIT(ihtmpp, V_device(vp), V_parentId(vp), ino);
+       IH_INIT(ihtmpp, V_device(vdp->vp), V_parentId(vdp->vp), ino);
        namei_HandleToName(&filename, ihtmpp);
 #if !defined(AFS_NT40_ENV)
        printf(" UFS-Filename: %s", filename.n_path);