diff options
author | kib <kib@FreeBSD.org> | 2008-11-30 13:18:35 +0000 |
---|---|---|
committer | kib <kib@FreeBSD.org> | 2008-11-30 13:18:35 +0000 |
commit | 0ad3ee03806e21a1a702bd108e68dff1c4f34afd (patch) | |
tree | 4dab11ccba8ad525d4ea2cc0a38817b16d6a63a4 /share/man/man9/ktr.9 | |
parent | e6c2c9e6552401e4b66d91c7e830d250d98f68f7 (diff) | |
download | FreeBSD-src-0ad3ee03806e21a1a702bd108e68dff1c4f34afd.zip FreeBSD-src-0ad3ee03806e21a1a702bd108e68dff1c4f34afd.tar.gz |
Clarify the reason to not use the volatile string as an argument to CTR
macros. Note the logging of file and line.
Reviewed by: dchagin
MFC after: 3 days
Diffstat (limited to 'share/man/man9/ktr.9')
-rw-r--r-- | share/man/man9/ktr.9 | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/share/man/man9/ktr.9 b/share/man/man9/ktr.9 index ca7d336..98b601d 100644 --- a/share/man/man9/ktr.9 +++ b/share/man/man9/ktr.9 @@ -24,7 +24,7 @@ .\" .\" $FreeBSD$ .\" -.Dd December 27, 2005 +.Dd November 30, 2008 .Dt KTR 9 .Os .Sh NAME @@ -84,9 +84,16 @@ Following the .Fa format string are zero to five arguments referenced by .Fa format . +Each event is logged with a file name and source line number of the +originating CTR call, and a timestamp in addition to the log message. +.Pp +The event is stored in the circular buffer with supplied arguments as is, +and formatting is done at the dump time. +Do not use pointers to the objects with limited lifetime, for instance, +strings, because the pointer may become invalid when buffer is printed. +.Pp Note that the different macros differ only in the number of arguments each one takes, as indicated by its name. -Each event is logged with a timestamp in addition to the log message. .Pp The .Va ktr_entries @@ -121,13 +128,11 @@ mi_switch() * Pick a new current process and record its start time. */ ... - CTR3(KTR_PROC, "mi_switch: old proc %p (pid %d, %s)", p, p->p_pid, - p->p_comm); + CTR3(KTR_PROC, "mi_switch: old proc %p (pid %d)", p, p->p_pid); ... cpu_switch(); ... - CTR3(KTR_PROC, "mi_switch: new proc %p (pid %d, %s)", p, p->p_pid, - p->p_comm); + CTR3(KTR_PROC, "mi_switch: new proc %p (pid %d)", p, p->p_pid); ... } .Ed |