Re: [PATCH] ftrace: Access ret_stack->subtime only in the function profiler

2016-08-30 Thread Namhyung Kim
On Tue, Aug 30, 2016 at 10:16:47PM -0400, Steven Rostedt wrote: > On Wed, 31 Aug 2016 11:11:38 +0900 > Namhyung Kim wrote: > > > On Tue, Aug 30, 2016 at 10:34:41AM +0900, Namhyung Kim wrote: > > > Hi Steve, > > > > > > On Mon, Aug 29, 2016 at 04:07:00PM -0400, Steven

Re: [PATCH] ftrace: Access ret_stack->subtime only in the function profiler

2016-08-30 Thread Namhyung Kim
On Tue, Aug 30, 2016 at 10:16:47PM -0400, Steven Rostedt wrote: > On Wed, 31 Aug 2016 11:11:38 +0900 > Namhyung Kim wrote: > > > On Tue, Aug 30, 2016 at 10:34:41AM +0900, Namhyung Kim wrote: > > > Hi Steve, > > > > > > On Mon, Aug 29, 2016 at 04:07:00PM -0400, Steven Rostedt wrote: > > > > On

Re: [PATCH] ftrace: Access ret_stack->subtime only in the function profiler

2016-08-30 Thread Steven Rostedt
On Wed, 31 Aug 2016 11:11:38 +0900 Namhyung Kim wrote: > On Tue, Aug 30, 2016 at 10:34:41AM +0900, Namhyung Kim wrote: > > Hi Steve, > > > > On Mon, Aug 29, 2016 at 04:07:00PM -0400, Steven Rostedt wrote: > > > On Mon, 29 Aug 2016 12:05:18 +0900 > > > Namhyung Kim

Re: [PATCH] ftrace: Access ret_stack->subtime only in the function profiler

2016-08-30 Thread Steven Rostedt
On Wed, 31 Aug 2016 11:11:38 +0900 Namhyung Kim wrote: > On Tue, Aug 30, 2016 at 10:34:41AM +0900, Namhyung Kim wrote: > > Hi Steve, > > > > On Mon, Aug 29, 2016 at 04:07:00PM -0400, Steven Rostedt wrote: > > > On Mon, 29 Aug 2016 12:05:18 +0900 > > > Namhyung Kim wrote: > > > > > > > The

Re: [PATCH] ftrace: Access ret_stack->subtime only in the function profiler

2016-08-30 Thread Namhyung Kim
On Tue, Aug 30, 2016 at 10:34:41AM +0900, Namhyung Kim wrote: > Hi Steve, > > On Mon, Aug 29, 2016 at 04:07:00PM -0400, Steven Rostedt wrote: > > On Mon, 29 Aug 2016 12:05:18 +0900 > > Namhyung Kim wrote: > > > > > The subtime is used only for function profiler with

Re: [PATCH] ftrace: Access ret_stack->subtime only in the function profiler

2016-08-30 Thread Namhyung Kim
On Tue, Aug 30, 2016 at 10:34:41AM +0900, Namhyung Kim wrote: > Hi Steve, > > On Mon, Aug 29, 2016 at 04:07:00PM -0400, Steven Rostedt wrote: > > On Mon, 29 Aug 2016 12:05:18 +0900 > > Namhyung Kim wrote: > > > > > The subtime is used only for function profiler with function graph > > > tracer

Re: [PATCH] ftrace: Access ret_stack->subtime only in the function profiler

2016-08-29 Thread Namhyung Kim
Hi Steve, On Mon, Aug 29, 2016 at 04:07:00PM -0400, Steven Rostedt wrote: > On Mon, 29 Aug 2016 12:05:18 +0900 > Namhyung Kim wrote: > > > The subtime is used only for function profiler with function graph > > tracer enabled. Move the definition of subtime under > >

Re: [PATCH] ftrace: Access ret_stack->subtime only in the function profiler

2016-08-29 Thread Namhyung Kim
Hi Steve, On Mon, Aug 29, 2016 at 04:07:00PM -0400, Steven Rostedt wrote: > On Mon, 29 Aug 2016 12:05:18 +0900 > Namhyung Kim wrote: > > > The subtime is used only for function profiler with function graph > > tracer enabled. Move the definition of subtime under > > CONFIG_FUNCTION_PROFILER to

Re: [PATCH] ftrace: Access ret_stack->subtime only in the function profiler

2016-08-29 Thread Steven Rostedt
On Mon, 29 Aug 2016 12:05:18 +0900 Namhyung Kim wrote: > The subtime is used only for function profiler with function graph > tracer enabled. Move the definition of subtime under > CONFIG_FUNCTION_PROFILER to reduce the memory usage. Also move the > initialization of

Re: [PATCH] ftrace: Access ret_stack->subtime only in the function profiler

2016-08-29 Thread Steven Rostedt
On Mon, 29 Aug 2016 12:05:18 +0900 Namhyung Kim wrote: > The subtime is used only for function profiler with function graph > tracer enabled. Move the definition of subtime under > CONFIG_FUNCTION_PROFILER to reduce the memory usage. Also move the > initialization of subtime into the graph

[PATCH] ftrace: Access ret_stack->subtime only in the function profiler

2016-08-28 Thread Namhyung Kim
The subtime is used only for function profiler with function graph tracer enabled. Move the definition of subtime under CONFIG_FUNCTION_PROFILER to reduce the memory usage. Also move the initialization of subtime into the graph entry callback. Cc: Josh Poimboeuf

[PATCH] ftrace: Access ret_stack->subtime only in the function profiler

2016-08-28 Thread Namhyung Kim
The subtime is used only for function profiler with function graph tracer enabled. Move the definition of subtime under CONFIG_FUNCTION_PROFILER to reduce the memory usage. Also move the initialization of subtime into the graph entry callback. Cc: Josh Poimboeuf Signed-off-by: Namhyung Kim