diff options
author | Steven Rostedt (Red Hat) <rostedt@goodmis.org> | 2016-04-29 18:10:21 -0400 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2016-04-29 18:10:21 -0400 |
commit | 9b9db275051cd9191e7776c4fd79ccd4318aa2dc (patch) | |
tree | 4ba7101de6e29b9e12c8bd1fd80661303cb14699 | |
parent | 33fddff24d05d71f97722cb7deec4964d39d10dc (diff) |
tracing: Remove one use of trace_current_buffer_lock_reserve()
The only user of trace_current_buffer_lock_reserve() is in the boot up self
tests. Restructure the code a little to have that code use what everything
else uses: trace_event_buffer_lock_reserve().
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
-rw-r--r-- | kernel/trace/trace_events.c | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c index e7cb983ee93c..da1eeb6190e3 100644 --- a/kernel/trace/trace_events.c +++ b/kernel/trace/trace_events.c | |||
@@ -3392,7 +3392,7 @@ static __init void event_trace_self_tests(void) | |||
3392 | 3392 | ||
3393 | static DEFINE_PER_CPU(atomic_t, ftrace_test_event_disable); | 3393 | static DEFINE_PER_CPU(atomic_t, ftrace_test_event_disable); |
3394 | 3394 | ||
3395 | static struct trace_array *event_tr; | 3395 | static struct trace_event_file event_trace_file __initdata; |
3396 | 3396 | ||
3397 | static void __init | 3397 | static void __init |
3398 | function_test_events_call(unsigned long ip, unsigned long parent_ip, | 3398 | function_test_events_call(unsigned long ip, unsigned long parent_ip, |
@@ -3416,17 +3416,17 @@ function_test_events_call(unsigned long ip, unsigned long parent_ip, | |||
3416 | 3416 | ||
3417 | local_save_flags(flags); | 3417 | local_save_flags(flags); |
3418 | 3418 | ||
3419 | event = trace_current_buffer_lock_reserve(&buffer, | 3419 | event = trace_event_buffer_lock_reserve(&buffer, &event_trace_file, |
3420 | TRACE_FN, sizeof(*entry), | 3420 | TRACE_FN, sizeof(*entry), |
3421 | flags, pc); | 3421 | flags, pc); |
3422 | if (!event) | 3422 | if (!event) |
3423 | goto out; | 3423 | goto out; |
3424 | entry = ring_buffer_event_data(event); | 3424 | entry = ring_buffer_event_data(event); |
3425 | entry->ip = ip; | 3425 | entry->ip = ip; |
3426 | entry->parent_ip = parent_ip; | 3426 | entry->parent_ip = parent_ip; |
3427 | 3427 | ||
3428 | trace_buffer_unlock_commit(event_tr, buffer, event, flags, pc); | 3428 | event_trigger_unlock_commit(&event_trace_file, buffer, event, |
3429 | 3429 | entry, flags, pc); | |
3430 | out: | 3430 | out: |
3431 | atomic_dec(&per_cpu(ftrace_test_event_disable, cpu)); | 3431 | atomic_dec(&per_cpu(ftrace_test_event_disable, cpu)); |
3432 | preempt_enable_notrace(); | 3432 | preempt_enable_notrace(); |
@@ -3441,9 +3441,11 @@ static struct ftrace_ops trace_ops __initdata = | |||
3441 | static __init void event_trace_self_test_with_function(void) | 3441 | static __init void event_trace_self_test_with_function(void) |
3442 | { | 3442 | { |
3443 | int ret; | 3443 | int ret; |
3444 | event_tr = top_trace_array(); | 3444 | |
3445 | if (WARN_ON(!event_tr)) | 3445 | event_trace_file.tr = top_trace_array(); |
3446 | if (WARN_ON(!event_trace_file.tr)) | ||
3446 | return; | 3447 | return; |
3448 | |||
3447 | ret = register_ftrace_function(&trace_ops); | 3449 | ret = register_ftrace_function(&trace_ops); |
3448 | if (WARN_ON(ret < 0)) { | 3450 | if (WARN_ON(ret < 0)) { |
3449 | pr_info("Failed to enable function tracer for event tests\n"); | 3451 | pr_info("Failed to enable function tracer for event tests\n"); |