Date: Sat, 16 Mar 2013 09:54:01 -0700 From: Rui Paulo <rpaulo@FreeBSD.org> To: John Baldwin <jhb@freebsd.org> Cc: svn-src-head@freebsd.org, "Kenneth D. Merry" <ken@freebsd.org>, svn-src-all@freebsd.org, src-committers@freebsd.org, Pawel Jakub Dawidek <pjd@freebsd.org> Subject: Re: svn commit: r247814 - in head: . sys/amd64/conf sys/cam/ctl sys/conf sys/i386/conf Message-ID: <6B59A612-B737-4E1F-AF56-0DA1D0733F0A@FreeBSD.org> In-Reply-To: <201303131108.27005.jhb@freebsd.org> References: <201303042118.r24LIj5e008913@svn.freebsd.org> <20130312210921.GB1390@garage.freebsd.pl> <201303131108.27005.jhb@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 2013/03/13, at 8:08, John Baldwin <jhb@freebsd.org> 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 :) I think this is not correct. The `disabled' hint comes from resource_disabled(), which checks for "resource_int_value(name, unit, "disabled", &value);" Regards, -- Rui Paulo
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?6B59A612-B737-4E1F-AF56-0DA1D0733F0A>