Re: [PATCH tip 1/1] tracing: Use a single trace_enabled variable

[Ingo Molnar <mingo@elte.hu>]

* Arnaldo Carvalho de Melo <acme@redhat.com> wrote:

> Impact: API change/cleanup
> 
> $ codiff kernel/trace/built-in.o.before kernel/trace/built-in.o
> kernel/trace/trace.c:
>   tracing_is_enabled |  -12
>   __find_next_entry  |   -5
>  2 functions changed, 17 bytes removed, diff: -17
> 
> kernel/trace/trace_sched_switch.c:
>   tracing_sched_switch_assign_trace |  -13
>   tracing_stop_sched_switch_record  |  -74
>   tracing_start_sched_switch_record |  -30
>  3 functions changed, 117 bytes removed, diff: -117
> 
> kernel/trace/trace_sysprof.c:
>   stack_trace_reset |  -10
>   stack_trace_init  |  -10
>  2 functions changed, 20 bytes removed, diff: -20
> 
> kernel/trace/trace_irqsoff.c:
>   irqsoff_tracer_start |  -16
>   irqsoff_tracer_stop  |  -16
>   irqsoff_tracer_reset |  -10
>   irqsoff_tracer_init  |  -19
>  4 functions changed, 61 bytes removed, diff: -61
> 
> kernel/trace/trace_sched_wakeup.c:
>   wakeup_tracer_stop  |  -16
>   wakeup_tracer_start |  -10
>   start_wakeup_tracer |  -19
>   wakeup_tracer_reset |  -10
>  4 functions changed, 55 bytes removed, diff: -55
> 
> kernel/trace/built-in.o:
>  15 functions changed, 270 bytes removed, diff: -270
> 
> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> ---
>  kernel/trace/trace.c              |   15 +-------------
>  kernel/trace/trace.h              |    3 +-
>  kernel/trace/trace_irqsoff.c      |   22 --------------------
>  kernel/trace/trace_sched_switch.c |   40 +------------------------------------
>  kernel/trace/trace_sched_wakeup.c |   26 ------------------------
>  kernel/trace/trace_sysprof.c      |    5 +---
>  6 files changed, 5 insertions(+), 106 deletions(-)

Applied to tip:tracing/ftrace, thanks Arnaldo!

	Ingo