Date: Thu, 07 Jun 2012 16:40:26 +0300 From: Alexander Motin <mav@FreeBSD.org> To: Benjamin Kaduk <bjk@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r236712 - in head: share/man/man4 sys/cam sys/cam/scsi sys/conf Message-ID: <4FD0AF4A.3070703@FreeBSD.org> In-Reply-To: <alpine.BSF.2.00.1206071325220.53782@freefall.freebsd.org> References: <201206071005.q57A5pSX075803@svn.freebsd.org> <alpine.BSF.2.00.1206071325220.53782@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4FD0AF4A.3070703>