diff options
author | Mark Johnston <markj@FreeBSD.org> | 2024-01-04 13:11:44 +0000 |
---|---|---|
committer | Mark Johnston <markj@FreeBSD.org> | 2024-01-11 14:19:45 +0000 |
commit | 208e68be1faed42ede97431476b33fc88cfd2429 (patch) | |
tree | f763c76196ce7003df2433522310e4ee9b93f6d1 | |
parent | b8233245b8d04816d13ea0c4441c0ded57323840 (diff) | |
download | src-208e68be1faed42ede97431476b33fc88cfd2429.tar.gz src-208e68be1faed42ede97431476b33fc88cfd2429.zip |
vfs: Simplify vrefact()
refcount_acquire() returns the old value, just use that. No functional
change intended.
Reviewed by: kib
MFC after: 1 week
Differential Revision: https://reviews.freebsd.org/D43255
(cherry picked from commit 27f4eda3ddce4009a0ec9be68f474fe3be38933f)
-rw-r--r-- | sys/kern/vfs_subr.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/sys/kern/vfs_subr.c b/sys/kern/vfs_subr.c index 0fe2bc46446c..811c5cef45d1 100644 --- a/sys/kern/vfs_subr.c +++ b/sys/kern/vfs_subr.c @@ -3360,14 +3360,11 @@ vref(struct vnode *vp) void vrefact(struct vnode *vp) { + int old __diagused; CTR2(KTR_VFS, "%s: vp %p", __func__, vp); -#ifdef INVARIANTS - int old = atomic_fetchadd_int(&vp->v_usecount, 1); + old = refcount_acquire(&vp->v_usecount); VNASSERT(old > 0, vp, ("%s: wrong use count %d", __func__, old)); -#else - refcount_acquire(&vp->v_usecount); -#endif } void |