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.6.3^2~29
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=96389a9996eeb8b6e03b2ddadacefae1ceb1cb44;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.

Reviewed-on: http://gerrit.openafs.org/8555
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
(cherry picked from commit 696db8667d12b1c5639515840f69c9a46d2b866d)

Change-Id: I81e4c7c7678a9f4a3e02952e243735d59b3af35c
Reviewed-on: http://gerrit.openafs.org/9502
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>
---

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