* Re: [PATCH] tracing/ftrace: fix the check on nopped sites
[not found] <1237239660-6342-1-git-send-email-fweisbec@gmail.com>
@ 2009-03-16 22:09 ` Steven Rostedt
2009-03-18 3:37 ` Zhaolei
2009-03-17 2:26 ` Steven Rostedt
1 sibling, 1 reply; 3+ messages in thread
From: Steven Rostedt @ 2009-03-16 22:09 UTC (permalink / raw)
To: Frederic Weisbecker; +Cc: Ingo Molnar, linux-kernel, Zhaolei
On Mon, 16 Mar 2009, Frederic Weisbecker wrote:
> Impact: fix a dynamic tracing failure
>
> Recently, the function and function graph tracers failed to use dynamic
> tracing after the following commit:
>
> fa9d13cf135efbd454453a53b6299976bea245a9
> (ftrace: don't try to __ftrace_replace_code on !FTRACE_FL_CONVERTED rec)
>
> The patch is right except a mistake on the check for the FTRACE_FL_CONVERTED
> flag. The code patching is aborted in case of successfully nopped sites.
> What we want is the opposite: ignore the callsites that haven't been nopped.
>
> Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
> ---
> 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 90d5729..7847806 100644
> --- a/kernel/trace/ftrace.c
> +++ b/kernel/trace/ftrace.c
> @@ -537,7 +537,7 @@ static void ftrace_replace_code(int enable)
> */
> if (rec->flags & FTRACE_FL_FREE ||
> rec->flags & FTRACE_FL_FAILED ||
> - rec->flags & FTRACE_FL_CONVERTED)
> + !(rec->flags & FTRACE_FL_CONVERTED))
> continue;
>
> /* ignore updates to this record's mcount site */
> --
Heh, I just found this bug too, and wrote a patch. But you posted before I
did, so I'll let you have it ;-)
Acked-by: Steven Rostedt <srostedt@redhat.com>
-- Steve
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] tracing/ftrace: fix the check on nopped sites
[not found] <1237239660-6342-1-git-send-email-fweisbec@gmail.com>
2009-03-16 22:09 ` [PATCH] tracing/ftrace: fix the check on nopped sites Steven Rostedt
@ 2009-03-17 2:26 ` Steven Rostedt
1 sibling, 0 replies; 3+ messages in thread
From: Steven Rostedt @ 2009-03-17 2:26 UTC (permalink / raw)
To: Frederic Weisbecker; +Cc: Ingo Molnar, linux-kernel, Zhaolei
Ingo,
I pulled this into my repo, and will post it shortly.
-- Steve
On Mon, 16 Mar 2009, Frederic Weisbecker wrote:
> Impact: fix a dynamic tracing failure
>
> Recently, the function and function graph tracers failed to use dynamic
> tracing after the following commit:
>
> fa9d13cf135efbd454453a53b6299976bea245a9
> (ftrace: don't try to __ftrace_replace_code on !FTRACE_FL_CONVERTED rec)
>
> The patch is right except a mistake on the check for the FTRACE_FL_CONVERTED
> flag. The code patching is aborted in case of successfully nopped sites.
> What we want is the opposite: ignore the callsites that haven't been nopped.
>
> Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
> ---
> 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 90d5729..7847806 100644
> --- a/kernel/trace/ftrace.c
> +++ b/kernel/trace/ftrace.c
> @@ -537,7 +537,7 @@ static void ftrace_replace_code(int enable)
> */
> if (rec->flags & FTRACE_FL_FREE ||
> rec->flags & FTRACE_FL_FAILED ||
> - rec->flags & FTRACE_FL_CONVERTED)
> + !(rec->flags & FTRACE_FL_CONVERTED))
> continue;
>
> /* ignore updates to this record's mcount site */
> --
> 1.6.1
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] tracing/ftrace: fix the check on nopped sites
2009-03-16 22:09 ` [PATCH] tracing/ftrace: fix the check on nopped sites Steven Rostedt
@ 2009-03-18 3:37 ` Zhaolei
0 siblings, 0 replies; 3+ messages in thread
From: Zhaolei @ 2009-03-18 3:37 UTC (permalink / raw)
To: Steven Rostedt, Frederic Weisbecker; +Cc: Ingo Molnar, linux-kernel
* From: "Steven Rostedt" <rostedt@goodmis.org>
>
>
> On Mon, 16 Mar 2009, Frederic Weisbecker wrote:
>
>> Impact: fix a dynamic tracing failure
>>
>> Recently, the function and function graph tracers failed to use dynamic
>> tracing after the following commit:
>>
>> fa9d13cf135efbd454453a53b6299976bea245a9
>> (ftrace: don't try to __ftrace_replace_code on !FTRACE_FL_CONVERTED rec)
>>
>> The patch is right except a mistake on the check for the FTRACE_FL_CONVERTED
>> flag. The code patching is aborted in case of successfully nopped sites.
>> What we want is the opposite: ignore the callsites that haven't been nopped.
>>
>> Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
>> ---
>> 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 90d5729..7847806 100644
>> --- a/kernel/trace/ftrace.c
>> +++ b/kernel/trace/ftrace.c
>> @@ -537,7 +537,7 @@ static void ftrace_replace_code(int enable)
>> */
>> if (rec->flags & FTRACE_FL_FREE ||
>> rec->flags & FTRACE_FL_FAILED ||
>> - rec->flags & FTRACE_FL_CONVERTED)
>> + !(rec->flags & FTRACE_FL_CONVERTED))
>> continue;
>>
>> /* ignore updates to this record's mcount site */
>> --
Hello,
Sorry for my fault.
B.R.
Zhaolei
>
> Heh, I just found this bug too, and wrote a patch. But you posted before I
> did, so I'll let you have it ;-)
>
> Acked-by: Steven Rostedt <srostedt@redhat.com>
>
> -- Steve
>
>
>ÿôèº{.nÇ+·®+%Ëÿ±éݶ\x17¥wÿº{.nÇ+·¥{±þG«éÿ{ayº\x1dÊÚë,j\a¢f£¢·hïêÿêçz_è®\x03(éÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?¨èÚ&£ø§~á¶iOæ¬z·vØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?I¥
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2009-03-18 3:38 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1237239660-6342-1-git-send-email-fweisbec@gmail.com>
2009-03-16 22:09 ` [PATCH] tracing/ftrace: fix the check on nopped sites Steven Rostedt
2009-03-18 3:37 ` Zhaolei
2009-03-17 2:26 ` Steven Rostedt
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.