git: 3454a7caa053 - main - kqueue: retire knlist_init_rw_reader()
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sun, 21 Aug 2022 05:23:34 UTC
The branch main has been updated by rew: URL: https://cgit.FreeBSD.org/src/commit/?id=3454a7caa053a2f44ad20216f908f472e9d43f1e commit 3454a7caa053a2f44ad20216f908f472e9d43f1e Author: Robert Wing <rew@FreeBSD.org> AuthorDate: 2022-08-21 05:17:39 +0000 Commit: Robert Wing <rew@FreeBSD.org> CommitDate: 2022-08-21 05:17:39 +0000 kqueue: retire knlist_init_rw_reader() Last usage was removed in afa85850e79c1839ec33efa1138206687b952cfa. Reviewed by: pauamma, melifaro, kib Differential Revision: https://reviews.freebsd.org/D36205 --- ObsoleteFiles.inc | 3 +++ share/man/man9/Makefile | 1 - share/man/man9/kqueue.9 | 24 ++++-------------------- sys/kern/kern_event.c | 33 --------------------------------- sys/sys/event.h | 1 - 5 files changed, 7 insertions(+), 55 deletions(-) diff --git a/ObsoleteFiles.inc b/ObsoleteFiles.inc index d7b622bc496c..66aa0c9c807d 100644 --- a/ObsoleteFiles.inc +++ b/ObsoleteFiles.inc @@ -52,6 +52,9 @@ # xargs -n1 | sort | uniq -d; # done +# 20220820: remove knlist_init_rw_reader() +OLD_FILES+=usr/share/man/man9/knlist_init_rw_reader.9.gz + # 20220813: minigzip(1) removed in favor of gzip(1) OLD_FILES+=usr/bin/minigzip OLD_FILES+=usr/share/man/man1/minigzip.1.gz diff --git a/share/man/man9/Makefile b/share/man/man9/Makefile index 1913f03a48fd..b99575b595ed 100644 --- a/share/man/man9/Makefile +++ b/share/man/man9/Makefile @@ -1366,7 +1366,6 @@ MLINKS+=kqueue.9 knlist_add.9 \ kqueue.9 knlist_empty.9 \ kqueue.9 knlist_init.9 \ kqueue.9 knlist_init_mtx.9 \ - kqueue.9 knlist_init_rw_reader.9 \ kqueue.9 knlist_remove.9 \ kqueue.9 knlist_remove_inevent.9 \ kqueue.9 knote_fdclose.9 \ diff --git a/share/man/man9/kqueue.9 b/share/man/man9/kqueue.9 index 53f55b8efe03..15a430d4a5a3 100644 --- a/share/man/man9/kqueue.9 +++ b/share/man/man9/kqueue.9 @@ -24,14 +24,14 @@ .\" .\" $FreeBSD$ .\" -.Dd October 12, 2021 +.Dd August 20, 2022 .Dt KQUEUE 9 .Os .Sh NAME .Nm kqueue_add_filteropts , kqueue_del_filteropts , .Nm kqfd_register , .Nm knote_fdclose , -.Nm knlist_init , knlist_init_mtx , knlist_init_rw_reader , +.Nm knlist_init , knlist_init_mtx , .Nm knlist_add , knlist_remove , knlist_remove_inevent , knlist_empty , .Nm knlist_clear , knlist_delete , knlist_destroy , .Nm KNOTE_LOCKED , KNOTE_UNLOCKED @@ -57,8 +57,6 @@ .Ft void .Fn knlist_init_mtx "struct knlist *knl" "struct mtx *lock" .Ft void -.Fn knlist_init_rw_reader "struct knlist *knl" "struct rwlock *lock" -.Ft void .Fn knlist_add "struct knlist *knl" "struct knote *kn" "int islocked" .Ft void .Fn knlist_remove "struct knlist *knl" "struct knote *kn" "int islocked" @@ -260,10 +258,9 @@ is not required, but is commonly used. If used, the .Vt knlist must be initialized with either -.Fn knlist_init , -.Fn knlist_init_mtx +.Fn knlist_init or -.Fn knlist_init_rw_reader . +.Fn knlist_init_mtx . The .Vt knlist structure may be embedded into the object structure. @@ -308,19 +305,6 @@ style lock. .Pp The function -.Fn knlist_init_rw_reader -may be used to initialize a -.Vt knlist -when -.Fa lock -is a -.Xr rwlock 9 -read lock. -Lock is acquired via -.Fn rw_rlock -function. -.Pp -The function .Fn knlist_empty returns true when there are no .Vt knotes diff --git a/sys/kern/kern_event.c b/sys/kern/kern_event.c index 5b24741028a9..f952a5f00340 100644 --- a/sys/kern/kern_event.c +++ b/sys/kern/kern_event.c @@ -45,7 +45,6 @@ __FBSDID("$FreeBSD$"); #include <sys/limits.h> #include <sys/lock.h> #include <sys/mutex.h> -#include <sys/rwlock.h> #include <sys/proc.h> #include <sys/malloc.h> #include <sys/unistd.h> @@ -2506,30 +2505,6 @@ knlist_mtx_assert_lock(void *arg, int what) mtx_assert((struct mtx *)arg, MA_NOTOWNED); } -static void -knlist_rw_rlock(void *arg) -{ - - rw_rlock((struct rwlock *)arg); -} - -static void -knlist_rw_runlock(void *arg) -{ - - rw_runlock((struct rwlock *)arg); -} - -static void -knlist_rw_assert_lock(void *arg, int what) -{ - - if (what == LA_LOCKED) - rw_assert((struct rwlock *)arg, RA_LOCKED); - else - rw_assert((struct rwlock *)arg, RA_UNLOCKED); -} - void knlist_init(struct knlist *knl, void *lock, void (*kl_lock)(void *), void (*kl_unlock)(void *), @@ -2575,14 +2550,6 @@ knlist_alloc(struct mtx *lock) return (knl); } -void -knlist_init_rw_reader(struct knlist *knl, struct rwlock *lock) -{ - - knlist_init(knl, lock, knlist_rw_rlock, knlist_rw_runlock, - knlist_rw_assert_lock); -} - void knlist_destroy(struct knlist *knl) { diff --git a/sys/sys/event.h b/sys/sys/event.h index 05eddfd681b1..1007b88828ae 100644 --- a/sys/sys/event.h +++ b/sys/sys/event.h @@ -337,7 +337,6 @@ int knlist_empty(struct knlist *knl); void knlist_init(struct knlist *knl, void *lock, void (*kl_lock)(void *), void (*kl_unlock)(void *), void (*kl_assert_lock)(void *, int)); void knlist_init_mtx(struct knlist *knl, struct mtx *lock); -void knlist_init_rw_reader(struct knlist *knl, struct rwlock *lock); void knlist_destroy(struct knlist *knl); void knlist_cleardel(struct knlist *knl, struct thread *td, int islocked, int killkn);