Re: [PATCH v7 04/10] ppc64 ftrace_with_regs configuration variables

2016-02-09 Thread Torsten Duwe
On Mon, Feb 08, 2016 at 10:49:28AM -0500, Steven Rostedt wrote: > On Mon, 8 Feb 2016 16:23:06 +0100 > Petr Mladek wrote: > > > >From 2b0fcb678d7720d03f9c9f233b61ed9ed4d420b3 Mon Sep 17 00:00:00 2001 > > From: Petr Mladek > > Date: Mon, 8 Feb 2016 16:03:03

Re: [PATCH v7 04/10] ppc64 ftrace_with_regs configuration variables

2016-02-08 Thread Petr Mladek
On Mon 2016-02-08 10:49:28, Steven Rostedt wrote: > On Mon, 8 Feb 2016 16:23:06 +0100 > Petr Mladek wrote: > > > >From 2b0fcb678d7720d03f9c9f233b61ed9ed4d420b3 Mon Sep 17 00:00:00 2001 > > From: Petr Mladek > > Date: Mon, 8 Feb 2016 16:03:03 +0100 > >

Re: [PATCH v7 04/10] ppc64 ftrace_with_regs configuration variables

2016-02-08 Thread Steven Rostedt
On Mon, 8 Feb 2016 16:23:06 +0100 Petr Mladek wrote: > >From 2b0fcb678d7720d03f9c9f233b61ed9ed4d420b3 Mon Sep 17 00:00:00 2001 > From: Petr Mladek > Date: Mon, 8 Feb 2016 16:03:03 +0100 > Subject: [PATCH] ftrace: Allow to explicitly disable the build of the

Re: [PATCH v7 04/10] ppc64 ftrace_with_regs configuration variables

2016-02-08 Thread Petr Mladek
On Sat 2016-02-06 11:32:43, Torsten Duwe wrote: > On Fri, Feb 05, 2016 at 05:18:34PM +0100, Petr Mladek wrote: > [...] > > more complicated. Whem I think about it, the change below does similar > > job and looks more strightforwad: > > Had I only looked closer. That's exactly how I thought it

Re: [PATCH v7 04/10] ppc64 ftrace_with_regs configuration variables

2016-02-08 Thread Torsten Duwe
On Mon, Feb 08, 2016 at 11:34:06AM +0100, Petr Mladek wrote: > On Sat 2016-02-06 11:32:43, Torsten Duwe wrote: > > On Fri, Feb 05, 2016 at 05:18:34PM +0100, Petr Mladek wrote: > > [...] > > > more complicated. Whem I think about it, the change below does similar > > > job and looks more

Re: [PATCH v7 04/10] ppc64 ftrace_with_regs configuration variables

2016-02-08 Thread Petr Mladek
On Mon 2016-02-08 13:12:45, Torsten Duwe wrote: > On Mon, Feb 08, 2016 at 11:34:06AM +0100, Petr Mladek wrote: > > On Sat 2016-02-06 11:32:43, Torsten Duwe wrote: > > > On Fri, Feb 05, 2016 at 05:18:34PM +0100, Petr Mladek wrote: > > > [...] > > > > more complicated. Whem I think about it, the

Re: [PATCH v7 04/10] ppc64 ftrace_with_regs configuration variables

2016-02-06 Thread Torsten Duwe
On Fri, Feb 05, 2016 at 05:18:34PM +0100, Petr Mladek wrote: [...] > more complicated. Whem I think about it, the change below does similar > job and looks more strightforwad: Had I only looked closer. That's exactly how I thought it would work in the first place. I'd call that a fix. Full ACK

Re: [PATCH v7 04/10] ppc64 ftrace_with_regs configuration variables

2016-02-05 Thread Petr Mladek
On Fri 2016-02-05 09:48:03, Steven Rostedt wrote: > On Fri, 5 Feb 2016 15:05:17 +0100 > Petr Mladek wrote: > > > On Mon 2016-01-25 16:29:54, Torsten Duwe wrote: > > > diff --git a/arch/powerpc/kernel/ftrace.c b/arch/powerpc/kernel/ftrace.c > > > index ef8b916..29b7014 100644 >

Re: [PATCH v7 04/10] ppc64 ftrace_with_regs configuration variables

2016-02-05 Thread Steven Rostedt
On Fri, 5 Feb 2016 17:18:34 +0100 Petr Mladek wrote: > diff --git a/kernel/trace/Kconfig b/kernel/trace/Kconfig > index a138f6d..de6dab0 100644 > --- a/kernel/trace/Kconfig > +++ b/kernel/trace/Kconfig > @@ -449,7 +449,7 @@ config PROBE_EVENTS > def_bool n > > config

Re: [PATCH v7 04/10] ppc64 ftrace_with_regs configuration variables

2016-02-05 Thread Petr Mladek
On Mon 2016-01-25 16:29:54, Torsten Duwe wrote: > diff --git a/arch/powerpc/kernel/ftrace.c b/arch/powerpc/kernel/ftrace.c > index ef8b916..29b7014 100644 > --- a/arch/powerpc/kernel/ftrace.c > +++ b/arch/powerpc/kernel/ftrace.c > @@ -28,6 +28,11 @@ > > > #ifdef CONFIG_DYNAMIC_FTRACE > +#if

Re: [PATCH v7 04/10] ppc64 ftrace_with_regs configuration variables

2016-02-05 Thread Steven Rostedt
On Fri, 5 Feb 2016 15:05:17 +0100 Petr Mladek wrote: > On Mon 2016-01-25 16:29:54, Torsten Duwe wrote: > > diff --git a/arch/powerpc/kernel/ftrace.c b/arch/powerpc/kernel/ftrace.c > > index ef8b916..29b7014 100644 > > --- a/arch/powerpc/kernel/ftrace.c > > +++

[PATCH v7 04/10] ppc64 ftrace_with_regs configuration variables

2016-02-04 Thread Torsten Duwe
* Makefile: - globally use -mprofile-kernel in case it's configured, available and bug-free. * arch/powerpc/gcc-mprofile-kernel-notrace.sh: - make sure -mprofile-kernel works and has none of the known bugs. * arch/powerpc/kernel/ftrace.c: - error out on compile with