Date: Tue, 12 Aug 2025 12:56:17 GMT From: Mark Johnston <markj@FreeBSD.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org Subject: git: 0188e350666d - stable/14 - p9fs: Set vnode state in p9fs_vget_common() Message-ID: <202508121256.57CCuHxn088544@gitrepo.freebsd.org>
next in thread | raw e-mail | index | archive | help
The branch stable/14 has been updated by markj: URL: https://cgit.FreeBSD.org/src/commit/?id=0188e350666dbbd68e4d4b4f2f900309a4c230bc commit 0188e350666dbbd68e4d4b4f2f900309a4c230bc Author: Mark Johnston <markj@FreeBSD.org> AuthorDate: 2025-08-04 15:07:05 +0000 Commit: Mark Johnston <markj@FreeBSD.org> CommitDate: 2025-08-12 12:49:47 +0000 p9fs: Set vnode state in p9fs_vget_common() Once a new vnode is visible from the mountpoint hash, we should set its state from VSTATE_UNINITIALIZED to VSTATE_CONSTRUCTED. I do not think this affects correctness at all, but the bug trips a check in vop_unlock_debugpost(), previously hidden under options DEBUG_VFS_LOCKS. Reviewed by: kib MFC after: 1 week Differential Revision: https://reviews.freebsd.org/D51720 (cherry picked from commit e2ac6de6e4edc1c6e7bfdfb0ec8fcf62f46d503f) --- sys/fs/p9fs/p9fs_vfsops.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sys/fs/p9fs/p9fs_vfsops.c b/sys/fs/p9fs/p9fs_vfsops.c index 3451bc052187..e0e91e7e1709 100644 --- a/sys/fs/p9fs/p9fs_vfsops.c +++ b/sys/fs/p9fs/p9fs_vfsops.c @@ -374,7 +374,7 @@ p9fs_vget_common(struct mount *mp, struct p9fs_node *np, int flags, STAILQ_INSERT_TAIL(&vses->virt_node_list, np, p9fs_node_next); np->flags |= P9FS_NODE_IN_SESSION; P9FS_UNLOCK(vses); - + vn_set_state(vp, VSTATE_CONSTRUCTED); *vpp = vp; } else { /*
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202508121256.57CCuHxn088544>