diff options
author | Andriy Gapon <avg@FreeBSD.org> | 2022-10-26 22:00:58 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-10-26 22:00:58 +0000 |
commit | 41133c97949af43daadee0503a9842a8dce8f0fd (patch) | |
tree | 61c7d4208226a4bde2acb65851a359aef18e33ab /include/os | |
parent | eeddd805728914c9bdbea394588f31ca22dcb657 (diff) | |
download | src-41133c97949af43daadee0503a9842a8dce8f0fd.tar.gz src-41133c97949af43daadee0503a9842a8dce8f0fd.zip |
FreeBSD: vn_flush_cached_data: observe vnode locking contract
vm_object_page_clean() expects that the associated vnode is locked
as VOP_PUTPAGES() may get called on the vnode.
Reviewed-by: Ryan Moeller <ryan@iXsystems.com>
Signed-off-by: Andriy Gapon <avg@FreeBSD.org>
Closes #14079
Diffstat (limited to 'include/os')
-rw-r--r-- | include/os/freebsd/spl/sys/vnode.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/include/os/freebsd/spl/sys/vnode.h b/include/os/freebsd/spl/sys/vnode.h index e0f79a1116b2..bbadf569c776 100644 --- a/include/os/freebsd/spl/sys/vnode.h +++ b/include/os/freebsd/spl/sys/vnode.h @@ -96,9 +96,11 @@ vn_flush_cached_data(vnode_t *vp, boolean_t sync) if (vp->v_object->flags & OBJ_MIGHTBEDIRTY) { #endif int flags = sync ? OBJPC_SYNC : 0; + vn_lock(vp, LK_SHARED | LK_RETRY); zfs_vmobject_wlock(vp->v_object); vm_object_page_clean(vp->v_object, 0, 0, flags); zfs_vmobject_wunlock(vp->v_object); + VOP_UNLOCK(vp); } } #endif |