Revert "Convert from using nvldbentry to uvldbentry"
[openafs.git] / src / volser / vsutils.c
index e6cfb79..13c7fd5 100644 (file)
@@ -60,16 +60,7 @@ ovlentry_to_nvlentry(struct vldbentry *oentryp,
     memset(nentryp, 0, sizeof(struct nvldbentry));
     strncpy(nentryp->name, oentryp->name, sizeof(nentryp->name));
     for (i = 0; i < oentryp->nServers; i++) {
-        bulkaddrs m_addrs;
-        ListAddrByAttributes m_attrs;
-        afs_int32 m_nentries;
-
-        m_attrs.Mask = VLADDR_IPADDR;
-        m_attrs.ipaddr = oentryp->serverNumber[i];
-        m_nentries = 0;
-        m_addrs.bulkaddrs_val = 0;
-        m_addrs.bulkaddrs_len = 0;
-        nentryp->serverNumber[i] = oentryp->serverNumber[i];
+       nentryp->serverNumber[i] = oentryp->serverNumber[i];
        nentryp->serverPartition[i] = oentryp->serverPartition[i];
        nentryp->serverFlags[i] = oentryp->serverFlags[i];
     }
@@ -80,8 +71,6 @@ ovlentry_to_nvlentry(struct vldbentry *oentryp,
     nentryp->flags = oentryp->flags;
 }
 
-#if 0
-/* It is here for when we need it.  Not currently used. */
 static int
 nvlentry_to_ovlentry(struct nvldbentry *nentryp,
                      struct vldbentry *oentryp)
@@ -109,136 +98,6 @@ nvlentry_to_ovlentry(struct nvldbentry *nentryp,
     oentryp->flags = nentryp->flags;
     return 0;
 }
-#endif
-
-static void
-ovlentry_to_uvlentry(struct vldbentry *oentryp,
-                     struct uvldbentry *uentryp)
-{
-    int i;
-
-    memset(uentryp, 0, sizeof(struct uvldbentry));
-    strncpy(uentryp->name, oentryp->name, sizeof(uentryp->name));
-    for (i = 0; i < oentryp->nServers; i++) {
-        afs_int32 vcode, m_uniq=0;
-        bulkaddrs m_addrs;
-        ListAddrByAttributes m_attrs;
-        afs_int32 m_nentries;
-
-        m_attrs.Mask = VLADDR_IPADDR;
-        m_attrs.ipaddr = oentryp->serverNumber[i];
-        m_nentries = 0;
-        m_addrs.bulkaddrs_val = 0;
-        m_addrs.bulkaddrs_len = 0;
-        vcode =
-            ubik_VL_GetAddrsU(cstruct, 0, &m_attrs,
-                               &uentryp->serverNumber[i],
-                               &m_uniq, &m_nentries,
-                               &m_addrs);
-        uentryp->serverUnique[i] = oentryp->serverNumber[i];
-       uentryp->serverPartition[i] = oentryp->serverPartition[i];
-       uentryp->serverFlags[i] = oentryp->serverFlags[i];
-    }
-    uentryp->nServers = oentryp->nServers;
-    for (i = 0; i < MAXTYPES; i++)
-       uentryp->volumeId[i] = oentryp->volumeId[i];
-    uentryp->cloneId = oentryp->cloneId;
-    uentryp->flags = oentryp->flags;
-}
-
-static int
-uvlentry_to_ovlentry(struct uvldbentry *uentryp,
-                     struct vldbentry *oentryp)
-{
-    int i;
-
-    memset(oentryp, 0, sizeof(struct vldbentry));
-    strncpy(oentryp->name, uentryp->name, sizeof(oentryp->name));
-    if (uentryp->nServers > OMAXNSERVERS) {
-       /*
-        * The alternative is to store OMAXSERVERS but it's always better
-        * to know what's going on...
-        */
-       return VL_BADSERVER;
-    }
-    for (i = 0; i < uentryp->nServers; i++) {
-       oentryp->serverNumber[i] = uentryp->serverUnique[i];
-       oentryp->serverPartition[i] = uentryp->serverPartition[i];
-       oentryp->serverFlags[i] = uentryp->serverFlags[i];
-    }
-    oentryp->nServers = i;
-    for (i = 0; i < MAXTYPES; i++)
-       oentryp->volumeId[i] = uentryp->volumeId[i];
-    oentryp->cloneId = uentryp->cloneId;
-    oentryp->flags = uentryp->flags;
-    return 0;
-}
-
-void
-nvlentry_to_uvlentry(struct nvldbentry *nentryp,
-                     struct uvldbentry *uentryp)
-{
-    int i;
-
-    memset(uentryp, 0, sizeof(struct uvldbentry));
-    strncpy(uentryp->name, nentryp->name, sizeof(uentryp->name));
-    for (i = 0; i < nentryp->nServers; i++) {
-        afs_int32 vcode, m_uniq=0;
-        bulkaddrs m_addrs;
-        ListAddrByAttributes m_attrs;
-        afs_int32 m_nentries;
-
-        m_attrs.Mask = VLADDR_IPADDR;
-        m_attrs.ipaddr = nentryp->serverNumber[i];
-        m_nentries = 0;
-        m_addrs.bulkaddrs_val = 0;
-        m_addrs.bulkaddrs_len = 0;
-        vcode =
-            ubik_VL_GetAddrsU(cstruct, 0, &m_attrs,
-                               &uentryp->serverNumber[i],
-                               &m_uniq, &m_nentries,
-                               &m_addrs);
-       uentryp->serverUnique[i] = m_uniq;
-       uentryp->serverPartition[i] = nentryp->serverPartition[i];
-       uentryp->serverFlags[i] = nentryp->serverFlags[i];
-       uentryp->serverFlags[i] |= VLSERVER_FLAG_UUID;
-    }
-    uentryp->nServers = nentryp->nServers;
-    for (i = 0; i < MAXTYPES; i++)
-       uentryp->volumeId[i] = nentryp->volumeId[i];
-    uentryp->cloneId = nentryp->cloneId;
-    uentryp->flags = nentryp->flags;
-    uentryp->matchindex = nentryp->matchindex;
-}
-
-int
-uvlentry_to_nvlentry(struct uvldbentry *uentryp,
-                     struct nvldbentry *nentryp)
-{
-    int i;
-
-    memset(nentryp, 0, sizeof(struct vldbentry));
-    strncpy(nentryp->name, uentryp->name, sizeof(nentryp->name));
-    if (uentryp->nServers > NMAXNSERVERS) {
-       /*
-        * The alternative is to store NMAXSERVERS but it's always better
-        * to know what's going on...
-        */
-       return VL_BADSERVER;
-    }
-    for (i = 0; i < uentryp->nServers; i++) {
-       nentryp->serverNumber[i] = uentryp->serverUnique[i];
-       nentryp->serverPartition[i] = uentryp->serverPartition[i];
-       nentryp->serverFlags[i] = uentryp->serverFlags[i];
-    }
-    nentryp->nServers = i;
-    for (i = 0; i < MAXTYPES; i++)
-       nentryp->volumeId[i] = uentryp->volumeId[i];
-    nentryp->cloneId = uentryp->cloneId;
-    nentryp->flags = uentryp->flags;
-    nentryp->matchindex = uentryp->matchindex;
-    return 0;
-}
 
 enum _vlserver_type {
     vltype_unknown = 0,
@@ -252,39 +111,25 @@ static enum _vlserver_type newvlserver = vltype_unknown;
 int
 VLDB_CreateEntry(struct nvldbentry *entryp)
 {
-    int code;
-    struct uvldbentry uentry;
-
-    nvlentry_to_uvlentry(entryp, &uentry);
-    code = VLDB_CreateEntryU(&uentry);
-    if (!code)
-        code = uvlentry_to_nvlentry(&uentry, entryp);
-    return code;
-}
-
-int
-VLDB_CreateEntryU(struct uvldbentry *entryp)
-{
-    struct nvldbentry nentry;
+    struct vldbentry oentry;
     int code;
 
     if (newvlserver == vltype_old) {
-        struct vldbentry oentry;
       tryold:
-       code = uvlentry_to_ovlentry(entryp, &oentry);
+       code = nvlentry_to_ovlentry(entryp, &oentry);
        if (code)
            return code;
        code = ubik_VL_CreateEntry(cstruct, 0, &oentry);
        return code;
     }
-
-    code = uvlentry_to_nvlentry(entryp, &nentry);
-    if (code)
-        return code;
-    code = ubik_VL_CreateEntryN(cstruct, 0, &nentry);
-    if (code == RXGEN_OPCODE) {
-        newvlserver = vltype_old;
-        goto tryold;
+    code = ubik_VL_CreateEntryN(cstruct, 0, entryp);
+    if (newvlserver == vltype_unknown) {
+       if (code == RXGEN_OPCODE) {
+           newvlserver = vltype_old;   /* Doesn't support new interface */
+           goto tryold;
+       } else if (!code) {
+           newvlserver = vltype_new;
+       }
     }
     return code;
 }
@@ -292,128 +137,80 @@ VLDB_CreateEntryU(struct uvldbentry *entryp)
 int
 VLDB_GetEntryByID(afs_uint32 volid, afs_int32 voltype, struct nvldbentry *entryp)
 {
-    int code;
-    struct uvldbentry uentry;
-
-    nvlentry_to_uvlentry(entryp, &uentry);
-    code = VLDB_GetEntryByIDU(volid, voltype, &uentry);
-    if (!code)
-        code = uvlentry_to_nvlentry(&uentry, entryp);
-    return code;
-}
-
-int
-VLDB_GetEntryByIDU(afs_uint32 volid, afs_int32 voltype, struct uvldbentry *entryp)
-{
-    struct nvldbentry nentry;
+    struct vldbentry oentry;
     int code;
 
     if (newvlserver == vltype_old) {
-        struct vldbentry oentry;
       tryold:
        code =
            ubik_VL_GetEntryByID(cstruct, 0, volid, voltype, &oentry);
        if (!code)
-           ovlentry_to_uvlentry(&oentry, entryp);
+           ovlentry_to_nvlentry(&oentry, entryp);
        return code;
     }
-
-    code = ubik_VL_GetEntryByIDN(cstruct, 0, volid, voltype, &nentry);
-    if (code == RXGEN_OPCODE) {
-        newvlserver = vltype_old;
-        goto tryold;
+    code = ubik_VL_GetEntryByIDN(cstruct, 0, volid, voltype, entryp);
+    if (newvlserver == vltype_unknown) {
+       if (code == RXGEN_OPCODE) {
+           newvlserver = vltype_old;   /* Doesn't support new interface */
+           goto tryold;
+       } else if (!code) {
+           newvlserver = vltype_new;
+       }
     }
-    if (!code)
-        nvlentry_to_uvlentry(&nentry, entryp);
     return code;
 }
 
 int
 VLDB_GetEntryByName(char *namep, struct nvldbentry *entryp)
 {
-    int code;
-    struct uvldbentry uentry;
-
-    nvlentry_to_uvlentry(entryp, &uentry);
-    code = VLDB_GetEntryByNameU(namep, &uentry);
-    if (!code)
-        code = uvlentry_to_nvlentry(&uentry, entryp);
-    return code;
-}
-
-int
-VLDB_GetEntryByNameU(char *namep, struct uvldbentry *entryp)
-{
+    struct vldbentry oentry;
     int code;
 
     if (newvlserver == vltype_old) {
-        struct vldbentry oentry;
       tryold:
        code = ubik_VL_GetEntryByNameO(cstruct, 0, namep, &oentry);
        if (!code)
-           ovlentry_to_uvlentry(&oentry, entryp);
+           ovlentry_to_nvlentry(&oentry, entryp);
        return code;
     }
-    if (newvlserver == vltype_new) {
-        struct nvldbentry nentry;
-      trynew:
-        code = ubik_VL_GetEntryByNameN(cstruct, 0, namep, &nentry);
-        if (code == RXGEN_OPCODE) {
-            newvlserver = vltype_old;
-            goto tryold;
-        }
-       if (!code)
-           nvlentry_to_uvlentry(&nentry, entryp);
-    }
-    code = ubik_VL_GetEntryByNameU(cstruct, 0, namep, entryp);
+    code = ubik_VL_GetEntryByNameN(cstruct, 0, namep, entryp);
     if (newvlserver == vltype_unknown) {
        if (code == RXGEN_OPCODE) {
-           newvlserver = vltype_new;
-           goto trynew;
+           newvlserver = vltype_old;   /* Doesn't support new interface */
+           goto tryold;
        } else if (!code) {
-           newvlserver = vltype_uuid;
+           newvlserver = vltype_new;
        }
     }
     return code;
 }
 
 int
-VLDB_ReplaceEntry(afs_uint32 volid, afs_int32 voltype, struct nvldbentry *nentryp, afs_int32 releasetype)
+VLDB_ReplaceEntry(afs_uint32 volid, afs_int32 voltype, struct nvldbentry *entryp, afs_int32 releasetype)
 {
-    struct uvldbentry uentry;
+    struct vldbentry oentry;
     int code;
 
-    nvlentry_to_uvlentry(nentryp, &uentry);
-    code = VLDB_ReplaceEntryU(volid, voltype, &uentry, releasetype);
-    if (!code)
-        code = uvlentry_to_nvlentry(&uentry, nentryp);
-
-    return code;
-}
-
-int
-VLDB_ReplaceEntryU(afs_uint32 volid, afs_int32 voltype, struct uvldbentry *entryp, afs_int32 releasetype)
-{
-    int code;
-    struct nvldbentry nentry;
-
     if (newvlserver == vltype_old) {
-        struct vldbentry oentry;
       tryold:
-       code = uvlentry_to_ovlentry(entryp, &oentry);
+       code = nvlentry_to_ovlentry(entryp, &oentry);
        if (code)
            return code;
        code =
-           ubik_VL_ReplaceEntry(cstruct, 0, volid, voltype, &oentry, releasetype);
+           ubik_VL_ReplaceEntry(cstruct, 0, volid, voltype, &oentry,
+                     releasetype);
        return code;
     }
-    code = uvlentry_to_nvlentry(entryp, &nentry);
-    if (code)
-        return code;
-    code = ubik_VL_ReplaceEntryN(cstruct, 0, volid, voltype, &nentry, releasetype);
-    if (code == RXGEN_OPCODE) {
-        newvlserver = vltype_old;
-            goto tryold;
+    code =
+       ubik_VL_ReplaceEntryN(cstruct, 0, volid, voltype, entryp,
+                 releasetype);
+    if (newvlserver == vltype_unknown) {
+       if (code == RXGEN_OPCODE) {
+           newvlserver = vltype_old;   /* Doesn't support new interface */
+           goto tryold;
+       } else if (!code) {
+           newvlserver = vltype_new;
+       }
     }
     return code;
 }
@@ -435,47 +232,15 @@ convertBulkToNBulk(bulkentries *bulk, nbulkentries *nbulk) {
     }
 }
 
-static void
-convertBulkToUBulk(bulkentries *bulk, ubulkentries *ubulk) {
-    int i;
-
-    if (bulk->bulkentries_len == 0)
-       return;
-
-    ubulk->ubulkentries_len = bulk->bulkentries_len;
-    ubulk->ubulkentries_val =
-       xdr_alloc(bulk->bulkentries_len * sizeof(struct uvldbentry));
-    for (i = 0; i < bulk->bulkentries_len; i++) {
-       ovlentry_to_uvlentry(&bulk->bulkentries_val[i],
-                            &ubulk->ubulkentries_val[i]);
-    }
-}
-
-static void
-convertNBulkToUBulk(nbulkentries *nbulk, ubulkentries *ubulk) {
-    int i;
-
-    if (nbulk->nbulkentries_len == 0)
-       return;
-
-    ubulk->ubulkentries_len = nbulk->nbulkentries_len;
-    ubulk->ubulkentries_val =
-        xdr_alloc(nbulk->nbulkentries_len * sizeof(struct uvldbentry));
-    for (i = 0; i < nbulk->nbulkentries_len; i++) {    /* process each entry */
-        nvlentry_to_uvlentry(&nbulk->nbulkentries_val[i],
-                             &ubulk->ubulkentries_val[i]);
-    }
-}
-
 int
 VLDB_ListAttributes(VldbListByAttributes *attrp,
                     afs_int32 *entriesp,
                     nbulkentries *blkentriesp)
 {
+    bulkentries arrayEntries;
     int code;
 
     if (newvlserver == vltype_old) {
-        bulkentries arrayEntries;
       tryold:
        memset(&arrayEntries, 0, sizeof(arrayEntries));
        code =
@@ -496,17 +261,17 @@ VLDB_ListAttributes(VldbListByAttributes *attrp,
        xdr_free((xdrproc_t) xdr_bulkentries, &arrayEntries);
        return code;
     }
-
     code =
         ubik_VL_ListAttributesN(cstruct, 0, attrp, entriesp, blkentriesp);
-    if (code == RXGEN_OPCODE) {
-        newvlserver = vltype_old;      /* Doesn't support new interface */
-        goto tryold;
+    if (newvlserver == vltype_unknown) {
+       if (code == RXGEN_OPCODE) {
+           newvlserver = vltype_old;   /* Doesn't support new interface */
+           goto tryold;
+       } else if (!code) {
+           newvlserver = vltype_new;
+       }
     }
 
-    if (code)
-       return code;
-
     /* Ensure the number of entries claimed matches the no. returned */
     if (*entriesp < 0)
        *entriesp = 0;
@@ -517,58 +282,6 @@ VLDB_ListAttributes(VldbListByAttributes *attrp,
 }
 
 int
-VLDB_ListAttributesU(VldbListByAttributes *attrp,
-                    afs_int32 *entriesp,
-                    ubulkentries *blkentriesp)
-{
-    nbulkentries narrayEntries;
-    int code;
-
-    if (newvlserver == vltype_old) {
-        bulkentries arrayEntries;
-      tryold:
-       memset(&arrayEntries, 0, sizeof(arrayEntries));
-       code =
-           ubik_VL_ListAttributes(cstruct, 0, attrp, entriesp,
-                     &arrayEntries);
-       if (code)
-           return code;
-
-       /* Ensure the number of entries claimed matches the no. returned */
-       if (*entriesp < 0)
-           *entriesp = 0;
-       if (*entriesp > arrayEntries.bulkentries_len)
-           *entriesp = arrayEntries.bulkentries_len;
-
-       convertBulkToUBulk(&arrayEntries, blkentriesp);
-
-       xdr_free((xdrproc_t) xdr_bulkentries, &arrayEntries);
-       return code;
-    }
-
-    memset(&narrayEntries, 0, sizeof(narrayEntries));
-    code =
-        ubik_VL_ListAttributesN(cstruct, 0, attrp, entriesp, &narrayEntries);
-    if (code == RXGEN_OPCODE) {
-        newvlserver = vltype_old;      /* Doesn't support new interface */
-        goto tryold;
-    }
-    if (code)
-       return code;
-
-    /* Ensure the number of entries claimed matches the no. returned */
-    if (*entriesp < 0)
-       *entriesp = 0;
-    if (*entriesp > narrayEntries.nbulkentries_len)
-       *entriesp = narrayEntries.nbulkentries_len;
-
-    convertNBulkToUBulk(&narrayEntries, blkentriesp);
-
-    xdr_free((xdrproc_t) xdr_bulkentries, &narrayEntries);
-    return code;
-}
-
-int
 VLDB_ListAttributesN2(VldbListByAttributes *attrp,
                       char *name,
                       afs_int32 thisindex,
@@ -594,41 +307,6 @@ VLDB_ListAttributesN2(VldbListByAttributes *attrp,
     return code;
 }
 
-int
-VLDB_ListAttributesN2U(VldbListByAttributes *attrp,
-                      char *name,
-                      afs_int32 thisindex,
-                      afs_int32 *nentriesp,
-                      ubulkentries *blkentriesp,
-                      afs_int32 *nextindexp)
-{
-    afs_int32 code = RXGEN_OPCODE;
-
-    if (newvlserver != vltype_old) {
-        nbulkentries narrayEntries;
-
-       memset(&narrayEntries, 0, sizeof(narrayEntries));       /*initialize to hint the stub  to alloc space */
-        code =
-            ubik_VL_ListAttributesN2(cstruct, 0, attrp, (name ? name : ""),
-                                     thisindex, nentriesp, &narrayEntries, nextindexp);
-        if (code)
-           return code;
-
-       /* Ensure the number of entries claimed matches the no. returned */
-       if (*nentriesp < 0)
-           *nentriesp = 0;
-       if (*nentriesp > narrayEntries.nbulkentries_len)
-           *nentriesp = narrayEntries.nbulkentries_len;
-
-       convertNBulkToUBulk(&narrayEntries, blkentriesp);
-
-       xdr_free((xdrproc_t) xdr_bulkentries, &narrayEntries);
-       return code;
-    }
-    return code;
-}
-
-
 struct cacheips {
     afs_uint32 server;
     afs_int32 count;
@@ -649,8 +327,8 @@ VLDB_IsSameAddrs(afs_uint32 serv1, afs_uint32 serv2, afs_int32 *errorp)
     int code;
     ListAddrByAttributes attrs;
     bulkaddrs addrs;
-    afs_uint32 *addrp, f1, f2;
-    afs_int32 i, j, unique, nentries;
+    afs_uint32 *addrp, i, j, f1, f2;
+    afs_int32 unique, nentries;
     afsUUID uuid;
     static int initcache = 0;
 
@@ -658,8 +336,7 @@ VLDB_IsSameAddrs(afs_uint32 serv1, afs_uint32 serv2, afs_int32 *errorp)
 
     if (serv1 == serv2)
        return 1;
-    if (newvlserver == vltype_old ||
-        newvlserver == vltype_new) {
+    if (newvlserver == vltype_old || newvlserver == vltype_new) {
        return 0;
     }
     if (!initcache) {
@@ -697,7 +374,6 @@ VLDB_IsSameAddrs(afs_uint32 serv1, afs_uint32 serv2, afs_int32 *errorp)
                  &addrs);
     if (newvlserver == vltype_unknown) {
        if (code == RXGEN_OPCODE) {
-           newvlserver = vltype_new;
            return 0;
        } else if (!code) {
            newvlserver = vltype_uuid;
@@ -767,7 +443,7 @@ int
 vsu_ExtractName(char rname[], char name[])
 {
     char sname[VOLSER_OLDMAXVOLNAME + 1];
-    size_t total;
+    int total;
 
     strncpy(sname, name, sizeof(sname));
     sname[sizeof(sname) - 1] = '\0';
@@ -794,9 +470,9 @@ afs_uint32
 vsu_GetVolumeID(char *astring, struct ubik_client *acstruct, afs_int32 *errp)
 {
     char volname[VOLSER_OLDMAXVOLNAME + 1];
-    struct uvldbentry entry;
+    struct nvldbentry entry;
     afs_int32 vcode = 0;
-    size_t total;
+    int total;
 
     *errp = 0;
 
@@ -811,7 +487,7 @@ vsu_GetVolumeID(char *astring, struct ubik_client *acstruct, afs_int32 *errp)
     /* It was not a volume number but something else */
     total = strlen(astring);
     vsu_ExtractName(volname, astring);
-    vcode = VLDB_GetEntryByNameU(volname, &entry);
+    vcode = VLDB_GetEntryByName(volname, &entry);
     if (!vcode) {
       if ((total >= 9) && (!strcmp(&astring[total - 9], ".readonly")))
        return entry.volumeId[ROVOL];