afs: Remove useless afs_GetVCache arguments 81/13681/7
authorAndrew Deason <adeason@sinenomine.net>
Wed, 10 Jul 2019 17:42:44 +0000 (12:42 -0500)
committerBenjamin Kaduk <kaduk@mit.edu>
Sun, 18 Aug 2019 17:26:12 +0000 (13:26 -0400)
The 'avc' argument in afs_GetVCache has never been used, all the way
back to OpenAFS 1.0. The 'cached' argument was set correctly, but none
of its callers ever looked at the result of 'cached'. Remove these
useless arguments.

afs_LookupVCache and afs_GetRootVCache also had the same 'cached'
argument, which was also never used by callers. Remove it for those,
as well.

Change-Id: I3536259f26536acc02fbb058787f417bf0f50b9a
Reviewed-on: https://gerrit.openafs.org/13681
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Tested-by: BuildBot <buildbot@rampaginggeek.com>

22 files changed:
src/afs/AIX/osi_vfsops.c
src/afs/DARWIN/osi_vfsops.c
src/afs/FBSD/osi_vfsops.c
src/afs/HPUX/osi_vfsops.c
src/afs/IRIX/osi_vfsops.c
src/afs/LINUX/osi_export.c
src/afs/LINUX/osi_vcache.c
src/afs/LINUX/osi_vfsops.c
src/afs/NBSD/osi_vfsops.c
src/afs/OBSD/osi_vfsops.c
src/afs/SOLARIS/osi_vfsops.c
src/afs/UKERNEL/osi_vfsops.c
src/afs/VNOPS/afs_vnop_access.c
src/afs/VNOPS/afs_vnop_create.c
src/afs/VNOPS/afs_vnop_dirops.c
src/afs/VNOPS/afs_vnop_lookup.c
src/afs/VNOPS/afs_vnop_remove.c
src/afs/VNOPS/afs_vnop_rename.c
src/afs/afs_osi_vget.c
src/afs/afs_pioctl.c
src/afs/afs_prototypes.h
src/afs/afs_vcache.c

