Re: [PATCH] arm: ftrace: Adds support for ftrace operations

[Jean-Jacques Hiblot <jjhiblot@traphandler.com>]

2014-11-07 18:01 GMT+01:00 Jean-Jacques Hiblot <jjhiblot@traphandler.com>:
>
>
> On 11/07/2014 05:52 PM, Steven Rostedt wrote:
>> On Fri,  7 Nov 2014 15:59:11 +0100
>> Jean-Jacques Hiblot <jjhiblot@traphandler.com> wrote:
>>
>>> Hi all,
>>>
>>> This patch adds support for ftrace operations on the ARM architecture.
>>> This support is needed by kgraft and possibly other ftrace-based tools.
>>> My goal is to have kgraft working on ARM and aarch64.
>>
>> Can you add this to the change log of the patch. As that would be what
>> goes into git history. Because honestly, I had no idea what you meant
>> by "ftrace operations". If I don't understand that term, how can you
>> expect anyone else to?
> I'll try to come up with a clearer commit log.
>>
>>>
>>> I tested this patch with a toy implementation of kgraft for ARM and basic
>>> usage of function and function_graph tracers
>>> It doesn't show any problem with the ftrace selftests and the tests found
>>> today in branch ftrace/testing-mystuff of linux-trace repository
>>> (git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git)
>>
>> Those tests are no where near what I have. Does this code work on the
>> beagle bone white? If so, I can probably run my full suite on it.
> It should work on any arm processor, It's not platform or SOC dependent
>>
Hi Steve,
did you get a chance of running your tests suite on real hardware with
this patch? I'd be really interested in the results.

Jiri,
would you be interested in the patchs to run kgraft on ARM ? The
modifications required in kgraft core are minimal, it's mostly
architecture specific.

Jean-Jacques

>> -- Steve
>>
>>>
>>> Jean-Jacques Hiblot (1):
>>>   arm: ftrace: Adds support for ftrace operations
>>>
>>>  arch/arm/Kconfig               |  2 ++
>>>  arch/arm/include/asm/ftrace.h  |  3 ++
>>>  arch/arm/kernel/entry-common.S | 68 +++++++++++++++++++++++++++++++++++++++++-
>>>  arch/arm/kernel/ftrace.c       | 34 +++++++++++++++++++++
>>>  4 files changed, 106 insertions(+), 1 deletion(-)
>>>
>>
>>
>> _______________________________________________
>> linux-arm-kernel mailing list
>> linux-arm-kernel@lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>>