From 696db8667d12b1c5639515840f69c9a46d2b866d Mon Sep 17 00:00:00 2001 From: Andrew Deason Date: Wed, 28 Nov 2012 15:12:12 -0600 Subject: [PATCH] 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 Reviewed-by: Derrick Brashear --- src/afs/afs_vcache.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/afs/afs_vcache.c b/src/afs/afs_vcache.c index 4acfd345e6..7620ed4adf 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");