SOLARIS: Use vn_renamepath as early as possible
[openafs.git] / src / afs / SOLARIS / osi_vnodeops.c
index 12a06e7..9f99157 100644 (file)
@@ -83,18 +83,18 @@ int afs_dump(), afs_cmp(), afs_realvp(), afs_GetOnePage();
 int afs_pvn_vptrunc;
 
 int
-afs_addmap(register struct vnode *avp, offset_t offset, struct as *asp, 
+afs_addmap(struct vnode *avp, offset_t offset, struct as *asp, 
           caddr_t addr, int length, int prot, int maxprot, int flags, 
-          AFS_UCRED *credp)
+          afs_ucred_t *credp)
 {
     /* XXX What should we do here?? XXX */
     return (0);
 }
 
 int
-afs_delmap(register struct vnode *avp, offset_t offset, struct as *asp, 
+afs_delmap(struct vnode *avp, offset_t offset, struct as *asp, 
           caddr_t addr, int length, int prot, int maxprot, int flags, 
-          AFS_UCRED *credp)
+          afs_ucred_t *credp)
 {
     /* XXX What should we do here?? XXX */
     return (0);
@@ -102,15 +102,15 @@ afs_delmap(register struct vnode *avp, offset_t offset, struct as *asp,
 
 #ifdef AFS_SUN510_ENV
 int
-afs_vmread(register struct vnode *avp, struct uio *auio, int ioflag, 
-          AFS_UCRED *acred, caller_context_t *ct)
+afs_vmread(struct vnode *avp, struct uio *auio, int ioflag, 
+          afs_ucred_t *acred, caller_context_t *ct)
 #else
 int
-afs_vmread(register struct vnode *avp, struct uio *auio, int ioflag, 
-          AFS_UCRED *acred)
+afs_vmread(struct vnode *avp, struct uio *auio, int ioflag, 
+          afs_ucred_t *acred)
 #endif
 {
-    register int code;
+    int code;
 
     if (!RW_READ_HELD(&(VTOAFS(avp))->rwlock))
        osi_Panic("afs_vmread: !rwlock");
@@ -123,15 +123,15 @@ afs_vmread(register struct vnode *avp, struct uio *auio, int ioflag,
 
 #ifdef AFS_SUN510_ENV
 int
-afs_vmwrite(register struct vnode *avp, struct uio *auio, int ioflag, 
-           AFS_UCRED *acred, caller_context_t *ct)
+afs_vmwrite(struct vnode *avp, struct uio *auio, int ioflag, 
+           afs_ucred_t *acred, caller_context_t *ct)
 #else
 int
-afs_vmwrite(register struct vnode *avp, struct uio *auio, int ioflag, 
-           AFS_UCRED *acred)
+afs_vmwrite(struct vnode *avp, struct uio *auio, int ioflag, 
+           afs_ucred_t *acred)
 #endif
 {
-    register int code;
+    int code;
 
     if (!RW_WRITE_HELD(&(VTOAFS(avp))->rwlock))
        osi_Panic("afs_vmwrite: !rwlock");
@@ -144,9 +144,9 @@ afs_vmwrite(register struct vnode *avp, struct uio *auio, int ioflag,
 int
 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, AFS_UCRED *acred)
+           enum seg_rw rw, afs_ucred_t *acred)
 {
-    register afs_int32 code = 0;
+    afs_int32 code = 0;
     AFS_STATCNT(afs_getpage);
 
     if (vp->v_flag & VNOMAP)   /* File doesn't allow mapping */
@@ -156,26 +156,41 @@ afs_getpage(struct vnode *vp, offset_t off, u_int len, u_int *protp,
 
     if (len <= PAGESIZE)
        code =
-           afs_GetOnePage(vp, 
-#if !defined(AFS_SUN56_ENV)
-                          (u_int)
-#endif
-                          off, len, protp, pl, plsz, seg, addr, rw, acred);
+           afs_GetOnePage(vp, off, len, protp, pl, plsz, seg, addr, rw, acred);
     else {
+       struct multiPage_range range;
        struct vcache *vcp = VTOAFS(vp);
+
+       /* We've been asked to get more than one page. We must return all
+        * requested pages at once, all of them locked, which means all of
+        * these dcache entries cannot be kicked out of the cache before we
+        * return (since their pages cannot be invalidated).
+        *
+        * afs_GetOnePage will be called multiple times by pvn_getpages in
+        * order to get all of the requested pages. One of the later
+        * afs_GetOnePage calls may need to evict some cache entries in order
+        * to perform its read. If we try to kick out one of the entries an
+        * earlier afs_GetOnePage call used, we will deadlock since we have
+        * the page locked. So, to tell afs_GetDownD that it should skip over
+        * any entries we've read in due to this afs_getpage call, record the
+        * offset and length in avc->multiPage.
+        *
+        * Ideally we would just set something in each dcache as we get it,
+        * but that is rather difficult, since pvn_getpages doesn't let us
+        * retain any information between calls to afs_GetOnePage. So instead
+        * just record the offset and length, and let afs_GetDownD calculate
+        * which dcache entries should be skipped. */
+
+       range.off = off;
+       range.len = len;
+
        ObtainWriteLock(&vcp->vlock, 548);
-       vcp->multiPage++;
+       QAdd(&vcp->multiPage, &range.q);
        ReleaseWriteLock(&vcp->vlock);
-       afs_BozonLock(&vcp->pvnLock, vcp);
        code =
-           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);
+           pvn_getpages(afs_GetOnePage, vp, off, len, protp, pl, plsz, seg, addr, rw, acred);
        ObtainWriteLock(&vcp->vlock, 549);
-       vcp->multiPage--;
+       QRemove(&range.q);
        ReleaseWriteLock(&vcp->vlock);
     }
     AFS_GUNLOCK();
@@ -184,23 +199,17 @@ afs_getpage(struct vnode *vp, offset_t off, u_int len, u_int *protp,
 
 /* Return all the pages from [off..off+len) in file */
 int
-#if    defined(AFS_SUN56_ENV)
 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, AFS_UCRED *acred)
-#else
-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, AFS_UCRED *acred)
-#endif
+              enum seg_rw rw, afs_ucred_t *acred)
 {
-    register struct page *page;
-    register afs_int32 code = 0;
+    struct page *page;
+    afs_int32 code = 0;
     u_int len;
     struct buf *buf;
     afs_int32 tlen;
-    register struct vcache *avc;
-    register struct dcache *tdc;
+    struct vcache *avc;
+    struct dcache *tdc;
     int i, s, pexists;
     int slot;
     afs_size_t offset, nlen = 0;
@@ -245,7 +254,8 @@ afs_GetOnePage(struct vnode *vp, u_int off, u_int alen, u_int *protp,
 
                tdc->mflags |= DFFetchReq;
                bp = afs_BQueue(BOP_FETCH, avc, B_DONTWAIT, 0, acred,
-                               (afs_size_t) off, (afs_size_t) 1, tdc);
+                               (afs_size_t) off, (afs_size_t) 1, tdc,
+                               (void *)0, (void *)0);
                if (!bp) {
                    /* Unable to start background fetch; might as well stop */
                    tdc->mflags &= ~DFFetchReq;
@@ -304,7 +314,6 @@ afs_GetOnePage(struct vnode *vp, u_int off, u_int alen, u_int *protp,
        return afs_CheckCode(code, &treq, 44);  /* failed to get it */
     }
 
-    afs_BozonLock(&avc->pvnLock, avc);
     ObtainReadLock(&avc->lock);
 
     afs_Trace4(afs_iclSetp, CM_TRACE_PAGEIN, ICL_TYPE_POINTER, (afs_int32) vp,
@@ -320,7 +329,6 @@ afs_GetOnePage(struct vnode *vp, u_int off, u_int alen, u_int *protp,
     if (avc->activeV) {
        ReleaseReadLock(&avc->lock);
        ReleaseWriteLock(&avc->vlock);
-       afs_BozonUnlock(&avc->pvnLock, avc);
        afs_PutDCache(tdc);
        /* Check activeV again, it may have been turned off
         * while we were waiting for a lock in afs_PutDCache */
@@ -344,7 +352,6 @@ afs_GetOnePage(struct vnode *vp, u_int off, u_int alen, u_int *protp,
        || !hsame(avc->f.m.DataVersion, tdc->f.versionNo)) {
        ReleaseReadLock(&tdc->lock);
        ReleaseReadLock(&avc->lock);
-       afs_BozonUnlock(&avc->pvnLock, avc);
        afs_PutDCache(tdc);
        goto retry;
     }
@@ -361,24 +368,14 @@ afs_GetOnePage(struct vnode *vp, u_int off, u_int alen, u_int *protp,
        /* if we make it here, we can't find the page in memory.  Do a real disk read
         * from the cache to get the data */
        Code |= 0x200;          /* XXX */
-#if    defined(AFS_SUN54_ENV)
        /* use PG_EXCL because we know the page does not exist already.  If it 
         * actually does exist, we have somehow raced between lookup and create.
         * As of 4/98, that shouldn't be possible, but we'll be defensive here
         * in case someone tries to relax all the serialization of read and write
         * operations with harmless things like stat. */
-#if    defined(AFS_SUN58_ENV)
        page =
            page_create_va(vp, toffset, PAGESIZE, PG_WAIT | PG_EXCL, seg,
                           addr);
-#else
-       page =
-           page_create_va(vp, toffset, PAGESIZE, PG_WAIT | PG_EXCL,
-                          seg->s_as, addr);
-#endif
-#else
-       page = page_create(vp, toffset, PAGESIZE, PG_WAIT);
-#endif
        if (!page) {
            continue;
        }
@@ -394,11 +391,7 @@ afs_GetOnePage(struct vnode *vp, u_int off, u_int alen, u_int *protp,
            buf = pageio_setup(page, PAGESIZE, vp, B_READ);     /* allocate a buf structure */
            buf->b_edev = 0;
            buf->b_dev = 0;
-#if defined(AFS_SUN56_ENV)
            buf->b_lblkno = lbtodb(toffset);
-#else
-           buf->b_blkno = btodb(toffset);
-#endif
            bp_mapin(buf);      /* map it in to our address space */
 
            AFS_GLOCK();
@@ -421,12 +414,7 @@ afs_GetOnePage(struct vnode *vp, u_int off, u_int alen, u_int *protp,
       nextpage:
        /* put page in array and continue */
        /* The p_selock must be downgraded to a shared lock after the page is read */
-#if    defined(AFS_SUN56_ENV)
-       if ((rw != S_CREATE) && !(PAGE_SHARED(page)))
-#else
-       if ((rw != S_CREATE) && !(se_shared_assert(&page->p_selock)))
-#endif
-       {
+       if ((rw != S_CREATE) && !(PAGE_SHARED(page))) {
            page_downgrade(page);
        }
        pl[slot++] = page;
@@ -457,7 +445,6 @@ afs_GetOnePage(struct vnode *vp, u_int off, u_int alen, u_int *protp,
     }
     afs_indexFlags[tdc->index] |= IFAnyPages;
     ReleaseWriteLock(&afs_xdcache);
-    afs_BozonUnlock(&avc->pvnLock, avc);
     afs_PutDCache(tdc);
     afs_Trace3(afs_iclSetp, CM_TRACE_PAGEINDONE, ICL_TYPE_LONG, code,
               ICL_TYPE_LONG, (int)page, ICL_TYPE_LONG, Code);
@@ -471,7 +458,6 @@ afs_GetOnePage(struct vnode *vp, u_int off, u_int alen, u_int *protp,
     if (page)
        pvn_read_done(page, B_ERROR);
     ReleaseReadLock(&avc->lock);
-    afs_BozonUnlock(&avc->pvnLock, avc);
     ReleaseReadLock(&tdc->lock);
     afs_PutDCache(tdc);
     return code;
@@ -479,23 +465,15 @@ afs_GetOnePage(struct vnode *vp, u_int off, u_int alen, u_int *protp,
 
 int
 afs_putpage(struct vnode *vp, offset_t off, u_int len, int flags, 
-           AFS_UCRED *cred)
+           afs_ucred_t *cred)
 {
     struct vcache *avc;
     struct page *pages;
     afs_int32 code = 0;
-#if    defined(AFS_SUN58_ENV)
     size_t tlen;
-#else
-    afs_int32 tlen;
-#endif
     afs_offs_t endPos;
     afs_int32 NPages = 0;
-#if    defined(AFS_SUN56_ENV)
     u_offset_t toff = off;
-#else
-    int toff = (int)off;
-#endif
     int didWriteLock;
 
     AFS_STATCNT(afs_putpage);
@@ -510,7 +488,6 @@ afs_putpage(struct vnode *vp, offset_t off, u_int len, int flags,
               (afs_int32) vp, ICL_TYPE_OFFSET, ICL_HANDLE_OFFSET(off),
               ICL_TYPE_INT32, (afs_int32) len, ICL_TYPE_LONG, (int)flags);
     avc = VTOAFS(vp);
-    afs_BozonLock(&avc->pvnLock, avc);
     ObtainSharedLock(&avc->lock, 247);
     didWriteLock = 0;
 
@@ -553,11 +530,7 @@ afs_putpage(struct vnode *vp, offset_t off, u_int len, int flags,
        }
 
        AFS_GUNLOCK();
-#if    defined(AFS_SUN56_ENV)
        code = pvn_vplist_dirty(vp, toff, afs_putapage, flags, cred);
-#else
-       code = pvn_vplist_dirty(vp, (u_int) off, afs_putapage, flags, cred);
-#endif
        AFS_GLOCK();
     }
 
@@ -573,7 +546,6 @@ afs_putpage(struct vnode *vp, offset_t off, u_int len, int flags,
        ReleaseWriteLock(&avc->lock);
     else
        ReleaseSharedLock(&avc->lock);
-    afs_BozonUnlock(&avc->pvnLock, avc);
     afs_Trace2(afs_iclSetp, CM_TRACE_PAGEOUTDONE, ICL_TYPE_LONG, code,
               ICL_TYPE_LONG, NPages);
     AFS_GUNLOCK();
@@ -582,16 +554,8 @@ afs_putpage(struct vnode *vp, offset_t off, u_int len, int flags,
 
 
 int
-#if defined(AFS_SUN58_ENV)
-afs_putapage(struct vnode *vp, struct page *pages, u_offset_t * offp,
-            size_t * lenp, int flags, 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, AFS_UCRED *credp)
-#else
-afs_putapage(struct vnode *vp, struct page *pages, u_int * offp,
-            u_int * lenp, int flags, AFS_UCRED *credp)
-#endif
+            size_t * lenp, int flags, afs_ucred_t *credp)
 {
     struct buf *tbuf;
     struct vcache *avc = VTOAFS(vp);
@@ -620,11 +584,7 @@ afs_putapage(struct vnode *vp, struct page *pages, u_int * offp,
            return (ENOMEM);
 
        tbuf->b_dev = 0;
-#if defined(AFS_SUN56_ENV)
        tbuf->b_lblkno = lbtodb(pages->p_offset);
-#else
-       tbuf->b_blkno = btodb(pages->p_offset);
-#endif
        bp_mapin(tbuf);
        AFS_GLOCK();
        afs_Trace4(afs_iclSetp, CM_TRACE_PAGEOUTONE, ICL_TYPE_LONG, avc,
@@ -645,21 +605,21 @@ afs_putapage(struct vnode *vp, struct page *pages, u_int * offp,
 }
 
 int
-afs_nfsrdwr(register struct vcache *avc, struct uio *auio, enum uio_rw arw,
-           int ioflag, AFS_UCRED *acred)
+afs_nfsrdwr(struct vcache *avc, struct uio *auio, enum uio_rw arw,
+           int ioflag, afs_ucred_t *acred)
 {
-    register afs_int32 code;
+    afs_int32 code;
     afs_int32 code2;
     int counter;
     afs_int32 mode, sflags;
-    register char *data;
+    char *data;
     struct dcache *dcp, *dcp_newpage;
     afs_size_t fileBase, size;
     afs_size_t pageBase;
-    register afs_int32 tsize;
-    register afs_int32 pageOffset, extraResid = 0;
-    register afs_size_t origLength;    /* length when reading/writing started */
-    register long appendLength;        /* length when this call will finish */
+    afs_int32 tsize;
+    afs_int32 pageOffset, extraResid = 0;
+    afs_size_t origLength;     /* length when reading/writing started */
+    long appendLength; /* length when this call will finish */
     int created;               /* created pages instead of faulting them */
     int lockCode;
     int didFakeOpen, eof;
@@ -710,29 +670,22 @@ afs_nfsrdwr(register struct vcache *avc, struct uio *auio, enum uio_rw arw,
     if (code)
        return afs_CheckCode(code, &treq, 45);
 
-    afs_BozonLock(&avc->pvnLock, avc);
-    osi_FlushPages(avc, acred);        /* hold bozon lock, but not basic vnode lock */
+    osi_FlushPages(avc, acred);
 
     ObtainWriteLock(&avc->lock, 250);
 
     /* adjust parameters when appending files */
     if ((ioflag & IO_APPEND) && arw == UIO_WRITE) {
-#if defined(AFS_SUN56_ENV)
        auio->uio_loffset = avc->f.m.Length;    /* write at EOF position */
-#else
-       auio->uio_offset = avc->f.m.Length;     /* write at EOF position */
-#endif
     }
     if (auio->afsio_offset < 0 || (auio->afsio_offset + auio->uio_resid) < 0) {
        ReleaseWriteLock(&avc->lock);
-       afs_BozonUnlock(&avc->pvnLock, avc);
        return EINVAL;
     }
 #ifndef AFS_64BIT_CLIENT
     /* file is larger than 2GB */
     if (AfsLargeFileSize(auio->uio_offset, auio->uio_resid)) {
        ReleaseWriteLock(&avc->lock);
-       afs_BozonUnlock(&avc->pvnLock, avc);
        return EFBIG;
     }
 #endif
@@ -740,11 +693,9 @@ afs_nfsrdwr(register struct vcache *avc, struct uio *auio, enum uio_rw arw,
     didFakeOpen = 0;           /* keep track of open so we can do close */
     if (arw == UIO_WRITE) {
        /* do ulimit processing; shrink resid or fail */
-#if    defined(AFS_SUN56_ENV)
        if (auio->uio_loffset + auio->afsio_resid > auio->uio_llimit) {
            if (auio->uio_loffset >= auio->uio_llimit) {
                ReleaseWriteLock(&avc->lock);
-               afs_BozonUnlock(&avc->pvnLock, avc);
                return EFBIG;
            } else {
                /* track # of bytes we should write, but won't because of
@@ -756,25 +707,6 @@ afs_nfsrdwr(register struct vcache *avc, struct uio *auio, enum uio_rw arw,
                extraResid -= auio->uio_resid;
            }
        }
-#else
-#ifdef AFS_SUN52_ENV
-       if (auio->afsio_offset + auio->afsio_resid > auio->uio_limit) {
-           if (auio->afsio_offset >= auio->uio_limit) {
-               ReleaseWriteLock(&avc->lock);
-               afs_BozonUnlock(&avc->pvnLock, avc);
-               return EFBIG;
-           } else {
-               /* track # of bytes we should write, but won't because of
-                * ulimit; we must add this into the final resid value
-                * so caller knows we punted some data.
-                */
-               extraResid = auio->uio_resid;
-               auio->uio_resid = auio->uio_limit - auio->afsio_offset;
-               extraResid -= auio->uio_resid;
-           }
-       }
-#endif
-#endif /* SUN56 */
        mode = S_WRITE;         /* segment map-in mode */
        afs_FakeOpen(avc);      /* do this for writes, so data gets put back
                                 * when we want it to be put back */
@@ -804,7 +736,6 @@ afs_nfsrdwr(register struct vcache *avc, struct uio *auio, enum uio_rw arw,
                (avc, PRSFS_READ, &treq,
                 CHECK_MODE_BITS | CMB_ALLOW_EXEC_AS_READ)) {
                ReleaseWriteLock(&avc->lock);
-               afs_BozonUnlock(&avc->pvnLock, avc);
                return EACCES;
            }
        }
@@ -897,13 +828,8 @@ afs_nfsrdwr(register struct vcache *avc, struct uio *auio, enum uio_rw arw,
        }
        ReleaseWriteLock(&avc->lock);   /* uiomove may page fault */
        AFS_GUNLOCK();
-#if    defined(AFS_SUN56_ENV)
        data = segmap_getmap(segkmap, AFSTOV(avc), (u_offset_t) pageBase);
        raddr = (caddr_t) (((uintptr_t) data + pageOffset) & PAGEMASK);
-#else
-       data = segmap_getmap(segkmap, AFSTOV(avc), pageBase);
-       raddr = (caddr_t) (((u_int) data + pageOffset) & PAGEMASK);
-#endif
        rsize =
            (((u_int) data + pageOffset + tsize + PAGEOFFSET) & PAGEMASK) -
            (u_int) raddr;
@@ -985,36 +911,27 @@ afs_nfsrdwr(register struct vcache *avc, struct uio *auio, enum uio_rw arw,
        code = avc->vc_error;
     }
     ReleaseWriteLock(&avc->lock);
-    afs_BozonUnlock(&avc->pvnLock, avc);
     if (!code) {
-#ifdef AFS_SUN53_ENV
        if ((ioflag & FSYNC) && (arw == UIO_WRITE)
            && !AFS_NFSXLATORREQ(acred))
            code = afs_fsync(avc, 0, acred);
-#else
-       if ((ioflag & IO_SYNC) && (arw == UIO_WRITE)
-           && !AFS_NFSXLATORREQ(acred))
-           code = afs_fsync(avc, acred);
-#endif
     }
-#ifdef AFS_SUN52_ENV
     /* 
      * If things worked, add in as remaining in request any bytes
      * we didn't write due to file size ulimit.
      */
     if (code == 0 && extraResid > 0)
        auio->uio_resid += extraResid;
-#endif
     return afs_CheckCode(code, &treq, 46);
 }
 
 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, AFS_UCRED *cred)
+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, afs_ucred_t *cred)
 {
     struct segvn_crargs crargs;
-    register afs_int32 code;
+    afs_int32 code;
     struct vrequest treq;
-    register struct vcache *avc = VTOAFS(vp);
+    struct vcache *avc = VTOAFS(vp);
 
     AFS_STATCNT(afs_map);
 
@@ -1049,21 +966,13 @@ afs_map(struct vnode *vp, offset_t off, struct as *as, caddr_t *addr, u_int len,
     if (code) {
        goto out;
     }
-    afs_BozonLock(&avc->pvnLock, avc);
     osi_FlushPages(avc, cred); /* ensure old pages are gone */
     avc->f.states |= CMAPPED;  /* flag cleared at afs_inactive */
-    afs_BozonUnlock(&avc->pvnLock, avc);
 
     AFS_GUNLOCK();
     as_rangelock(as);
     if ((flags & MAP_FIXED) == 0) {
-#if    defined(AFS_SUN57_ENV)
        map_addr(addr, len, off, 1, flags);
-#elif  defined(AFS_SUN56_ENV)
-       map_addr(addr, len, off, 1);
-#else
-       map_addr(addr, len, (off_t) off, 1);
-#endif
        if (*addr == NULL) {
            as_rangeunlock(as);
            code = ENOMEM;
@@ -1100,8 +1009,13 @@ afs_map(struct vnode *vp, offset_t off, struct as *as, caddr_t *addr, u_int len,
  * at some point.
  */
 int
-afs_pathconf(struct vnode *vp, int cmd, u_long *outdatap, 
-            register AFS_UCRED *credp)
+#ifdef AFS_SUN511_ENV
+afs_pathconf(struct vnode *vp, int cmd, u_long *outdatap,
+            afs_ucred_t *credp, caller_context_t *ct)
+#else
+afs_pathconf(struct vnode *vp, int cmd, u_long *outdatap,
+            afs_ucred_t *credp)
+#endif /* AFS_SUN511_ENV */
 {
     AFS_STATCNT(afs_cntl);
     switch (cmd) {
@@ -1128,7 +1042,11 @@ afs_pathconf(struct vnode *vp, int cmd, u_long *outdatap,
 #endif
        break;
     default:
-       return EINVAL;
+#ifdef AFS_SUN511_ENV
+       return fs_pathconf(vp, cmd, outdatap, credp, ct);
+#else
+       return fs_pathconf(vp, cmd, outdatap, credp);
+#endif /* AFS_SUN511_ENV */
     }
     return 0;
 }
@@ -1158,7 +1076,7 @@ afs_rwunlock(struct vnode *vnp, int wlock)
 int
 afs_seek(struct vnode *vnp, offset_t ooff, offset_t *noffp)
 {
-    register int code = 0;
+    int code = 0;
 
 #ifndef AFS_64BIT_CLIENT
 # define __MAXOFF_T MAXOFF_T
@@ -1174,16 +1092,13 @@ afs_seek(struct vnode *vnp, offset_t ooff, offset_t *noffp)
 int
 #ifdef AFS_SUN59_ENV
 afs_frlock(struct vnode *vnp, int cmd, struct flock64 *ap, int flag, 
-          offset_t off, struct flk_callback *flkcb, AFS_UCRED *credp)
-#elif defined(AFS_SUN56_ENV)
-afs_frlock(struct vnode *vnp, int cmd, struct flock64 *ap, int flag, 
-          offset_t off, AFS_UCRED *credp)
+          offset_t off, struct flk_callback *flkcb, afs_ucred_t *credp)
 #else
-afs_frlock(struct vnode *vnp, int cmd, struct flock *ap, int flag, 
-          offset_t off, AFS_UCRED *credp)
+afs_frlock(struct vnode *vnp, int cmd, struct flock64 *ap, int flag,
+          offset_t off, afs_ucred_t *credp)
 #endif
 {
-    register afs_int32 code = 0;
+    afs_int32 code = 0;
     /*
      * Implement based on afs_lockctl
      */
@@ -1194,20 +1109,11 @@ afs_frlock(struct vnode *vnp, int cmd, struct flock *ap, int flag,
 #endif
     if ((cmd == F_GETLK) || (cmd == F_O_GETLK) || (cmd == F_SETLK)
        || (cmd == F_SETLKW)) {
-#ifdef AFS_SUN53_ENV
        ap->l_pid = ttoproc(curthread)->p_pid;
        ap->l_sysid = 0;
-#else
-       ap->l_pid = ttoproc(curthread)->p_epid;
-       ap->l_sysid = ttoproc(curthread)->p_sysid;
-#endif
 
        AFS_GUNLOCK();
-#ifdef AFS_SUN56_ENV
        code = convoff(vnp, ap, 0, off);
-#else
-       code = convoff(vnp, ap, 0, (off_t) off);
-#endif
        if (code)
            return code;
        AFS_GLOCK();
@@ -1220,23 +1126,14 @@ afs_frlock(struct vnode *vnp, int cmd, struct flock *ap, int flag,
 
 
 int
-#if    defined(AFS_SUN56_ENV)
 afs_space(struct vnode *vnp, int cmd, struct flock64 *ap, int flag, 
-         offset_t off, AFS_UCRED *credp)
-#else
-afs_space(struct vnode *vnp, int cmd, struct flock *ap, int flag, 
-         offset_t off, AFS_UCRED *credp)
-#endif
+         offset_t off, afs_ucred_t *credp)
 {
-    register afs_int32 code = EINVAL;
+    afs_int32 code = EINVAL;
     struct vattr vattr;
 
     if ((cmd == F_FREESP)
-#ifdef AFS_SUN56_ENV
        && ((code = convoff(vnp, ap, 0, off)) == 0)) {
-#else
-       && ((code = convoff(vnp, ap, 0, (off_t) off)) == 0)) {
-#endif
        AFS_GLOCK();
        if (!ap->l_len) {
            vattr.va_mask = AT_SIZE;
@@ -1293,7 +1190,6 @@ afs_dumpctl(struct vnode *vp, int i)
     return EINVAL;
 }
 
-#ifdef AFS_SUN54_ENV
 #ifdef AFS_SUN511_ENV
 extern void
 afs_dispose(struct vnode *vp, struct page *p, int fl, int dn, struct cred *cr, struct caller_context_t *ct)
@@ -1332,59 +1228,58 @@ afs_getsecattr(struct vnode *vp, vsecattr_t *vsecattr, int flag, struct cred *cr
     return fs_fab_acl(vp, vsecattr, flag, creds);
 }
 #endif
-#endif
 
 #ifdef AFS_GLOBAL_SUNLOCK
-extern int gafs_open(register struct vcache **avcp, afs_int32 aflags, 
-                    AFS_UCRED *acred);
-extern int gafs_close(register struct vcache *avc, afs_int32 aflags, 
-                     int count, offset_t offset, AFS_UCRED *acred);
+extern int gafs_open(struct vcache **avcp, afs_int32 aflags, 
+                    afs_ucred_t *acred);
+extern int gafs_close(struct vcache *avc, afs_int32 aflags, 
+                     int count, offset_t offset, afs_ucred_t *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, AFS_UCRED *acred);
-extern int gafs_getattr(register struct vcache *avc, 
-                       register struct vattr *attrs, int flags, 
-                       AFS_UCRED *acred);
-extern int gafs_setattr(register struct vcache *avc, 
-                       register struct vattr *attrs, int flags, 
-                       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, AFS_UCRED *acred);
-extern int gafs_remove(register struct vcache *adp, char *aname, 
-                      AFS_UCRED *acred);
-extern int gafs_link(register struct vcache *adp, register struct vcache *avc,
-                    char *aname, AFS_UCRED *acred);
-extern int gafs_rename(register struct vcache *aodp, char *aname1,
-                      register struct vcache *andp, char *aname2,
-                      AFS_UCRED *acred);
-extern int gafs_symlink(register struct vcache *adp, char *aname, 
-                       struct vattr *attrs, register char *atargetName, 
-                       AFS_UCRED *acred);
-extern int gafs_rmdir(register struct vcache *adp, char *aname, 
-                     struct vnode *cdirp, AFS_UCRED *acred);
-extern int gafs_mkdir(register struct vcache *adp, char *aname, 
-                     struct vattr *attrs, register struct vcache **avcp, 
-                     AFS_UCRED *acred);
-extern int
-#ifdef  AFS_SUN53_ENV
-gafs_fsync(register struct vcache *avc, int flag, AFS_UCRED *acred);
-#else
-gafs_fsync(register struct vcache *avc, AFS_UCRED *acred);
-#endif
-extern int gafs_readlink(register struct vcache *avc, struct uio *auio, 
-                        AFS_UCRED *acred);
-extern int gafs_readdir(register struct vcache *avc, struct uio *auio,
-                       AFS_UCRED *acred, int *eofp);
-extern void gafs_inactive(register struct vcache *avc, 
-                         AFS_UCRED *acred);
+extern int gafs_access(struct vcache *avc, afs_int32 amode,
+                      int flags, afs_ucred_t *acred);
+extern int gafs_getattr(struct vcache *avc, 
+                       struct vattr *attrs, int flags, 
+                       afs_ucred_t *acred);
+extern int gafs_setattr(struct vcache *avc, 
+                       struct vattr *attrs, int flags, 
+                       afs_ucred_t *acred);
+extern int gafs_lookup(struct vcache *adp, char *aname, 
+                      struct vcache **avcp, struct pathname *pnp,
+                      int flags, struct vnode *rdir, afs_ucred_t *acred);
+extern int gafs_remove(struct vcache *adp, char *aname, 
+                      afs_ucred_t *acred);
+extern int gafs_link(struct vcache *adp, struct vcache *avc,
+                    char *aname, afs_ucred_t *acred);
+extern int gafs_rename(struct vcache *aodp, char *aname1,
+                      struct vcache *andp, char *aname2,
+                      afs_ucred_t *acred);
+extern int gafs_symlink(struct vcache *adp, char *aname, 
+                       struct vattr *attrs, char *atargetName, 
+                       afs_ucred_t *acred);
+extern int gafs_rmdir(struct vcache *adp, char *aname, 
+                     struct vnode *cdirp, afs_ucred_t *acred);
+extern int gafs_mkdir(struct vcache *adp, char *aname, 
+                     struct vattr *attrs, struct vcache **avcp, 
+                     afs_ucred_t *acred);
+extern int gafs_fsync(struct vcache *avc, int flag, afs_ucred_t *acred);
+extern int gafs_readlink(struct vcache *avc, struct uio *auio, 
+                        afs_ucred_t *acred);
+extern int gafs_readdir(struct vcache *avc, struct uio *auio,
+                       afs_ucred_t *acred, int *eofp);
+extern void gafs_inactive(struct vcache *avc, 
+                         afs_ucred_t *acred);
 extern int gafs_fid(struct vcache *avc, struct fid **fidpp);
-extern int gafs_create(register struct vcache *adp, char *aname, 
+extern int gafs_create(struct vcache *adp, char *aname, 
                       struct vattr *attrs, enum vcexcl aexcl, int amode, 
-                      struct vcache **avcp, AFS_UCRED *acred);
+                      struct vcache **avcp, afs_ucred_t *acred);
+#ifdef AFS_SUN511_ENV
+extern int afs_pathconf(struct vnode *vp, int cmd, u_long *outdatap,
+                       afs_ucred_t *credp, caller_context_t *ct);
+#else
 extern int afs_pathconf(struct vnode *vp, int cmd, u_long *outdatap,
-                       register AFS_UCRED *credp);
+                       afs_ucred_t *credp);
+#endif /* AFS_SUN511_ENV */
 
 #if defined(AFS_SUN511_ENV)
 /* The following list must always be NULL-terminated */
@@ -1525,23 +1420,19 @@ struct vnodeops Afs_vnodeops = {
     afs_pathconf,
     afs_pageio,
     afs_dumpctl,
-#ifdef AFS_SUN54_ENV
     afs_dispose,
     afs_setsecattr,
     afs_getsecattr,
-#endif
-#if    defined(AFS_SUN56_ENV)
     fs_shrlock,
-#endif
 };
 struct vnodeops *afs_ops = &Afs_vnodeops;
 #endif
 
 int
-gafs_open(register struct vcache **avcp, afs_int32 aflags, 
-         AFS_UCRED *acred)
+gafs_open(struct vcache **avcp, afs_int32 aflags, 
+         afs_ucred_t *acred)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
     code = afs_open(avcp, aflags, acred);
@@ -1550,10 +1441,10 @@ gafs_open(register struct vcache **avcp, afs_int32 aflags,
 }
 
 int
-gafs_close(register struct vcache *avc, afs_int32 aflags, int count, 
-          offset_t offset, AFS_UCRED *acred)
+gafs_close(struct vcache *avc, afs_int32 aflags, int count, 
+          offset_t offset, afs_ucred_t *acred)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
     code = afs_close(avc, aflags, count, offset, acred);
@@ -1562,10 +1453,10 @@ gafs_close(register struct vcache *avc, afs_int32 aflags, int count,
 }
 
 int
-gafs_getattr(register struct vcache *avc, register struct vattr *attrs, 
-            int flags, AFS_UCRED *acred)
+gafs_getattr(struct vcache *avc, struct vattr *attrs, 
+            int flags, afs_ucred_t *acred)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
     code = afs_getattr(avc, attrs, flags, acred);
@@ -1575,10 +1466,10 @@ gafs_getattr(register struct vcache *avc, register struct vattr *attrs,
 
 
 int
-gafs_setattr(register struct vcache *avc, register struct vattr *attrs, 
-            int flags, AFS_UCRED *acred)
+gafs_setattr(struct vcache *avc, struct vattr *attrs, 
+            int flags, afs_ucred_t *acred)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
     code = afs_setattr(avc, attrs, flags, acred);
@@ -1588,10 +1479,10 @@ gafs_setattr(register struct vcache *avc, register struct vattr *attrs,
 
 
 int
-gafs_access(register struct vcache *avc, register afs_int32 amode, int flags, 
-           AFS_UCRED *acred)
+gafs_access(struct vcache *avc, afs_int32 amode, int flags, 
+           afs_ucred_t *acred)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
     code = afs_access(avc, amode, flags, acred);
@@ -1601,11 +1492,11 @@ gafs_access(register struct vcache *avc, register afs_int32 amode, int flags,
 
 
 int
-gafs_lookup(register struct vcache *adp, char *aname, 
-           register struct vcache **avcp, struct pathname *pnp, int flags, 
-           struct vnode *rdir, AFS_UCRED *acred)
+gafs_lookup(struct vcache *adp, char *aname, 
+           struct vcache **avcp, struct pathname *pnp, int flags, 
+           struct vnode *rdir, afs_ucred_t *acred)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
     code = afs_lookup(adp, aname, avcp, pnp, flags, rdir, acred);
@@ -1615,11 +1506,11 @@ gafs_lookup(register struct vcache *adp, char *aname,
 
 
 int
-gafs_create(register struct vcache *adp, char *aname, struct vattr *attrs, 
+gafs_create(struct vcache *adp, char *aname, struct vattr *attrs, 
            enum vcexcl aexcl, int amode, struct vcache **avcp, 
-           AFS_UCRED *acred)
+           afs_ucred_t *acred)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
     code = afs_create(adp, aname, attrs, aexcl, amode, avcp, acred);
@@ -1628,9 +1519,9 @@ gafs_create(register struct vcache *adp, char *aname, struct vattr *attrs,
 }
 
 int
-gafs_remove(register struct vcache *adp, char *aname, AFS_UCRED *acred)
+gafs_remove(struct vcache *adp, char *aname, afs_ucred_t *acred)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
     code = afs_remove(adp, aname, acred);
@@ -1639,10 +1530,10 @@ gafs_remove(register struct vcache *adp, char *aname, AFS_UCRED *acred)
 }
 
 int
-gafs_link(register struct vcache *adp, register struct vcache *avc, 
-         char *aname, AFS_UCRED *acred)
+gafs_link(struct vcache *adp, struct vcache *avc, 
+         char *aname, afs_ucred_t *acred)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
     code = afs_link(adp, avc, aname, acred);
@@ -1651,11 +1542,11 @@ gafs_link(register struct vcache *adp, register struct vcache *avc,
 }
 
 int
-gafs_rename(register struct vcache *aodp, char *aname1, 
-           register struct vcache *andp, char *aname2, 
-           AFS_UCRED *acred)
+gafs_rename(struct vcache *aodp, char *aname1, 
+           struct vcache *andp, char *aname2, 
+           afs_ucred_t *acred)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
     code = afs_rename(aodp, aname1, andp, aname2, acred);
@@ -1666,7 +1557,10 @@ gafs_rename(register struct vcache *aodp, char *aname1,
        (void) afs_lookup(andp, aname2, &avcp, NULL, 0, NULL, acred);
        if (avcp) {
            struct vnode *vp = AFSTOV(avcp), *pvp = AFSTOV(andp);
-           
+
+# ifdef HAVE_VN_RENAMEPATH
+           vn_renamepath(pvp, vp, aname2, strlen(aname2));
+# else
            mutex_enter(&vp->v_lock);
            if (vp->v_path != NULL) {
                kmem_free(vp->v_path, strlen(vp->v_path) + 1);
@@ -1674,6 +1568,7 @@ gafs_rename(register struct vcache *aodp, char *aname1,
            }
            mutex_exit(&vp->v_lock);
            vn_setpath(afs_globalVp, pvp, vp, aname2, strlen(aname2));
+# endif /* !HAVE_VN_RENAMEPATH */
 
            AFS_RELE(avcp);
        }
@@ -1684,10 +1579,10 @@ gafs_rename(register struct vcache *aodp, char *aname1,
 }
 
 int
-gafs_mkdir(register struct vcache *adp, char *aname, struct vattr *attrs, 
-          register struct vcache **avcp, AFS_UCRED *acred)
+gafs_mkdir(struct vcache *adp, char *aname, struct vattr *attrs, 
+          struct vcache **avcp, afs_ucred_t *acred)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
     code = afs_mkdir(adp, aname, attrs, avcp, acred);
@@ -1696,10 +1591,10 @@ gafs_mkdir(register struct vcache *adp, char *aname, struct vattr *attrs,
 }
 
 int
-gafs_rmdir(register struct vcache *adp, char *aname, struct vnode *cdirp, 
-          AFS_UCRED *acred)
+gafs_rmdir(struct vcache *adp, char *aname, struct vnode *cdirp, 
+          afs_ucred_t *acred)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
     code = afs_rmdir(adp, aname, cdirp, acred);
@@ -1709,10 +1604,10 @@ gafs_rmdir(register struct vcache *adp, char *aname, struct vnode *cdirp,
 
 
 int
-gafs_readdir(register struct vcache *avc, struct uio *auio,
-            AFS_UCRED *acred, int *eofp)
+gafs_readdir(struct vcache *avc, struct uio *auio,
+            afs_ucred_t *acred, int *eofp)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
     code = afs_readdir(avc, auio, acred, eofp);
@@ -1721,10 +1616,10 @@ gafs_readdir(register struct vcache *avc, struct uio *auio,
 }
 
 int
-gafs_symlink(register struct vcache *adp, char *aname, struct vattr *attrs,
-            register char *atargetName, AFS_UCRED *acred)
+gafs_symlink(struct vcache *adp, char *aname, struct vattr *attrs,
+            char *atargetName, afs_ucred_t *acred)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
     code = afs_symlink(adp, aname, attrs, atargetName, acred);
@@ -1734,9 +1629,9 @@ gafs_symlink(register struct vcache *adp, char *aname, struct vattr *attrs,
 
 
 int
-gafs_readlink(register struct vcache *avc, struct uio *auio, AFS_UCRED *acred)
+gafs_readlink(struct vcache *avc, struct uio *auio, afs_ucred_t *acred)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
     code = afs_readlink(avc, auio, acred);
@@ -1745,26 +1640,18 @@ gafs_readlink(register struct vcache *avc, struct uio *auio, AFS_UCRED *acred)
 }
 
 int
-#ifdef AFS_SUN53_ENV
-gafs_fsync(register struct vcache *avc, int flag, AFS_UCRED *acred)
-#else
-gafs_fsync(register struct vcache *avc, AFS_UCRED *acred)
-#endif
+gafs_fsync(struct vcache *avc, int flag, afs_ucred_t *acred)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
-#ifdef AFS_SUN53_ENV
     code = afs_fsync(avc, flag, acred);
-#else
-    code = afs_fsync(avc, acred);
-#endif
     AFS_GUNLOCK();
     return (code);
 }
 
 int
-afs_inactive(struct vcache *avc, AFS_UCRED *acred)
+afs_inactive(struct vcache *avc, afs_ucred_t *acred)
 {
     struct vnode *vp = AFSTOV(avc);
     if (afs_shuttingdown)
@@ -1788,6 +1675,7 @@ afs_inactive(struct vcache *avc, AFS_UCRED *acred)
     }
     mutex_exit(&vp->v_lock);
 
+#ifndef AFS_SUN511_ENV
     /*
      * Solaris calls VOP_OPEN on exec, but doesn't call VOP_CLOSE when
      * the executable exits.  So we clean up the open count here.
@@ -1798,13 +1686,21 @@ afs_inactive(struct vcache *avc, AFS_UCRED *acred)
      */
     if (avc->opens > 0 && avc->mvstat == 0 && !(avc->f.states & CCore))
        avc->opens = avc->execsOrWriters = 0;
+#endif
 
     afs_InactiveVCache(avc, acred);
+
+    AFS_GUNLOCK();
+    /* VFS_RELE must be called outside of GLOCK, since it can potentially
+     * call afs_freevfs, which acquires GLOCK */
+    VFS_RELE(afs_globalVFS);
+    AFS_GLOCK();
+
     return 0;
 }
 
 void
-gafs_inactive(register struct vcache *avc, AFS_UCRED *acred)
+gafs_inactive(struct vcache *avc, afs_ucred_t *acred)
 {
     AFS_GLOCK();
     (void)afs_inactive(avc, acred);
@@ -1815,7 +1711,7 @@ gafs_inactive(register struct vcache *avc, AFS_UCRED *acred)
 int
 gafs_fid(struct vcache *avc, struct fid **fidpp)
 {
-    register int code;
+    int code;
 
     AFS_GLOCK();
     code = afs_fid(avc, fidpp);