]> git.michaelhowe.org Git - packages/o/openafs.git/commitdiff
afs: Apply VLRU safety check for Linux too
authorAndrew Deason <adeason@sinenomine.net>
Wed, 28 Nov 2012 21:12:12 +0000 (15:12 -0600)
committerStephan Wiesand <stephan.wiesand@desy.de>
Thu, 4 Apr 2013 12:59:48 +0000 (05:59 -0700)
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>
src/afs/afs_vcache.c

index 6f0bf50e75e0ea26e1363623a8e6107ba9bfedee..a9a323292589bd7b6a08cae5b140d49ca2a38599 100644 (file)
@@ -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");