2015-11-30 09:26:25

by Li Bin

Subject: [PATCH] ftrace: fix a typo in comment

s/ARCH_SUPPORT_FTARCE_OPS/ARCH_SUPPORTS_FTARCE_OPS

Signed-off-by: Li Bin <[email protected]>
---
kernel/trace/ftrace.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c
index 3f743b1..eb4a881 100644
--- a/kernel/trace/ftrace.c
+++ b/kernel/trace/ftrace.c
@@ -5195,7 +5195,7 @@ out:
* being NULL, or CONFIG_DYNAMIC_FTRACE_WITH_REGS.
* Note, CONFIG_DYNAMIC_FTRACE_WITH_REGS expects a full regs to be saved.
* An architecture can pass partial regs with ftrace_ops and still
- * set the ARCH_SUPPORT_FTARCE_OPS.
+ * set the ARCH_SUPPORTS_FTARCE_OPS.
*/
#if ARCH_SUPPORTS_FTRACE_OPS
static void ftrace_ops_list_func(unsigned long ip, unsigned long parent_ip,
--
1.7.1

2015-11-30 09:54:04

by Daniel K.

Subject: Re: [PATCH] ftrace: fix a typo in comment

On 11/30/2015 09:24 AM, Li Bin wrote:
> s/ARCH_SUPPORT_FTARCE_OPS/ARCH_SUPPORTS_FTARCE_OPS

How about s/FTARCE/FTRACE/ as well?


Daniel K.

2015-11-30 10:20:34

by Li Bin

Subject: Re: [PATCH] ftrace: fix a typo in comment

Sorry for my mistake. And please ignore this patch.

Thanks,
Li Bin

on 2015/11/30 17:53, Daniel K. wrote:
> On 11/30/2015 09:24 AM, Li Bin wrote:
>> s/ARCH_SUPPORT_FTARCE_OPS/ARCH_SUPPORTS_FTARCE_OPS
> How about s/FTARCE/FTRACE/ as well?
>
>
> Daniel K.
>
>
>

2015-11-30 15:04:28

by Steven Rostedt

Subject: Re: [PATCH] ftrace: fix a typo in comment

On Mon, 30 Nov 2015 17:24:28 +0800
Li Bin <[email protected]> wrote:

> s/ARCH_SUPPORT_FTARCE_OPS/ARCH_SUPPORTS_FTARCE_OPS
>
> Signed-off-by: Li Bin <[email protected]>
> ---
> kernel/trace/ftrace.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c
> index 3f743b1..eb4a881 100644
> --- a/kernel/trace/ftrace.c
> +++ b/kernel/trace/ftrace.c
> @@ -5195,7 +5195,7 @@ out:
> * being NULL, or CONFIG_DYNAMIC_FTRACE_WITH_REGS.
> * Note, CONFIG_DYNAMIC_FTRACE_WITH_REGS expects a full regs to be saved.
> * An architecture can pass partial regs with ftrace_ops and still
> - * set the ARCH_SUPPORT_FTARCE_OPS.
> + * set the ARCH_SUPPORTS_FTARCE_OPS.

Nice, except can you change the typo of s/FTARCE/FTRACE/ too ;-)

Although I have to admit that FTRACE is sometimes F'ng Tarce!

-- Steve

> */
> #if ARCH_SUPPORTS_FTRACE_OPS
> static void ftrace_ops_list_func(unsigned long ip, unsigned long parent_ip,