all pools have their ipl set via pool_setipl, so fold it into pool_init.

the ioff argument to pool_init() is unused and has been for many
years, so this replaces it with an ipl argument. because the ipl
will be set on init we no longer need pool_setipl.

most of these changes have been done with coccinelle using the spatch
below. cocci sucks at formatting code though, so i fixed that by hand.

the manpage and subr_pool.c bits i did myself.

ok tedu@ jmatthew@

@ipl@
expression pp;
expression ipl;
expression s, a, o, f, m, p;
@@
-pool_init(pp, s, a, o, f, m, p);
-pool_setipl(pp, ipl);
+pool_init(pp, s, a, ipl, f, m, p);

1 files changed, 4 insertions, 7 deletions

diff --git a/sys/isofs/udf/udf_vfsops.c b/sys/isofs/udf/udf_vfsops.c
index 45df2b77535..d7ecaf4d8d9 100644
--- a/sys/isofs/udf/udf_vfsops.c
+++ b/sys/isofs/udf/udf_vfsops.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: udf_vfsops.c,v 1.55 2016/09/07 17:30:12 natano Exp $	*/
+/*	$OpenBSD: udf_vfsops.c,v 1.56 2016/09/15 02:00:18 dlg Exp $	*/
 
 /*
  * Copyright (c) 2001, 2002 Scott Long <scottl@freebsd.org>
@@ -102,15 +102,12 @@ const struct vfsops udf_vfsops = {
 int
 udf_init(struct vfsconf *foo)
 {
-	pool_init(&udf_trans_pool, MAXNAMLEN * sizeof(unicode_t), 0, 0,
+	pool_init(&udf_trans_pool, MAXNAMLEN * sizeof(unicode_t), 0, IPL_NONE,
 	    PR_WAITOK, "udftrpl", NULL);
-	pool_setipl(&udf_trans_pool, IPL_NONE);
-	pool_init(&unode_pool, sizeof(struct unode), 0, 0,
+	pool_init(&unode_pool, sizeof(struct unode), 0, IPL_NONE,
 	    PR_WAITOK, "udfndpl", NULL);
-	pool_setipl(&unode_pool, IPL_NONE);
-	pool_init(&udf_ds_pool, sizeof(struct udf_dirstream), 0, 0,
+	pool_init(&udf_ds_pool, sizeof(struct udf_dirstream), 0, IPL_NONE,
 	    PR_WAITOK, "udfdspl", NULL);
-	pool_setipl(&udf_ds_pool, IPL_NONE);
 
 	return (0);
 }