From owner-cvs-src@FreeBSD.ORG Sat Nov 4 09:12:04 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 09E6416A4E0; Sat, 4 Nov 2006 09:12:04 +0000 (UTC) (envelope-from simon@zaphod.nitro.dk) Received: from mx.nitro.dk (zarniwoop.nitro.dk [83.92.207.38]) by mx1.FreeBSD.org (Postfix) with ESMTP id 4AFE343D45; Sat, 4 Nov 2006 09:11:57 +0000 (GMT) (envelope-from simon@zaphod.nitro.dk) Received: from zaphod.nitro.dk (unknown [192.168.3.39]) by mx.nitro.dk (Postfix) with ESMTP id 7323D386C08; Sat, 4 Nov 2006 09:11:28 +0000 (UTC) Received: by zaphod.nitro.dk (Postfix, from userid 3000) id C69ED11434; Sat, 4 Nov 2006 10:11:56 +0100 (CET) Date: Sat, 4 Nov 2006 10:11:56 +0100 From: "Simon L. Nielsen" To: John Birrell Message-ID: <20061104091155.GA12770@zaphod.nitro.dk> References: <200611040458.kA44wAKN097086@repoman.freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200611040458.kA44wAKN097086@repoman.freebsd.org> User-Agent: Mutt/1.5.11 Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@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 09:12:04 -0000 On 2006.11.04 04:58:10 +0000, John Birrell wrote: > jb 2006-11-04 04:58:10 UTC > > FreeBSD src repository > > Modified files: > sys/amd64/conf DEFAULTS > sys/i386/conf DEFAULTS > sys/ia64/conf DEFAULTS > sys/pc98/conf DEFAULTS > sys/powerpc/conf DEFAULTS > sys/sparc64/conf DEFAULTS > sys/sparc64/sparc64 machdep.c > sys/sun4v/conf DEFAULTS > Log: > Build in kernel support for loading DTrace modules by default. This > adds the hooks that DTrace modules register with, and adds a few functions > which have the dtrace_ prefix to allow the DTrace FBT (function boundary > trace) provider to avoid tracing because they are called from the DTtrace > probe context. 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. On a side note, I think it's really great with the DTrace support and I look forward to trying it out. Thanks for working on it! -- Simon L. Nielsen