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

Ingo Molnar
2009-02-11T12:06:50Z

* 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