Re: CVS commit: src/sys/kern

[christos%zoulas.com@localhost (Christos Zoulas)]

On Jul 27,  9:46pm, martin%duskware.de@localhost (Martin Husemann) wrote:
-- Subject: Re: CVS commit: src/sys/kern

| On Sun, Jul 27, 2014 at 07:37:14PM +0000, Christos Zoulas wrote:
| > to wrap all the KMEM_ related diagnostic options in one, can separate
| > them from DIAGNOSTIC, so that low memory machines can choose not to turn
| > them on, but still have DIAGNOSTIC turned on.
| 
| I am not convinced this will be used ever. All "low memory" machines I
| can think of (at least all the ones I run) do not use DIAGNOSTIC in normal
| operation, and when debugging it hardly matters.
| 
| How about enabling the kmem stuff by default with DIAGNOSTIC, but add a
| negative option to turn it off (KMEM_NO_EXTRA_CHECKS or something)?

That's fine too. The all-or-nothing approach is the issue here.

christos