vol: Avoid FDH_SEEK/FDH_READ
[openafs.git] / src / vol / vol-info.c
index 64ced42..22ac94a 100644 (file)
@@ -29,6 +29,9 @@
 #include <afs/dir.h>
 #include <afs/afsint.h>
 #include <afs/errors.h>
+#include <afs/acl.h>
+#include <afs/prs_fs.h>
+#include <rx/rx_queue.h>
 
 #include "nfs.h"
 #include "lock.h"
 #include "vnode.h"
 #include "volume.h"
 #include "partition.h"
+#include "salvage.h"
+#include "daemon_com_inline.h"
+#include "fssync_inline.h"
+#include "vol-info.h"
+
+static int volinfo_init = 0;
+static const char *progname = "";
+static const char *PLACEHOLDER = "-";
+
+
+/* Numeric column type ids */
+typedef enum columnType {
+#define c(x) col_##x,
+    VOLSCAN_COLUMNS
+#undef c
+    max_column_type
+} columnType;
+
+struct columnName {
+    columnType type;
+    const char *name;
+};
 
-#ifndef AFS_NT40_ENV
-#include "AFS_component_version_number.c"
-#endif
+/* Table of id:name tuples of possible columns. */
+struct columnName ColumnName[] = {
+#define c(x) { col_##x, #x },
+    VOLSCAN_COLUMNS
+#undef c
+    {max_column_type, NULL}
+};
 
-static const char *progname = "volinfo";
 
-/* Command line options */
+/* VnodeDetails union descriminator */
 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;
-
-/* Modes */
-static int DumpInfo = 1;            /**< Dump volume information, defualt mode*/
-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)
-static int PrintFileNames = 0;      /**< Dump vnode and special file name filenames */
-#endif
-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. */
+    VNODE_U_NONE,
+    VNODE_U_MOUNT,
+    VNODE_U_SYMLINK,
+    VNODE_U_POS_ACCESS,
+    VNODE_U_NEG_ACCESS
+} vnode_details_u_t;
+
+struct VnodeDetails {
+    Volume *vp;
+    VnodeClass class;
+    VnodeDiskObject *vnode;
+    VnodeId vnodeNumber;
+    afs_foff_t offset;
+    int index;
+    char *path;
+    vnode_details_u_t t;
+    union {
+       struct {
+           char type;
+           char *cell;
+           char *vol;
+       } mnt;
+       char *target;
+       struct acl_accessEntry *access;
+    } u;
+};
+
+
+static int NeedDirIndex;        /**< Large vnode index handle is needed for path lookups. */
+static FdHandle_t *DirIndexFd = NULL; /**< Current large vnode index handle for path lookups. */
+
+static int NumOutputColumns = 0;
+static columnType OutputColumn[max_column_type];
 
 /**
  * Volume size running totals
@@ -92,17 +124,33 @@ 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 VolInfoOpt *opt, struct VnodeDetails * vdp);
+};
+static struct opr_queue VnodeScanLists[nVNODECLASSES];
+
 /* Forward Declarations */
