vos: Remove dead code
[openafs.git] / src / volser / vos.c
index 360e217..6fcdec8 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * Copyright 2000, International Business Machines Corporation and others.
  * All Rights Reserved.
- * 
+ *
  * This software has been released under the terms of the IBM Public
  * License.  For details, see the LICENSE file in the top-level source
  * directory or online at http://www.openafs.org/dl/license10.html
 #include <afsconfig.h>
 #include <afs/param.h>
 
+#include <roken.h>
+
 #ifdef IGNORE_SOME_GCC_WARNINGS
 # pragma GCC diagnostic warning "-Wimplicit-function-declaration"
 #endif
 
-#include <sys/types.h>
-#include <string.h>
 #ifdef AFS_NT40_ENV
-#include <fcntl.h>
-#include <io.h>
-#include <winsock2.h>
 #include <WINNT/afsreg.h>
-#else
-#include <sys/time.h>
-#include <sys/file.h>
-#include <netdb.h>
-#include <netinet/in.h>
-#include <arpa/inet.h>
 #endif
-#include <sys/stat.h>
+
 #ifdef AFS_AIX_ENV
 #include <sys/statfs.h>
 #endif
-#include <errno.h>
 
 #include <lock.h>
 #include <afs/stds.h>
+#include <rx/rx_queue.h>
 #include <rx/xdr.h>
 #include <rx/rx.h>
 #include <rx/rx_globals.h>
@@ -57,9 +48,6 @@
 #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"
@@ -71,8 +59,8 @@
 
 /* Local Prototypes */
 int PrintDiagnostics(char *astring, afs_int32 acode);
-int GetVolumeInfo(afs_uint32 volid, afs_int32 *server, afs_int32 *part, 
-                  afs_int32 *voltype, struct nvldbentry *rentry);
+int GetVolumeInfo(afs_uint32 volid, afs_uint32 *server, afs_int32 *part,
+                 afs_int32 *voltype, struct nvldbentry *rentry);
 
 struct tqElem {
     afs_uint32 volid;
@@ -84,19 +72,42 @@ struct tqHead {
     struct tqElem *next;
 };
 
-#define COMMONPARMS     cmd_Seek(ts, 12);\
-cmd_AddParm(ts, "-cell", CMD_SINGLE, CMD_OPTIONAL, "cell name");\
-cmd_AddParm(ts, "-noauth", CMD_FLAG, CMD_OPTIONAL, "don't authenticate");\
-cmd_AddParm(ts, "-localauth",CMD_FLAG,CMD_OPTIONAL,"use server tickets");\
-cmd_AddParm(ts, "-verbose", CMD_FLAG, CMD_OPTIONAL, "verbose");\
-cmd_AddParm(ts, "-encrypt", CMD_FLAG, CMD_OPTIONAL, "encrypt commands");\
-cmd_AddParm(ts, "-noresolve", CMD_FLAG, CMD_OPTIONAL, "don't resolve addresses"); \
+enum {
+    COMMONPARM_OFFSET_CELL      = 25,
+    COMMONPARM_OFFSET_NOAUTH    = 26,
+    COMMONPARM_OFFSET_LOCALAUTH = 27,
+    COMMONPARM_OFFSET_VERBOSE   = 28,
+    COMMONPARM_OFFSET_ENCRYPT   = 29,
+    COMMONPARM_OFFSET_NORESOLVE = 30,
+    COMMONPARM_OFFSET_CONFIG    = 31,
+    COMMONPARM_OFFSET_RXGK      = 32,
+};
 
-#define ERROR_EXIT(code) {error=(code); goto error_exit;}
+#define COMMONPARMS \
+cmd_AddParmAtOffset(ts, COMMONPARM_OFFSET_CELL, \
+    "-cell", CMD_SINGLE, CMD_OPTIONAL, "cell name");\
+cmd_AddParmAlias(ts, COMMONPARM_OFFSET_CELL, "-c");   /* original -cell option */ \
+cmd_AddParmAtOffset(ts, COMMONPARM_OFFSET_NOAUTH, \
+    "-noauth", CMD_FLAG, CMD_OPTIONAL, "don't authenticate");\
+cmd_AddParmAtOffset(ts, COMMONPARM_OFFSET_LOCALAUTH, \
+    "-localauth",CMD_FLAG,CMD_OPTIONAL,"use server tickets");\
+cmd_AddParmAtOffset(ts, COMMONPARM_OFFSET_VERBOSE, \
+    "-verbose", CMD_FLAG, CMD_OPTIONAL, "verbose");\
+cmd_AddParmAtOffset(ts, COMMONPARM_OFFSET_ENCRYPT, \
+    "-encrypt", CMD_FLAG, CMD_OPTIONAL, "encrypt commands");\
+cmd_AddParmAtOffset(ts, COMMONPARM_OFFSET_NORESOLVE, \
+    "-noresolve", CMD_FLAG, CMD_OPTIONAL, "don't resolve addresses"); \
+cmd_AddParmAtOffset(ts, COMMONPARM_OFFSET_CONFIG, \
+    "-config", CMD_SINGLE, CMD_OPTIONAL, "config location"); \
+cmd_AddParmAtOffset(ts, COMMONPARM_OFFSET_RXGK, \
+    "-rxgk", CMD_SINGLE, CMD_OPTIONAL, "rxgk security level to use"); \
+
+#define ERROR_EXIT(code) do { \
+    error = (code); \
+    goto error_exit; \
+} while (0)
 
 int rxInitDone = 0;
-struct rx_connection *tconn;
-afs_int32 tserver;
 extern struct ubik_client *cstruct;
 const char *confdir;
 
@@ -109,13 +120,12 @@ qInit(struct tqHead *ahead)
     return;
 }
 
