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

[Jean-Jacques Hiblot]

On 11/07/2014 05:52 PM, Steven Rostedt wrote:
> On Fri,  7 Nov 2014 15:59:11 +0100
> Jean-Jacques Hiblot <jjhiblot@xxxxxxxxxxxxxxx> 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
> 
> -- 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@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/