Subject: Re: no listeners loaded [was: Re: CVS commit: src/sys/kern]
To: YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp>
From: Elad Efrat <elad@NetBSD.org>
List: tech-kern
Date: 09/14/2006 19:47:26
YAMAMOTO Takashi wrote:

> then, why not simply:
> 	if (!listeners_have_been_loaded) {
> 		KASSERT(SIMPLEQ_EMPTY(&scope->listenq));
> 		return 0;
> 	}
> 
> if you have a reason, (eg. to avoid a cache fill?)
> please add a comment.

Why do we want to KASSERT it?

IIRC, KASSERT will panic if DIAGNOSTIC, and by default DIAGNOSTIC
is not set... I'm not sure I like the dependency on DIAGNOSTIC for
this specific case.

-e.

-- 
Elad Efrat