svn commit: r247814 - in head: . sys/amd64/conf sys/cam/ctl sys/conf sys/i386/conf

[Pawel Jakub Dawidek]

On Wed, Mar 13, 2013 at 11:08:26AM -0400, John Baldwin wrote:
> On Tuesday, March 12, 2013 5:09:21 pm Pawel Jakub Dawidek wrote:
> > On Mon, Mar 04, 2013 at 09:18:45PM +0000, Kenneth D. Merry wrote:
> > > Author: ken
> > > Date: Mon Mar  4 21:18:45 2013
> > > New Revision: 247814
> > > URL: http://svnweb.freebsd.org/changeset/base/247814
> > > 
> > > Log:
> > >   Re-enable CTL in GENERIC on i386 and amd64, but turn on the CTL disable
> > >   tunable by default.
> > >   
> > >   This will allow GENERIC configurations to boot on small memory boxes, but
> > >   not require end users who want to use CTL to recompile their kernel.  They
> > >   can simply set kern.cam.ctl.disable=0 in loader.conf.
> > 
> > Could you rename it to kern.cam.ctl.enable(d)? There was discussion at
> > some point about sysctl/tunable names and the consensus was, AFAIR, to
> > use positive(?) names as they are more obvious.
> 
> Except that all the hints we use for devices are hint.foo.X.disable=1 :)

Well we have many sysctls too, but would be nice not to introduce new ones.

-- 
Pawel Jakub Dawidek                       http://www.wheelsystems.com
FreeBSD committer                         http://www.FreeBSD.org
Am I Evil? Yes, I Am!                     http://tupytaj.pl
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 196 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-src-head/attachments/20130313/76ae7e28/attachment.sig>