ubik_VL_GetAddrsU does not accept a VLCallBack parameter
[openafs.git] / src / volser / vos.c
index 765b777..360e217 100644 (file)
@@ -10,8 +10,9 @@
 #include <afsconfig.h>
 #include <afs/param.h>
 
-RCSID
-    ("$Header$");
+#ifdef IGNORE_SOME_GCC_WARNINGS
+# pragma GCC diagnostic warning "-Wimplicit-function-declaration"
+#endif
 
 #include <sys/types.h>
 #include <string.h>
@@ -19,6 +20,7 @@ RCSID
 #include <fcntl.h>
 #include <io.h>
 #include <winsock2.h>
+#include <WINNT/afsreg.h>
 #else
 #include <sys/time.h>
 #include <sys/file.h>
@@ -51,12 +53,14 @@ RCSID
 #include <afs/ihandle.h>
 #include <afs/vnode.h>
 #include <afs/volume.h>
+#include <afs/com_err.h>
 #include "dump.h"
 #include "lockdata.h"
 
 #ifdef AFS_AIX32_ENV
 #include <signal.h>
 #endif
+#include "volser_internal.h"
 #include "volser_prototypes.h"
 #include "vsutils_prototypes.h"
 #include "lockprocs_prototypes.h"
@@ -67,11 +71,11 @@ RCSID
 
 /* Local Prototypes */
 int PrintDiagnostics(char *astring, afs_int32 acode);
-int GetVolumeInfo(afs_int32 volid, afs_int32 *server, afs_int32 *part, 
+int GetVolumeInfo(afs_uint32 volid, afs_int32 *server, afs_int32 *part, 
                   afs_int32 *voltype, struct nvldbentry *rentry);
 
 struct tqElem {
-    afs_int32 volid;
+    afs_uint32 volid;
     struct tqElem *next;
 };
 
@@ -101,13 +105,13 @@ static struct tqHead busyHead, notokHead;
 static void
 qInit(struct tqHead *ahead)
 {
-    memset((char *)ahead, 0, sizeof(struct tqHead));
+    memset(ahead, 0, sizeof(struct tqHead));
     return;
 }
 
 
 static void
-qPut(struct tqHead *ahead, afs_int32 volid)
+qPut(struct tqHead *ahead, afs_uint32 volid)
 {
     struct tqElem *elem;
 
@@ -120,7 +124,7 @@ qPut(struct tqHead *ahead, afs_int32 volid)
 }
 
 static void
-qGet(struct tqHead *ahead, afs_int32 *volid)
+qGet(struct tqHead *ahead, afs_uint32 *volid)
 {
     struct tqElem *tmp;
 
@@ -285,17 +289,18 @@ SendFile(usd_handle_t ufd, register struct rx_call *call, long blksize)
 #ifndef AFS_NT40_ENV           /* NT csn't select on non-socket fd's */
        fd_set in;
        FD_ZERO(&in);
-       FD_SET((int)(ufd->handle), &in);
+       FD_SET((intptr_t)(ufd->handle), &in);
        /* don't timeout if read blocks */
 #if defined(AFS_PTHREAD_ENV)
        select(((int)(ufd->handle)) + 1, &in, 0, 0, 0);
 #else
-       IOMGR_Select(((int)(ufd->handle)) + 1, &in, 0, 0, 0);
+       IOMGR_Select(((intptr_t)(ufd->handle)) + 1, &in, 0, 0, 0);
 #endif
 #endif
        error = USD_READ(ufd, buffer, blksize, &nbytes);
        if (error) {
-           fprintf(STDERR, "File system read failed\n");
+           fprintf(STDERR, "File system read failed: %s\n",
+                   afs_error_message(error));
            break;
        }
        if (nbytes == 0) {
@@ -315,9 +320,9 @@ SendFile(usd_handle_t ufd, register struct rx_call *call, long blksize)
 /* function invoked by UV_RestoreVolume, reads the data from rx_trx_stream and
  * writes it out to the volume. */
 afs_int32
-WriteData(struct rx_call *call, char *rock)
+WriteData(struct rx_call *call, void *rock)
 {
-    char *filename;
+    char *filename = (char *) rock;
     usd_handle_t ufd;
     long blksize;
     afs_int32 error, code;
@@ -328,7 +333,6 @@ WriteData(struct rx_call *call, char *rock)
 
     error = 0;
 
-    filename = rock;
     if (!filename || !*filename) {
        usd_StandardInput(&ufd);
        blksize = 4096;
@@ -340,7 +344,8 @@ WriteData(struct rx_call *call, char *rock)
            code = USD_IOCTL(ufd, USD_IOCTL_GETBLKSIZE, &blksize);
        }
        if (code) {
-           fprintf(STDERR, "Could not access file '%s'\n", filename);
+           fprintf(STDERR, "Could not access file '%s': %s\n", filename,
+                   afs_error_message(code));
            error = VOLSERBADOP;
            goto wfail;
        }
@@ -399,18 +404,19 @@ ReceiveFile(usd_handle_t ufd, struct rx_call *call, long blksize)
 #ifndef AFS_NT40_ENV           /* NT csn't select on non-socket fd's */
            fd_set out;
            FD_ZERO(&out);
-           FD_SET((int)(ufd->handle), &out);
+           FD_SET((intptr_t)(ufd->handle), &out);
            /* don't timeout if write blocks */
 #if defined(AFS_PTHREAD_ENV)
            select(((int)(ufd->handle)) + 1, &out, 0, 0, 0);
 #else
-           IOMGR_Select(((int)(ufd->handle)) + 1, 0, &out, 0, 0);
+           IOMGR_Select(((intptr_t)(ufd->handle)) + 1, 0, &out, 0, 0);
 #endif
 #endif
            error =
                USD_WRITE(ufd, &buffer[bytesread - bytesleft], bytesleft, &w);
            if (error) {
-               fprintf(STDERR, "File system write failed\n");
+               fprintf(STDERR, "File system write failed: %s\n",
+                       afs_error_message(error));
                ERROR_EXIT(-1);
            }
        }
@@ -423,8 +429,9 @@ ReceiveFile(usd_handle_t ufd, struct rx_call *call, long blksize)
 }
 
 afs_int32
-DumpFunction(struct rx_call *call, char *filename)
+DumpFunction(struct rx_call *call, void *rock)
 {
+    char *filename = (char *)rock;
     usd_handle_t ufd;          /* default is to stdout */
     afs_int32 error = 0, code;
     afs_hyper_t size;
@@ -448,7 +455,8 @@ DumpFunction(struct rx_call *call, char *filename)
            code = USD_IOCTL(ufd, USD_IOCTL_GETBLKSIZE, &blksize);
        }
        if (code) {
-           fprintf(STDERR, "Could not create file '%s'\n", filename);
+           fprintf(STDERR, "Could not create file '%s': %s\n", filename,
+                   afs_error_message(code));
            ERROR_EXIT(VOLSERBADOP);
        }
     }
@@ -472,22 +480,13 @@ DumpFunction(struct rx_call *call, char *filename)
     return (error);
 }
 
-static char *
-vos_ctime(afs_int32 *timep)
-{
-    time_t foo = *timep;
-    return ctime(&foo);
-}
-
 static void
