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

[Kenneth D. Merry]

On Wed, Mar 13, 2013 at 22:09:51 +0100, Pawel Jakub Dawidek wrote:
> 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.

This particular sysctl/tunable is not new.  It was in 9.1.

My concern with changing it is that it will lead to surprises for users who
already have it set.

Ken
-- 
Kenneth Merry
ken at FreeBSD.ORG