Re: [PATCH v5 2/6] arm64: ftrace: fix a stack tracer's output under function graph tracer

2015-11-16 Thread AKASHI Takahiro
Jungseok, On 11/14/2015 12:01 AM, Jungseok Lee wrote: (+ Li Bin in CC) On Nov 10, 2015, at 11:42 AM, AKASHI Takahiro wrote: On 11/09/2015 11:04 PM, Jungseok Lee wrote: On Nov 6, 2015, at 3:44 PM, AKASHI Takahiro wrote: Hi Akashi, Function graph tracer modifies a return address (LR) in a

Re: [PATCH v5 2/6] arm64: ftrace: fix a stack tracer's output under function graph tracer

2015-11-16 Thread AKASHI Takahiro
Jungseok, On 11/14/2015 12:01 AM, Jungseok Lee wrote: (+ Li Bin in CC) On Nov 10, 2015, at 11:42 AM, AKASHI Takahiro wrote: On 11/09/2015 11:04 PM, Jungseok Lee wrote: On Nov 6, 2015, at 3:44 PM, AKASHI Takahiro wrote: Hi Akashi, Function graph tracer modifies a return address (LR) in a

Re: [PATCH v5 2/6] arm64: ftrace: fix a stack tracer's output under function graph tracer

2015-11-13 Thread Jungseok Lee
(+ Li Bin in CC) On Nov 10, 2015, at 11:42 AM, AKASHI Takahiro wrote: > On 11/09/2015 11:04 PM, Jungseok Lee wrote: >> On Nov 6, 2015, at 3:44 PM, AKASHI Takahiro wrote: >> >> Hi Akashi, >> >>> Function graph tracer modifies a return address (LR) in a stack frame >>> to hook a function return.

Re: [PATCH v5 2/6] arm64: ftrace: fix a stack tracer's output under function graph tracer

2015-11-13 Thread Jungseok Lee
(+ Li Bin in CC) On Nov 10, 2015, at 11:42 AM, AKASHI Takahiro wrote: > On 11/09/2015 11:04 PM, Jungseok Lee wrote: >> On Nov 6, 2015, at 3:44 PM, AKASHI Takahiro wrote: >> >> Hi Akashi, >> >>> Function graph tracer modifies a return address (LR) in a stack frame >>> to hook a function return.

Re: [PATCH v5 2/6] arm64: ftrace: fix a stack tracer's output under function graph tracer

2015-11-09 Thread AKASHI Takahiro
On 11/09/2015 11:04 PM, Jungseok Lee wrote: On Nov 6, 2015, at 3:44 PM, AKASHI Takahiro wrote: Hi Akashi, Function graph tracer modifies a return address (LR) in a stack frame to hook a function return. This will result in many useless entries (return_to_handler) showing up in a stack

Re: [PATCH v5 2/6] arm64: ftrace: fix a stack tracer's output under function graph tracer

2015-11-09 Thread Jungseok Lee
On Nov 6, 2015, at 3:44 PM, AKASHI Takahiro wrote: Hi Akashi, > Function graph tracer modifies a return address (LR) in a stack frame > to hook a function return. This will result in many useless entries > (return_to_handler) showing up in a stack tracer's output. > > This patch replaces such

Re: [PATCH v5 2/6] arm64: ftrace: fix a stack tracer's output under function graph tracer

2015-11-09 Thread Jungseok Lee
On Nov 6, 2015, at 3:44 PM, AKASHI Takahiro wrote: Hi Akashi, > Function graph tracer modifies a return address (LR) in a stack frame > to hook a function return. This will result in many useless entries > (return_to_handler) showing up in a stack tracer's output. > > This patch replaces such

Re: [PATCH v5 2/6] arm64: ftrace: fix a stack tracer's output under function graph tracer

2015-11-09 Thread AKASHI Takahiro
On 11/09/2015 11:04 PM, Jungseok Lee wrote: On Nov 6, 2015, at 3:44 PM, AKASHI Takahiro wrote: Hi Akashi, Function graph tracer modifies a return address (LR) in a stack frame to hook a function return. This will result in many useless entries (return_to_handler) showing up in a stack

[PATCH v5 2/6] arm64: ftrace: fix a stack tracer's output under function graph tracer

2015-11-05 Thread AKASHI Takahiro
Function graph tracer modifies a return address (LR) in a stack frame to hook a function return. This will result in many useless entries (return_to_handler) showing up in a stack tracer's output. This patch replaces such entries with originals values preserved in current->ret_stack[].

[PATCH v5 2/6] arm64: ftrace: fix a stack tracer's output under function graph tracer

2015-11-05 Thread AKASHI Takahiro
Function graph tracer modifies a return address (LR) in a stack frame to hook a function return. This will result in many useless entries (return_to_handler) showing up in a stack tracer's output. This patch replaces such entries with originals values preserved in current->ret_stack[].