-DisplayFormat(pntr, server, part, totalOK, totalNotOK, totalBusy, fast,
-             longlist, disp)
-     volintInfo *pntr;
-     afs_int32 server, part;
-     int *totalOK, *totalNotOK, *totalBusy;
-     int fast, longlist, disp;
+DisplayFormat(volintInfo *pntr, afs_int32 server, afs_int32 part,
+             int *totalOK, int *totalNotOK, int *totalBusy, int fast,
+             int longlist, int disp)
 {
     char pname[10];
+    time_t t;
 
     if (fast) {
        fprintf(STDOUT, "%-10lu\n", (unsigned long)pntr->volid);
@@ -520,29 +519,35 @@ DisplayFormat(pntr, server, part, totalOK, totalNotOK, totalBusy, fast,
                    (unsigned long)pntr->cloneID,
                    (unsigned long)pntr->backupID);
            fprintf(STDOUT, "    MaxQuota %10d K \n", pntr->maxquota);
+           t = pntr->creationDate;
            fprintf(STDOUT, "    Creation    %s",
-                   vos_ctime(& pntr->creationDate));
+                   ctime(&t));
 #ifdef FULL_LISTVOL_SWITCH
+           t = pntr->copyDate;
            fprintf(STDOUT, "    Copy        %s",
-                   vos_ctime( & pntr->copyDate));
-           if (!pntr->backupDate)
+                   ctime(&t));
+
+           t = pntr->backupDate;
+           if (!t)
                fprintf(STDOUT, "    Backup      Never\n");
            else
                fprintf(STDOUT, "    Backup      %s",
-                       vos_ctime( & pntr->backupDate));
-           if (pntr->accessDate)
+                       ctime(&t));
+
+           t = pntr->accessDate;
+           if (t)
                fprintf(STDOUT, "    Last Access %s",
-                       vos_ctime( & pntr->accessDate));
+                       ctime(&t));
 #endif
-           if (!pntr->updateDate)
+           t = pntr->updateDate;
+           if (!t)
                fprintf(STDOUT, "    Last Update Never\n");
-           else {
+           else
                fprintf(STDOUT, "    Last Update %s",
-                       vos_ctime( & pntr->updateDate));
-               fprintf(STDOUT,
-                       "    %d accesses in the past day (i.e., vnode references)\n",
-                       pntr->dayUse);
-           }
+                       ctime(&t));
+           fprintf(STDOUT,
+                   "    %d accesses in the past day (i.e., vnode references)\n",
+                   pntr->dayUse);
        } else if (pntr->status == VBUSY) {
            *totalBusy += 1;
            qPut(&busyHead, pntr->volid);
@@ -622,20 +627,11 @@ DisplayFormat(pntr, server, part, totalOK, totalNotOK, totalBusy, fast,
  *------------------------------------------------------------------------*/
 
 static void
-XDisplayFormat(a_xInfoP, a_servID, a_partID, a_totalOKP, a_totalNotOKP,
-              a_totalBusyP, a_fast, a_int32, a_showProblems)
-     volintXInfo *a_xInfoP;
-     afs_int32 a_servID;
-     afs_int32 a_partID;
-     int *a_totalOKP;
-     int *a_totalNotOKP;
-     int *a_totalBusyP;
-     int a_fast;
-     int a_int32;
-     int a_showProblems;
-
+XDisplayFormat(volintXInfo *a_xInfoP, afs_int32 a_servID, afs_int32 a_partID,
+              int *a_totalOKP, int *a_totalNotOKP, int *a_totalBusyP,
+              int a_fast, int a_int32, int a_showProblems)
 {                              /*XDisplayFormat */
-
+    time_t t;
     char pname[10];
 
     if (a_fast) {
@@ -677,29 +673,36 @@ XDisplayFormat(a_xInfoP, a_servID, a_partID, a_totalOKP, a_totalNotOKP,
                    (unsigned long)a_xInfoP->cloneID,
                    (unsigned long)a_xInfoP->backupID);
            fprintf(STDOUT, "    MaxQuota %10d K \n", a_xInfoP->maxquota);
+
+           t = a_xInfoP->creationDate;
            fprintf(STDOUT, "    Creation    %s",
-                   vos_ctime( & a_xInfoP->creationDate));
+                   ctime(&t));
 #ifdef FULL_LISTVOL_SWITCH
+           t = a_xInfoP->copyDate;
            fprintf(STDOUT, "    Copy        %s",
-                   vos_ctime( & a_xInfoP->copyDate));
-           if (!a_xInfoP->backupDate)
+                   ctime(&t));
+
+           t = a_xInfoP->backupDate;
+           if (!t)
                fprintf(STDOUT, "    Backup      Never\n");
            else
                fprintf(STDOUT, "    Backup      %s",
-                       vos_ctime( & a_xInfoP->backupDate));
-           if (a_xInfoP->accessDate)
+                       ctime(&t));
+
+           t = a_xInfoP->accessDate;
+           if (t)
                fprintf(STDOUT, "    Last Access %s",
-                       vos_ctime( & a_xInfoP->accessDate));
+                       ctime(&t));
 #endif
-           if (!a_xInfoP->updateDate)
+           t = a_xInfoP->updateDate;
+           if (!t)
                fprintf(STDOUT, "    Last Update Never\n");
-           else {
+           else
                fprintf(STDOUT, "    Last Update %s",
-                       vos_ctime( & a_xInfoP->updateDate));
-               fprintf(STDOUT,
-                       "    %d accesses in the past day (i.e., vnode references)\n",
-                       a_xInfoP->dayUse);
-           }
+                       ctime(&t));
+           fprintf(STDOUT,
+                   "    %d accesses in the past day (i.e., vnode references)\n",
+                   a_xInfoP->dayUse);
 
            /*
             * Print all the read/write and authorship stats.
@@ -858,19 +861,11 @@ XDisplayFormat(a_xInfoP, a_servID, a_partID, a_totalOKP, a_totalNotOKP,
  *------------------------------------------------------------------------*/
 
 static void
