afs: remove dead code 10/13210/3
authorMark Vitale <mvitale@sinenomine.net>
Thu, 14 Jun 2018 18:38:54 +0000 (14:38 -0400)
committerBenjamin Kaduk <kaduk@mit.edu>
Tue, 19 Jun 2018 03:24:00 +0000 (23:24 -0400)
afs_CheckLocks has been dead code since openafs-ibm-1_0.

No functional change incurred.

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

src/afs/afs_prototypes.h
src/afs/afs_stats.h
src/afs/afs_util.c
src/venus/kdump.c

index fb8ddf4..1f371fc 100644 (file)
@@ -1028,7 +1028,6 @@ extern void print_internet_address(char *preamble, struct srvAddr *sa,
                                   struct rx_connection *rxconn);
 extern afs_int32 afs_data_pointer_to_int32(const void *p);
 
-extern void afs_CheckLocks(void);
 extern int afs_badop(void);
 extern int afs_noop(void);
 extern afs_int32 afs_data_pointer_to_int32(const void *p);
index 0d9eb91..bf383c8 100644 (file)
@@ -501,7 +501,7 @@ struct afs_MeanStats {
     AFS_CS(fprf)               /* afs_resource.c */ \
     AFS_CS(fprint)             /* afs_resource.c */ \
     AFS_CS(fprintn)            /* afs_resource.c */ \
-    AFS_CS(afs_CheckLocks)     /* afs_resource.c */ \
+    AFS_CS(afs_CheckLocks)     /* UNUSED */ \
     AFS_CS(puttofile)          /* afs_resource.c */ \
     AFS_CS(shutdown_AFS)       /* afs_resource.c */ \
     AFS_CS(afs_CheckCacheResets)       /* afs_resource.c */ \
index 8bd595c..491f522 100644 (file)
@@ -230,93 +230,6 @@ print_internet_address(char *preamble, struct srvAddr *sa, char *postamble,
     }
 }                              /*print_internet_address */
 
-
-
-/* run everywhere, checking locks */
-void
-afs_CheckLocks(void)
-{
-    int i;
-
-    afs_warn("Looking for locked data structures.\n");
-    afs_warn("conn %p, volume %p, user %p, cell %p, server %p\n", &afs_xconn,
-            &afs_xvolume, &afs_xuser, &afs_xcell, &afs_xserver);
-    {
-       struct vcache *tvc;
-       AFS_STATCNT(afs_CheckLocks);
-
-       for (i = 0; i < VCSIZE; i++) {
-           for (tvc = afs_vhashT[i]; tvc; tvc = tvc->hnext) {
-                if (tvc->f.states & CVInit) continue;
-#ifdef AFS_DARWIN80_ENV
-               if (vnode_isinuse(AFSTOV(tvc), 0))
-#else
-               if (VREFCOUNT(tvc))
-#endif
-                   afs_warn("Stat cache entry at %p is held\n", tvc);
-               if (CheckLock(&tvc->lock))
-                   afs_warn("Stat entry at %p is locked\n", tvc);
-           }
-       }
-    }
-    {
-       struct dcache *tdc;
-       for (i = 0; i < afs_cacheFiles; i++) {
-           tdc = afs_indexTable[i];
-           if (tdc) {
-               if (tdc->refCount)
-                   afs_warn("Disk entry %d at %p is held\n", i, tdc);
-           }
-           if (afs_indexFlags[i] & IFDataMod)
-               afs_warn("Disk entry %d at %p has IFDataMod flag set.\n", i,
-                        tdc);
-       }
-    }
-    {
-       struct srvAddr *sa;
-       struct server *ts;
-        struct sa_conn_vector *tcv;
-       for (i = 0; i < NSERVERS; i++) {
-           for (ts = afs_servers[i]; ts; ts = ts->next) {
-               if (ts->flags & SRVR_ISDOWN)
-                   afs_warn("Server entry %p is marked down\n", ts);
-               for (sa = ts->addr; sa; sa = sa->next_sa) {
-                    for (tcv = sa->conns; tcv; tcv = tcv->next) {
-                        if (tcv->refCount)
-                            afs_warn("conn at %p (server %x) is held\n", tcv,
-                                    sa->sa_ip);
-                   }
-               }
-           }
-       }
-    }
-    {
-       struct volume *tv;
-       for (i = 0; i < NVOLS; i++) {
-           for (tv = afs_volumes[i]; tv; tv = tv->next) {
-               if (CheckLock(&tv->lock))
-                   afs_warn("volume at %p is locked\n", tv);
-               if (tv->refCount)
-                   afs_warn("volume at %p is held\n", tv);
-           }
-       }
-    }
-    {
-       struct unixuser *tu;
-
-       for (i = 0; i < NUSERS; i++) {
-           for (tu = afs_users[i]; tu; tu = tu->next) {
-               if (CheckLock(&tu->lock))
-                   afs_warn("user at %p is locked\n", tu);
-               if (tu->refCount)
-                   afs_warn("user at %lx is held\n", (unsigned long)tu);
-           }
-       }
-    }
-    afs_warn("Done.\n");
-}
-
-
 int
 afs_noop(void)
 {
index 4ed65c4..596109c 100644 (file)
@@ -4350,7 +4350,6 @@ print_cmstats(struct afs_CMStats *cmp)
     printf("\t%10d afs_ConnByHost\n", cmp->callInfo.C_afs_ConnByHost);
     printf("\t%10d afs_ConnByMHosts\n", cmp->callInfo.C_afs_ConnByMHosts);
     printf("\t%10d afs_Analyze\n", cmp->callInfo.C_afs_Analyze);
-    printf("\t%10d afs_CheckLocks\n", cmp->callInfo.C_afs_CheckLocks);
     printf("\t%10d CheckVLServer\n", cmp->callInfo.C_CheckVLServer);
     printf("\t%10d afs_CheckCacheResets\n",
           cmp->callInfo.C_afs_CheckCacheResets);