PERFORCE change 132274 for review

John Birrell jb at FreeBSD.org
Tue Jan 1 16:01:13 PST 2008


http://perforce.freebsd.org/chv.cgi?CH=132274

Change 132274 by jb at 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


More information about the p4-projects mailing list