index 5265944..3c895d8 100644 (file)
@@ -125,7 +125,7 @@ afs_root_nolock(struct vfs *afsp, struct vnode **avpp)
        }
        credp = crref();
        if (!(code = afs_InitReq(&treq, credp)) && !(code = afs_CheckInit())) {
-           tvp = afs_GetVCache(&afs_rootFid, &treq, NULL, NULL);
+           tvp = afs_GetVCache(&afs_rootFid, &treq);
            /* we really want this to stay around */
            if (tvp) {
                afs_globalVp = tvp;
index 064410d..50ac152 100644 (file)
@@ -251,7 +251,7 @@ again:
            ? &afs_rootFid : (struct VenusFid *)mdata;
 
        if (!(error = afs_InitReq(&treq, cr)) && !(error = afs_CheckInit())) {
-           tvp = afs_GetVCache(rootFid, &treq, NULL, NULL);
+           tvp = afs_GetVCache(rootFid, &treq);
 #ifdef AFS_DARWIN80_ENV
             if (tvp) {
                AFS_GUNLOCK();
index 8ca88c3..c9361fe 100644 (file)
@@ -278,7 +278,7 @@ tryagain:
        error = 0;
     } else {
        if (!(error = afs_InitReq(&treq, cr)) && !(error = afs_CheckInit())) {
-           tvp = afs_GetVCache(&afs_rootFid, &treq, NULL, NULL);
+           tvp = afs_GetVCache(&afs_rootFid, &treq);
            /* we really want this to stay around */
            if (tvp) {
                gvp = afs_globalVp;
index ee3741c..5d53926 100644 (file)
@@ -128,7 +128,7 @@ afs_root(struct vfs *afsp, struct vnode **avpp, char *unused1)
 
        if (!(code = afs_InitReq(&treq, p_cred(u.u_procp)))
            && !(code = afs_CheckInit())) {
-           tvp = afs_GetVCache(&afs_rootFid, &treq, NULL, NULL);
+           tvp = afs_GetVCache(&afs_rootFid, &treq);
            /* we really want this to stay around */
            if (tvp) {
                afs_globalVp = tvp;
index a7e8387..ef800a3 100644 (file)
@@ -274,7 +274,7 @@ afs_root(OSI_VFS_ARG(afsp), avpp)
 
        if (!(code = afs_InitReq(&treq, OSI_GET_CURRENT_CRED()))
            && !(code = afs_CheckInit())) {
-           tvp = afs_GetVCache(&afs_rootFid, &treq, NULL, NULL);
+           tvp = afs_GetVCache(&afs_rootFid, &treq);
            /* we really want this to stay around */
            if (tvp) {
                afs_globalVp = tvp;
@@ -541,7 +541,7 @@ afs_vget(OSI_VFS_DECL(afsp), vnode_t ** avcp, struct fid * fidp)
        if (code = afs_InitReq(&treq, OSI_GET_CURRENT_CRED()))
            goto out;
        *avcp =
-           (vnode_t *) afs_GetVCache(&vfid, &treq, NULL, (struct vcache *)0);
+           (vnode_t *) afs_GetVCache(&vfid, &treq);
        if (!*avcp) {
            code = EIO;
        }
index 926bd0d..76dd229 100644 (file)
@@ -424,7 +424,7 @@ static int UnEvalFakeStat(struct vrequest *areq, struct vcache **vcpp)
     tfid = tvp->mtpoint;
     afs_PutVolume(tvp, READ_LOCK);
 
-    tvc = afs_GetVCache(&tfid, areq, NULL, NULL);
+    tvc = afs_GetVCache(&tfid, areq);
     if (!tvc) {
 #ifdef OSI_EXPORT_DEBUG
        printk("afs: UnEvalFakeStat(0x%08x/%d/%d.%d): GetVCache(0x%08x/%d/%d.%d) failed\n",
@@ -486,7 +486,7 @@ static struct dentry *get_dentry_from_fid(cred_t *credp, struct VenusFid *afid)
        afs_DestroyAttr(vattr);
        return ERR_PTR(-afs_CheckCode(code, NULL, 101));
     }
-    vcp = afs_GetVCache(afid, treq, NULL, NULL);
+    vcp = afs_GetVCache(afid, treq);
     if (vcp == NULL) {
 #ifdef OSI_EXPORT_DEBUG
        printk("afs: get_dentry_from_fid(0x%08x/%d/%d.%d): no vcache\n",
@@ -689,7 +689,7 @@ static int afs_export_get_name(struct dentry *parent, char *name,
               data.fid.Cell,      data.fid.Fid.Volume,
               data.fid.Fid.Vnode, data.fid.Fid.Unique);
 #endif
-       vcp = afs_GetVCache(&data.fid, treq, NULL, NULL);
+       vcp = afs_GetVCache(&data.fid, treq);
        if (vcp) {
            ObtainReadLock(&vcp->lock);
            if (strlen(vcp->linkData + 1) <= NAME_MAX)
index 19840d1..3b82cc6 100644 (file)
@@ -205,7 +205,7 @@ osi_ResetRootVCache(afs_uint32 volid)
     credp = crref();
     if (afs_CreateReq(&treq, credp))
        goto out;
-    vcp = afs_GetVCache(&afs_rootFid, treq, NULL, NULL);
+    vcp = afs_GetVCache(&afs_rootFid, treq);
     if (!vcp)
        goto out;
     afs_getattr(vcp, &vattr, credp);
index 0adfc38..949092a 100644 (file)
@@ -202,7 +202,7 @@ afs_root(struct super_block *afsp)
        }
 
        if (!(code = afs_CreateReq(&treq, credp)) && !(code = afs_CheckInit())) {
-           tvp = afs_GetVCache(&afs_rootFid, treq, NULL, NULL);
+           tvp = afs_GetVCache(&afs_rootFid, treq);
            if (tvp) {
                struct inode *ip = AFSTOV(tvp);
                struct vattr *vattr = NULL;
index a52629b..2b662c1 100644 (file)
@@ -355,7 +355,7 @@ tryagain:
 
        if (!(code = afs_InitReq(&treq, osi_curcred()))
            && !(code = afs_CheckInit())) {
-           tvp = afs_GetVCache(&afs_rootFid, &treq, NULL, NULL);
+           tvp = afs_GetVCache(&afs_rootFid, &treq);
 
            if (tvp) {
                if (afs_globalVp) {
index 4377494..78b7ed6 100644 (file)
@@ -344,7 +344,7 @@ afs_root(struct mount *mp, struct vnode **vpp)
     AFS_GLOCK();
     if (!(code = afs_InitReq(&treq, osi_curcred()))
        && !(code = afs_CheckInit())) {
-       tvp = afs_GetVCache(&afs_rootFid, &treq, NULL, NULL);
+       tvp = afs_GetVCache(&afs_rootFid, &treq);
        if (tvp) {
            /* There is really no reason to over-hold this bugger--it's held
             * by the root filesystem reference. */
index cd127e4..26c49d7 100644 (file)
@@ -178,7 +178,7 @@ again:
 
        if (!(code = afs_InitReq(&treq, proc->p_cred))
            && !(code = afs_CheckInit())) {
-           tvp = afs_GetVCache(&afs_rootFid, &treq, NULL, NULL);
+           tvp = afs_GetVCache(&afs_rootFid, &treq);
            /* we really want this to stay around */
            if (tvp) {
                if (afs_globalVp) {
index b50af8f..2dfa201 100644 (file)
@@ -76,7 +76,7 @@ afs_root(OSI_VFS_DECL(afsp), struct vnode **avpp)
 
        if (!(code = afs_InitReq(&treq, get_user_struct()->u_cred))
            && !(code = afs_CheckInit())) {
-           tvp = afs_GetVCache(&afs_rootFid, &treq, NULL, NULL);
+           tvp = afs_GetVCache(&afs_rootFid, &treq);
            /* we really want this to stay around */
            if (tvp) {
                afs_globalVp = tvp;
index 149dae9..8dd885b 100644 (file)
@@ -145,7 +145,7 @@ afs_AccessOK(struct vcache *avc, afs_int32 arights, struct vrequest *areq,
            dirFid.Fid.Vnode = avc->f.parent.vnode;
            dirFid.Fid.Unique = avc->f.parent.unique;
            /* Avoid this GetVCache call */
-           tvc = afs_GetVCache(&dirFid, areq, NULL, NULL);
+           tvc = afs_GetVCache(&dirFid, areq);
            if (tvc) {
                dirBits = afs_GetAccessBits(tvc, arights, areq);
                afs_PutVCache(tvc);
index 6d1df4f..dc0966d 100644 (file)
@@ -178,10 +178,10 @@ afs_create(OSI_VC_DECL(adp), char *aname, struct vattr *attrs,
            newFid.Fid.Volume = adp->f.fid.Fid.Volume;
            tvc = NULL;
            if (newFid.Fid.Unique == 0) {
-               tvc = afs_LookupVCache(&newFid, treq, NULL, adp, aname);
+               tvc = afs_LookupVCache(&newFid, treq, adp, aname);
            }
            if (!tvc)           /* lookup failed or wasn't called */
-               tvc = afs_GetVCache(&newFid, treq, NULL, NULL);
+               tvc = afs_GetVCache(&newFid, treq);
 
            if (tvc) {
                /* if the thing exists, we need the right access to open it.
index 5966513..1ddda67 100644 (file)
@@ -228,7 +228,7 @@ afs_mkdir(OSI_VC_DECL(adp), char *aname, struct vattr *attrs,
        ReleaseWriteLock(&tvc->lock);
     } else {
        /* now we're done with parent dir, create the real dir's cache entry */
-       tvc = afs_GetVCache(&newFid, treq, NULL, NULL);
+       tvc = afs_GetVCache(&newFid, treq);
        if (tvc) {
            code = 0;
            *avcp = tvc;
@@ -323,13 +323,11 @@ afs_rmdir(OSI_VC_DECL(adp), char *aname, afs_ucred_t *acred)
        unlinkFid.Fid.Vnode = 0;
        code = afs_dir_Lookup(tdc, aname, &unlinkFid.Fid);
        if (code == 0) {
-           afs_int32 cached = 0;
-
            unlinkFid.Cell = adp->f.fid.Cell;
            unlinkFid.Fid.Volume = adp->f.fid.Fid.Volume;
            if (unlinkFid.Fid.Unique == 0) {
                tvc =
-                   afs_LookupVCache(&unlinkFid, treq, &cached, adp, aname);
+                   afs_LookupVCache(&unlinkFid, treq, adp, aname);
            } else {
                ObtainReadLock(&afs_xvcache);
                tvc = afs_FindVCache(&unlinkFid, 0, 1 /* do xstats */ );
index 2ca0969..e54e382 100644 (file)
@@ -408,7 +408,7 @@ afs_EvalFakeStat_int(struct vcache **avcp, struct afs_fakestat_state *state,
            } while (root_vp && retry);
            ReleaseWriteLock(&afs_xvcache);
        } else {
-           root_vp = afs_GetVCache(tvc->mvid.target_root, areq, NULL, NULL);
+           root_vp = afs_GetVCache(tvc->mvid.target_root, areq);
        }
        if (!root_vp) {
            code = canblock ? EIO : 0;
@@ -1458,7 +1458,7 @@ afs_lookup(OSI_VC_DECL(adp), char *aname, struct vcache **avcp, afs_ucred_t *acr
        }
        /* otherwise we have the fid here, so we use it */
        /*printf("Getting vcache\n");*/
-       tvc = afs_GetVCache(adp->mvid.parent, treq, NULL, NULL);
+       tvc = afs_GetVCache(adp->mvid.parent, treq);
        afs_Trace3(afs_iclSetp, CM_TRACE_GETVCDOTDOT, ICL_TYPE_FID, adp->mvid.parent,
                   ICL_TYPE_POINTER, tvc, ICL_TYPE_INT32, code);
        *avcp = tvc;
@@ -1554,7 +1554,7 @@ afs_lookup(OSI_VC_DECL(adp), char *aname, struct vcache **avcp, afs_ucred_t *acr
            tfid.Fid.Vnode = VNUM_FROM_TYPEID(VN_TYPE_MOUNT, cellidx << 2);
            tfid.Fid.Unique = volid;
        }
-       *avcp = tvc = afs_GetVCache(&tfid, treq, NULL, NULL);
+       *avcp = tvc = afs_GetVCache(&tfid, treq);
        code = (tvc ? 0 : EIO);
        hit = 1;
        goto done;
@@ -1570,7 +1570,7 @@ afs_lookup(OSI_VC_DECL(adp), char *aname, struct vcache **avcp, afs_ucred_t *acr
        struct VenusFid tfid;
 
        afs_GetDynrootMountFid(&tfid);
-       *avcp = tvc = afs_GetVCache(&tfid, treq, NULL, NULL);
+       *avcp = tvc = afs_GetVCache(&tfid, treq);
        code = 0;
        hit = 1;
        goto done;
@@ -1780,12 +1780,11 @@ afs_lookup(OSI_VC_DECL(adp), char *aname, struct vcache **avcp, afs_ucred_t *acr
         * the file has not yet been looked up.
         */
        if (!tvc) {
-           afs_int32 cached = 0;
            if (!tfid.Fid.Unique && (adp->f.states & CForeign)) {
-               tvc = afs_LookupVCache(&tfid, treq, &cached, adp, tname);
+               tvc = afs_LookupVCache(&tfid, treq, adp, tname);
            }
            if (!tvc && !bulkcode) {    /* lookup failed or wasn't called */
-               tvc = afs_GetVCache(&tfid, treq, &cached, NULL);
+               tvc = afs_GetVCache(&tfid, treq);
            }
        }                       /* if !tvc */
     }                          /* sub-block just to reduce stack usage */
@@ -1845,9 +1844,9 @@ afs_lookup(OSI_VC_DECL(adp), char *aname, struct vcache **avcp, afs_ucred_t *acr
                    if (tvolp && (tvolp->states & VForeign)) {
                        /* XXXX tvolp has ref cnt on but not locked! XXX */
                        tvc =
-                           afs_GetRootVCache(tvc->mvid.target_root, treq, NULL, tvolp);
+                           afs_GetRootVCache(tvc->mvid.target_root, treq, tvolp);
                    } else {
-                       tvc = afs_GetVCache(tvc->mvid.target_root, treq, NULL, NULL);
+                       tvc = afs_GetVCache(tvc->mvid.target_root, treq);
                    }
                    afs_PutVCache(uvc); /* we're done with it */
 
index a563652..015ea3a 100644 (file)
@@ -255,13 +255,11 @@ afs_remove(OSI_VC_DECL(adp), char *aname, afs_ucred_t *acred)
        if (tdc) {
            code = afs_dir_Lookup(tdc, aname, &unlinkFid.Fid);
            if (code == 0) {
-               afs_int32 cached = 0;
-
                unlinkFid.Cell = adp->f.fid.Cell;
                unlinkFid.Fid.Volume = adp->f.fid.Fid.Volume;
                if (unlinkFid.Fid.Unique == 0) {
                    tvc =
-                       afs_LookupVCache(&unlinkFid, treq, &cached, adp,
+                       afs_LookupVCache(&unlinkFid, treq, adp,
                                         aname);
                } else {
                    ObtainReadLock(&afs_xvcache);
@@ -437,7 +435,7 @@ afs_remunlink(struct vcache *avc, int doit)
            dirFid.Fid.Volume = avc->f.fid.Fid.Volume;
            dirFid.Fid.Vnode = avc->f.parent.vnode;
            dirFid.Fid.Unique = avc->f.parent.unique;
-           adp = afs_GetVCache(&dirFid, treq, NULL, NULL);
+           adp = afs_GetVCache(&dirFid, treq);
 
            if (adp) {
                tdc = afs_FindDCache(adp, (afs_size_t) 0);
index fcf2bcb..f899d31 100644 (file)
@@ -361,10 +361,10 @@ afsrename(struct vcache *aodp, char *aname1, struct vcache *andp,
        unlinkFid.Cell = aodp->f.fid.Cell;
        tvc = NULL;
        if (!unlinkFid.Fid.Unique) {
-           tvc = afs_LookupVCache(&unlinkFid, areq, NULL, aodp, aname1);
+           tvc = afs_LookupVCache(&unlinkFid, areq, aodp, aname1);
        }
        if (!tvc)               /* lookup failed or wasn't called */
-           tvc = afs_GetVCache(&unlinkFid, areq, NULL, NULL);
+           tvc = afs_GetVCache(&unlinkFid, areq);
 
        if (tvc) {
            ObtainWriteLock(&tvc->lock, 151);
@@ -392,9 +392,9 @@ afsrename(struct vcache *aodp, char *aname1, struct vcache *andp,
        fileFid.Fid.Volume = aodp->f.fid.Fid.Volume;
        fileFid.Cell = aodp->f.fid.Cell;
        if (!fileFid.Fid.Unique)
-           tvc = afs_LookupVCache(&fileFid, areq, NULL, andp, aname2);
+           tvc = afs_LookupVCache(&fileFid, areq, andp, aname2);
        else
-           tvc = afs_GetVCache(&fileFid, areq, NULL, (struct vcache *)0);
+           tvc = afs_GetVCache(&fileFid, areq);
        if (tvc && (vType(tvc) == VDIR)) {
            ObtainWriteLock(&tvc->lock, 152);
            tdc1 = afs_FindDCache(tvc, (afs_size_t) 0);
index 5fd36bf..cb9c017 100644 (file)
@@ -62,7 +62,7 @@ afs_osi_vget(struct vcache **avcpp, struct fid *afidp, struct vrequest *areqp)
        /* didn't find an entry. */
        code = afs_CreateReq(&treq, afs_osi_credp);
        if (code == 0) {
-           *avcpp = afs_GetVCache(&vfid, treq, NULL, NULL);
+           *avcpp = afs_GetVCache(&vfid, treq);
            afs_DestroyReq(treq);
        }
     }
index cd6907f..6848138 100644 (file)
@@ -2226,9 +2226,9 @@ DECL_PIOCTL(PNewStatMount)
     tfid.Cell = avc->f.fid.Cell;
     tfid.Fid.Volume = avc->f.fid.Fid.Volume;
     if (!tfid.Fid.Unique && (avc->f.states & CForeign)) {
-       tvc = afs_LookupVCache(&tfid, areq, NULL, avc, bufp);
+       tvc = afs_LookupVCache(&tfid, areq, avc, bufp);
     } else {
-       tvc = afs_GetVCache(&tfid, areq, NULL, NULL);
+       tvc = afs_GetVCache(&tfid, areq);
     }
     if (!tvc) {
        code = EIO;
@@ -3287,9 +3287,9 @@ DECL_PIOCTL(PRemoveMount)
     tfid.Cell = avc->f.fid.Cell;
     tfid.Fid.Volume = avc->f.fid.Fid.Volume;
     if (!tfid.Fid.Unique && (avc->f.states & CForeign)) {
-       tvc = afs_LookupVCache(&tfid, areq, NULL, avc, bufp);
+       tvc = afs_LookupVCache(&tfid, areq, avc, bufp);
     } else {
-       tvc = afs_GetVCache(&tfid, areq, NULL, NULL);
+       tvc = afs_GetVCache(&tfid, areq);
     }
     if (!tvc) {
        code = EIO;
@@ -4844,9 +4844,9 @@ DECL_PIOCTL(PFlushMount)
     tfid.Cell = avc->f.fid.Cell;
     tfid.Fid.Volume = avc->f.fid.Fid.Volume;
     if (!tfid.Fid.Unique && (avc->f.states & CForeign)) {
-       tvc = afs_LookupVCache(&tfid, areq, NULL, avc, bufp);
+       tvc = afs_LookupVCache(&tfid, areq, avc, bufp);
     } else {
-       tvc = afs_GetVCache(&tfid, areq, NULL, NULL);
+       tvc = afs_GetVCache(&tfid, areq);
     }
     if (!tvc) {
        code = EIO;
@@ -4982,7 +4982,7 @@ DECL_PIOCTL(PPrefetchFromTape)
     tfid.Fid.Vnode = Fid->Vnode;
     tfid.Fid.Unique = Fid->Unique;
 
-    tvc = afs_GetVCache(&tfid, areq, NULL, NULL);
+    tvc = afs_GetVCache(&tfid, areq);
     if (!tvc) {
        afs_Trace3(afs_iclSetp, CM_TRACE_PREFETCHCMD, ICL_TYPE_POINTER, tvc,
                   ICL_TYPE_FID, &tfid, ICL_TYPE_FID, &avc->f.fid);
@@ -5053,7 +5053,7 @@ DECL_PIOCTL(PFsCmd)
     tfid.Fid.Vnode = Fid->Vnode;
     tfid.Fid.Unique = Fid->Unique;
 
-    tvc = afs_GetVCache(&tfid, areq, NULL, NULL);
+    tvc = afs_GetVCache(&tfid, areq);
     afs_Trace3(afs_iclSetp, CM_TRACE_RESIDCMD, ICL_TYPE_POINTER, tvc,
               ICL_TYPE_INT32, Inputs->command, ICL_TYPE_FID, &tfid);
     if (!tvc)
index ec28e80..e2355e9 100644 (file)
@@ -1083,13 +1083,12 @@ extern afs_int32 afs_FlushVCBs(afs_int32 lockit);
 extern void afs_InactiveVCache(struct vcache *avc, afs_ucred_t *acred);
 extern struct vcache *afs_LookupVCache(struct VenusFid *afid,
                                       struct vrequest *areq,
-                                      afs_int32 * cached, struct vcache *adp,
+                                      struct vcache *adp,
                                       char *aname);
 extern void afs_FlushAllVCaches(void);
 extern int afs_FlushVCache(struct vcache *avc, int *slept);
 extern struct vcache *afs_GetRootVCache(struct VenusFid *afid,
                                        struct vrequest *areq,
-                                       afs_int32 * cached,
                                        struct volume *tvolp);
 extern struct vcache *afs_NewVCache(struct VenusFid *afid,
                                    struct server *serverp);
@@ -1097,8 +1096,7 @@ extern struct vcache *afs_NewBulkVCache(struct VenusFid *afid,
                                        struct server *serverp, int seq);
 extern int afs_VerifyVCache2(struct vcache *avc, struct vrequest *areq);
 extern struct vcache *afs_GetVCache(struct VenusFid *afid,
-                                   struct vrequest *areq, afs_int32 * cached,
-                                   struct vcache *avc);
+                                   struct vrequest *areq);
 extern void afs_PutVCache(struct vcache *avc);
 extern int afs_RefVCache(struct vcache *avc);
 
index 84cd6de..97e2acf 100644 (file)
@@ -1246,7 +1246,7 @@ afs_VerifyVCache2(struct vcache *avc, struct vrequest *areq)
     ReleaseWriteLock(&avc->lock);
 
     /* fetch the status info */
-    tvc = afs_GetVCache(&avc->f.fid, areq, NULL, avc);
+    tvc = afs_GetVCache(&avc->f.fid, areq);
     if (!tvc)
        return EIO;
     /* Put it back; caller has already incremented vrefCount */
@@ -1643,8 +1643,6 @@ afs_RemoteLookup(struct VenusFid *afid, struct vrequest *areq,
  * \param afid File ID.
  * \param areq Ptr to associated vrequest structure, specifying the
  *  user whose authentication tokens will be used.
- * \param avc Caller may already have a vcache for this file, which is
- *  already held.
  *
  * \note Environment:
  *     The cache entry is returned with an increased vrefCount field.
@@ -1664,13 +1662,9 @@ afs_RemoteLookup(struct VenusFid *afid, struct vrequest *areq,
  *     locking directories in a constant order.
  *
  * \note NB.  NewVCache -> FlushVCache presently (4/10/95) drops the xvcache lock.
- *
- * \note Might have a vcache structure already, which must
- *  already be held by the caller
  */
 struct vcache *
-afs_GetVCache(struct VenusFid *afid, struct vrequest *areq,
-             afs_int32 * cached, struct vcache *avc)
+afs_GetVCache(struct VenusFid *afid, struct vrequest *areq)
 {
 
     afs_int32 code, newvcache = 0;
@@ -1680,9 +1674,6 @@ afs_GetVCache(struct VenusFid *afid, struct vrequest *areq,
 
     AFS_STATCNT(afs_GetVCache);
 
-    if (cached)
-       *cached = 0;            /* Init just in case */
-
 #if    defined(AFS_SGI_ENV) && !defined(AFS_SGI53_ENV)
   loop:
 #endif
@@ -1698,8 +1689,6 @@ afs_GetVCache(struct VenusFid *afid, struct vrequest *areq,
 #endif
     }
     if (tvc) {
-       if (cached)
-           *cached = 1;
        osi_Assert((tvc->f.states & CVInit) == 0);
        /* If we are in readdir, return the vnode even if not statd */
        if ((tvc->f.states & CStatd) || afs_InReadDir(tvc)) {
@@ -1905,7 +1894,6 @@ afs_GetVCache(struct VenusFid *afid, struct vrequest *areq,
  *
  * \param afid
  * \param areq
- * \param cached Is element cached? If NULL, don't answer.
  * \param adp
  * \param aname
  *
@@ -1913,7 +1901,7 @@ afs_GetVCache(struct VenusFid *afid, struct vrequest *areq,
  */
 struct vcache *
 afs_LookupVCache(struct VenusFid *afid, struct vrequest *areq,
-                afs_int32 * cached, struct vcache *adp, char *aname)
+                struct vcache *adp, char *aname)
 {
     afs_int32 code, now, newvcache = 0;
     struct VenusFid nfid;
@@ -1927,8 +1915,6 @@ afs_LookupVCache(struct VenusFid *afid, struct vrequest *areq,
     afs_int32 retry;
 
     AFS_STATCNT(afs_GetVCache);
-    if (cached)
-       *cached = 0;            /* Init just in case */
 
 #if    defined(AFS_SGI_ENV) && !defined(AFS_SGI53_ENV)
   loop1:
@@ -1948,9 +1934,6 @@ afs_LookupVCache(struct VenusFid *afid, struct vrequest *areq,
        ObtainReadLock(&tvc->lock);
 
        if (tvc->f.states & CStatd) {
-           if (cached) {
-               *cached = 1;
-           }
            ReleaseReadLock(&tvc->lock);
            return tvc;
        }
@@ -2075,7 +2058,7 @@ afs_LookupVCache(struct VenusFid *afid, struct vrequest *areq,
 
 struct vcache *
 afs_GetRootVCache(struct VenusFid *afid, struct vrequest *areq,
-                 afs_int32 * cached, struct volume *tvolp)
+                 struct volume *tvolp)
 {
     afs_int32 code = 0, i, newvcache = 0, haveStatus = 0;
     afs_int32 getNewFid = 0;
@@ -2177,8 +2160,6 @@ afs_GetRootVCache(struct VenusFid *afid, struct vrequest *areq,
        newvcache = 1;
        afs_stats_cmperf.vcacheMisses++;
     } else {
-       if (cached)
-           *cached = 1;
        afs_stats_cmperf.vcacheHits++;
 #if    defined(AFS_DARWIN80_ENV)
        /* we already bumped the ref count in the for loop above */