PERFORCE change 132686 for review

John Birrell jb at FreeBSD.org
Sun Jan 6 23:46:54 PST 2008


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

Change 132686 by jb at jb_freebsd1 on 2008/01/07 07:46:37

	Revise a couple of comments. I'm beginning to dislike the name
	'invop' used by Sun. It applies to i386, but on amd64 it's actually
	a breakpoint trap, not a fault.

Affected files ...

.. //depot/projects/dtrace/src/sys/cddl/dev/dtrace/dtrace_load.c#26 edit
.. //depot/projects/dtrace/src/sys/cddl/dev/dtrace/dtrace_unload.c#22 edit

Differences ...

==== //depot/projects/dtrace/src/sys/cddl/dev/dtrace/dtrace_load.c#26 (text+ko) ====

@@ -53,7 +53,7 @@
 	/* Hang our hook for thread switches. */
 	dtrace_vtime_switch_func = dtrace_vtime_switch;
 
-	/* Hang our hook for invalid opcodes. */
+	/* Hang our hook for exceptions. */
 	dtrace_invop_init();
 
 	/*

==== //depot/projects/dtrace/src/sys/cddl/dev/dtrace/dtrace_unload.c#22 (text+ko) ====

@@ -159,7 +159,7 @@
 	/* XXX Hack */
 	mutex_destroy(&mod_lock);
 
-	/* Reset out hook for invalid opcodes. */
+	/* Reset our hook for exceptions. */
 	dtrace_invop_uninit();
 
 	/*


More information about the p4-projects mailing list