Re: [PATCH] ftrace/x86: Fix missing ops arg on static function tracing

From: Steven Rostedt
Date: Mon Nov 16 2015 - 11:25:23 EST


On Mon, 16 Nov 2015 10:54:53 -0500
Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

> This isn't needed, nor will the parameter be used if added. A comment
> should probably be added here instead.
>
> /*
> * When DYNAMIC_FTRACE is not defined, ARCH_SUPPORTS_FTRACE_OPS is not
> * set (see include/asm/ftrace.h). Only the ip and parent ip are used

Probably should also add "and include/linux/ftrace.h" as that's where
the ARCH_SUPPORTS_FTRACE_OPS is described.

-- Steve

> * and the list function is called when function tracing is enabled.
> */
>
--
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/