diff options
author | Frederic Weisbecker <fweisbec@gmail.com> | 2009-09-06 21:23:20 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-09-07 02:22:10 -0400 |
commit | d6a65dffb30d8636b1e5d4c201564ef401a246cf (patch) | |
tree | 34684202b58da377836d4acb8dcde5a33a075782 /kernel | |
parent | a1922ed661ab2c1637d0b10cde933bd9cd33d965 (diff) |
tracing: Fix ring-buffer and ksym tracer merge interaction
The compiler warns us about:
kernel/trace/trace_ksym.c: In function ksym_hbp_handler:
kernel/trace/trace_ksym.c:92: attention : passing argument 1 of trace_buffer_lock_reserve from incompatible pointer type
kernel/trace/trace_ksym.c:106: attention : passing argument 1 of trace_buffer_unlock_commit from incompatible pointer type
Commit "e77405ad" (tracing: pass around ring buffer
instead of tracer) has changed the central tracing
APIs. And this change has updated every callsites of
these APIs except those that aren't in tracing/core,
such as the ksym tracer.
Cc: Steven Rostedt <srostedt@redhat.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/trace/trace_ksym.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/kernel/trace/trace_ksym.c b/kernel/trace/trace_ksym.c index 2fde875ead4c..6d5609c67378 100644 --- a/kernel/trace/trace_ksym.c +++ b/kernel/trace/trace_ksym.c | |||
@@ -78,17 +78,18 @@ void ksym_collect_stats(unsigned long hbp_hit_addr) | |||
78 | void ksym_hbp_handler(struct hw_breakpoint *hbp, struct pt_regs *regs) | 78 | void ksym_hbp_handler(struct hw_breakpoint *hbp, struct pt_regs *regs) |
79 | { | 79 | { |
80 | struct ring_buffer_event *event; | 80 | struct ring_buffer_event *event; |
81 | struct trace_array *tr; | ||
82 | struct ksym_trace_entry *entry; | 81 | struct ksym_trace_entry *entry; |
82 | struct ring_buffer *buffer; | ||
83 | int pc; | 83 | int pc; |
84 | 84 | ||
85 | if (!ksym_tracing_enabled) | 85 | if (!ksym_tracing_enabled) |
86 | return; | 86 | return; |
87 | 87 | ||
88 | tr = ksym_trace_array; | 88 | buffer = ksym_trace_array->buffer; |
89 | |||
89 | pc = preempt_count(); | 90 | pc = preempt_count(); |
90 | 91 | ||
91 | event = trace_buffer_lock_reserve(tr, TRACE_KSYM, | 92 | event = trace_buffer_lock_reserve(buffer, TRACE_KSYM, |
92 | sizeof(*entry), 0, pc); | 93 | sizeof(*entry), 0, pc); |
93 | if (!event) | 94 | if (!event) |
94 | return; | 95 | return; |
@@ -103,7 +104,7 @@ void ksym_hbp_handler(struct hw_breakpoint *hbp, struct pt_regs *regs) | |||
103 | ksym_collect_stats(hbp->info.address); | 104 | ksym_collect_stats(hbp->info.address); |
104 | #endif /* CONFIG_PROFILE_KSYM_TRACER */ | 105 | #endif /* CONFIG_PROFILE_KSYM_TRACER */ |
105 | 106 | ||
106 | trace_buffer_unlock_commit(tr, event, 0, pc); | 107 | trace_buffer_unlock_commit(buffer, event, 0, pc); |
107 | } | 108 | } |
108 | 109 | ||
109 | /* Valid access types are represented as | 110 | /* Valid access types are represented as |