From owner-cvs-src@FreeBSD.ORG Sat Nov 4 21:00:09 2006 Return-Path: X-Original-To: cvs-src@FreeBSD.org Delivered-To: cvs-src@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id C49C316A4A0; Sat, 4 Nov 2006 21:00:08 +0000 (UTC) (envelope-from sam@errno.com) Received: from ebb.errno.com (ebb.errno.com [69.12.149.25]) by mx1.FreeBSD.org (Postfix) with ESMTP id 401B343D8C; Sat, 4 Nov 2006 21:00:02 +0000 (GMT) (envelope-from sam@errno.com) Received: from [10.0.0.248] (trouble.errno.com [10.0.0.248]) (authenticated bits=0) by ebb.errno.com (8.13.6/8.12.6) with ESMTP id kA4L00x5070885 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Sat, 4 Nov 2006 13:00:01 -0800 (PST) (envelope-from sam@errno.com) Message-ID: <454CFF50.8010708@errno.com> Date: Sat, 04 Nov 2006 13:00:00 -0800 From: Sam Leffler User-Agent: Thunderbird 1.5.0.7 (X11/20060920) MIME-Version: 1.0 To: John Birrell References: <200611040458.kA44wAKN097086@repoman.freebsd.org> <20061104091155.GA12770@zaphod.nitro.dk> <20061104203150.GB2027@what-creek.com> In-Reply-To: <20061104203150.GB2027@what-creek.com> X-Enigmail-Version: 0.94.0.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: cvs-src@FreeBSD.org, John Birrell , cvs-all@FreeBSD.org, "Simon L. Nielsen" , src-committers@FreeBSD.org Subject: Re: cvs commit: src/sys/amd64/conf DEFAULTS src/sys/i386/conf DEFAULTS src/sys/ia64/conf DEFAULTS src/sys/pc98/conf DEFAULTS src/sys/powerpc/conf DEFAULTS src/sys/sparc64/conf DEFAULTS src/sys/sparc64/sparc64 machdep.c src/sys/sun4v/conf DEFAULTS X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 04 Nov 2006 21:00:09 -0000 John Birrell wrote: > On Sat, Nov 04, 2006 at 10:11:56AM +0100, Simon L. Nielsen wrote: >> Why isn't it just in GENERIC so people can easily disable it if they >> like? I thought DEFAULTS was just meant for things which caused users >> pain (and people answering support mails...) if they accidentally >> missed the option? I don't really see DTrace falling in that >> category. > > I added it to DEFAULTS so that it would be available automatically for > people who didn't know that they want it. 8-) > > Adding it to GENERIC doesn't cover the people who have their own > kernel configs. And they can still disable it in GENERIC or whatever > config they are using just as easily as they could add it. > > The point is that there is negligible run-time impact and only a > small increase in code size to have it enabled. Remember it's only > the hooks and a few support functions. The guts of DTrace is in the > 'dtrace' kernel module (and the associated providers) which still > have to be loaded as modules. Like he said, this is contrary to the purpose of DEFAULTS. You are forcing policy on people. Please put it in GENERIC where it belongs. Sam