unpin-out-of-order-oops-20050125
[openafs.git] / src / afs / afs_call.c
index 00648ef..9266fd7 100644 (file)
@@ -8,25 +8,26 @@
  */
 
 #include <afsconfig.h>
-#include "../afs/param.h"
+#include "afs/param.h"
 
-RCSID("$Header$");
+RCSID
+    ("$Header$");
 
-#include "../afs/sysincludes.h"        /* Standard vendor system headers */
-#include "../afs/afsincludes.h"        /* Afs-based standard headers */
-#include "../afs/afs_stats.h"
-#include "../rx/rx_globals.h"
+#include "afs/sysincludes.h"   /* Standard vendor system headers */
+#include "afsincludes.h"       /* Afs-based standard headers */
+#include "afs/afs_stats.h"
+#include "rx/rx_globals.h"
 #if !defined(UKERNEL) && !defined(AFS_LINUX20_ENV)
 #include "net/if.h"
 #ifdef AFS_SGI62_ENV
-#include "../h/hashing.h"
+#include "h/hashing.h"
 #endif
 #if !defined(AFS_HPUX110_ENV) && !defined(AFS_DARWIN60_ENV)
 #include "netinet/in_var.h"
 #endif
 #endif /* !defined(UKERNEL) */
 #ifdef AFS_LINUX22_ENV
-#include "../h/smp_lock.h"
+#include "h/smp_lock.h"
 #endif
 
 
@@ -42,11 +43,12 @@ struct afsop_cell {
 };
 
 char afs_zeros[AFS_ZEROS];
-char afs_rootVolumeName[64]="";
-struct afs_icl_set *afs_iclSetp = (struct afs_icl_set*)0;
-struct afs_icl_set *afs_iclLongTermSetp = (struct afs_icl_set*)0;
+char afs_rootVolumeName[64] = "";
+struct afs_icl_set *afs_iclSetp = (struct afs_icl_set *)0;
+struct afs_icl_set *afs_iclLongTermSetp = (struct afs_icl_set *)0;
+afs_uint32 rx_bindhost;
 
-#if defined(AFS_SUN5_ENV) || defined(AFS_SGI_ENV) 
+#if defined(AFS_SUN5_ENV) || defined(AFS_SGI_ENV)
 kmutex_t afs_global_lock;
 kmutex_t afs_rxglobal_lock;
 #endif
@@ -63,10 +65,13 @@ simple_lock_data_t afs_global_lock;
 struct lock__bsd__ afs_global_lock;
 #endif
 
-#if defined(AFS_FBSD_ENV)
+#if defined(AFS_XBSD_ENV) && !defined(AFS_FBSD50_ENV)
 struct lock afs_global_lock;
 struct proc *afs_global_owner;
 #endif
+#ifdef AFS_FBSD50_ENV
+struct mtx afs_global_mtx;
+#endif
 
 #if defined(AFS_OSF_ENV) || defined(AFS_DARWIN_ENV)
 thread_t afs_global_owner;
@@ -89,8 +94,11 @@ extern struct interfaceAddr afs_cb_interface;
 static int afs_RX_Running = 0;
 static int afs_InitSetup_done = 0;
 
+afs_int32 afs_rx_deadtime = AFS_RXDEADTIME;
+afs_int32 afs_rx_harddead = AFS_HARDDEADTIME;
+
 static int
-Afscall_icl(long opcode, long p1, long p2, long p3, long p4, long *retval);
+  Afscall_icl(long opcode, long p1, long p2, long p3, long p4, long *retval);
 
 #if defined(AFS_HPUX_ENV)
 extern int afs_vfs_mount();
@@ -99,13 +107,14 @@ extern int afs_vfs_mount();
 /* This is code which needs to be called once when the first daemon enters
  * the client. A non-zero return means an error and AFS should not start.
  */
-static int afs_InitSetup(int preallocs)
+static int
+afs_InitSetup(int preallocs)
 {
     extern void afs_InitStats();
     int code;
 
     if (afs_InitSetup_done)
-       return;
+       return EAGAIN;
 
 #ifndef AFS_NOSTATS
     /*
@@ -115,17 +124,17 @@ static int afs_InitSetup(int preallocs)
      */
     afs_InitStats();
 #endif /* AFS_NOSTATS */
-    
+
     memset(afs_zeros, 0, AFS_ZEROS);
 
     /* start RX */
     rx_extraPackets = AFS_NRXPACKETS;  /* smaller # of packets */
-    code = rx_Init(htons(7001));
+    code = rx_InitHost(rx_bindhost, htons(7001));
     if (code) {
-       printf("AFS: RX failed to initialize.\n");
+       printf("AFS: RX failed to initialize %d).\n", code);
        return code;
     }
-    rx_SetRxDeadTime(AFS_RXDEADTIME);
+    rx_SetRxDeadTime(afs_rx_deadtime);
     /* resource init creates the services */
     afs_ResourceInit(preallocs);
 
@@ -135,67 +144,267 @@ static int afs_InitSetup(int preallocs)
     return code;
 }
 
+#if defined(AFS_LINUX24_ENV) && defined(COMPLETION_H_EXISTS)
+struct afsd_thread_info {
+    unsigned long parm;
+    struct completion *complete;
+};
+
+static int
+afsd_thread(void *rock)
+{
+    struct afsd_thread_info *arg = rock;
+    unsigned long parm = arg->parm;
+#ifdef SYS_SETPRIORITY_EXPORTED
+    int (*sys_setpriority) (int, int, int) = sys_call_table[__NR_setpriority];
+#endif
+#if defined(AFS_LINUX26_ENV)
+    daemonize("afsd");
+#else
+    daemonize();
+#endif
+                               /* doesn't do much, since we were forked from keventd, but
+                                * does call mm_release, which wakes up our parent (since it
+                                * used CLONE_VFORK) */
+#if !defined(AFS_LINUX26_ENV)
+    reparent_to_init();
+#endif
+    afs_osi_MaskSignals();
+    switch (parm) {
+    case AFSOP_START_RXCALLBACK:
+       sprintf(current->comm, "afs_cbstart");
+       AFS_GLOCK();
+       complete(arg->complete);
+       afs_CB_Running = 1;
+       while (afs_RX_Running != 2)
+           afs_osi_Sleep(&afs_RX_Running);
+       sprintf(current->comm, "afs_callback");
+       afs_RXCallBackServer();
+       AFS_GUNLOCK();
+       complete_and_exit(0, 0);
+       break;
+    case AFSOP_START_AFS:
+       sprintf(current->comm, "afs_afsstart");
+       AFS_GLOCK();
+       complete(arg->complete);
+       AFS_Running = 1;
+       while (afs_initState < AFSOP_START_AFS)
+           afs_osi_Sleep(&afs_initState);
+       afs_initState = AFSOP_START_BKG;
+       afs_osi_Wakeup(&afs_initState);
+       sprintf(current->comm, "afsd");
+       afs_Daemon();
+       AFS_GUNLOCK();
+       complete_and_exit(0, 0);
+       break;
+    case AFSOP_START_BKG:
+       sprintf(current->comm, "afs_bkgstart");
+       AFS_GLOCK();
+       complete(arg->complete);
+       while (afs_initState < AFSOP_START_BKG)
+           afs_osi_Sleep(&afs_initState);
+       if (afs_initState < AFSOP_GO) {
+           afs_initState = AFSOP_GO;
+           afs_osi_Wakeup(&afs_initState);
+       }
+       sprintf(current->comm, "afs_background");
+       afs_BackgroundDaemon();
+       AFS_GUNLOCK();
+       complete_and_exit(0, 0);
+       break;
+    case AFSOP_START_TRUNCDAEMON:
+       sprintf(current->comm, "afs_trimstart");
+       AFS_GLOCK();
+       complete(arg->complete);
+       while (afs_initState < AFSOP_GO)
+           afs_osi_Sleep(&afs_initState);
+       sprintf(current->comm, "afs_cachetrim");
+       afs_CacheTruncateDaemon();
+       AFS_GUNLOCK();
+       complete_and_exit(0, 0);
+       break;
+    case AFSOP_START_CS:
+       sprintf(current->comm, "afs_checkserver");
+       AFS_GLOCK();
+       complete(arg->complete);
+       afs_CheckServerDaemon();
+       AFS_GUNLOCK();
+       complete_and_exit(0, 0);
+       break;
+    case AFSOP_RXEVENT_DAEMON:
+       sprintf(current->comm, "afs_evtstart");
+#ifdef SYS_SETPRIORITY_EXPORTED
+       sys_setpriority(PRIO_PROCESS, 0, -10);
+#else
+#ifdef CURRENT_INCLUDES_NICE
+       current->nice = -10;
+#endif
+#endif
+       AFS_GLOCK();
+       complete(arg->complete);
+       while (afs_initState < AFSOP_START_BKG)
+           afs_osi_Sleep(&afs_initState);
+       sprintf(current->comm, "afs_rxevent");
+       afs_rxevent_daemon();
+       AFS_GUNLOCK();
+       complete_and_exit(0, 0);
+       break;
+    case AFSOP_RXLISTENER_DAEMON:
+       sprintf(current->comm, "afs_lsnstart");
+#ifdef SYS_SETPRIORITY_EXPORTED
+       sys_setpriority(PRIO_PROCESS, 0, -10);
+#else
+#ifdef CURRENT_INCLUDES_NICE
+       current->nice = -10;
+#endif
+#endif
+       AFS_GLOCK();
+       complete(arg->complete);
+       afs_initState = AFSOP_START_AFS;
+       afs_osi_Wakeup(&afs_initState);
+       afs_RX_Running = 2;
+       afs_osi_Wakeup(&afs_RX_Running);
+       afs_osi_RxkRegister();
+       sprintf(current->comm, "afs_rxlistener");
+       rxk_Listener();
+       AFS_GUNLOCK();
+       complete_and_exit(0, 0);
+       break;
+    default:
+       printf("Unknown op %ld in StartDaemon()\n", (long)parm);
+       break;
+    }
+    return 0;
+}
+
+void
+afsd_launcher(void *rock)
+{
+    if (!kernel_thread(afsd_thread, rock, CLONE_VFORK | SIGCHLD))
+       printf("kernel_thread failed. afs startup will not complete\n");
+}
+
+void
+afs_DaemonOp(long parm, long parm2, long parm3, long parm4, long parm5,
+            long parm6)
+{
+    int code;
+    DECLARE_COMPLETION(c);
+#if defined(AFS_LINUX26_ENV)
+    struct work_struct tq;
+#else
+    struct tq_struct tq;
+#endif
+    struct afsd_thread_info info;
+    if (parm == AFSOP_START_RXCALLBACK) {
+       if (afs_CB_Running)
+           return;
+    } else if (parm == AFSOP_RXLISTENER_DAEMON) {
+       if (afs_RX_Running)
+           return;
+       afs_RX_Running = 1;
+       code = afs_InitSetup(parm2);
+       if (parm3) {
+           rx_enablePeerRPCStats();
+       }
+       if (parm4) {
+           rx_enableProcessRPCStats();
+       }
+       if (code)
+           return;
+    } else if (parm == AFSOP_START_AFS) {
+       if (AFS_Running)
+           return;
+    }                          /* other functions don't need setup in the parent */
+    info.complete = &c;
+    info.parm = parm;
+#if defined(AFS_LINUX26_ENV)
+    INIT_WORK(&tq, afsd_launcher, &info);
+    schedule_work(&tq);
+#else
+    tq.sync = 0;
+    INIT_LIST_HEAD(&tq.list);
+    tq.routine = afsd_launcher;
+    tq.data = &info;
+    schedule_task(&tq);
+#endif
+    AFS_GUNLOCK();
+    /* we need to wait cause we passed stack pointers around.... */
+    wait_for_completion(&c);
+    AFS_GLOCK();
+}
+#endif
+
 /* leaving as is, probably will barf if we add prototypes here since it's likely being called
 with partial list */
+int
 afs_syscall_call(parm, parm2, parm3, parm4, parm5, parm6)
