diff options
| author | Steven Rostedt <srostedt@redhat.com> | 2009-03-03 14:43:50 (GMT) |
|---|---|---|
| committer | Steven Rostedt <srostedt@redhat.com> | 2009-03-03 14:43:50 (GMT) |
| commit | 633ddaa7f471e9db181f993c1458d6f4bae321ca (patch) | |
| tree | 2bf2e419c004cdbce77b1ba3b4cbd5c28cf45bd7 /kernel | |
| parent | c79a61f55773d2519fd0525bf58385f7d20752d3 (diff) | |
| download | linux-fsl-qoriq-633ddaa7f471e9db181f993c1458d6f4bae321ca.tar.xz | |
tracing: fix return value to registering events
The registering of events had the return value check backwards.
A zero returned is success, the check had it as a failure.
This patch also fixes a missing "\n" in the warning that the check
failed.
Reported-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Steven Rostedt <srostedt@redhat.com>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/trace/trace_events_stage_3.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/kernel/trace/trace_events_stage_3.h b/kernel/trace/trace_events_stage_3.h index c62a4d2..041789f 100644 --- a/kernel/trace/trace_events_stage_3.h +++ b/kernel/trace/trace_events_stage_3.h @@ -120,9 +120,9 @@ static int ftrace_reg_event_##call(void) \ int ret; \ \ ret = register_trace_##call(ftrace_event_##call); \ - if (!ret) \ + if (ret) \ pr_info("event trace: Could not activate trace point " \ - "probe to " #call); \ + "probe to " #call "\n"); \ return ret; \ } \ \ @@ -195,9 +195,9 @@ static int ftrace_raw_reg_event_##call(void) \ int ret; \ \ ret = register_trace_##call(ftrace_raw_event_##call); \ - if (!ret) \ + if (ret) \ pr_info("event trace: Could not activate trace point " \ - "probe to " #call); \ + "probe to " #call "\n"); \ return ret; \ } \ \ |