libafs: remove linux conditionals for md5 inode number calculation
[openafs.git] / src / afs / afs_util.c
index 4f572e2..2879613 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
@@ -15,8 +15,6 @@
 #include <afsconfig.h>
 #include "afs/param.h"
 
-RCSID
-    ("$Header$");
 
 #include "afs/stds.h"
 #include "afs/sysincludes.h"   /* Standard vendor system headers */
@@ -30,22 +28,14 @@ RCSID
 #ifdef AFS_SGI62_ENV
 #include "h/hashing.h"
 #endif
-#if !defined(AFS_HPUX110_ENV) && !defined(AFS_LINUX20_ENV) && !defined(AFS_DARWIN60_ENV)
+#if !defined(AFS_HPUX110_ENV) && !defined(AFS_LINUX20_ENV) && !defined(AFS_DARWIN_ENV)
 #include <netinet/in_var.h>
 #endif /* ! AFS_HPUX110_ENV */
 #endif /* !defined(UKERNEL) */
 
 #include "afsincludes.h"       /* Afs-based standard headers */
 #include "afs/afs_stats.h"     /* afs statistics */
-
-#if    defined(AFS_SUN56_ENV)
-#include <inet/led.h>
-#include <inet/common.h>
-#if     defined(AFS_SUN58_ENV)
-#include <netinet/ip6.h>
-#endif
-#include <inet/ip.h>
-#endif
+#include "hcrypto/md5.h"
 
 #if    defined(AFS_AIX_ENV)
 #include <sys/fp_io.h>
@@ -57,8 +47,8 @@ afs_int32 afs_new_inum = 0;
 char *
 afs_cv2string(char *ttp, afs_uint32 aval)
 {
-    register char *tp = ttp;
-    register int i;
+    char *tp = ttp;
+    int i;
     int any;
 
     AFS_STATCNT(afs_cv2string);
@@ -77,9 +67,37 @@ afs_cv2string(char *ttp, afs_uint32 aval)
 }                              /*afs_cv2string */
 #endif
 
+/* not a generic strtoul replacement. for vol/vno/uniq, portable */
+
+afs_int32
+afs_strtoi_r(const char *str, char **endptr, afs_uint32 *ret)
+{
+    char *x;
+
+    *ret = 0;
+    *endptr = (char *)str;
+
+    if (!str)
+       return ERANGE;
+
+    for (x = (char *)str; *x >= '0' && *x <= '9'; x++) {
+       /* Check for impending overflow */
+       if (*ret > 429496729) { /* ULONG_MAX/10 */
+           *ret = 0;
+           *endptr = (char *)str;
+           return EINVAL;
+       }
+
+       *ret = (*ret * 10) + (*x - '0');
+    }
+
+    *endptr = x;
+    return 0;
+}
+
 #ifndef afs_strcasecmp
 int
-afs_strcasecmp(char *s1, char *s2)
+afs_strcasecmp(const char *s1, const char *s2)
 {
     while (*s1 && *s2) {
        char c1, c2;
@@ -136,6 +154,19 @@ afs_strchr(char *s, int c)
     return NULL;
 }
 #endif
+#ifndef afs_strrchr
+char *
+afs_strrchr(char *s, int c)
+{
+    char *p = NULL;
+
+    do {
+       if (*s == c)
+           p = (char*) s;
+    } while (*s++);
+    return p;
+}
+#endif
 
 char *
 afs_strdup(char *s)
@@ -144,7 +175,7 @@ afs_strdup(char *s)
     int cc;
 
     cc = strlen(s) + 1;
-    n = (char *)afs_osi_Alloc(cc);
+    n = afs_osi_Alloc(cc);
     if (n)
        memcpy(n, s, cc);
 
@@ -153,9 +184,9 @@ afs_strdup(char *s)
 
 void
 print_internet_address(char *preamble, struct srvAddr *sa, char *postamble,
-                      int flag)
+                      int flag, int code, struct rx_connection *rxconn)
 {
-    register struct server *aserver = sa->server;
+    struct server *aserver = sa->server;
     char *ptr = "\n";
     afs_uint32 address;
 
@@ -173,146 +204,86 @@ print_internet_address(char *preamble, struct srvAddr *sa, char *postamble,
                " (multi-homed address; other same-host interfaces may still be down)\n";
        }
     }
