From 96389a9996eeb8b6e03b2ddadacefae1ceb1cb44 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. Reviewed-on: http://gerrit.openafs.org/8555 Tested-by: BuildBot Reviewed-by: Derrick Brashear (cherry picked from commit 696db8667d12b1c5639515840f69c9a46d2b866d) Change-Id: I81e4c7c7678a9f4a3e02952e243735d59b3af35c Reviewed-on: http://gerrit.openafs.org/9502 Tested-by: BuildBot Reviewed-by: Derrick Brashear Reviewed-by: Andrew Deason Reviewed-by: Stephan Wiesand --- 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 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"); -- 2.39.5