svn commit: r244112 - head/sys/kern

[Andriy Gapon]

on 16/12/2012 10:06 Alfred Perlstein said the following:
> On 12/15/12 11:45 PM, Andriy Gapon wrote:
>> on 16/12/2012 07:00 Ian Lepore said the following:
>>> Some developer already made that choice by coding a
>>> KASSERT() instead of a panic().

>> Please don't perpetuate this argument.  The point of KASSERT is not that the
>> developer intended that the system should try to keep running in production.
>> The point is that (1) the KASSERT should not be hit in production as was
>> established in testing *and* (2) having all KASSERTs enabled in production is
>> too expensive.  That's all.
>>
> I don't understand, we have a few partners running KASSERT enabled kernels.
> 
> Depending on workload our machines can have enough CPU free for this.

Yes, sure.  I have stated the reason why KASSERT is controlled by INVARIANTS.
I haven't said that it must turned off in production.

-- 
Andriy Gapon