-    afs_warn("%s%d.%d.%d.%d in cell %s%s%s", preamble, (address >> 24),
+    afs_warnall("%s%d.%d.%d.%d in cell %s%s (code %d)%s", preamble, (address >> 24),
             (address >> 16) & 0xff, (address >> 8) & 0xff, (address) & 0xff,
-            aserver->cell->cellName, postamble, ptr);
-    afs_warnuser("%s%d.%d.%d.%d in cell %s%s%s", preamble, (address >> 24),
-                (address >> 16) & 0xff, (address >> 8) & 0xff,
-                (address) & 0xff, aserver->cell->cellName, postamble, ptr);
-
-}                              /*print_internet_address */
-
-
-
-/* * * * * * *
- * this code badly needs to be cleaned up...  too many ugly ifdefs.
- * XXX
- */
-#if 0
-void
-afs_warn(char *a, long b, long c, long d, long e, long f, long g, long h,
-        long i, long j)
-#else
-void
-afs_warn(a, b, c, d, e, f, g, h, i, j)
-     char *a;
-#if defined( AFS_USE_VOID_PTR)
-     void *b, *c, *d, *e, *f, *g, *h, *i, *j;
-#else
-     long b, c, d, e, f, g, h, i, j;
-#endif
-#endif
-{
-    AFS_STATCNT(afs_warn);
-
-    if (afs_showflags & GAGCONSOLE) {
-#if defined(AFS_AIX_ENV)
-       struct file *fd;
-
-       /* cf. console_printf() in oncplus/kernext/nfs/serv/shared.c */
-       if (fp_open
-           ("/dev/console", O_WRONLY | O_NOCTTY | O_NDELAY, 0666, 0, FP_SYS,
-            &fd) == 0) {
-           char buf[1024];
-           ssize_t len;
-           ssize_t count;
-
-           sprintf(buf, a, b, c, d, e, f, g, h, i, j);
-           len = strlen(buf);
-           fp_write(fd, buf, len, 0, UIO_SYSSPACE, &count);
-           fp_close(fd);
+            aserver->cell->cellName, postamble, code, ptr);
+
+    if (flag == 1 && rxconn) {
+       /* server was marked down, check Rx to see if this was possibly due to
+        * an ICMP error received from the network */
+       int errorigin, errtype, errcode;
+       const char *errmsg;
+       if (rx_GetNetworkError(rxconn, &errorigin, &errtype, &errcode, &errmsg) == 0) {
+           const char *str1 = " (";
+           const char *str2 = ")";
+           if (!errmsg) {
+               errmsg = str1 = str2 = "";
+           }
+           afs_warnall("afs: network error for %d.%d.%d.%d:%d: origin %d type %d code %d%s%s%s\n",
+                    (address >> 24),
+                    (address >> 16) & 0xff,
+                    (address >> 8) & 0xff,
+                    (address) & 0xff,
+                    (int)ntohs(sa->sa_portal),
+                    errorigin, errtype, errcode, str1, errmsg, str2);
        }
-#else
-       printf(a, b, c, d, e, f, g, h, i, j);
-#endif
     }
-}
+}                              /*print_internet_address */
 
