Re: [PATCHv2] arm: ftrace: Adds support for CONFIG_DYNAMIC_FTRACE_WITH_REGS

2017-02-10 Thread Jean-Jacques Hiblot
2017-02-07 20:18 GMT+01:00 Abel Vesa : > On Wed, Jan 25, 2017 at 07:30:32PM +0100, Jean-Jacques Hiblot wrote: > >> Abel, I tested it partially. > > OK, so I managed to find exactly what is happening. > >> It's seems OK if I have FRAME_POINTER enabled and it breaks otherwise. > > FRAME_POINTER is ne

Re: [PATCHv2] arm: ftrace: Adds support for CONFIG_DYNAMIC_FTRACE_WITH_REGS

2017-02-07 Thread Abel Vesa
On Wed, Jan 25, 2017 at 07:30:32PM +0100, Jean-Jacques Hiblot wrote: > Abel, I tested it partially. OK, so I managed to find exactly what is happening. > It's seems OK if I have FRAME_POINTER enabled and it breaks otherwise. FRAME_POINTER is needed by OLD_MCOUNT. OLD_MCOUNT is needed by __ftr

Re: [PATCHv2] arm: ftrace: Adds support for CONFIG_DYNAMIC_FTRACE_WITH_REGS

2017-01-30 Thread Russell King - ARM Linux
On Tue, Jan 24, 2017 at 11:24:51PM +, Abel Vesa wrote: > The DYNAMIC_FTRACE_WITH_REGS configuration makes it possible for a ftrace > operation to specify if registers need to saved/restored by the ftrace > handler. > This is needed by kgraft and possibly other ftrace-based tools, and the ARM >

Re: [PATCHv2] arm: ftrace: Adds support for CONFIG_DYNAMIC_FTRACE_WITH_REGS

2017-01-25 Thread Jean-Jacques Hiblot
Abel, I tested it partially. It's seems OK if I have FRAME_POINTER enabled and it breaks otherwise. My test case attaches a tracing handler to a function "myfunc": static struct ftrace_ops kprobe_ftrace_ops __read_mostly = { .func = jjh_ftrace_handler, .flags = FTRACE_OPS_FL_SAVE_REGS | FTRACE_O

[PATCHv2] arm: ftrace: Adds support for CONFIG_DYNAMIC_FTRACE_WITH_REGS

2017-01-24 Thread Abel Vesa
The DYNAMIC_FTRACE_WITH_REGS configuration makes it possible for a ftrace operation to specify if registers need to saved/restored by the ftrace handler. This is needed by kgraft and possibly other ftrace-based tools, and the ARM architecture is currently lacking this feature. It would also be the