Re: [PATCH v2 01/32] tracing: Allow gcov profiling on only ftrace subsystem

2018-08-21 Thread Masami Hiramatsu
Hi Steve, On Mon, 20 Aug 2018 11:33:49 -0400 Steven Rostedt wrote: > On Mon, 20 Aug 2018 11:32:13 -0400 > Steven Rostedt wrote: > > > Other than that, it looks fine to me. > > This patch also seems out of place for the series. Yes, this one is the patch for ftrace, not ftracetest. > Want

Re: [PATCH v2 01/32] tracing: Allow gcov profiling on only ftrace subsystem

2018-08-21 Thread Masami Hiramatsu
Hi Steve, On Mon, 20 Aug 2018 11:33:49 -0400 Steven Rostedt wrote: > On Mon, 20 Aug 2018 11:32:13 -0400 > Steven Rostedt wrote: > > > Other than that, it looks fine to me. > > This patch also seems out of place for the series. Yes, this one is the patch for ftrace, not ftracetest. > Want

Re: [PATCH v2 01/32] tracing: Allow gcov profiling on only ftrace subsystem

2018-08-20 Thread Steven Rostedt
On Mon, 20 Aug 2018 11:32:13 -0400 Steven Rostedt wrote: > Other than that, it looks fine to me. This patch also seems out of place for the series. Want to resend it separately, and I'll apply it? Thanks! -- Steve

Re: [PATCH v2 01/32] tracing: Allow gcov profiling on only ftrace subsystem

2018-08-20 Thread Steven Rostedt
On Mon, 20 Aug 2018 11:32:13 -0400 Steven Rostedt wrote: > Other than that, it looks fine to me. This patch also seems out of place for the series. Want to resend it separately, and I'll apply it? Thanks! -- Steve

Re: [PATCH v2 01/32] tracing: Allow gcov profiling on only ftrace subsystem

2018-08-20 Thread Steven Rostedt
On Fri, 17 Aug 2018 01:29:39 +0900 Masami Hiramatsu wrote: > Add GCOV_PROFILE_FTRACE to allow gcov profiling > on only files in ftrace subsystem. This kconfig > flag will be used for checking kselftest/ftrace Note, I haven't heard of "kconfig flag" as the terminology. Usually, we just say

Re: [PATCH v2 01/32] tracing: Allow gcov profiling on only ftrace subsystem

2018-08-20 Thread Steven Rostedt
On Fri, 17 Aug 2018 01:29:39 +0900 Masami Hiramatsu wrote: > Add GCOV_PROFILE_FTRACE to allow gcov profiling > on only files in ftrace subsystem. This kconfig > flag will be used for checking kselftest/ftrace Note, I haven't heard of "kconfig flag" as the terminology. Usually, we just say

[PATCH v2 01/32] tracing: Allow gcov profiling on only ftrace subsystem

2018-08-16 Thread Masami Hiramatsu
Add GCOV_PROFILE_FTRACE to allow gcov profiling on only files in ftrace subsystem. This kconfig flag will be used for checking kselftest/ftrace coverage. Signed-off-by: Masami Hiramatsu --- kernel/trace/Kconfig | 13 + kernel/trace/Makefile |5 + 2 files changed, 18

[PATCH v2 01/32] tracing: Allow gcov profiling on only ftrace subsystem

2018-08-16 Thread Masami Hiramatsu
Add GCOV_PROFILE_FTRACE to allow gcov profiling on only files in ftrace subsystem. This kconfig flag will be used for checking kselftest/ftrace coverage. Signed-off-by: Masami Hiramatsu --- kernel/trace/Kconfig | 13 + kernel/trace/Makefile |5 + 2 files changed, 18