-void PrintHeader(Volume * vp);
-void HandleAllPart(void);
-void HandlePart(struct DiskPartition64 *partP);
-void HandleVolume(struct DiskPartition64 *partP, char *name);
-struct DiskPartition64 *FindCurrentPartition(void);
-Volume *AttachVolume(struct DiskPartition64 *dp, char *volname,
+static void PrintHeader(Volume * vp);
+static void HandleAllPart(struct VolInfoOpt *opt);
+static void HandlePart(struct VolInfoOpt *opt, struct DiskPartition64 *partP);
+static void HandleVolume(struct VolInfoOpt *opt, struct DiskPartition64 *partP, char *name);
+static struct DiskPartition64 *FindCurrentPartition(void);
+static Volume *AttachVolume(struct VolInfoOpt *opt, struct DiskPartition64 *dp, char *volname,
                     struct VolumeHeader *header);
-void PrintVnode(afs_foff_t offset, VnodeDiskObject * vnode, VnodeId vnodeNumber,
-               Inode ino, Volume * vp);
-void HandleVnodes(Volume * vp, VnodeClass class);
+static void HandleVnodes(struct VolInfoOpt *opt, Volume * vp, VnodeClass class);
+static void PrintColumnHeading(struct VolInfoOpt *opt);
+static void PrintColumns(struct VolInfoOpt *opt, struct VnodeDetails *vdp, const char *desc);
+
+/* externs */
+extern void SetSalvageDirHandle(DirHandle * dir, afs_int32 volume,
+                               Device device, Inode inode,
+                               int *volumeChanged);
+extern void FidZap(DirHandle * file);
 
 /**
  * Format time as a timestamp string
@@ -116,7 +164,7 @@ void HandleVnodes(Volume * vp, VnodeClass class);
  *       caller must not reference the returned string after
  *       seven additional calls to this function.
  */
-char *
+static char *
 date(time_t date)
 {
 #define MAX_DATE_RESULT        100
@@ -144,7 +192,7 @@ date(time_t date)
  *       caller must not reference the returned string after
  *       seven additional calls to this function.
  */
-char *
+static char *
 NT_date(FILETIME * ft)
 {
     static char result[8][64];
@@ -164,6 +212,25 @@ NT_date(FILETIME * ft)
 #endif
 
 /**
+ * Add vnode size to the running volume totals.
+ *
+ * @param[in]  vdp   vnode details object
+ *
+ * @return none
+ */
+void
+volinfo_AddVnodeToSizeTotals(struct VolInfoOpt *opt, struct VnodeDetails *vdp)
+{
+    afs_fsize_t fileLength;
+
+    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
@@ -205,8 +272,8 @@ 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),
+    printf("%" AFS_VOLID_FMT "\t%9llu%9llu%10llu%10llu%9lld\t%24s\n",
+          afs_printable_VolumeId_lu(V_id(vp)),
           volumeTotals.diskused_k,
           volumeTotals.auxsize_k, volumeTotals.vnodesize_k,
           volumeTotals.size_k, diff_k, V_name(vp));
@@ -257,8 +324,9 @@ PrintServerTotals(void)
  *   @retval 0 success
  *   @retval -1 failed to read file
  */
-int
-ReadHdr1(IHandle_t * ih, char *to, int size, u_int magic, u_int version)
+static int
+ReadHdr1(struct VolInfoOpt *opt, IHandle_t * ih, char *to, afs_sfsize_t size,
+        u_int magic, u_int version)
 {
     struct versionStamp *vsn;
     int bad = 0;
@@ -283,7 +351,7 @@ ReadHdr1(IHandle_t * ih, char *to, int size, u_int magic, u_int version)
                progname,
                PrintInode(NULL, ih->ih_ino), vsn->version, version);
     }
-    if (bad && FixHeader) {
+    if (bad && opt->fixHeader) {
        vsn->magic = magic;
        vsn->version = version;
        printf("Special index inode %s has a bad header. Reconstructing...\n",
@@ -295,7 +363,7 @@ ReadHdr1(IHandle_t * ih, char *to, int size, u_int magic, u_int version)
                    progname, PrintInode(NULL, ih->ih_ino));
        }
     }
-    if (!bad && DumpInfo) {
+    if (!bad && opt->dumpInfo) {
        printf("Inode %s: Good magic %x and version %x\n",
               PrintInode(NULL, ih->ih_ino), magic, version);
     }
@@ -311,13 +379,37 @@ ReadHdr1(IHandle_t * ih, char *to, int size, u_int magic, u_int version)
  *
  * @return volume object or null on error
  */
-Volume *
-AttachVolume(struct DiskPartition64 * dp, char *volname,
-            struct VolumeHeader * header)
+static Volume *
+AttachVolume(struct VolInfoOpt *opt, struct DiskPartition64 *dp,
+            char *volname, struct VolumeHeader *header)
 {
     Volume *vp;
     afs_int32 ec = 0;
 
+    if (opt->checkout) {
+       afs_int32 code;
+       SYNC_response response;
+       VolumeId volid = header->id;
+
+       memset(&response, 0, sizeof(response));
+       code =
+           FSYNC_VolOp(volid, dp->name, FSYNC_VOL_NEEDVOLUME, V_DUMP,
+                       &response);
+       if (code != SYNC_OK) {
+           if (response.hdr.reason == FSYNC_SALVAGE) {
+               fprintf(stderr,
+                       "%s: file server says volume %lu is salvaging.\n",
+                       progname, afs_printable_uint32_lu(volid));
+               return NULL;
+           } else {
+               fprintf(stderr,
+                       "%s: attach of volume %lu denied by file server.\n",
+                       progname, afs_printable_uint32_lu(volid));
+               return NULL;
+           }
+       }
+    }
+
     vp = (Volume *) calloc(1, sizeof(Volume));
     if (!vp) {
        fprintf(stderr, "%s: Failed to allocate volume object.\n", progname);
@@ -337,28 +429,28 @@ AttachVolume(struct DiskPartition64 * dp, char *volname,
     vp->shuttingDown = 0;
     vp->goingOffline = 0;
     vp->nUsers = 1;
-    vp->header = (struct volHeader *)calloc(1, sizeof(*vp->header));
+    vp->header = 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),
+    ec = ReadHdr1(opt, V_diskDataHandle(vp), (char *)&V_disk(vp),
                  sizeof(V_disk(vp)), VOLUMEINFOMAGIC, VOLUMEINFOVERSION);
     if (!ec) {
        struct IndexFileHeader iHead;
-       ec = ReadHdr1(vp->vnodeIndex[vSmall].handle, (char *)&iHead,
+       ec = ReadHdr1(opt, vp->vnodeIndex[vSmall].handle, (char *)&iHead,
                      sizeof(iHead), SMALLINDEXMAGIC, SMALLINDEXVERSION);
     }
     if (!ec) {
        struct IndexFileHeader iHead;
-       ec = ReadHdr1(vp->vnodeIndex[vLarge].handle, (char *)&iHead,
+       ec = ReadHdr1(opt, vp->vnodeIndex[vLarge].handle, (char *)&iHead,
                      sizeof(iHead), LARGEINDEXMAGIC, LARGEINDEXVERSION);
     }
 #ifdef AFS_NAMEI_ENV
     if (!ec) {
        struct versionStamp stamp;
-       ec = ReadHdr1(V_linkHandle(vp), (char *)&stamp, sizeof(stamp),
+       ec = ReadHdr1(opt, V_linkHandle(vp), (char *)&stamp, sizeof(stamp),
                      LINKTABLEMAGIC, LINKTABLEVERSION);
     }
 #endif
@@ -375,8 +467,27 @@ AttachVolume(struct DiskPartition64 * dp, char *volname,
  * @return none
  */
 static void
-DetachVolume(Volume * vp)
+DetachVolume(struct VolInfoOpt *opt, Volume * vp)
 {
+    if (opt->checkout) {
+       afs_int32 code;
+       SYNC_response response;
+       memset(&response, 0, sizeof(response));
+
+       code = FSYNC_VolOp(V_id(vp), V_partition(vp)->name,
+                          FSYNC_VOL_ON, FSYNC_WHATEVER, &response);
+       if (code != SYNC_OK) {
+           fprintf(stderr, "%s: FSSYNC error %d (%s)\n", progname, code,
+                   SYNC_res2string(code));
+           fprintf(stderr, "%s:  protocol response code was %d (%s)\n",
+                   progname, response.hdr.response,
+                   SYNC_res2string(response.hdr.response));
+           fprintf(stderr, "%s:  protocol reason code was %d (%s)\n",
+                   progname, response.hdr.reason,
+                   FSYNC_reason2string(response.hdr.reason));
+       }
+    }
+
     IH_RELEASE(vp->vnodeIndex[vLarge].handle);
     IH_RELEASE(vp->vnodeIndex[vSmall].handle);
     IH_RELEASE(vp->diskDataHandle);
@@ -400,7 +511,7 @@ DetachVolume(Volume * vp)
 static int
 GetPartitionName(afs_uint32 partId, char *partName, afs_size_t partNameSize)
 {
-    const int OLD_NUM_DEVICES = 26;    /* a..z */
+    const afs_uint32 OLD_NUM_DEVICES = 26;     /* a..z */
 
     if (partId < OLD_NUM_DEVICES) {
        if (partNameSize < 8) {
@@ -426,90 +537,35 @@ GetPartitionName(afs_uint32 partId, char *partName, afs_size_t partNameSize)
 }
 
 /**
- * Process command line options and start scanning
+ * Scan the volumes in the partitions
  *
- * @param[in] as     command syntax object
- * @param[in] arock  opaque object; not used
+ * 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.
  *
- * @return error code
+ * @param[in] partNameOrId   partition name or id to be scannned
+ * @param[in] volumeId       volume id to be scanned
+ *
+ * @return 0 if successful
  */
-static int
-handleit(struct cmd_syndesc *as, void *arock)
+int
+volinfo_ScanPartitions(struct VolInfoOpt *opt, char *partNameOrId, VolumeId volumeId)
 {
-    struct cmd_item *ti;
     int err = 0;
-    afs_uint32 volumeId = 0;
-    char *partNameOrId = 0;
     char partName[64] = "";
     struct DiskPartition64 *partP = NULL;
 
-
-#ifndef AFS_NT40_ENV
-    if (geteuid() != 0) {
-       fprintf(stderr, "%s: Must be run as root; sorry\n", progname);
-       return 1;
-    }
-#endif
-
-    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;
+    if (opt->checkout) {
+       if (!FSYNC_clientInit()) {
+           fprintf(stderr, "%s: Failed to connect to fileserver.\n",
+                   progname);
+           return 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
-    }
 
     /* Allow user to specify partition by name or id. */
     if (partNameOrId) {
@@ -517,22 +573,24 @@ handleit(struct cmd_syndesc *as, void *arock)
        if (partId == -1) {
            fprintf(stderr, "%s: Could not parse '%s' as a partition name.\n",
                    progname, partNameOrId);
-           return 1;
+           err = 1;
+           goto cleanup;
        }
        if (GetPartitionName(partId, partName, sizeof(partName))) {
            fprintf(stderr,
                    "%s: Could not format '%s' as a partition name.\n",
                    progname, partNameOrId);
-           return 1;
+           err = 1;
+           goto cleanup;
        }
     }
 
-    DInit(10);
-
     err = VAttachPartitions();
     if (err) {
        fprintf(stderr, "%s: %d partitions had errors during attach.\n",
                progname, err);
+       err = 1;
+       goto cleanup;
     }
 
     if (partName[0]) {
@@ -541,15 +599,19 @@ handleit(struct cmd_syndesc *as, void *arock)
            fprintf(stderr,
                    "%s: %s is not an AFS partition name on this server.\n",
                    progname, partName);
-           return 1;
+           err = 1;
+           goto cleanup;
        }
     }
 
     if (!volumeId) {
+       if (opt->printHeading) {
+           PrintColumnHeading(opt);
+       }
        if (!partP) {
-           HandleAllPart();
+           HandleAllPart(opt);
        } else {
-           HandlePart(partP);
+           HandlePart(opt, partP);
        }
     } else {
        char name1[128];
@@ -560,24 +622,138 @@ handleit(struct cmd_syndesc *as, void *arock)
                fprintf(stderr,
                        "%s: Current partition is not a vice partition.\n",
                        progname);
-               return 1;
+               err = 1;
+               goto cleanup;
            }
        }
        snprintf(name1, sizeof name1, VFORMAT,
-                afs_printable_uint32_lu(volumeId));
-       HandleVolume(partP, name1);
+                afs_printable_VolumeId_lu(volumeId));
+       if (opt->printHeading) {
+           PrintColumnHeading(opt);
+       }
+       HandleVolume(opt, partP, name1);
+    }
+
+  cleanup:
+    if (opt->checkout) {
+       FSYNC_clientFinis();
+    }
+    return err;
+}
+
+/**
+ * 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
+ */
+void
+volinfo_AddVnodeHandler(int vnodeClass,
+               void (*proc) (struct VolInfoOpt * opt,
+                             struct VnodeDetails * vdp), const char *heading)
+{
+    struct VnodeScanProc *entry = malloc(sizeof(struct VnodeScanProc));
+    entry->proc = proc;
+    entry->heading = heading;
+    opr_queue_Append(&VnodeScanLists[vnodeClass], (struct opr_queue *)entry);
+}
+
+/**
+ * Initialize the vol-info module.
+ */
+int
+volinfo_Init(const char *aprogname)
+{
+    if (!volinfo_init) {
+       progname = aprogname;  /* for error messages */
+#ifndef AFS_NT40_ENV
+       if (geteuid() != 0) {
+           fprintf(stderr, "%s: Must be run as root.\n", progname);
+           return EPERM;
+       }
+#endif
+       DInit(1024);
+       VInitVnodes(vLarge, 0);
+       VInitVnodes(vSmall, 0);
+       opr_queue_Init(&VnodeScanLists[vLarge]);
+       opr_queue_Init(&VnodeScanLists[vSmall]);
+        volinfo_init = 1;
     }
     return 0;
 }
 
 /**
+ * Alloc an options structure with default values.
+ *
+ * The caller must free the options pointer after use.
+ *
+ * @param[out] optp address of options pointer
+ * @return 0 on success
+ */
+int
+volinfo_Options(struct VolInfoOpt **optp)
+{
+    struct VolInfoOpt *opt;
+
+    opt = calloc(1, sizeof(struct VolInfoOpt));
+    if (!opt) {
+       fprintf(stderr, "%s: Failed to allocate options (%d).\n", progname, ENOMEM);
+        return ENOMEM;
+    }
+    opt->dumpInfo = 1;
+    gethostname(opt->hostname, sizeof(opt->hostname));
+    opt->columnDelim[0] = ' ';
+    opt->checkMagic = 1;
+    *optp = opt;
+    return 0;
+}
+
+/**
+ * Add a column type to be displayed.
+ *
+ * @param[in]  name   column type name
+ *
+ * @return error code
+ *   @retval 0 success
+ *   @retval 1 too many columns
+ *   @retval 2 invalid column name
+ */
+int
+volinfo_AddOutputColumn(char *name)
+{
+    int i;
+
+    if (NumOutputColumns >= sizeof(OutputColumn) / sizeof(*OutputColumn)) {
+       fprintf(stderr, "%s: Too many output columns (%d).\n", progname,
+               NumOutputColumns);
+       return 1;
+    }
+    for (i = 0; i < max_column_type; i++) {
+       if (!strcmp(ColumnName[i].name, name)) {
+           columnType t = ColumnName[i].type;
+           OutputColumn[NumOutputColumns++] = t;
+
+           if (t == col_path) {
+               NeedDirIndex = 1;
+           }
+           return 0;
+       }
+    }
+    return 2;
+}
+
+
+/**
  * Determine if the current directory is a vice partition
  *
  * @return disk partition object
  */
 #ifdef AFS_NT40_ENV
 #include <direct.h>
-struct DiskPartition64 *
+static struct DiskPartition64 *
 FindCurrentPartition(void)
 {
     int dr = _getdrive();
@@ -595,7 +771,7 @@ FindCurrentPartition(void)
     return dp;
 }
 #else
-struct DiskPartition64 *
+static struct DiskPartition64 *
 FindCurrentPartition(void)
 {
     char partName[1024];
@@ -630,21 +806,22 @@ FindCurrentPartition(void)
  *
  * @return none
  */
-void
-HandleAllPart(void)
+static void
+HandleAllPart(struct VolInfoOpt *opt)
 {
     struct DiskPartition64 *partP;
 
-
     for (partP = DiskPartitionList; partP; partP = partP->next) {
-       printf("Processing Partition %s:\n", partP->name);
-       HandlePart(partP);
-       if (ShowSizes) {
+       if (opt->dumpInfo || opt->saveInodes || opt->showSizes) {
+           printf("Processing Partition %s:\n", partP->name);
+       }
+       HandlePart(opt, partP);
+       if (opt->showSizes) {
            AddSizeTotals(&serverTotals, &partitionTotals);
        }
     }
 
-    if (ShowSizes) {
+    if (opt->showSizes) {
        PrintServerTotals();
     }
 }
@@ -656,8 +833,8 @@ HandleAllPart(void)
  *
  * @return none
  */
-void
-HandlePart(struct DiskPartition64 *partP)
+static void
+HandlePart(struct VolInfoOpt *opt, struct DiskPartition64 *partP)
 {
     afs_int64 nvols = 0;
     DIR *dirp;
@@ -678,15 +855,15 @@ HandlePart(struct DiskPartition64 *partP)
     while ((dp = readdir(dirp))) {
        p = (char *)strrchr(dp->d_name, '.');
        if (p != NULL && strcmp(p, VHDREXT) == 0) {
-           HandleVolume(partP, dp->d_name);
-           if (ShowSizes) {
+           HandleVolume(opt, partP, dp->d_name);
+           if (opt->showSizes) {
                nvols++;
                AddSizeTotals(&partitionTotals, &volumeTotals);
            }
        }
     }
     closedir(dirp);
-    if (ShowSizes) {
+    if (opt->showSizes) {
        PrintPartitionTotals(nvols);
     }
 }
@@ -694,20 +871,20 @@ HandlePart(struct DiskPartition64 *partP)
 /**
  * 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
+ * @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[inout] psize      total of the header special file sizes
  *
  * @return none
  */
-void
-HandleSpecialFile(const char *name, struct DiskPartition64 *dp,
+static void
+HandleSpecialFile(struct VolInfoOpt *opt, const char *name, struct DiskPartition64 *dp,
                  struct VolumeHeader *header, Inode inode,
                  afs_sfsize_t * psize)
 {
-    afs_sfsize_t size = 0;
+    afs_sfsize_t size = -1;
     IHandle_t *ih = NULL;
     FdHandle_t *fdP = NULL;
 #ifdef AFS_NAMEI_ENV
@@ -718,30 +895,35 @@ HandleSpecialFile(const char *name, struct DiskPartition64 *dp,
     fdP = IH_OPEN(ih);
     if (fdP == NULL) {
        fprintf(stderr,
-               "%s: Error opening header file '%s' for volume %u", progname,
-               name, header->id);
+               "%s: Error opening header file '%s' for volume %" AFS_VOLID_FMT "\n", progname,
+               name, afs_printable_VolumeId_lu(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);
+               "%s: Error getting size of header file '%s' for volume %" AFS_VOLID_FMT "\n",
+               progname, name, afs_printable_VolumeId_lu(header->id));
        perror("fstat");
        goto error;
     }
-    if (DumpInfo) {
-       printf("\t%s inode\t= %s (size = %lld)\n",
-              name, PrintInode(NULL, inode), size);
+    *psize += size;
+
+  error:
+    if (opt->dumpInfo) {
+       printf("\t%s inode\t= %s (size = ", name, PrintInode(NULL, inode));
+       if (size != -1) {
+           printf("%lld)\n", size);
+       } else {
+           printf("unknown)\n");
+       }
 #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);
     }
@@ -756,44 +938,75 @@ HandleSpecialFile(const char *name, struct DiskPartition64 *dp,
  * @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,
+static void
+HandleHeaderFiles(struct VolInfoOpt *opt, struct DiskPartition64 *dp, FD_t header_fd,
                  struct VolumeHeader *header)
 {
     afs_sfsize_t size = 0;
 
-    if (DumpInfo) {
+    if (opt->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);
+       printf("\tVolId\t= %" AFS_VOLID_FMT "\n", afs_printable_VolumeId_lu(header->id));
+       printf("\tparent\t= %" AFS_VOLID_FMT "\n", afs_printable_VolumeId_lu(header->parent));
     }
 
-    HandleSpecialFile("Info", dp, header, header->volumeInfo, &size);
-    HandleSpecialFile("Small", dp, header, header->smallVnodeIndex,
+    HandleSpecialFile(opt, "Info", dp, header, header->volumeInfo, &size);
+    HandleSpecialFile(opt, "Small", dp, header, header->smallVnodeIndex,
                      &size);
-    HandleSpecialFile("Large", dp, header, header->largeVnodeIndex,
+    HandleSpecialFile(opt, "Large", dp, header, header->largeVnodeIndex,
                      &size);
 #ifdef AFS_NAMEI_ENV
-    HandleSpecialFile("Link", dp, header, header->linkTable, &size);
+    HandleSpecialFile(opt, "Link", dp, header, header->linkTable, &size);
 #endif /* AFS_NAMEI_ENV */
 
-    if (DumpInfo) {
+    if (opt->dumpInfo) {
        printf("Total aux volume size = %lld\n\n", size);
     }
 
-    if (ShowSizes) {
+    if (opt->showSizes) {
         volumeTotals.auxsize = size;
         volumeTotals.auxsize_k = size / 1024;
     }
 }
 
 /**
+ * Determine if the vnodes of this volume should be scanned.
+ *
+ * @param[in]  vp   volume object
+ *
+ * @return true if vnodes should be scanned
+ */
+static int
+IsScannable(struct VolInfoOpt *opt, Volume * vp)
+{
+    if (opr_queue_IsEmpty(&VnodeScanLists[vLarge]) &&
+       opr_queue_IsEmpty(&VnodeScanLists[vSmall])) {
+       return 0;
+    }
+    if (!opt->scanVolType) {
+       return 1;               /* filtering disabled; do not check vol type */
+    }
+    switch (V_type(vp)) {
+    case RWVOL:
+       return opt->scanVolType & SCAN_RW;
+    case ROVOL:
+       return opt->scanVolType & SCAN_RO;
+    case BACKVOL:
+       return opt->scanVolType & SCAN_BK;
+    default:
+       fprintf(stderr, "%s: Volume %" AFS_VOLID_FMT "; Unknown volume type %d\n", progname,
+               afs_printable_VolumeId_lu(V_id(vp)), V_type(vp));
+       break;
+    }
+    return 0;
+}
+
+/**
  * Attach and scan the volume and handle the header and vnodes
  *
  * Print the volume header and vnode information, depending on the
@@ -804,8 +1017,8 @@ HandleHeaderFiles(struct DiskPartition64 *dp, FD_t header_fd,
  *
  * @return none
  */
-void
-HandleVolume(struct DiskPartition64 *dp, char *name)
+static void
+HandleVolume(struct VolInfoOpt *opt, struct DiskPartition64 *dp, char *name)
 {
     struct VolumeHeader header;
     struct VolumeDiskHeader diskHeader;
@@ -839,30 +1052,45 @@ HandleVolume(struct DiskPartition64 *dp, char *name)
     }
 
     DiskToVolumeHeader(&header, &diskHeader);
-    if (DumpHeader || ShowSizes) {
-       HandleHeaderFiles(dp, fd, &header);
+    if (opt->dumpHeader || opt->showSizes) {
+       HandleHeaderFiles(opt, dp, fd, &header);
     }
 
-    vp = AttachVolume(dp, name, &header);
+    vp = AttachVolume(opt, dp, name, &header);
     if (!vp) {
        fprintf(stderr, "%s: Error attaching volume header %s\n",
                progname, name);
        goto cleanup;
     }
 
-    if (DumpInfo) {
+    if (opt->dumpInfo) {
        PrintHeader(vp);
     }
-    if (DumpVnodes || ShowSizes || ShowOrphaned) {
-       HandleVnodes(vp, vLarge);
-    }
-    if (DumpVnodes || ShowSizes || SaveInodes || ShowOrphaned) {
-       HandleVnodes(vp, vSmall);
+    if (IsScannable(opt, vp)) {
+       if (NeedDirIndex) {
+           IHandle_t *ih = vp->vnodeIndex[vLarge].handle;
+           DirIndexFd = IH_OPEN(ih);
+           if (DirIndexFd == NULL) {
+               fprintf(stderr, "%s: Failed to open index for directories.",
+                       progname);
+           }
+       }
+
+       HandleVnodes(opt, vp, vLarge);
+       HandleVnodes(opt, vp, vSmall);
+
+       if (DirIndexFd) {
+           FDH_CLOSE(DirIndexFd);
+           DirIndexFd = NULL;
+       }
     }
-    if (ShowSizes) {
+    if (opt->showSizes) {
        volumeTotals.diskused_k = V_diskused(vp);
        volumeTotals.size_k =
            volumeTotals.auxsize_k + volumeTotals.vnodesize_k;
+       if (opt->saveInodes) {
+           PrintingVolumeSizes = 0;    /* print heading again */
+       }
        PrintVolumeSizes(vp);
     }
 
@@ -871,58 +1099,40 @@ HandleVolume(struct DiskPartition64 *dp, char *name)
        OS_CLOSE(fd);
     }
     if (vp) {
-       DetachVolume(vp);
+       DetachVolume(opt, vp);
     }
 }
 
 /**
- * volinfo program entry
+ * Return a display string for the volume type.
+ *
+ * @param[in]  type  volume type
+ *
+ * @return volume type description string
  */
-int
-main(int argc, char **argv)
+static_inline char *
+volumeTypeString(int type)
 {
-    struct cmd_syndesc *ts;
-    afs_int32 code;
-
-    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_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;
+    return
+       (type == RWVOL ? "read/write" :
+        (type == ROVOL ? "readonly" :
+         (type == BACKVOL ? "backup" : "unknown")));
 }
 
-#define typestring(type) (type == RWVOL? "read/write": type == ROVOL? "readonly": type == BACKVOL? "backup": "unknown")
+/**
+ * Return a short display string for the volume type.
+ *
+ * @param[in]  type  volume type
+ *
+ * @return volume type short description string
+ */
+static_inline char *
+volumeTypeShortString(int type)
+{
+    return
+       (type == RWVOL ? "RW" :
+        (type == ROVOL ? "RO" : (type == BACKVOL ? "BK" : "??")));
+}
 
 /**
  * Print the volume header information
@@ -931,10 +1141,10 @@ main(int argc, char **argv)
  *
  * @return none
  */
-void
+static void
 PrintHeader(Volume * vp)
 {
-    printf("Volume header for volume %u (%s)\n", V_id(vp), V_name(vp));
+    printf("Volume header for volume %" AFS_VOLID_FMT " (%s)\n", afs_printable_VolumeId_lu(V_id(vp)), V_name(vp));
     printf("stamp.magic = %x, stamp.version = %u\n", V_stamp(vp).magic,
           V_stamp(vp).version);
     printf
@@ -943,12 +1153,11 @@ 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",
-        V_id(vp), V_parentId(vp), V_cloneId(vp), V_backupId(vp),
-        V_restoredFromId(vp));
+       ("id = %" AFS_VOLID_FMT ", parentId = %" AFS_VOLID_FMT ", cloneId = %" AFS_VOLID_FMT ", backupId = %" AFS_VOLID_FMT ", restoredFromId = %" AFS_VOLID_FMT "\n",
+        afs_printable_VolumeId_lu(V_id(vp)), afs_printable_VolumeId_lu(V_parentId(vp)), afs_printable_VolumeId_lu(V_cloneId(vp)), afs_printable_VolumeId_lu(V_backupId(vp)), afs_printable_VolumeId_lu(V_restoredFromId(vp)));
     printf
        ("maxquota = %d, minquota = %d, maxfiles = %d, filecount = %d, diskused = %d\n",
         V_maxquota(vp), V_minquota(vp), V_maxfiles(vp), V_filecount(vp),
@@ -966,7 +1175,7 @@ PrintHeader(Volume * vp)
         V_dayUse(vp), V_weekUse(vp)[0], V_weekUse(vp)[1], V_weekUse(vp)[2],
         V_weekUse(vp)[3], V_weekUse(vp)[4], V_weekUse(vp)[5],
         V_weekUse(vp)[6], date(V_dayUseDate(vp)));
-    printf("volUpdateCounter = %u\n", V_volUpCounter(vp));
+    printf("volUpdateCounter = %u\n", V_volUpdateCounter(vp));
 }
 
 /**
@@ -1018,14 +1227,12 @@ GetFileInfo(FD_t fd, afs_sfsize_t * size, char **ctime, char **mtime,
 /**
  * Copy the inode data to a file in the current directory.
  *
- * @param[in] vp     volume object
- * @param[in] vnode  vnode object
- * @param[in] inode  inode of the source file
+ * @param[in] vdp     vnode details object
  *
  * @return none
  */
-static void
-SaveInode(Volume * vp, struct VnodeDiskObject *vnode, Inode ino)
+void
+volinfo_SaveInode(struct VolInfoOpt *opt, struct VnodeDetails *vdp)
 {
     IHandle_t *ih;
     FdHandle_t *fdP;
@@ -1033,12 +1240,13 @@ SaveInode(Volume * vp, struct VnodeDiskObject *vnode, Inode ino)
     int ofd = 0;
     afs_foff_t total;
     ssize_t len;
+    Inode ino = VNDISK_GET_INO(vdp->vnode);
 
     if (!VALID_INO(ino)) {
-        return;
+       return;
     }
 
-    IH_INIT(ih, V_device(vp), V_parentId(vp), ino);
+    IH_INIT(ih, V_device(vdp->vp), V_parentId(vdp->vp), ino);
     fdP = IH_OPEN(ih);
     if (fdP == NULL) {
        fprintf(stderr,
@@ -1073,7 +1281,7 @@ SaveInode(Volume * vp, struct VnodeDiskObject *vnode, Inode ino)
        }
        if (len == 0)
            break;              /* No more input */
-       nBytes = write(ofd, buffer, len);
+       nBytes = write(ofd, buffer, (size_t)len);
        if (nBytes != len) {
            FDH_REALLYCLOSE(fdP);
            IH_RELEASE(ih);
@@ -1095,6 +1303,400 @@ SaveInode(Volume * vp, struct VnodeDiskObject *vnode, Inode ino)
 }
 
 /**
+ * get the VnodeDiskObject for a directory given its vnode id.
+ *
+ * @param[in]   vp       volume object
+ * @param[in]   parent   vnode id to read
+ * @param[out]  pvn      vnode disk object to populate
+ *
+ * @post pvn contains copy of disk object for parent id
+ *
+ * @return operation status
+ *   @retval 0   success
+ *   @retval -1  failure
+ */
+static int
+GetDirVnode(struct VolInfoOpt *opt, Volume * vp, VnodeId parent, VnodeDiskObject * pvn)
+{
+    afs_int32 code;
+    afs_foff_t offset;
+
+    if (!DirIndexFd) {
+       return -1;              /* previously failed to open the large vnode index. */
+    }
+    if (parent % 2 == 0) {
+       fprintf(stderr, "%s: Invalid parent vnode id %lu in volume %lu\n",
+               progname,
+               afs_printable_uint32_lu(parent),
+               afs_printable_uint32_lu(V_id(vp)));
+    }
+    offset = vnodeIndexOffset(&VnodeClassInfo[vLarge], parent);
+    code = FDH_PREAD(DirIndexFd, pvn, SIZEOF_LARGEDISKVNODE, offset);
+    if (code != SIZEOF_LARGEDISKVNODE) {
+       fprintf(stderr,
+               "%s: GetDirVnode: read failed for %lu.%lu at offset %llu\n",
+               progname, afs_printable_uint32_lu(V_id(vp)),
+               afs_printable_uint32_lu(parent), (long long unsigned)offset);
+       return -1;
+    }
+    if (opt->checkMagic && (pvn->vnodeMagic != LARGEVNODEMAGIC)) {
+       fprintf(stderr, "%s: GetDirVnode: bad vnode magic for %lu.%lu at offset %llu\n",
+               progname, afs_printable_uint32_lu(V_id(vp)),
+               afs_printable_uint32_lu(parent), (long long unsigned)offset);
+       return -1;
+    }
+    if (!pvn->dataVersion) {
+       fprintf(stderr, "%s: GetDirVnode: dv is zero for %lu.%lu at offset %llu\n",
+               progname, afs_printable_uint32_lu(V_id(vp)),
+               afs_printable_uint32_lu(parent), (long long unsigned)offset);
+       return -1;
+    }
+    return 0;
+}
+
+/**
+ * Perform inverse lookup on a vice directory object to map a fid onto a dirent string.
+ *
+ * @param[in]   vp          volume object
+ * @param[in]   pvnode      parent directory vnode object
+ * @param[in]   cvnid       child vnode id to inverse lookup
+ * @param[in]   cuniq       child uniquifier to inverse lookup
+ * @param[out]  dirent      buffer in which to store dirent string
+ * @param[out]  dirent_len  length of dirent buffer
+ *
+ * @post dirent contains string for the (cvnid, cuniq) entry
+ *
+ * @return operation status
+ *    @retval 0 success
+ */
+static int
+GetDirEntry(Volume * vp, VnodeDiskObject * pvnode, VnodeId cvnid,
+           afs_uint32 cuniq, char *dirent, size_t dirent_len)
+{
+    DirHandle dir;
+    Inode ino;
+    int volumeChanged;
+    afs_int32 code;
+
+    ino = VNDISK_GET_INO(pvnode);
+    if (!VALID_INO(ino)) {
+       fprintf(stderr, "%s: GetDirEntry invalid parent ino\n", progname);
+       return -1;
+    }
+    SetSalvageDirHandle(&dir, V_parentId(vp), V_device(vp), ino,
+                       &volumeChanged);
+    code = afs_dir_InverseLookup(&dir, cvnid, cuniq, dirent, dirent_len);
+    if (code) {
+       fprintf(stderr, "%s: afs_dir_InverseLookup failed with code %d\n",
+               progname, code);
+    }
+    FidZap(&dir);
+    return code;
+}
+
+/**
+ * Lookup the path of this vnode, relative to the root of the volume.
+ *
+ * @param[in] vdp    vnode details
+ *
+ * @return status
+ *   @retval 0 success
+ *   @retval -1 error
+ */
+static int
+LookupPath(struct VolInfoOpt *opt, struct VnodeDetails *vdp)
+{
+#define MAX_PATH_LEN 1023
+    static char path_buffer[MAX_PATH_LEN + 1];
+    static char dirent[MAX_PATH_LEN + 1];
+    char vnode_buffer[SIZEOF_LARGEDISKVNODE];
+    struct VnodeDiskObject *pvn = (struct VnodeDiskObject *)vnode_buffer;
+    int code = 0;
+    int space;
+    char *cursor;
+    Volume *vp = vdp->vp;
+    VnodeId parent = vdp->vnode->parent;
+    VnodeId cvnid = vdp->vnodeNumber;
+    afs_uint32 cuniq = vdp->vnode->uniquifier;
+
+    if (!parent) {
+       vdp->path = "/";        /* this is root */
+       return 0;
+    }
+
+    space = sizeof(path_buffer) - 1;
+    cursor = &path_buffer[space];
+    *cursor = '\0';
+
+    while (parent) {
+       int len;
+
+       code = GetDirVnode(opt, vp, parent, pvn);
+       if (code) {
+           cursor = NULL;
+           break;
+       }
+       code = GetDirEntry(vp, pvn, cvnid, cuniq, dirent, MAX_PATH_LEN);
+       if (code) {
+           cursor = NULL;
+           break;
+       }
+
+       len = strlen(dirent);
+       if (len == 0) {
+           fprintf(stderr,
+                   "%s: Failed to lookup path for fid %lu.%lu.%lu: empty dir entry\n",
+                   progname, afs_printable_uint32_lu(V_id(vdp->vp)),
+                   afs_printable_uint32_lu(vdp->vnodeNumber),
+                   afs_printable_uint32_lu(vdp->vnode->uniquifier));
+           cursor = NULL;
+           code = -1;
+           break;
+       }
+
+       if (space < (len + 1)) {
+           fprintf(stderr,
+                   "%s: Failed to lookup path for fid %lu.%lu.%lu: path exceeds max length (%u).\n",
+                   progname, afs_printable_uint32_lu(V_id(vdp->vp)),
+                   afs_printable_uint32_lu(vdp->vnodeNumber),
+                   afs_printable_uint32_lu(vdp->vnode->uniquifier),
+                   MAX_PATH_LEN);
+           cursor = NULL;
+           code = -1;
+           break;
+       }
+
+       /* prepend path component */
+       cursor -= len;
+       memcpy(cursor, dirent, len);
+       *--cursor = '/';
+       space -= (len + 1);
+
+       /* next parent */
+       cvnid = parent;
+       cuniq = pvn->uniquifier;
+       parent = pvn->parent;
+    }
+
+    if (cursor) {
+       vdp->path = cursor;
+    }
+    return code;
+}
+
+/**
+ * Read the symlink target and determine if this vnode is a mount point.
+ *
+ * @param[inout]   vdp    vnode details object
+ *
+ * @return error code
+ *   @retval 0 success
+ *   @retval -1 failure
+ */
+static int
+ReadSymlinkTarget(struct VnodeDetails *vdp)
+{
+#define MAX_SYMLINK_LEN 1023
+    static char buffer[MAX_SYMLINK_LEN + 1];
+    int code;
+    Volume *vp = vdp->vp;
+    VnodeDiskObject *vnode = vdp->vnode;
+    VnodeId vnodeNumber = vdp->vnodeNumber;
+    IHandle_t *ihP = NULL;
+    FdHandle_t *fdP = NULL;
+    afs_fsize_t fileLength;
+    int readLength;
+    Inode ino;
+
+    ino = VNDISK_GET_INO(vnode);
+    VNDISK_GET_LEN(fileLength, vnode);
+
+    if (fileLength > MAX_SYMLINK_LEN) {
+       fprintf(stderr,
+               "%s: Symlink contents for fid (%lu.%lu.%lu.%lu) exceeds "
+               "%u, file length is %llu)!\n", progname,
+               afs_printable_uint32_lu(V_id(vp)),
+               afs_printable_uint32_lu(vnodeNumber),
+               afs_printable_uint32_lu(vnode->uniquifier),
+               afs_printable_uint32_lu(vnode->dataVersion),
+                MAX_SYMLINK_LEN,
+               fileLength);
+       return -1;
+    }
+    if (fileLength == 0) {
+       fprintf(stderr,
+               "%s: Symlink contents for fid (%lu.%lu.%lu.%lu) is empty.\n",
+               progname,
+               afs_printable_uint32_lu(V_id(vp)),
+               afs_printable_uint32_lu(vnodeNumber),
+               afs_printable_uint32_lu(vnode->uniquifier),
+               afs_printable_uint32_lu(vnode->dataVersion));
+       return -1;
+    }
+
+    IH_INIT(ihP, V_device(vp), V_parentId(vp), ino);
+    fdP = IH_OPEN(ihP);
+    if (fdP == NULL) {
+       code = -1;
+       goto cleanup;
+    }
+    readLength = FDH_PREAD(fdP, buffer, fileLength, 0);
+    if (readLength < 0) {
+       fprintf(stderr,
+               "%s: Error reading symlink contents for fid (%lu.%lu.%lu.%lu); "
+               "errno %d\n",
+               progname,
+               afs_printable_uint32_lu(V_id(vp)),
+               afs_printable_uint32_lu(vnodeNumber),
+               afs_printable_uint32_lu(vnode->uniquifier),
+               afs_printable_uint32_lu(vnode->dataVersion), errno);
+       code = -1;
+       goto cleanup;
+    } else if (readLength != fileLength) {
+       fprintf(stderr,
+               "%s: Symlink contents for fid (%lu.%lu.%lu.%lu) don't match "
+               "vnode file length metadata (len=%llu, actual=%lld)!\n",
+               progname,
+               afs_printable_uint32_lu(V_id(vp)),
+               afs_printable_uint32_lu(vnodeNumber),
+               afs_printable_uint32_lu(vnode->uniquifier),
+               afs_printable_uint32_lu(vnode->dataVersion), fileLength,
+               (long long)readLength);
+       code = -1;
+       goto cleanup;
+    }
+    code = 0;
+
+    if (readLength > 1 && (buffer[0] == '#' || buffer[0] == '%')
+       && buffer[readLength - 1] == '.') {
+       char *sep;
+       buffer[readLength - 1] = '\0';  /* stringify; clobbers trailing dot */
+       sep = strchr(buffer, ':');
+       vdp->t = VNODE_U_MOUNT;
+       vdp->u.mnt.type = buffer[0];
+       if (!sep) {
+           vdp->u.mnt.cell = NULL;
+           vdp->u.mnt.vol = buffer + 1;
+       } else {
+           *sep = '\0';
+           vdp->u.mnt.cell = buffer + 1;
+           vdp->u.mnt.vol = sep + 1;
+       }
+    } else {
+       buffer[readLength] = '\0';
+       vdp->t = VNODE_U_SYMLINK;
+       vdp->u.target = buffer;
+    }
+
+  cleanup:
+    if (fdP) {
+       FDH_CLOSE(fdP);
+    }
+    if (ihP) {
+       IH_RELEASE(ihP);
+    }
+    return code;
+}
+
+/**
+ * Print vnode details line
+ *
+ * @param[inout]  vdp   vnode details object
+ *
+ * @return none
+ */
+void
+volinfo_PrintVnodeDetails(struct VolInfoOpt *opt, struct VnodeDetails *vdp)
+{
+    switch (vdp->vnode->type) {
+    case vNull:
+       break;
+    case vFile:
+       if (opt->findVnType & FIND_FILE) {
+           PrintColumns(opt, vdp, "file");
+       }
+       break;
+    case vDirectory:
+       if (opt->findVnType & FIND_DIR) {
+           PrintColumns(opt, vdp, "dir");
+       }
+       break;
+    case vSymlink:
+       if (opt->findVnType & (FIND_MOUNT | FIND_SYMLINK)) {
+           ReadSymlinkTarget(vdp);
+           if ((opt->findVnType & FIND_MOUNT) && (vdp->t == VNODE_U_MOUNT)) {
+               PrintColumns(opt, vdp, "mount");
+           }
+           if ((opt->findVnType & FIND_SYMLINK) && (vdp->t == VNODE_U_SYMLINK)) {
+               PrintColumns(opt, vdp, "symlink");
+           }
+       }
+       break;
+    default:
+       fprintf(stderr,
+               "%s: Warning: unexpected vnode type %u on fid %lu.%lu.%lu",
+               progname, vdp->vnode->type,
+               afs_printable_uint32_lu(V_id(vdp->vp)),
+               afs_printable_uint32_lu(vdp->vnodeNumber),
+               afs_printable_uint32_lu(vdp->vnode->uniquifier));
+    }
+}
+
+/**
+ * Print each access entry of a vnode
+ *
+ * @param[in]  vdp   vnode details object
+ *
+ * @return none
+ */
+void
+volinfo_ScanAcl(struct VolInfoOpt *opt, struct VnodeDetails *vdp)
+{
+    int i;
+    struct acl_accessList *acl;
+    VnodeDiskObject *vnode = vdp->vnode;
+
+    if (vnode->type == vNull) {
+       return;
+    }
+
+    acl = VVnodeDiskACL(vnode);
+    for (i = 0; i < acl->positive; i++) {
+       vdp->t = VNODE_U_POS_ACCESS;
+       vdp->u.access = &(acl->entries[i]);
+       PrintColumns(opt, vdp, "acl");
+    }
+    for (i = (acl->total - 1); i >= (acl->total - acl->negative); i--) {
+       vdp->t = VNODE_U_NEG_ACCESS;
+       vdp->u.access = &(acl->entries[i]);
+       PrintColumns(opt, vdp, "acl");
+    }
+}
+
+/**
+ * Determine if the mode matches all the given masks.
+ *
+ * Returns true if the mode bits match all the given masks. A mask matches if at
+ * least one bit in the mask is present in the mode bits.  An empty mode mask
+ * list matches all modes (even if all the mode bits are zero.)
+ *
+ * param[in]  modeBits  unix mode bits of a vnode
+ *
+ */
+static int
+ModeMaskMatch(struct VolInfoOpt *opt, unsigned int modeBits)
+{
+    int i;
+
+    for (i = 0; i < sizeof(opt->modeMask) / sizeof(*opt->modeMask) && opt->modeMask[i]; i++) {
+       if ((opt->modeMask[i] & modeBits) == 0) {
+           return 0;           /* at least one mode bit is not present */
+       }
+    }
+    return 1;
+}
+
+/**
  * Scan a volume index and handle each vnode
  *
  * @param[in] vp      volume object
@@ -1102,8 +1704,8 @@ SaveInode(Volume * vp, struct VnodeDiskObject *vnode, Inode ino)
  *
  * @return none
  */
-void
-HandleVnodes(Volume * vp, VnodeClass class)
+static void
+HandleVnodes(struct VolInfoOpt *opt, Volume * vp, VnodeClass class)
 {
     afs_int32 diskSize =
        (class == vSmall ? SIZEOF_SMALLDISKVNODE : SIZEOF_LARGEDISKVNODE);
@@ -1113,27 +1715,21 @@ HandleVnodes(Volume * vp, VnodeClass class)
     int vnodeIndex;
     afs_sfsize_t nVnodes;
     afs_foff_t offset = 0;
-    Inode ino;
     IHandle_t *ih = vp->vnodeIndex[class].handle;
     FdHandle_t *fdP = NULL;
     afs_sfsize_t size;
     char *ctime, *atime, *mtime;
+    struct opr_queue *scanList = &VnodeScanLists[class];
+    struct opr_queue *cursor;
 
-    /* print vnode table heading */
-    if (class == vLarge) {
-       if (DumpInfo) {
-           printf("\nLarge vnodes (directories)\n");
-       }
-    } else {
-       if (DumpInfo) {
-           printf("\nSmall vnodes(files, symbolic links)\n");
-           fflush(stdout);
-       }
-       if (SaveInodes) {
-           printf("Saving all volume files to current directory ...\n");
-           if (ShowSizes) {
-               PrintingVolumeSizes = 0;        /* print heading again */
-           }
+    if (opr_queue_IsEmpty(scanList)) {
+       return;
+    }
+
+    for (opr_queue_Scan(scanList, cursor)) {
+       struct VnodeScanProc *entry = (struct VnodeScanProc *)cursor;
+       if (entry->heading) {
+           printf("%s", entry->heading);
        }
     }
 
@@ -1152,7 +1748,7 @@ HandleVnodes(Volume * vp, VnodeClass class)
     if (GetFileInfo(fdP->fd_fd, &size, &ctime, &atime, &mtime) != 0) {
        goto error;
     }
-    if (InodeTimes) {
+    if (opt->dumpInodeTimes) {
        printf("ichanged : %s\nimodified: %s\niaccessed: %s\n\n", ctime,
               mtime, atime);
     }
@@ -1167,22 +1763,25 @@ HandleVnodes(Volume * vp, VnodeClass class)
         nVnodes && STREAM_READ(vnode, diskSize, 1, file) == 1;
         nVnodes--, vnodeIndex++, offset += diskSize) {
 
-       ino = VNDISK_GET_INO(vnode);
-       if (ShowSizes) {
-           afs_fsize_t fileLength;
+       struct VnodeDetails vnodeDetails;
 
-           VNDISK_GET_LEN(fileLength, vnode);
-           if (fileLength > 0) {
-               volumeTotals.vnodesize += fileLength;
-               volumeTotals.vnodesize_k += fileLength / 1024;
-           }
-       }
-       if (SaveInodes && (class == vSmall)) {
-           SaveInode(vp, vnode, ino);
+       if (!ModeMaskMatch(opt, vnode->modeBits)) {
+           continue;
        }
-       if (DumpVnodes || ShowOrphaned) {
-           PrintVnode(offset, vnode,
-                      bitNumberToVnodeNumber(vnodeIndex, class), ino, vp);
+
+       memset(&vnodeDetails, 0, sizeof(struct 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) (opt, &vnodeDetails);
+           }
        }
     }
 
@@ -1198,44 +1797,43 @@ HandleVnodes(Volume * vp, VnodeClass class)
 /**
  * Print vnode information
  *
- * @param[in] offset       index offset of this vnode
- * @param[in] vnode        vnode object to be printed
- * @param[in] vnodeNumber  vnode number
- * @param[in] ino          fileserver inode number
- * @param[in] vp           parent volume of the vnode
+ * @param[in] vdp          vnode details object
  *
  * @return none
  */
 void
-PrintVnode(afs_foff_t offset, VnodeDiskObject * vnode, VnodeId vnodeNumber,
-          Inode ino, Volume * vp)
+volinfo_PrintVnode(struct VolInfoOpt *opt, 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);
 
     /* 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))
+    if (opt->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);
-    if (DumpInodeNumber)
+        (long long)offset, vdp->vnodeNumber, vnode->uniquifier,
+        vnode->dataVersion, vnode->cloned, (afs_uintmax_t) fileLength,
+        vnode->linkCount, vnode->parent);
+    if (opt->dumpInodeNumber)
        printf(" inode: %s", PrintInode(NULL, ino));
-    if (DumpDate)
+    if (opt->dumpDate)
        printf(" ServerModTime: %s", date(vnode->serverModifyTime));
 #if defined(AFS_NAMEI_ENV)
-    if (PrintFileNames) {
-       IH_INIT(ihtmpp, V_device(vp), V_parentId(vp), ino);
+    if (opt->dumpFileNames) {
+       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);
@@ -1246,3 +1844,312 @@ PrintVnode(afs_foff_t offset, VnodeDiskObject * vnode, VnodeId vnodeNumber,
 #endif
     printf("\n");
 }
+
+/**
+ * Print the volume partition id
+ *
+ * @param[in]  vp  volume object
+ *
+ * @return none
+ */
+static void
+PrintPartitionId(Volume * vp)
+{
+    char *partition = VPartitionPath(V_partition(vp));
+
+    if (!strncmp(partition, "/vicep", 6)) {
+       printf("%s", partition + 6);
+    } else if (!strncmp(partition, "vicep", 5)) {
+       printf("%s", partition + 5);
+    } else {
+       fprintf(stderr, "Invalid partition for volume id %lu\n",
+               afs_printable_uint32_lu(V_id(vp)));
+       printf("%s", PLACEHOLDER);
+    }
+}
+
+/**
+ * Print the vnode type description string
+ *
+ * @param[in]  type  vnode type
+ *
+ * @return none
+ */
+static void
+volinfo_PrintVnodeType(int type)
+{
+    switch (type) {
+    case vNull:
+       printf("null");
+       break;
+    case vFile:
+       printf("file");
+       break;
+    case vDirectory:
+       printf("dir");
+       break;
+    case vSymlink:
+       printf("symlink");
+       break;
+    default:
+       printf("unknown");
+    }
+}
+
+/**
+ * Print right bits as string.
+ *
+ * param[in] rights  rights bitmap
+ */
+static void
+PrintRights(int rights)
+{
+    if (rights & PRSFS_READ) {
+       printf("r");
+    }
+    if (rights & PRSFS_LOOKUP) {
+       printf("l");
+    }
+    if (rights & PRSFS_INSERT) {
+       printf("i");
+    }
+    if (rights & PRSFS_DELETE) {
+       printf("d");
+    }
+    if (rights & PRSFS_WRITE) {
+       printf("w");
+    }
+    if (rights & PRSFS_LOCK) {
+       printf("k");
+    }
+    if (rights & PRSFS_ADMINISTER) {
+       printf("a");
+    }
+    if (rights & PRSFS_USR0) {
+       printf("A");
+    }
+    if (rights & PRSFS_USR1) {
+       printf("B");
+    }
+    if (rights & PRSFS_USR2) {
+       printf("C");
+    }
+    if (rights & PRSFS_USR3) {
+       printf("D");
+    }
+    if (rights & PRSFS_USR4) {
+       printf("E");
+    }
+    if (rights & PRSFS_USR5) {
+       printf("F");
+    }
+    if (rights & PRSFS_USR6) {
+       printf("G");
+    }
+    if (rights & PRSFS_USR7) {
+       printf("H");
+    }
+}
+
+/**
+ * Print the path to the namei file.
+ */
+static void
+PrintNamei(Volume * vp, VnodeDiskObject * vnode)
+{
+#ifdef AFS_NAMEI_ENV
+    namei_t name;
+    IHandle_t *ihP = NULL;
+    Inode ino;
+    ino = VNDISK_GET_INO(vnode);
+    IH_INIT(ihP, V_device(vp), V_parentId(vp), ino);
+    namei_HandleToName(&name, ihP);
+    printf("%s", name.n_path);
+    IH_RELEASE(ihP);
+#else
+    printf("%s", PLACEHOLDER);
+#endif
+}
+
+/**
+ * Print the column heading line.
+ */
+static void
+PrintColumnHeading(struct VolInfoOpt *opt)
+{
+    int i;
+    const char *name;
+
+    for (i = 0; i < NumOutputColumns; i++) {
+       if (i > 0) {
+           printf("%s", opt->columnDelim);
+       }
+       name = ColumnName[OutputColumn[i]].name;
+       while (*name) {
+           putchar(toupper(*name++));
+       }
+    }
+    printf("\n");
+}
+
+/**
+ * Print output columns for the vnode/acess entry.
+ *
+ * @param[in]  vdp   vnode details object
+ * @param[in]  desc  type of line to be printed
+ *
+ * @return none
+ */
+static void
+PrintColumns(struct VolInfoOpt *opt, struct VnodeDetails *vdp, const char *desc)
+{
+    int i;
+    afs_fsize_t length;
+
+    for (i = 0; i < NumOutputColumns; i++) {
+       if (i > 0) {
+           printf("%s", opt->columnDelim);
+       }
+       switch (OutputColumn[i]) {
+       case col_host:
+           printf("%s", opt->hostname);
+           break;
+       case col_desc:
+           printf("%s", desc);
+           break;
+       case col_vid:
+           printf("%lu", afs_printable_uint32_lu(V_id(vdp->vp)));
+           break;
+       case col_offset:
+           printf("%llu", vdp->offset);
+           break;
+       case col_vtype:
+           printf("%s", volumeTypeShortString(V_type(vdp->vp)));
+           break;
+       case col_vname:
+           printf("%s", V_name(vdp->vp));
+           break;
+       case col_part:
+           printf("%s", VPartitionPath(V_partition(vdp->vp)));
+           break;
+       case col_partid:
+           PrintPartitionId(vdp->vp);
+           break;
+       case col_fid:
+           printf("%lu.%lu.%lu",
+                  afs_printable_uint32_lu(V_id(vdp->vp)),
+                  afs_printable_uint32_lu(vdp->vnodeNumber),
+                  afs_printable_uint32_lu(vdp->vnode->uniquifier));
+           break;
+       case col_path:
+           if (!vdp->path) {
+               LookupPath(opt, vdp);
+           }
+           printf("%s", vdp->path ? vdp->path : PLACEHOLDER);
+           break;
+       case col_target:
+           printf("%s",
+                  (vdp->t == VNODE_U_SYMLINK ? vdp->u.target : PLACEHOLDER));
+           break;
+       case col_mount:
+           if (vdp->t != VNODE_U_MOUNT) {
+               printf("%s", PLACEHOLDER);
+           } else {
+               printf("%c", vdp->u.mnt.type);
+               if (vdp->u.mnt.cell) {
+                   printf("%s:", vdp->u.mnt.cell);
+               }
+               printf("%s.", vdp->u.mnt.vol);
+           }
+           break;
+       case col_mtype:
+           printf("%c", (vdp->t == VNODE_U_MOUNT ? vdp->u.mnt.type : '-'));
+           break;
+       case col_mcell:
+           printf("%s",
+                  (vdp->t == VNODE_U_MOUNT && vdp->u.mnt.cell ? vdp->u.mnt.cell : PLACEHOLDER));
+           break;
+       case col_mvol:
+           printf("%s",
+                  (vdp->t == VNODE_U_MOUNT ? vdp->u.mnt.vol : PLACEHOLDER));
+           break;
+       case col_aid:
+           if (vdp->t == VNODE_U_POS_ACCESS || vdp->t == VNODE_U_NEG_ACCESS) {
+               printf("%d", vdp->u.access->id);
+           } else {
+               printf("%s", PLACEHOLDER);
+           }
+           break;
+       case col_arights:
+           if (vdp->t == VNODE_U_POS_ACCESS) {
+               printf("+");
+               PrintRights(vdp->u.access->rights);
+           } else if (vdp->t == VNODE_U_NEG_ACCESS) {
+               printf("-");
+               PrintRights(vdp->u.access->rights);
+           }
+           break;
+       case col_vntype:
+           volinfo_PrintVnodeType(vdp->vnode->type);
+           break;
+       case col_cloned:
+           printf("%c", vdp->vnode->cloned ? 'y' : 'n');
+           break;
+       case col_mode:
+           printf("0%o", vdp->vnode->modeBits);
+           break;
+       case col_links:
+           printf("%lu", afs_printable_uint32_lu(vdp->vnode->linkCount));
+           break;
+       case col_length:
+           VNDISK_GET_LEN(length, vdp->vnode);
+           printf("%llu", length);
+           break;
+       case col_uniq:
+           printf("%lu", afs_printable_uint32_lu(vdp->vnode->uniquifier));
+           break;
+       case col_dv:
+           printf("%lu", afs_printable_uint32_lu(vdp->vnode->dataVersion));
+           break;
+       case col_inode:
+           printf("%" AFS_UINT64_FMT, VNDISK_GET_INO(vdp->vnode));
+           break;
+       case col_namei:
+           PrintNamei(vdp->vp, vdp->vnode);
+           break;
+       case col_modtime:
+           printf("%lu",
+                  afs_printable_uint32_lu(vdp->vnode->unixModifyTime));
+           break;
+       case col_author:
+           printf("%lu", afs_printable_uint32_lu(vdp->vnode->author));
+           break;
+       case col_owner:
+           printf("%lu", afs_printable_uint32_lu(vdp->vnode->owner));
+           break;
+       case col_parent:
+           printf("%lu", afs_printable_uint32_lu(vdp->vnode->parent));
+           break;
+       case col_magic:
+           printf("0x%08X", vdp->vnode->vnodeMagic);
+           break;
+       case col_lock:
+           printf("%lu.%lu",
+                  afs_printable_uint32_lu(vdp->vnode->lock.lockCount),
+                  afs_printable_uint32_lu(vdp->vnode->lock.lockTime));
+           break;
+       case col_smodtime:
+           printf("%lu",
+                  afs_printable_uint32_lu(vdp->vnode->serverModifyTime));
+           break;
+       case col_group:
+           printf("%lu", afs_printable_uint32_lu(vdp->vnode->group));
+           break;
+       default:
+           fprintf(stderr, "%s: Unknown column type: %d (%d)\n", progname,
+                   OutputColumn[i], i);
+           break;
+       }
+    }
+    printf("\n");
+}