From: Andrew Deason <adeason@sinenomine.net>
Date: Wed, 28 Nov 2012 21:12:12 +0000 (-0600)
Subject: afs: Apply VLRU safety check for Linux too
X-Git-Tag: upstream/1.8.0_pre1^2~1740
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=696db8667d12b1c5639515840f69c9a46d2b866d;p=packages%2Fo%2Fopenafs.git

afs: Apply VLRU safety check for Linux too

This invariant should apply to all platforms, not just those with
dynamic vcaches. Since this prevents an infinite loop if the list os
corrupt or something, having this around everywhere seems useful. So,
drop the check for afsd_dynamic_vcaches.

Change-Id: I90e438eb33f40762346d1c5a7138ed127fdcdcc6
Reviewed-on: http://gerrit.openafs.org/8555
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
---

diff --git a/src/afs/afs_vcache.c b/src/afs/afs_vcache.c
index 4acfd345e..7620ed4ad 100644
--- a/src/afs/afs_vcache.c
+++ b/src/afs/afs_vcache.c
@@ -716,7 +716,7 @@ afs_ShakeLooseVCaches(afs_int32 anumber)
 	uq = QPrev(tq);
 	if (tvc->f.states & CVFlushed) {
 	    refpanic("CVFlushed on VLRU");
-	} else if (!afsd_dynamic_vcaches && i++ > afs_vcount) {
+	} else if (i++ > afs_vcount) {
 	    refpanic("Found too many AFS vnodes on VLRU (VLRU cycle?)");
 	} else if (QNext(uq) != tq) {
 	    refpanic("VLRU inconsistent");