From owner-p4-projects@FreeBSD.ORG Wed Jan 2 00:01:13 2008 Return-Path: Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id 820E616A41B; Wed, 2 Jan 2008 00:01:13 +0000 (UTC) Delivered-To: perforce@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id B222816A419 for ; Wed, 2 Jan 2008 00:01:12 +0000 (UTC) (envelope-from jb@freebsd.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id A060C13C448 for ; Wed, 2 Jan 2008 00:01:12 +0000 (UTC) (envelope-from jb@freebsd.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.1/8.14.1) with ESMTP id m0201Cv7087486 for ; Wed, 2 Jan 2008 00:01:12 GMT (envelope-from jb@freebsd.org) Received: (from perforce@localhost) by repoman.freebsd.org (8.14.1/8.14.1/Submit) id m0201CCp087483 for perforce@freebsd.org; Wed, 2 Jan 2008 00:01:12 GMT (envelope-from jb@freebsd.org) Date: Wed, 2 Jan 2008 00:01:12 GMT Message-Id: <200801020001.m0201CCp087483@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: perforce set sender to jb@freebsd.org using -f From: John Birrell To: Perforce Change Reviews Cc: Subject: PERFORCE change 132274 for review X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 02 Jan 2008 00:01:13 -0000 http://perforce.freebsd.org/chv.cgi?CH=132274 Change 132274 by jb@jb_freebsd1 on 2008/01/02 00:00:35 When booting with DTrace enabled, just use the 'dtraceall' module. It's dependencies will cause the set of DTrace modules to be loaded. This means that the boot loader doesn't have to change as new providers are added. Affected files ... .. //depot/projects/dtrace/src/sys/boot/common/boot.c#7 edit Differences ... ==== //depot/projects/dtrace/src/sys/boot/common/boot.c#7 (text+ko) ==== @@ -107,27 +107,12 @@ /* Check if DTrace is to be loaded prior to boot. */ if (getenv("dtrace_boot")) { /* - * Load the DTrace modules. This would be better done - * in 4th so that extra providers can be added easily. + * Load the DTrace modules using the module that + * depends on all other modules -- dtraceall. */ - error = mod_load("cyclic", NULL, 0, NULL); + error = mod_load("dtraceall", NULL, 0, NULL); if (error != 0) - printf("cyclic autoload failed - %s\n", strerror(error)); - error = mod_load("dtrace", NULL, 0, NULL); - if (error != 0) - printf("dtrace autoload failed - %s\n", strerror(error)); - error = mod_load("profile", NULL, 0, NULL); - if (error != 0) - printf("profile autoload failed - %s\n", strerror(error)); - error = mod_load("systrace", NULL, 0, NULL); - if (error != 0) - printf("systrace autoload failed - %s\n", strerror(error)); - error = mod_load("fbt", NULL, 0, NULL); - if (error != 0) - printf("fbt autoload failed - %s\n", strerror(error)); - error = mod_load("sdt", NULL, 0, NULL); - if (error != 0) - printf("sdt autoload failed - %s\n", strerror(error)); + printf("dtraceall autoload failed - %s\n", strerror(error)); /* * Open the DTrace DOF file if it exists. Don't worry if