Re: [PATCH] ftrace: Build without frame pointers on Microblaze

[Michal Simek <monstr@monstr.eu>]

Hi Ingo, [cc: Steven and Frederic]

can you please add this patch to your tree?

Thanks,
Michal

Michal Simek wrote:
> Microblaze doesn't need/support FRAME_POINTERS in order to have a working
> function tracer.
> 
> The patch remove Kconfig warning.
> 
> Warning log:
> warning: (LOCKDEP && FAULT_INJECTION_STACKTRACE_FILTER && LATENCYTOP &&
> FUNCTION_TRACER && KMEMCHECK) selects FRAME_POINTER which has unmet direct
> dependencies (DEBUG_KERNEL && (CRIS || M68K || FRV || UML || AVR32 ||
> SUPERH || BLACKFIN || MN10300) || ARCH_WANT_FRAME_POINTERS)
> 
> Signed-off-by: Michal Simek <monstr@monstr.eu>
> CC: Steven Rostedt <rostedt@goodmis.org>
> CC: Frederic Weisbecker <fweisbec@gmail.com>
> CC: Ingo Molnar <mingo@redhat.com>
> ---
>  kernel/trace/Kconfig |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/kernel/trace/Kconfig b/kernel/trace/Kconfig
> index 61d7d59..2ad39e5 100644
> --- a/kernel/trace/Kconfig
> +++ b/kernel/trace/Kconfig
> @@ -141,7 +141,7 @@ if FTRACE
>  config FUNCTION_TRACER
>  	bool "Kernel Function Tracer"
>  	depends on HAVE_FUNCTION_TRACER
> -	select FRAME_POINTER if !ARM_UNWIND && !S390
> +	select FRAME_POINTER if !ARM_UNWIND && !S390 && !MICROBLAZE
>  	select KALLSYMS
>  	select GENERIC_TRACER
>  	select CONTEXT_SWITCH_TRACER


-- 
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian