git: 5d8e32a66c17 - main - vfs: retire VNODE_REFCOUNT_FENCE_* macros

Mateusz Guzik mjg at FreeBSD.org
Sat Sep 18 10:18:20 UTC 2021


The branch main has been updated by mjg:

URL: https://cgit.FreeBSD.org/src/commit/?id=5d8e32a66c1700323c570d25b03672f35f4e0110

commit 5d8e32a66c1700323c570d25b03672f35f4e0110
Author:     Mateusz Guzik <mjg at FreeBSD.org>
AuthorDate: 2021-09-18 08:14:35 +0000
Commit:     Mateusz Guzik <mjg at FreeBSD.org>
CommitDate: 2021-09-18 10:16:00 +0000

    vfs: retire VNODE_REFCOUNT_FENCE_* macros
    
    They are unused as of last year.
---
 sys/kern/vfs_subr.c | 16 ----------------
 1 file changed, 16 deletions(-)

diff --git a/sys/kern/vfs_subr.c b/sys/kern/vfs_subr.c
index 23f6ec9cf3fd..33a556fbfa2b 100644
--- a/sys/kern/vfs_subr.c
+++ b/sys/kern/vfs_subr.c
@@ -121,22 +121,6 @@ static int	v_inval_buf_range_locked(struct vnode *vp, struct bufobj *bo,
 		    daddr_t startlbn, daddr_t endlbn);
 static void	vnlru_recalc(void);
 
-/*
- * These fences are intended for cases where some synchronization is
- * needed between access of v_iflags and lockless vnode refcount (v_holdcnt
- * and v_usecount) updates.  Access to v_iflags is generally synchronized
- * by the interlock, but we have some internal assertions that check vnode
- * flags without acquiring the lock.  Thus, these fences are INVARIANTS-only
- * for now.
- */
-#ifdef INVARIANTS
-#define	VNODE_REFCOUNT_FENCE_ACQ()	atomic_thread_fence_acq()
-#define	VNODE_REFCOUNT_FENCE_REL()	atomic_thread_fence_rel()
-#else
-#define	VNODE_REFCOUNT_FENCE_ACQ()
-#define	VNODE_REFCOUNT_FENCE_REL()
-#endif
-
 /*
  * Number of vnodes in existence.  Increased whenever getnewvnode()
  * allocates a new vnode, decreased in vdropl() for VIRF_DOOMED vnode.


More information about the dev-commits-src-all mailing list