Don't clear afs_stats_cmperf too early during shutdown
authorMarc Dionne <marc.c.dionne@gmail.com>
Thu, 4 Feb 2010 02:34:24 +0000 (21:34 -0500)
committerDerrick Brashear <shadow|account-1000005@unknown>
Thu, 4 Feb 2010 02:56:12 +0000 (18:56 -0800)
Commit 21cbf7fee0a089d94f62baa7df2422e7bc8293f7 activated some
previously unused cleanup code.  Part of this clears afs_stats_cmperf,
but these stats are used later to check for leaks of chunks allocated
with afs_AllocSmallSpace and afs_AllocLargeSpace.  The result is a
message about unfreed blocks in the syslog with negative counts.

Since the structure is already cleared later in the shutdown process,
just remove this instance.

Change-Id: I6bf2ab37752d2623d072469fb87fa6d06e85b9ec
Reviewed-on: http://gerrit.openafs.org/1229
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>

src/afs/afs_init.c

index 8e511cb..a0dde1b 100644 (file)
@@ -712,7 +712,6 @@ shutdown_cache(void)
        cacheInfoModTime = 0;
 
        afs_fsfragsize = 1023;
-       memset(&afs_stats_cmperf, 0, sizeof(afs_stats_cmperf));
        memset(&cacheDev, 0, sizeof(struct osi_dev));
        osi_dnlc_shutdown();
     }