svn commit: r236712 - in head: share/man/man4 sys/cam sys/cam/scsi sys/conf

[Alexander Motin]

On 06/07/12 16:26, Benjamin Kaduk wrote:
> On Thu, 7 Jun 2012, Alexander Motin wrote:
>
>> Author: mav
>> Date: Thu Jun 7 10:05:51 2012
>> New Revision: 236712
>> URL: http://svn.freebsd.org/changeset/base/236712
>>
>> Log:
>> To make CAM debugging easier, compile in some debug flags
>> (CAM_DEBUG_INFO,
>> CAM_DEBUG_CDB, CAM_DEBUG_PERIPH and CAM_DEBUG_PROBE) by default.
>> List of these flags can be modified with CAM_DEBUG_COMPILE kernel option.
>> CAMDEBUG kernel option still enables all possible debug, if not
>> overriden.
>>
>> Additional 50KB of kernel size is a good price for the ability to debug
>> problems without rebuilding the kernel. In case where size is important,
>> debugging can be compiled out by setting CAM_DEBUG_COMPILE option to 0.
>
> Hmm, it looks like re@ will want to add this to the list of things to
> remove from GENERIC for building release kernels. Is that true?

I don't think so. That only enables compilation of some debugging 
printf. It changes nothing in CAM behavior until user explicitly enable 
some of that through the loader tunables, sysctl or `camcontrol debug` 
command. It it like bootverbose, not like INVARIANTS. We are not 
blocking bootverbose on releases.

-- 
Alexander Motin