]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
vfs: fix vnlru marker handling for filtered/unfiltered cases
authorMateusz Guzik <mjg@FreeBSD.org>
Wed, 17 Mar 2021 21:33:47 +0000 (22:33 +0100)
committerMateusz Guzik <mjg@FreeBSD.org>
Thu, 18 Mar 2021 15:00:50 +0000 (15:00 +0000)
commit4f2e8545f7fd842bb54f0a23166965f1ae58e522
tree4a964f6d03edb4dcaf3b0c73a5ff71aa7f700ca9
parentb65d2c31b1de93e697e6429b962d9fa0b78e4f70
vfs: fix vnlru marker handling for filtered/unfiltered cases

The global list has a marker with an invariant that free vnodes are
placed somewhere past that. A caller which performs filtering (like ZFS)
can move said marker all the way to the end, across free vnodes which
don't match. Then a caller which does not perform filtering will fail to
find them. This makes vn_alloc_hard sleep for 1 second instead of
reclaiming, resulting in significant stalls.

Fix the problem by requiring an explicit marker by callers which do
filtering.

As a temporary measure extend vnlru_free to restart if it fails to
reclaim anything.

Big thanks go to the reporter for testing several iterations of the
patch.

Reported by: Yamagi <lists yamagi.org>
Tested by: Yamagi <lists yamagi.org>
Reviewed by: kib
Differential Revision: https://reviews.freebsd.org/D29324

(cherry picked from commit e9272225e6bed840b00eef1c817b188c172338ee)
sys/contrib/openzfs/module/os/freebsd/zfs/arc_os.c
sys/kern/vfs_subr.c
sys/sys/vnode.h