]> 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)
committerDerrick Brashear <shadow@your-file-system.com>
Wed, 5 Dec 2012 16:22:03 +0000 (08:22 -0800)
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>
src/afs/afs_vcache.c

index 4acfd345e6dd88f464684616200aabb17e712f11..7620ed4adfa418b2d2eb0fb0d51b4b787457752d 100644 (file)
@@ -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");