Subject: Re: CVS commit: src/sys/kern
To: Antti Kantee <pooka@netbsd.org>
From: Thor Lancelot Simon <tls@netbsd.org>
List: tech-kern
Date: 10/22/2006 04:16:02
On Sat, Oct 21, 2006 at 05:01:56PM +0000, Antti Kantee wrote:
> 
> Module Name:	src
> Committed By:	pooka
> Date:		Sat Oct 21 17:01:56 UTC 2006
> 
> Modified Files:
> 	src/sys/kern: kern_proc.c
> 
> Log Message:
> pool_get(): use PR_WAITOK, not M_WAITOK (which happens to be PR_NOWAIT ..)
> 
> XXX: not the first time this mistake has been made

find /usr/src/sys -name *.[ch"]" -exec grep -l pool_.*M_ "{"} ";"
./dev/raidframe/rf_reconmap.c
./kern/kern_kcont.c
./kern/kern_proc.c

There are probably more, in cases where a function argument is passed to
pool_get() as the flag.

I think we should use different values for the constants and, if DIAGNOSTIC,
assert that the constant is in fact in the correct range.

Thor