]> git.michaelhowe.org Git - packages/o/openafs.git/commitdiff
solaris-vnop-prototyping-20081213
authorDerrick Brashear <shadow@dementia.org>
Sat, 13 Dec 2008 21:30:55 +0000 (21:30 +0000)
committerDerrick Brashear <shadow@dementia.org>
Sat, 13 Dec 2008 21:30:55 +0000 (21:30 +0000)
LICENSE IPL10

make the head sane on solaris again

src/afs/SOLARIS/osi_vnodeops.c

index c3e7007cba7485f0148029cb5aab3deb94a28cf6..d5f5d27111beecaf6d1d0ac288e83c431de5ca95 100644 (file)
@@ -55,27 +55,6 @@ RCSID
 #include <sys/debug.h>
 #include <sys/fs_subr.h>
 
-/* 
- * XXX Temporary fix for problems with Solaris rw_tryupgrade() lock.
- * It isn't very persistent in getting the upgrade when others are
- * waiting for it and returns 0.  So the UpgradeSToW() macro that the
- * rw_tryupgrade used to map to wasn't good enough and we need to use
- * the following code instead.  Obviously this isn't the proper place
- * for it but it's only called from here for now
- * 
- */
-#ifndef        AFS_SUN54_ENV
-AFS_TRYUP(lock)
-     afs_rwlock_t *lock;
-{
-    if (!rw_tryupgrade(lock)) {
-       rw_exit(lock);
-       rw_enter(lock, RW_WRITER);
-    }
-}
-#endif
-
-
 /* Translate a faultcode_t as returned by some of the vm routines
  * into a suitable errno value.
  */
@@ -106,26 +85,18 @@ int afs_dump(), afs_cmp(), afs_realvp(), afs_GetOnePage();
 int afs_pvn_vptrunc;
 
 int
-afs_addmap(avp, offset, asp, addr, length, prot, maxprot, flags, credp)
-     register struct vnode *avp;
-     offset_t offset;
-     struct as *asp;
-     caddr_t addr;
-     int length, prot, maxprot, flags;
-     struct AFS_UCRED *credp;
+afs_addmap(register struct vnode *avp, offset_t offset, struct as *asp, 
+          caddr_t addr, int length, int prot, int maxprot, int flags, 
+          struct AFS_UCRED *credp)
 {
     /* XXX What should we do here?? XXX */
     return (0);
 }
 
 int
