Date: Sat, 18 Sep 2021 10:18:20 GMT From: Mateusz Guzik <mjg@FreeBSD.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Subject: git: 5d8e32a66c17 - main - vfs: retire VNODE_REFCOUNT_FENCE_* macros Message-ID: <202109181018.18IAIK0I065958@gitrepo.freebsd.org>
next in thread | raw e-mail | index | archive | help
The branch main has been updated by mjg: URL: https://cgit.FreeBSD.org/src/commit/?id=5d8e32a66c1700323c570d25b03672f35f4e0110 commit 5d8e32a66c1700323c570d25b03672f35f4e0110 Author: Mateusz Guzik <mjg@FreeBSD.org> AuthorDate: 2021-09-18 08:14:35 +0000 Commit: Mateusz Guzik <mjg@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.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202109181018.18IAIK0I065958>