diff options
| author | David Gwynne <dlg@cvs.openbsd.org> | 2017-02-08 05:28:31 +0000 |
|---|---|---|
| committer | David Gwynne <dlg@cvs.openbsd.org> | 2017-02-08 05:28:31 +0000 |
| commit | 15f503533d2e69feae2ac479e33277951b12840e (patch) | |
| tree | 6085853d279add286e901606d223779f7f59595d | |
| parent | dce59fe6e1e128ed03c16e493686ecd655007b83 (diff) | |
the splvm() in pool_gc_pages is unecessary now.
all pools set their ipls unconditionally now, so there isn't a need
to second guess them.
pointed out by and ok jmatthew@
| -rw-r--r-- | sys/kern/subr_pool.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/sys/kern/subr_pool.c b/sys/kern/subr_pool.c index 06955059936..177b4083c8e 100644 --- a/sys/kern/subr_pool.c +++ b/sys/kern/subr_pool.c @@ -1,4 +1,4 @@ -/* $OpenBSD: subr_pool.c,v 1.205 2017/01/24 09:54:41 mpi Exp $ */ +/* $OpenBSD: subr_pool.c,v 1.206 2017/02/08 05:28:30 dlg Exp $ */ /* $NetBSD: subr_pool.c,v 1.61 2001/09/26 07:14:56 chs Exp $ */ /*- @@ -1446,10 +1446,8 @@ pool_gc_pages(void *null) { struct pool *pp; struct pool_page_header *ph, *freeph; - int s; rw_enter_read(&pool_lock); - s = splvm(); /* XXX go to splvm until all pools _setipl properly */ SIMPLEQ_FOREACH(pp, &pool_head, pr_poollist) { if (pp->pr_nidle <= pp->pr_minpages || /* guess */ !mtx_enter_try(&pp->pr_mtx)) /* try */ @@ -1469,7 +1467,6 @@ pool_gc_pages(void *null) if (freeph != NULL) pool_p_free(pp, freeph); } - splx(s); rw_exit_read(&pool_lock); timeout_add_sec(&pool_gc_tick, 1); |