-XDisplayFormat2(a_xInfoP, a_servID, a_partID, a_totalOKP, a_totalNotOKP,
-              a_totalBusyP, a_fast, a_int32, a_showProblems)
-     volintXInfo *a_xInfoP;
-     afs_int32 a_servID;
-     afs_int32 a_partID;
-     int *a_totalOKP;
-     int *a_totalNotOKP;
-     int *a_totalBusyP;
-     int a_fast;
-     int a_int32;
-     int a_showProblems;
-
+XDisplayFormat2(volintXInfo *a_xInfoP, afs_int32 a_servID, afs_int32 a_partID,
+               int *a_totalOKP, int *a_totalNotOKP, int *a_totalBusyP,
+               int a_fast, int a_int32, int a_showProblems)
 {                              /*XDisplayFormat */
+    time_t t;
     if (a_fast) {
        /*
         * Short & sweet.
@@ -904,23 +899,16 @@ XDisplayFormat2(a_xInfoP, a_servID, a_partID, a_totalOKP, a_totalNotOKP,
                }
 
                fprintf(STDOUT, "name\t\t%s\n", a_xInfoP->name);
-               fprintf(STDOUT, "id\t\t%lu\n", a_xInfoP->volid);
+               fprintf(STDOUT, "id\t\t%lu\n", afs_printable_uint32_lu(a_xInfoP->volid));
                fprintf(STDOUT, "serv\t\t%s\t%s\n", address, hostname);
                fprintf(STDOUT, "part\t\t%s\n", pname);
-               switch (a_xInfoP->status) {
-               case VOK:
-                       fprintf(STDOUT, "status\t\tOK\n");
-                       break;
-               case VBUSY:
-                       fprintf(STDOUT, "status\t\tBUSY\n");
-                       return;
-               default:
-                       fprintf(STDOUT, "status\t\tUNATTACHABLE\n");
-                       return;
-               }
-               fprintf(STDOUT, "backupID\t%lu\n", a_xInfoP->backupID);
-               fprintf(STDOUT, "parentID\t%lu\n", a_xInfoP->parentID);
-               fprintf(STDOUT, "cloneID\t\t%lu\n", a_xInfoP->cloneID);
+                fprintf(STDOUT, "status\t\tOK\n");
+               fprintf(STDOUT, "backupID\t%lu\n", 
+                       afs_printable_uint32_lu(a_xInfoP->backupID));
+               fprintf(STDOUT, "parentID\t%lu\n", 
+                       afs_printable_uint32_lu(a_xInfoP->parentID));
+               fprintf(STDOUT, "cloneID\t\t%lu\n", 
+                       afs_printable_uint32_lu(a_xInfoP->cloneID));
                fprintf(STDOUT, "inUse\t\t%s\n", a_xInfoP->inUse ? "Y" : "N");
                switch (a_xInfoP->type) {
                case 0:
@@ -936,16 +924,30 @@ XDisplayFormat2(a_xInfoP, a_servID, a_partID, a_totalOKP, a_totalNotOKP,
                        fprintf(STDOUT, "type\t\t?\n");
                        break;
                }
-               fprintf(STDOUT, "creationDate\t%-9lu\t%s", a_xInfoP->creationDate,
-                       vos_ctime(&a_xInfoP->creationDate));
-               fprintf(STDOUT, "accessDate\t%-9lu\t%s", a_xInfoP->accessDate,
-                       vos_ctime(&a_xInfoP->accessDate));
-               fprintf(STDOUT, "updateDate\t%-9lu\t%s", a_xInfoP->updateDate,
-                       vos_ctime(&a_xInfoP->updateDate));
-               fprintf(STDOUT, "backupDate\t%-9lu\t%s", a_xInfoP->backupDate,
-                       vos_ctime(&a_xInfoP->backupDate));
-               fprintf(STDOUT, "copyDate\t%-9lu\t%s", a_xInfoP->copyDate,
-                       vos_ctime(&a_xInfoP->copyDate));
+               t = a_xInfoP->creationDate;
+               fprintf(STDOUT, "creationDate\t%-9lu\t%s", 
+                       afs_printable_uint32_lu(a_xInfoP->creationDate),
+                       ctime(&t));
+
+               t = a_xInfoP->accessDate;
+               fprintf(STDOUT, "accessDate\t%-9lu\t%s", 
+                       afs_printable_uint32_lu(a_xInfoP->accessDate),
+                       ctime(&t));
+
+               t = a_xInfoP->updateDate;
+               fprintf(STDOUT, "updateDate\t%-9lu\t%s", 
+                       afs_printable_uint32_lu(a_xInfoP->updateDate),
+                       ctime(&t));
+
+               t = a_xInfoP->backupDate;
+               fprintf(STDOUT, "backupDate\t%-9lu\t%s", 
+                       afs_printable_uint32_lu(a_xInfoP->backupDate),
+                       ctime(&t));
+
+               t = a_xInfoP->copyDate;
+               fprintf(STDOUT, "copyDate\t%-9lu\t%s", 
+                       afs_printable_uint32_lu(a_xInfoP->copyDate),
+                       ctime(&t));
                
                fprintf(STDOUT, "diskused\t%u\n", a_xInfoP->size);
                fprintf(STDOUT, "maxquota\t%u\n", a_xInfoP->maxquota);
@@ -1031,12 +1033,11 @@ XDisplayFormat2(a_xInfoP, a_servID, a_partID, a_totalOKP, a_totalNotOKP,
 
 #ifdef FULL_LISTVOL_SWITCH
 static void
-DisplayFormat2(server, partition, pntr)
-     long server, partition;
-     volintInfo *pntr;
+DisplayFormat2(long server, long partition, volintInfo *pntr)
 {
     static long server_cache = -1, partition_cache = -1;
     static char hostname[256], address[32], pname[16];
+    time_t t;
 
     if (server != server_cache) {
        struct in_addr s;
@@ -1050,8 +1051,12 @@ DisplayFormat2(server, partition, pntr)
        MapPartIdIntoName(partition, pname);
        partition_cache = partition;
     }
-    fprintf(STDOUT, "name\t\t%s\n", pntr->name);
-    fprintf(STDOUT, "id\t\t%lu\n", pntr->volid);
+
+    if (pntr->status == VOK)
+        fprintf(STDOUT, "name\t\t%s\n", pntr->name);
+
+    fprintf(STDOUT, "id\t\t%lu\n", 
+           afs_printable_uint32_lu(pntr->volid));
     fprintf(STDOUT, "serv\t\t%s\t%s\n", address, hostname);
     fprintf(STDOUT, "part\t\t%s\n", pname);
     switch (pntr->status) {
@@ -1065,9 +1070,12 @@ DisplayFormat2(server, partition, pntr)
        fprintf(STDOUT, "status\t\tUNATTACHABLE\n");
        return;
     }
-    fprintf(STDOUT, "backupID\t%lu\n", pntr->backupID);
-    fprintf(STDOUT, "parentID\t%lu\n", pntr->parentID);
-    fprintf(STDOUT, "cloneID\t\t%lu\n", pntr->cloneID);
+    fprintf(STDOUT, "backupID\t%lu\n", 
+           afs_printable_uint32_lu(pntr->backupID));
+    fprintf(STDOUT, "parentID\t%lu\n", 
+           afs_printable_uint32_lu(pntr->parentID));
+    fprintf(STDOUT, "cloneID\t\t%lu\n", 
+           afs_printable_uint32_lu(pntr->cloneID));
     fprintf(STDOUT, "inUse\t\t%s\n", pntr->inUse ? "Y" : "N");
     fprintf(STDOUT, "needsSalvaged\t%s\n", pntr->needsSalvaged ? "Y" : "N");
     /* 0xD3 is from afs/volume.h since I had trouble including the file */
@@ -1086,32 +1094,50 @@ DisplayFormat2(server, partition, pntr)
        fprintf(STDOUT, "type\t\t?\n");
        break;
     }
-    fprintf(STDOUT, "creationDate\t%-9lu\t%s", pntr->creationDate,
-           vos_ctime(&pntr->creationDate));
-    fprintf(STDOUT, "accessDate\t%-9lu\t%s", pntr->accessDate,
-           vos_ctime(&pntr->accessDate));
-    fprintf(STDOUT, "updateDate\t%-9lu\t%s", pntr->updateDate,
-           vos_ctime(&pntr->updateDate));
-    fprintf(STDOUT, "backupDate\t%-9lu\t%s", pntr->backupDate,
-           vos_ctime(&pntr->backupDate));
-    fprintf(STDOUT, "copyDate\t%-9lu\t%s", pntr->copyDate,
-           vos_ctime(&pntr->copyDate));
-    fprintf(STDOUT, "flags\t\t%#lx\t(Optional)\n", pntr->flags);
+    t = pntr->creationDate;
+    fprintf(STDOUT, "creationDate\t%-9lu\t%s", 
+           afs_printable_uint32_lu(pntr->creationDate),
+           ctime(&t));
+
+    t = pntr->accessDate;
+    fprintf(STDOUT, "accessDate\t%-9lu\t%s", 
+           afs_printable_uint32_lu(pntr->accessDate),
+           ctime(&t));
+
+    t = pntr->updateDate;
+    fprintf(STDOUT, "updateDate\t%-9lu\t%s", 
+           afs_printable_uint32_lu(pntr->updateDate),
+           ctime(&t));
+
+    t = pntr->backupDate;
+    fprintf(STDOUT, "backupDate\t%-9lu\t%s", 
+           afs_printable_uint32_lu(pntr->backupDate),
+           ctime(&t));
+
+    t = pntr->copyDate;
+    fprintf(STDOUT, "copyDate\t%-9lu\t%s", 
+           afs_printable_uint32_lu(pntr->copyDate),
+           ctime(&t));
+
+    fprintf(STDOUT, "flags\t\t%#lx\t(Optional)\n", 
+           afs_printable_uint32_lu(pntr->flags));
     fprintf(STDOUT, "diskused\t%u\n", pntr->size);
     fprintf(STDOUT, "maxquota\t%u\n", pntr->maxquota);
-    fprintf(STDOUT, "minquota\t%lu\t(Optional)\n", pntr->spare0);
+    fprintf(STDOUT, "minquota\t%lu\t(Optional)\n", 
+           afs_printable_uint32_lu(pntr->spare0));
     fprintf(STDOUT, "filecount\t%u\n", pntr->filecount);
     fprintf(STDOUT, "dayUse\t\t%u\n", pntr->dayUse);
-    fprintf(STDOUT, "weekUse\t\t%lu\t(Optional)\n", pntr->spare1);
-    fprintf(STDOUT, "spare2\t\t%lu\t(Optional)\n", pntr->spare2);
-    fprintf(STDOUT, "spare3\t\t%lu\t(Optional)\n", pntr->spare3);
+    fprintf(STDOUT, "weekUse\t\t%lu\t(Optional)\n",
+           afs_printable_uint32_lu(pntr->spare1));
+    fprintf(STDOUT, "spare2\t\t%lu\t(Optional)\n", 
+           afs_printable_uint32_lu(pntr->spare2));
+    fprintf(STDOUT, "spare3\t\t%lu\t(Optional)\n", 
+           afs_printable_uint32_lu(pntr->spare3));
     return;
 }
 
 static void
