diff options
| author | John Baldwin <jhb@FreeBSD.org> | 2025-12-09 15:59:19 +0000 |
|---|---|---|
| committer | John Baldwin <jhb@FreeBSD.org> | 2025-12-09 15:59:19 +0000 |
| commit | a35eae89cbddaac73d3b6c2b5c1b90e3a0c00997 (patch) | |
| tree | 89a2d3c9e3de046fba1764fe01707b90f6fc95c8 | |
| parent | e2b8be511e9064de29a847e7c1cfc9200e69394b (diff) | |
rman: Simplify initialization of internal globals
Use TAILQ_HEAD_INITIALIZER and MTX_SYSINIT to remove the 'once' code
from rman_init.
Reviewed by: des
Differential Revision: https://reviews.freebsd.org/D54142
| -rw-r--r-- | sys/kern/subr_rman.c | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/sys/kern/subr_rman.c b/sys/kern/subr_rman.c index dbb09850d049..b8528104ba28 100644 --- a/sys/kern/subr_rman.c +++ b/sys/kern/subr_rman.c @@ -102,8 +102,10 @@ SYSCTL_INT(_debug, OID_AUTO, rman_debug, CTLFLAG_RWTUN, static MALLOC_DEFINE(M_RMAN, "rman", "Resource manager"); -struct rman_head rman_head; +struct rman_head rman_head = TAILQ_HEAD_INITIALIZER(rman_head); static struct mtx rman_mtx; /* mutex to protect rman_head */ +MTX_SYSINIT(rman_mtx, &rman_mtx, "rman head", MTX_DEF); + static int int_rman_release_resource(struct rman *rm, struct resource_i *r); static __inline struct resource_i * @@ -121,14 +123,6 @@ int_alloc_resource(int malloc_flag) int rman_init(struct rman *rm) { - static int once = 0; - - if (once == 0) { - once = 1; - TAILQ_INIT(&rman_head); - mtx_init(&rman_mtx, "rman head", NULL, MTX_DEF); - } - if (rm->rm_start == 0 && rm->rm_end == 0) rm->rm_end = ~0; if (rm->rm_type == RMAN_UNINIT) |