Re: CVS commit: src/sys/arch

[Manuel Bouyer <bouyer%antioche.eu.org@localhost>]

On Sun, Jul 03, 2011 at 03:57:25PM +0100, Mindaugas Rasiukevicius wrote:
> Thanks for enabling this, however you have ignored points raised on
> tech-kern@.  In such case, at least DEBUG

DEBUG is not as usefull as DIAGNOSTIC, and is very verbose.
My main point is to get KASSERT() back again.

DEBUG and LOCKDEBUG were mentionned, but I don't remmeber a clear consensus
about thoses

> and DDB on panic (plus trace
> command) should be re-enabled as well.

I also don't think we did reach a consensus about this either.
With DDB on panic you don't get a core dump, you just see the system
hang if running X11.
With trace (assuming you're not running X11), you have the usefull panic
message scroll away on a standard VGA screen.

> 
> And, most importantly, this is not communicated to -current users who
> will get performance decrease after kernel update.  Apart from updating
> motd, I was thinking about some printf()ed warning in the kernel..

Was this ever evaluated ? I have DIAGNOSTIC in all my kernels,
and I never noticed a difference in performances for real usages.

-- 
Manuel Bouyer <bouyer%antioche.eu.org@localhost>
     NetBSD: 26 ans d'experience feront toujours la difference
--