diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2009-10-21 15:34:06 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-10-23 02:22:47 -0400 |
commit | 6beba7adbe092e63dfe8d09fbd1e3ec140474a13 (patch) | |
tree | d45a3347de224513c6d3d0e1061b78641f9aaf8a /tools/perf/builtin-timechart.c | |
parent | 802da5f2289bbe363acef084805195c11f453c48 (diff) |
perf tools: Unify debug messages mechanisms
We were using eprintf in some places, that looks at a global
'verbose' level, and at other places passing a 'v' parameter to
specify the verbosity level, unify it by introducing
pr_{err,warning,debug,etc}, just like in the kernel.
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Mike Galbraith <efault@gmx.de>
LKML-Reference: <1256153646-10097-1-git-send-email-acme@redhat.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'tools/perf/builtin-timechart.c')
-rw-r--r-- | tools/perf/builtin-timechart.c | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/tools/perf/builtin-timechart.c b/tools/perf/builtin-timechart.c index 34fad57087f9..0a2f22261c3a 100644 --- a/tools/perf/builtin-timechart.c +++ b/tools/perf/builtin-timechart.c | |||
@@ -1162,12 +1162,10 @@ more: | |||
1162 | size = event->header.size; | 1162 | size = event->header.size; |
1163 | 1163 | ||
1164 | if (!size || process_event(event) < 0) { | 1164 | if (!size || process_event(event) < 0) { |
1165 | 1165 | pr_warning("%p [%p]: skipping unknown header type: %d\n", | |
1166 | printf("%p [%p]: skipping unknown header type: %d\n", | 1166 | (void *)(offset + head), |
1167 | (void *)(offset + head), | 1167 | (void *)(long)(event->header.size), |
1168 | (void *)(long)(event->header.size), | 1168 | event->header.type); |
1169 | event->header.type); | ||
1170 | |||
1171 | /* | 1169 | /* |
1172 | * assume we lost track of the stream, check alignment, and | 1170 | * assume we lost track of the stream, check alignment, and |
1173 | * increment a single u64 in the hope to catch on again 'soon'. | 1171 | * increment a single u64 in the hope to catch on again 'soon'. |
@@ -1200,7 +1198,8 @@ done: | |||
1200 | 1198 | ||
1201 | write_svg_file(output_name); | 1199 | write_svg_file(output_name); |
1202 | 1200 | ||
1203 | printf("Written %2.1f seconds of trace to %s.\n", (last_time - first_time) / 1000000000.0, output_name); | 1201 | pr_info("Written %2.1f seconds of trace to %s.\n", |
1202 | (last_time - first_time) / 1000000000.0, output_name); | ||
1204 | 1203 | ||
1205 | return rc; | 1204 | return rc; |
1206 | } | 1205 | } |