[PATCH v2 1/4] Add a new element for the struct perf_tool, and add the --per-event-dump option for perf script

2017-09-26 Thread yuzhoujian
This patch will add a new bool variable per_event_dump to mark the per-event-dump option, and add the OPT_BOOLEAN for per-event-dump in the perf_data_file struct. Changes since v1: - remove the set for script.tool.per_event_dump variable,since the OPT_BOOLEAN will do the same thing.

[PATCH v2 1/4] Add a new element for the struct perf_tool, and add the --per-event-dump option for perf script

2017-09-26 Thread yuzhoujian
This patch will add a new bool variable per_event_dump to mark the per-event-dump option, and add the OPT_BOOLEAN for per-event-dump in the perf_data_file struct. Changes since v1: - remove the set for script.tool.per_event_dump variable,since the OPT_BOOLEAN will do the same thing.

Re: [PATCH v2 1/4] Add a new element for the struct perf_tool, and add the --per-event-dump option for perf script

2017-09-25 Thread David Ahern
On 9/25/17 8:41 PM, 禹舟键 wrote: > The changelogs are in the cover-letter. > changes in v2: >         - remove the set of script.tool.per_event_dump variable. >         - add the __maybe_unused attribute for the fp argument in the > second patch. >         - remove the fp_selection_helper function

Re: [PATCH v2 1/4] Add a new element for the struct perf_tool, and add the --per-event-dump option for perf script

2017-09-25 Thread David Ahern
On 9/25/17 8:41 PM, 禹舟键 wrote: > The changelogs are in the cover-letter. > changes in v2: >         - remove the set of script.tool.per_event_dump variable. >         - add the __maybe_unused attribute for the fp argument in the > second patch. >         - remove the fp_selection_helper function

Re: [PATCH v2 1/4] Add a new element for the struct perf_tool, and add the --per-event-dump option for perf script

2017-09-25 Thread Jiri Olsa
On Sun, Sep 24, 2017 at 07:12:24PM +0800, yuzhoujian wrote: > Signed-off-by: yuzhoujian you're missing changelogs in all 4 patches.. you need to describe the change in here.. check 'git log' on kernel tee for examples jirka > --- > tools/perf/builtin-script.c | 3

Re: [PATCH v2 1/4] Add a new element for the struct perf_tool, and add the --per-event-dump option for perf script

2017-09-25 Thread Jiri Olsa
On Sun, Sep 24, 2017 at 07:12:24PM +0800, yuzhoujian wrote: > Signed-off-by: yuzhoujian you're missing changelogs in all 4 patches.. you need to describe the change in here.. check 'git log' on kernel tee for examples jirka > --- > tools/perf/builtin-script.c | 3 +++ > tools/perf/util/tool.h

[PATCH v2 1/4] Add a new element for the struct perf_tool, and add the --per-event-dump option for perf script

2017-09-24 Thread yuzhoujian
Signed-off-by: yuzhoujian --- tools/perf/builtin-script.c | 3 +++ tools/perf/util/tool.h | 1 + 2 files changed, 4 insertions(+) diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c index 9092de0..936af8b 100644 ---

[PATCH v2 1/4] Add a new element for the struct perf_tool, and add the --per-event-dump option for perf script

2017-09-24 Thread yuzhoujian
Signed-off-by: yuzhoujian --- tools/perf/builtin-script.c | 3 +++ tools/perf/util/tool.h | 1 + 2 files changed, 4 insertions(+) diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c index 9092de0..936af8b 100644 --- a/tools/perf/builtin-script.c +++

[PATCH v2 1/4] Add a new element for the struct perf_tool, and add the --per-event-dump option for perf script

2017-09-23 Thread yuzhoujian
Signed-off-by: yuzhoujian --- tools/perf/builtin-script.c | 3 +++ tools/perf/util/tool.h | 1 + 2 files changed, 4 insertions(+) diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c index 9092de0..936af8b 100644 ---

[PATCH v2 1/4] Add a new element for the struct perf_tool, and add the --per-event-dump option for perf script

2017-09-23 Thread yuzhoujian
Signed-off-by: yuzhoujian --- tools/perf/builtin-script.c | 3 +++ tools/perf/util/tool.h | 1 + 2 files changed, 4 insertions(+) diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c index 9092de0..936af8b 100644 --- a/tools/perf/builtin-script.c +++