Re: [PATCH 1/6] ftrace: move function tracer functions out oftrace.c

From: Andrew Morton
Date: Fri Jan 16 2009 - 01:50:59 EST


On Thu, 15 Jan 2009 23:53:43 -0500 Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

> static void
> +function_trace_call_preempt_only(unsigned long ip, unsigned long parent_ip)
> +{
> + struct trace_array *tr = func_trace;
> + struct trace_array_cpu *data;
> + unsigned long flags;
> + long disabled;
> + int cpu, resched;
> + int pc;
> +
> + if (unlikely(!ftrace_function_enabled))
> + return;

We're optimizing for the tracing-is-enabled case. What's the thinking
here?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/