From: Tom Keiser <tkeiser@sinenomine.net>
Date: Wed, 30 Jun 2010 22:40:20 +0000 (-0400)
Subject: DAFS: fix VOL_HDR_IN_LRU state bit tracking
X-Git-Tag: openafs-devel-1_5_75~34
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=3330b9d17b2767318eef6222aceabdee47bbc703;p=packages%2Fo%2Fopenafs.git

DAFS: fix VOL_HDR_IN_LRU state bit tracking

For the case where the cached vp->header can be re-used by
GetVolumeHeader(), we have not been re-setting the VOL_HDR_IN_LRU
state bit.  Although this has not affected correctness (this
bit is merely used to aid in debugging), we should be resetting
it in all relevant cases...

Change-Id: I2ad2b8c5070470beb067efdcda6f76f6b72c006c
Reviewed-on: http://gerrit.openafs.org/2308
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
---

diff --git a/src/vol/volume.c b/src/vol/volume.c
index ffd76aa11..c28f16774 100644
--- a/src/vol/volume.c
+++ b/src/vol/volume.c
@@ -7309,6 +7309,9 @@ GetVolumeHeader(register Volume * vp)
 	    hd = vp->header;
 	    queue_Remove(hd);
 	    assert(hd->back == vp);
+#ifdef AFS_DEMAND_ATTACH_FS
+            V_attachFlags(vp) &= ~(VOL_HDR_IN_LRU);
+#endif
 	} else {
 	    /* we need to grab a new element off the LRU */
 	    if (queue_IsNotEmpty(&volume_hdr_LRU)) {