-
 static void
 qPut(struct tqHead *ahead, afs_uint32 volid)
 {
     struct tqElem *elem;
 
-    elem = (struct tqElem *)malloc(sizeof(struct tqElem));
+    elem = malloc(sizeof(struct tqElem));
     elem->next = ahead->next;
     elem->volid = volid;
     ahead->next = elem;
@@ -144,7 +154,7 @@ FileExists(char *filename)
 {
     usd_handle_t ufd;
     int code;
-    afs_hyper_t size;
+    afs_int64 size;
 
     code = usd_Open(filename, USD_OPEN_RDONLY, 0, &ufd);
     if (code) {
@@ -162,8 +172,7 @@ FileExists(char *filename)
 static int
 VolNameOK(char *name)
 {
-    int total;
-
+    size_t total;
 
     total = strlen(name);
     if (!strcmp(&name[total - 9], ".readonly")) {
@@ -179,7 +188,8 @@ VolNameOK(char *name)
 static int
 IsNumeric(char *name)
 {
-    int result, len, i;
+    int result, i;
+    size_t len;
     char *ptr;
 
     result = 1;
@@ -196,59 +206,85 @@ IsNumeric(char *name)
     return result;
 }
 
-
 /*
- * Parse a server name/address and return the address in HOST BYTE order
+ * Parse a server dotted address and return the address in network byte order
  */
-afs_int32
-GetServer(char *aname)
+afs_uint32
+GetServerNoresolve(char *aname)
 {
-    register struct hostent *th;
-    afs_int32 addr;
     int b1, b2, b3, b4;
-    register afs_int32 code;
-    char hostname[MAXHOSTCHARS];
+    afs_uint32 addr;
+    afs_int32 code;
 
     code = sscanf(aname, "%d.%d.%d.%d", &b1, &b2, &b3, &b4);
     if (code == 4) {
        addr = (b1 << 24) | (b2 << 16) | (b3 << 8) | b4;
-       addr = ntohl(addr);     /* convert to host order */
-    } else {
-       th = gethostbyname(aname);
-       if (!th)
-           return 0;
-       memcpy(&addr, th->h_addr, sizeof(addr));
-    }
+       addr = htonl(addr);     /* convert to network byte order */
+       return addr;
+    } else
+       return 0;
+}
+/*
+ * Parse a server name/address and return a non-loopback address in network byte order
+ */
+afs_uint32
+GetServer(char *aname)
+{
+    struct hostent *th;
+    afs_uint32 addr; /* in network byte order */
+    afs_int32 code;
+    char hostname[MAXHOSTCHARS];
+    afs_uint32 **addr_list;
+    int i;
 
-    if (addr == htonl(0x7f000001)) {   /* local host */
-       code = gethostname(hostname, MAXHOSTCHARS);
-       if (code)
-           return 0;
-       th = gethostbyname(hostname);   /* returns host byte order */
-       if (!th)
+    addr = GetServerNoresolve(aname);
+    if (addr != 0) {
+       if (!rx_IsLoopbackAddr(ntohl(addr)))
+           return addr;
+       else
            return 0;
-       memcpy(&addr, th->h_addr, sizeof(addr));
     }
 
-    return (addr);
-}
+    th = gethostbyname(aname);
+    if (th != NULL && th->h_addrtype == AF_INET) {
+       addr_list = (afs_uint32 **)th->h_addr_list;
+       for(i = 0; addr_list[i] != NULL; i++) {
+           if (!rx_IsLoopbackAddr(ntohl(*addr_list[i]))) {
+               memcpy(&addr, addr_list[i], sizeof(addr));
+               return addr;
+           }
+       }
 
-afs_int32
-GetVolumeType(char *aname)
-{
+       /*
+        * If we reach this point all of the addresses returned by
+        * gethostbyname() are loopback addresses.  We assume that means
+        * that the name is supposed to describe the machine this code
+        * is executing on.  Try gethostname() to and check to see if
+        * that name can provide us a non-loopback address.
+        */
+       code = gethostname(hostname, MAXHOSTCHARS);
+       if (code == 0) {
+           th = gethostbyname(hostname);
+           if (th != NULL && th->h_addrtype == AF_INET) {
+               addr_list = (afs_uint32 **)th->h_addr_list;
+               for (i=0; addr_list[i] != NULL; i++) {
+                   if (!rx_IsLoopbackAddr(ntohl(*addr_list[i]))) {
+                       memcpy(&addr, addr_list[i], sizeof(addr));
+                       return addr;
+                   }
+               }
+           }
+       }
+    }
 
-    if (!strcmp(aname, "ro"))
-       return (ROVOL);
-    else if (!strcmp(aname, "rw"))
-       return (RWVOL);
-    else if (!strcmp(aname, "bk"))
-       return (BACKVOL);
-    else
-       return (-1);
+    /*
+     * No non-loopback address could be obtained for 'aname'.
+     */
+    return 0;
 }
 
 int
-IsPartValid(afs_int32 partId, afs_int32 server, afs_int32 *code)
+IsPartValid(afs_int32 partId, afs_uint32 server, afs_int32 *code)
 {
     struct partList dummyPartList;
     int i, success, cnt;
@@ -267,46 +303,42 @@ IsPartValid(afs_int32 partId, afs_int32 server, afs_int32 *code)
     return success;
 }
 
-
-
- /*sends the contents of file associated with <fd> and <blksize>  to Rx Stream 
+ /*sends the contents of file associated with <fd> and <blksize>  to Rx Stream
   * associated  with <call> */
-int 
-SendFile(usd_handle_t ufd, register struct rx_call *call, long blksize)
+int
+SendFile(usd_handle_t ufd, struct rx_call *call, long blksize)
 {
     char *buffer = (char *)0;
     afs_int32 error = 0;
-    int done = 0;
     afs_uint32 nbytes;
 
-    buffer = (char *)malloc(blksize);
+    buffer = malloc(blksize);
     if (!buffer) {
        fprintf(STDERR, "malloc failed\n");
        return -1;
     }
 
-    while (!error && !done) {
-#ifndef AFS_NT40_ENV           /* NT csn't select on non-socket fd's */
+    while (!error) {
+#if !defined(AFS_NT40_ENV) && !defined(AFS_PTHREAD_ENV)
+       /* Only for this for non-NT, non-pthread. For NT, we can't select on
+        * non-socket FDs. For pthread environments, we don't need to select at
+        * all, since the following read() will block. */
        fd_set in;
        FD_ZERO(&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(((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: %s\n",
-                   afs_error_message(error));
+                   afs_error_message(error));
            break;
        }
-       if (nbytes == 0) {
-           done = 1;
+
+       if (nbytes == 0)
            break;
-       }
+
        if (rx_Write(call, buffer, nbytes) != nbytes) {
            error = -1;
            break;
@@ -327,9 +359,9 @@ WriteData(struct rx_call *call, void *rock)
     long blksize;
     afs_int32 error, code;
     int ufdIsOpen = 0;
-    afs_hyper_t filesize, currOffset; 
-    afs_uint32 buffer;         
-    afs_uint32 got;            
+    afs_int64 currOffset;
+    afs_uint32 buffer;
+    afs_uint32 got;
 
     error = 0;
 
@@ -345,24 +377,20 @@ WriteData(struct rx_call *call, void *rock)
        }
        if (code) {
            fprintf(STDERR, "Could not access file '%s': %s\n", filename,
-                   afs_error_message(code));
+                   afs_error_message(code));
            error = VOLSERBADOP;
            goto wfail;
        }
        /* test if we have a valid dump */
-       hset64(filesize, 0, 0);
-       USD_SEEK(ufd, filesize, SEEK_END, &currOffset);
-       hset64(filesize, hgethi(currOffset), hgetlo(currOffset)-sizeof(afs_uint32));
-       USD_SEEK(ufd, filesize, SEEK_SET, &currOffset);
+       USD_SEEK(ufd, 0, SEEK_END, &currOffset);
+       USD_SEEK(ufd, currOffset - sizeof(afs_uint32), SEEK_SET, &currOffset);
        USD_READ(ufd, (char *)&buffer, sizeof(afs_uint32), &got);
        if ((got != sizeof(afs_uint32)) || (ntohl(buffer) != DUMPENDMAGIC)) {
            fprintf(STDERR, "Signature missing from end of file '%s'\n", filename);
            error = VOLSERBADOP;
            goto wfail;
        }
-       /* rewind, we are done */
-       hset64(filesize, 0, 0);
-       USD_SEEK(ufd, filesize, SEEK_SET, &currOffset);
+       USD_SEEK(ufd, 0, SEEK_SET, &currOffset);
     }
     code = SendFile(ufd, call, blksize);
     if (code) {
@@ -393,7 +421,7 @@ ReceiveFile(usd_handle_t ufd, struct rx_call *call, long blksize)
     afs_uint32 bytesleft, w;
     afs_int32 error = 0;
 
-    buffer = (char *)malloc(blksize);
+    buffer = malloc(blksize);
     if (!buffer) {
        fprintf(STDERR, "memory allocation failed\n");
        ERROR_EXIT(-1);
@@ -401,22 +429,21 @@ ReceiveFile(usd_handle_t ufd, struct rx_call *call, long blksize)
 
     while ((bytesread = rx_Read(call, buffer, blksize)) > 0) {
        for (bytesleft = bytesread; bytesleft; bytesleft -= w) {
-#ifndef AFS_NT40_ENV           /* NT csn't select on non-socket fd's */
+#if !defined(AFS_NT40_ENV) && !defined(AFS_PTHREAD_ENV)
+           /* Only for this for non-NT, non-pthread. For NT, we can't select
+            * on non-socket FDs. For pthread environments, we don't need to
+            * select at all, since the following write() will block. */
            fd_set out;
            FD_ZERO(&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(((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: %s\n",
-                       afs_error_message(error));
+                       afs_error_message(error));
                ERROR_EXIT(-1);
            }
        }
@@ -434,7 +461,7 @@ 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;
+    afs_int64 size;
     long blksize;
     int ufdIsOpen = 0;
 
@@ -448,7 +475,7 @@ DumpFunction(struct rx_call *call, void *rock)
            usd_Open(filename, USD_OPEN_CREATE | USD_OPEN_RDWR, 0666, &ufd);
        if (code == 0) {
            ufdIsOpen = 1;
-           hzero(size);
+           size = 0;
            code = USD_IOCTL(ufd, USD_IOCTL_SETSIZE, &size);
        }
        if (code == 0) {
@@ -456,7 +483,7 @@ DumpFunction(struct rx_call *call, void *rock)
        }
        if (code) {
            fprintf(STDERR, "Could not create file '%s': %s\n", filename,
-                   afs_error_message(code));
+                   afs_error_message(code));
            ERROR_EXIT(VOLSERBADOP);
        }
     }
@@ -481,7 +508,7 @@ DumpFunction(struct rx_call *call, void *rock)
 }
 
 static void
-DisplayFormat(volintInfo *pntr, afs_int32 server, afs_int32 part,
+DisplayFormat(volintInfo *pntr, afs_uint32 server, afs_int32 part,
              int *totalOK, int *totalNotOK, int *totalBusy, int fast,
              int longlist, int disp)
 {
@@ -522,7 +549,6 @@ DisplayFormat(volintInfo *pntr, afs_int32 server, afs_int32 part,
            t = pntr->creationDate;
            fprintf(STDOUT, "    Creation    %s",
                    ctime(&t));
-#ifdef FULL_LISTVOL_SWITCH
            t = pntr->copyDate;
            fprintf(STDOUT, "    Copy        %s",
                    ctime(&t));
@@ -538,7 +564,7 @@ DisplayFormat(volintInfo *pntr, afs_int32 server, afs_int32 part,
            if (t)
                fprintf(STDOUT, "    Last Access %s",
                        ctime(&t));
-#endif
+
            t = pntr->updateDate;
            if (!t)
                fprintf(STDOUT, "    Last Update Never\n");
@@ -627,7 +653,7 @@ DisplayFormat(volintInfo *pntr, afs_int32 server, afs_int32 part,
  *------------------------------------------------------------------------*/
 
 static void
-XDisplayFormat(volintXInfo *a_xInfoP, afs_int32 a_servID, afs_int32 a_partID,
+XDisplayFormat(volintXInfo *a_xInfoP, afs_uint32 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 */
@@ -677,7 +703,7 @@ XDisplayFormat(volintXInfo *a_xInfoP, afs_int32 a_servID, afs_int32 a_partID,
            t = a_xInfoP->creationDate;
            fprintf(STDOUT, "    Creation    %s",
                    ctime(&t));
-#ifdef FULL_LISTVOL_SWITCH
+
            t = a_xInfoP->copyDate;
            fprintf(STDOUT, "    Copy        %s",
                    ctime(&t));
@@ -693,7 +719,7 @@ XDisplayFormat(volintXInfo *a_xInfoP, afs_int32 a_servID, afs_int32 a_partID,
            if (t)
                fprintf(STDOUT, "    Last Access %s",
                        ctime(&t));
-#endif
+
            t = a_xInfoP->updateDate;
            if (!t)
                fprintf(STDOUT, "    Last Update Never\n");
@@ -832,7 +858,6 @@ XDisplayFormat(volintXInfo *a_xInfoP, afs_int32 a_servID, afs_int32 a_partID,
     }                          /*Default listing */
 }                              /*XDisplayFormat */
 
-#ifdef FULL_LISTVOL_SWITCH
 /*------------------------------------------------------------------------
  * PRIVATE XDisplayFormat2
  *
@@ -861,7 +886,7 @@ XDisplayFormat(volintXInfo *a_xInfoP, afs_int32 a_servID, afs_int32 a_partID,
  *------------------------------------------------------------------------*/
 
 static void
-XDisplayFormat2(volintXInfo *a_xInfoP, afs_int32 a_servID, afs_int32 a_partID,
+XDisplayFormat2(volintXInfo *a_xInfoP, afs_uint32 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 */
@@ -880,10 +905,10 @@ XDisplayFormat2(volintXInfo *a_xInfoP, afs_int32 a_servID, afs_int32 a_partID,
             * Volume's status is OK - all the fields are valid.
             */
 
-                static long server_cache = -1, partition_cache = -1;
-                static char hostname[256], address[32], pname[16];
-                int i,ai[] = {VOLINT_STATS_TIME_IDX_0,VOLINT_STATS_TIME_IDX_1,VOLINT_STATS_TIME_IDX_2,
-                              VOLINT_STATS_TIME_IDX_3,VOLINT_STATS_TIME_IDX_4,VOLINT_STATS_TIME_IDX_5};
+               static long server_cache = -1, partition_cache = -1;
+               static char hostname[256], address[32], pname[16];
+               int i,ai[] = {VOLINT_STATS_TIME_IDX_0,VOLINT_STATS_TIME_IDX_1,VOLINT_STATS_TIME_IDX_2,
+                             VOLINT_STATS_TIME_IDX_3,VOLINT_STATS_TIME_IDX_4,VOLINT_STATS_TIME_IDX_5};
 
                if (a_servID != server_cache) {
                        struct in_addr s;
@@ -902,12 +927,12 @@ XDisplayFormat2(volintXInfo *a_xInfoP, afs_int32 a_servID, afs_int32 a_partID,
                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);
-                fprintf(STDOUT, "status\t\tOK\n");
-               fprintf(STDOUT, "backupID\t%lu\n", 
+               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", 
+               fprintf(STDOUT, "parentID\t%lu\n",
                        afs_printable_uint32_lu(a_xInfoP->parentID));
-               fprintf(STDOUT, "cloneID\t\t%lu\n", 
+               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) {
@@ -925,38 +950,36 @@ XDisplayFormat2(volintXInfo *a_xInfoP, afs_int32 a_servID, afs_int32 a_partID,
                        break;
                }
                t = a_xInfoP->creationDate;
-               fprintf(STDOUT, "creationDate\t%-9lu\t%s", 
+               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", 
+               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", 
+               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", 
+               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", 
+               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);
 
                fprintf(STDOUT, "filecount\t%u\n", a_xInfoP->filecount);
                fprintf(STDOUT, "dayUse\t\t%u\n", a_xInfoP->dayUse);
 
-
-
                fprintf(STDOUT,"reads_same_net\t%8d\n",a_xInfoP->stat_reads[VOLINT_STATS_SAME_NET]);
                fprintf(STDOUT,"reads_same_net_auth\t%8d\n",a_xInfoP->stat_reads[VOLINT_STATS_SAME_NET_AUTH]);
                fprintf(STDOUT,"reads_diff_net\t%8d\n",a_xInfoP->stat_reads[VOLINT_STATS_DIFF_NET]);
@@ -1008,7 +1031,7 @@ XDisplayFormat2(volintXInfo *a_xInfoP, afs_int32 a_servID, afs_int32 a_partID,
 
            fprintf(STDOUT, "inUse\t%d\n",a_xInfoP->inUse);
            if (a_xInfoP->inUse == 1)
-               (*a_totalOKP)++;
+               (*a_totalOKP)++;
            else
                (*a_totalNotOKP)++;
 
@@ -1029,9 +1052,7 @@ XDisplayFormat2(volintXInfo *a_xInfoP, afs_int32 a_servID, afs_int32 a_partID,
        }                       /*Screwed volume */
     }                          /*Default listing */
 }                              /*XDisplayFormat */
-#endif /*FULL_LISTVOL_SWITCH*/
 
-#ifdef FULL_LISTVOL_SWITCH
 static void
 DisplayFormat2(long server, long partition, volintInfo *pntr)
 {
@@ -1053,9 +1074,9 @@ DisplayFormat2(long server, long partition, volintInfo *pntr)
     }
 
     if (pntr->status == VOK)
-        fprintf(STDOUT, "name\t\t%s\n", pntr->name);
+       fprintf(STDOUT, "name\t\t%s\n", pntr->name);
 
-    fprintf(STDOUT, "id\t\t%lu\n", 
+    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);
@@ -1070,11 +1091,11 @@ DisplayFormat2(long server, long partition, volintInfo *pntr)
        fprintf(STDOUT, "status\t\tUNATTACHABLE\n");
        return;
     }
-    fprintf(STDOUT, "backupID\t%lu\n", 
+    fprintf(STDOUT, "backupID\t%lu\n",
            afs_printable_uint32_lu(pntr->backupID));
-    fprintf(STDOUT, "parentID\t%lu\n", 
+    fprintf(STDOUT, "parentID\t%lu\n",
            afs_printable_uint32_lu(pntr->parentID));
-    fprintf(STDOUT, "cloneID\t\t%lu\n", 
+    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");
@@ -1095,43 +1116,43 @@ DisplayFormat2(long server, long partition, volintInfo *pntr)
        break;
     }
     t = pntr->creationDate;
-    fprintf(STDOUT, "creationDate\t%-9lu\t%s", 
+    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", 
+    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", 
+    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", 
+    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", 
+    fprintf(STDOUT, "copyDate\t%-9lu\t%s",
            afs_printable_uint32_lu(pntr->copyDate),
            ctime(&t));
 
-    fprintf(STDOUT, "flags\t\t%#lx\t(Optional)\n", 
+    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", 
+    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",
            afs_printable_uint32_lu(pntr->spare1));
-    fprintf(STDOUT, "spare2\t\t%lu\t(Optional)\n", 
+    fprintf(STDOUT, "spare2\t\t%lu\t(Optional)\n",
            afs_printable_uint32_lu(pntr->spare2));
-    fprintf(STDOUT, "spare3\t\t%lu\t(Optional)\n", 
+    fprintf(STDOUT, "spare3\t\t%lu\t(Optional)\n",
            afs_printable_uint32_lu(pntr->spare3));
     return;
 }
@@ -1149,10 +1170,9 @@ DisplayVolumes2(long server, long partition, volintInfo *pntr, long count)
     }
     return;
 }
-#endif /* FULL_LISTVOL_SWITCH */
 
 static void
-DisplayVolumes(afs_int32 server, afs_int32 part, volintInfo *pntr,
+DisplayVolumes(afs_uint32 server, afs_int32 part, volintInfo *pntr,
               afs_int32 count, afs_int32 longlist, afs_int32 fast,
               int quiet)
 {
@@ -1218,7 +1238,7 @@ DisplayVolumes(afs_int32 server, afs_int32 part, volintInfo *pntr,
  *------------------------------------------------------------------------*/
 
 static void
-XDisplayVolumes(afs_int32 a_servID, afs_int32 a_partID, volintXInfo *a_xInfoP,
+XDisplayVolumes(afs_uint32 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 */
@@ -1275,7 +1295,7 @@ XDisplayVolumes(afs_int32 a_servID, afs_int32 a_partID, volintXInfo *a_xInfoP,
     }
 
 }                              /*XDisplayVolumes */
-#ifdef FULL_LISTVOL_SWITCH
+
 /*------------------------------------------------------------------------
  * PRIVATE XDisplayVolumes2
  *
@@ -1302,7 +1322,7 @@ XDisplayVolumes(afs_int32 a_servID, afs_int32 a_partID, volintXInfo *a_xInfoP,
  *------------------------------------------------------------------------*/
 
 static void
-XDisplayVolumes2(afs_int32 a_servID, afs_int32 a_partID, volintXInfo *a_xInfoP,
+XDisplayVolumes2(afs_uint32 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 */
@@ -1361,27 +1381,33 @@ XDisplayVolumes2(afs_int32 a_servID, afs_int32 a_partID, volintXInfo *a_xInfoP,
     }
 
 }                              /*XDisplayVolumes2 */
-#endif /* FULL_LISTVOL_SWITCH */
-
 
-/* set <server> and <part> to the correct values depending on 
- * <voltype> and <entry> */
-static void
-GetServerAndPart(struct nvldbentry *entry, int voltype, afs_int32 *server,
+/**
+ * Retrieve the sites (i.e., server id, partition id) from a vldb entry for a
+ * given volume type.  May be called repeatedly to get each read-only site.
+ *
+ * @param[in] entry       vldb entry from a previous vldb query
+ * @param[in] voltype     type of volume of interest (rw, ro, bk)
+ * @param[out] server     vldb entry server number for voltype
+ * @param[out] part       vldb part id for voltype
+ * @param[inout] previdx  cursor; should be initialized to -1
+ *                        before first call on a given entry.
+ *
+ * @returns true when a volume site has been found
+ */
+static int
+GetServerAndPart(struct nvldbentry *entry, int voltype, afs_uint32 *server,
                 afs_int32 *part, int *previdx)
 {
     int i, istart, vtype;
 
-    *server = -1;
-    *part = -1;
-
-    /* Doesn't check for non-existance of backup volume */
+    /* Doesn't check for non-existence of backup volume */
     if ((voltype == RWVOL) || (voltype == BACKVOL)) {
-       vtype = ITSRWVOL;
-       istart = 0;             /* seach the entire entry */
+       vtype = VLSF_RWVOL;
+       istart = 0;             /* search the entire entry */
     } else {
-       vtype = ITSROVOL;
-       /* Seach from beginning of entry or pick up where we left off */
+       vtype = VLSF_ROVOL;
+       /* Search from beginning of entry or pick up where we left off */
        istart = ((*previdx < 0) ? 0 : *previdx + 1);
     }
 
@@ -1390,13 +1416,39 @@ GetServerAndPart(struct nvldbentry *entry, int voltype, afs_int32 *server,
            *server = entry->serverNumber[i];
            *part = entry->serverPartition[i];
            *previdx = i;
-           return;
+           return 1;
        }
     }
 
-    /* Didn't find any, return -1 */
+    /* Didn't find any more, reset index. */
     *previdx = -1;
-    return;
+    return 0;
+}
+
+static void
+PrintLocked(afs_int32 aflags)
+{
+    afs_int32 flags = aflags & VLOP_ALLOPERS;
+
+    if (flags) {
+       fprintf(STDOUT, "    Volume is currently LOCKED  \n");
+
+       if (flags & VLOP_MOVE) {
+           fprintf(STDOUT, "    Volume is locked for a move operation\n");
+       }
+       if (flags & VLOP_RELEASE) {
+           fprintf(STDOUT, "    Volume is locked for a release operation\n");
+       }
+       if (flags & VLOP_BACKUP) {
+           fprintf(STDOUT, "    Volume is locked for a backup operation\n");
+       }
+       if (flags & VLOP_DELETE) {
+           fprintf(STDOUT, "    Volume is locked for a delete/misc operation\n");
+       }
+       if (flags & VLOP_DUMP) {
+           fprintf(STDOUT, "    Volume is locked for a dump/restore operation\n");
+       }
+    }
 }
 
 static void
@@ -1404,8 +1456,7 @@ PostVolumeStats(struct nvldbentry *entry)
 {
     SubEnumerateEntry(entry);
     /* Check for VLOP_ALLOPERS */
-    if (entry->flags & VLOP_ALLOPERS)
-       fprintf(STDOUT, "    Volume is currently LOCKED  \n");
+    PrintLocked(entry->flags);
     return;
 }
 
@@ -1453,10 +1504,12 @@ XVolumeStats(volintXInfo *a_xInfoP, struct nvldbentry *a_entryP,
 }                              /*XVolumeStats */
 
 static void
-VolumeStats_int(volintInfo *pntr, struct nvldbentry *entry, afs_int32 server, 
+VolumeStats_int(volintInfo *pntr, struct nvldbentry *entry, afs_uint32 server,
             afs_int32 part, int voltype)
 {
-    int totalOK, totalNotOK, totalBusy;
+    int totalOK = 0;
+    int totalNotOK = 0;
+    int totalBusy = 0;
 
     DisplayFormat(pntr, server, part, &totalOK, &totalNotOK, &totalBusy, 0, 1,
                  1);
@@ -1465,14 +1518,14 @@ VolumeStats_int(volintInfo *pntr, struct nvldbentry *entry, afs_int32 server,
 
 /* command to forcibly remove a volume */
 static int
-NukeVolume(register struct cmd_syndesc *as)
+NukeVolume(struct cmd_syndesc *as)
 {
-    register afs_int32 code;
+    afs_int32 code;
     afs_uint32 volID;
     afs_int32  err;
     afs_int32 partID;
-    afs_int32 server;
-    register char *tp;
+    afs_uint32 server;
+    char *tp;
 
     server = GetServer(tp = as->parms[0].items->data);
     if (!server) {
@@ -1508,7 +1561,6 @@ NukeVolume(register struct cmd_syndesc *as)
     return code;
 }
 
-
 /*------------------------------------------------------------------------
  * PRIVATE ExamineVolume
  *
@@ -1531,7 +1583,7 @@ NukeVolume(register struct cmd_syndesc *as)
  *------------------------------------------------------------------------
  */
 static int
-ExamineVolume(register struct cmd_syndesc *as, void *arock)
+ExamineVolume(struct cmd_syndesc *as, void *arock)
 {
     struct nvldbentry entry;
     afs_int32 vcode = 0;
@@ -1540,10 +1592,11 @@ ExamineVolume(register struct cmd_syndesc *as, void *arock)
     afs_uint32 volid;
     afs_int32 code, err, error = 0;
     int voltype, foundserv = 0, foundentry = 0;
-    afs_int32 aserver, apart;
+    afs_uint32 aserver;
+    afs_int32 apart;
     int previdx = -1;
     int wantExtendedInfo;      /*Do we want extended vol info? */
-
+    int isSubEnum=0;           /* Keep track whether sub enumerate called. */
     wantExtendedInfo = (as->parms[1].items ? 1 : 0);   /* -extended */
 
     volid = vsu_GetVolumeID(as->parms[0].items->data, cstruct, &err);  /* -id */
@@ -1584,17 +1637,14 @@ ExamineVolume(register struct cmd_syndesc *as, void *arock)
         * It its a BK vol, get the RW entry (even if VLDB may say the BK doen't exist).
         * If its a RO vol, get the next RO entry.
         */
-       GetServerAndPart(&entry, ((voltype == ROVOL) ? ROVOL : RWVOL),
-                        &aserver, &apart, &previdx);
-       if (previdx == -1) {    /* searched all entries */
-           if (!foundentry) {
-               fprintf(STDERR, "Volume %s does not exist in VLDB\n\n",
-                       as->parms[0].items->data);
-               error = ENOENT;
-           }
+       foundentry = GetServerAndPart(&entry, ((voltype == ROVOL) ? ROVOL : RWVOL),
+                                     &aserver, &apart, &previdx);
+       if (!foundentry) {      /* searched all entries */
+           fprintf(STDERR, "Volume %s does not exist in VLDB\n\n",
+                   as->parms[0].items->data);
+           error = ENOENT;
            break;
        }
-       foundentry = 1;
 
        /* Get information about the volume from the server */
        if (verbose) {
@@ -1612,7 +1662,7 @@ ExamineVolume(register struct cmd_syndesc *as, void *arock)
        if (code) {
            error = code;
            if (code == ENODEV) {
-               if ((voltype == BACKVOL) && !(entry.flags & BACK_EXISTS)) {
+               if ((voltype == BACKVOL) && !(entry.flags & VLF_BACKEXISTS)) {
                    /* The VLDB says there is no backup volume and its not on disk */
                    fprintf(STDERR, "Volume %s does not exist\n",
                            as->parms[0].items->data);
@@ -1630,16 +1680,14 @@ ExamineVolume(register struct cmd_syndesc *as, void *arock)
            foundserv = 1;
            if (wantExtendedInfo)
                XVolumeStats(xInfoP, &entry, aserver, apart, voltype);
-           else
-#ifdef FULL_LISTVOL_SWITCH
-           if (as->parms[2].items) {
+           else if (as->parms[2].items) {
                DisplayFormat2(aserver, apart, pntr);
                EnumerateEntry(&entry);
+               isSubEnum = 1;
            } else
-#endif /* FULL_LISTVOL_SWITCH */
                VolumeStats_int(pntr, &entry, aserver, apart, voltype);
 
-           if ((voltype == BACKVOL) && !(entry.flags & BACK_EXISTS)) {
+           if ((voltype == BACKVOL) && !(entry.flags & VLF_BACKEXISTS)) {
                /* The VLDB says there is no backup volume yet we found one on disk */
                fprintf(STDERR, "Volume %s does not exist in VLDB\n",
                        as->parms[0].items->data);
@@ -1657,7 +1705,9 @@ ExamineVolume(register struct cmd_syndesc *as, void *arock)
        fprintf(STDERR, "Dump only information from VLDB\n\n");
        fprintf(STDOUT, "%s \n", entry.name);   /* PostVolumeStats doesn't print name */
     }
-    PostVolumeStats(&entry);
+
+    if (!isSubEnum)
+       PostVolumeStats(&entry);
 
     return (error);
 }
@@ -1683,14 +1733,17 @@ ExamineVolume(register struct cmd_syndesc *as, void *arock)
  *------------------------------------------------------------------------
  */
 static int
-SetFields(register struct cmd_syndesc *as, void *arock)
+SetFields(struct cmd_syndesc *as, void *arock)
 {
     struct nvldbentry entry;
     volintInfo info;
     afs_uint32 volid;
     afs_int32 code, err;
-    afs_int32 aserver, apart;
+    afs_uint32 aserver;
+    afs_int32 apart;
     int previdx = -1;
+    int have_field = 0;
+    int found;
 
     volid = vsu_GetVolumeID(as->parms[0].items->data, cstruct, &err);  /* -id */
     if (volid == 0) {
@@ -1711,8 +1764,8 @@ SetFields(register struct cmd_syndesc *as, void *arock)
     }
     MapHostToNetwork(&entry);
 
-    GetServerAndPart(&entry, RWVOL, &aserver, &apart, &previdx);
-    if (previdx == -1) {
+    found = GetServerAndPart(&entry, RWVOL, &aserver, &apart, &previdx);
+    if (!found) {
        fprintf(STDERR, "Volume %s does not exist in VLDB\n\n",
                as->parms[0].items->data);
        return (ENOENT);
@@ -1724,6 +1777,7 @@ SetFields(register struct cmd_syndesc *as, void *arock)
 
     if (as->parms[1].items) {
        /* -max <quota> */
+       have_field = 1;
        code = util_GetHumanInt32(as->parms[1].items->data, &info.maxquota);
        if (code) {
            fprintf(STDERR, "invalid quota value\n");
@@ -1732,12 +1786,18 @@ SetFields(register struct cmd_syndesc *as, void *arock)
     }
     if (as->parms[2].items) {
        /* -clearuse */
+       have_field = 1;
        info.dayUse = 0;
     }
     if (as->parms[3].items) {
        /* -clearVolUpCounter */
+       have_field = 1;
        info.spare2 = 0;
     }
+    if (!have_field) {
+       fprintf(STDERR,"Nothing to set.\n");
+       return (1);
+    }
     code = UV_SetVolumeInfo(aserver, apart, volid, &info);
     if (code)
        fprintf(STDERR,
@@ -1766,9 +1826,10 @@ SetFields(register struct cmd_syndesc *as, void *arock)
  *------------------------------------------------------------------------
  */
 static int
-volOnline(register struct cmd_syndesc *as, void *arock)
+volOnline(struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 server, partition;
+    afs_uint32 server;
+    afs_int32 partition;
     afs_uint32 volid;
     afs_int32 code, err = 0;
 
@@ -1826,9 +1887,10 @@ volOnline(register struct cmd_syndesc *as, void *arock)
  *------------------------------------------------------------------------
  */
 static int
-volOffline(register struct cmd_syndesc *as, void *arock)
+volOffline(struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 server, partition;
+    afs_uint32 server;
+    afs_int32 partition;
     afs_uint32 volid;
     afs_int32 code, err = 0;
     afs_int32 transflag, sleeptime, transdone;
@@ -1859,7 +1921,7 @@ volOffline(register struct cmd_syndesc *as, void *arock)
 
     transflag = (as->parms[4].items ? ITBusy : ITOffline);
     sleeptime = (as->parms[3].items ? atol(as->parms[3].items->data) : 0);
-    transdone = (sleeptime ? 0 /*online */ : VTOutOfService);
+    transdone = ((sleeptime || as->parms[4].items) ? 0 /*online */ : VTOutOfService);
     if (as->parms[4].items && !as->parms[3].items) {
        fprintf(STDERR, "-sleep option must be used with -busy flag\n");
        return -1;
@@ -1877,7 +1939,7 @@ volOffline(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-CreateVolume(register struct cmd_syndesc *as, void *arock)
+CreateVolume(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 pnum;
     char part[10];
@@ -1887,6 +1949,7 @@ CreateVolume(register struct cmd_syndesc *as, void *arock)
     struct nvldbentry entry;
     afs_int32 vcode;
     afs_int32 quota;
+    afs_uint32 tserver;
 
     arovolid = &rovolid;
 
@@ -1991,41 +2054,21 @@ CreateVolume(register struct cmd_syndesc *as, void *arock)
     return 0;
 }
 
-#if 0
-static afs_int32
-DeleteAll(struct nvldbentry *entry)
-{
-    int i;
-    afs_int32 error, code, curserver, curpart;
-    afs_uint32 volid;
-
-    MapHostToNetwork(entry);
-    error = 0;
-    for (i = 0; i < entry->nServers; i++) {
-       curserver = entry->serverNumber[i];
-       curpart = entry->serverPartition[i];
-       if (entry->serverFlags[i] & ITSROVOL) {
-           volid = entry->volumeId[ROVOL];
-       } else {
-           volid = entry->volumeId[RWVOL];
-       }
-       code = UV_DeleteVolume(curserver, curpart, volid);
-       if (code && !error)
-           error = code;
-    }
-    return error;
-}
-#endif
-
 static int
 DeleteVolume(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 err, code = 0;
-    afs_int32 server = 0, partition = -1;
+    afs_uint32 server = 0;
+    afs_int32 partition = -1;
     afs_uint32 volid;
     char pname[10];
     afs_int32 idx, j;
 
+    if (as->parms[1].items && !as->parms[0].items) {
+       fprintf(STDERR, "vos: The -partition option requires the -server option.\n");
+       return EINVAL;
+    }
+
     if (as->parms[0].items) {
        server = GetServer(as->parms[0].items->data);
        if (!server) {
@@ -2080,9 +2123,9 @@ DeleteVolume(struct cmd_syndesc *as, void *arock)
            return (code);
        }
 
-       if (((volid == entry.volumeId[RWVOL]) && (entry.flags & RW_EXISTS))
+       if (((volid == entry.volumeId[RWVOL]) && (entry.flags & VLF_RWEXISTS))
            || ((volid == entry.volumeId[BACKVOL])
-               && (entry.flags & BACK_EXISTS))) {
+               && (entry.flags & VLF_BACKEXISTS))) {
            idx = Lp_GetRwIndex(&entry);
            if ((idx == -1) || (server && (server != entry.serverNumber[idx]))
                || ((partition != -1)
@@ -2092,9 +2135,9 @@ DeleteVolume(struct cmd_syndesc *as, void *arock)
                return ENOENT;
            }
        } else if ((volid == entry.volumeId[ROVOL])
-                  && (entry.flags & RO_EXISTS)) {
+                  && (entry.flags & VLF_ROEXISTS)) {
            for (idx = -1, j = 0; j < entry.nServers; j++) {
-               if (entry.serverFlags[j] != ITSROVOL)
+               if (!(entry.serverFlags[j] & VLSF_ROVOL))
                    continue;
 
                if (((server == 0) || (server == entry.serverNumber[j]))
@@ -2124,7 +2167,6 @@ DeleteVolume(struct cmd_syndesc *as, void *arock)
        partition = entry.serverPartition[idx];
     }
 
-
     code = UV_DeleteVolume(server, partition, volid);
     if (code) {
        PrintDiagnostics("remove", code);
@@ -2139,11 +2181,12 @@ DeleteVolume(struct cmd_syndesc *as, void *arock)
 
 #define TESTM  0               /* set for move space tests, clear for production */
 static int
-MoveVolume(register struct cmd_syndesc *as, void *arock)
+MoveVolume(struct cmd_syndesc *as, void *arock)
 {
 
     afs_uint32 volid;
-    afs_int32 fromserver, toserver, frompart, topart;
+    afs_uint32 fromserver, toserver;
+    afs_int32 frompart, topart;
     afs_int32 flags, code, err;
     char fromPartName[10], toPartName[10];
 
@@ -2267,10 +2310,11 @@ MoveVolume(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-CopyVolume(register struct cmd_syndesc *as, void *arock)
+CopyVolume(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 volid;
-    afs_int32 fromserver, toserver, frompart, topart, code, err, flags;
+    afs_uint32 fromserver, toserver;
+    afs_int32 frompart, topart, code, err, flags;
     char fromPartName[10], toPartName[10], *tovolume;
     struct nvldbentry entry;
     struct diskPartition64 partition;  /* for space check */
@@ -2412,12 +2456,12 @@ CopyVolume(register struct cmd_syndesc *as, void *arock)
     return 0;
 }
 
-
 static int
-ShadowVolume(register struct cmd_syndesc *as, void *arock)
+ShadowVolume(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 volid, tovolid;
-    afs_int32 fromserver, toserver, frompart, topart;
+    afs_uint32 fromserver, toserver;
+    afs_int32 frompart, topart;
     afs_int32 code, err, flags;
     char fromPartName[10], toPartName[10], toVolName[32], *tovolume;
     struct diskPartition64 partition;  /* for space check */
@@ -2608,12 +2652,12 @@ ShadowVolume(register struct cmd_syndesc *as, void *arock)
     return 0;
 }
 
-
 static int
-CloneVolume(register struct cmd_syndesc *as, void *arock)
+CloneVolume(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 volid, cloneid;
-    afs_int32 server, part, voltype;
+    afs_uint32 server;
+    afs_int32 part, voltype;
     char partName[10], *volname;
     afs_int32 code, err, flags;
     struct nvldbentry entry;
@@ -2654,7 +2698,7 @@ CloneVolume(register struct cmd_syndesc *as, void *arock)
                    "vos : partition %s does not exist on the server\n",
                    as->parms[2].items->data);
            return ENOENT;
-        }
+       }
     } else {
        code = GetVolumeInfo(volid, &server, &part, &voltype, &entry);
        if (code)
@@ -2670,18 +2714,6 @@ CloneVolume(register struct cmd_syndesc *as, void *arock)
                volname, VOLSER_OLDMAXVOLNAME - 1);
            return E2BIG;
        }
-#if 0
-       /* 
-        * In order that you be able to make clones of RO or BK, this
-        * check must be omitted.
-        */
-       if (!VolNameOK(volname)) {
-           fprintf(STDERR,
-               "Illegal volume name %s, should not end in .readonly or .backup\n",
-               volname);
-           return EINVAL;
-       }
-#endif
        if (IsNumeric(volname)) {
            fprintf(STDERR,
                "Illegal volume name %s, should not be a number\n",
@@ -2705,10 +2737,14 @@ CloneVolume(register struct cmd_syndesc *as, void *arock)
 
     flags = 0;
     if (as->parms[5].items) flags |= RV_OFFLINE;
+    if (as->parms[6].items && as->parms[7].items) {
+       fprintf(STDERR, "vos: cannot specify that a volume be -readwrite and -readonly\n");
+       return EINVAL;
+    }
     if (as->parms[6].items) flags |= RV_RDONLY;
+    if (as->parms[7].items) flags |= RV_RWONLY;
 
-
-    code = 
+    code =
        UV_CloneVolume(server, part, volid, cloneid, volname, flags);
 
     if (code) {
@@ -2722,16 +2758,17 @@ CloneVolume(register struct cmd_syndesc *as, void *arock)
     return 0;
 }
 
-
 static int
-BackupVolume(register struct cmd_syndesc *as, void *arock)
+BackupVolume(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
-    afs_int32 aserver, apart, vtype, code, err;
+    afs_uint32 aserver;
+    afs_int32 apart, vtype, code, err;
     struct nvldbentry entry;
 
     afs_uint32 buvolid;
-    afs_int32 buserver, bupart, butype;
+    afs_uint32 buserver;
+    afs_int32 bupart, butype;
     struct nvldbentry buentry;
 
     avolid = vsu_GetVolumeID(as->parms[0].items->data, cstruct, &err);
@@ -2756,7 +2793,7 @@ BackupVolume(register struct cmd_syndesc *as, void *arock)
 
     /* is there a backup volume already? */
 
-    if (entry.flags & BACK_EXISTS) {
+    if (entry.flags & VLF_BACKEXISTS) {
        /* yep, where is it? */
 
        buvolid = entry.volumeId[BACKVOL];
@@ -2794,16 +2831,24 @@ BackupVolume(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-ReleaseVolume(register struct cmd_syndesc *as, void *arock)
+ReleaseVolume(struct cmd_syndesc *as, void *arock)
 {
 
     struct nvldbentry entry;
     afs_uint32 avolid;
-    afs_int32 aserver, apart, vtype, code, err;
-    int force = 0;
+    afs_uint32 aserver;
+    afs_int32 apart, vtype, code, err;
+    int flags = 0;
+
+    if (as->parms[1].items) /* -force */
+       flags |= (REL_COMPLETE | REL_FULLDUMPS);
+    if (as->parms[2].items) { /* -stayonline */
+       fprintf(STDERR, "vos: -stayonline not supported\n");
+       return EINVAL;
+    }
+    if (as->parms[3].items) /* -force-reclone */
+       flags |= REL_COMPLETE;
 
-    if (as->parms[1].items)
-       force = 1;
     avolid = vsu_GetVolumeID(as->parms[0].items->data, cstruct, &err);
     if (avolid == 0) {
        if (err)
@@ -2829,7 +2874,8 @@ ReleaseVolume(register struct cmd_syndesc *as, void *arock)
        return E2BIG;
     }
 
-    code = UV_ReleaseVolume(avolid, aserver, apart, force);
+    code = UV_ReleaseVolume(avolid, aserver, apart, flags);
+
     if (code) {
        PrintDiagnostics("release", code);
        return code;
@@ -2840,10 +2886,11 @@ ReleaseVolume(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-DumpVolumeCmd(register struct cmd_syndesc *as, void *arock)
+DumpVolumeCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
-    afs_int32 aserver, apart, voltype, fromdate = 0, code, err, i, flags;
+    afs_uint32 aserver;
+    afs_int32 apart, voltype, fromdate = 0, code, err, i, flags;
     char filename[MAXPATHLEN];
     struct nvldbentry entry;
 
@@ -2853,8 +2900,8 @@ DumpVolumeCmd(register struct cmd_syndesc *as, void *arock)
        if (rxConn == 0)
            break;
        rx_SetConnDeadTime(rxConn, rx_connDeadTime);
-       if (rxConn->service)
-           rxConn->service->connDeadTime = rx_connDeadTime;
+       if (rx_ServiceOf(rxConn))
+           rx_ServiceOf(rxConn)->connDeadTime = rx_connDeadTime;
     }
 
     avolid = vsu_GetVolumeID(as->parms[0].items->data, cstruct, &err);
@@ -2941,21 +2988,19 @@ retry_dump:
 #define TS_NEW 3
 
 static int
-RestoreVolumeCmd(register struct cmd_syndesc *as, void *arock)
+RestoreVolumeCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid, aparentid;
-    afs_int32 aserver, apart, code, vcode, err;
+    afs_uint32 aserver;
+    afs_int32 apart, code, vcode, err;
     afs_int32 aoverwrite = ASK;
     afs_int32 acreation = 0, alastupdate = 0;
     int restoreflags = 0;
     int readonly = 0, offline = 0, voltype = RWVOL;
-    char prompt;
     char afilename[MAXPATHLEN], avolname[VOLSER_MAXVOLNAME + 1], apartName[10];
     char volname[VOLSER_MAXVOLNAME + 1];
     struct nvldbentry entry;
 
-    prompt = 'n';
-
     aparentid = 0;
     if (as->parms[4].items) {
        avolid = vsu_GetVolumeID(as->parms[4].items->data, cstruct, &err);
@@ -3098,11 +3143,12 @@ RestoreVolumeCmd(register struct cmd_syndesc *as, void *arock)
                   && entry.volumeId[voltype] != avolid) {
            avolid = entry.volumeId[voltype];
        }
-        aparentid = entry.volumeId[RWVOL];
+       aparentid = entry.volumeId[RWVOL];
     }
 
     else {                     /* volume exists - do we do a full incremental or abort */
-       int Oserver, Opart, Otype, vol_elsewhere = 0;
+       afs_uint32 Oserver;
+       afs_int32 Opart, Otype, vol_elsewhere = 0;
        struct nvldbentry Oentry;
        int c, dc;
 
@@ -3112,7 +3158,7 @@ RestoreVolumeCmd(register struct cmd_syndesc *as, void *arock)
                   && entry.volumeId[voltype] != avolid) {
            avolid = entry.volumeId[voltype];
        }
-        aparentid = entry.volumeId[RWVOL];
+       aparentid = entry.volumeId[RWVOL];
 
        /* A file name was specified  - check if volume is on another partition */
        vcode = GetVolumeInfo(avolid, &Oserver, &Opart, &Otype, &Oentry);
@@ -3216,28 +3262,23 @@ RestoreVolumeCmd(register struct cmd_syndesc *as, void *arock)
     if (as->parms[10].items) {
        restoreflags |= RV_NODEL;
     }
-    
 
     code =
        UV_RestoreVolume2(aserver, apart, avolid, aparentid,
-                          avolname, restoreflags, WriteData, afilename);
+                         avolname, restoreflags, WriteData, afilename);
     if (code) {
        PrintDiagnostics("restore", code);
        exit(1);
     }
     MapPartIdIntoName(apart, apartName);
 
-    /*
-     * patch typo here - originally "parms[1]", should be "parms[0]"
-     */
-
     fprintf(STDOUT, "Restored volume %s on %s %s\n", avolname,
            as->parms[0].items->data, apartName);
     return 0;
 }
 
 static int
-LockReleaseCmd(register struct cmd_syndesc *as, void *arock)
+LockReleaseCmd(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
     afs_int32 code, err;
@@ -3263,10 +3304,11 @@ LockReleaseCmd(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-AddSite(register struct cmd_syndesc *as, void *arock)
+AddSite(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
-    afs_int32 aserver, apart, code, err, arovolid, valid = 0;
+    afs_uint32 aserver;
+    afs_int32 apart, code, err, arovolid, valid = 0;
     char apartName[10], avolname[VOLSER_MAXVOLNAME + 1];
 
     vsu_ExtractName(avolname, as->parms[2].items->data);;
@@ -3325,11 +3367,12 @@ AddSite(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-RemoveSite(register struct cmd_syndesc *as, void *arock)
+RemoveSite(struct cmd_syndesc *as, void *arock)
 {
 
     afs_uint32 avolid;
-    afs_int32 aserver, apart, code, err;
+    afs_uint32 aserver;
+    afs_int32 apart, code, err;
     char apartName[10], avolname[VOLSER_MAXVOLNAME + 1];
 
     vsu_ExtractName(avolname, as->parms[2].items->data);
@@ -3354,17 +3397,11 @@ RemoveSite(register struct cmd_syndesc *as, void *arock)
                as->parms[1].items->data);
        exit(1);
     }
-/*
- *skip the partition validity check, since it is possible that the partition
- *has since been decomissioned.
- */
-/*
-       if (!IsPartValid(apart,aserver,&code)){
-           if(code) PrintError("",code);
-           else fprintf(STDERR,"vos : partition %s does not exist on the server\n",as->parms[1].items->data);
-           exit(1);
-       }
-*/
+    /*
+     * The partition validity check is skipped, since it is possible that the
+     * partition has already been decomissioned.
+     */
+
     code = UV_RemoveSite(aserver, apart, avolid);
     if (code) {
        PrintDiagnostics("remsite", code);
@@ -3377,10 +3414,11 @@ RemoveSite(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-ChangeLocation(register struct cmd_syndesc *as, void *arock)
+ChangeLocation(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
-    afs_int32 aserver, apart, code, err;
+    afs_uint32 aserver;
+    afs_int32 apart, code, err;
     char apartName[10];
 
     avolid = vsu_GetVolumeID(as->parms[2].items->data, cstruct, &err);
@@ -3415,7 +3453,7 @@ ChangeLocation(register struct cmd_syndesc *as, void *arock)
     }
     code = UV_ChangeLocation(aserver, apart, avolid);
     if (code) {
-       PrintDiagnostics("addsite", code);
+       PrintDiagnostics("changeloc", code);
        exit(1);
     }
     MapPartIdIntoName(apart, apartName);
@@ -3425,9 +3463,10 @@ ChangeLocation(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-ListPartitions(register struct cmd_syndesc *as, void *arock)
+ListPartitions(struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 aserver, code;
+    afs_uint32 aserver;
+    afs_int32 code;
     struct partList dummyPartList;
     int i;
     char pname[10];
@@ -3440,7 +3479,6 @@ ListPartitions(register struct cmd_syndesc *as, void *arock)
        exit(1);
     }
 
-
     code = UV_ListPartitions(aserver, &dummyPartList, &cnt);
     if (code) {
        PrintDiagnostics("listpart", code);
@@ -3583,10 +3621,11 @@ XCompareVolID(const void *a_obj1P, const void *a_obj2P)
  *------------------------------------------------------------------------*/
 
 static int
-ListVolumes(register struct cmd_syndesc *as, void *arock)
+ListVolumes(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 apart, int32list, fast;
-    afs_int32 aserver, code;
+    afs_uint32 aserver;
+    afs_int32 code;
     volintInfo *pntr;
     volintInfo *oldpntr = NULL;
     afs_int32 count;
@@ -3681,8 +3720,6 @@ ListVolumes(register struct cmd_syndesc *as, void *arock)
                                   &pntr, &count);
            if (code) {
                PrintDiagnostics("listvol", code);
-               if (pntr)
-                   free(pntr);
                exit(1);
            }
            if (wantExtendedInfo) {
@@ -3711,24 +3748,20 @@ ListVolumes(register struct cmd_syndesc *as, void *arock)
                        as->parms[0].items->data, pname,
                        (unsigned long)count);
            if (wantExtendedInfo) {
-#ifdef FULL_LISTVOL_SWITCH
                if (as->parms[6].items)
                    XDisplayVolumes2(aserver, dummyPartList.partId[i], origxInfoP,
                                count, int32list, fast, quiet);
                else
-#endif /* FULL_LISTVOL_SWITCH */
-               XDisplayVolumes(aserver, dummyPartList.partId[i], origxInfoP,
+                   XDisplayVolumes(aserver, dummyPartList.partId[i], origxInfoP,
                                count, int32list, fast, quiet);
                if (xInfoP)
                    free(xInfoP);
                xInfoP = (volintXInfo *) 0;
            } else {
-#ifdef FULL_LISTVOL_SWITCH
                if (as->parms[6].items)
                    DisplayVolumes2(aserver, dummyPartList.partId[i], oldpntr,
                                    count);
                else
-#endif /* FULL_LISTVOL_SWITCH */
                    DisplayVolumes(aserver, dummyPartList.partId[i], oldpntr,
                                   count, int32list, fast, quiet);
                if (pntr)
@@ -3741,14 +3774,21 @@ ListVolumes(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-SyncVldb(register struct cmd_syndesc *as, void *arock)
+SyncVldb(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 pnum = 0, code;  /* part name */
     char part[10];
     int flags = 0;
     char *volname = 0;
+    afs_uint32 tserver;
 
     tserver = 0;
+
+    if (as->parms[1].items && !as->parms[0].items) {
+       fprintf(STDERR, "vos: The -partition option requires the -server option.\n");
+       exit(1);
+    }
+
     if (as->parms[0].items) {
        tserver = GetServer(as->parms[0].items->data);
        if (!tserver) {
@@ -3775,12 +3815,6 @@ SyncVldb(register struct cmd_syndesc *as, void *arock)
            exit(1);
        }
        flags = 1;
-
-       if (!tserver) {
-           fprintf(STDERR,
-                   "The -partition option requires a -server option\n");
-           exit(1);
-       }
     }
 
     if (as->parms[3].items) {
@@ -3823,10 +3857,11 @@ SyncVldb(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-SyncServer(register struct cmd_syndesc *as, void *arock)
+SyncServer(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 pnum, code;      /* part name */
     char part[10];
+    afs_uint32 tserver;
 
     int flags = 0;
 
@@ -3854,7 +3889,7 @@ SyncServer(register struct cmd_syndesc *as, void *arock)
        }
        flags = 1;
     } else {
-        pnum = -1;
+       pnum = -1;
     }
 
     if (as->parms[2].items) {
@@ -3897,14 +3932,13 @@ VolumeInfoCmd(char *name)
      * If VLOP_ALLOPERS is set, the entry is locked.
      * Leave this routine as is, but put in correct check.
      */
-    if (entry.flags & VLOP_ALLOPERS)
-       fprintf(STDOUT, "    Volume is currently LOCKED  \n");
+    PrintLocked(entry.flags);
 
     return 0;
 }
 
 static int
-VolumeZap(register struct cmd_syndesc *as, void *arock)
+VolumeZap(struct cmd_syndesc *as, void *arock)
 {
     struct nvldbentry entry;
     afs_uint32 volid, zapbackupid = 0, backupid = 0;
@@ -3990,9 +4024,10 @@ VolumeZap(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-VolserStatus(register struct cmd_syndesc *as, void *arock)
+VolserStatus(struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 server, code;
+    afs_uint32 server;
+    afs_int32 code;
     transDebugInfo *pntr, *oldpntr;
     afs_int32 count;
     int i;
@@ -4020,42 +4055,43 @@ 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;
+       t = pntr->creationTime;
        fprintf(STDOUT, "transaction: %lu  created: %s",
                (unsigned long)pntr->tid, ctime(&t));
+       t = pntr->time;
+       fprintf(STDOUT, "lastActiveTime: %s", ctime(&t));
        if (pntr->returnCode) {
            fprintf(STDOUT, "returnCode: %lu\n",
                    (unsigned long)pntr->returnCode);
        }
        if (pntr->iflags) {
            fprintf(STDOUT, "attachFlags:  ");
-           switch (pntr->iflags) {
-           case ITOffline:
+           if ((pntr->iflags & ITOffline) != 0) {
                fprintf(STDOUT, "offline ");
-               break;
-           case ITBusy:
+           }
+           if ((pntr->iflags & ITBusy) != 0) {
                fprintf(STDOUT, "busy ");
-               break;
-           case ITReadOnly:
+           }
+           if ((pntr->iflags & ITReadOnly) != 0) {
                fprintf(STDOUT, "readonly ");
-               break;
-           case ITCreate:
+           }
+           if ((pntr->iflags & ITCreate) != 0) {
                fprintf(STDOUT, "create ");
-               break;
-           case ITCreateVolID:
+           }
+           if ((pntr->iflags & ITCreateVolID) != 0) {
                fprintf(STDOUT, "create volid ");
-               break;
            }
            fprintf(STDOUT, "\n");
        }
        if (pntr->vflags) {
            fprintf(STDOUT, "volumeStatus: ");
-           switch (pntr->vflags) {
-           case VTDeleteOnSalvage:
+           if ((pntr->vflags & VTDeleteOnSalvage) != 0) {
                fprintf(STDOUT, "deleteOnSalvage ");
-           case VTOutOfService:
+           }
+           if ((pntr->vflags & VTOutOfService) != 0) {
                fprintf(STDOUT, "outOfService ");
-           case VTDeleted:
+           }
+           if ((pntr->vflags & VTDeleted) != 0) {
                fprintf(STDOUT, "deleted ");
            }
            fprintf(STDOUT, "\n");
@@ -4068,10 +4104,12 @@ VolserStatus(register struct cmd_syndesc *as, void *arock)
        fprintf(STDOUT, "volume: %lu  partition: %s  procedure: %s\n",
                (unsigned long)pntr->volid, pname, pntr->lastProcName);
        if (pntr->callValid) {
-           fprintf(STDOUT,
-                   "packetRead: %lu  lastReceiveTime: %d  packetSend: %lu  lastSendTime: %d\n",
-                   (unsigned long)pntr->readNext, pntr->lastReceiveTime,
-                   (unsigned long)pntr->transmitNext, pntr->lastSendTime);
+           t = pntr->lastReceiveTime;
+           fprintf(STDOUT, "packetRead: %lu  lastReceiveTime: %s",
+                   (unsigned long)pntr->readNext, ctime(&t));
+           t = pntr->lastSendTime;
+           fprintf(STDOUT, "packetSend: %lu  lastSendTime: %s",
+                   (unsigned long)pntr->transmitNext, ctime(&t));
        }
        pntr++;
        fprintf(STDOUT, "--------------------------------------\n");
@@ -4083,27 +4121,41 @@ VolserStatus(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-RenameVolume(register struct cmd_syndesc *as, void *arock)
+RenameVolume(struct cmd_syndesc *as, void *arock)
 {
-    afs_int32 code1, code2, code;
+    afs_int32 code;
     struct nvldbentry entry;
+    struct nvldbentry entry2;
 
-    code1 = VLDB_GetEntryByName(as->parms[0].items->data, &entry);
-    if (code1) {
+    /* Get the entry of the volume to be renamed (-oldname), by name or id. */
+    code = VLDB_GetEntryByName(as->parms[0].items->data, &entry);
+    if (code) {
        fprintf(STDERR, "vos: Could not find entry for volume %s\n",
                as->parms[0].items->data);
+       PrintError("", code);
        exit(1);
     }
-    code2 = VLDB_GetEntryByName(as->parms[1].items->data, &entry);
-    if ((!code1) && (!code2)) {        /*the newname already exists */
-       fprintf(STDERR, "vos: volume %s already exists\n",
+
+    /*
+     * Verify the new name is available before attempting to rename.
+     * Allow renaming of the same volume in order to complete a
+     * previously interrupted rename.
+     */
+    code = VLDB_GetEntryByName(as->parms[1].items->data, &entry2);
+    if (code != 0 && code != VL_NOENT) {
+       fprintf(STDERR, "vos: Could not check entry for volume %s\n",
                as->parms[1].items->data);
+       PrintError("", code);
        exit(1);
     }
-
-    if (code1 && code2) {
-       fprintf(STDERR, "vos: Could not find entry for volume %s or %s\n",
-               as->parms[0].items->data, as->parms[1].items->data);
+    if (code == 0 && entry.volumeId[RWVOL] != entry2.volumeId[RWVOL]) {
+       fprintf(STDERR, "vos: Cannot rename volume %s (%lu) to %s;"
+                       " volume %s (%lu) already exists\n",
+                       as->parms[0].items->data,
+                       (unsigned long)entry.volumeId[RWVOL],
+                       as->parms[1].items->data,
+                       as->parms[1].items->data,
+                       (unsigned long)entry2.volumeId[RWVOL]);
        exit(1);
     }
     if (!VolNameOK(as->parms[0].items->data)) {
@@ -4143,8 +4195,8 @@ RenameVolume(register struct cmd_syndesc *as, void *arock)
 }
 
 int
-GetVolumeInfo(afs_uint32 volid, afs_int32 *server, afs_int32 *part, afs_int32 *voltype, 
-              struct nvldbentry *rentry)
+GetVolumeInfo(afs_uint32 volid, afs_uint32 *server, afs_int32 *part, afs_int32 *voltype,
+             struct nvldbentry *rentry)
 {
     afs_int32 vcode;
     int i, index = -1;
@@ -4161,8 +4213,8 @@ GetVolumeInfo(afs_uint32 volid, afs_int32 *server, afs_int32 *part, afs_int32 *v
     if (volid == rentry->volumeId[ROVOL]) {
        *voltype = ROVOL;
        for (i = 0; i < rentry->nServers; i++) {
-           if ((index == -1) && (rentry->serverFlags[i] & ITSROVOL)
-               && !(rentry->serverFlags[i] & RO_DONTUSE))
+           if ((index == -1) && (rentry->serverFlags[i] & VLSF_ROVOL)
+               && !(rentry->serverFlags[i] & VLSF_DONTUSE))
                index = i;
        }
        if (index == -1) {
@@ -4197,20 +4249,20 @@ GetVolumeInfo(afs_uint32 volid, afs_int32 *server, afs_int32 *part, afs_int32 *v
        return 0;
     }
     fprintf(STDERR,
-            "unexpected volume type for volume %lu\n",
-            (unsigned long)volid);
+           "unexpected volume type for volume %lu\n",
+           (unsigned long)volid);
     return -1;
 }
 
 static int
-DeleteEntry(register struct cmd_syndesc *as, void *arock)
+DeleteEntry(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 apart = 0;
     afs_uint32 avolid;
     afs_int32 vcode;
     struct VldbListByAttributes attributes;
     nbulkentries arrayEntries;
-    register struct nvldbentry *vllist;
+    struct nvldbentry *vllist;
     struct cmd_item *itp;
     afs_int32 nentries;
     int j;
@@ -4234,7 +4286,8 @@ DeleteEntry(register struct cmd_syndesc *as, void *arock)
                            itp->data);
                continue;
            }
-           if (as->parms[4].items) {   /* -noexecute */
+           if (as->parms[4].items || as->parms[5].items) {
+               /* -noexecute (hidden) or -dryrun */
                fprintf(STDOUT, "Would have deleted VLDB entry for %s \n",
                        itp->data);
                fflush(STDOUT);
@@ -4276,7 +4329,7 @@ DeleteEntry(register struct cmd_syndesc *as, void *arock)
     }
 
     if (as->parms[2].items) {  /* -server */
-       afs_int32 aserver;
+       afs_uint32 aserver;
        aserver = GetServer(as->parms[2].items->data);
        if (aserver == 0) {
            fprintf(STDERR, "vos: server '%s' not found in host table\n",
@@ -4344,7 +4397,8 @@ DeleteEntry(register struct cmd_syndesc *as, void *arock)
            }
        }
 
-       if (as->parms[4].items) {       /* -noexecute */
+       if (as->parms[4].items || as->parms[5].items) {
+           /* -noexecute (hidden) or -dryrun */
            fprintf(STDOUT, "Would have deleted VLDB entry for %s \n",
                    vllist->name);
            fflush(STDOUT);
@@ -4355,7 +4409,7 @@ DeleteEntry(register struct cmd_syndesc *as, void *arock)
        avolid = vllist->volumeId[RWVOL];
        vcode = ubik_VL_DeleteEntry(cstruct, 0, avolid, RWVOL);
        if (vcode) {
-           fprintf(STDOUT, "Could not delete VDLB entry for  %s\n",
+           fprintf(STDOUT, "Could not delete VLDB entry for  %s\n",
                    vllist->name);
            totalFail++;
            PrintError("", vcode);
@@ -4372,12 +4426,11 @@ DeleteEntry(register struct cmd_syndesc *as, void *arock)
     fprintf(STDOUT,
            "Total VLDB entries deleted: %lu; failed to delete: %lu\n",
            (unsigned long)totalBack, (unsigned long)totalFail);
-    if (arrayEntries.nbulkentries_val)
-       free(arrayEntries.nbulkentries_val);
+
+    xdr_free((xdrproc_t) xdr_nbulkentries, &arrayEntries);
     return 0;
 }
 
-
 static int
 CompareVldbEntryByName(const void *p1, const void *p2)
 {
@@ -4388,46 +4441,11 @@ CompareVldbEntryByName(const void *p1, const void *p2)
     return (strcmp(arg1->name, arg2->name));
 }
 
-/*
-static int CompareVldbEntry(char *p1, char *p2)
-{
-    struct nvldbentry *arg1,*arg2;
-    int i;
-    int pos1, pos2;
-    char comp1[100],comp2[100];
-    char temp1[20],temp2[20];
-
-    arg1 = (struct nvldbentry *)p1;
-    arg2 = (struct nvldbentry *)p2;
-    pos1 = -1;
-    pos2 = -1;
-
-    for(i = 0; i < arg1->nServers; i++)
-       if(arg1->serverFlags[i] & ITSRWVOL) pos1 = i;
-    for(i = 0; i < arg2->nServers; i++)
-       if(arg2->serverFlags[i] & ITSRWVOL) pos2 = i;
-    if(pos1 == -1 || pos2 == -1){
-       pos1 = 0;
-       pos2 = 0;
-    }
-    sprintf(comp1,"%10u",arg1->serverNumber[pos1]);
-    sprintf(comp2,"%10u",arg2->serverNumber[pos2]);
-    sprintf(temp1,"%10u",arg1->serverPartition[pos1]);
-    sprintf(temp2,"%10u",arg2->serverPartition[pos2]);
-    strcat(comp1,temp1);
-    strcat(comp2,temp2);
-    strcat(comp1,arg1->name);
-    strcat(comp1,arg2->name);
-    return(strcmp(comp1,comp2));
-
-}
-
-*/
 static int
 ListVLDB(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 apart;
-    afs_int32 aserver, code;
+    afs_int32 code;
     afs_int32 vcode;
     struct VldbListByAttributes attributes;
     nbulkentries arrayEntries;
@@ -4440,10 +4458,9 @@ ListVLDB(struct cmd_syndesc *as, void *arock)
     int quiet, sort, lock;
     afs_int32 thisindex, nextindex;
 
-    aserver = 0;
     apart = 0;
 
-    attributes.Mask = 0;
+    memset(&attributes, 0, sizeof(attributes));
     lock = (as->parms[3].items ? 1 : 0);       /* -lock   flag */
     quiet = (as->parms[4].items ? 1 : 0);      /* -quit   flag */
     sort = (as->parms[5].items ? 0 : 1);       /* -nosort flag */
@@ -4467,6 +4484,8 @@ ListVLDB(struct cmd_syndesc *as, void *arock)
 
     /* Server specified */
     if (as->parms[1].items) {
+       afs_uint32 aserver;
+
        aserver = GetServer(as->parms[1].items->data);
        if (aserver == 0) {
            fprintf(STDERR, "vos: server '%s' not found in host table\n",
@@ -4533,8 +4552,7 @@ ListVLDB(struct cmd_syndesc *as, void *arock)
                MapHostToNetwork(vllist);
                EnumerateEntry(vllist);
 
-               if (vllist->flags & VLOP_ALLOPERS)
-                   fprintf(STDOUT, "    Volume is currently LOCKED  \n");
+               PrintLocked(vllist->flags);
            }
        }
 
@@ -4543,16 +4561,19 @@ ListVLDB(struct cmd_syndesc *as, void *arock)
         */
        else if (centries > 0) {
            if (!tarray) {
-               /* steal away the first bulk entries array */
-               tarray = (struct nvldbentry *)arrayEntries.nbulkentries_val;
+               /* malloc the first bulk entries array */
                tarraysize = centries * sizeof(struct nvldbentry);
-               arrayEntries.nbulkentries_val = 0;
+               tarray = malloc(tarraysize);
+               if (!tarray) {
+                   fprintf(STDERR,
+                           "Could not allocate enough space for the VLDB entries\n");
+                   goto bypass;
+               }
+               memcpy((char*)tarray, arrayEntries.nbulkentries_val, tarraysize);
            } else {
                /* Grow the tarray to keep the extra entries */
                parraysize = (centries * sizeof(struct nvldbentry));
-               ttarray =
-                   (struct nvldbentry *)realloc(tarray,
-                                                tarraysize + parraysize);
+               ttarray = realloc(tarray, tarraysize + parraysize);
                if (!ttarray) {
                    fprintf(STDERR,
                            "Could not allocate enough space for  the VLDB entries\n");
@@ -4568,10 +4589,7 @@ ListVLDB(struct cmd_syndesc *as, void *arock)
        }
 
        /* Free the bulk array */
-       if (arrayEntries.nbulkentries_val) {
-           free(arrayEntries.nbulkentries_val);
-           arrayEntries.nbulkentries_val = 0;
-       }
+       xdr_free((xdrproc_t) xdr_nbulkentries, &arrayEntries);
     }
 
     /* Here is where we now sort all the entries and print them */
@@ -4582,8 +4600,7 @@ ListVLDB(struct cmd_syndesc *as, void *arock)
            MapHostToNetwork(vllist);
            EnumerateEntry(vllist);
 
-           if (vllist->flags & VLOP_ALLOPERS)
-               fprintf(STDOUT, "    Volume is currently LOCKED  \n");
+           PrintLocked(vllist->flags);
        }
     }
 
@@ -4596,15 +4613,16 @@ ListVLDB(struct cmd_syndesc *as, void *arock)
 }
 
 static int
-BackSys(register struct cmd_syndesc *as, void *arock)
+BackSys(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
     afs_int32 apart = 0;
-    afs_int32 aserver = 0, code, aserver1, apart1;
+    afs_uint32 aserver = 0, aserver1;
+    afs_int32 code, apart1;
     afs_int32 vcode;
     struct VldbListByAttributes attributes;
     nbulkentries arrayEntries;
-    register struct nvldbentry *vllist;
+    struct nvldbentry *vllist;
     afs_int32 nentries;
     int j;
     char pname[10];
@@ -4619,6 +4637,7 @@ BackSys(register struct cmd_syndesc *as, void *arock)
 #ifndef HAVE_POSIX_REGEX
     char *ccode;
 #endif
+    int found;
 
     memset(&attributes, 0, sizeof(struct VldbListByAttributes));
     attributes.Mask = 0;
@@ -4863,7 +4882,7 @@ BackSys(register struct cmd_syndesc *as, void *arock)
            continue;
        }
 
-       if (!(vllist->flags & RW_EXISTS)) {
+       if (!(vllist->flags & VLF_RWEXISTS)) {
            if (verbose) {
                fprintf(STDOUT,
                        "Omitting to backup %s since RW volume does not exist \n",
@@ -4876,8 +4895,8 @@ BackSys(register struct cmd_syndesc *as, void *arock)
 
        avolid = vllist->volumeId[RWVOL];
        MapHostToNetwork(vllist);
-       GetServerAndPart(vllist, RWVOL, &aserver1, &apart1, &previdx);
-       if (aserver1 == -1 || apart1 == -1) {
+       found = GetServerAndPart(vllist, RWVOL, &aserver1, &apart1, &previdx);
+       if (!found) {
            fprintf(STDOUT, "could not backup %s, invalid VLDB entry\n",
                    vllist->name);
            totalFail++;
@@ -4923,21 +4942,20 @@ BackSys(register struct cmd_syndesc *as, void *arock)
     fprintf(STDOUT, "Total volumes backed up: %lu; failed to backup: %lu\n",
            (unsigned long)totalBack, (unsigned long)totalFail);
     fflush(STDOUT);
-    if (arrayEntries.nbulkentries_val)
-       free(arrayEntries.nbulkentries_val);
+    xdr_free((xdrproc_t) xdr_nbulkentries, &arrayEntries);
     return 0;
 }
 
 static int
-UnlockVLDB(register struct cmd_syndesc *as, void *arock)
+UnlockVLDB(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 apart;
-    afs_int32 aserver = 0;
+    afs_uint32 aserver = 0;
     afs_int32 code;
     afs_int32 vcode;
     struct VldbListByAttributes attributes;
     nbulkentries arrayEntries;
-    register struct nvldbentry *vllist;
+    struct nvldbentry *vllist;
     afs_int32 nentries;
     int j;
     afs_uint32 volid;
@@ -4946,7 +4964,7 @@ UnlockVLDB(register struct cmd_syndesc *as, void *arock)
 
     apart = -1;
     totalE = 0;
-    attributes.Mask = 0;
+    memset(&attributes, 0, sizeof(attributes));
 
     if (as->parms[0].items) {  /* server specified */
        aserver = GetServer(as->parms[0].items->data);
@@ -4965,14 +4983,17 @@ UnlockVLDB(register struct cmd_syndesc *as, void *arock)
                    as->parms[1].items->data);
            exit(1);
        }
-       if (!IsPartValid(apart, aserver, &code)) {      /*check for validity of the partition */
-           if (code)
-               PrintError("", code);
-           else
-               fprintf(STDERR,
-                       "vos : partition %s does not exist on the server\n",
-                       as->parms[1].items->data);
-           exit(1);
+       if (aserver) {
+           /* Check for validity of the partition if a server was given. */
+           if (!IsPartValid(apart, aserver, &code)) {
+               if (code)
+                   PrintError("", code);
+               else
+                   fprintf(STDERR,
+                           "vos : partition %s does not exist on the server\n",
+                           as->parms[1].items->data);
+               exit(1);
+           }
        }
        attributes.partition = apart;
        attributes.Mask |= VLLIST_PARTITION;
@@ -4991,7 +5012,7 @@ UnlockVLDB(register struct cmd_syndesc *as, void *arock)
        volid = vllist->volumeId[RWVOL];
        vcode =
            ubik_VL_ReleaseLock(cstruct, 0, volid, -1,
-                               LOCKREL_OPCODE | LOCKREL_AFSID | 
+                               LOCKREL_OPCODE | LOCKREL_AFSID |
                                LOCKREL_TIMESTAMP);
        if (vcode) {
            fprintf(STDERR, "Could not unlock entry for volume %s\n",
@@ -5025,44 +5046,44 @@ UnlockVLDB(register struct cmd_syndesc *as, void *arock)
        }
     }
 
-    if (arrayEntries.nbulkentries_val)
-       free(arrayEntries.nbulkentries_val);
+    xdr_free((xdrproc_t) xdr_nbulkentries, &arrayEntries);
     return 0;
 }
 
 static char *
 PrintInt64Size(afs_uint64 in)
 {
-    register afs_uint32 hi, lo;
-    register char * units;
+    afs_uint32 hi, lo;
+    char * units;
     static char output[16];
 
     SplitInt64(in,hi,lo);
 
     if (hi == 0) {
-        units = "KB";
+       units = "KB";
     } else if (!(hi & 0xFFFFFC00)) {
-        units = "MB";
-        lo = (hi << 22) | (lo >> 10);
+       units = "MB";
+       lo = (hi << 22) | (lo >> 10);
     } else if (!(hi & 0xFFF00000)) {
-        units = "GB";
-        lo = (hi << 12) | (lo >> 20);
+       units = "GB";
+       lo = (hi << 12) | (lo >> 20);
     } else if (!(hi & 0xC0000000)) {
-        units = "TB";
-        lo = (hi << 2) | (lo >> 30);
+       units = "TB";
+       lo = (hi << 2) | (lo >> 30);
     } else {
-        units = "PB";
-        lo = (hi >> 8);
+       units = "PB";
+       lo = (hi >> 8);
     }
     sprintf(output,"%u %s", lo, units);
     return output;
 }
 
 static int
-PartitionInfo(register struct cmd_syndesc *as, void *arock)
+PartitionInfo(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 apart;
-    afs_int32 aserver, code;
+    afs_uint32 aserver;
+    afs_int32 code;
     char pname[10];
     struct diskPartition64 partition;
     struct partList dummyPartList;
@@ -5091,7 +5112,7 @@ PartitionInfo(register struct cmd_syndesc *as, void *arock)
        cnt = 1;
     }
     if (as->parms[2].items) {
-        printSummary = 1;
+       printSummary = 1;
     }
     if (apart != -1) {
        if (!IsPartValid(apart, aserver, &code)) {      /*check for validity of the partition */
@@ -5124,29 +5145,33 @@ PartitionInfo(register struct cmd_syndesc *as, void *arock)
                    "Free space on partition %s: %" AFS_INT64_FMT " K blocks out of total %" AFS_INT64_FMT "\n",
                    pname, partition.free, partition.minFree);
            sumPartitions++;
-            AddUInt64(sumFree,partition.free,&sumFree);
-            AddUInt64(sumStorage,partition.minFree,&sumStorage);
+           AddUInt64(sumFree,partition.free,&sumFree);
+           AddUInt64(sumStorage,partition.minFree,&sumStorage);
        }
     }
     if (printSummary) {
-        fprintf(STDOUT,
+       fprintf(STDOUT,
                "Summary: %s free out of ",
                PrintInt64Size(sumFree));
-        fprintf(STDOUT,
-                "%s on %d partitions\n",
-                PrintInt64Size(sumStorage), 
-                sumPartitions);
+       fprintf(STDOUT,
+               "%s on %d partitions\n",
+               PrintInt64Size(sumStorage),
+               sumPartitions);
     }
     return 0;
 }
 
 static int
-ChangeAddr(register struct cmd_syndesc *as, void *arock)
+ChangeAddr(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 ip1, ip2, vcode;
     int remove = 0;
+    int force = 0;
 
-    ip1 = GetServer(as->parms[0].items->data);
+    if (noresolve)
+       ip1 = GetServerNoresolve(as->parms[0].items->data);
+    else
+       ip1 = GetServer(as->parms[0].items->data);
     if (!ip1) {
        fprintf(STDERR, "vos: invalid host address\n");
        return (EINVAL);
@@ -5159,15 +5184,22 @@ ChangeAddr(register struct cmd_syndesc *as, void *arock)
        return (EINVAL);
     }
 
+    if (as->parms[3].items) {
+       force = 1;
+    }
+
     if (as->parms[1].items) {
-       ip2 = GetServer(as->parms[1].items->data);
+       if (noresolve)
+           ip2 = GetServerNoresolve(as->parms[1].items->data);
+       else
+           ip2 = GetServer(as->parms[1].items->data);
        if (!ip2) {
            fprintf(STDERR, "vos: invalid host address\n");
            return (EINVAL);
        }
     } else {
        /* Play a trick here. If we are removing an address, ip1 will be -1
-        * and ip2 will be the original address. This switch prevents an 
+        * and ip2 will be the original address. This switch prevents an
         * older revision vlserver from removing the IP address.
         */
        remove = 1;
@@ -5175,18 +5207,58 @@ ChangeAddr(register struct cmd_syndesc *as, void *arock)
        ip1 = 0xffffffff;
     }
 
+    if (!remove && !force) {
+       afs_int32 m_nentries;
+       bulkaddrs m_addrs;
+       afs_int32 m_uniq = 0;
+       afsUUID m_uuid;
+       ListAddrByAttributes m_attrs;
+       struct uuid_fmtbuf buffer;
+
+       memset(&m_attrs, 0, sizeof(m_attrs));
+       memset(&m_uuid, 0, sizeof(m_uuid));
+       memset(&m_addrs, 0, sizeof(m_addrs));
+
+       m_attrs.Mask = VLADDR_IPADDR;
+       m_attrs.ipaddr = ntohl(ip1);    /* -oldaddr */
+
+       vcode =
+           ubik_VL_GetAddrsU(cstruct, UBIK_CALL_NEW, &m_attrs, &m_uuid,
+                             &m_uniq, &m_nentries, &m_addrs);
+       xdr_free((xdrproc_t) xdr_bulkaddrs, &m_addrs);
+       switch (vcode) {
+       case 0:         /* mh entry detected */
+           afsUUID_to_string(&m_uuid, &buffer);
+           fprintf(STDERR, "vos: Refusing to change address in multi-homed server entry.\n");
+           fprintf(STDERR, "     -oldaddr address is registered to file server UUID %s\n",
+                           buffer.buffer);
+           fprintf(STDERR, "     Please restart the file server or use vos setaddrs.\n");
+           return EINVAL;
+       case VL_NOENT:
+           break;
+       default:
+           fprintf(STDERR, "vos: could not list the server addresses\n");
+           PrintError("", vcode);
+           return vcode;
+       }
+    }
+
     vcode = ubik_VL_ChangeAddr(cstruct, UBIK_CALL_NEW, ntohl(ip1), ntohl(ip2));
     if (vcode) {
+       char hoststr1[16], hoststr2[16];
        if (remove) {
+           afs_inet_ntoa_r(ip2, hoststr2);
            fprintf(STDERR, "Could not remove server %s from the VLDB\n",
-                   as->parms[0].items->data);
+                   hoststr2);
            if (vcode == VL_NOENT) {
                fprintf(STDERR,
                        "vlserver does not support the remove flag or ");
            }
        } else {
+           afs_inet_ntoa_r(ip1, hoststr1);
+           afs_inet_ntoa_r(ip2, hoststr2);
            fprintf(STDERR, "Could not change server %s to server %s\n",
-                   as->parms[0].items->data, as->parms[1].items->data);
+                   hoststr1, hoststr2);
        }
        PrintError("", vcode);
        return (vcode);
@@ -5206,79 +5278,23 @@ static void
 print_addrs(const bulkaddrs * addrs, afsUUID * m_uuid, int nentries,
            int print)
 {
-    afs_int32 vcode, m_uniq=0;
-    afs_int32 i, j;
-    afs_int32 *addrp;
-    bulkaddrs m_addrs;
-    ListAddrByAttributes m_attrs;
-    afs_int32 m_nentries, *m_addrp;
-    afs_int32 base, index;
-    char buf[1024];
+    int i;
+    afs_uint32 addr;
+    struct uuid_fmtbuf buf;
 
     if (print) {
-       afsUUID_to_string(m_uuid, buf, sizeof(buf));
-       printf("UUID: %s\n", buf);
+       afsUUID_to_string(m_uuid, &buf);
+       printf("UUID: %s\n", buf.buffer);
     }
 
     /* print out the list of all the server */
-    addrp = (afs_int32 *) addrs->bulkaddrs_val;
-    for (i = 0; i < nentries; i++, addrp++) {
-       /* If it is a multihomed address, then we will need to 
-        * get the addresses for this multihomed server from
-        * the vlserver and print them.
-        */
-       if (((*addrp & 0xff000000) == 0xff000000) && ((*addrp) & 0xffff)) {
-           /* Get the list of multihomed fileservers */
-           base = (*addrp >> 16) & 0xff;
-           index = (*addrp) & 0xffff;
-
-           if ((base >= 0) && (base <= VL_MAX_ADDREXTBLKS) && (index >= 1)
-               && (index <= VL_MHSRV_PERBLK)) {
-               m_attrs.Mask = VLADDR_INDEX;
-               m_attrs.index = (base * VL_MHSRV_PERBLK) + index;
-               m_nentries = 0;
-               m_addrs.bulkaddrs_val = 0;
-               m_addrs.bulkaddrs_len = 0;
-               vcode =
-                   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");
-                   PrintError("", vcode);
-               }
-
-               /* Print the list */
-               m_addrp = (afs_int32 *) m_addrs.bulkaddrs_val;
-               for (j = 0; j < m_nentries; j++, m_addrp++) {
-                   *m_addrp = htonl(*m_addrp);
-                   if (noresolve) {
-                       char hoststr[16];
-                       printf("%s ", afs_inet_ntoa_r(*m_addrp, hoststr));
-                   } else {
-                       printf("%s ", hostutil_GetNameByINet(*m_addrp));
-                   }
-               }
-               if (j == 0) {
-                   printf("<unknown>\n");
-               } else {
-                   printf("\n");
-               }
-
-               continue;
-           }
-       }
-
-       /* Otherwise, it is a non-multihomed entry and contains
-        * the IP address of the server - print it.
-        */
-       *addrp = htonl(*addrp);
+    for (i = 0; i < addrs->bulkaddrs_len; i++) {
+       addr = htonl(addrs->bulkaddrs_val[i]);
        if (noresolve) {
            char hoststr[16];
-           printf("%s\n", afs_inet_ntoa_r(*addrp, hoststr));
+           printf("%s\n", afs_inet_ntoa_r(addr, hoststr));
        } else {
-           printf("%s\n", hostutil_GetNameByINet(*addrp));
+           printf("%s\n", hostutil_GetNameByINet(addr));
        }
     }
 
@@ -5289,7 +5305,7 @@ print_addrs(const bulkaddrs * addrs, afsUUID * m_uuid, int nentries,
 }
 
 static int
-ListAddrs(register struct cmd_syndesc *as, void *arock)
+ListAddrs(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 vcode, m_uniq=0;
     afs_int32 i, printuuid = 0;
@@ -5300,25 +5316,26 @@ ListAddrs(register struct cmd_syndesc *as, void *arock)
     afsUUID m_uuid, askuuid;
     afs_int32 m_nentries;
 
-    memset(&m_attrs, 0, sizeof(struct ListAddrByAttributes));
-    m_attrs.Mask = VLADDR_INDEX;
+    if (as->parms[0].items && as->parms[1].items) {
+       fprintf(STDERR, "vos: Can't use the -uuid and -host flags together\n");
+       exit(-1);
+    }
 
-    memset(&m_addrs, 0, sizeof(bulkaddrs));
+    memset(&m_attrs, 0, sizeof(struct ListAddrByAttributes));
     memset(&askuuid, 0, sizeof(afsUUID));
     if (as->parms[0].items) {
        /* -uuid */
-        if (afsUUID_from_string(as->parms[0].items->data, &askuuid) < 0) {
-           fprintf(STDERR, "vos: invalid UUID '%s'\n", 
+       if (afsUUID_from_string(as->parms[0].items->data, &askuuid) < 0) {
+           fprintf(STDERR, "vos: invalid UUID '%s'\n",
                    as->parms[0].items->data);
            exit(-1);
        }
        m_attrs.Mask = VLADDR_UUID;
        m_attrs.uuid = askuuid;
-    }
-    if (as->parms[1].items) {
+    } else if (as->parms[1].items) {
        /* -host */
        struct hostent *he;
-       afs_int32 saddr;
+       afs_uint32 saddr;
        he = hostutil_GetHostByName((char *)as->parms[1].items->data);
        if (he == NULL) {
            fprintf(STDERR, "vos: Can't get host info for '%s'\n",
@@ -5328,13 +5345,17 @@ ListAddrs(register struct cmd_syndesc *as, void *arock)
        memcpy(&saddr, he->h_addr, 4);
        m_attrs.Mask = VLADDR_IPADDR;
        m_attrs.ipaddr = ntohl(saddr);
+    } else {
+       /* by index */
+       m_attrs.Mask = VLADDR_INDEX;
     }
+
     if (as->parms[2].items) {
        printuuid = 1;
     }
 
-    m_addrs.bulkaddrs_val = 0;
-    m_addrs.bulkaddrs_len = 0;
+    memset(&m_addrs, 0, sizeof(bulkaddrs));
+    memset(&vlcb, 0, sizeof(struct VLCallBack));
 
     vcode =
        ubik_VL_GetAddrs(cstruct, UBIK_CALL_NEW, 0, 0, &vlcb, &nentries,
@@ -5342,58 +5363,181 @@ ListAddrs(register struct cmd_syndesc *as, void *arock)
     if (vcode) {
        fprintf(STDERR, "vos: could not list the server addresses\n");
        PrintError("", vcode);
-       return (vcode);
+       goto out;
     }
 
-    m_nentries = 0;
-    m_addrs.bulkaddrs_val = 0;
-    m_addrs.bulkaddrs_len = 0;
-    i = 1;
-    while (1) {
+    for (i = 1, m_nentries = 0; nentries; i++) {
        m_attrs.index = i;
 
+       xdr_free((xdrproc_t)xdr_bulkaddrs, &m_addrs); /* reset addr list */
        vcode =
            ubik_VL_GetAddrsU(cstruct, UBIK_CALL_NEW, &m_attrs, &m_uuid,
                              &m_uniq, &m_nentries, &m_addrs);
+       switch (vcode) {
+       case 0: /* success */
+           print_addrs(&m_addrs, &m_uuid, m_nentries, printuuid);
+           nentries--;
+           break;
 
-       if (vcode == VL_NOENT) {
-           if (m_attrs.Mask == VLADDR_UUID) {
-               fprintf(STDERR, "vos: no entry for UUID '%s' found in VLDB\n",
+       case VL_NOENT:
+           if (m_attrs.Mask == VLADDR_UUID) {
+               fprintf(STDERR, "vos: no entry for UUID '%s' found in VLDB\n",
                        as->parms[0].items->data);
                exit(-1);
            } else if (m_attrs.Mask == VLADDR_IPADDR) {
-               fprintf(STDERR, "vos: no entry for host '%s' [0x%08x] found in VLDB\n",
+               fprintf(STDERR, "vos: no entry for host '%s' [0x%08x] found in VLDB\n",
                        as->parms[1].items->data, m_attrs.ipaddr);
                exit(-1);
-           } else {
-               i++;
-               nentries++;
-               continue;
            }
-       }
+           continue;
 
-       if (vcode == VL_INDEXERANGE) {
-           break;
-       }
+       case VL_INDEXERANGE:
+           vcode = 0; /* not an error, just means we're done */
+           goto out;
 
-       if (vcode) {
+       default: /* any other error */
            fprintf(STDERR, "vos: could not list the server addresses\n");
            PrintError("", vcode);
-           return (vcode);
+           goto out;
        }
 
-       print_addrs(&m_addrs, &m_uuid, m_nentries, printuuid);
-       i++;
-
-       if ((as->parms[1].items) || (as->parms[0].items) || (i > nentries))
+       /* if -uuid or -host, only list one response */
+       if (as->parms[1].items || as->parms[0].items)
            break;
     }
 
+out:
+    xdr_free((xdrproc_t)xdr_bulkaddrs, &m_addrs);
+    return vcode;
+}
+
+static int
+SetAddrs(struct cmd_syndesc *as, void *arock)
+{
+    afs_int32 vcode;
+    bulkaddrs m_addrs;
+    afsUUID askuuid;
+    afs_uint32 FS_HostAddrs_HBO[ADDRSPERSITE];
+
+    memset(&m_addrs, 0, sizeof(bulkaddrs));
+    memset(&askuuid, 0, sizeof(afsUUID));
+    if (as->parms[0].items) {
+       /* -uuid */
+       if (afsUUID_from_string(as->parms[0].items->data, &askuuid) < 0) {
+           fprintf(STDERR, "vos: invalid UUID '%s'\n",
+                   as->parms[0].items->data);
+           exit(-1);
+       }
+    }
+    if (as->parms[1].items) {
+       /* -host */
+       struct cmd_item *ti;
+       afs_uint32 saddr;
+       int i = 0;
+
+       for (ti = as->parms[1].items; ti && i < ADDRSPERSITE; ti = ti->next) {
+
+           if (noresolve)
+               saddr = GetServerNoresolve(ti->data);
+           else
+               saddr = GetServer(ti->data);
+
+           if (!saddr) {
+               fprintf(STDERR, "vos: Can't get host info for '%s'\n",
+                       ti->data);
+               exit(-1);
+           }
+           /* Convert it to host byte order */
+           FS_HostAddrs_HBO[i] = ntohl(saddr);
+           i++;
+       }
+       m_addrs.bulkaddrs_len = i;
+       m_addrs.bulkaddrs_val = FS_HostAddrs_HBO;
+    }
+
+    vcode = ubik_VL_RegisterAddrs(cstruct, 0, &askuuid, 0, &m_addrs);
+
+    if (vcode) {
+       if (vcode == VL_MULTIPADDR) {
+           fprintf(STDERR, "vos: VL_RegisterAddrs rpc failed; The IP address exists on a different server; repair it\n");
+       } else if (vcode == RXGEN_OPCODE) {
+           fprintf(STDERR, "vlserver doesn't support VL_RegisterAddrs rpc; ignored\n");
+       } else {
+           fprintf(STDERR, "vos: VL_RegisterAddrs rpc failed\n");
+       }
+       PrintError("", vcode);
+       return vcode;
+    }
+    if (verbose) {
+       fprintf(STDOUT, "vos: Changed UUID with addresses:\n");
+       print_addrs(&m_addrs, &askuuid, m_addrs.bulkaddrs_len, 1);
+    }
     return 0;
 }
 
 static int
-LockEntry(register struct cmd_syndesc *as, void *arock)
+RemoveAddrs(struct cmd_syndesc *as, void *arock)
+{
+    afs_int32 code;
+    ListAddrByAttributes attrs;
+    afsUUID uuid;
+    afs_int32 uniq = 0;
+    afs_int32 nentries = 0;
+    bulkaddrs addrs;
+    afs_uint32 ip1;
+    afs_uint32 ip2;
+
+    memset(&attrs, 0, sizeof(ListAddrByAttributes));
+    memset(&addrs, 0, sizeof(bulkaddrs));
+    memset(&uuid, 0, sizeof(afsUUID));
+    attrs.Mask = VLADDR_UUID;
+
+    if (as->parms[0].items) {  /* -uuid */
+       if (afsUUID_from_string(as->parms[0].items->data, &attrs.uuid) < 0) {
+           fprintf(STDERR, "vos: invalid UUID '%s'\n",
+                   as->parms[0].items->data);
+           return EINVAL;
+       }
+    }
+
+    code =
+       ubik_VL_GetAddrsU(cstruct, UBIK_CALL_NEW, &attrs, &uuid, &uniq,
+                         &nentries, &addrs);
+    if (code == VL_NOENT) {
+       fprintf(STDERR, "vos: UUID not found\n");
+       goto out;
+    }
+    if (code != 0) {
+       fprintf(STDERR, "vos: could not list the server addresses\n");
+       PrintError("", code);
+       goto out;
+    }
+    if (addrs.bulkaddrs_len == 0) {
+       fprintf(STDERR, "vos: no addresses found for UUID\n");
+       goto out;
+    }
+
+    ip2 = addrs.bulkaddrs_val[0]; /* network byte order */
+    ip1 = 0xffffffff;             /* indicates removal mode */
+
+    if (verbose) {
+       printf("vos: Removing UUID with hosts:\n");
+       print_addrs(&addrs, &uuid, nentries, 1);
+    }
+
+    code = ubik_VL_ChangeAddr(cstruct, UBIK_CALL_NEW, ip1, ip2);
+    if (code) {
+       fprintf(STDERR, "Could not remove server entry from the VLDB.\n");
+       PrintError("", code);
+    }
+
+  out:
+    xdr_free((xdrproc_t) xdr_bulkaddrs, &addrs);
+    return code;
+}
+
+static int
+LockEntry(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
     afs_int32 vcode, err;
@@ -5420,22 +5564,20 @@ LockEntry(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-ConvertRO(register struct cmd_syndesc *as, void *arock)
+ConvertRO(struct cmd_syndesc *as, void *arock)
 {
     afs_int32 partition = -1;
     afs_uint32 volid;
-    afs_int32 server, code, i, same;
-    struct nvldbentry entry, storeEntry;
+    afs_uint32 server;
+    afs_int32 code, i, same;
+    struct nvldbentry entry;
     afs_int32 vcode;
-    afs_int32 rwindex = 0;
-    afs_int32 rwserver = 0;
+    afs_uint32 rwserver = 0;
     afs_int32 rwpartition = 0;
-    afs_int32 roindex = 0;
-    afs_int32 roserver = 0;
+    afs_uint32 roserver = 0;
     afs_int32 ropartition = 0;
     int force = 0;
-    struct rx_connection *aconn;
-    char c, dc;
+    int c, dc;
 
     server = GetServer(as->parms[0].items->data);
     if (!server) {
@@ -5464,34 +5606,34 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
            PrintError("", code);
        else
            fprintf(STDERR, "Unknown volume ID or name '%s'\n",
-                   as->parms[0].items->data);
+                   as->parms[2].items->data);
        return -1;
     }
     if (as->parms[3].items)
        force = 1;
 
+    memset(&entry, 0, sizeof(entry));
     vcode = VLDB_GetEntryByID(volid, -1, &entry);
     if (vcode) {
        fprintf(STDERR,
                "Could not fetch the entry for volume %lu from VLDB\n",
                (unsigned long)volid);
-       PrintError("convertROtoRW", code);
+       PrintError("convertROtoRW ", vcode);
        return vcode;
     }
 
     /* use RO volid even if user specified RW or BK volid */
-
     if (volid != entry.volumeId[ROVOL])
        volid = entry.volumeId[ROVOL];
 
     MapHostToNetwork(&entry);
     for (i = 0; i < entry.nServers; i++) {
-       if (entry.serverFlags[i] & ITSRWVOL) {
-           rwindex = i;
+       if (entry.serverFlags[i] & VLSF_RWVOL) {
            rwserver = entry.serverNumber[i];
            rwpartition = entry.serverPartition[i];
-       }
-       if (entry.serverFlags[i] & ITSROVOL) {
+           if (roserver)
+               break;
+       } else if ((entry.serverFlags[i] & VLSF_ROVOL) && !roserver) {
            same = VLDB_IsSameAddrs(server, entry.serverNumber[i], &code);
            if (code) {
                fprintf(STDERR,
@@ -5500,17 +5642,17 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
                return ENOENT;
            }
            if (same) {
-               roindex = i;
                roserver = entry.serverNumber[i];
                ropartition = entry.serverPartition[i];
-               break;
+               if (rwserver)
+                    break;
            }
        }
     }
     if (!roserver) {
        fprintf(STDERR, "Warning: RO volume didn't exist in vldb!\n");
     }
-    if (ropartition != partition) {
+    if (roserver && (ropartition != partition)) {
        fprintf(STDERR,
                "Warning: RO volume should be in partition %d instead of %d (vldb)\n",
                ropartition, partition);
@@ -5533,62 +5675,17 @@ ConvertRO(register struct cmd_syndesc *as, void *arock)
        }
     }
 
-    vcode =
-       ubik_VL_SetLock(cstruct, 0, entry.volumeId[RWVOL], RWVOL,
-                 VLOP_MOVE);
-    aconn = UV_Bind(server, AFSCONF_VOLUMEPORT);
-    code = AFSVolConvertROtoRWvolume(aconn, partition, volid);
-    if (code) {
-       fprintf(STDERR,
-               "Converting RO volume %lu to RW volume failed with code %d\n",
-               (unsigned long)volid, code);
-       PrintError("convertROtoRW ", code);
-       return -1;
-    }
-    entry.serverFlags[roindex] = ITSRWVOL;
-    entry.flags |= RW_EXISTS;
-    entry.flags &= ~BACK_EXISTS;
-    if (rwserver) {
-       (entry.nServers)--;
-       if (rwindex != entry.nServers) {
-           entry.serverNumber[rwindex] = entry.serverNumber[entry.nServers];
-           entry.serverPartition[rwindex] =
-               entry.serverPartition[entry.nServers];
-           entry.serverFlags[rwindex] = entry.serverFlags[entry.nServers];
-           entry.serverNumber[entry.nServers] = 0;
-           entry.serverPartition[entry.nServers] = 0;
-           entry.serverFlags[entry.nServers] = 0;
-       }
-    }
-    entry.flags &= ~RO_EXISTS;
-    for (i = 0; i < entry.nServers; i++) {
-       if (entry.serverFlags[i] & ITSROVOL) {
-           if (!(entry.serverFlags[i] & (RO_DONTUSE | NEW_REPSITE)))
-               entry.flags |= RO_EXISTS;
-       }
-    }
-    MapNetworkToHost(&entry, &storeEntry);
-    code =
-       VLDB_ReplaceEntry(entry.volumeId[RWVOL], RWVOL, &storeEntry,
-                         (LOCKREL_OPCODE | LOCKREL_AFSID |
-                          LOCKREL_TIMESTAMP));
-    if (code) {
-       fprintf(STDERR,
-               "Warning: volume converted, but vldb update failed with code %d!\n",
-               code);
-    }
-    vcode = UV_LockRelease(entry.volumeId[RWVOL]);
-    if (vcode) {
-       PrintDiagnostics("unlock", vcode);
-    }
+    code = UV_ConvertRO(server, partition, volid, &entry);
+
     return code;
 }
 
 static int
-Sizes(register struct cmd_syndesc *as, void *arock)
+Sizes(struct cmd_syndesc *as, void *arock)
 {
     afs_uint32 avolid;
-    afs_int32 aserver, apart, voltype, fromdate = 0, code, err, i;
+    afs_uint32 aserver;
+    afs_int32 apart, voltype, fromdate = 0, code, err, i;
     struct nvldbentry entry;
     volintSize vol_size;
 
@@ -5598,8 +5695,8 @@ Sizes(register struct cmd_syndesc *as, void *arock)
        if (rxConn == 0)
            break;
        rx_SetConnDeadTime(rxConn, rx_connDeadTime);
-       if (rxConn->service)
-           rxConn->service->connDeadTime = rx_connDeadTime;
+       if (rx_ServiceOf(rxConn))
+           rx_ServiceOf(rxConn)->connDeadTime = rx_connDeadTime;
     }
 
     avolid = vsu_GetVolumeID(as->parms[0].items->data, cstruct, &err);
@@ -5648,11 +5745,7 @@ Sizes(register struct cmd_syndesc *as, void *arock)
     fprintf(STDOUT, "Volume: %s\n", as->parms[0].items->data);
 
     if (as->parms[3].items) {  /* do the dump estimate */
-#ifdef AFS_64BIT_ENV
        vol_size.dump_size = 0;
-#else
-   FillInt64(vol_size.dump_size,0, 1);
-#endif
        code = UV_GetSize(avolid, aserver, apart, fromdate, &vol_size);
        if (code) {
            PrintDiagnostics("size", code);
@@ -5669,21 +5762,61 @@ Sizes(register struct cmd_syndesc *as, void *arock)
     return 0;
 }
 
+static int
+EndTrans(struct cmd_syndesc *as, void *arock)
+{
+    afs_uint32 server;
+    afs_int32 code, tid, rcode;
+    struct rx_connection *aconn;
+
+    server = GetServer(as->parms[0].items->data);
+    if (!server) {
+       fprintf(STDERR, "vos: host '%s' not found in host table\n",
+               as->parms[0].items->data);
+       return EINVAL;
+    }
+
+    code = util_GetInt32(as->parms[1].items->data, &tid);
+    if (code) {
+       fprintf(STDERR, "vos: bad integer specified for transaction ID.\n");
+       return code;
+    }
+
+    aconn = UV_Bind(server, AFSCONF_VOLUMEPORT);
+    code = AFSVolEndTrans(aconn, tid, &rcode);
+    if (!code) {
+       code = rcode;
+    }
+
+    if (code) {
+       PrintDiagnostics("endtrans", code);
+       return 1;
+    }
+
+    return 0;
+}
+
 int
 PrintDiagnostics(char *astring, afs_int32 acode)
 {
-    if (acode == EACCES) {
+    switch (acode) {
+    case EACCES:
        fprintf(STDERR,
                "You are not authorized to perform the 'vos %s' command (%d)\n",
                astring, acode);
-    } else {
+       break;
+    case EXDEV:
+       fprintf(STDERR, "Error in vos %s command.\n", astring);
+       fprintf(STDERR, "Clone volume is not in the same partition as the read-write volume.\n");
+       break;
+    default:
        fprintf(STDERR, "Error in vos %s command.\n", astring);
        PrintError("", acode);
+       break;
     }
     return 0;
 }
 
-
 #ifdef AFS_NT40_ENV
 static DWORD
 win32_enableCrypt(void)
@@ -5695,11 +5828,11 @@ win32_enableCrypt(void)
 
     /* 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);
+                       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);
+       dummyLen = sizeof(cryptall);
+       RegQueryValueEx(parmKey, "SecurityLevel", NULL, NULL,
+                       (BYTE *) &cryptall, &dummyLen);
     }
     RegCloseKey (parmKey);
 
@@ -5710,50 +5843,71 @@ win32_enableCrypt(void)
 static int
 MyBeforeProc(struct cmd_syndesc *as, void *arock)
 {
-    register char *tcell;
-    register afs_int32 code;
-    register afs_int32 sauth;
+    char *tcell;
+    char *rxgk_seclevel_str = NULL;
+    afs_int32 code;
+    int secFlags;
 
     /* Initialize the ubik_client connection */
     rx_SetRxDeadTime(90);
-    cstruct = (struct ubik_client *)0;
+    cstruct = NULL;
+    secFlags = AFSCONF_SECOPTS_FALLBACK_NULL;
 
-    sauth = 0;
     tcell = NULL;
-    if (as->parms[12].items)   /* if -cell specified */
-       tcell = as->parms[12].items->data;
-    if (as->parms[14].items)   /* -serverauth specified */
-       sauth = 1;
-    if (as->parms[16].items     /* -encrypt specified */
+    if (as->parms[COMMONPARM_OFFSET_CELL].items)       /* if -cell specified */
+       tcell = as->parms[COMMONPARM_OFFSET_CELL].items->data;
+
+    if (as->parms[COMMONPARM_OFFSET_NOAUTH].items)
+       secFlags |= AFSCONF_SECOPTS_NOAUTH;
+
+    if (as->parms[COMMONPARM_OFFSET_LOCALAUTH].items) {        /* -localauth specified */
+       secFlags |= AFSCONF_SECOPTS_LOCALAUTH;
+       confdir = AFSDIR_SERVER_ETC_DIRPATH;
+    }
+
+    if (as->parms[COMMONPARM_OFFSET_ENCRYPT].items     /* -encrypt specified */
 #ifdef AFS_NT40_ENV
-        || win32_enableCrypt()
+       || win32_enableCrypt()
 #endif /* AFS_NT40_ENV */
-         )
-       vsu_SetCrypt(1);
-    if ((code =
-        vsu_ClientInit((as->parms[13].items != 0), confdir, tcell, sauth,
-                       &cstruct, UV_SetSecurity))) {
+        )
+       secFlags |= AFSCONF_SECOPTS_ALWAYSENCRYPT;
+
+    if (as->parms[COMMONPARM_OFFSET_CONFIG].items)   /* -config flag set */
+       confdir = as->parms[COMMONPARM_OFFSET_CONFIG].items->data;
+
+    if (cmd_OptionAsString(as, COMMONPARM_OFFSET_RXGK, &rxgk_seclevel_str) == 0) {
+       if (strcmp(rxgk_seclevel_str, "clear") == 0)
+           secFlags |= AFSCONF_SECOPTS_ALWAYSCLEAR;
+       else if (strcmp(rxgk_seclevel_str, "auth") == 0)
+           secFlags |= AFSCONF_SECOPTS_NEVERENCRYPT;
+       else if (strcmp(rxgk_seclevel_str, "crypt") == 0) {
+           /* don't need to set any flags; this is the default for rxgk */
+       } else {
+           fprintf(STDERR, "Invalid argument to -rxgk: %s\n", rxgk_seclevel_str);
+           exit(1);
+       }
+       secFlags |= AFSCONF_SECOPTS_RXGK;
+
+       free(rxgk_seclevel_str);
+       rxgk_seclevel_str = NULL;
+    }
+
+    if ((code = vsu_ClientInit(confdir, tcell, secFlags, UV_SetSecurity,
+                              &cstruct))) {
        fprintf(STDERR, "could not initialize VLDB library (code=%lu) \n",
                (unsigned long)code);
        exit(1);
     }
     rxInitDone = 1;
-    if (as->parms[15].items)   /* -verbose flag set */
+    if (as->parms[COMMONPARM_OFFSET_VERBOSE].items)    /* -verbose flag set */
        verbose = 1;
     else
        verbose = 0;
-    if (as->parms[17].items)   /* -noresolve flag set */
+    if (as->parms[COMMONPARM_OFFSET_NORESOLVE].items)  /* -noresolve flag set */
        noresolve = 1;
     else
        noresolve = 0;
-    return 0;
-}
 
-int
-osi_audit(void)
-{
-/* this sucks but it works for now.
-*/
     return 0;
 }
 
@@ -5762,14 +5916,14 @@ osi_audit(void)
 int
 main(int argc, char **argv)
 {
-    register afs_int32 code;
+    afs_int32 code;
 
-    register struct cmd_syndesc *ts;
+    struct cmd_syndesc *ts;
 
 #ifdef AFS_AIX32_ENV
     /*
-     * The following signal action for AIX is necessary so that in case of a 
-     * crash (i.e. core is generated) we can include the user's data section 
+     * The following signal action for AIX is necessary so that in case of a
+     * crash (i.e. core is generated) we can include the user's data section
      * in the core dump. Unfortunately, by default, only a partial core is
      * generated which, in many cases, isn't too useful.
      */
@@ -5785,7 +5939,7 @@ main(int argc, char **argv)
 
     cmd_SetBeforeProc(MyBeforeProc, NULL);
 
-    ts = cmd_CreateSyntax("create", CreateVolume, NULL, "create a new volume");
+    ts = cmd_CreateSyntax("create", CreateVolume, NULL, 0, "create a new 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, "volume name");
@@ -5793,19 +5947,16 @@ main(int argc, char **argv)
                "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
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("remove", DeleteVolume, NULL, "delete a volume");
+    ts = cmd_CreateSyntax("remove", DeleteVolume, NULL, 0, "delete a volume");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_OPTIONAL, "machine name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, CMD_OPTIONAL, "partition name");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
 
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("move", MoveVolume, NULL, "move a volume");
+    ts = cmd_CreateSyntax("move", MoveVolume, NULL, 0, "move a volume");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
     cmd_AddParm(ts, "-fromserver", CMD_SINGLE, 0, "machine name on source");
     cmd_AddParm(ts, "-frompartition", CMD_SINGLE, 0,
@@ -5818,7 +5969,7 @@ main(int argc, char **argv)
                "copy live volume without cloning");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("copy", CopyVolume, NULL, "copy a volume");
+    ts = cmd_CreateSyntax("copy", CopyVolume, NULL, 0, "copy a volume");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID on source");
     cmd_AddParm(ts, "-fromserver", CMD_SINGLE, 0, "machine name on source");
     cmd_AddParm(ts, "-frompartition", CMD_SINGLE, 0,
@@ -5836,7 +5987,7 @@ main(int argc, char **argv)
                "copy live volume without cloning");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("shadow", ShadowVolume, NULL,
+    ts = cmd_CreateSyntax("shadow", ShadowVolume, NULL, 0,
                          "make or update a shadow volume");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID on source");
     cmd_AddParm(ts, "-fromserver", CMD_SINGLE, 0, "machine name on source");
@@ -5860,12 +6011,12 @@ main(int argc, char **argv)
                "do incremental update if target exists");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("backup", BackupVolume, NULL,
+    ts = cmd_CreateSyntax("backup", BackupVolume, NULL, 0,
                          "make backup of a volume");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("clone", CloneVolume, NULL,
+    ts = cmd_CreateSyntax("clone", CloneVolume, NULL, 0,
                          "make clone of a volume");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_OPTIONAL, "server");
@@ -5878,15 +6029,23 @@ main(int argc, char **argv)
                "leave clone volume offline");
     cmd_AddParm(ts, "-readonly", CMD_FLAG, CMD_OPTIONAL,
                "make clone volume read-only, not readwrite");
+    cmd_AddParm(ts, "-readwrite", CMD_FLAG, CMD_OPTIONAL,
+               "make clone volume readwrite, not read-only");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("release", ReleaseVolume, NULL, "release a volume");
+    ts = cmd_CreateSyntax("release", ReleaseVolume, NULL, 0, "release a volume");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
     cmd_AddParm(ts, "-force", CMD_FLAG, CMD_OPTIONAL,
-               "force a complete release");
+               "force a complete release and full dumps");
+    cmd_AddParmAlias(ts, 1, "-f"); /* original force option */
+    /* keep this reserved */
+    cmd_AddParm(ts, "-stayonline", CMD_FLAG, CMD_OPTIONAL|CMD_HIDDEN,
+               "release to cloned temp vol, then clone back to repsite RO");
+    cmd_AddParm(ts, "-force-reclone", CMD_FLAG, CMD_OPTIONAL,
+               "force a reclone and complete release with incremental dumps");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("dump", DumpVolumeCmd, NULL, "dump a volume");
+    ts = cmd_CreateSyntax("dump", DumpVolumeCmd, NULL, 0, "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");
@@ -5898,7 +6057,7 @@ main(int argc, char **argv)
                "omit unchanged directories from an incremental dump");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("restore", RestoreVolumeCmd, NULL,
+    ts = cmd_CreateSyntax("restore", RestoreVolumeCmd, NULL, 0,
                          "restore a volume");
     cmd_AddParm(ts, "-server", CMD_SINGLE, 0, "machine name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, 0, "partition name");
@@ -5919,12 +6078,12 @@ main(int argc, char **argv)
                "do not delete old site when restoring to a new site");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("unlock", LockReleaseCmd, NULL,
+    ts = cmd_CreateSyntax("unlock", LockReleaseCmd, NULL, 0,
                          "release lock on VLDB entry for a volume");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("changeloc", ChangeLocation, NULL,
+    ts = cmd_CreateSyntax("changeloc", ChangeLocation, NULL, 0,
                          "change an RW volume's location in the VLDB");
     cmd_AddParm(ts, "-server", CMD_SINGLE, 0,
                "machine name for new location");
@@ -5933,7 +6092,7 @@ main(int argc, char **argv)
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("addsite", AddSite, NULL, "add a replication site");
+    ts = cmd_CreateSyntax("addsite", AddSite, NULL, 0, "add a replication site");
     cmd_AddParm(ts, "-server", CMD_SINGLE, 0, "machine name for new site");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, 0,
                "partition name for new site");
@@ -5942,18 +6101,18 @@ main(int argc, char **argv)
     cmd_AddParm(ts, "-valid", CMD_FLAG, CMD_OPTIONAL, "publish as an up-to-date site in VLDB");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("remsite", RemoveSite, NULL,
+    ts = cmd_CreateSyntax("remsite", RemoveSite, NULL, 0,
                          "remove a replication site");
     cmd_AddParm(ts, "-server", CMD_SINGLE, 0, "machine name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, 0, "partition name");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("listpart", ListPartitions, NULL, "list partitions");
+    ts = cmd_CreateSyntax("listpart", ListPartitions, NULL, 0, "list partitions");
     cmd_AddParm(ts, "-server", CMD_SINGLE, 0, "machine name");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("listvol", ListVolumes, NULL,
+    ts = cmd_CreateSyntax("listvol", ListVolumes, NULL, 0,
                          "list volumes on server (bypass VLDB)");
     cmd_AddParm(ts, "-server", CMD_SINGLE, 0, "machine name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, CMD_OPTIONAL, "partition name");
@@ -5964,40 +6123,37 @@ main(int argc, char **argv)
                "generate minimal information");
     cmd_AddParm(ts, "-extended", CMD_FLAG, CMD_OPTIONAL,
                "list extended volume fields");
-#ifdef FULL_LISTVOL_SWITCH
     cmd_AddParm(ts, "-format", CMD_FLAG, CMD_OPTIONAL,
                "machine readable format");
-#endif /* FULL_LISTVOL_SWITCH */
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("syncvldb", SyncVldb, NULL,
+    ts = cmd_CreateSyntax("syncvldb", SyncVldb, NULL, 0,
                          "synchronize VLDB with server");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_OPTIONAL, "machine name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, CMD_OPTIONAL, "partition name");
     cmd_AddParm(ts, "-volume", CMD_SINGLE, CMD_OPTIONAL, "volume name or ID");
-    cmd_AddParm(ts, "-dryrun", CMD_FLAG, CMD_OPTIONAL, "report without updating");
+    cmd_AddParm(ts, "-dryrun", CMD_FLAG, CMD_OPTIONAL, "list what would be done, don't do it");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("syncserv", SyncServer, NULL,
+    ts = cmd_CreateSyntax("syncserv", SyncServer, NULL, 0,
                          "synchronize server with VLDB");
     cmd_AddParm(ts, "-server", CMD_SINGLE, 0, "machine name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, CMD_OPTIONAL, "partition name");
-    cmd_AddParm(ts, "-dryrun", CMD_FLAG, CMD_OPTIONAL, "report without updating");
+    cmd_AddParm(ts, "-dryrun", CMD_FLAG, CMD_OPTIONAL, "list what would be done, don't do it");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("examine", ExamineVolume, NULL,
+    ts = cmd_CreateSyntax("examine", ExamineVolume, NULL, 0,
                          "everything about the volume");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
     cmd_AddParm(ts, "-extended", CMD_FLAG, CMD_OPTIONAL,
                "list extended volume fields");
-#ifdef FULL_LISTVOL_SWITCH
     cmd_AddParm(ts, "-format", CMD_FLAG, CMD_OPTIONAL,
                "machine readable format");
-#endif /* FULL_LISTVOL_SWITCH */
     COMMONPARMS;
     cmd_CreateAlias(ts, "volinfo");
+    cmd_CreateAlias(ts, "e");
 
-    ts = cmd_CreateSyntax("setfields", SetFields, NULL,
+    ts = cmd_CreateSyntax("setfields", SetFields, NULL, 0,
                          "change volume info fields");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
     cmd_AddParm(ts, "-maxquota", CMD_SINGLE, CMD_OPTIONAL, "quota (KB)");
@@ -6005,7 +6161,7 @@ main(int argc, char **argv)
     cmd_AddParm(ts, "-clearVolUpCounter", CMD_FLAG, CMD_OPTIONAL, "clear volUpdateCounter");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("offline", volOffline, NULL, "force the volume status to offline");
+    ts = cmd_CreateSyntax("offline", volOffline, NULL, 0, "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");
@@ -6013,13 +6169,13 @@ main(int argc, char **argv)
     cmd_AddParm(ts, "-busy", CMD_FLAG, CMD_OPTIONAL, "busy volume");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("online", volOnline, NULL, "force the volume status to online");
+    ts = cmd_CreateSyntax("online", volOnline, NULL, 0, "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");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("zap", VolumeZap, NULL,
+    ts = cmd_CreateSyntax("zap", VolumeZap, NULL, 0,
                          "delete the volume, don't bother with VLDB");
     cmd_AddParm(ts, "-server", CMD_SINGLE, 0, "machine name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, 0, "partition name");
@@ -6030,17 +6186,17 @@ main(int argc, char **argv)
                "also delete backup volume if one is found");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("status", VolserStatus, NULL,
+    ts = cmd_CreateSyntax("status", VolserStatus, NULL, 0,
                          "report on volser status");
     cmd_AddParm(ts, "-server", CMD_SINGLE, 0, "machine name");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("rename", RenameVolume, NULL, "rename a volume");
+    ts = cmd_CreateSyntax("rename", RenameVolume, NULL, 0, "rename a volume");
     cmd_AddParm(ts, "-oldname", CMD_SINGLE, 0, "old volume name ");
     cmd_AddParm(ts, "-newname", CMD_SINGLE, 0, "new volume name ");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("listvldb", ListVLDB, NULL,
+    ts = cmd_CreateSyntax("listvldb", ListVLDB, NULL, 0,
                          "list volumes in the VLDB");
     cmd_AddParm(ts, "-name", CMD_SINGLE, CMD_OPTIONAL, "volume name or ID");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_OPTIONAL, "machine name");
@@ -6052,7 +6208,7 @@ main(int argc, char **argv)
                "do not alphabetically sort the volume names");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("backupsys", BackSys, NULL, "en masse backups");
+    ts = cmd_CreateSyntax("backupsys", BackSys, NULL, 0, "en masse backups");
     cmd_AddParm(ts, "-prefix", CMD_LIST, CMD_OPTIONAL,
                "common prefix on volume(s)");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_OPTIONAL, "machine name");
@@ -6061,21 +6217,22 @@ main(int argc, char **argv)
                "exclude common prefix volumes");
     cmd_AddParm(ts, "-xprefix", CMD_LIST, CMD_OPTIONAL,
                "negative prefix on volume(s)");
-    cmd_AddParm(ts, "-dryrun", CMD_FLAG, CMD_OPTIONAL, "no action");
+    cmd_AddParm(ts, "-dryrun", CMD_FLAG, CMD_OPTIONAL, "list what would be done, don't do it");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("delentry", DeleteEntry, NULL,
+    ts = cmd_CreateSyntax("delentry", DeleteEntry, NULL, 0,
                          "delete VLDB entry for a volume");
     cmd_AddParm(ts, "-id", CMD_LIST, CMD_OPTIONAL, "volume name or ID");
     cmd_AddParm(ts, "-prefix", CMD_SINGLE, CMD_OPTIONAL,
                "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,
-               "no execute");
+    cmd_AddParm(ts, "-noexecute", CMD_FLAG, CMD_OPTIONAL|CMD_HIDDEN, "");
+    cmd_AddParm(ts, "-dryrun", CMD_FLAG, CMD_OPTIONAL,
+               "list what would be done, don't do it");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("partinfo", PartitionInfo, NULL,
+    ts = cmd_CreateSyntax("partinfo", PartitionInfo, NULL, 0,
                          "list partition information");
     cmd_AddParm(ts, "-server", CMD_SINGLE, 0, "machine name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, CMD_OPTIONAL, "partition name");
@@ -6083,26 +6240,28 @@ main(int argc, char **argv)
                "print storage summary");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("unlockvldb", UnlockVLDB, NULL,
+    ts = cmd_CreateSyntax("unlockvldb", UnlockVLDB, NULL, 0,
                          "unlock all the locked entries in the VLDB");
     cmd_AddParm(ts, "-server", CMD_SINGLE, CMD_OPTIONAL, "machine name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, CMD_OPTIONAL, "partition name");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("lock", LockEntry, NULL,
+    ts = cmd_CreateSyntax("lock", LockEntry, NULL, 0,
                          "lock VLDB entry for a volume");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("changeaddr", ChangeAddr, NULL,
+    ts = cmd_CreateSyntax("changeaddr", ChangeAddr, NULL, 0,
                          "change the IP address of a file server");
     cmd_AddParm(ts, "-oldaddr", CMD_SINGLE, 0, "original IP address");
     cmd_AddParm(ts, "-newaddr", CMD_SINGLE, CMD_OPTIONAL, "new IP address");
     cmd_AddParm(ts, "-remove", CMD_FLAG, CMD_OPTIONAL,
                "remove the IP address from the VLDB");
+    cmd_AddParm(ts, "-force", CMD_FLAG, CMD_OPTIONAL,
+               "allow multi-homed server entry change (not recommended)");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("listaddrs", ListAddrs, NULL,
+    ts = cmd_CreateSyntax("listaddrs", ListAddrs, NULL, 0,
                          "list the IP address of all file servers registered in the VLDB");
     cmd_AddParm(ts, "-uuid", CMD_SINGLE, CMD_OPTIONAL, "uuid of server");
     cmd_AddParm(ts, "-host", CMD_SINGLE, CMD_OPTIONAL, "address of host");
@@ -6110,7 +6269,7 @@ main(int argc, char **argv)
                "print uuid of hosts");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("convertROtoRW", ConvertRO, NULL,
+    ts = cmd_CreateSyntax("convertROtoRW", ConvertRO, NULL, 0,
                          "convert a RO volume into a RW volume (after loss of old RW volume)");
     cmd_AddParm(ts, "-server", CMD_SINGLE, 0, "machine name");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, 0, "partition name");
@@ -6118,7 +6277,7 @@ main(int argc, char **argv)
     cmd_AddParm(ts, "-force", CMD_FLAG, CMD_OPTIONAL, "don't ask");
     COMMONPARMS;
 
-    ts = cmd_CreateSyntax("size", Sizes, NULL,
+    ts = cmd_CreateSyntax("size", Sizes, NULL, 0,
                          "obtain various sizes of the volume.");
     cmd_AddParm(ts, "-id", CMD_SINGLE, 0, "volume name or ID");
     cmd_AddParm(ts, "-partition", CMD_SINGLE, CMD_OPTIONAL, "partition name");
@@ -6128,6 +6287,24 @@ main(int argc, char **argv)
     cmd_AddParm(ts, "-time", CMD_SINGLE, CMD_OPTIONAL, "dump from time");
     COMMONPARMS;
 
+    ts = cmd_CreateSyntax("endtrans", EndTrans, NULL, 0,
+                         "end a volserver transaction");
+    cmd_AddParm(ts, "-server", CMD_SINGLE, 0, "machine name");
+    cmd_AddParm(ts, "-transaction", CMD_SINGLE, 0,
+               "transaction ID");
+    COMMONPARMS;
+
+    ts = cmd_CreateSyntax("setaddrs", SetAddrs, NULL, 0,
+                         "set the list of IP addresses for a given UUID in the VLDB");
+    cmd_AddParm(ts, "-uuid", CMD_SINGLE, 0, "uuid of server");
+    cmd_AddParm(ts, "-host", CMD_LIST, 0, "address of host");
+    COMMONPARMS;
+
+    ts = cmd_CreateSyntax("remaddrs", RemoveAddrs, NULL, 0,
+                         "remove the list of IP addresses for a given UUID in the VLDB");
+    cmd_AddParm(ts, "-uuid", CMD_SINGLE, 0, "uuid of server");
+    COMMONPARMS;
+
     code = cmd_Dispatch(argc, argv);
     if (rxInitDone) {
        /* Shut down the ubik_client and rx connections */