-#if 0
-void
-afs_warnuser(char *a, long b, long c, long d, long e, long f, long g, long h,
-            long i, long j)
-#else
-void
-afs_warnuser(a, b, c, d, e, f, g, h, i, j)
-     char *a;
-     long b, c, d, e, f, g, h, i, j;
-#endif
-{
-    AFS_STATCNT(afs_warnuser);
-    if (afs_showflags & GAGUSER) {
-#ifdef AFS_GLOBAL_SUNLOCK
-       int haveGlock = ISAFS_GLOCK();
-       if (haveGlock)
-           AFS_GUNLOCK();
-#endif /* AFS_GLOBAL_SUNLOCK */
-
-       uprintf(a, b, c, d, e, f, g, h, i, j);
-
-#ifdef AFS_GLOBAL_SUNLOCK
-       if (haveGlock)
-           AFS_GLOCK();
-#endif /* AFS_GLOBAL_SUNLOCK */
-    }
-}
 
 
 /* run everywhere, checking locks */
 void
 afs_CheckLocks(void)
 {
-    register int i;
+    int i;
 
     afs_warn("Looking for locked data structures.\n");
-    afs_warn("conn %lx, volume %lx, user %lx, cell %lx, server %lx\n", &afs_xconn,
+    afs_warn("conn %p, volume %p, user %p, cell %p, server %p\n", &afs_xconn,
             &afs_xvolume, &afs_xuser, &afs_xcell, &afs_xserver);
     {
-       register struct vcache *tvc;
+       struct vcache *tvc;
        AFS_STATCNT(afs_CheckLocks);
 
        for (i = 0; i < VCSIZE; i++) {
            for (tvc = afs_vhashT[i]; tvc; tvc = tvc->hnext) {
-                if (tvc->states & CVInit) continue;
-#ifdef AFS_OSF_ENV
-               if (VREFCOUNT(tvc) > 1)
-#else /* AFS_OSF_ENV */
+                if (tvc->f.states & CVInit) continue;
 #ifdef AFS_DARWIN80_ENV
                if (vnode_isinuse(AFSTOV(tvc), 0))
 #else
                if (VREFCOUNT(tvc))
 #endif
-#endif
-                   afs_warn("Stat cache entry at %x is held\n", tvc);
+                   afs_warn("Stat cache entry at %p is held\n", tvc);
                if (CheckLock(&tvc->lock))
-                   afs_warn("Stat entry at %x is locked\n", tvc);
+                   afs_warn("Stat entry at %p is locked\n", tvc);
            }
        }
     }
     {
-       register struct dcache *tdc;
+       struct dcache *tdc;
        for (i = 0; i < afs_cacheFiles; i++) {
            tdc = afs_indexTable[i];
            if (tdc) {
                if (tdc->refCount)
-                   afs_warn("Disk entry %d at %x is held\n", i, tdc);
+                   afs_warn("Disk entry %d at %p is held\n", i, tdc);
            }
            if (afs_indexFlags[i] & IFDataMod)
-               afs_warn("Disk entry %d at %x has IFDataMod flag set.\n", i,
+               afs_warn("Disk entry %d at %p has IFDataMod flag set.\n", i,
                         tdc);
        }
     }
     {
        struct srvAddr *sa;
        struct server *ts;
-       struct conn *tc;
+        struct sa_conn_vector *tcv;
        for (i = 0; i < NSERVERS; i++) {
            for (ts = afs_servers[i]; ts; ts = ts->next) {
                if (ts->flags & SRVR_ISDOWN)
-                   printf("Server entry %lx is marked down\n", (unsigned long)ts);
+                   afs_warn("Server entry %p is marked down\n", ts);
                for (sa = ts->addr; sa; sa = sa->next_sa) {
-                   for (tc = sa->conns; tc; tc = tc->next) {
-                       if (tc->refCount)
-                           afs_warn("conn at %x (server %x) is held\n", tc,
+                    for (tcv = sa->conns; tcv; tcv = tcv->next) {
+                        if (tcv->refCount)
+                            afs_warn("conn at %p (server %x) is held\n", tcv,
                                     sa->sa_ip);
                    }
                }
@@ -324,9 +295,9 @@ afs_CheckLocks(void)
        for (i = 0; i < NVOLS; i++) {
            for (tv = afs_volumes[i]; tv; tv = tv->next) {
                if (CheckLock(&tv->lock))
-                   afs_warn("volume at %x is locked\n", tv);
+                   afs_warn("volume at %p is locked\n", tv);
                if (tv->refCount)
-                   afs_warn("volume at %x is held\n", tv);
+                   afs_warn("volume at %p is held\n", tv);
            }
        }
     }
@@ -335,8 +306,10 @@ afs_CheckLocks(void)
 
        for (i = 0; i < NUSERS; i++) {
            for (tu = afs_users[i]; tu; tu = tu->next) {
+               if (CheckLock(&tu->lock))
+                   afs_warn("user at %p is locked\n", tu);
                if (tu->refCount)
-                   printf("user at %lx is held\n", (unsigned long)tu);
+                   afs_warn("user at %lx is held\n", (unsigned long)tu);
            }
        }
     }
@@ -348,11 +321,7 @@ int
 afs_noop(void)
 {
     AFS_STATCNT(afs_noop);
-#ifdef AFS_OSF30_ENV
-    return (EOPNOTSUPP);
-#else
     return EINVAL;
-#endif
 }
 
 int
@@ -405,273 +374,48 @@ afs_data_pointer_to_int32(const void *p)
     return ip.i32[i32_sub];
 }
 
-#ifdef AFS_LINUX20_ENV
-
-struct afs_md5 {
-    unsigned int sz[2];
-    afs_int32 counter[4];
-    unsigned char save[64];
-};
-
-static void AFS_MD5_Init (struct afs_md5 *m);
-static void AFS_MD5_Update (struct afs_md5 *m, const void *p, size_t len);
-static void AFS_MD5_Final (void *res, struct afs_md5 *m); /* u_int32 res[4] */
-
-#define A m->counter[0]
-#define B m->counter[1]
-#define C m->counter[2]
-#define D m->counter[3]
-#define X data
-
-static void
-AFS_MD5_Init (struct afs_md5 *m)
-{
-    m->sz[0] = 0;
-    m->sz[1] = 0;
-    D = 0x10325476;
-    C = 0x98badcfe;
-    B = 0xefcdab89;
-    A = 0x67452301;
-}
-
-#define F(x,y,z) ((x & y) | (~x & z))
-#define G(x,y,z) ((x & z) | (y & ~z))
-#define H(x,y,z) (x ^ y ^ z)
-#define I(x,y,z) (y ^ (x | ~z))
-
-static inline afs_uint32
-cshift (afs_uint32 x, unsigned int n)
-{
-    return ((x << n) | (x >> (32 - n)));
-}
-
-#define DOIT(a,b,c,d,k,s,i,OP) \
-a = b + cshift(a + OP(b,c,d) + X[k] + (i), s)
-
-#define DO1(a,b,c,d,k,s,i) DOIT(a,b,c,d,k,s,i,F)
-#define DO2(a,b,c,d,k,s,i) DOIT(a,b,c,d,k,s,i,G)
-#define DO3(a,b,c,d,k,s,i) DOIT(a,b,c,d,k,s,i,H)
-#define DO4(a,b,c,d,k,s,i) DOIT(a,b,c,d,k,s,i,I)
-
-static inline void
-calc (struct afs_md5 *m, afs_uint32 *data)
-{
-    afs_uint32 AA, BB, CC, DD;
-    
-    AA = A;
-    BB = B;
-    CC = C;
-    DD = D;
-    
-    /* Round 1 */
-    
-    DO1(A,B,C,D,0,7,0xd76aa478);
-    DO1(D,A,B,C,1,12,0xe8c7b756);
-    DO1(C,D,A,B,2,17,0x242070db);
-    DO1(B,C,D,A,3,22,0xc1bdceee);
-    
-    DO1(A,B,C,D,4,7,0xf57c0faf);
-    DO1(D,A,B,C,5,12,0x4787c62a);
-    DO1(C,D,A,B,6,17,0xa8304613);
-    DO1(B,C,D,A,7,22,0xfd469501);
-    
-    DO1(A,B,C,D,8,7,0x698098d8);
-    DO1(D,A,B,C,9,12,0x8b44f7af);
-    DO1(C,D,A,B,10,17,0xffff5bb1);
-    DO1(B,C,D,A,11,22,0x895cd7be);
-    
-    DO1(A,B,C,D,12,7,0x6b901122);
-    DO1(D,A,B,C,13,12,0xfd987193);
-    DO1(C,D,A,B,14,17,0xa679438e);
-    DO1(B,C,D,A,15,22,0x49b40821);
-    
-    /* Round 2 */
-    
-    DO2(A,B,C,D,1,5,0xf61e2562);
-    DO2(D,A,B,C,6,9,0xc040b340);
-    DO2(C,D,A,B,11,14,0x265e5a51);
-    DO2(B,C,D,A,0,20,0xe9b6c7aa);
-    
-    DO2(A,B,C,D,5,5,0xd62f105d);
-    DO2(D,A,B,C,10,9,0x2441453);
-    DO2(C,D,A,B,15,14,0xd8a1e681);
-    DO2(B,C,D,A,4,20,0xe7d3fbc8);
-    
-    DO2(A,B,C,D,9,5,0x21e1cde6);
-    DO2(D,A,B,C,14,9,0xc33707d6);
-    DO2(C,D,A,B,3,14,0xf4d50d87);
-    DO2(B,C,D,A,8,20,0x455a14ed);
-    
-    DO2(A,B,C,D,13,5,0xa9e3e905);
-    DO2(D,A,B,C,2,9,0xfcefa3f8);
-    DO2(C,D,A,B,7,14,0x676f02d9);
-    DO2(B,C,D,A,12,20,0x8d2a4c8a);
-    
-    /* Round 3 */
-    
-    DO3(A,B,C,D,5,4,0xfffa3942);
-    DO3(D,A,B,C,8,11,0x8771f681);
-    DO3(C,D,A,B,11,16,0x6d9d6122);
-    DO3(B,C,D,A,14,23,0xfde5380c);
-    
-    DO3(A,B,C,D,1,4,0xa4beea44);
-    DO3(D,A,B,C,4,11,0x4bdecfa9);
-    DO3(C,D,A,B,7,16,0xf6bb4b60);
-    DO3(B,C,D,A,10,23,0xbebfbc70);
-    
-    DO3(A,B,C,D,13,4,0x289b7ec6);
-    DO3(D,A,B,C,0,11,0xeaa127fa);
-    DO3(C,D,A,B,3,16,0xd4ef3085);
-    DO3(B,C,D,A,6,23,0x4881d05);
-    
-    DO3(A,B,C,D,9,4,0xd9d4d039);
-    DO3(D,A,B,C,12,11,0xe6db99e5);
-    DO3(C,D,A,B,15,16,0x1fa27cf8);
-    DO3(B,C,D,A,2,23,0xc4ac5665);
-    
-    /* Round 4 */
-    
-    DO4(A,B,C,D,0,6,0xf4292244);
-    DO4(D,A,B,C,7,10,0x432aff97);
-    DO4(C,D,A,B,14,15,0xab9423a7);
-    DO4(B,C,D,A,5,21,0xfc93a039);
-    
-    DO4(A,B,C,D,12,6,0x655b59c3);
-    DO4(D,A,B,C,3,10,0x8f0ccc92);
-    DO4(C,D,A,B,10,15,0xffeff47d);
-    DO4(B,C,D,A,1,21,0x85845dd1);
-    
-    DO4(A,B,C,D,8,6,0x6fa87e4f);
-    DO4(D,A,B,C,15,10,0xfe2ce6e0);
-    DO4(C,D,A,B,6,15,0xa3014314);
-    DO4(B,C,D,A,13,21,0x4e0811a1);
-    
-    DO4(A,B,C,D,4,6,0xf7537e82);
-    DO4(D,A,B,C,11,10,0xbd3af235);
-    DO4(C,D,A,B,2,15,0x2ad7d2bb);
-    DO4(B,C,D,A,9,21,0xeb86d391);
-    
-    A += AA;
-    B += BB;
-    C += CC;
-    D += DD;
-}
-
-/*
- * From `Performance analysis of MD5' by Joseph D. Touch <touch@isi.edu>
- */
-
-#if defined(WORDS_BIGENDIAN)
-static inline afs_uint32
-swap_u_int32_t (afs_uint32 t)
+static_inline afs_int32
+afs_calc_inum_md5(afs_int32 cell, afs_int32 volume, afs_int32 vnode)
 {
-    afs_uint32 temp1, temp2;
-    
-    temp1   = cshift(t, 16);
-    temp2   = temp1 >> 8;
-    temp1  &= 0x00ff00ff;
-    temp2  &= 0x00ff00ff;
-    temp1 <<= 8;
-    return temp1 | temp2;
-}
-#endif
-
-struct x32{
-    unsigned int a:32;
-    unsigned int b:32;
-};
+    afs_int32 ino = 0, vno = vnode;
+    char digest[16];
+    struct md5 ct;
 
-static void
-AFS_MD5_Update (struct afs_md5 *m, const void *v, size_t len)
-{
-    const unsigned char *p = v;
-    size_t old_sz = m->sz[0];
-    size_t offset;
-    
-    m->sz[0] += len * 8;
-    if (m->sz[0] < old_sz)
-       ++m->sz[1];
-    offset = (old_sz / 8)  % 64;
-    while(len > 0){
-       size_t l = MIN(len, 64 - offset);
-       memcpy(m->save + offset, p, l);
-       offset += l;
-       p += l;
-       len -= l;
-       if(offset == 64){
-#if defined(WORDS_BIGENDIAN)
-           int i;
-           afs_uint32 temp[16];
-           struct x32 *us = (struct x32*)m->save;
-           for(i = 0; i < 8; i++){
-               temp[2*i+0] = swap_u_int32_t(us[i].a);
-               temp[2*i+1] = swap_u_int32_t(us[i].b);
-           }
-           calc(m, temp);
-#else
-           calc(m, (afs_uint32*)m->save);
-#endif
-           offset = 0;
+    if (afs_new_inum) {
+       int offset;
+       MD5_Init(&ct);
+       MD5_Update(&ct, &cell, 4);
+       MD5_Update(&ct, &volume, 4);
+       MD5_Update(&ct, &vnode, 4);
+       MD5_Final(digest, &ct);
+
+       /* Userspace may react oddly to an inode number of 0 or 1, so keep
+        * reading more of the md5 digest if we get back one of those.
+        * Make sure not to read beyond the end of the digest; if we somehow
+        * still have a 0, we will fall through to the non-md5 calculation. */
+       for (offset = 0;
+            (ino == 0 || ino == 1) &&
+             offset + sizeof(ino) <= sizeof(digest);
+            offset++) {
+
+           memcpy(&ino, &digest[offset], sizeof(ino));
+           ino ^= (ino ^ vno) & 1;
+           ino &= 0x7fffffff;      /* Assumes 32 bit ino_t ..... */
        }
     }
+    return ino;
 }
 
-static void
-AFS_MD5_Final (void *res, struct afs_md5 *m)
+afs_int32
+afs_calc_inum(afs_int32 cell, afs_int32 volume, afs_int32 vnode)
 {
-    unsigned char zeros[72];
-    unsigned offset = (m->sz[0] / 8) % 64;
-    unsigned int dstart = (120 - offset - 1) % 64 + 1;
-    
-    *zeros = 0x80;
-    memset (zeros + 1, 0, sizeof(zeros) - 1);
-    zeros[dstart+0] = (m->sz[0] >> 0) & 0xff;
-    zeros[dstart+1] = (m->sz[0] >> 8) & 0xff;
-    zeros[dstart+2] = (m->sz[0] >> 16) & 0xff;
-    zeros[dstart+3] = (m->sz[0] >> 24) & 0xff;
-    zeros[dstart+4] = (m->sz[1] >> 0) & 0xff;
-    zeros[dstart+5] = (m->sz[1] >> 8) & 0xff;
-    zeros[dstart+6] = (m->sz[1] >> 16) & 0xff;
-    zeros[dstart+7] = (m->sz[1] >> 24) & 0xff;
-    AFS_MD5_Update (m, zeros, dstart + 8);
-    {
-       int i;
-       unsigned char *r = (unsigned char *)res;
-       
-       for (i = 0; i < 4; ++i) {
-           r[4*i]   = m->counter[i] & 0xFF;
-           r[4*i+1] = (m->counter[i] >> 8) & 0xFF;
-           r[4*i+2] = (m->counter[i] >> 16) & 0xFF;
-           r[4*i+3] = (m->counter[i] >> 24) & 0xFF;
-       }
-    }
-}
-
-afs_int32 afs_calc_inum (afs_int32 volume, afs_int32 vnode)
-{ 
     afs_int32 ino;
-    char digest[16];
-    struct afs_md5 ct;
-    
-    if (afs_new_inum) {
-       AFS_MD5_Init(&ct);
-       AFS_MD5_Update(&ct, &volume, 4);
-       AFS_MD5_Update(&ct, &vnode, 4);
-       AFS_MD5_Final(digest, &ct);
-       memcpy(&ino, digest, sizeof(ino_t));
-       ino ^= (ino ^ vnode) & 1;
-    } else {
+
+    ino = afs_calc_inum_md5(cell, volume, vnode);
+
+    if (ino == 0 || ino == 1) {
        ino = (volume << 16) + vnode;
-       ino &= 0x7fffffff;      /* Assumes 32 bit ino_t ..... */
     }
+    ino &= 0x7fffffff;      /* Assumes 32 bit ino_t ..... */
     return ino;
 }
-
-#else
-
-afs_int32 afs_calc_inum (afs_int32 volume, afs_int32 vnode)
-{
-    return (volume << 16) + vnode;
-}
-
-#endif