HomeFreeBSD

vfs: avoid exposing partially constructed vnodes

Description

vfs: avoid exposing partially constructed vnodes

If multiple threads race calling vfs_hash_insert() while creating vnodes
with the same identity, all of the vnodes which lose the race must be
destroyed before any other thread can see them. Previously this was
accomplished by the vput() in vfs_hash_insert() resulting in the vnode's
VOP_INACTIVE() method calling vgone() before the vnode lock was unlocked,
but at some point changes to the the vnode refcount/inactive logic have caused
that to no longer work, leading to crashes, so instead vfs_hash_insert()
must call vgone() itself before calling vput() on vnodes which lose the race.

Reviewed by: mjg, kib
Sponsored by: Netflix
Differential Revision: https://reviews.freebsd.org/D26291

Details

Provenance
chsAuthored on
Reviewer
mjg
Differential Revision
D26291: vfs: avoid exposing partially constructed vnodes
Parents
rS365350: bbr: remove unused static function
Branches
Unknown
Tags
Unknown