Subject: Re: CVS commit: src/sys
To: Jeff Rizzo <riz@NetBSD.org>
From: Eric Haszlakiewicz <erh@netbsd.org>
List: source-changes
Date: 03/31/2006 00:01:59
On Thu, Mar 30, 2006 at 04:09:16PM -0800, Jeff Rizzo wrote:
> On Sun, Mar 26, 2006 at 08:19:52PM +0000, Eric Haszlakiewicz wrote:
> >=20
> > Module Name:	src
> > Committed By:	erh
> > Date:		Sun Mar 26 20:19:52 UTC 2006
> >=20
> > Modified Files:
> > 	src/sys/kern: kern_lock.c
> > 	src/sys/sys: lock.h sched.h
> >=20
> > Log Message:
> > Add simple_lock_assert_locked/simple_lock_assert_unlocked to provide ad=
ditional
> > useful information when panic'ing because the assertion fails.
> > Use these to define the SCHED_ASSERT_LOCKED/SCHED_ASSERT_UNLOCKED macro=
s.
> >=20
>=20
> This change needs to be reworked a bit - I can't compile a kernel with
> MULTIPROCESSOR and DEBUG options because _simple_lock_assert_locked()=20
> is only compiled under LOCKDEBUG.  If I add LOCKDEBUG, though, I
> discover that _kernel_lock_assert_locked() tries to call it
> with the wrong number of arguments...

	whoops, didn't try that particular combination of options.
It should be fixed now, for any combination of those.
(although it seems cpu.c is broken, so I wasn't actually able to link)

eric