-DisplayVolumes2(server, partition, pntr, count)
-     volintInfo *pntr;
-     long server, partition, count;
+DisplayVolumes2(long server, long partition, volintInfo *pntr, long count)
 {
     long i;
 
@@ -1126,14 +1152,12 @@ DisplayVolumes2(server, partition, pntr, count)
 #endif /* FULL_LISTVOL_SWITCH */
 
 static void
-DisplayVolumes(server, part, pntr, count, longlist, fast, quiet)
-     afs_int32 server, part;
-     volintInfo *pntr;
-     afs_int32 count, longlist, fast;
-     int quiet;
+DisplayVolumes(afs_int32 server, afs_int32 part, volintInfo *pntr,
+              afs_int32 count, afs_int32 longlist, afs_int32 fast,
+              int quiet)
 {
     int totalOK, totalNotOK, totalBusy, i;
-    afs_int32 volid = 0;
+    afs_uint32 volid = 0;
 
     totalOK = 0;
     totalNotOK = 0;
@@ -1194,23 +1218,16 @@ DisplayVolumes(server, part, pntr, count, longlist, fast, quiet)
  *------------------------------------------------------------------------*/
 
 static void
-XDisplayVolumes(a_servID, a_partID, a_xInfoP, a_count, a_int32, a_fast,
-               a_quiet)
-     afs_int32 a_servID;
-     afs_int32 a_partID;
-     volintXInfo *a_xInfoP;
-     afs_int32 a_count;
-     afs_int32 a_int32;
-     afs_int32 a_fast;
-     int a_quiet;
-
+XDisplayVolumes(afs_int32 a_servID, afs_int32 a_partID, volintXInfo *a_xInfoP,
+               afs_int32 a_count, afs_int32 a_int32, afs_int32 a_fast,
+               int a_quiet)
 {                              /*XDisplayVolumes */
 
     int totalOK;               /*Total OK volumes */
     int totalNotOK;            /*Total screwed volumes */
     int totalBusy;             /*Total busy volumes */
     int i;                     /*Loop variable */
-    afs_int32 volid = 0;       /*Current volume ID */
+    afs_uint32 volid = 0;      /*Current volume ID */
 
     /*
      * Initialize counters and (global!!) queues.
@@ -1285,23 +1302,16 @@ XDisplayVolumes(a_servID, a_partID, a_xInfoP, a_count, a_int32, a_fast,
  *------------------------------------------------------------------------*/
 
 static void
-XDisplayVolumes2(a_servID, a_partID, a_xInfoP, a_count, a_int32, a_fast,
-               a_quiet)
-     afs_int32 a_servID;
-     afs_int32 a_partID;
-     volintXInfo *a_xInfoP;
-     afs_int32 a_count;
-     afs_int32 a_int32;
-     afs_int32 a_fast;
-     int a_quiet;
-
+XDisplayVolumes2(afs_int32 a_servID, afs_int32 a_partID, volintXInfo *a_xInfoP,
+                afs_int32 a_count, afs_int32 a_int32, afs_int32 a_fast,
+                int a_quiet)
 {                              /*XDisplayVolumes */
 
     int totalOK;               /*Total OK volumes */
     int totalNotOK;            /*Total screwed volumes */
     int totalBusy;             /*Total busy volumes */
     int i;                     /*Loop variable */
-    afs_int32 volid = 0;       /*Current volume ID */
+    afs_uint32 volid = 0;      /*Current volume ID */
 
     /*
      * Initialize counters and (global!!) queues.
@@ -1357,11 +1367,8 @@ XDisplayVolumes2(a_servID, a_partID, a_xInfoP, a_count, a_int32, a_fast,
 /* set <server> and <part> to the correct values depending on 
  * <voltype> and <entry> */
 static void
-GetServerAndPart(entry, voltype, server, part, previdx)
-     struct nvldbentry *entry;
-     afs_int32 *server, *part;
-     int voltype;
-     int *previdx;
+GetServerAndPart(struct nvldbentry *entry, int voltype, afs_int32 *server,
+                afs_int32 *part, int *previdx)
 {
     int i, istart, vtype;
 
@@ -1426,13 +1433,8 @@ PostVolumeStats(struct nvldbentry *entry)
  *------------------------------------------------------------------------*/
 
 static void
-XVolumeStats(a_xInfoP, a_entryP, a_srvID, a_partID, a_volType)
-     volintXInfo *a_xInfoP;
-     struct nvldbentry *a_entryP;
-     afs_int32 a_srvID;
-     afs_int32 a_partID;
-     int a_volType;
-
+XVolumeStats(volintXInfo *a_xInfoP, struct nvldbentry *a_entryP,
+            afs_int32 a_srvID, afs_int32 a_partID, int a_volType)
 {                              /*XVolumeStats */
 
     int totalOK, totalNotOK, totalBusy;        /*Dummies - we don't really count here */
@@ -1466,7 +1468,8 @@ static int
 NukeVolume(register struct cmd_syndesc *as)
 {
     register afs_int32 code;
-    afs_int32 volID, err;
+    afs_uint32 volID;
+    afs_int32  err;
     afs_int32 partID;
     afs_int32 server;
     register char *tp;
@@ -1534,7 +1537,7 @@ ExamineVolume(register struct cmd_syndesc *as, void *arock)
     afs_int32 vcode = 0;
     volintInfo *pntr = (volintInfo *) 0;
     volintXInfo *xInfoP = (volintXInfo *) 0;
-    afs_int32 volid;
+    afs_uint32 volid;
     afs_int32 code, err, error = 0;
     int voltype, foundserv = 0, foundentry = 0;
     afs_int32 aserver, apart;
@@ -1683,9 +1686,8 @@ static int
 SetFields(register struct cmd_syndesc *as, void *arock)
 {
     struct nvldbentry entry;
-    afs_int32 vcode = 0;
     volintInfo info;
-    afs_int32 volid;
+    afs_uint32 volid;
     afs_int32 code, err;
     afs_int32 aserver, apart;
     int previdx = -1;
@@ -1722,7 +1724,7 @@ SetFields(register struct cmd_syndesc *as, void *arock)
 
     if (as->parms[1].items) {
        /* -max <quota> */
-       code = util_GetInt32(as->parms[1].items->data, &info.maxquota);
+       code = util_GetHumanInt32(as->parms[1].items->data, &info.maxquota);
        if (code) {
            fprintf(STDERR, "invalid quota value\n");
            return code;
@@ -1766,7 +1768,8 @@ SetFields(register struct cmd_syndesc *as, void *arock)
 static int
 volOnline(register struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 server, partition, volid;
+    afs_int32 server, partition;
+    afs_uint32 volid;
     afs_int32 code, err = 0;
 
     server = GetServer(as->parms[0].items->data);
@@ -1825,7 +1828,8 @@ volOnline(register struct cmd_syndesc *as, void *arock)
 static int
 volOffline(register struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 server, partition, volid;
+    afs_int32 server, partition;
+    afs_uint32 volid;
     afs_int32 code, err = 0;
     afs_int32 transflag, sleeptime, transdone;
 
@@ -1877,11 +1881,15 @@ CreateVolume(register struct cmd_syndesc *as, void *arock)
 {
     afs_int32 pnum;
     char part[10];
-    afs_int32 volid, code;
+    afs_uint32 volid = 0, rovolid = 0, bkvolid = 0;
+    afs_uint32 *arovolid;
+    afs_int32 code;
     struct nvldbentry entry;
     afs_int32 vcode;
     afs_int32 quota;
 
+    arovolid = &rovolid;
+
     quota = 5000;
     tserver = GetServer(as->parms[0].items->data);
     if (!tserver) {
@@ -1930,22 +1938,48 @@ CreateVolume(register struct cmd_syndesc *as, void *arock)
     }
 
     if (as->parms[3].items) {
-       if (!IsNumeric(as->parms[3].items->data)) {
-           fprintf(STDERR, "Initial quota %s should be numeric.\n",
-                   as->parms[3].items->data);
+       code = util_GetHumanInt32(as->parms[3].items->data, &quota);
+       if (code) {
+           fprintf(STDERR, "vos: bad integer specified for quota.\n");
+           return code;
+       }
+    }
+
+    if (as->parms[4].items) {
+       if (!IsNumeric(as->parms[4].items->data)) {
+           fprintf(STDERR, "vos: Given volume ID %s should be numeric.\n",
+                   as->parms[4].items->data);
            return EINVAL;
        }
 
-       code = util_GetInt32(as->parms[3].items->data, &quota);
+       code = util_GetUInt32(as->parms[4].items->data, &volid);
        if (code) {
-           fprintf(STDERR, "vos: bad integer specified for quota.\n");
+           fprintf(STDERR, "vos: bad integer specified for volume ID.\n");
            return code;
        }
     }
 
+    if (as->parms[5].items) {
+       if (!IsNumeric(as->parms[5].items->data)) {
+           fprintf(STDERR, "vos: Given RO volume ID %s should be numeric.\n",
+                   as->parms[5].items->data);
+           return EINVAL;
+       }
+
+       code = util_GetUInt32(as->parms[5].items->data, &rovolid);
+       if (code) {
+           fprintf(STDERR, "vos: bad integer specified for volume ID.\n");
+           return code;
+       }
+
+       if (rovolid == 0) {
+           arovolid = NULL;
+       }
+    }
+
     code =
-       UV_CreateVolume2(tserver, pnum, as->parms[2].items->data, quota, 0,
-                        0, 0, 0, &volid);
+       UV_CreateVolume3(tserver, pnum, as->parms[2].items->data, quota, 0,
+                        0, 0, 0, &volid, arovolid, &bkvolid);
     if (code) {
        PrintDiagnostics("create", code);
        return code;
@@ -1957,12 +1991,13 @@ CreateVolume(register struct cmd_syndesc *as, void *arock)
     return 0;
 }
 
+#if 0
 static afs_int32
-DeleteAll(entry)
-     struct nvldbentry *entry;
+DeleteAll(struct nvldbentry *entry)
 {
     int i;
-    afs_int32 error, code, curserver, curpart, volid;
+    afs_int32 error, code, curserver, curpart;
+    afs_uint32 volid;
 
     MapHostToNetwork(entry);
     error = 0;
@@ -1980,12 +2015,14 @@ DeleteAll(entry)
     }
     return error;
 }
+#endif
 
 static int
 DeleteVolume(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 err, code = 0;
-    afs_int32 server = 0, partition = -1, volid;
+    afs_int32 server = 0, partition = -1;
+    afs_uint32 volid;
     char pname[10];
     afs_int32 idx, j;
 
@@ -2101,11 +2138,12 @@ DeleteVolume(struct cmd_syndesc *as, void *arock)
 }
 
 #define TESTM  0               /* set for move space tests, clear for production */
-static
+static int
 MoveVolume(register struct cmd_syndesc *as, void *arock)
 {
 
-    afs_int32 volid, fromserver, toserver, frompart, topart;
+    afs_uint32 volid;
+    afs_int32 fromserver, toserver, frompart, topart;
     afs_int32 flags, code, err;
     char fromPartName[10], toPartName[10];
 
@@ -2231,7 +2269,8 @@ MoveVolume(register struct cmd_syndesc *as, void *arock)
 static int
 CopyVolume(register struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 volid, fromserver, toserver, frompart, topart, code, err, flags;
+    afs_uint32 volid;
+    afs_int32 fromserver, toserver, frompart, topart, code, err, flags;
     char fromPartName[10], toPartName[10], *tovolume;
     struct nvldbentry entry;
     struct diskPartition64 partition;  /* for space check */
@@ -2377,7 +2416,8 @@ CopyVolume(register struct cmd_syndesc *as, void *arock)
 static int
 ShadowVolume(register struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 volid, fromserver, toserver, frompart, topart, tovolid;
+    afs_uint32 volid, tovolid;
+    afs_int32 fromserver, toserver, frompart, topart;
     afs_int32 code, err, flags;
     char fromPartName[10], toPartName[10], toVolName[32], *tovolume;
     struct diskPartition64 partition;  /* for space check */
@@ -2572,7 +2612,8 @@ ShadowVolume(register struct cmd_syndesc *as, void *arock)
 static int
 CloneVolume(register struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 server, part, volid, cloneid, voltype;
+    afs_uint32 volid, cloneid;
+    afs_int32 server, part, voltype;
     char partName[10], *volname;
     afs_int32 code, err, flags;
     struct nvldbentry entry;
@@ -2685,10 +2726,12 @@ CloneVolume(register struct cmd_syndesc *as, void *arock)
 static int
 BackupVolume(register struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 avolid, aserver, apart, vtype, code, err;
+    afs_uint32 avolid;
+    afs_int32 aserver, apart, vtype, code, err;
     struct nvldbentry entry;
 
-    afs_int32 buvolid, buserver, bupart, butype;
+    afs_uint32 buvolid;
+    afs_int32 buserver, bupart, butype;
     struct nvldbentry buentry;
 
     avolid = vsu_GetVolumeID(as->parms[0].items->data, cstruct, &err);
@@ -2755,7 +2798,8 @@ ReleaseVolume(register struct cmd_syndesc *as, void *arock)
 {
 
     struct nvldbentry entry;
-    afs_int32 avolid, aserver, apart, vtype, code, err;
+    afs_uint32 avolid;
+    afs_int32 aserver, apart, vtype, code, err;
     int force = 0;
 
     if (as->parms[1].items)
@@ -2795,10 +2839,11 @@ ReleaseVolume(register struct cmd_syndesc *as, void *arock)
     return 0;
 }
 
-static
-DumpVolume(register struct cmd_syndesc *as, void *arock)
+static int
+DumpVolumeCmd(register struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 avolid, aserver, apart, voltype, fromdate = 0, code, err, i, flags;
+    afs_uint32 avolid;
+    afs_int32 aserver, apart, voltype, fromdate = 0, code, err, i, flags;
     char filename[MAXPATHLEN];
     struct nvldbentry entry;
 
@@ -2896,9 +2941,10 @@ retry_dump:
 #define TS_NEW 3
 
 static int
-RestoreVolume(register struct cmd_syndesc *as, void *arock)
+RestoreVolumeCmd(register struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 avolid, aparentid, aserver, apart, code, vcode, err;
+    afs_uint32 avolid, aparentid;
+    afs_int32 aserver, apart, code, vcode, err;
     afs_int32 aoverwrite = ASK;
     afs_int32 acreation = 0, alastupdate = 0;
     int restoreflags = 0;
@@ -3193,7 +3239,8 @@ RestoreVolume(register struct cmd_syndesc *as, void *arock)
 static int
 LockReleaseCmd(register struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 avolid, code, err;
+    afs_uint32 avolid;
+    afs_int32 code, err;
 
     avolid = vsu_GetVolumeID(as->parms[0].items->data, cstruct, &err);
     if (avolid == 0) {
@@ -3218,7 +3265,8 @@ LockReleaseCmd(register struct cmd_syndesc *as, void *arock)
 static int
 AddSite(register struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 avolid, aserver, apart, code, err, valid = 0;
+    afs_uint32 avolid;
+    afs_int32 aserver, apart, code, err, arovolid, valid = 0;
     char apartName[10], avolname[VOLSER_MAXVOLNAME + 1];
 
     vsu_ExtractName(avolname, as->parms[2].items->data);;
@@ -3231,6 +3279,16 @@ AddSite(register struct cmd_syndesc *as, void *arock)
                    as->parms[2].items->data);
        exit(1);
     }
+    arovolid = 0;
+    if (as->parms[3].items) {
+       vsu_ExtractName(avolname, as->parms[3].items->data);
+       arovolid = vsu_GetVolumeID(avolname, cstruct, &err);
+       if (!arovolid) {
+           fprintf(STDERR, "vos: invalid ro volume id '%s'\n",
+                   as->parms[3].items->data);
+           exit(1);
+       }
+    }
     aserver = GetServer(as->parms[0].items->data);
     if (aserver == 0) {
        fprintf(STDERR, "vos: server '%s' not found in host table\n",
@@ -3252,10 +3310,10 @@ AddSite(register struct cmd_syndesc *as, void *arock)
                    as->parms[1].items->data);
        exit(1);
     }
-    if (as->parms[3].items) {
+    if (as->parms[4].items) {
        valid = 1;
     }
-    code = UV_AddSite(aserver, apart, avolid, valid);
+    code = UV_AddSite2(aserver, apart, avolid, arovolid, valid);
     if (code) {
        PrintDiagnostics("addsite", code);
        exit(1);
@@ -3270,7 +3328,8 @@ static int
 RemoveSite(register struct cmd_syndesc *as, void *arock)
 {
 
-    afs_int32 avolid, aserver, apart, code, err;
+    afs_uint32 avolid;
+    afs_int32 aserver, apart, code, err;
     char apartName[10], avolname[VOLSER_MAXVOLNAME + 1];
 
     vsu_ExtractName(avolname, as->parms[2].items->data);
@@ -3320,7 +3379,8 @@ RemoveSite(register struct cmd_syndesc *as, void *arock)
 static int
 ChangeLocation(register struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 avolid, aserver, apart, code, err;
+    afs_uint32 avolid;
+    afs_int32 aserver, apart, code, err;
     char apartName[10];
 
     avolid = vsu_GetVolumeID(as->parms[2].items->data, cstruct, &err);
@@ -3405,8 +3465,7 @@ ListPartitions(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-CompareVolName(p1, p2)
-     char *p1, *p2;
+CompareVolName(const void *p1, const void *p2)
 {
     volintInfo *arg1, *arg2;
 
@@ -3439,9 +3498,7 @@ CompareVolName(p1, p2)
  *------------------------------------------------------------------------*/
 
 static int
-XCompareVolName(a_obj1P, a_obj2P)
-     char *a_obj1P, *a_obj2P;
-
+XCompareVolName(const void *a_obj1P, const void *a_obj2P)
 {                              /*XCompareVolName */
 
     return (strcmp
@@ -3451,8 +3508,7 @@ XCompareVolName(a_obj1P, a_obj2P)
 }                              /*XCompareVolName */
 
 static int
-CompareVolID(p1, p2)
-     char *p1, *p2;
+CompareVolID(const void *p1, const void *p2)
 {
     volintInfo *arg1, *arg2;
 
@@ -3490,9 +3546,7 @@ CompareVolID(p1, p2)
  *------------------------------------------------------------------------*/
 
 static int
-XCompareVolID(a_obj1P, a_obj2P)
-     char *a_obj1P, *a_obj2P;
-
+XCompareVolID(const void *a_obj1P, const void *a_obj2P)
 {                              /*XCompareVolID */
 
     afs_int32 id1, id2;                /*Volume IDs we're comparing */
@@ -3528,7 +3582,7 @@ XCompareVolID(a_obj1P, a_obj2P)
  *     As advertised.
  *------------------------------------------------------------------------*/
 
-static
+static int
 ListVolumes(register struct cmd_syndesc *as, void *arock)
 {
     afs_int32 apart, int32list, fast;
@@ -3822,9 +3876,8 @@ SyncServer(register struct cmd_syndesc *as, void *arock)
 
 }
 
-static
-VolumeInfoCmd(name)
-     char *name;
+static int
+VolumeInfoCmd(char *name)
 {
     struct nvldbentry entry;
     afs_int32 vcode;
@@ -3854,7 +3907,8 @@ static int
 VolumeZap(register struct cmd_syndesc *as, void *arock)
 {
     struct nvldbentry entry;
-    afs_int32 volid, code, server, part, zapbackupid = 0, backupid = 0, err;
+    afs_uint32 volid, zapbackupid = 0, backupid = 0;
+    afs_int32 code, server, part, err;
 
     if (as->parms[3].items) {
        /* force flag is on, use the other version */
@@ -3943,6 +3997,7 @@ VolserStatus(register struct cmd_syndesc *as, void *arock)
     afs_int32 count;
     int i;
     char pname[10];
+    time_t t;
 
     server = GetServer(as->parms[0].items->data);
     if (!server) {
@@ -3965,8 +4020,9 @@ VolserStatus(register struct cmd_syndesc *as, void *arock)
     for (i = 0; i < count; i++) {
        /*print out the relevant info */
        fprintf(STDOUT, "--------------------------------------\n");
+       t = pntr->time;
        fprintf(STDOUT, "transaction: %lu  created: %s",
-               (unsigned long)pntr->tid, vos_ctime( & pntr->time));
+               (unsigned long)pntr->tid, ctime(&t));
        if (pntr->returnCode) {
            fprintf(STDOUT, "returnCode: %lu\n",
                    (unsigned long)pntr->returnCode);
@@ -4087,7 +4143,7 @@ RenameVolume(register struct cmd_syndesc *as, void *arock)
 }
 
 int
-GetVolumeInfo(afs_int32 volid, afs_int32 *server, afs_int32 *part, afs_int32 *voltype, 
+GetVolumeInfo(afs_uint32 volid, afs_int32 *server, afs_int32 *part, afs_int32 *voltype, 
               struct nvldbentry *rentry)
 {
     afs_int32 vcode;
@@ -4150,7 +4206,7 @@ static int
 DeleteEntry(register struct cmd_syndesc *as, void *arock)
 {
     afs_int32 apart = 0;
-    afs_int32 avolid;
+    afs_uint32 avolid;
     afs_int32 vcode;
     struct VldbListByAttributes attributes;
     nbulkentries arrayEntries;
@@ -4323,8 +4379,7 @@ DeleteEntry(register struct cmd_syndesc *as, void *arock)
 
 
 static int
-CompareVldbEntryByName(p1, p2)
-     char *p1, *p2;
+CompareVldbEntryByName(const void *p1, const void *p2)
 {
     struct nvldbentry *arg1, *arg2;
 
@@ -4334,8 +4389,7 @@ CompareVldbEntryByName(p1, p2)
 }
 
 /*
-static int CompareVldbEntry(p1,p2)
-char *p1,*p2;
+static int CompareVldbEntry(char *p1, char *p2)
 {
     struct nvldbentry *arg1,*arg2;
     int i;
@@ -4544,7 +4598,8 @@ ListVLDB(struct cmd_syndesc *as, void *arock)
 static int
 BackSys(register struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 apart = 0, avolid;
+    afs_uint32 avolid;
+    afs_int32 apart = 0;
     afs_int32 aserver = 0, code, aserver1, apart1;
     afs_int32 vcode;
     struct VldbListByAttributes attributes;
@@ -4556,11 +4611,14 @@ BackSys(register struct cmd_syndesc *as, void *arock)
     int seenprefix, seenxprefix, exclude, ex, exp, noaction;
     afs_int32 totalBack = 0;
     afs_int32 totalFail = 0;
-    int previdx = -1, error, same;
-    int comp = 0;
+    int previdx = -1;
+    int error;
+    int same = 0;
     struct cmd_item *ti;
-    char *ccode;
     int match = 0;
+#ifndef HAVE_POSIX_REGEX
+    char *ccode;
+#endif
 
     memset(&attributes, 0, sizeof(struct VldbListByAttributes));
     attributes.Mask = 0;
@@ -4874,7 +4932,7 @@ static int
 UnlockVLDB(register struct cmd_syndesc *as, void *arock)
 {
     afs_int32 apart;
-    afs_int32 aserver = NULL;
+    afs_int32 aserver = 0;
     afs_int32 code;
     afs_int32 vcode;
     struct VldbListByAttributes attributes;
@@ -4882,7 +4940,7 @@ UnlockVLDB(register struct cmd_syndesc *as, void *arock)
     register struct nvldbentry *vllist;
     afs_int32 nentries;
     int j;
-    afs_int32 volid;
+    afs_uint32 volid;
     afs_int32 totalE;
     char pname[10];
 
@@ -5117,7 +5175,7 @@ ChangeAddr(register struct cmd_syndesc *as, void *arock)
        ip1 = 0xffffffff;
     }
 
-    vcode = ubik_Call_New(VL_ChangeAddr, cstruct, 0, ntohl(ip1), ntohl(ip2));
+    vcode = ubik_VL_ChangeAddr(cstruct, UBIK_CALL_NEW, ntohl(ip1), ntohl(ip2));
     if (vcode) {
        if (remove) {
            fprintf(STDERR, "Could not remove server %s from the VLDB\n",
@@ -5145,12 +5203,11 @@ ChangeAddr(register struct cmd_syndesc *as, void *arock)
 }
 
 static void
-print_addrs(const bulkaddrs * addrs, const afsUUID * m_uuid, int nentries,
+print_addrs(const bulkaddrs * addrs, afsUUID * m_uuid, int nentries,
            int print)
 {
-    afs_int32 vcode;
+    afs_int32 vcode, m_uniq=0;
     afs_int32 i, j;
-    struct VLCallBack vlcb;
     afs_int32 *addrp;
     bulkaddrs m_addrs;
     ListAddrByAttributes m_attrs;
@@ -5183,8 +5240,9 @@ print_addrs(const bulkaddrs * addrs, const afsUUID * m_uuid, int nentries,
                m_addrs.bulkaddrs_val = 0;
                m_addrs.bulkaddrs_len = 0;
                vcode =
-                   ubik_VL_GetAddrsU(cstruct, 0, &m_attrs, &m_uuid,
-                                     (afs_int32 *)&vlcb, &m_nentries, &m_addrs);
+                   ubik_VL_GetAddrsU(cstruct, 0, &m_attrs, m_uuid,
+                                     &m_uniq, &m_nentries,
+                                     &m_addrs);
                if (vcode) {
                    fprintf(STDERR,
                            "vos: could not list the multi-homed server addresses\n");
@@ -5233,7 +5291,7 @@ print_addrs(const bulkaddrs * addrs, const afsUUID * m_uuid, int nentries,
 static int
 ListAddrs(register struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 vcode;
+    afs_int32 vcode, m_uniq=0;
     afs_int32 i, printuuid = 0;
     struct VLCallBack vlcb;
     afs_int32 nentries;
@@ -5279,8 +5337,8 @@ ListAddrs(register struct cmd_syndesc *as, void *arock)
     m_addrs.bulkaddrs_len = 0;
 
     vcode =
-       ubik_Call_New(VL_GetAddrs, cstruct, 0, 0, 0, &vlcb, &nentries,
-                     &m_addrs);
+       ubik_VL_GetAddrs(cstruct, UBIK_CALL_NEW, 0, 0, &vlcb, &nentries,
+                        &m_addrs);
     if (vcode) {
        fprintf(STDERR, "vos: could not list the server addresses\n");
        PrintError("", vcode);
@@ -5295,8 +5353,8 @@ ListAddrs(register struct cmd_syndesc *as, void *arock)
        m_attrs.index = i;
 
        vcode =
-           ubik_Call_New(VL_GetAddrsU, cstruct, 0, &m_attrs, &m_uuid,
-                         &vlcb, &m_nentries, &m_addrs);
+           ubik_VL_GetAddrsU(cstruct, UBIK_CALL_NEW, &m_attrs, &m_uuid,
+                             &m_uniq, &m_nentries, &m_addrs);
 
        if (vcode == VL_NOENT) {
            if (m_attrs.Mask == VLADDR_UUID) {
@@ -5337,7 +5395,8 @@ ListAddrs(register struct cmd_syndesc *as, void *arock)
 static int
 LockEntry(register struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 avolid, vcode, err;
+    afs_uint32 avolid;
+    afs_int32 vcode, err;
 
     avolid = vsu_GetVolumeID(as->parms[0].items->data, cstruct, &err);
     if (avolid == 0) {
@@ -5364,7 +5423,8 @@ static int
 ConvertRO(register struct cmd_syndesc *as, void *arock)
 {
     afs_int32 partition = -1;
-    afs_int32 server, volid, code, i, same;
+    afs_uint32 volid;
+    afs_int32 server, code, i, same;
     struct nvldbentry entry, storeEntry;
     afs_int32 vcode;
     afs_int32 rwindex = 0;
@@ -5527,7 +5587,8 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
 static int
 Sizes(register struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 avolid, aserver, apart, voltype, fromdate = 0, code, err, i;
+    afs_uint32 avolid;
+    afs_int32 aserver, apart, voltype, fromdate = 0, code, err, i;
     struct nvldbentry entry;
     volintSize vol_size;
 
@@ -5623,6 +5684,29 @@ PrintDiagnostics(char *astring, afs_int32 acode)
 }
 
 
+#ifdef AFS_NT40_ENV
+static DWORD
+win32_enableCrypt(void)
+{
+    HKEY parmKey;
+    DWORD dummyLen;
+    DWORD cryptall = 0;
+    DWORD code;
+
+    /* Look up configuration parameters in Registry */
+    code = RegOpenKeyEx(HKEY_LOCAL_MACHINE, AFSREG_CLT_SVC_PARAM_SUBKEY,
+                        0, (IsWow64()?KEY_WOW64_64KEY:0)|KEY_QUERY_VALUE, &parmKey);
+    if (code != ERROR_SUCCESS) {
+        dummyLen = sizeof(cryptall);
+        RegQueryValueEx(parmKey, "SecurityLevel", NULL, NULL,
+                        (BYTE *) &cryptall, &dummyLen);
+    }
+    RegCloseKey (parmKey);
+
+    return cryptall;
+}
+#endif /* AFS_NT40_ENV */
+
 static int
 MyBeforeProc(struct cmd_syndesc *as, void *arock)
 {
@@ -5640,7 +5724,11 @@ MyBeforeProc(struct cmd_syndesc *as, void *arock)
        tcell = as->parms[12].items->data;
     if (as->parms[14].items)   /* -serverauth specified */
        sauth = 1;
-    if (as->parms[16].items)   /* -crypt specified */
+    if (as->parms[16].items     /* -encrypt specified */
+#ifdef AFS_NT40_ENV
+        || win32_enableCrypt()
+#endif /* AFS_NT40_ENV */
+         )
        vsu_SetCrypt(1);
     if ((code =
         vsu_ClientInit((as->parms[13].items != 0), confdir, tcell, sauth,
@@ -5662,7 +5750,7 @@ MyBeforeProc(struct cmd_syndesc *as, void *arock)
 }
 
 int
-osi_audit()
+osi_audit(void)
 {
 /* this sucks but it works for now.
 */
@@ -5671,9 +5759,8 @@ osi_audit()
 
 #include "AFS_component_version_number.c"
 
-main(argc, argv)
-     int argc;
-     char **argv;
+int
+main(int argc, char **argv)
 {
     register afs_int32 code;
 
@@ -5704,6 +5791,8 @@ main(argc, argv)
     cmd_AddParm(ts, "-name", CMD_SINGLE, 0, "volume name");
     cmd_AddParm(ts, "-maxquota", CMD_SINGLE, CMD_OPTIONAL,
                "initial quota (KB)");
+    cmd_AddParm(ts, "-id", CMD_SINGLE, CMD_OPTIONAL, "volume ID");
+    cmd_AddParm(ts, "-roid", CMD_SINGLE, CMD_OPTIONAL, "readonly volume ID");
 #ifdef notdef
     cmd_AddParm(ts, "-minquota", CMD_SINGLE, CMD_OPTIONAL, "");
 #endif
@@ -5797,7 +5886,7 @@ main(argc, argv)
                "force a complete release");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("dump", DumpVolume, NULL, "dump a volume");
+    ts = cmd_CreateSyntax("dump", DumpVolumeCmd, NULL, "dump a volume");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
     cmd_AddParm(ts, "-time", CMD_SINGLE, CMD_OPTIONAL, "dump from time");
     cmd_AddParm(ts, "-file", CMD_SINGLE, CMD_OPTIONAL, "dump file");
@@ -5809,7 +5898,8 @@ main(argc, argv)
                "omit unchanged directories from an incremental dump");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("restore", RestoreVolume, NULL, "restore a volume");
+    ts = cmd_CreateSyntax("restore", RestoreVolumeCmd, NULL,
+                         "restore a volume");
     cmd_AddParm(ts, "-server", CMD_SINGLE, 0, "machine name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, 0, "partition name");
     cmd_AddParm(ts, "-name", CMD_SINGLE, 0, "name of volume to be restored");
@@ -5848,7 +5938,8 @@ main(argc, argv)
     cmd_AddParm(ts, "-partition", CMD_SINGLE, 0,
                "partition name for new site");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
-    cmd_AddParm(ts, "-valid", CMD_FLAG, CMD_OPTIONAL | CMD_HIDE, "publish as an up-to-date site in VLDB");
+    cmd_AddParm(ts, "-roid", CMD_SINGLE, CMD_OPTIONAL, "volume name or ID for RO");
+    cmd_AddParm(ts, "-valid", CMD_FLAG, CMD_OPTIONAL, "publish as an up-to-date site in VLDB");
     COMMONPARMS;
 
     ts = cmd_CreateSyntax("remsite", RemoveSite, NULL,
@@ -5914,7 +6005,7 @@ main(argc, argv)
     cmd_AddParm(ts, "-clearVolUpCounter", CMD_FLAG, CMD_OPTIONAL, "clear volUpdateCounter");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("offline", volOffline, NULL, (char *)CMD_HIDDEN);
+    ts = cmd_CreateSyntax("offline", volOffline, NULL, "force the volume status to offline");
     cmd_AddParm(ts, "-server", CMD_SINGLE, 0, "server name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, 0, "partition name");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
@@ -5922,7 +6013,7 @@ main(argc, argv)
     cmd_AddParm(ts, "-busy", CMD_FLAG, CMD_OPTIONAL, "busy volume");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("online", volOnline, NULL, (char *)CMD_HIDDEN);
+    ts = cmd_CreateSyntax("online", volOnline, NULL, "force the volume status to online");
     cmd_AddParm(ts, "-server", CMD_SINGLE, 0, "server name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, 0, "partition name");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
@@ -5980,7 +6071,7 @@ main(argc, argv)
                "prefix of the volume whose VLDB entry is to be deleted");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_OPTIONAL, "machine name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, CMD_OPTIONAL, "partition name");
-    cmd_AddParm(ts, "-noexecute", CMD_FLAG, CMD_OPTIONAL | CMD_HIDE,
+    cmd_AddParm(ts, "-noexecute", CMD_FLAG, CMD_OPTIONAL,
                "no execute");
     COMMONPARMS;