-long parm, parm2, parm3, parm4, parm5, parm6;
+     long parm, parm2, parm3, parm4, parm5, parm6;
 {
     afs_int32 code = 0;
-#if defined(AFS_SGI61_ENV) || defined(AFS_SUN57_ENV) || defined(AFS_DARWIN_ENV) || defined(AFS_FBSD_ENV)
-    size_t bufferSize; 
+#if defined(AFS_SGI61_ENV) || defined(AFS_SUN57_ENV) || defined(AFS_DARWIN_ENV) || defined(AFS_XBSD_ENV)
+    size_t bufferSize;
 #else /* AFS_SGI61_ENV */
-    u_int bufferSize;  
+    u_int bufferSize;
 #endif /* AFS_SGI61_ENV */
 
     AFS_STATCNT(afs_syscall_call);
 #ifdef AFS_SUN5_ENV
-    if (!afs_suser(CRED()) && (parm != AFSOP_GETMTU) 
+    if (!afs_suser(CRED()) && (parm != AFSOP_GETMTU)
        && (parm != AFSOP_GETMASK)) {
-      /* only root can run this code */
+       /* only root can run this code */
        return (EACCES);
 #else
-    if (!afs_suser() && (parm != AFSOP_GETMTU)
+    if (!afs_suser(NULL) && (parm != AFSOP_GETMTU)
        && (parm != AFSOP_GETMASK)) {
-      /* only root can run this code */
-#if defined(KERNEL_HAVE_SETUERROR)
+       /* only root can run this code */
+#if defined(KERNEL_HAVE_UERROR)
        setuerror(EACCES);
-       return(EACCES);
+       return (EACCES);
 #else
-#if    defined(AFS_OSF_ENV)
+#if defined(AFS_OSF_ENV)
        return EACCES;
-#else  /* AFS_OSF_ENV */
+#else /* AFS_OSF_ENV */
        return EPERM;
-#endif
+#endif /* AFS_OSF_ENV */
 #endif
 #endif
     }
     AFS_GLOCK();
+#if defined(AFS_LINUX24_ENV) && defined(COMPLETION_H_EXISTS) && !defined(UKERNEL)
+    if (parm < AFSOP_ADDCELL || parm == AFSOP_RXEVENT_DAEMON
+       || parm == AFSOP_RXLISTENER_DAEMON) {
+       afs_DaemonOp(parm, parm2, parm3, parm4, parm5, parm6);
+    }
+#else /* !(AFS_LINUX24_ENV && !UKERNEL) */
     if (parm == AFSOP_START_RXCALLBACK) {
-       if (afs_CB_Running) goto out;
+       if (afs_CB_Running)
+           goto out;
        afs_CB_Running = 1;
 #ifndef RXK_LISTENER_ENV
        code = afs_InitSetup(parm2);
-       if (!code) 
-#endif /* RXK_LISTENER_ENV */
-           {
+       if (!code)
+#endif /* !RXK_LISTENER_ENV */
+       {
 #ifdef RXK_LISTENER_ENV
-               while (afs_RX_Running != 2)
-                   afs_osi_Sleep(&afs_RX_Running);
-#else
-               afs_initState = AFSOP_START_AFS;
-               afs_osi_Wakeup(&afs_initState);
+           while (afs_RX_Running != 2)
+               afs_osi_Sleep(&afs_RX_Running);
+#else /* !RXK_LISTENER_ENV */
+           afs_initState = AFSOP_START_AFS;
+           afs_osi_Wakeup(&afs_initState);
 #endif /* RXK_LISTENER_ENV */
-               afs_osi_Invisible();
-               afs_RXCallBackServer();
-           }
-#ifdef AFS_SGI_ENV
+           afs_osi_Invisible();
+           afs_RXCallBackServer();
+       }
+#ifdef AFS_SGI_ENV
        AFS_GUNLOCK();
        exit(CLD_EXITED, code);
-#endif
+#endif /* AFS_SGI_ENV */
     }
 #ifdef RXK_LISTENER_ENV
     else if (parm == AFSOP_RXLISTENER_DAEMON) {
-       if (afs_RX_Running) goto out;
+       if (afs_RX_Running)
+           goto out;
        afs_RX_Running = 1;
        code = afs_InitSetup(parm2);
        if (parm3) {
@@ -212,39 +421,21 @@ long parm, parm2, parm3, parm4, parm5, parm6;
            afs_osi_Wakeup(&afs_RX_Running);
 #ifndef UKERNEL
            afs_osi_RxkRegister();
-#endif
+#endif /* !UKERNEL */
            rxk_Listener();
        }
 #ifdef AFS_SGI_ENV
        AFS_GUNLOCK();
        exit(CLD_EXITED, code);
-#endif
-    }
-#endif
-    else if (parm == AFSOP_BASIC_INIT) {
-       afs_int32 temp;
-
-       while (!afs_InitSetup_done)
-           afs_osi_Sleep(&afs_InitSetup_done);
-
-#if defined(AFS_SUN_ENV) || defined(AFS_SGI_ENV) || defined(AFS_HPUX_ENV) || defined(AFS_LINUX20_ENV) || defined(AFS_DARWIN_ENV) || defined(AFS_FBSD_ENV)
-       temp = AFS_MINBUFFERS;  /* Should fix this soon */
-#else
-       /* number of 2k buffers we could get from all of the buffer space */
-       temp = ((afs_bufferpages * NBPG)>>11);
-       temp = temp>>2; /* don't take more than 25% (our magic parameter) */
-       if (temp < AFS_MINBUFFERS)
-           temp = AFS_MINBUFFERS; /* though we really should have this many */
-#endif
-       DInit(temp);
-       afs_rootFid.Fid.Volume = 0;
-       code = 0;
+#endif /* AFS_SGI_ENV */
     }
+#endif /* RXK_LISTENER_ENV */
     else if (parm == AFSOP_START_AFS) {
        /* afs daemon */
-       if (AFS_Running) goto out;
+       if (AFS_Running)
+           goto out;
        AFS_Running = 1;
-       while (afs_initState < AFSOP_START_AFS) 
+       while (afs_initState < AFSOP_START_AFS)
            afs_osi_Sleep(&afs_initState);
 
        afs_initState = AFSOP_START_BKG;
@@ -254,18 +445,16 @@ long parm, parm2, parm3, parm4, parm5, parm6;
 #ifdef AFS_SGI_ENV
        AFS_GUNLOCK();
        exit(CLD_EXITED, 0);
-#endif
-    }
-    else if (parm == AFSOP_START_CS) {
+#endif /* AFS_SGI_ENV */
+    } else if (parm == AFSOP_START_CS) {
        afs_osi_Invisible();
        afs_CheckServerDaemon();
 #ifdef AFS_SGI_ENV
        AFS_GUNLOCK();
        exit(CLD_EXITED, 0);
-#endif
-    }
-    else if (parm == AFSOP_START_BKG) {
-       while (afs_initState < AFSOP_START_BKG) 
+#endif /* AFS_SGI_ENV */
+    } else if (parm == AFSOP_START_BKG) {
+       while (afs_initState < AFSOP_START_BKG)
            afs_osi_Sleep(&afs_initState);
        if (afs_initState < AFSOP_GO) {
            afs_initState = AFSOP_GO;
@@ -273,19 +462,18 @@ long parm, parm2, parm3, parm4, parm5, parm6;
        }
        /* start the bkg daemon */
        afs_osi_Invisible();
-#ifdef AFS_AIX32_ENV
+#ifdef AFS_AIX32_ENV
        if (parm2)
            afs_BioDaemon(parm2);
        else
-#endif
+#endif /* AFS_AIX32_ENV */
            afs_BackgroundDaemon();
 #ifdef AFS_SGI_ENV
        AFS_GUNLOCK();
        exit(CLD_EXITED, 0);
-#endif
-    }
-    else if (parm == AFSOP_START_TRUNCDAEMON) {
-       while (afs_initState < AFSOP_GO) 
+#endif /* AFS_SGI_ENV */
+    } else if (parm == AFSOP_START_TRUNCDAEMON) {
+       while (afs_initState < AFSOP_GO)
            afs_osi_Sleep(&afs_initState);
        /* start the bkg daemon */
        afs_osi_Invisible();
@@ -293,65 +481,94 @@ long parm, parm2, parm3, parm4, parm5, parm6;
 #ifdef AFS_SGI_ENV
        AFS_GUNLOCK();
        exit(CLD_EXITED, 0);
-#endif
+#endif /* AFS_SGI_ENV */
     }
 #if defined(AFS_SUN5_ENV) || defined(RXK_LISTENER_ENV)
     else if (parm == AFSOP_RXEVENT_DAEMON) {
-       while (afs_initState < AFSOP_START_BKG) afs_osi_Sleep(&afs_initState);
+       while (afs_initState < AFSOP_START_BKG)
+           afs_osi_Sleep(&afs_initState);
        afs_osi_Invisible();
        afs_rxevent_daemon();
 #ifdef AFS_SGI_ENV
        AFS_GUNLOCK();
        exit(CLD_EXITED, 0);
-#endif
+#endif /* AFS_SGI_ENV */
     }
-#endif 
-    else if (parm == AFSOP_ADDCELL) {
+#endif /* AFS_SUN5_ENV || RXK_LISTENER_ENV */
+#endif /* AFS_LINUX24_ENV && !UKERNEL */
+    else if (parm == AFSOP_BASIC_INIT) {
+       afs_int32 temp;
+
+       while (!afs_InitSetup_done)
+           afs_osi_Sleep(&afs_InitSetup_done);
+
+#if defined(AFS_SUN_ENV) || defined(AFS_SGI_ENV) || defined(AFS_HPUX_ENV) || defined(AFS_LINUX20_ENV) || defined(AFS_DARWIN_ENV) || defined(AFS_XBSD_ENV)
+       temp = AFS_MINBUFFERS;  /* Should fix this soon */
+#else
+       /* number of 2k buffers we could get from all of the buffer space */
+       temp = ((afs_bufferpages * NBPG) >> 11);
+       temp = temp >> 2;       /* don't take more than 25% (our magic parameter) */
+       if (temp < AFS_MINBUFFERS)
+           temp = AFS_MINBUFFERS;      /* though we really should have this many */
+#endif
+       DInit(temp);
+       afs_rootFid.Fid.Volume = 0;
+       code = 0;
+    } else if (parm == AFSOP_ADDCELL) {
        /* add a cell.  Parameter 2 is 8 hosts (in net order),  parm 3 is the null-terminated
-        name.  Parameter 4 is the length of the name, including the null.  Parm 5 is the
-        home cell flag (0x1 bit) and the nosuid flag (0x2 bit) */
-       struct afsop_cell tcell;
+        * name.  Parameter 4 is the length of the name, including the null.  Parm 5 is the
+        * home cell flag (0x1 bit) and the nosuid flag (0x2 bit) */
+       struct afsop_cell *tcell = afs_osi_Alloc(sizeof(struct afsop_cell));
 
-       AFS_COPYIN((char *)parm2, (char *)tcell.hosts, sizeof(tcell.hosts), code);
+       AFS_COPYIN((char *)parm2, (char *)tcell->hosts, sizeof(tcell->hosts),
+                  code);
        if (!code) {
-           if (parm4 > sizeof(tcell.cellName)) 
+           if (parm4 > sizeof(tcell->cellName))
                code = EFAULT;
            else {
-               AFS_COPYIN((char *)parm3, tcell.cellName, parm4, code);
-               if (!code) 
-                   afs_NewCell(tcell.cellName, tcell.hosts, parm5,
-                               NULL, 0, 0, 0);
+               AFS_COPYIN((char *)parm3, tcell->cellName, parm4, code);
+               if (!code)
+                   afs_NewCell(tcell->cellName, tcell->hosts, parm5, NULL, 0,
+                               0, 0);
            }
        }
+       afs_osi_Free(tcell, sizeof(struct afsop_cell));
     } else if (parm == AFSOP_ADDCELL2) {
-       struct afsop_cell tcell;
+       struct afsop_cell *tcell = afs_osi_Alloc(sizeof(struct afsop_cell));
        char *tbuffer = osi_AllocSmallSpace(AFS_SMALLOCSIZ), *lcnamep = 0;
-       char *tbuffer1 = osi_AllocSmallSpace(AFS_SMALLOCSIZ), *cnamep = 0;
+       char *tbuffer1 = osi_AllocSmallSpace(AFS_SMALLOCSIZ);
        int cflags = parm4;
 
-       /* wait for basic init */
-       while (afs_initState < AFSOP_START_BKG) afs_osi_Sleep(&afs_initState);
+#if 0
+       /* wait for basic init - XXX can't find any reason we need this? */
+       while (afs_initState < AFSOP_START_BKG)
+           afs_osi_Sleep(&afs_initState);
+#endif
 
-       AFS_COPYIN((char *)parm2, (char *)tcell.hosts, sizeof(tcell.hosts), code);
+       AFS_COPYIN((char *)parm2, (char *)tcell->hosts, sizeof(tcell->hosts),
+                  code);
        if (!code) {
-           AFS_COPYINSTR((char *)parm3, tbuffer1, AFS_SMALLOCSIZ, &bufferSize, code);
+           AFS_COPYINSTR((char *)parm3, tbuffer1, AFS_SMALLOCSIZ,
+                         &bufferSize, code);
            if (!code) {
                if (parm4 & 4) {
-                   AFS_COPYINSTR((char *)parm5, tbuffer, AFS_SMALLOCSIZ, &bufferSize, code);
+                   AFS_COPYINSTR((char *)parm5, tbuffer, AFS_SMALLOCSIZ,
+                                 &bufferSize, code);
                    if (!code) {
                        lcnamep = tbuffer;
                        cflags |= CLinkedCell;
                    }
                }
                if (!code)
-                   code = afs_NewCell(tbuffer1, tcell.hosts, cflags,
-                                      lcnamep, 0, 0, 0);
+                   code =
+                       afs_NewCell(tbuffer1, tcell->hosts, cflags, lcnamep,
+                                   0, 0, 0);
            }
        }
+       afs_osi_Free(tcell, sizeof(struct afsop_cell));
        osi_FreeSmallSpace(tbuffer);
        osi_FreeSmallSpace(tbuffer1);
-    }
-    else if (parm == AFSOP_ADDCELLALIAS) {
+    } else if (parm == AFSOP_ADDCELLALIAS) {
        /*
         * Call arguments:
         * parm2 is the alias name
@@ -360,100 +577,98 @@ long parm, parm2, parm3, parm4, parm5, parm6;
        char *aliasName = osi_AllocSmallSpace(AFS_SMALLOCSIZ);
        char *cellName = osi_AllocSmallSpace(AFS_SMALLOCSIZ);
 
-       AFS_COPYINSTR((char *)parm2, aliasName, AFS_SMALLOCSIZ, &bufferSize, code);
-       if (!code) AFS_COPYINSTR((char *)parm3, cellName, AFS_SMALLOCSIZ, &bufferSize, code);
-       if (!code) afs_NewCellAlias(aliasName, cellName);
+       AFS_COPYINSTR((char *)parm2, aliasName, AFS_SMALLOCSIZ, &bufferSize,
+                     code);
+       if (!code)
+           AFS_COPYINSTR((char *)parm3, cellName, AFS_SMALLOCSIZ,
+                         &bufferSize, code);
+       if (!code)
+           afs_NewCellAlias(aliasName, cellName);
        osi_FreeSmallSpace(aliasName);
        osi_FreeSmallSpace(cellName);
-    }
-    else if (parm == AFSOP_SET_THISCELL) {
+    } else if (parm == AFSOP_SET_THISCELL) {
        /*
         * Call arguments:
         * parm2 is the primary cell name
         */
        char *cell = osi_AllocSmallSpace(AFS_SMALLOCSIZ);
 
-       AFS_COPYINSTR((char *) parm2, cell, AFS_SMALLOCSIZ, &bufferSize, code);
+       AFS_COPYINSTR((char *)parm2, cell, AFS_SMALLOCSIZ, &bufferSize, code);
        if (!code)
            afs_SetPrimaryCell(cell);
        osi_FreeSmallSpace(cell);
-    }
-    else if (parm == AFSOP_CACHEINIT) {
+    } else if (parm == AFSOP_CACHEINIT) {
        struct afs_cacheParams cparms;
 
-       if (afs_CacheInit_Done) goto out;
+       if (afs_CacheInit_Done)
+           goto out;
 
-       AFS_COPYIN((char *)parm2, (caddr_t) &cparms, sizeof(cparms), code);
+       AFS_COPYIN((char *)parm2, (caddr_t) & cparms, sizeof(cparms), code);
        if (code) {
-#if defined(KERNEL_HAVE_SETUERROR)
+#if defined(KERNEL_HAVE_UERROR)
            setuerror(code);
            code = -1;
 #endif
            goto out;
        }
        afs_CacheInit_Done = 1;
-    {
-       struct afs_icl_log *logp;
-       /* initialize the ICL system */
-       code = afs_icl_CreateLog("cmfx", 60*1024, &logp);
-       if (code == 0)
-           code = afs_icl_CreateSetWithFlags("cm", logp,
-                                             (struct icl_log *) 0,
-                                             ICL_CRSET_FLAG_DEFAULT_OFF,
-                                             &afs_iclSetp);
-           code = afs_icl_CreateSet("cmlongterm", logp, (struct icl_log*) 0,
-                                &afs_iclLongTermSetp);
-    }
+       {
+           struct afs_icl_log *logp;
+           /* initialize the ICL system */
+           code = afs_icl_CreateLog("cmfx", 60 * 1024, &logp);
+           if (code == 0)
+               code =
+                   afs_icl_CreateSetWithFlags("cm", logp, NULL,
+                                              ICL_CRSET_FLAG_DEFAULT_OFF,
+                                              &afs_iclSetp);
+           code =
+               afs_icl_CreateSet("cmlongterm", logp, NULL,
+                                 &afs_iclLongTermSetp);
+       }
        afs_setTime = cparms.setTimeFlag;
 
-       code = afs_CacheInit(cparms.cacheScaches,
-                            cparms.cacheFiles,
-                            cparms.cacheBlocks,
-                            cparms.cacheDcaches,
-                            cparms.cacheVolumes,
-                            cparms.chunkSize,
-                            cparms.memCacheFlag,
-                            cparms.inodes,
-                            cparms.users);
+       code =
+           afs_CacheInit(cparms.cacheScaches, cparms.cacheFiles,
+                         cparms.cacheBlocks, cparms.cacheDcaches,
+                         cparms.cacheVolumes, cparms.chunkSize,
+                         cparms.memCacheFlag, cparms.inodes, cparms.users);
 
-    }
-    else if (parm == AFSOP_CACHEINODE) {
+    } else if (parm == AFSOP_CACHEINODE) {
        ino_t ainode = parm2;
        /* wait for basic init */
-       while (afs_initState < AFSOP_START_BKG) afs_osi_Sleep(&afs_initState);
+       while (afs_initState < AFSOP_START_BKG)
+           afs_osi_Sleep(&afs_initState);
 
        /* do it by inode */
 #ifdef AFS_SGI62_ENV
        ainode = (ainode << 32) | (parm3 & 0xffffffff);
 #endif
        code = afs_InitCacheFile(NULL, ainode);
-    }
-    else if (parm == AFSOP_ROOTVOLUME) {
+    } else if (parm == AFSOP_ROOTVOLUME) {
        /* wait for basic init */
-       while (afs_initState < AFSOP_START_BKG) afs_osi_Sleep(&afs_initState);
+       while (afs_initState < AFSOP_START_BKG)
+           afs_osi_Sleep(&afs_initState);
 
        if (parm2) {
-           AFS_COPYINSTR((char *)parm2, afs_rootVolumeName, sizeof(afs_rootVolumeName), &bufferSize, code);
-           afs_rootVolumeName[sizeof(afs_rootVolumeName)-1] = 0;
-       }
-       else code = 0;
-    }
-    else if (parm == AFSOP_CACHEFILE ||
-            parm == AFSOP_CACHEINFO ||
-            parm == AFSOP_VOLUMEINFO ||
-            parm == AFSOP_AFSLOG ||
-            parm == AFSOP_CELLINFO) {
+           AFS_COPYINSTR((char *)parm2, afs_rootVolumeName,
+                         sizeof(afs_rootVolumeName), &bufferSize, code);
+           afs_rootVolumeName[sizeof(afs_rootVolumeName) - 1] = 0;
+       } else
+           code = 0;
+    } else if (parm == AFSOP_CACHEFILE || parm == AFSOP_CACHEINFO
+              || parm == AFSOP_VOLUMEINFO || parm == AFSOP_AFSLOG
+              || parm == AFSOP_CELLINFO) {
        char *tbuffer = osi_AllocSmallSpace(AFS_SMALLOCSIZ);
 
        code = 0;
-       AFS_COPYINSTR((char *) parm2, tbuffer, AFS_SMALLOCSIZ,
-                     &bufferSize, code);
+       AFS_COPYINSTR((char *)parm2, tbuffer, AFS_SMALLOCSIZ, &bufferSize,
+                     code);
        if (code) {
            osi_FreeSmallSpace(tbuffer);
            goto out;
        }
        if (!code) {
-           tbuffer[AFS_SMALLOCSIZ-1] = '\0';   /* null-terminate the name */
+           tbuffer[AFS_SMALLOCSIZ - 1] = '\0'; /* null-terminate the name */
            /* We have the cache dir copied in.  Call the cache init routine */
            if (parm == AFSOP_CACHEFILE)
                code = afs_InitCacheFile(tbuffer, 0);
@@ -465,85 +680,107 @@ long parm, parm2, parm3, parm4, parm5, parm6;
                code = afs_InitCellInfo(tbuffer);
        }
        osi_FreeSmallSpace(tbuffer);
-    }
-    else if (parm == AFSOP_GO) {
+    } else if (parm == AFSOP_GO) {
        /* the generic initialization calls come here.  One parameter: should we do the
-             set-time operation on this workstation */
-       if (afs_Go_Done) goto out;
+        * set-time operation on this workstation */
+       if (afs_Go_Done)
+           goto out;
        afs_Go_Done = 1;
-       while (afs_initState < AFSOP_GO) afs_osi_Sleep(&afs_initState);
+       while (afs_initState < AFSOP_GO)
+           afs_osi_Sleep(&afs_initState);
        afs_initState = 101;
        afs_setTime = parm2;
        afs_osi_Wakeup(&afs_initState);
 #if    (!defined(AFS_NONFSTRANS) && !defined(AFS_DEC_ENV)) || defined(AFS_AIX_IAUTH_ENV)
        afs_nfsclient_init();
 #endif
-       printf("found %d non-empty cache files (%d%%).\n", afs_stats_cmperf.cacheFilesReused,
-              (100*afs_stats_cmperf.cacheFilesReused) /
-              (afs_stats_cmperf.cacheNumEntries?afs_stats_cmperf.cacheNumEntries : 1));
-    }
-    else if (parm == AFSOP_ADVISEADDR) {
+       printf("found %d non-empty cache files (%d%%).\n",
+              afs_stats_cmperf.cacheFilesReused,
+              (100 * afs_stats_cmperf.cacheFilesReused) /
+              (afs_stats_cmperf.cacheNumEntries ? afs_stats_cmperf.
+               cacheNumEntries : 1));
+    } else if (parm == AFSOP_ADVISEADDR) {
        /* pass in the host address to the rx package */
-       afs_int32       count        = parm2;
-       afs_int32       buffer[AFS_MAX_INTERFACE_ADDR];
-       afs_int32       maskbuffer[AFS_MAX_INTERFACE_ADDR];
-       afs_int32       mtubuffer[AFS_MAX_INTERFACE_ADDR];
-       int     i;
-       int     code;
-
-       if (  count > AFS_MAX_INTERFACE_ADDR ) {
-          code = ENOMEM;
-          count = AFS_MAX_INTERFACE_ADDR;
+       int rxbind = 0;
+       afs_int32 count = parm2;
+       afs_int32 *buffer =
+           afs_osi_Alloc(sizeof(afs_int32) * AFS_MAX_INTERFACE_ADDR);
+       afs_int32 *maskbuffer =
+           afs_osi_Alloc(sizeof(afs_int32) * AFS_MAX_INTERFACE_ADDR);
+       afs_int32 *mtubuffer =
+           afs_osi_Alloc(sizeof(afs_int32) * AFS_MAX_INTERFACE_ADDR);
+       int i;
+
+       /* Bind, but only if there's only one address configured */ 
+       if ( count & 0x80000000) {
+           count &= ~0x80000000;
+           if (count == 1)
+               rxbind=1;
+       }
+
+       if (count > AFS_MAX_INTERFACE_ADDR) {
+           code = ENOMEM;
+           count = AFS_MAX_INTERFACE_ADDR;
        }
-          
-       AFS_COPYIN( (char *)parm3, (char *)buffer, count*sizeof(afs_int32), code);
+
+       AFS_COPYIN((char *)parm3, (char *)buffer, count * sizeof(afs_int32),
+                  code);
        if (parm4)
-         AFS_COPYIN((char *)parm4, (char *)maskbuffer, count*sizeof(afs_int32), code);
+           AFS_COPYIN((char *)parm4, (char *)maskbuffer,
+                      count * sizeof(afs_int32), code);
        if (parm5)
-         AFS_COPYIN((char *)parm5, (char *)mtubuffer, count*sizeof(afs_int32), code);
+           AFS_COPYIN((char *)parm5, (char *)mtubuffer,
+                      count * sizeof(afs_int32), code);
 
        afs_cb_interface.numberOfInterfaces = count;
-       for (i=0; i < count ; i++) {
-          afs_cb_interface.addr_in[i] = buffer[i];
-#ifdef AFS_USERSPACE_IP_ADDR   
-          /* AFS_USERSPACE_IP_ADDR means we have no way of finding the
-           * machines IP addresses when in the kernel (the in_ifaddr
-           * struct is not available), so we pass the info in at
-           * startup. We also pass in the subnetmask and mtu size. The
-           * subnetmask is used when setting the rank:
-           * afsi_SetServerIPRank(); and the mtu size is used when
-           * finding the best mtu size. rxi_FindIfnet() is replaced
-           * with rxi_Findcbi().
-           */
-          afs_cb_interface.subnetmask[i] = (parm4 ? maskbuffer[i] : 0xffffffff);
-          afs_cb_interface.mtu[i]        = (parm5 ? mtubuffer[i]  : htonl(1500));
+       for (i = 0; i < count; i++) {
+           afs_cb_interface.addr_in[i] = buffer[i];
+#ifdef AFS_USERSPACE_IP_ADDR
+           /* AFS_USERSPACE_IP_ADDR means we have no way of finding the
+            * machines IP addresses when in the kernel (the in_ifaddr
+            * struct is not available), so we pass the info in at
+            * startup. We also pass in the subnetmask and mtu size. The
+            * subnetmask is used when setting the rank:
+            * afsi_SetServerIPRank(); and the mtu size is used when
+            * finding the best mtu size. rxi_FindIfnet() is replaced
+            * with rxi_Findcbi().
+            */
+           afs_cb_interface.subnetmask[i] =
+               (parm4 ? maskbuffer[i] : 0xffffffff);
+           afs_cb_interface.mtu[i] = (parm5 ? mtubuffer[i] : htonl(1500));
 #endif
        }
        afs_uuid_create(&afs_cb_interface.uuid);
        rxi_setaddr(buffer[0]);
-    }
+       if (rxbind)
+           rx_bindhost = buffer[0];
+       else
+           rx_bindhost = htonl(INADDR_ANY);
 
+       afs_osi_Free(buffer, sizeof(afs_int32) * AFS_MAX_INTERFACE_ADDR);
+       afs_osi_Free(maskbuffer, sizeof(afs_int32) * AFS_MAX_INTERFACE_ADDR);
+       afs_osi_Free(mtubuffer, sizeof(afs_int32) * AFS_MAX_INTERFACE_ADDR);
+    }
 #ifdef AFS_SGI53_ENV
     else if (parm == AFSOP_NFSSTATICADDR) {
-       extern int (*nfs_rfsdisptab_v2)();
+       extern int (*nfs_rfsdisptab_v2) ();
        nfs_rfsdisptab_v2 = (int (*)())parm2;
-    }
-    else if (parm == AFSOP_NFSSTATICADDR2) {
-       extern int (*nfs_rfsdisptab_v2)();
+    } else if (parm == AFSOP_NFSSTATICADDR2) {
+       extern int (*nfs_rfsdisptab_v2) ();
 #ifdef _K64U64
-       nfs_rfsdisptab_v2 = (int (*)())((parm2<<32) | (parm3 & 0xffffffff));
+       nfs_rfsdisptab_v2 = (int (*)())((parm2 << 32) | (parm3 & 0xffffffff));
 #else /* _K64U64 */
        nfs_rfsdisptab_v2 = (int (*)())(parm3 & 0xffffffff);
 #endif /* _K64U64 */
     }
 #if defined(AFS_SGI62_ENV) && !defined(AFS_SGI65_ENV)
     else if (parm == AFSOP_SBLOCKSTATICADDR2) {
-       extern int (*afs_sblockp)();
-       extern void (*afs_sbunlockp)();
+       extern int (*afs_sblockp) ();
+       extern void (*afs_sbunlockp) ();
 #ifdef _K64U64
-       afs_sblockp = (int (*)())((parm2<<32) | (parm3 & 0xffffffff));
-       afs_sbunlockp = (void (*)())((parm4<<32) | (parm5 & 0xffffffff));
-#else 
+       afs_sblockp = (int (*)())((parm2 << 32) | (parm3 & 0xffffffff));
+       afs_sbunlockp = (void (*)())((parm4 << 32) | (parm5 & 0xffffffff));
+#else
        afs_sblockp = (int (*)())(parm3 & 0xffffffff);
        afs_sbunlockp = (void (*)())(parm5 & 0xffffffff);
 #endif /* _K64U64 */
@@ -551,51 +788,45 @@ long parm, parm2, parm3, parm4, parm5, parm6;
 #endif /* AFS_SGI62_ENV && !AFS_SGI65_ENV */
 #endif /* AFS_SGI53_ENV */
     else if (parm == AFSOP_SHUTDOWN) {
-#if    defined(AFS_OSF_ENV) || defined(AFS_DARWIN_ENV) || defined(AFS_FBSD_ENV)
-       extern struct mount *afs_globalVFS;
-#else  /* AFS_OSF_ENV */
-       extern struct vfs *afs_globalVFS;
-#endif
        afs_cold_shutdown = 0;
-       if (parm == 1) afs_cold_shutdown = 1;
+       if (parm2 == 1)
+           afs_cold_shutdown = 1;
+#ifndef AFS_DARWIN_ENV
        if (afs_globalVFS != 0) {
            afs_warn("AFS isn't unmounted yet! Call aborted\n");
            code = EACCES;
        } else
+#endif
            afs_shutdown();
-    }
-    else if (parm == AFSOP_AFS_VFSMOUNT) {
+    } else if (parm == AFSOP_AFS_VFSMOUNT) {
 #ifdef AFS_HPUX_ENV
        vfsmount(parm2, parm3, parm4, parm5);
 #else /* defined(AFS_HPUX_ENV) */
-#if defined(KERNEL_HAVE_SETUERROR)
-      setuerror(EINVAL);
+#if defined(KERNEL_HAVE_UERROR)
+       setuerror(EINVAL);
 #else
-      code = EINVAL;
+       code = EINVAL;
 #endif
 #endif /* defined(AFS_HPUX_ENV) */
-    }
-    else if (parm == AFSOP_CLOSEWAIT) {
+    } else if (parm == AFSOP_CLOSEWAIT) {
        afs_SynchronousCloses = 'S';
-    }
-    else if (parm == AFSOP_GETMTU) { 
-      afs_uint32 mtu = 0;
+    } else if (parm == AFSOP_GETMTU) {
+       afs_uint32 mtu = 0;
 #if    !defined(AFS_SUN5_ENV) && !defined(AFS_LINUX20_ENV)
 #ifdef AFS_USERSPACE_IP_ADDR
-      afs_int32 i;
-      i = rxi_Findcbi(parm2);
-      mtu = ((i == -1) ? htonl(1500) : afs_cb_interface.mtu[i]);
+       afs_int32 i;
+       i = rxi_Findcbi(parm2);
+       mtu = ((i == -1) ? htonl(1500) : afs_cb_interface.mtu[i]);
 #else /* AFS_USERSPACE_IP_ADDR */
-      struct ifnet *tifnp;
-      struct in_ifaddr *tifadp = (struct in_ifaddr *) 0;
-      extern struct ifnet *rxi_FindIfnet();
-      tifnp = rxi_FindIfnet(parm2, &tifadp);  /*  make iterative */
-      mtu = (tifnp ? tifnp->if_mtu : htonl(1500));
+       struct ifnet *tifnp;
+
+       tifnp = rxi_FindIfnet(parm2, NULL);     /*  make iterative */
+       mtu = (tifnp ? tifnp->if_mtu : htonl(1500));
 #endif /* else AFS_USERSPACE_IP_ADDR */
 #endif /* !AFS_SUN5_ENV */
-      if (!code) 
-        AFS_COPYOUT ((caddr_t)&mtu, (caddr_t)parm3, sizeof(afs_int32), code);
+       if (!code)
+           AFS_COPYOUT((caddr_t) & mtu, (caddr_t) parm3, sizeof(afs_int32),
+                       code);
 #ifdef AFS_AIX32_ENV
 /* this is disabled for now because I can't figure out how to get access
  * to these kernel variables.  It's only for supporting user-mode rx
@@ -612,40 +843,28 @@ long parm, parm2, parm3, parm4, parm5, parm6;
        }
       } */
 #endif /* AFS_AIX32_ENV */
-    }
-    else if (parm == AFSOP_GETMASK) {  /* parm2 == addr in net order */
-      afs_uint32 mask = 0;
+    } else if (parm == AFSOP_GETMASK) {        /* parm2 == addr in net order */
+       afs_uint32 mask = 0;
 #if    !defined(AFS_SUN5_ENV)
 #ifdef AFS_USERSPACE_IP_ADDR
-      afs_int32 i;
-      i = rxi_Findcbi(parm2);
-      if (i != -1) {
-        mask = afs_cb_interface.subnetmask[i];
-      } else {
-        code = -1;
-      }
+       afs_int32 i;
+       i = rxi_Findcbi(parm2);
+       if (i != -1) {
+           mask = afs_cb_interface.subnetmask[i];
+       } else {
+           code = -1;
+       }
 #else /* AFS_USERSPACE_IP_ADDR */
-      struct ifnet *tifnp;
-#ifdef AFS_DARWIN60_ENV
-      struct ifaddr *tifadp = (struct ifaddr *) 0;
-#else
-      struct in_ifaddr *tifadp = (struct in_ifaddr *) 0;
-#endif
-      extern struct ifnet *rxi_FindIfnet();
-      tifnp = rxi_FindIfnet(parm2, &tifadp);  /* make iterative */
-      if (tifnp && tifadp) {
-#ifdef AFS_DARWIN60_ENV
-        mask = ((struct sockaddr_in *)tifadp->ifa_netmask)->sin_addr.s_addr;
-#else
-        mask = tifadp->ia_subnetmask;
-#endif
-      } else {
-        code = -1;
-      }
+       struct ifnet *tifnp;
+
+       tifnp = rxi_FindIfnet(parm2, &mask);    /* make iterative */
+       if (!tifnp)
+           code = -1;
 #endif /* else AFS_USERSPACE_IP_ADDR */
 #endif /* !AFS_SUN5_ENV */
-      if (!code) 
-        AFS_COPYOUT ((caddr_t)&mask, (caddr_t)parm3, sizeof(afs_int32), code);
+       if (!code)
+           AFS_COPYOUT((caddr_t) & mask, (caddr_t) parm3, sizeof(afs_int32),
+                       code);
     }
 #ifdef AFS_AFSDB_ENV
     else if (parm == AFSOP_AFSDB_HANDLER) {
@@ -656,39 +875,41 @@ long parm, parm2, parm3, parm4, parm5, parm6;
        char *cellname = afs_osi_Alloc(cellLen);
 
 #ifndef UKERNEL
-       afs_osi_MaskSignals();
+       afs_osi_MaskUserLoop();
 #endif
-       AFS_COPYIN((afs_int32 *)parm2, cellname, cellLen, code);
-       AFS_COPYIN((afs_int32 *)parm3, kmsg, kmsgLen, code);
+       AFS_COPYIN((afs_int32 *) parm2, cellname, cellLen, code);
+       AFS_COPYIN((afs_int32 *) parm3, kmsg, kmsgLen, code);
        if (!code) {
            code = afs_AFSDBHandler(cellname, cellLen, kmsg);
-           if (*cellname == 1) *cellname = 0;
+           if (*cellname == 1)
+               *cellname = 0;
            if (code == -2) {   /* Shutting down? */
                *cellname = 1;
                code = 0;
            }
        }
-       if (!code) AFS_COPYOUT(cellname, (char *)parm2, cellLen, code);
+       if (!code)
+           AFS_COPYOUT(cellname, (char *)parm2, cellLen, code);
        afs_osi_Free(kmsg, kmsgLen);
        afs_osi_Free(cellname, cellLen);
     }
 #endif
     else if (parm == AFSOP_SET_DYNROOT) {
        code = afs_SetDynrootEnable(parm2);
-    }
-    else if (parm == AFSOP_SET_FAKESTAT) {
+    } else if (parm == AFSOP_SET_FAKESTAT) {
        afs_fakestat_enable = parm2;
        code = 0;
-    }
-    else
-      code = EINVAL;
+    } else if (parm == AFSOP_SET_BACKUPTREE) {
+       afs_bkvolpref = parm2;
+    } else
+       code = EINVAL;
 
-out:
-  AFS_GUNLOCK();
+  out:
+    AFS_GUNLOCK();
 #ifdef AFS_LINUX20_ENV
-  return -code;
+    return -code;
 #else
-  return code;
+    return code;
 #endif
 }
 
@@ -703,76 +924,80 @@ out:
  *     THIS SHOULD BE CHANGED TO afs_syscall(), but requires
  *     all the user-level calls to `syscall' to change.
  */
-syscall(syscall, p1, p2, p3, p4, p5, p6) {
-       register rval1=0, code;
-       register monster;
-       int retval=0;
+syscall(syscall, p1, p2, p3, p4, p5, p6)
+{
+    register rval1 = 0, code;
+    register monster;
+    int retval = 0;
 #ifndef AFS_AIX41_ENV
-       extern lock_t kernel_lock;
-       monster = lockl(&kernel_lock, LOCK_SHORT);
+    extern lock_t kernel_lock;
+    monster = lockl(&kernel_lock, LOCK_SHORT);
 #endif /* !AFS_AIX41_ENV */
 
-       AFS_STATCNT(syscall);
-       setuerror(0);
-       switch (syscall) {
-           case AFSCALL_CALL:
-               rval1 = afs_syscall_call(p1, p2, p3, p4, p5, p6);
-               break;
+    AFS_STATCNT(syscall);
+    setuerror(0);
+    switch (syscall) {
+    case AFSCALL_CALL:
+       rval1 = afs_syscall_call(p1, p2, p3, p4, p5, p6);
+       break;
 
-           case AFSCALL_SETPAG:
-               AFS_GLOCK();
-               rval1 = afs_setpag();
-               AFS_GUNLOCK();
-               break;
+    case AFSCALL_SETPAG:
+       AFS_GLOCK();
+       rval1 = afs_setpag();
+       AFS_GUNLOCK();
+       break;
 
-           case AFSCALL_PIOCTL:
-               AFS_GLOCK();
-               rval1 = afs_syscall_pioctl(p1, p2, p3, p4);
-               AFS_GUNLOCK();
-               break;
+    case AFSCALL_PIOCTL:
+       AFS_GLOCK();
+       rval1 = afs_syscall_pioctl(p1, p2, p3, p4);
+       AFS_GUNLOCK();
+       break;
 
-           case AFSCALL_ICREATE:
-               rval1 = afs_syscall_icreate(p1, p2, p3, p4, p5, p6);
-               break;
+    case AFSCALL_ICREATE:
+       rval1 = afs_syscall_icreate(p1, p2, p3, p4, p5, p6);
+       break;
 
-           case AFSCALL_IOPEN:
-               rval1 = afs_syscall_iopen(p1, p2, p3);
-               break;
+    case AFSCALL_IOPEN:
+       rval1 = afs_syscall_iopen(p1, p2, p3);
+       break;
 
-           case AFSCALL_IDEC:
-               rval1 = afs_syscall_iincdec(p1, p2, p3, -1);
-               break;
+    case AFSCALL_IDEC:
+       rval1 = afs_syscall_iincdec(p1, p2, p3, -1);
+       break;
 
-           case AFSCALL_IINC:
-               rval1 = afs_syscall_iincdec(p1, p2, p3, 1);
-               break;
+    case AFSCALL_IINC:
+       rval1 = afs_syscall_iincdec(p1, p2, p3, 1);
+       break;
 
-           case AFSCALL_ICL:
-               AFS_GLOCK();
-               code = Afscall_icl(p1, p2, p3, p4, p5, &retval);
-               AFS_GUNLOCK();
-               if (!code) rval1 = retval;
-               if (!rval1) rval1 = code;
-               break;
+    case AFSCALL_ICL:
+       AFS_GLOCK();
+       code = Afscall_icl(p1, p2, p3, p4, p5, &retval);
+       AFS_GUNLOCK();
+       if (!code)
+           rval1 = retval;
+       if (!rval1)
+           rval1 = code;
+       break;
 
-           default:
-               rval1 = EINVAL;
-               setuerror(EINVAL);
-               break;
-       }
+    default:
+       rval1 = EINVAL;
+       setuerror(EINVAL);
+       break;
+    }
 
-    out:
+  out:
 #ifndef AFS_AIX41_ENV
-       if (monster != LOCK_NEST)
-               unlockl(&kernel_lock);
+    if (monster != LOCK_NEST)
+       unlockl(&kernel_lock);
 #endif /* !AFS_AIX41_ENV */
-       return getuerror() ? -1 : rval1;
+    return getuerror()? -1 : rval1;
 }
 
 /*
  * lsetpag -   interface to afs_setpag().
  */
-lsetpag() {
+lsetpag()
+{
 
     AFS_STATCNT(lsetpag);
     return syscall(AFSCALL_SETPAG, 0, 0, 0, 0, 0);
@@ -782,57 +1007,63 @@ lsetpag() {
  * lpioctl -   interface to pioctl()
  */
 lpioctl(path, cmd, cmarg, follow)
-char *path, *cmarg; {
+     char *path, *cmarg;
+{
 
     AFS_STATCNT(lpioctl);
     return syscall(AFSCALL_PIOCTL, path, cmd, cmarg, follow);
 }
 
-#else  /* !AFS_AIX32_ENV       */
+#else /* !AFS_AIX32_ENV       */
 
 #if defined(AFS_SGI_ENV)
-struct afsargs
-{
-       sysarg_t syscall;
-       sysarg_t parm1;
-       sysarg_t parm2;
-       sysarg_t parm3;
-       sysarg_t parm4;
-       sysarg_t parm5;
+struct afsargs {
+    sysarg_t syscall;
+    sysarg_t parm1;
+    sysarg_t parm2;
+    sysarg_t parm3;
+    sysarg_t parm4;
+    sysarg_t parm5;
 };
 
 
 int
-Afs_syscall (struct afsargs *uap, rval_t *rvp)
+Afs_syscall(struct afsargs *uap, rval_t * rvp)
 {
     int error;
     long retval;
 
     AFS_STATCNT(afs_syscall);
-    switch(uap->syscall) {
+    switch (uap->syscall) {
     case AFSCALL_ICL:
        retval = 0;
        AFS_GLOCK();
-       error=Afscall_icl(uap->parm1,uap->parm2,uap->parm3,uap->parm4,uap->parm5, &retval);
+       error =
+           Afscall_icl(uap->parm1, uap->parm2, uap->parm3, uap->parm4,
+                       uap->parm5, &retval);
        AFS_GUNLOCK();
        rvp->r_val1 = retval;
        break;
 #ifdef AFS_SGI_XFS_IOPS_ENV
     case AFSCALL_IDEC64:
-       error = afs_syscall_idec64(uap->parm1, uap->parm2, uap->parm3,
-                                  uap->parm4, uap->parm5);
+       error =
+           afs_syscall_idec64(uap->parm1, uap->parm2, uap->parm3, uap->parm4,
+                              uap->parm5);
        break;
     case AFSCALL_IINC64:
-       error = afs_syscall_iinc64(uap->parm1, uap->parm2, uap->parm3,
-                                  uap->parm4, uap->parm5);
+       error =
+           afs_syscall_iinc64(uap->parm1, uap->parm2, uap->parm3, uap->parm4,
+                              uap->parm5);
        break;
     case AFSCALL_ILISTINODE64:
-       error = afs_syscall_ilistinode64(uap->parm1, uap->parm2, uap->parm3,
-                                  uap->parm4, uap->parm5);
+       error =
+           afs_syscall_ilistinode64(uap->parm1, uap->parm2, uap->parm3,
+                                    uap->parm4, uap->parm5);
        break;
     case AFSCALL_ICREATENAME64:
-       error = afs_syscall_icreatename64(uap->parm1, uap->parm2, uap->parm3,
-                                  uap->parm4, uap->parm5);
+       error =
+           afs_syscall_icreatename64(uap->parm1, uap->parm2, uap->parm3,
+                                     uap->parm4, uap->parm5);
        break;
 #endif
 #ifdef AFS_SGI_VNODE_GLUE
@@ -841,8 +1072,9 @@ Afs_syscall (struct afsargs *uap, rval_t *rvp)
        break;
 #endif
     default:
-       error = afs_syscall_call(uap->syscall, uap->parm1, uap->parm2,
-                                uap->parm3, uap->parm4, uap->parm5);
+       error =
+           afs_syscall_call(uap->syscall, uap->parm1, uap->parm2, uap->parm3,
+                            uap->parm4, uap->parm5);
     }
     return error;
 }
@@ -864,14 +1096,16 @@ struct iparam32 {
 };
 
 
+#if defined(AFS_HPUX_64BIT_ENV) || defined(AFS_SUN57_64BIT_ENV) || (defined(AFS_LINUX_64BIT_KERNEL) && !defined(AFS_ALPHA_LINUX20_ENV) && !defined(AFS_IA64_LINUX20_ENV) && !defined(AFS_AMD64_LINUX20_ENV))
 static void
 iparam32_to_iparam(const struct iparam32 *src, struct iparam *dst)
 {
-       dst->param1 = src->param1;
-       dst->param2 = src->param2;
-       dst->param3 = src->param3;
-       dst->param4 = src->param4;
+    dst->param1 = src->param1;
+    dst->param2 = src->param2;
+    dst->param3 = src->param3;
+    dst->param4 = src->param4;
 }
+#endif
 
 /*
  * If you need to change copyin_iparam(), you may also need to change
@@ -881,51 +1115,56 @@ iparam32_to_iparam(const struct iparam32 *src, struct iparam *dst)
 static int
 copyin_iparam(caddr_t cmarg, struct iparam *dst)
 {
-       int code;
+    int code;
 
 #if defined(AFS_HPUX_64BIT_ENV)
-       struct iparam32 dst32;
+    struct iparam32 dst32;
 
-       if (is_32bit(u.u_procp))        /* is_32bit() in proc_iface.h */
-       {
-               AFS_COPYIN(cmarg, (caddr_t) &dst32, sizeof dst32, code);
-               if (!code)
-                       iparam32_to_iparam(&dst32, dst);
-               return code;
-       }
+    if (is_32bit(u.u_procp)) { /* is_32bit() in proc_iface.h */
+       AFS_COPYIN(cmarg, (caddr_t) & dst32, sizeof dst32, code);
+       if (!code)
+           iparam32_to_iparam(&dst32, dst);
+       return code;
+    }
 #endif /* AFS_HPUX_64BIT_ENV */
 
 #if defined(AFS_SUN57_64BIT_ENV)
-       struct iparam32 dst32;
+    struct iparam32 dst32;
 
-       if (get_udatamodel() == DATAMODEL_ILP32) {
-               AFS_COPYIN(cmarg, (caddr_t) &dst32, sizeof dst32, code);
-               if (!code)
-                       iparam32_to_iparam(&dst32, dst);
-               return code;
-       }
+    if (get_udatamodel() == DATAMODEL_ILP32) {
+       AFS_COPYIN(cmarg, (caddr_t) & dst32, sizeof dst32, code);
+       if (!code)
+           iparam32_to_iparam(&dst32, dst);
+       return code;
+    }
 #endif /* AFS_SUN57_64BIT_ENV */
 
-#if defined(AFS_LINUX_64BIT_KERNEL) && !defined(AFS_ALPHA_LINUX20_ENV) && !defined(AFS_IA64_LINUX20_ENV)
-       struct iparam32 dst32;
+#if defined(AFS_LINUX_64BIT_KERNEL) && !defined(AFS_ALPHA_LINUX20_ENV) && !defined(AFS_IA64_LINUX20_ENV) && !defined(AFS_AMD64_LINUX20_ENV)
+    struct iparam32 dst32;
 
 #ifdef AFS_SPARC64_LINUX24_ENV
-       if (current->thread.flags & SPARC_FLAG_32BIT) 
-#elif AFS_SPARC64_LINUX20_ENV
-       if (current->tss.flags & SPARC_FLAG_32BIT) 
+    if (current->thread.flags & SPARC_FLAG_32BIT)
+#elif defined(AFS_SPARC64_LINUX20_ENV)
+    if (current->tss.flags & SPARC_FLAG_32BIT)
+#elif defined(AFS_AMD64_LINUX20_ENV)
+    if (current->thread.flags & THREAD_IA32)
+#elif defined(AFS_PPC64_LINUX20_ENV)
+    if (current->thread.flags & PPC_FLAG_32BIT) 
+#elif defined(AFS_S390X_LINUX20_ENV)
+    if (current->thread.flags & S390_FLAG_31BIT) 
 #else
 #error Not done for this linux version
-#endif /* AFS_SPARC64_LINUX20_ENV */
-       {
-               AFS_COPYIN(cmarg, (caddr_t) &dst32, sizeof dst32, code);
-               if (!code)
-                       iparam32_to_iparam(&dst32, dst);
-               return code;
-       }
+#endif
+    {
+       AFS_COPYIN(cmarg, (caddr_t) & dst32, sizeof dst32, code);
+       if (!code)
+           iparam32_to_iparam(&dst32, dst);
+       return code;
+    }
 #endif /* AFS_LINUX_64BIT_KERNEL */
 
-       AFS_COPYIN(cmarg, (caddr_t) dst, sizeof *dst, code);
-       return code;
+    AFS_COPYIN(cmarg, (caddr_t) dst, sizeof *dst, code);
+    return code;
 }
 
 /* Main entry of all afs system calls */
@@ -959,26 +1198,31 @@ struct afssysa {
 };
 #endif
 
-Afs_syscall(register struct afssysa *uap, rval_t *rvp)
+Afs_syscall(register struct afssysa *uap, rval_t * rvp)
 {
     int *retval = &rvp->r_val1;
 #else /* AFS_SUN5_ENV */
-#if    defined(AFS_OSF_ENV) || defined(AFS_DARWIN_ENV) || defined(AFS_FBSD_ENV)
+#if    defined(AFS_OSF_ENV) || defined(AFS_DARWIN_ENV) || defined(AFS_XBSD_ENV)
+int
 afs3_syscall(p, args, retval)
-       struct proc *p;
-       void *args;
-       int *retval;
+#ifdef AFS_FBSD50_ENV
+     struct thread *p;
+#else
+     struct proc *p;
+#endif
+     void *args;
+     int *retval;
 {
     register struct a {
-           long syscall;
-           long parm1;
-           long parm2;
-           long parm3;
-           long parm4;
-           long parm5;
-           long parm6;
-       } *uap = (struct a *)args;
-#else  /* AFS_OSF_ENV */
+       long syscall;
+       long parm1;
+       long parm2;
+       long parm3;
+       long parm4;
+       long parm5;
+       long parm6;
+    } *uap = (struct a *)args;
+#else /* AFS_OSF_ENV */
 #ifdef AFS_LINUX20_ENV
 struct afssysargs {
     long syscall;
@@ -987,59 +1231,58 @@ struct afssysargs {
     long parm3;
     long parm4;
     long parm5;
-    long parm6; /* not actually used - should be removed */
+    long parm6;                        /* not actually used - should be removed */
 };
 /* Linux system calls only set up for 5 arguments. */
-asmlinkage int afs_syscall(long syscall, long parm1, long parm2, long parm3,
-                          long parm4)
+asmlinkage long
+afs_syscall(long syscall, long parm1, long parm2, long parm3, long parm4)
 {
     struct afssysargs args, *uap = &args;
-    long linux_ret=0;
+    long linux_ret = 0;
     long *retval = &linux_ret;
-    long eparm[4]; /* matches AFSCALL_ICL in fstrace.c */
+    long eparm[4];             /* matches AFSCALL_ICL in fstrace.c */
 #ifdef AFS_SPARC64_LINUX24_ENV
     afs_int32 eparm32[4];
 #endif
     /* eparm is also used by AFSCALL_CALL in afsd.c */
 #else
 #if defined(UKERNEL)
-Afs_syscall ()
+Afs_syscall()
 {
     register struct a {
-           long syscall;
-           long parm1;
-           long parm2;
-           long parm3;
-           long parm4;
-           long parm5;
-           long parm6;
-       } *uap = (struct a *)u.u_ap;
+       long syscall;
+       long parm1;
+       long parm2;
+       long parm3;
+       long parm4;
+       long parm5;
+       long parm6;
+    } *uap = (struct a *)u.u_ap;
 #else /* UKERNEL */
+int
 #if defined(AFS_SUN_ENV) && !defined(AFS_SUN5_ENV)
-afs_syscall ()
+afs_syscall()
 #else
-Afs_syscall ()
-#endif /* SUN && !SUN5 */
+Afs_syscall()
+#endif                         /* SUN && !SUN5 */
 {
     register struct a {
-           long syscall;
-           long parm1;
-           long parm2;
-           long parm3;
-           long parm4;
-           long parm5;
-           long parm6;
-       } *uap = (struct a *)u.u_ap;
+       long syscall;
+       long parm1;
+       long parm2;
+       long parm3;
+       long parm4;
+       long parm5;
+       long parm6;
+    } *uap = (struct a *)u.u_ap;
 #endif /* UKERNEL */
 #if  defined(AFS_DEC_ENV)
     int *retval = &u.u_r.r_val1;
-#else
-#if defined(AFS_HPUX_ENV)
+#elif defined(AFS_HPUX_ENV)
     long *retval = &u.u_rval1;
 #else
     int *retval = &u.u_rval1;
 #endif
-#endif
 #endif /* AFS_LINUX20_ENV */
 #endif /* AFS_OSF_ENV */
 #endif /* AFS_SUN5_ENV */
@@ -1074,19 +1317,18 @@ Afs_syscall ()
 
 
        if (current->thread.flags & SPARC_FLAG_32BIT) {
-       AFS_COPYIN((char*)parm4, (char*)eparm32, sizeof(eparm32), code);
-       eparm[0]=AA(eparm32[0]);
-       eparm[1]=AA(eparm32[1]);
-       eparm[2]=AA(eparm32[2]);
+           AFS_COPYIN((char *)parm4, (char *)eparm32, sizeof(eparm32), code);
+           eparm[0] = AA(eparm32[0]);
+           eparm[1] = AA(eparm32[1]);
+           eparm[2] = AA(eparm32[2]);
 #undef AA
-} else
+       } else
 #endif
-       AFS_COPYIN((char*)parm4, (char*)eparm, sizeof(eparm), code);
+           AFS_COPYIN((char *)parm4, (char *)eparm, sizeof(eparm), code);
        uap->parm4 = eparm[0];
        uap->parm5 = eparm[1];
        uap->parm6 = eparm[2];
-    }
-    else {
+    } else {
        uap->parm4 = parm4;
        uap->parm5 = 0;
        uap->parm6 = 0;
@@ -1104,10 +1346,13 @@ Afs_syscall ()
 #endif
     if (uap->syscall == AFSCALL_CALL) {
 #ifdef AFS_SUN5_ENV
-       code =  afs_syscall_call(uap->parm1, uap->parm2, uap->parm3, 
-                               uap->parm4, uap->parm5, uap->parm6, rvp, CRED());
+       code =
+           afs_syscall_call(uap->parm1, uap->parm2, uap->parm3, uap->parm4,
+                            uap->parm5, uap->parm6, rvp, CRED());
 #else
-       code = afs_syscall_call(uap->parm1, uap->parm2, uap->parm3, uap->parm4, uap->parm5, uap->parm6);
+       code =
+           afs_syscall_call(uap->parm1, uap->parm2, uap->parm3, uap->parm4,
+                            uap->parm5, uap->parm6);
 #endif
     } else if (uap->syscall == AFSCALL_SETPAG) {
 #ifdef AFS_SUN5_ENV
@@ -1115,27 +1360,35 @@ Afs_syscall ()
 
        procp = ttoproc(curthread);
        AFS_GLOCK();
-       code =  afs_setpag(&procp->p_cred);
+       code = afs_setpag(&procp->p_cred);
        AFS_GUNLOCK();
 #else
        AFS_GLOCK();
-#if    defined(AFS_OSF_ENV) || defined(AFS_DARWIN_ENV) || defined(AFS_FBSD_ENV)
+#if    defined(AFS_OSF_ENV) || defined(AFS_DARWIN_ENV) || defined(AFS_XBSD_ENV)
        code = afs_setpag(p, args, retval);
-#else  /* AFS_OSF_ENV */
+#else /* AFS_OSF_ENV */
        code = afs_setpag();
 #endif
        AFS_GUNLOCK();
 #endif
     } else if (uap->syscall == AFSCALL_PIOCTL) {
        AFS_GLOCK();
-#ifdef AFS_SUN5_ENV
-        code = afs_syscall_pioctl(uap->parm1, uap->parm2, uap->parm3, uap->parm4, rvp, CRED());
-#else
-#if defined(AFS_DARWIN_ENV) || defined(AFS_FBSD_ENV)
-        code = afs_syscall_pioctl(uap->parm1, uap->parm2, uap->parm3, uap->parm4, p->p_cred->pc_ucred);
+#if defined(AFS_SUN5_ENV)
+       code =
+           afs_syscall_pioctl(uap->parm1, uap->parm2, uap->parm3, uap->parm4,
+                              rvp, CRED());
+#elif defined(AFS_FBSD50_ENV)
+       code =
+           afs_syscall_pioctl(uap->parm1, uap->parm2, uap->parm3, uap->parm4,
+                              p->td_ucred);
+#elif defined(AFS_DARWIN_ENV) || defined(AFS_XBSD_ENV)
+       code =
+           afs_syscall_pioctl(uap->parm1, uap->parm2, uap->parm3, uap->parm4,
+                              p->p_cred->pc_ucred);
 #else
-       code = afs_syscall_pioctl(uap->parm1, uap->parm2, uap->parm3, uap->parm4);
-#endif
+       code =
+           afs_syscall_pioctl(uap->parm1, uap->parm2, uap->parm3,
+                              uap->parm4);
 #endif
        AFS_GUNLOCK();
     } else if (uap->syscall == AFSCALL_ICREATE) {
@@ -1143,67 +1396,79 @@ Afs_syscall ()
 
        code = copyin_iparam((char *)uap->parm3, &iparams);
        if (code) {
-#if defined(KERNEL_HAVE_SETUERROR)
+#if defined(KERNEL_HAVE_UERROR)
            setuerror(code);
 #endif
        } else
 #ifdef AFS_SUN5_ENV
-       code =  afs_syscall_icreate(uap->parm1, uap->parm2, iparams.param1, iparams.param2, 
-                                  iparams.param3, iparams.param4, rvp, CRED());
+           code =
+               afs_syscall_icreate(uap->parm1, uap->parm2, iparams.param1,
+                                   iparams.param2, iparams.param3,
+                                   iparams.param4, rvp, CRED());
 #else
-       code =  afs_syscall_icreate(uap->parm1, uap->parm2, iparams.param1, iparams.param2,
-#if defined(AFS_OSF_ENV) || defined(AFS_DARWIN_ENV) || defined(AFS_FBSD_ENV)
-                                  iparams.param3, iparams.param4, retval);
+           code =
+               afs_syscall_icreate(uap->parm1, uap->parm2, iparams.param1,
+                                   iparams.param2,
+#if defined(AFS_OSF_ENV) || defined(AFS_DARWIN_ENV) || defined(AFS_XBSD_ENV)
+                                   iparams.param3, iparams.param4, retval);
 #else
-                                  iparams.param3, iparams.param4);
+                                   iparams.param3, iparams.param4);
 #endif
-#endif /* AFS_SUN5_ENV */
+#endif /* AFS_SUN5_ENV */
     } else if (uap->syscall == AFSCALL_IOPEN) {
 #ifdef AFS_SUN5_ENV
-       code = afs_syscall_iopen(uap->parm1, uap->parm2, uap->parm3, rvp, CRED());
+       code =
+           afs_syscall_iopen(uap->parm1, uap->parm2, uap->parm3, rvp,
+                             CRED());
 #else
-#if defined(AFS_OSF_ENV) || defined(AFS_DARWIN_ENV) || defined(AFS_FBSD_ENV)
+#if defined(AFS_OSF_ENV) || defined(AFS_DARWIN_ENV) || defined(AFS_XBSD_ENV)
        code = afs_syscall_iopen(uap->parm1, uap->parm2, uap->parm3, retval);
 #else
        code = afs_syscall_iopen(uap->parm1, uap->parm2, uap->parm3);
 #endif
-#endif /* AFS_SUN5_ENV */
+#endif /* AFS_SUN5_ENV */
     } else if (uap->syscall == AFSCALL_IDEC) {
 #ifdef AFS_SUN5_ENV
-       code = afs_syscall_iincdec(uap->parm1, uap->parm2, uap->parm3, -1, rvp, CRED());
+       code =
+           afs_syscall_iincdec(uap->parm1, uap->parm2, uap->parm3, -1, rvp,
+                               CRED());
 #else
        code = afs_syscall_iincdec(uap->parm1, uap->parm2, uap->parm3, -1);
-#endif /* AFS_SUN5_ENV */
+#endif /* AFS_SUN5_ENV */
     } else if (uap->syscall == AFSCALL_IINC) {
 #ifdef AFS_SUN5_ENV
-       code = afs_syscall_iincdec(uap->parm1, uap->parm2, uap->parm3, 1, rvp, CRED());
+       code =
+           afs_syscall_iincdec(uap->parm1, uap->parm2, uap->parm3, 1, rvp,
+                               CRED());
 #else
        code = afs_syscall_iincdec(uap->parm1, uap->parm2, uap->parm3, 1);
-#endif /* AFS_SUN5_ENV */
+#endif /* AFS_SUN5_ENV */
     } else if (uap->syscall == AFSCALL_ICL) {
        AFS_GLOCK();
-       code = Afscall_icl(uap->parm1, uap->parm2, uap->parm3, uap->parm4, uap->parm5, retval);
+       code =
+           Afscall_icl(uap->parm1, uap->parm2, uap->parm3, uap->parm4,
+                       uap->parm5, retval);
        AFS_GUNLOCK();
 #ifdef AFS_LINUX20_ENV
        if (!code) {
            /* ICL commands can return values. */
-           code = -linux_ret; /* Gets negated again at exit below */
+           code = -linux_ret;  /* Gets negated again at exit below */
        }
 #else
-        if (code) {
-#if defined(KERNEL_HAVE_SETUERROR)
+       if (code) {
+#if defined(KERNEL_HAVE_UERROR)
            setuerror(code);
 #endif
-        }
+       }
 #endif /* !AFS_LINUX20_ENV */
     } else {
-#if defined(KERNEL_HAVE_SETUERROR)
-        setuerror(EINVAL);
+#if defined(KERNEL_HAVE_UERROR)
+       setuerror(EINVAL);
 #else
        code = EINVAL;
 #endif
     }
-out:
+
 #ifdef AFS_LINUX20_ENV
     code = -code;
     unlock_kernel();
@@ -1211,7 +1476,7 @@ out:
     return code;
 }
 #endif /* AFS_SGI_ENV */
-#endif /* !AFS_AIX32_ENV       */
+#endif /* !AFS_AIX32_ENV       */
 
 /*
  * Initstate in the range 0 < x < 100 are early initialization states.
@@ -1223,22 +1488,25 @@ out:
  *  incorrectly.
  * Initstate of 300 means that the volume has been *successfully* looked up.
  */
-int afs_CheckInit(void)
+int
+afs_CheckInit(void)
 {
     register int code = 0;
 
     AFS_STATCNT(afs_CheckInit);
     if (afs_initState <= 100)
-       code =  ENXIO;   /* never finished init phase */
+       code = ENXIO;           /* never finished init phase */
     else if (afs_initState == 101) {   /* init done, wait for afs_daemon */
-       while (afs_initState < 200) afs_osi_Sleep(&afs_initState);
-    } else  if (afs_initState == 200) 
-       code =  ETIMEDOUT; /* didn't find root volume */
+       while (afs_initState < 200)
+           afs_osi_Sleep(&afs_initState);
+    } else if (afs_initState == 200)
+       code = ETIMEDOUT;       /* didn't find root volume */
     return code;
 }
 
-int afs_shuttingdown = 0; 
-void afs_shutdown(void)
+int afs_shuttingdown = 0;
+void
+afs_shutdown(void)
 {
     extern short afs_brsDaemons;
     extern afs_int32 afs_CheckServerDaemonStarted;
@@ -1246,14 +1514,25 @@ void afs_shutdown(void)
     extern struct osi_file *afs_cacheInodep;
 
     AFS_STATCNT(afs_shutdown);
-    if (afs_shuttingdown) return;
+    if (afs_initState == 0) {
+        afs_warn("AFS not initialized - not shutting down\n");
+      return;
+    }
+
+    if (afs_shuttingdown)
+       return;
     afs_shuttingdown = 1;
-    if (afs_cold_shutdown) afs_warn("COLD ");
-    else afs_warn("WARM ");
-    afs_warn("shutting down of: CB... "); 
+    if (afs_cold_shutdown)
+       afs_warn("COLD ");
+    else
+       afs_warn("WARM ");
+    afs_warn("shutting down of: CB... ");
 
     afs_termState = AFSOP_STOP_RXCALLBACK;
     rx_WakeupServerProcs();
+#ifdef AFS_AIX51_ENV
+    shutdown_rxkernel();
+#endif
     /* shutdown_rxkernel(); */
     while (afs_termState == AFSOP_STOP_RXCALLBACK)
        afs_osi_Sleep(&afs_termState);
@@ -1278,7 +1557,7 @@ void afs_shutdown(void)
     afs_warn("CTrunc... ");
     /* Cancel cache truncate daemon. */
     while (afs_termState == AFSOP_STOP_TRUNCDAEMON) {
-       afs_osi_Wakeup((char*)&afs_CacheTruncateDaemon);
+       afs_osi_Wakeup((char *)&afs_CacheTruncateDaemon);
        afs_osi_Sleep(&afs_termState);
     }
 #ifdef AFS_AFSDB_ENV
@@ -1289,8 +1568,8 @@ void afs_shutdown(void)
 #endif
 #if    defined(AFS_SUN5_ENV) || defined(RXK_LISTENER_ENV)
     afs_warn("RxEvent... ");
-    /* cancel rx event deamon */
-    while (afs_termState == AFSOP_STOP_RXEVENT) 
+    /* cancel rx event daemon */
+    while (afs_termState == AFSOP_STOP_RXEVENT)
        afs_osi_Sleep(&afs_termState);
 #if defined(RXK_LISTENER_ENV)
 #ifndef UKERNEL
@@ -1299,33 +1578,36 @@ void afs_shutdown(void)
 #endif
     /* cancel rx listener */
     afs_warn("RxListener... ");
-    osi_StopListener(); /* This closes rx_socket. */
+    osi_StopListener();                /* This closes rx_socket. */
     while (afs_termState == AFSOP_STOP_RXK_LISTENER) {
        afs_warn("Sleep... ");
        afs_osi_Sleep(&afs_termState);
     }
 #endif
 #else
-    afs_termState =  AFSOP_STOP_COMPLETE;
+    afs_termState = AFSOP_STOP_COMPLETE;
 #endif
     afs_warn("\n");
 
     /* Close file only after daemons which can write to it are stopped. */
-    if (afs_cacheInodep)       /* memcache won't set this */
-    {
-       osi_UFSClose(afs_cacheInodep);    /* Since we always leave it open */
+    if (afs_cacheInodep) {     /* memcache won't set this */
+       osi_UFSClose(afs_cacheInodep);  /* Since we always leave it open */
        afs_cacheInodep = 0;
     }
-    return;    /* Just kill daemons for now */
+    return;                    /* Just kill daemons for now */
 #ifdef notdef
-    shutdown_CB();  
+    shutdown_CB();
     shutdown_AFS();
     shutdown_rxkernel();
-    shutdown_rxevent(); 
+    shutdown_rxevent();
     shutdown_rx();
-    afs_shutdown_BKG(); 
+    afs_shutdown_BKG();
     shutdown_bufferpackage();
+#endif
+#ifdef AFS_AIX51_ENV
     shutdown_daemons();
+#endif
+#ifdef notdef
     shutdown_cache();
     shutdown_osi();
     shutdown_osinet();
@@ -1352,22 +1634,24 @@ void afs_shutdown(void)
 #endif
 }
 
-void shutdown_afstest(void)
+void
+shutdown_afstest(void)
 {
     AFS_STATCNT(shutdown_afstest);
     afs_initState = afs_termState = afs_setTime = 0;
     AFS_Running = afs_CB_Running = 0;
     afs_CacheInit_Done = afs_Go_Done = 0;
     if (afs_cold_shutdown) {
-      *afs_rootVolumeName = 0;
+       *afs_rootVolumeName = 0;
     }
 }
 
 
 /* In case there is a bunch of dynamically build bkg daemons to free */
-void afs_shutdown_BKG(void)
+void
+afs_shutdown_BKG(void)
 {
-  AFS_STATCNT(shutdown_BKG);
+    AFS_STATCNT(shutdown_BKG);
 }
 
 
@@ -1379,13 +1663,18 @@ int afs_icl_sizeofLong = 1;
 int afs_icl_sizeofLong = 2;
 #endif /* SGI62 */
 #else
+#if defined(AFS_AIX51_ENV) && defined(AFS_64BIT_KERNEL)
+int afs_icl_sizeofLong = 2;
+#else
 int afs_icl_sizeofLong = 1;
 #endif
+#endif
 
 int afs_icl_inited = 0;
 
 /* init function, called once, under afs_icl_lock */
-int afs_icl_Init(void)
+int
+afs_icl_Init(void)
 {
     afs_icl_inited = 1;
     return 0;
@@ -1398,15 +1687,18 @@ extern struct afs_icl_set *afs_icl_FindSet();
 static int
 Afscall_icl(long opcode, long p1, long p2, long p3, long p4, long *retval)
 {
-    register int i;
     afs_int32 *lp, elts, flags;
     register afs_int32 code;
     struct afs_icl_log *logp;
     struct afs_icl_set *setp;
-#if defined(AFS_SGI61_ENV) || defined(AFS_SUN57_ENV) || defined(AFS_DARWIN_ENV) || defined(AFS_FBSD_ENV)
+#if defined(AFS_SGI61_ENV) || defined(AFS_SUN57_ENV) || defined(AFS_DARWIN_ENV) || defined(AFS_XBSD_ENV)
     size_t temp;
 #else /* AFS_SGI61_ENV */
+#if defined(AFS_AIX51_ENV) && defined(AFS_64BIT_KERNEL)
+    afs_uint64 temp;
+#else
     afs_uint32 temp;
+#endif
 #endif /* AFS_SGI61_ENV */
     char tname[65];
     afs_int32 startCookie;
@@ -1418,8 +1710,8 @@ Afscall_icl(long opcode, long p1, long p2, long p3, long p4, long *retval)
        return (EACCES);
     }
 #else
-    if (!afs_suser()) {        /* only root can run this code */
-#if defined(KERNEL_HAVE_SETUERROR)
+    if (!afs_suser(NULL)) {    /* only root can run this code */
+#if defined(KERNEL_HAVE_UERROR)
        setuerror(EACCES);
        return EACCES;
 #else
@@ -1436,98 +1728,123 @@ Afscall_icl(long opcode, long p1, long p2, long p3, long p4, long *retval)
         * skip some records.
         */
        AFS_COPYINSTR((char *)p1, tname, sizeof(tname), &temp, code);
-       if (code) return code;
+       if (code)
+           return code;
        AFS_COPYIN((char *)p4, (char *)&startCookie, sizeof(afs_int32), code);
-       if (code) return code;
+       if (code)
+           return code;
        logp = afs_icl_FindLog(tname);
-       if (!logp) return ENOENT;
+       if (!logp)
+           return ENOENT;
 #define        BUFFERSIZE      AFS_LRALLOCSIZ
        lp = (afs_int32 *) osi_AllocLargeSpace(AFS_LRALLOCSIZ);
        elts = BUFFERSIZE / sizeof(afs_int32);
-       if (p3 < elts) elts = p3;
+       if (p3 < elts)
+           elts = p3;
        flags = (opcode == ICL_OP_COPYOUT) ? 0 : ICL_COPYOUTF_CLRAFTERREAD;
-       code = afs_icl_CopyOut(logp, lp, &elts, (afs_uint32 *) &startCookie,
-                          &flags);
+       code =
+           afs_icl_CopyOut(logp, lp, &elts, (afs_uint32 *) & startCookie,
+                           &flags);
        if (code) {
-           osi_FreeLargeSpace((struct osi_buffer *) lp);
+           osi_FreeLargeSpace((struct osi_buffer *)lp);
            break;
        }
        AFS_COPYOUT((char *)lp, (char *)p2, elts * sizeof(afs_int32), code);
-       if (code) goto done;
-       AFS_COPYOUT((char *) &startCookie, (char *)p4, sizeof(afs_int32), code);
-       if (code) goto done;
-       *retval = (flags<<24) | (elts & 0xffffff);
+       if (code)
+           goto done;
+       AFS_COPYOUT((char *)&startCookie, (char *)p4, sizeof(afs_int32),
+                   code);
+       if (code)
+           goto done;
+#if defined(AFS_AIX51_ENV) && defined(AFS_64BIT_KERNEL)
+       if (!(IS64U))
+           *retval = ((long)((flags << 24) | (elts & 0xffffff))) << 32;
+       else
+#endif
+           *retval = (flags << 24) | (elts & 0xffffff);
       done:
        afs_icl_LogRele(logp);
-       osi_FreeLargeSpace((struct osi_buffer *) lp);
+       osi_FreeLargeSpace((struct osi_buffer *)lp);
        break;
 
     case ICL_OP_ENUMLOGS:      /* enumerate logs */
        /* enumerate logs: p1=index, p2=&name, p3=sizeof(name), p4=&size.
         * return 0 for success, otherwise error.
         */
-       for(tlp = afs_icl_allLogs; tlp; tlp=tlp->nextp) {
-           if (p1-- == 0) break;
+       for (tlp = afs_icl_allLogs; tlp; tlp = tlp->nextp) {
+           if (p1-- == 0)
+               break;
        }
-       if (!tlp) return ENOENT;     /* past the end of file */
-       temp = strlen(tlp->name)+1;
-       if (temp > p3) return EINVAL;
-       AFS_COPYOUT(tlp->name, (char *) p2, temp, code);
-       if (!code)      /* copy out size of log */
-           AFS_COPYOUT((char *)&tlp->logSize, (char *)p4, sizeof (afs_int32), code);
+       if (!tlp)
+           return ENOENT;      /* past the end of file */
+       temp = strlen(tlp->name) + 1;
+       if (temp > p3)
+           return EINVAL;
+       AFS_COPYOUT(tlp->name, (char *)p2, temp, code);
+       if (!code)              /* copy out size of log */
+           AFS_COPYOUT((char *)&tlp->logSize, (char *)p4, sizeof(afs_int32),
+                       code);
        break;
 
     case ICL_OP_ENUMLOGSBYSET: /* enumerate logs by set name */
        /* enumerate logs: p1=setname, p2=index, p3=&name, p4=sizeof(name).
         * return 0 for success, otherwise error.
         */
-       AFS_COPYINSTR((char *)p1, tname, sizeof (tname), &temp, code);
-       if (code) return code;
+       AFS_COPYINSTR((char *)p1, tname, sizeof(tname), &temp, code);
+       if (code)
+           return code;
        setp = afs_icl_FindSet(tname);
-       if (!setp) return ENOENT;
+       if (!setp)
+           return ENOENT;
        if (p2 > ICL_LOGSPERSET)
            return EINVAL;
        if (!(tlp = setp->logs[p2]))
            return EBADF;
-       temp = strlen(tlp->name)+1;
-       if (temp > p4) return EINVAL;
+       temp = strlen(tlp->name) + 1;
+       if (temp > p4)
+           return EINVAL;
        AFS_COPYOUT(tlp->name, (char *)p3, temp, code);
        break;
 
-    case ICL_OP_CLRLOG:                /* clear specified log */
+    case ICL_OP_CLRLOG:        /* clear specified log */
        /* zero out the specified log: p1=logname */
-       AFS_COPYINSTR((char *)p1, tname, sizeof (tname), &temp, code);
-       if (code) return code;
+       AFS_COPYINSTR((char *)p1, tname, sizeof(tname), &temp, code);
+       if (code)
+           return code;
        logp = afs_icl_FindLog(tname);
-       if (!logp) return ENOENT;
+       if (!logp)
+           return ENOENT;
        code = afs_icl_ZeroLog(logp);
        afs_icl_LogRele(logp);
        break;
 
-    case ICL_OP_CLRSET:                /* clear specified set */
+    case ICL_OP_CLRSET:        /* clear specified set */
        /* zero out the specified set: p1=setname */
-       AFS_COPYINSTR((char *)p1, tname, sizeof (tname), &temp, code);
-       if (code) return code;
+       AFS_COPYINSTR((char *)p1, tname, sizeof(tname), &temp, code);
+       if (code)
+           return code;
        setp = afs_icl_FindSet(tname);
-       if (!setp) return ENOENT;
+       if (!setp)
+           return ENOENT;
        code = afs_icl_ZeroSet(setp);
        afs_icl_SetRele(setp);
        break;
 
-    case ICL_OP_CLRALL:                /* clear all logs */
+    case ICL_OP_CLRALL:        /* clear all logs */
        /* zero out all logs -- no args */
        code = 0;
-       ObtainWriteLock(&afs_icl_lock,178);
-       for(tlp = afs_icl_allLogs; tlp; tlp=tlp->nextp) {
+       ObtainWriteLock(&afs_icl_lock, 178);
+       for (tlp = afs_icl_allLogs; tlp; tlp = tlp->nextp) {
            tlp->refCount++;    /* hold this guy */
            ReleaseWriteLock(&afs_icl_lock);
            /* don't clear persistent logs */
            if ((tlp->states & ICL_LOGF_PERSISTENT) == 0)
                code = afs_icl_ZeroLog(tlp);
-           ObtainWriteLock(&afs_icl_lock,179);
+           ObtainWriteLock(&afs_icl_lock, 179);
            if (--tlp->refCount == 0)
                afs_icl_ZapLog(tlp);
-           if (code) break;
+           if (code)
+               break;
        }
        ReleaseWriteLock(&afs_icl_lock);
        break;
@@ -1536,23 +1853,29 @@ Afscall_icl(long opcode, long p1, long p2, long p3, long p4, long *retval)
        /* enumerate sets: p1=index, p2=&name, p3=sizeof(name), p4=&states.
         * return 0 for success, otherwise error.
         */
-       for(setp = afs_icl_allSets; setp; setp = setp->nextp) {
-           if (p1-- == 0) break;
+       for (setp = afs_icl_allSets; setp; setp = setp->nextp) {
+           if (p1-- == 0)
+               break;
        }
-       if (!setp) return ENOENT;       /* past the end of file */
-       temp = strlen(setp->name)+1;
-       if (temp > p3) return EINVAL;
+       if (!setp)
+           return ENOENT;      /* past the end of file */
+       temp = strlen(setp->name) + 1;
+       if (temp > p3)
+           return EINVAL;
        AFS_COPYOUT(setp->name, (char *)p2, temp, code);
-       if (!code)      /* copy out size of log */
-           AFS_COPYOUT((char *)&setp->states,(char *)p4, sizeof (afs_int32), code);
+       if (!code)              /* copy out size of log */
+           AFS_COPYOUT((char *)&setp->states, (char *)p4, sizeof(afs_int32),
+                       code);
        break;
 
     case ICL_OP_SETSTAT:       /* set status on a set */
        /* activate the specified set: p1=setname, p2=op */
        AFS_COPYINSTR((char *)p1, tname, sizeof(tname), &temp, code);
-       if (code) return code;
+       if (code)
+           return code;
        setp = afs_icl_FindSet(tname);
-       if (!setp) return ENOENT;
+       if (!setp)
+           return ENOENT;
        code = afs_icl_SetSetStat(setp, p2);
        afs_icl_SetRele(setp);
        break;
@@ -1560,51 +1883,61 @@ Afscall_icl(long opcode, long p1, long p2, long p3, long p4, long *retval)
     case ICL_OP_SETSTATALL:    /* set status on all sets */
        /* activate the specified set: p1=op */
        code = 0;
-       ObtainWriteLock(&afs_icl_lock,180);
-       for(setp = afs_icl_allSets; setp; setp=setp->nextp) {
+       ObtainWriteLock(&afs_icl_lock, 180);
+       for (setp = afs_icl_allSets; setp; setp = setp->nextp) {
            setp->refCount++;   /* hold this guy */
            ReleaseWriteLock(&afs_icl_lock);
            /* don't set states on persistent sets */
            if ((setp->states & ICL_SETF_PERSISTENT) == 0)
                code = afs_icl_SetSetStat(setp, p1);
-           ObtainWriteLock(&afs_icl_lock,181);
+           ObtainWriteLock(&afs_icl_lock, 181);
            if (--setp->refCount == 0)
                afs_icl_ZapSet(setp);
-           if (code) break;
+           if (code)
+               break;
        }
        ReleaseWriteLock(&afs_icl_lock);
        break;
 
-    case ICL_OP_SETLOGSIZE:            /* set size of log */
+    case ICL_OP_SETLOGSIZE:    /* set size of log */
        /* set the size of the specified log: p1=logname, p2=size (in words) */
        AFS_COPYINSTR((char *)p1, tname, sizeof(tname), &temp, code);
-       if (code) return code;
+       if (code)
+           return code;
        logp = afs_icl_FindLog(tname);
-       if (!logp) return ENOENT;
+       if (!logp)
+           return ENOENT;
        code = afs_icl_LogSetSize(logp, p2);
        afs_icl_LogRele(logp);
        break;
 
-    case ICL_OP_GETLOGINFO:            /* get size of log */
+    case ICL_OP_GETLOGINFO:    /* get size of log */
        /* zero out the specified log: p1=logname, p2=&logSize, p3=&allocated */
        AFS_COPYINSTR((char *)p1, tname, sizeof(tname), &temp, code);
-       if (code) return code;
+       if (code)
+           return code;
        logp = afs_icl_FindLog(tname);
-       if (!logp) return ENOENT;
+       if (!logp)
+           return ENOENT;
        allocated = !!logp->datap;
-       AFS_COPYOUT((char *)&logp->logSize, (char *) p2, sizeof(afs_int32), code);
+       AFS_COPYOUT((char *)&logp->logSize, (char *)p2, sizeof(afs_int32),
+                   code);
        if (!code)
-           AFS_COPYOUT((char *)&allocated, (char *) p3, sizeof(afs_int32), code);
+           AFS_COPYOUT((char *)&allocated, (char *)p3, sizeof(afs_int32),
+                       code);
        afs_icl_LogRele(logp);
        break;
 
-    case ICL_OP_GETSETINFO:            /* get state of set */
+    case ICL_OP_GETSETINFO:    /* get state of set */
        /* zero out the specified set: p1=setname, p2=&state */
        AFS_COPYINSTR((char *)p1, tname, sizeof(tname), &temp, code);
-       if (code) return code;
+       if (code)
+           return code;
        setp = afs_icl_FindSet(tname);
-       if (!setp) return ENOENT;
-       AFS_COPYOUT((char *)&setp->states, (char *) p2, sizeof(afs_int32), code);
+       if (!setp)
+           return ENOENT;
+       AFS_COPYOUT((char *)&setp->states, (char *)p2, sizeof(afs_int32),
+                   code);
        afs_icl_SetRele(setp);
        break;
 
@@ -1619,61 +1952,70 @@ Afscall_icl(long opcode, long p1, long p2, long p3, long p4, long *retval)
 afs_lock_t afs_icl_lock;
 
 /* exported routine: a 4 parameter event */
-int afs_icl_Event4(register struct afs_icl_set *setp, afs_int32 eventID, 
-       afs_int32 lAndT, long p1, long p2, long p3, long p4)
+int
+afs_icl_Event4(register struct afs_icl_set *setp, afs_int32 eventID,
+              afs_int32 lAndT, long p1, long p2, long p3, long p4)
 {
-    register struct afs_icl_log *logp;
     afs_int32 mask;
     register int i;
     register afs_int32 tmask;
     int ix;
 
     /* If things aren't init'ed yet (or the set is inactive), don't panic */
-    if (!ICL_SETACTIVE(setp)) return;
+    if (!ICL_SETACTIVE(setp))
+       return 0;
 
     AFS_ASSERT_GLOCK();
-    mask = lAndT>>24 & 0xff;   /* mask of which logs to log to */
+    mask = lAndT >> 24 & 0xff; /* mask of which logs to log to */
     ix = ICL_EVENTBYTE(eventID);
     ObtainReadLock(&setp->lock);
     if (setp->eventFlags[ix] & ICL_EVENTMASK(eventID)) {
-       for(i=0, tmask = 1; i<ICL_LOGSPERSET; i++, tmask <<= 1) {
+       for (i = 0, tmask = 1; i < ICL_LOGSPERSET; i++, tmask <<= 1) {
            if (mask & tmask) {
                afs_icl_AppendRecord(setp->logs[i], eventID, lAndT & 0xffffff,
-                             p1, p2, p3, p4);
+                                    p1, p2, p3, p4);
            }
            mask &= ~tmask;
-           if (mask == 0) break;       /* break early */
+           if (mask == 0)
+               break;          /* break early */
        }
     }
     ReleaseReadLock(&setp->lock);
+    return 0;
 }
 
 /* Next 4 routines should be implemented via var-args or something.
  * Whole purpose is to avoid compiler warnings about parameter # mismatches.
  * Otherwise, could call afs_icl_Event4 directly.
  */
-int afs_icl_Event3(register struct afs_icl_set *setp, afs_int32 eventID, 
-       afs_int32 lAndT, long p1, long p2, long p3)
+int
+afs_icl_Event3(register struct afs_icl_set *setp, afs_int32 eventID,
+              afs_int32 lAndT, long p1, long p2, long p3)
 {
     return afs_icl_Event4(setp, eventID, lAndT, p1, p2, p3, (long)0);
 }
 
-int afs_icl_Event2(register struct afs_icl_set *setp, afs_int32 eventID, 
-       afs_int32 lAndT, long p1, long p2)
+int
+afs_icl_Event2(register struct afs_icl_set *setp, afs_int32 eventID,
+              afs_int32 lAndT, long p1, long p2)
 {
     return afs_icl_Event4(setp, eventID, lAndT, p1, p2, (long)0, (long)0);
 }
 
-int afs_icl_Event1(register struct afs_icl_set *setp, afs_int32 eventID, 
-       afs_int32 lAndT, long p1)
+int
+afs_icl_Event1(register struct afs_icl_set *setp, afs_int32 eventID,
+              afs_int32 lAndT, long p1)
 {
-    return afs_icl_Event4(setp, eventID, lAndT, p1, (long)0, (long)0, (long)0);
+    return afs_icl_Event4(setp, eventID, lAndT, p1, (long)0, (long)0,
+                         (long)0);
 }
 
-int afs_icl_Event0(register struct afs_icl_set *setp, afs_int32 eventID, 
-       afs_int32 lAndT)
+int
+afs_icl_Event0(register struct afs_icl_set *setp, afs_int32 eventID,
+              afs_int32 lAndT)
 {
-    return afs_icl_Event4(setp, eventID, lAndT, (long)0, (long)0, (long)0, (long)0);
+    return afs_icl_Event4(setp, eventID, lAndT, (long)0, (long)0, (long)0,
+                         (long)0);
 }
 
 struct afs_icl_log *afs_icl_allLogs = 0;
@@ -1684,7 +2026,8 @@ struct afs_icl_log *afs_icl_allLogs = 0;
  *
  * Log must be write-locked.
  */
-static afs_icl_GetLogSpace(register struct afs_icl_log *logp, afs_int32 minSpace)
+static void
+afs_icl_GetLogSpace(register struct afs_icl_log *logp, afs_int32 minSpace)
 {
     register unsigned int tsize;
 
@@ -1704,14 +2047,15 @@ static afs_icl_GetLogSpace(register struct afs_icl_log *logp, afs_int32 minSpace
  * log must be write-locked.
  */
 #define ICL_CHARSPERLONG       4
-static afs_int32 afs_icl_AppendString(struct afs_icl_log *logp, char *astr)
+static void
+afs_icl_AppendString(struct afs_icl_log *logp, char *astr)
 {
-    char *op;          /* ptr to char to write */
+    char *op;                  /* ptr to char to write */
     int tc;
-    register int bib;  /* bytes in buffer */
+    register int bib;          /* bytes in buffer */
 
     bib = 0;
-    op = (char *) &(logp->datap[logp->firstFree]);
+    op = (char *)&(logp->datap[logp->firstFree]);
     while (1) {
        tc = *astr++;
        *op++ = tc;
@@ -1720,11 +2064,12 @@ static afs_int32 afs_icl_AppendString(struct afs_icl_log *logp, char *astr)
            bib = 0;
            if (++(logp->firstFree) >= logp->logSize) {
                logp->firstFree = 0;
-               op = (char *) &(logp->datap[0]);
+               op = (char *)&(logp->datap[0]);
            }
            logp->logElements++;
        }
-       if (tc == 0) break;
+       if (tc == 0)
+           break;
     }
     if (bib > 0) {
        /* if we've used this word at all, allocate it */
@@ -1736,7 +2081,6 @@ static afs_int32 afs_icl_AppendString(struct afs_icl_log *logp, char *astr)
 }
 
 /* add a long to the log, ignoring overflow (checked already) */
-#if defined(AFS_ALPHA_ENV) || (defined(AFS_SGI61_ENV) && (_MIPS_SZLONG==64))
 #define ICL_APPENDINT32(lp, x) \
     MACRO_BEGIN \
         (lp)->datap[(lp)->firstFree] = (x); \
@@ -1746,6 +2090,7 @@ static afs_int32 afs_icl_AppendString(struct afs_icl_log *logp, char *astr)
         (lp)->logElements++; \
     MACRO_END
 
+#if defined(AFS_ALPHA_ENV) || (defined(AFS_SGI61_ENV) && (_MIPS_SZLONG==64)) || (defined(AFS_AIX51_ENV) && defined(AFS_64BIT_KERNEL))
 #define ICL_APPENDLONG(lp, x) \
     MACRO_BEGIN \
        ICL_APPENDINT32((lp), ((x) >> 32) & 0xffffffffL); \
@@ -1753,21 +2098,14 @@ static afs_int32 afs_icl_AppendString(struct afs_icl_log *logp, char *astr)
     MACRO_END
 
 #else /* AFS_ALPHA_ENV */
-#define ICL_APPENDLONG(lp, x) \
-    MACRO_BEGIN \
-        (lp)->datap[(lp)->firstFree] = (x); \
-       if (++((lp)->firstFree) >= (lp)->logSize) { \
-               (lp)->firstFree = 0; \
-       } \
-        (lp)->logElements++; \
-    MACRO_END
-#define ICL_APPENDINT32(lp, x) ICL_APPENDLONG((lp), (x))
+#define ICL_APPENDLONG(lp, x) ICL_APPENDINT32((lp), (x))
 #endif /* AFS_ALPHA_ENV */
 
 /* routine to tell whether we're dealing with the address or the
  * object itself
  */
-int afs_icl_UseAddr(int type)
+int
+afs_icl_UseAddr(int type)
 {
     if (type == ICL_TYPE_HYPER || type == ICL_TYPE_STRING
        || type == ICL_TYPE_FID || type == ICL_TYPE_INT64)
@@ -1781,8 +2119,9 @@ int afs_icl_UseAddr(int type)
  * pretty soon, anyway.  The log must be unlocked.
  */
 
-int afs_icl_AppendRecord(register struct afs_icl_log *logp, afs_int32 op, 
-       afs_int32 types, long p1, long p2, long p3, long p4)
+void
+afs_icl_AppendRecord(register struct afs_icl_log *logp, afs_int32 op,
+                    afs_int32 types, long p1, long p2, long p3, long p4)
 {
     int rsize;                 /* record size in longs */
     register int tsize;                /* temp size */
@@ -1798,7 +2137,7 @@ int afs_icl_AppendRecord(register struct afs_icl_log *logp, afs_int32 op,
     t1 = types & 0x3f;
 
     osi_GetTime(&tv);          /* It panics for solaris if inside */
-    ObtainWriteLock(&logp->lock,182);
+    ObtainWriteLock(&logp->lock, 182);
     if (!logp->datap) {
        ReleaseWriteLock(&logp->lock);
        return;
@@ -1808,23 +2147,24 @@ int afs_icl_AppendRecord(register struct afs_icl_log *logp, afs_int32 op,
      * change that often.  This algorithm ticks over every 20 minutes
      * or so (1000 seconds).  Write a timestamp record if it has.
      */
-    if (tv.tv_sec - logp->lastTS > 1024)
-    {
+    if (tv.tv_sec - logp->lastTS > 1024) {
        /* the timer has wrapped -- write a timestamp record */
        if (logp->logSize - logp->logElements <= 5)
            afs_icl_GetLogSpace(logp, 5);
 
-       ICL_APPENDINT32(logp, (afs_int32)(5<<24) + (ICL_TYPE_UNIXDATE<<18));
-       ICL_APPENDINT32(logp, (afs_int32)ICL_INFO_TIMESTAMP);
-       ICL_APPENDINT32(logp, (afs_int32)0); /* use thread ID zero for clocks */
        ICL_APPENDINT32(logp,
-                       (afs_int32)(tv.tv_sec & 0x3ff) * 1000000 + tv.tv_usec);
-       ICL_APPENDINT32(logp, (afs_int32)tv.tv_sec);
+                       (afs_int32) (5 << 24) + (ICL_TYPE_UNIXDATE << 18));
+       ICL_APPENDINT32(logp, (afs_int32) ICL_INFO_TIMESTAMP);
+       ICL_APPENDINT32(logp, (afs_int32) 0);   /* use thread ID zero for clocks */
+       ICL_APPENDINT32(logp,
+                       (afs_int32) (tv.tv_sec & 0x3ff) * 1000000 +
+                       tv.tv_usec);
+       ICL_APPENDINT32(logp, (afs_int32) tv.tv_sec);
 
        logp->lastTS = tv.tv_sec;
     }
 
-    rsize = 4;                         /* base case */
+    rsize = 4;                 /* base case */
     if (t1) {
        /* compute size of parameter p1.  Only tricky case is string.
         * In that case, we have to call strlen to get the string length.
@@ -1857,171 +2197,179 @@ int afs_icl_AppendRecord(register struct afs_icl_log *logp, afs_int32 op,
      */
     if (rsize > 255) {
        ReleaseWriteLock(&logp->lock);
-       return;         /* log record too big to express */
+       return;                 /* log record too big to express */
     }
 
     if (logp->logSize - logp->logElements <= rsize)
        afs_icl_GetLogSpace(logp, rsize);
 
     ICL_APPENDINT32(logp,
-                   (afs_int32)(rsize<<24) + (t1<<18) + (t2<<12) + (t3<<6) + t4);
-    ICL_APPENDINT32(logp, (afs_int32)op);
-    ICL_APPENDINT32(logp, (afs_int32)osi_ThreadUnique());
-    ICL_APPENDINT32(logp, (afs_int32)(tv.tv_sec & 0x3ff) * 1000000 + tv.tv_usec);
+                   (afs_int32) (rsize << 24) + (t1 << 18) + (t2 << 12) +
+                   (t3 << 6) + t4);
+    ICL_APPENDINT32(logp, (afs_int32) op);
+    ICL_APPENDINT32(logp, (afs_int32) osi_ThreadUnique());
+    ICL_APPENDINT32(logp,
+                   (afs_int32) (tv.tv_sec & 0x3ff) * 1000000 + tv.tv_usec);
 
     if (t1) {
        /* marshall parameter 1 now */
        if (t1 == ICL_TYPE_STRING) {
-           afs_icl_AppendString(logp, (char *) p1);
-       }
-       else if (t1 == ICL_TYPE_HYPER) {
-           ICL_APPENDINT32(logp, (afs_int32)((struct afs_hyper_t *)p1)->high);
-           ICL_APPENDINT32(logp, (afs_int32)((struct afs_hyper_t *)p1)->low);
-       }
-       else if (t1 == ICL_TYPE_INT64) {
+           afs_icl_AppendString(logp, (char *)p1);
+       } else if (t1 == ICL_TYPE_HYPER) {
+           ICL_APPENDINT32(logp,
+                           (afs_int32) ((struct afs_hyper_t *)p1)->high);
+           ICL_APPENDINT32(logp,
+                           (afs_int32) ((struct afs_hyper_t *)p1)->low);
+       } else if (t1 == ICL_TYPE_INT64) {
 #ifdef AFSLITTLE_ENDIAN
 #ifdef AFS_64BIT_CLIENT
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p1)[1]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p1)[0]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p1)[1]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p1)[0]);
 #else /* AFS_64BIT_CLIENT */
            ICL_APPENDINT32(logp, (afs_int32) p1);
            ICL_APPENDINT32(logp, (afs_int32) 0);
 #endif /* AFS_64BIT_CLIENT */
 #else /* AFSLITTLE_ENDIAN */
 #ifdef AFS_64BIT_CLIENT
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p1)[0]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p1)[1]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p1)[0]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p1)[1]);
 #else /* AFS_64BIT_CLIENT */
            ICL_APPENDINT32(logp, (afs_int32) 0);
            ICL_APPENDINT32(logp, (afs_int32) p1);
 #endif /* AFS_64BIT_CLIENT */
 #endif /* AFSLITTLE_ENDIAN */
-        }
-       else if (t1 == ICL_TYPE_FID) {
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p1)[0]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p1)[1]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p1)[2]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p1)[3]);
+       } else if (t1 == ICL_TYPE_FID) {
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p1)[0]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p1)[1]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p1)[2]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p1)[3]);
        }
-#if defined(AFS_ALPHA_ENV) || (defined(AFS_SGI61_ENV) && (_MIPS_SZLONG==64))
+#if defined(AFS_ALPHA_ENV) || (defined(AFS_SGI61_ENV) && (_MIPS_SZLONG==64)) || (defined(AFS_AIX51_ENV) && defined(AFS_64BIT_KERNEL))
        else if (t1 == ICL_TYPE_INT32)
-           ICL_APPENDINT32(logp, (afs_int32)p1);
+           ICL_APPENDINT32(logp, (afs_int32) p1);
 #endif /* AFS_ALPHA_ENV */
-       else ICL_APPENDLONG(logp, p1); 
+       else
+           ICL_APPENDLONG(logp, p1);
     }
     if (t2) {
        /* marshall parameter 2 now */
-       if (t2 == ICL_TYPE_STRING) afs_icl_AppendString(logp, (char *) p2);
+       if (t2 == ICL_TYPE_STRING)
+           afs_icl_AppendString(logp, (char *)p2);
        else if (t2 == ICL_TYPE_HYPER) {
-           ICL_APPENDINT32(logp, (afs_int32)((struct afs_hyper_t *)p2)->high);
-           ICL_APPENDINT32(logp, (afs_int32)((struct afs_hyper_t *)p2)->low);
-       }
-       else if (t2 == ICL_TYPE_INT64) {
+           ICL_APPENDINT32(logp,
+                           (afs_int32) ((struct afs_hyper_t *)p2)->high);
+           ICL_APPENDINT32(logp,
+                           (afs_int32) ((struct afs_hyper_t *)p2)->low);
+       } else if (t2 == ICL_TYPE_INT64) {
 #ifdef AFSLITTLE_ENDIAN
 #ifdef AFS_64BIT_CLIENT
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p2)[1]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p2)[0]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p2)[1]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p2)[0]);
 #else /* AFS_64BIT_CLIENT */
            ICL_APPENDINT32(logp, (afs_int32) p2);
            ICL_APPENDINT32(logp, (afs_int32) 0);
 #endif /* AFS_64BIT_CLIENT */
 #else /* AFSLITTLE_ENDIAN */
 #ifdef AFS_64BIT_CLIENT
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p2)[0]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p2)[1]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p2)[0]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p2)[1]);
 #else /* AFS_64BIT_CLIENT */
            ICL_APPENDINT32(logp, (afs_int32) 0);
            ICL_APPENDINT32(logp, (afs_int32) p2);
 #endif /* AFS_64BIT_CLIENT */
 #endif /* AFSLITTLE_ENDIAN */
-        }
-       else if (t2 == ICL_TYPE_FID) {
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p2)[0]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p2)[1]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p2)[2]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p2)[3]);
+       } else if (t2 == ICL_TYPE_FID) {
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p2)[0]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p2)[1]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p2)[2]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p2)[3]);
        }
-#if defined(AFS_ALPHA_ENV) || (defined(AFS_SGI61_ENV) && (_MIPS_SZLONG==64))
+#if defined(AFS_ALPHA_ENV) || (defined(AFS_SGI61_ENV) && (_MIPS_SZLONG==64)) || (defined(AFS_AIX51_ENV) && defined(AFS_64BIT_KERNEL))
        else if (t2 == ICL_TYPE_INT32)
-           ICL_APPENDINT32(logp, (afs_int32)p2);
+           ICL_APPENDINT32(logp, (afs_int32) p2);
 #endif /* AFS_ALPHA_ENV */
-       else ICL_APPENDLONG(logp, p2);
+       else
+           ICL_APPENDLONG(logp, p2);
     }
     if (t3) {
        /* marshall parameter 3 now */
-       if (t3 == ICL_TYPE_STRING) afs_icl_AppendString(logp, (char *) p3);
+       if (t3 == ICL_TYPE_STRING)
+           afs_icl_AppendString(logp, (char *)p3);
        else if (t3 == ICL_TYPE_HYPER) {
-           ICL_APPENDINT32(logp, (afs_int32)((struct afs_hyper_t *)p3)->high);
-           ICL_APPENDINT32(logp, (afs_int32)((struct afs_hyper_t *)p3)->low);
-       }
-       else if (t3 == ICL_TYPE_INT64) {
+           ICL_APPENDINT32(logp,
+                           (afs_int32) ((struct afs_hyper_t *)p3)->high);
+           ICL_APPENDINT32(logp,
+                           (afs_int32) ((struct afs_hyper_t *)p3)->low);
+       } else if (t3 == ICL_TYPE_INT64) {
 #ifdef AFSLITTLE_ENDIAN
 #ifdef AFS_64BIT_CLIENT
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p3)[1]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p3)[0]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p3)[1]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p3)[0]);
 #else /* AFS_64BIT_CLIENT */
            ICL_APPENDINT32(logp, (afs_int32) p3);
            ICL_APPENDINT32(logp, (afs_int32) 0);
 #endif /* AFS_64BIT_CLIENT */
 #else /* AFSLITTLE_ENDIAN */
 #ifdef AFS_64BIT_CLIENT
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p3)[0]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p3)[1]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p3)[0]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p3)[1]);
 #else /* AFS_64BIT_CLIENT */
            ICL_APPENDINT32(logp, (afs_int32) 0);
            ICL_APPENDINT32(logp, (afs_int32) p3);
 #endif /* AFS_64BIT_CLIENT */
 #endif /* AFSLITTLE_ENDIAN */
-        }
-       else if (t3 == ICL_TYPE_FID) {
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p3)[0]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p3)[1]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p3)[2]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p3)[3]);
+       } else if (t3 == ICL_TYPE_FID) {
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p3)[0]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p3)[1]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p3)[2]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p3)[3]);
        }
-#if defined(AFS_ALPHA_ENV) || (defined(AFS_SGI61_ENV) && (_MIPS_SZLONG==64))
+#if defined(AFS_ALPHA_ENV) || (defined(AFS_SGI61_ENV) && (_MIPS_SZLONG==64)) || (defined(AFS_AIX51_ENV) && defined(AFS_64BIT_KERNEL))
        else if (t3 == ICL_TYPE_INT32)
-           ICL_APPENDINT32(logp, (afs_int32)p3);
+           ICL_APPENDINT32(logp, (afs_int32) p3);
 #endif /* AFS_ALPHA_ENV */
-       else ICL_APPENDLONG(logp, p3);
+       else
+           ICL_APPENDLONG(logp, p3);
     }
     if (t4) {
        /* marshall parameter 4 now */
-       if (t4 == ICL_TYPE_STRING) afs_icl_AppendString(logp, (char *) p4);
+       if (t4 == ICL_TYPE_STRING)
+           afs_icl_AppendString(logp, (char *)p4);
        else if (t4 == ICL_TYPE_HYPER) {
-           ICL_APPENDINT32(logp, (afs_int32)((struct afs_hyper_t *)p4)->high);
-           ICL_APPENDINT32(logp, (afs_int32)((struct afs_hyper_t *)p4)->low);
-       }
-       else if (t4 == ICL_TYPE_INT64) {
+           ICL_APPENDINT32(logp,
+                           (afs_int32) ((struct afs_hyper_t *)p4)->high);
+           ICL_APPENDINT32(logp,
+                           (afs_int32) ((struct afs_hyper_t *)p4)->low);
+       } else if (t4 == ICL_TYPE_INT64) {
 #ifdef AFSLITTLE_ENDIAN
 #ifdef AFS_64BIT_CLIENT
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p4)[1]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p4)[0]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p4)[1]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p4)[0]);
 #else /* AFS_64BIT_CLIENT */
            ICL_APPENDINT32(logp, (afs_int32) p4);
            ICL_APPENDINT32(logp, (afs_int32) 0);
 #endif /* AFS_64BIT_CLIENT */
 #else /* AFSLITTLE_ENDIAN */
 #ifdef AFS_64BIT_CLIENT
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p4)[0]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p4)[1]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p4)[0]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p4)[1]);
 #else /* AFS_64BIT_CLIENT */
            ICL_APPENDINT32(logp, (afs_int32) 0);
            ICL_APPENDINT32(logp, (afs_int32) p4);
 #endif /* AFS_64BIT_CLIENT */
 #endif /* AFSLITTLE_ENDIAN */
-        }
-       else if (t4 == ICL_TYPE_FID) {
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p4)[0]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p4)[1]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p4)[2]);
-           ICL_APPENDINT32(logp, (afs_int32)((afs_int32 *)p4)[3]);
+       } else if (t4 == ICL_TYPE_FID) {
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p4)[0]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p4)[1]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p4)[2]);
+           ICL_APPENDINT32(logp, (afs_int32) ((afs_int32 *) p4)[3]);
        }
-#if defined(AFS_ALPHA_ENV) || (defined(AFS_SGI61_ENV) && (_MIPS_SZLONG==64))
+#if defined(AFS_ALPHA_ENV) || (defined(AFS_SGI61_ENV) && (_MIPS_SZLONG==64)) || (defined(AFS_AIX51_ENV) && defined(AFS_64BIT_KERNEL))
        else if (t4 == ICL_TYPE_INT32)
-           ICL_APPENDINT32(logp, (afs_int32)p4);
+           ICL_APPENDINT32(logp, (afs_int32) p4);
 #endif /* AFS_ALPHA_ENV */
-       else ICL_APPENDLONG(logp, p4);
+       else
+           ICL_APPENDLONG(logp, p4);
     }
     ReleaseWriteLock(&logp->lock);
 }
@@ -2029,31 +2377,34 @@ int afs_icl_AppendRecord(register struct afs_icl_log *logp, afs_int32 op,
 /* create a log with size logSize; return it in *outLogpp and tag
  * it with name "name."
  */
-int afs_icl_CreateLog(char *name, afs_int32 logSize, struct afs_icl_log **outLogpp)
+int
+afs_icl_CreateLog(char *name, afs_int32 logSize,
+                 struct afs_icl_log **outLogpp)
 {
-    return afs_icl_CreateLogWithFlags(name, logSize, /*flags*/0, outLogpp);
+    return afs_icl_CreateLogWithFlags(name, logSize, /*flags */ 0, outLogpp);
 }
 
 /* create a log with size logSize; return it in *outLogpp and tag
  * it with name "name."  'flags' can be set to make the log unclearable.
  */
-int afs_icl_CreateLogWithFlags(char *name, afs_int32 logSize, afs_uint32 flags, 
-       struct afs_icl_log **outLogpp)
+int
+afs_icl_CreateLogWithFlags(char *name, afs_int32 logSize, afs_uint32 flags,
+                          struct afs_icl_log **outLogpp)
 {
     register struct afs_icl_log *logp;
 
     /* add into global list under lock */
-    ObtainWriteLock(&afs_icl_lock,183);
-    if (!afs_icl_inited) afs_icl_Init();
+    ObtainWriteLock(&afs_icl_lock, 183);
+    if (!afs_icl_inited)
+       afs_icl_Init();
 
-    for (logp = afs_icl_allLogs; logp; logp=logp->nextp) {
+    for (logp = afs_icl_allLogs; logp; logp = logp->nextp) {
        if (strcmp(logp->name, name) == 0) {
            /* found it already created, just return it */
            logp->refCount++;
            *outLogpp = logp;
-           if (flags & ICL_CRLOG_FLAG_PERSISTENT)
-           {
-               ObtainWriteLock(&logp->lock,184);
+           if (flags & ICL_CRLOG_FLAG_PERSISTENT) {
+               ObtainWriteLock(&logp->lock, 184);
                logp->states |= ICL_LOGF_PERSISTENT;
                ReleaseWriteLock(&logp->lock);
            }
@@ -2061,17 +2412,17 @@ int afs_icl_CreateLogWithFlags(char *name, afs_int32 logSize, afs_uint32 flags,
            return 0;
        }
     }
-    
+
     logp = (struct afs_icl_log *)
        osi_AllocSmallSpace(sizeof(struct afs_icl_log));
-    memset((caddr_t)logp, 0, sizeof(*logp));
+    memset((caddr_t) logp, 0, sizeof(*logp));
 
     logp->refCount = 1;
-    logp->name = osi_AllocSmallSpace(strlen(name)+1);
+    logp->name = osi_AllocSmallSpace(strlen(name) + 1);
     strcpy(logp->name, name);
     LOCK_INIT(&logp->lock, "logp lock");
     logp->logSize = logSize;
-    logp->datap = NULL;        /* don't allocate it until we need it */
+    logp->datap = NULL;                /* don't allocate it until we need it */
 
     if (flags & ICL_CRLOG_FLAG_PERSISTENT)
        logp->states |= ICL_LOGF_PERSISTENT;
@@ -2090,16 +2441,17 @@ int afs_icl_CreateLogWithFlags(char *name, afs_int32 logSize, afs_uint32 flags,
  * in *flagsp.  The flag ICL_COPYOUTF_MISSEDSOME is set if we can't
  * find the record with cookie value cookie.
  */
-int afs_icl_CopyOut(register struct afs_icl_log *logp, afs_int32 *bufferp, 
-       afs_int32 *bufSizep, afs_uint32 *cookiep, afs_int32 *flagsp)
+int
+afs_icl_CopyOut(register struct afs_icl_log *logp, afs_int32 * bufferp,
+               afs_int32 * bufSizep, afs_uint32 * cookiep,
+               afs_int32 * flagsp)
 {
     afs_int32 nwords;          /* number of words to copy out */
     afs_uint32 startCookie;    /* first cookie to use */
-    register afs_int32 i;
     afs_int32 outWords;                /* words we've copied out */
     afs_int32 inWords;         /* max words to copy out */
-    afs_int32 code;                    /* return code */
-    afs_int32 ix;                      /* index we're copying from */
+    afs_int32 code;            /* return code */
+    afs_int32 ix;              /* index we're copying from */
     afs_int32 outFlags;                /* return flags */
     afs_int32 inFlags;         /* flags passed in */
     afs_int32 end;
@@ -2111,7 +2463,7 @@ int afs_icl_CopyOut(register struct afs_icl_log *logp, afs_int32 *bufferp,
     inFlags = *flagsp;
     code = 0;
 
-    ObtainWriteLock(&logp->lock,185);
+    ObtainWriteLock(&logp->lock, 185);
     if (!logp->datap) {
        ReleaseWriteLock(&logp->lock);
        goto done;
@@ -2121,7 +2473,7 @@ int afs_icl_CopyOut(register struct afs_icl_log *logp, afs_int32 *bufferp,
     while (1) {
        /* (re-)compute where we should start */
        if (startCookie < logp->baseCookie) {
-           if (startCookie)  /* missed some output */
+           if (startCookie)    /* missed some output */
                outFlags |= ICL_COPYOUTF_MISSEDSOME;
            /* skip to the first available record */
            startCookie = logp->baseCookie;
@@ -2130,14 +2482,15 @@ int afs_icl_CopyOut(register struct afs_icl_log *logp, afs_int32 *bufferp,
 
        /* compute where we find the first element to copy out */
        ix = logp->firstUsed + startCookie - logp->baseCookie;
-       if (ix >= logp->logSize) ix -= logp->logSize;
+       if (ix >= logp->logSize)
+           ix -= logp->logSize;
 
        /* if have some data now, break out and process it */
-       if (startCookie - logp->baseCookie < logp->logElements) break;
+       if (startCookie - logp->baseCookie < logp->logElements)
+           break;
 
        /* At end of log, so clear it if we need to */
-       if (inFlags & ICL_COPYOUTF_CLRAFTERREAD)
-       {
+       if (inFlags & ICL_COPYOUTF_CLRAFTERREAD) {
            logp->firstUsed = logp->firstFree = 0;
            logp->logElements = 0;
        }
@@ -2150,7 +2503,7 @@ int afs_icl_CopyOut(register struct afs_icl_log *logp, afs_int32 *bufferp,
        logp->states |= ICL_LOGF_WAITING;
        ReleaseWriteLock(&logp->lock);
        afs_osi_Sleep(&logp->lock);
-       ObtainWriteLock(&logp->lock,186);
+       ObtainWriteLock(&logp->lock, 186);
     }
     /* copy out data from ix to logSize or firstFree, depending
      * upon whether firstUsed <= firstFree (no wrap) or otherwise.
@@ -2166,7 +2519,8 @@ int afs_icl_CopyOut(register struct afs_icl_log *logp, afs_int32 *bufferp,
        if (end - ix < nwords)
            nwords = end - ix;
        if (nwords > 0) {
-           memcpy((char *) bufferp, (char *) &logp->datap[ix], sizeof(afs_int32) * nwords);
+           memcpy((char *)bufferp, (char *)&logp->datap[ix],
+                  sizeof(afs_int32) * nwords);
            outWords += nwords;
            inWords -= nwords;
            bufferp += nwords;
@@ -2177,8 +2531,8 @@ int afs_icl_CopyOut(register struct afs_icl_log *logp, afs_int32 *bufferp,
     /* now, if active part of the log has wrapped, there's more stuff
      * starting at the head of the log.  Copy out more from there.
      */
-    if (logp->firstUsed > logp->firstFree
-       && ix < logp->firstFree && inWords > 0) {
+    if (logp->firstUsed > logp->firstFree && ix < logp->firstFree
+       && inWords > 0) {
        /* (more to) copy out from the wrapped section at the
         * start of the log.  May get here even if didn't copy any
         * above, if the cookie points directly into the wrapped section.
@@ -2186,7 +2540,8 @@ int afs_icl_CopyOut(register struct afs_icl_log *logp, afs_int32 *bufferp,
        nwords = inWords;
        if (logp->firstFree - ix < nwords)
            nwords = logp->firstFree - ix;
-       memcpy((char *) bufferp, (char *) &logp->datap[ix], sizeof(afs_int32) * nwords);
+       memcpy((char *)bufferp, (char *)&logp->datap[ix],
+              sizeof(afs_int32) * nwords);
        outWords += nwords;
        inWords -= nwords;
        bufferp += nwords;
@@ -2203,8 +2558,9 @@ int afs_icl_CopyOut(register struct afs_icl_log *logp, afs_int32 *bufferp,
 }
 
 /* return basic parameter information about a log */
-int afs_icl_GetLogParms(struct afs_icl_log *logp, afs_int32 *maxSizep, 
-       afs_int32 *curSizep)
+int
+afs_icl_GetLogParms(struct afs_icl_log *logp, afs_int32 * maxSizep,
+                   afs_int32 * curSizep)
 {
     ObtainReadLock(&logp->lock);
     *maxSizep = logp->logSize;
@@ -2215,32 +2571,36 @@ int afs_icl_GetLogParms(struct afs_icl_log *logp, afs_int32 *maxSizep,
 
 
 /* hold and release logs */
-int afs_icl_LogHold(register struct afs_icl_log *logp)
+int
+afs_icl_LogHold(register struct afs_icl_log *logp)
 {
-    ObtainWriteLock(&afs_icl_lock,187);
+    ObtainWriteLock(&afs_icl_lock, 187);
     logp->refCount++;
     ReleaseWriteLock(&afs_icl_lock);
     return 0;
 }
 
 /* hold and release logs, called with lock already held */
-int afs_icl_LogHoldNL(register struct afs_icl_log *logp)
+int
+afs_icl_LogHoldNL(register struct afs_icl_log *logp)
 {
     logp->refCount++;
     return 0;
 }
 
 /* keep track of how many sets believe the log itself is allocated */
-int afs_icl_LogUse(register struct afs_icl_log *logp)
+int
+afs_icl_LogUse(register struct afs_icl_log *logp)
 {
-    ObtainWriteLock(&logp->lock,188);
+    ObtainWriteLock(&logp->lock, 188);
     if (logp->setCount == 0) {
        /* this is the first set actually using the log -- allocate it */
-       if (logp->logSize == 0) {
+       if (logp->logSize == 0) {
            /* we weren't passed in a hint and it wasn't set */
            logp->logSize = ICL_DEFAULT_LOGSIZE;
        }
-       logp->datap = (afs_int32 *) afs_osi_Alloc(sizeof(afs_int32) * logp->logSize);
+       logp->datap =
+           (afs_int32 *) afs_osi_Alloc(sizeof(afs_int32) * logp->logSize);
 #ifdef KERNEL_HAVE_PIN
        pin((char *)logp->datap, sizeof(afs_int32) * logp->logSize);
 #endif
@@ -2251,15 +2611,16 @@ int afs_icl_LogUse(register struct afs_icl_log *logp)
 }
 
 /* decrement the number of real users of the log, free if possible */
-int afs_icl_LogFreeUse(register struct afs_icl_log *logp)
+int
+afs_icl_LogFreeUse(register struct afs_icl_log *logp)
 {
-    ObtainWriteLock(&logp->lock,189);
+    ObtainWriteLock(&logp->lock, 189);
     if (--logp->setCount == 0) {
-       /* no more users -- free it (but keep log structure around)*/
-       afs_osi_Free(logp->datap, sizeof(afs_int32) * logp->logSize);
+       /* no more users -- free it (but keep log structure around) */
 #ifdef KERNEL_HAVE_PIN
        unpin((char *)logp->datap, sizeof(afs_int32) * logp->logSize);
 #endif
+       afs_osi_Free(logp->datap, sizeof(afs_int32) * logp->logSize);
        logp->firstUsed = logp->firstFree = 0;
        logp->logElements = 0;
        logp->datap = NULL;
@@ -2269,24 +2630,25 @@ int afs_icl_LogFreeUse(register struct afs_icl_log *logp)
 }
 
 /* set the size of the log to 'logSize' */
-int afs_icl_LogSetSize(register struct afs_icl_log *logp, afs_int32 logSize)
-{  
-    ObtainWriteLock(&logp->lock,190);
+int
+afs_icl_LogSetSize(register struct afs_icl_log *logp, afs_int32 logSize)
+{
+    ObtainWriteLock(&logp->lock, 190);
     if (!logp->datap) {
        /* nothing to worry about since it's not allocated */
        logp->logSize = logSize;
-    }
-    else {
+    } else {
        /* reset log */
        logp->firstUsed = logp->firstFree = 0;
        logp->logElements = 0;
 
        /* free and allocate a new one */
-       afs_osi_Free(logp->datap, sizeof(afs_int32) * logp->logSize);
 #ifdef KERNEL_HAVE_PIN
        unpin((char *)logp->datap, sizeof(afs_int32) * logp->logSize);
 #endif
-       logp->datap = (afs_int32 *) afs_osi_Alloc(sizeof(afs_int32) * logSize);
+       afs_osi_Free(logp->datap, sizeof(afs_int32) * logp->logSize);
+       logp->datap =
+           (afs_int32 *) afs_osi_Alloc(sizeof(afs_int32) * logSize);
 #ifdef KERNEL_HAVE_PIN
        pin((char *)logp->datap, sizeof(afs_int32) * logSize);
 #endif
@@ -2298,27 +2660,32 @@ int afs_icl_LogSetSize(register struct afs_icl_log *logp, afs_int32 logSize)
 }
 
 /* free a log.  Called with afs_icl_lock locked. */
-int afs_icl_ZapLog(register struct afs_icl_log *logp)
+int
+afs_icl_ZapLog(register struct afs_icl_log *logp)
 {
     register struct afs_icl_log **lpp, *tp;
 
-    for(lpp = &afs_icl_allLogs, tp = *lpp; tp; lpp = &tp->nextp, tp = *lpp) {
+    for (lpp = &afs_icl_allLogs, tp = *lpp; tp; lpp = &tp->nextp, tp = *lpp) {
        if (tp == logp) {
            /* found the dude we want to remove */
            *lpp = logp->nextp;
            osi_FreeSmallSpace(logp->name);
-           osi_FreeSmallSpace(logp->datap);
+#ifdef KERNEL_HAVE_PIN
+           unpin((char *)logp->datap, sizeof(afs_int32) * logp->logSize);
+#endif
+           afs_osi_Free(logp->datap, sizeof(afs_int32) * logp->logSize);
            osi_FreeSmallSpace(logp);
-           break;      /* won't find it twice */
+           break;              /* won't find it twice */
        }
     }
     return 0;
 }
 
 /* do the release, watching for deleted entries */
-int afs_icl_LogRele(register struct afs_icl_log *logp)
+int
+afs_icl_LogRele(register struct afs_icl_log *logp)
 {
-    ObtainWriteLock(&afs_icl_lock,191);
+    ObtainWriteLock(&afs_icl_lock, 191);
     if (--logp->refCount == 0 && (logp->states & ICL_LOGF_DELETED)) {
        afs_icl_ZapLog(logp);   /* destroys logp's lock! */
     }
@@ -2327,7 +2694,8 @@ int afs_icl_LogRele(register struct afs_icl_log *logp)
 }
 
 /* do the release, watching for deleted entries, log already held */
-int afs_icl_LogReleNL(register struct afs_icl_log *logp)
+int
+afs_icl_LogReleNL(register struct afs_icl_log *logp)
 {
     if (--logp->refCount == 0 && (logp->states & ICL_LOGF_DELETED)) {
        afs_icl_ZapLog(logp);   /* destroys logp's lock! */
@@ -2336,9 +2704,10 @@ int afs_icl_LogReleNL(register struct afs_icl_log *logp)
 }
 
 /* zero out the log */
-int afs_icl_ZeroLog(register struct afs_icl_log *logp)
+int
+afs_icl_ZeroLog(register struct afs_icl_log *logp)
 {
-    ObtainWriteLock(&logp->lock,192);
+    ObtainWriteLock(&logp->lock, 192);
     logp->firstUsed = logp->firstFree = 0;
     logp->logElements = 0;
     logp->baseCookie = 0;
@@ -2347,9 +2716,10 @@ int afs_icl_ZeroLog(register struct afs_icl_log *logp)
 }
 
 /* free a log entry, and drop its reference count */
-int afs_icl_LogFree(register struct afs_icl_log *logp)
+int
+afs_icl_LogFree(register struct afs_icl_log *logp)
 {
-    ObtainWriteLock(&logp->lock,193);
+    ObtainWriteLock(&logp->lock, 193);
     logp->states |= ICL_LOGF_DELETED;
     ReleaseWriteLock(&logp->lock);
     afs_icl_LogRele(logp);
@@ -2357,11 +2727,12 @@ int afs_icl_LogFree(register struct afs_icl_log *logp)
 }
 
 /* find a log by name, returning it held */
-struct afs_icl_log *afs_icl_FindLog(char *name)
+struct afs_icl_log *
+afs_icl_FindLog(char *name)
 {
     register struct afs_icl_log *tp;
-    ObtainWriteLock(&afs_icl_lock,194);
-    for(tp = afs_icl_allLogs; tp; tp=tp->nextp) {
+    ObtainWriteLock(&afs_icl_lock, 194);
+    for (tp = afs_icl_allLogs; tp; tp = tp->nextp) {
        if (strcmp(tp->name, name) == 0) {
            /* this is the dude we want */
            tp->refCount++;
@@ -2372,23 +2743,27 @@ struct afs_icl_log *afs_icl_FindLog(char *name)
     return tp;
 }
 
-int afs_icl_EnumerateLogs(int (*aproc)(), char *arock)
+int
+afs_icl_EnumerateLogs(int (*aproc)
+                       (char *name, char *arock, struct afs_icl_log * tp),
+                     char *arock)
 {
     register struct afs_icl_log *tp;
     register afs_int32 code;
 
     code = 0;
-    ObtainWriteLock(&afs_icl_lock,195);
-    for(tp = afs_icl_allLogs; tp; tp=tp->nextp) {
-       tp->refCount++; /* hold this guy */
+    ObtainWriteLock(&afs_icl_lock, 195);
+    for (tp = afs_icl_allLogs; tp; tp = tp->nextp) {
+       tp->refCount++;         /* hold this guy */
        ReleaseWriteLock(&afs_icl_lock);
        ObtainReadLock(&tp->lock);
-       code = (*aproc)(tp->name, arock, tp);
+       code = (*aproc) (tp->name, arock, tp);
        ReleaseReadLock(&tp->lock);
-       ObtainWriteLock(&afs_icl_lock,196);
+       ObtainWriteLock(&afs_icl_lock, 196);
        if (--tp->refCount == 0)
            afs_icl_ZapLog(tp);
-       if (code) break;
+       if (code)
+           break;
     }
     ReleaseWriteLock(&afs_icl_lock);
     return code;
@@ -2396,11 +2771,13 @@ int afs_icl_EnumerateLogs(int (*aproc)(), char *arock)
 
 struct afs_icl_set *afs_icl_allSets = 0;
 
-int afs_icl_CreateSet(char *name, struct afs_icl_log *baseLogp, 
-       struct afs_icl_log *fatalLogp, struct afs_icl_set **outSetpp)
+int
+afs_icl_CreateSet(char *name, struct afs_icl_log *baseLogp,
+                 struct afs_icl_log *fatalLogp,
+                 struct afs_icl_set **outSetpp)
 {
     return afs_icl_CreateSetWithFlags(name, baseLogp, fatalLogp,
-                                     /*flags*/0, outSetpp);
+                                     /*flags */ 0, outSetpp);
 }
 
 /* create a set, given pointers to base and fatal logs, if any.
@@ -2409,23 +2786,25 @@ int afs_icl_CreateSet(char *name, struct afs_icl_log *baseLogp,
  * addds references from the new afs_icl_set.  When the set is destroyed,
  * those references will be released.
  */
-int afs_icl_CreateSetWithFlags(char *name, struct afs_icl_log *baseLogp, 
-       struct afs_icl_log *fatalLogp, afs_uint32 flags, struct afs_icl_set **outSetpp)
+int
+afs_icl_CreateSetWithFlags(char *name, struct afs_icl_log *baseLogp,
+                          struct afs_icl_log *fatalLogp, afs_uint32 flags,
+                          struct afs_icl_set **outSetpp)
 {
     register struct afs_icl_set *setp;
     register int i;
     afs_int32 states = ICL_DEFAULT_SET_STATES;
 
-    ObtainWriteLock(&afs_icl_lock,197);
-    if (!afs_icl_inited) afs_icl_Init();
+    ObtainWriteLock(&afs_icl_lock, 197);
+    if (!afs_icl_inited)
+       afs_icl_Init();
 
     for (setp = afs_icl_allSets; setp; setp = setp->nextp) {
        if (strcmp(setp->name, name) == 0) {
            setp->refCount++;
            *outSetpp = setp;
-           if (flags & ICL_CRSET_FLAG_PERSISTENT)
-           {
-               ObtainWriteLock(&setp->lock,198);
+           if (flags & ICL_CRSET_FLAG_PERSISTENT) {
+               ObtainWriteLock(&setp->lock, 198);
                setp->states |= ICL_SETF_PERSISTENT;
                ReleaseWriteLock(&setp->lock);
            }
@@ -2433,7 +2812,7 @@ int afs_icl_CreateSetWithFlags(char *name, struct afs_icl_log *baseLogp,
            return 0;
        }
     }
-    
+
     /* determine initial state */
     if (flags & ICL_CRSET_FLAG_DEFAULT_ON)
        states = ICL_SETF_ACTIVE;
@@ -2442,8 +2821,8 @@ int afs_icl_CreateSetWithFlags(char *name, struct afs_icl_log *baseLogp,
     if (flags & ICL_CRSET_FLAG_PERSISTENT)
        states |= ICL_SETF_PERSISTENT;
 
-    setp = (struct afs_icl_set *) afs_osi_Alloc(sizeof(struct afs_icl_set));
-    memset((caddr_t)setp, 0, sizeof(*setp));
+    setp = (struct afs_icl_set *)afs_osi_Alloc(sizeof(struct afs_icl_set));
+    memset((caddr_t) setp, 0, sizeof(*setp));
     setp->refCount = 1;
     if (states & ICL_SETF_FREED)
        states &= ~ICL_SETF_ACTIVE;     /* if freed, can't be active */
@@ -2454,15 +2833,15 @@ int afs_icl_CreateSetWithFlags(char *name, struct afs_icl_log *baseLogp,
      * it doesn't matter, since no one can find this lock yet, since
      * the afs_icl_lock is still held, and thus the obtain can't block.
      */
-    ObtainWriteLock(&setp->lock,199);
-    setp->name = osi_AllocSmallSpace(strlen(name)+1);
+    ObtainWriteLock(&setp->lock, 199);
+    setp->name = osi_AllocSmallSpace(strlen(name) + 1);
     strcpy(setp->name, name);
     setp->nevents = ICL_DEFAULTEVENTS;
     setp->eventFlags = afs_osi_Alloc(ICL_DEFAULTEVENTS);
 #ifdef KERNEL_HAVE_PIN
     pin((char *)setp->eventFlags, ICL_DEFAULTEVENTS);
 #endif
-    for(i=0; i<ICL_DEFAULTEVENTS; i++)
+    for (i = 0; i < ICL_DEFAULTEVENTS; i++)
        setp->eventFlags[i] = 0xff;     /* default to enabled */
 
     /* update this global info under the afs_icl_lock */
@@ -2490,11 +2869,12 @@ int afs_icl_CreateSetWithFlags(char *name, struct afs_icl_log *baseLogp,
 }
 
 /* function to change event enabling information for a particular set */
-int afs_icl_SetEnable(struct afs_icl_set *setp, afs_int32 eventID, int setValue)
+int
+afs_icl_SetEnable(struct afs_icl_set *setp, afs_int32 eventID, int setValue)
 {
     char *tp;
 
-    ObtainWriteLock(&setp->lock,200);
+    ObtainWriteLock(&setp->lock, 200);
     if (!ICL_EVENTOK(setp, eventID)) {
        ReleaseWriteLock(&setp->lock);
        return -1;
@@ -2512,8 +2892,8 @@ int afs_icl_SetEnable(struct afs_icl_set *setp, afs_int32 eventID, int setValue)
  * for tracing.  If *getValuep is set to 0, the event is disabled,
  * otherwise it is enabled.  All events start out enabled by default.
  */
-int afs_icl_GetEnable(struct afs_icl_set *setp, afs_int32 eventID, 
-       int *getValuep)
+int
+afs_icl_GetEnable(struct afs_icl_set *setp, afs_int32 eventID, int *getValuep)
 {
     ObtainReadLock(&setp->lock);
     if (!ICL_EVENTOK(setp, eventID)) {
@@ -2529,45 +2909,48 @@ int afs_icl_GetEnable(struct afs_icl_set *setp, afs_int32 eventID,
 }
 
 /* hold and release event sets */
-int afs_icl_SetHold(register struct afs_icl_set *setp)
+int
+afs_icl_SetHold(register struct afs_icl_set *setp)
 {
-    ObtainWriteLock(&afs_icl_lock,201);
+    ObtainWriteLock(&afs_icl_lock, 201);
     setp->refCount++;
     ReleaseWriteLock(&afs_icl_lock);
     return 0;
 }
 
 /* free a set.  Called with afs_icl_lock locked */
-int afs_icl_ZapSet(register struct afs_icl_set *setp)
+int
+afs_icl_ZapSet(register struct afs_icl_set *setp)
 {
     register struct afs_icl_set **lpp, *tp;
     int i;
     register struct afs_icl_log *tlp;
 
-    for(lpp = &afs_icl_allSets, tp = *lpp; tp; lpp = &tp->nextp, tp = *lpp) {
+    for (lpp = &afs_icl_allSets, tp = *lpp; tp; lpp = &tp->nextp, tp = *lpp) {
        if (tp == setp) {
            /* found the dude we want to remove */
            *lpp = setp->nextp;
            osi_FreeSmallSpace(setp->name);
-           afs_osi_Free(setp->eventFlags, ICL_DEFAULTEVENTS);
 #ifdef KERNEL_HAVE_PIN
            unpin((char *)setp->eventFlags, ICL_DEFAULTEVENTS);
 #endif
-           for(i=0; i < ICL_LOGSPERSET; i++) {
+           afs_osi_Free(setp->eventFlags, ICL_DEFAULTEVENTS);
+           for (i = 0; i < ICL_LOGSPERSET; i++) {
                if ((tlp = setp->logs[i]))
                    afs_icl_LogReleNL(tlp);
            }
            osi_FreeSmallSpace(setp);
-           break;      /* won't find it twice */
+           break;              /* won't find it twice */
        }
     }
     return 0;
 }
 
 /* do the release, watching for deleted entries */
-int afs_icl_SetRele(register struct afs_icl_set *setp)
+int
+afs_icl_SetRele(register struct afs_icl_set *setp)
 {
-    ObtainWriteLock(&afs_icl_lock,202);
+    ObtainWriteLock(&afs_icl_lock, 202);
     if (--setp->refCount == 0 && (setp->states & ICL_SETF_DELETED)) {
        afs_icl_ZapSet(setp);   /* destroys setp's lock! */
     }
@@ -2576,9 +2959,10 @@ int afs_icl_SetRele(register struct afs_icl_set *setp)
 }
 
 /* free a set entry, dropping its reference count */
-int afs_icl_SetFree(register struct afs_icl_set *setp)
+int
+afs_icl_SetFree(register struct afs_icl_set *setp)
 {
-    ObtainWriteLock(&setp->lock,203);
+    ObtainWriteLock(&setp->lock, 203);
     setp->states |= ICL_SETF_DELETED;
     ReleaseWriteLock(&setp->lock);
     afs_icl_SetRele(setp);
@@ -2586,11 +2970,12 @@ int afs_icl_SetFree(register struct afs_icl_set *setp)
 }
 
 /* find a set by name, returning it held */
-struct afs_icl_set *afs_icl_FindSet(char *name)
+struct afs_icl_set *
+afs_icl_FindSet(char *name)
 {
     register struct afs_icl_set *tp;
-    ObtainWriteLock(&afs_icl_lock,204);
-    for(tp = afs_icl_allSets; tp; tp=tp->nextp) {
+    ObtainWriteLock(&afs_icl_lock, 204);
+    for (tp = afs_icl_allSets; tp; tp = tp->nextp) {
        if (strcmp(tp->name, name) == 0) {
            /* this is the dude we want */
            tp->refCount++;
@@ -2602,20 +2987,22 @@ struct afs_icl_set *afs_icl_FindSet(char *name)
 }
 
 /* zero out all the logs in the set */
-int afs_icl_ZeroSet(struct afs_icl_set *setp)
+int
+afs_icl_ZeroSet(struct afs_icl_set *setp)
 {
     register int i;
     int code = 0;
     int tcode;
     struct afs_icl_log *logp;
-    
+
     ObtainReadLock(&setp->lock);
-    for(i = 0; i < ICL_LOGSPERSET; i++) {
+    for (i = 0; i < ICL_LOGSPERSET; i++) {
        logp = setp->logs[i];
        if (logp) {
            afs_icl_LogHold(logp);
            tcode = afs_icl_ZeroLog(logp);
-           if (tcode != 0) code = tcode;       /* save the last bad one */
+           if (tcode != 0)
+               code = tcode;   /* save the last bad one */
            afs_icl_LogRele(logp);
        }
     }
@@ -2623,35 +3010,39 @@ int afs_icl_ZeroSet(struct afs_icl_set *setp)
     return code;
 }
 
-int afs_icl_EnumerateSets(int (*aproc)(), char *arock)
+int
+afs_icl_EnumerateSets(int (*aproc)
+                       (char *name, char *arock, struct afs_icl_log * tp),
+                     char *arock)
 {
     register struct afs_icl_set *tp, *np;
     register afs_int32 code;
 
     code = 0;
-    ObtainWriteLock(&afs_icl_lock,205);
-    for(tp = afs_icl_allSets; tp; tp=np) {
-       tp->refCount++; /* hold this guy */
+    ObtainWriteLock(&afs_icl_lock, 205);
+    for (tp = afs_icl_allSets; tp; tp = np) {
+       tp->refCount++;         /* hold this guy */
        ReleaseWriteLock(&afs_icl_lock);
-       code = (*aproc)(tp->name, arock, tp);
-       ObtainWriteLock(&afs_icl_lock,206);
-       np = tp->nextp; /* tp may disappear next, but not np */
+       code = (*aproc) (tp->name, arock, (struct afs_icl_log *)tp);
+       ObtainWriteLock(&afs_icl_lock, 206);
+       np = tp->nextp;         /* tp may disappear next, but not np */
        if (--tp->refCount == 0 && (tp->states & ICL_SETF_DELETED))
            afs_icl_ZapSet(tp);
-       if (code) break;
+       if (code)
+           break;
     }
     ReleaseWriteLock(&afs_icl_lock);
     return code;
 }
 
-int afs_icl_AddLogToSet(struct afs_icl_set *setp, struct afs_icl_log *newlogp)
+int
+afs_icl_AddLogToSet(struct afs_icl_set *setp, struct afs_icl_log *newlogp)
 {
     register int i;
     int code = -1;
-    struct afs_icl_log *logp;
-    
-    ObtainWriteLock(&setp->lock,207);
-    for(i = 0; i < ICL_LOGSPERSET; i++) {
+
+    ObtainWriteLock(&setp->lock, 207);
+    for (i = 0; i < ICL_LOGSPERSET; i++) {
        if (!setp->logs[i]) {
            setp->logs[i] = newlogp;
            code = i;
@@ -2667,14 +3058,15 @@ int afs_icl_AddLogToSet(struct afs_icl_set *setp, struct afs_icl_log *newlogp)
     return code;
 }
 
-int afs_icl_SetSetStat(struct afs_icl_set *setp, int op)
+int
+afs_icl_SetSetStat(struct afs_icl_set *setp, int op)
 {
     int i;
     afs_int32 code;
     struct afs_icl_log *logp;
 
-    ObtainWriteLock(&setp->lock,208);
-    switch(op) {
+    ObtainWriteLock(&setp->lock, 208);
+    switch (op) {
     case ICL_OP_SS_ACTIVATE:   /* activate a log */
        /*
         * If we are not already active, see if we have released
@@ -2684,7 +3076,7 @@ int afs_icl_SetSetStat(struct afs_icl_set *setp, int op)
        if (!(setp->states & ICL_SETF_ACTIVE)) {
            if (setp->states & ICL_SETF_FREED) {
                /* have to reassert desire for logs */
-               for(i = 0; i < ICL_LOGSPERSET; i++) {
+               for (i = 0; i < ICL_LOGSPERSET; i++) {
                    logp = setp->logs[i];
                    if (logp) {
                        afs_icl_LogHold(logp);
@@ -2714,7 +3106,7 @@ int afs_icl_SetSetStat(struct afs_icl_set *setp, int op)
            code = EINVAL;
        else {
            if (!(setp->states & ICL_SETF_FREED)) {
-               for(i = 0; i < ICL_LOGSPERSET; i++) {
+               for (i = 0; i < ICL_LOGSPERSET; i++) {
                    logp = setp->logs[i];
                    if (logp) {
                        afs_icl_LogHold(logp);
@@ -2734,4 +3126,3 @@ int afs_icl_SetSetStat(struct afs_icl_set *setp, int op)
     ReleaseWriteLock(&setp->lock);
     return code;
 }
-