git: 4e91a0b9fece - main - nullfs: stop using insmntque1
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Thu, 27 Jan 2022 00:12:25 UTC
The branch main has been updated by mjg: URL: https://cgit.FreeBSD.org/src/commit/?id=4e91a0b9fece53711d5ff8fbc56762bd6d5fee4f commit 4e91a0b9fece53711d5ff8fbc56762bd6d5fee4f Author: Mateusz Guzik <mjg@FreeBSD.org> AuthorDate: 2022-01-26 23:53:59 +0000 Commit: Mateusz Guzik <mjg@FreeBSD.org> CommitDate: 2022-01-26 23:54:47 +0000 nullfs: stop using insmntque1 It adds nothing of value over insmntque. --- sys/fs/nullfs/null_subr.c | 17 ++++++----------- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/sys/fs/nullfs/null_subr.c b/sys/fs/nullfs/null_subr.c index fb6a13449df7..5f8cbfd4ce31 100644 --- a/sys/fs/nullfs/null_subr.c +++ b/sys/fs/nullfs/null_subr.c @@ -182,14 +182,6 @@ null_destroy_proto(struct vnode *vp, void *xp) free(xp, M_NULLFSNODE); } -static void -null_insmntque_dtr(struct vnode *vp, void *xp) -{ - - vput(((struct null_node *)xp)->null_lowervp); - null_destroy_proto(vp, xp); -} - /* * Make a new or get existing nullfs node. * Vp is the alias vnode, lowervp is the lower vnode. @@ -219,7 +211,7 @@ null_nodeget(mp, lowervp, vpp) } /* - * The insmntque1() call below requires the exclusive lock on + * The insmntque() call below requires the exclusive lock on * the nullfs vnode. Upgrade the lock now if hash failed to * provide ready to use vnode. */ @@ -252,9 +244,12 @@ null_nodeget(mp, lowervp, vpp) vp->v_type = lowervp->v_type; vp->v_data = xp; vp->v_vnlock = lowervp->v_vnlock; - error = insmntque1(vp, mp, null_insmntque_dtr, xp); - if (error != 0) + error = insmntque(vp, mp); + if (error != 0) { + vput(lowervp); + null_destroy_proto(vp, xp); return (error); + } if (lowervp == MOUNTTONULLMOUNT(mp)->nullm_lowerrootvp) vp->v_vflag |= VV_ROOT;