-afs_delmap(avp, offset, asp, addr, length, prot, maxprot, flags, credp)
-     register struct vnode *avp;
-     offset_t offset;
-     struct as *asp;
-     caddr_t addr;
-     int length, prot, maxprot, flags;
-     struct AFS_UCRED *credp;
+afs_delmap(register struct vnode *avp, offset_t offset, struct as *asp, 
+          caddr_t addr, int length, int prot, int maxprot, int flags, 
+          struct AFS_UCRED *credp)
 {
     /* XXX What should we do here?? XXX */
     return (0);
@@ -133,19 +104,12 @@ afs_delmap(avp, offset, asp, addr, length, prot, maxprot, flags, credp)
 
 #ifdef AFS_SUN510_ENV
 int
-afs_vmread(avp, auio, ioflag, acred, ct)
-     register struct vnode *avp;
-     struct uio *auio;
-     int ioflag;
-     struct AFS_UCRED *acred;
-     caller_context_t *ct;
+afs_vmread(register struct vnode *avp, struct uio *auio, int ioflag, 
+          struct AFS_UCRED *acred, caller_context_t *ct)
 #else
 int
-afs_vmread(avp, auio, ioflag, acred)
-     register struct vnode *avp;
-     struct uio *auio;
-     int ioflag;
-     struct AFS_UCRED *acred;
+afs_vmread(register struct vnode *avp, struct uio *auio, int ioflag, 
+          struct AFS_UCRED *acred)
 #endif
 {
     register int code;
@@ -161,19 +125,12 @@ afs_vmread(avp, auio, ioflag, acred)
 
 #ifdef AFS_SUN510_ENV
 int
-afs_vmwrite(avp, auio, ioflag, acred, ct)
-     register struct vnode *avp;
-     struct uio *auio;
-     int ioflag;
-     struct AFS_UCRED *acred;
-     caller_context_t *ct;
+afs_vmwrite(register struct vnode *avp, struct uio *auio, int ioflag, 
+           struct AFS_UCRED *acred, caller_context_t *ct)
 #else
 int
-afs_vmwrite(avp, auio, ioflag, acred)
-     register struct vnode *avp;
-     struct uio *auio;
-     int ioflag;
-     struct AFS_UCRED *acred;
+afs_vmwrite(register struct vnode *avp, struct uio *auio, int ioflag, 
+           struct AFS_UCRED *acred)
 #endif
 {
     register int code;
@@ -187,17 +144,9 @@ afs_vmwrite(avp, auio, ioflag, acred)
 }
 
 int
-afs_getpage(vp, off, len, protp, pl, plsz, seg, addr, rw, acred)
-     struct vnode *vp;
-     u_int len;
-     u_int *protp;
-     struct page *pl[];
-     u_int plsz;
-     struct seg *seg;
-     offset_t off;
-     caddr_t addr;
-     enum seg_rw rw;
-     struct AFS_UCRED *acred;
+afs_getpage(struct vnode *vp, offset_t off, u_int len, u_int *protp, 
+           struct page *pl[], u_int plsz, struct seg *seg, caddr_t addr, 
+           enum seg_rw rw, struct AFS_UCRED *acred)
 {
     register afs_int32 code = 0;
     AFS_STATCNT(afs_getpage);
@@ -207,32 +156,25 @@ afs_getpage(vp, off, len, protp, pl, plsz, seg, addr, rw, acred)
 
     AFS_GLOCK();
 
-#if    defined(AFS_SUN56_ENV)
-    if (len <= PAGESIZE)
-       code =
-           afs_GetOnePage(vp, off, len, protp, pl, plsz, seg, addr, rw,
-                          acred);
-#else
     if (len <= PAGESIZE)
        code =
-           afs_GetOnePage(vp, (u_int) off, len, protp, pl, plsz, seg, addr,
-                          rw, acred);
+           afs_GetOnePage(vp, 
+#if !defined(AFS_SUN56_ENV)
+                          (u_int)
 #endif
+                          off, len, protp, pl, plsz, seg, addr, rw, acred);
     else {
        struct vcache *vcp = VTOAFS(vp);
        ObtainWriteLock(&vcp->vlock, 548);
        vcp->multiPage++;
        ReleaseWriteLock(&vcp->vlock);
        afs_BozonLock(&vcp->pvnLock, vcp);
-#if    defined(AFS_SUN56_ENV)
-       code =
-           pvn_getpages(afs_GetOnePage, vp, off, len, protp, pl, plsz, seg,
-                        addr, rw, acred);
-#else
        code =
-           pvn_getpages(afs_GetOnePage, vp, (u_int) off, len, protp, pl,
-                        plsz, seg, addr, rw, acred);
+           pvn_getpages(afs_GetOnePage, vp, 
+#if !defined(AFS_SUN56_ENV)
+                        (u_int)
 #endif
+                        off, len, protp, pl, plsz, seg, addr, rw, acred);
        afs_BozonUnlock(&vcp->pvnLock, vcp);
        ObtainWriteLock(&vcp->vlock, 549);
        vcp->multiPage--;
@@ -244,21 +186,15 @@ afs_getpage(vp, off, len, protp, pl, plsz, seg, addr, rw, acred)
 
 /* Return all the pages from [off..off+len) in file */
 int
-afs_GetOnePage(vp, off, alen, protp, pl, plsz, seg, addr, rw, acred)
-     u_int alen;
-     struct vnode *vp;
 #if    defined(AFS_SUN56_ENV)
-     u_offset_t off;
+afs_GetOnePage(struct vnode *vp, u_offset_t off, u_int alen, u_int *protp, 
+              struct page *pl[], u_int plsz, struct seg *seg, caddr_t addr, 
+              enum seg_rw rw, struct AFS_UCRED *acred)
 #else
-     u_int off;
+afs_GetOnePage(struct vnode *vp, u_int off, u_int alen, u_int *protp, 
+              struct page *pl[], u_int plsz, struct seg *seg, caddr_t addr, 
+              enum seg_rw rw, struct AFS_UCRED *acred)
 #endif
-     u_int *protp;
-     struct page *pl[];
-     u_int plsz;
-     struct seg *seg;
-     caddr_t addr;
-     enum seg_rw rw;
-     struct AFS_UCRED *acred;
 {
     register struct page *page;
     register afs_int32 code = 0;
@@ -544,12 +480,8 @@ afs_GetOnePage(vp, off, alen, protp, pl, plsz, seg, addr, rw, acred)
 }
 
 int
-afs_putpage(vp, off, len, flags, cred)
-     struct vnode *vp;
-     offset_t off;
-     u_int len;
-     int flags;
-     struct AFS_UCRED *cred;
+afs_putpage(struct vnode *vp, offset_t off, u_int len, int flags, 
+           struct AFS_UCRED *cred)
 {
     struct vcache *avc;
     struct page *pages;
@@ -652,18 +584,16 @@ afs_putpage(vp, off, len, flags, cred)
 
 
 int
-afs_putapage(struct vnode *vp, struct page *pages,
-#if    defined(AFS_SUN56_ENV)
-            u_offset_t * offp,
-#else
-            u_int * offp,
-#endif
-#if    defined(AFS_SUN58_ENV)
-            size_t * lenp,
+#if defined(AFS_SUN58_ENV)
+afs_putapage(struct vnode *vp, struct page *pages, u_offset_t * offp,
+            size_t * lenp, int flags, struct AFS_UCRED *credp)
+#elif defined(AFS_SUN56_ENV)
+afs_putapage(struct vnode *vp, struct page *pages, u_offset_t * offp,
+            u_int * lenp, int flags, struct AFS_UCRED *credp)
 #else
-            u_int * lenp,
+afs_putapage(struct vnode *vp, struct page *pages, u_int * offp,
+            u_int * lenp, int flags, struct AFS_UCRED *credp)
 #endif
-            int flags, struct AFS_UCRED *credp)
 {
     struct buf *tbuf;
     struct vcache *avc = VTOAFS(vp);
@@ -717,12 +647,8 @@ afs_putapage(struct vnode *vp, struct page *pages,
 }
 
 int
-afs_nfsrdwr(avc, auio, arw, ioflag, acred)
-     register struct vcache *avc;
-     struct uio *auio;
-     enum uio_rw arw;
-     int ioflag;
-     struct AFS_UCRED *acred;
+afs_nfsrdwr(register struct vcache *avc, struct uio *auio, enum uio_rw arw,
+           int ioflag, struct AFS_UCRED *acred)
 {
     register afs_int32 code;
     afs_int32 code2;
@@ -1084,15 +1010,8 @@ afs_nfsrdwr(avc, auio, arw, ioflag, acred)
     return afs_CheckCode(code, &treq, 46);
 }
 
-afs_map(vp, off, as, addr, len, prot, maxprot, flags, cred)
-     struct vnode *vp;
-     struct as *as;
-     offset_t off;
-     caddr_t *addr;
-     u_int len;
-     u_char prot, maxprot;
-     u_int flags;
-     struct AFS_UCRED *cred;
+int
+afs_map(struct vnode *vp, offset_t off, struct as *as, caddr_t *addr, u_int len, u_char prot, u_char maxprot, u_int flags, struct AFS_UCRED *cred)
 {
     struct segvn_crargs crargs;
     register afs_int32 code;
@@ -1182,11 +1101,9 @@ afs_map(vp, off, as, addr, len, prot, maxprot, flags, cred)
  * For Now We use standard local kernel params for AFS system values. Change this
  * at some point.
  */
-afs_pathconf(vp, cmd, outdatap, credp)
-     register struct AFS_UCRED *credp;
-     struct vnode *vp;
-     int cmd;
-     u_long *outdatap;
+int
+afs_pathconf(struct vnode *vp, int cmd, u_long *outdatap, 
+            register struct AFS_UCRED *credp)
 {
     AFS_STATCNT(afs_cntl);
     switch (cmd) {
@@ -1211,38 +1128,30 @@ afs_pathconf(vp, cmd, outdatap, credp)
     return 0;
 }
 
-afs_ioctl(vnp, com, arg, flag, credp, rvalp)
-     struct vnode *vnp;
-     int com, arg, flag;
-     cred_t *credp;
-     int *rvalp;
+int
+afs_ioctl(struct vnode *vnp, int com, int arg, int flag, cred_t *credp, 
+         int *rvalp)
 {
     return (ENOTTY);
 }
 
 void
-afs_rwlock(vnp, wlock)
-     struct vnode *vnp;
-     int wlock;
+afs_rwlock(struct vnode *vnp, int wlock)
 {
     rw_enter(&(VTOAFS(vnp))->rwlock, (wlock ? RW_WRITER : RW_READER));
 }
 
 
 void
-afs_rwunlock(vnp, wlock)
-     struct vnode *vnp;
-     int wlock;
+afs_rwunlock(struct vnode *vnp, int wlock)
 {
     rw_exit(&(VTOAFS(vnp))->rwlock);
 }
 
 
 /* NOT SUPPORTED */
-afs_seek(vnp, ooff, noffp)
-     struct vnode *vnp;
-     offset_t ooff;
-     offset_t *noffp;
+int
+afs_seek(struct vnode *vnp, offset_t ooff, offset_t *noffp)
 {
     register int code = 0;
 
@@ -1258,24 +1167,16 @@ afs_seek(vnp, ooff, noffp)
 }
 
 int
-afs_frlock(vnp, cmd, ap, flag, off,
 #ifdef AFS_SUN59_ENV
-          flkcb,
-#endif
-          credp)
-     struct vnode *vnp;
-     int cmd;
-#if    defined(AFS_SUN56_ENV)
-     struct flock64 *ap;
+afs_frlock(struct vnode *vnp, int cmd, struct flock64 *ap, int flag, 
+          offset_t off, struct flk_callback *flkcb, struct AFS_UCRED *credp)
+#elif defined(AFS_SUN56_ENV)
+afs_frlock(struct vnode *vnp, int cmd, struct flock64 *ap, int flag, 
+          offset_t off, struct AFS_UCRED *credp)
 #else
-     struct flock *ap;
-#endif
-     int flag;
-     offset_t off;
-#ifdef AFS_SUN59_ENV
-     struct flk_callback *flkcb;
+afs_frlock(struct vnode *vnp, int cmd, struct flock *ap, int flag, 
+          offset_t off, struct AFS_UCRED *credp)
 #endif
-     struct AFS_UCRED *credp;
 {
     register afs_int32 code = 0;
     /*
@@ -1314,17 +1215,13 @@ afs_frlock(vnp, cmd, ap, flag, off,
 
 
 int
-afs_space(vnp, cmd, ap, flag, off, credp)
-     struct vnode *vnp;
-     int cmd;
 #if    defined(AFS_SUN56_ENV)
-     struct flock64 *ap;
+afs_space(struct vnode *vnp, int cmd, struct flock64 *ap, int flag, 
+         offset_t off, struct AFS_UCRED *credp)
 #else
-     struct flock *ap;
+afs_space(struct vnode *vnp, int cmd, struct flock *ap, int flag, 
+         offset_t off, struct AFS_UCRED *credp)
 #endif
-     int flag;
-     offset_t off;
-     struct AFS_UCRED *credp;
 {
     register afs_int32 code = EINVAL;
     struct vattr vattr;
@@ -1347,10 +1244,7 @@ afs_space(vnp, cmd, ap, flag, off, credp)
 }
 
 int
-afs_dump(vp, addr, i1, i2)
-     struct vnode *vp;
-     caddr_t addr;
-     int i1, i2;
+afs_dump(struct vnode *vp, caddr_t addr, int i1, int i2)
 {
     AFS_STATCNT(afs_dump);
     afs_warn("AFS_DUMP. MUST IMPLEMENT THIS!!!\n");
@@ -1359,8 +1253,8 @@ afs_dump(vp, addr, i1, i2)
 
 
 /* Nothing fancy here; just compare if vnodes are identical ones */
-afs_cmp(vp1, vp2)
-     struct vnode *vp1, *vp2;
+int
+afs_cmp(struct vnode *vp1, struct vnode *vp2)
 {
     AFS_STATCNT(afs_cmp);
     return (vp1 == vp2);
@@ -1376,27 +1270,18 @@ afs_realvp(struct vnode *vp, struct vnode **vpp)
 
 
 int
-afs_pageio(vp, pp, ui1, ui2, i1, credp)
-     struct vnode *vp;
-     struct page *pp;
-     u_int ui1, ui2;
-     int i1;
-     struct cred *credp;
+afs_pageio(struct vnode *vp, struct page *pp, u_int ui1, u_int ui2, int i1, 
+          struct cred *credp)
 {
     afs_warn("afs_pageio: Not implemented\n");
     return EINVAL;
 }
 
 int
-afs_dumpctl(vp, i
-#ifdef AFS_SUN59_ENV
-           , blkp
-#endif
-    )
-     struct vnode *vp;
-     int i;
 #ifdef AFS_SUN59_ENV
-     int *blkp;
+afs_dumpctl(struct vnode *vp, int i, int *blkp)
+#else
+afs_dumpctl(struct vnode *vp, int i)
 #endif
 {
     afs_warn("afs_dumpctl: Not implemented\n");
@@ -1424,31 +1309,20 @@ afs_getsecattr(struct vnode *vp, vsecattr_t *vsecattr, int flag, struct cred *cr
 }
 #else
 extern void
-afs_dispose(vp, p, fl, dn, cr)
-     struct vnode *vp;
-     struct page *p;
-     int fl, dn;
-     struct cred *cr;
+afs_dispose(struct vnode *vp, struct page *p, int fl, int dn, struct cred *cr)
 {
     fs_dispose(vp, p, fl, dn, cr);
 }
 
 int
-afs_setsecattr(vp, vsecattr, flag, creds)
-     struct vnode *vp;
-     vsecattr_t *vsecattr;
-     int flag;
-     struct cred *creds;
+afs_setsecattr(struct vnode *vp, vsecattr_t *vsecattr, int flag, 
+              struct cred *creds)
 {
     return ENOSYS;
 }
 
 int
-afs_getsecattr(vp, vsecattr, flag, creds)
-     struct vnode *vp;
-     vsecattr_t *vsecattr;
-     int flag;
-     struct cred *creds;
+afs_getsecattr(struct vnode *vp, vsecattr_t *vsecattr, int flag, struct cred *creds)
 {
     return fs_fab_acl(vp, vsecattr, flag, creds);
 }
@@ -1456,13 +1330,56 @@ afs_getsecattr(vp, vsecattr, flag, creds)
 #endif
 
 #ifdef AFS_GLOBAL_SUNLOCK
-extern int gafs_open(), gafs_close(), afs_ioctl(), gafs_access();
-extern int gafs_getattr(), gafs_setattr(), gafs_lookup(), gafs_create();
-extern int gafs_remove(), gafs_link(), gafs_rename(), gafs_mkdir();
-extern int gafs_rmdir(), gafs_readdir(), gafs_fsync(), gafs_symlink();
-extern int gafs_fid(), gafs_readlink(), fs_setfl(), afs_pathconf();
-extern int afs_lockctl();
-extern void gafs_inactive();
+extern int gafs_open(register struct vcache **avcp, afs_int32 aflags, 
+                    struct AFS_UCRED *acred);
+extern int gafs_close(register struct vcache *avc, afs_int32 aflags, 
+                     int count, offset_t offset, struct AFS_UCRED *acred);
+extern int afs_ioctl(struct vnode *vnp, int com, int arg, int flag, 
+                    cred_t *credp, int *rvalp);
+extern int gafs_access(register struct vcache *avc, register afs_int32 amode,
+                      int flags, struct AFS_UCRED *acred);
+extern int gafs_getattr(register struct vcache *avc, 
+                       register struct vattr *attrs, int flags, 
+                       struct AFS_UCRED *acred);
+extern int gafs_setattr(register struct vcache *avc, 
+                       register struct vattr *attrs, int flags, 
+                       struct AFS_UCRED *acred);
+extern int gafs_lookup(register struct vcache *adp, char *aname, 
+                      register struct vcache **avcp, struct pathname *pnp,
+                      int flags, struct vnode *rdir, struct AFS_UCRED *acred);
+extern int gafs_remove(register struct vcache *adp, char *aname, 
+                      struct AFS_UCRED *acred);
+extern int gafs_link(register struct vcache *adp, register struct vcache *avc,
+                    char *aname, struct AFS_UCRED *acred);
+extern int gafs_rename(register struct vcache *aodp, char *aname1,
+                      register struct vcache *andp, char *aname2,
+                      struct AFS_UCRED *acred);
+extern int gafs_symlink(register struct vcache *adp, char *aname, 
+                       struct vattr *attrs, register char *atargetName, 
+                       struct AFS_UCRED *acred);
+extern int gafs_rmdir(register struct vcache *adp, char *aname, 
+                     struct vnode *cdirp, struct AFS_UCRED *acred);
+extern int gafs_mkdir(register struct vcache *adp, char *aname, 
+                     struct vattr *attrs, register struct vcache **avcp, 
+                     struct AFS_UCRED *acred);
+extern int
+#ifdef  AFS_SUN53_ENV
+gafs_fsync(register struct vcache *avc, int flag, struct AFS_UCRED *acred);
+#else
+gafs_fsync(register struct vcache *avc, struct AFS_UCRED *acred);
+#endif
+extern int gafs_readlink(register struct vcache *avc, struct uio *auio, 
+                        struct AFS_UCRED *acred);
+extern int gafs_readdir(register struct vcache *avc, struct uio *auio,
+                       struct AFS_UCRED *acred, int *eofp);
+extern void gafs_inactive(register struct vcache *avc, 
+                         struct AFS_UCRED *acred);
+extern int gafs_fid(struct vcache *avc, struct fid **fidpp);
+extern int gafs_create(register struct vcache *adp, char *aname, 
+                      struct vattr *attrs, enum vcexcl aexcl, int amode, 
+                      struct vcache **avcp, struct AFS_UCRED *acred);
+extern int afs_pathconf(struct vnode *vp, int cmd, u_long *outdatap,
+                       register struct AFS_UCRED *credp);
 
 #if defined(AFS_SUN511_ENV)
 /* The following list must always be NULL-terminated */
@@ -1615,12 +1532,9 @@ struct vnodeops Afs_vnodeops = {
 struct vnodeops *afs_ops = &Afs_vnodeops;
 #endif
 
-
-
-gafs_open(avcp, aflags, acred)
-     register struct vcache **avcp;
-     afs_int32 aflags;
-     struct AFS_UCRED *acred;
+int
+gafs_open(register struct vcache **avcp, afs_int32 aflags, 
+         struct AFS_UCRED *acred)
 {
     register int code;
 
@@ -1630,13 +1544,9 @@ gafs_open(avcp, aflags, acred)
     return (code);
 }
 
-
-gafs_close(avc, aflags, count, offset, acred)
-     offset_t offset;
-     int count;
-     register struct vcache *avc;
-     afs_int32 aflags;
-     struct AFS_UCRED *acred;
+int
+gafs_close(register struct vcache *avc, afs_int32 aflags, int count, 
+          offset_t offset, struct AFS_UCRED *acred)
 {
     register int code;
 
@@ -1646,12 +1556,9 @@ gafs_close(avc, aflags, count, offset, acred)
     return (code);
 }
 
-
-gafs_getattr(avc, attrs, flags, acred)
-     int flags;
-     register struct vcache *avc;
-     register struct vattr *attrs;
-     struct AFS_UCRED *acred;
+int
+gafs_getattr(register struct vcache *avc, register struct vattr *attrs, 
+            int flags, struct AFS_UCRED *acred)
 {
     register int code;
 
@@ -1662,11 +1569,9 @@ gafs_getattr(avc, attrs, flags, acred)
 }
 
 
-gafs_setattr(avc, attrs, flags, acred)
-     int flags;
-     register struct vcache *avc;
-     register struct vattr *attrs;
-     struct AFS_UCRED *acred;
+int
+gafs_setattr(register struct vcache *avc, register struct vattr *attrs, 
+            int flags, struct AFS_UCRED *acred)
 {
     register int code;
 
@@ -1677,11 +1582,9 @@ gafs_setattr(avc, attrs, flags, acred)
 }
 
 
-gafs_access(avc, amode, flags, acred)
-     int flags;
-     register struct vcache *avc;
-     register afs_int32 amode;
-     struct AFS_UCRED *acred;
+int
+gafs_access(register struct vcache *avc, register afs_int32 amode, int flags, 
+           struct AFS_UCRED *acred)
 {
     register int code;
 
@@ -1692,13 +1595,10 @@ gafs_access(avc, amode, flags, acred)
 }
 
 
-gafs_lookup(adp, aname, avcp, pnp, flags, rdir, acred)
-     struct pathname *pnp;
-     int flags;
-     struct vnode *rdir;
-     register struct vcache *adp, **avcp;
-     char *aname;
-     struct AFS_UCRED *acred;
+int
+gafs_lookup(register struct vcache *adp, char *aname, 
+           register struct vcache **avcp, struct pathname *pnp, int flags, 
+           struct vnode *rdir, struct AFS_UCRED *acred)
 {
     register int code;
 
@@ -1709,14 +1609,10 @@ gafs_lookup(adp, aname, avcp, pnp, flags, rdir, acred)
 }
 
 
-gafs_create(adp, aname, attrs, aexcl, amode, avcp, acred)
-     register struct vcache *adp;
-     char *aname;
-     struct vattr *attrs;
-     enum vcexcl aexcl;
-     int amode;
-     struct vcache **avcp;
-     struct AFS_UCRED *acred;
+int
+gafs_create(register struct vcache *adp, char *aname, struct vattr *attrs, 
+           enum vcexcl aexcl, int amode, struct vcache **avcp, 
+           struct AFS_UCRED *acred)
 {
     register int code;
 
@@ -1726,10 +1622,8 @@ gafs_create(adp, aname, attrs, aexcl, amode, avcp, acred)
     return (code);
 }
 
-gafs_remove(adp, aname, acred)
-     register struct vcache *adp;
-     char *aname;
-     struct AFS_UCRED *acred;
+int
+gafs_remove(register struct vcache *adp, char *aname, struct AFS_UCRED *acred)
 {
     register int code;
 
@@ -1739,11 +1633,9 @@ gafs_remove(adp, aname, acred)
     return (code);
 }
 
-gafs_link(adp, avc, aname, acred)
-     register struct vcache *avc;
-     register struct vcache *adp;
-     char *aname;
-     struct AFS_UCRED *acred;
+int
+gafs_link(register struct vcache *adp, register struct vcache *avc, 
+         char *aname, struct AFS_UCRED *acred)
 {
     register int code;
 
@@ -1753,10 +1645,10 @@ gafs_link(adp, avc, aname, acred)
     return (code);
 }
 
-gafs_rename(aodp, aname1, andp, aname2, acred)
-     register struct vcache *aodp, *andp;
-     char *aname1, *aname2;
-     struct AFS_UCRED *acred;
+int
+gafs_rename(register struct vcache *aodp, char *aname1, 
+           register struct vcache *andp, char *aname2, 
+           struct AFS_UCRED *acred)
 {
     register int code;
 
@@ -1786,12 +1678,9 @@ gafs_rename(aodp, aname1, andp, aname2, acred)
     return (code);
 }
 
-gafs_mkdir(adp, aname, attrs, avcp, acred)
-     register struct vcache *adp;
-     register struct vcache **avcp;
-     char *aname;
-     struct vattr *attrs;
-     struct AFS_UCRED *acred;
+int
+gafs_mkdir(register struct vcache *adp, char *aname, struct vattr *attrs, 
+          register struct vcache **avcp, struct AFS_UCRED *acred)
 {
     register int code;
 
@@ -1801,12 +1690,9 @@ gafs_mkdir(adp, aname, attrs, avcp, acred)
     return (code);
 }
 
-
-gafs_rmdir(adp, aname, cdirp, acred)
-     struct vnode *cdirp;
-     register struct vcache *adp;
-     char *aname;
-     struct AFS_UCRED *acred;
+int
+gafs_rmdir(register struct vcache *adp, char *aname, struct vnode *cdirp, 
+          struct AFS_UCRED *acred)
 {
     register int code;
 
@@ -1817,11 +1703,9 @@ gafs_rmdir(adp, aname, cdirp, acred)
 }
 
 
-gafs_readdir(avc, auio, acred, eofp)
-     int *eofp;
-     register struct vcache *avc;
-     struct uio *auio;
-     struct AFS_UCRED *acred;
+int
+gafs_readdir(register struct vcache *avc, struct uio *auio,
+            struct AFS_UCRED *acred, int *eofp)
 {
     register int code;
 
@@ -1831,12 +1715,9 @@ gafs_readdir(avc, auio, acred, eofp)
     return (code);
 }
 
-gafs_symlink(adp, aname, attrs, atargetName, acred)
-     register struct vcache *adp;
-     register char *atargetName;
-     char *aname;
-     struct vattr *attrs;
-     struct AFS_UCRED *acred;
+int
+gafs_symlink(register struct vcache *adp, char *aname, struct vattr *attrs,
+            register char *atargetName, struct AFS_UCRED *acred)
 {
     register int code;
 
@@ -1847,10 +1728,8 @@ gafs_symlink(adp, aname, attrs, atargetName, acred)
 }
 
 
-gafs_readlink(avc, auio, acred)
-     register struct vcache *avc;
-     struct uio *auio;
-     struct AFS_UCRED *acred;
+int
+gafs_readlink(register struct vcache *avc, struct uio *auio, struct AFS_UCRED *acred)
 {
     register int code;
 
@@ -1860,14 +1739,12 @@ gafs_readlink(avc, auio, acred)
     return (code);
 }
 
+int
 #ifdef AFS_SUN53_ENV
-gafs_fsync(avc, flag, acred)
-     int flag;
+gafs_fsync(register struct vcache *avc, int flag, struct AFS_UCRED *acred)
 #else
-gafs_fsync(avc, acred)
+gafs_fsync(register struct vcache *avc, struct AFS_UCRED *acred)
 #endif
-     register struct vcache *avc;
-     struct AFS_UCRED *acred;
 {
     register int code;
 
@@ -1881,12 +1758,12 @@ gafs_fsync(avc, acred)
     return (code);
 }
 
-void
+int
 afs_inactive(struct vcache *avc, struct AFS_UCRED *acred)
 {
     struct vnode *vp = AFSTOV(avc);
     if (afs_shuttingdown)
-       return;
+       return 0;
 
     /*
      * In Solaris and HPUX s800 and HP-UX10.0 they actually call us with
@@ -1902,7 +1779,7 @@ afs_inactive(struct vcache *avc, struct AFS_UCRED *acred)
     vp->v_count--;
     if (vp->v_count > 0) {
        mutex_exit(&vp->v_lock);
-       return;
+       return 0;
     }
     mutex_exit(&vp->v_lock);
 
@@ -1918,22 +1795,20 @@ afs_inactive(struct vcache *avc, struct AFS_UCRED *acred)
        avc->opens = avc->execsOrWriters = 0;
 
     afs_InactiveVCache(avc, acred);
+    return 0;
 }
 
 void
-gafs_inactive(avc, acred)
-     register struct vcache *avc;
-     struct AFS_UCRED *acred;
+gafs_inactive(register struct vcache *avc, struct AFS_UCRED *acred)
 {
     AFS_GLOCK();
-    afs_inactive(avc, acred);
+    (void)afs_inactive(avc, acred);
     AFS_GUNLOCK();
 }
 
 
-gafs_fid(avc, fidpp)
-     struct vcache *avc;
-     struct fid **fidpp;
+int
+gafs_fid(struct vcache *avc, struct fid **fidpp)
 {
     register int code;