From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932337AbbJ0NQN (ORCPT ); Tue, 27 Oct 2015 09:16:13 -0400 Received: from mail.kernel.org ([198.145.29.136]:50033 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754276AbbJ0NQM (ORCPT ); Tue, 27 Oct 2015 09:16:12 -0400 Date: Tue, 27 Oct 2015 10:16:05 -0300 From: Arnaldo Carvalho de Melo To: Jiri Olsa Cc: lkml , David Ahern , Ingo Molnar , Namhyung Kim , Peter Zijlstra , "Liang, Kan" Subject: Re: [PATCH 11/52] perf tools: Add stat config event Message-ID: <20151027131605.GA9405@kernel.org> References: <1445784728-21732-1-git-send-email-jolsa@kernel.org> <1445784728-21732-12-git-send-email-jolsa@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1445784728-21732-12-git-send-email-jolsa@kernel.org> X-Url: http://acmel.wordpress.com User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Em Sun, Oct 25, 2015 at 03:51:27PM +0100, Jiri Olsa escreveu: > Adding stat config event to pass/store stat config data, > so report tools (report/script) know how to interpret > stat data. > > The config data are stored in 'tag|value' way to allow > easy extension and backward compatibility. I wonder if this couldn't be renamed 'PERF_RECORD_CONFIG' and just go on using one of those 2^64-1 tags for the 'stat config' needs, but then this is just a matter of changing the name of this event, which won't break anything when done. Other stuff we may want to have stored like this include sysctl, sysfs values, kernel command line options used, etc. - Arnaldo > Link: http://lkml.kernel.org/n/tip-1npdsfez8635vogthpqwtkd0@git.kernel.org > Signed-off-by: Jiri Olsa > --- > tools/perf/util/event.c | 1 + > tools/perf/util/event.h | 20 ++++++++++++++++++++ > tools/perf/util/session.c | 24 ++++++++++++++++++++++++ > tools/perf/util/tool.h | 3 ++- > 4 files changed, 47 insertions(+), 1 deletion(-) > > diff --git a/tools/perf/util/event.c b/tools/perf/util/event.c > index 36dc992b072f..1165cb1d891f 100644 > --- a/tools/perf/util/event.c > +++ b/tools/perf/util/event.c > @@ -39,6 +39,7 @@ static const char *perf_event__names[] = { > [PERF_RECORD_AUXTRACE_ERROR] = "AUXTRACE_ERROR", > [PERF_RECORD_THREAD_MAP] = "THREAD_MAP", > [PERF_RECORD_CPU_MAP] = "CPU_MAP", > + [PERF_RECORD_STAT_CONFIG] = "STAT_CONFIG", > }; > > const char *perf_event__name(unsigned int id) > diff --git a/tools/perf/util/event.h b/tools/perf/util/event.h > index e46f95285350..0bc3393dd28c 100644 > --- a/tools/perf/util/event.h > +++ b/tools/perf/util/event.h > @@ -228,6 +228,7 @@ enum perf_user_event_type { /* above any possible kernel type */ > PERF_RECORD_AUXTRACE_ERROR = 72, > PERF_RECORD_THREAD_MAP = 73, > PERF_RECORD_CPU_MAP = 74, > + PERF_RECORD_STAT_CONFIG = 75, > PERF_RECORD_HEADER_MAX > }; > > @@ -395,6 +396,24 @@ struct thread_map_event { > struct thread_map_data_event data[]; > }; > > +enum { > + PERF_STAT_CONFIG_TERM__AGGR_MODE = 0, > + PERF_STAT_CONFIG_TERM__INTERVAL = 1, > + PERF_STAT_CONFIG_TERM__SCALE = 2, > + PERF_STAT_CONFIG_TERM__MAX = 3, > +}; > + > +struct stat_config_term_event { > + u64 tag; > + u64 val; > +}; > + > +struct stat_config_event { > + struct perf_event_header header; > + u64 nr; > + struct stat_config_term_event data[]; > +}; > + > union perf_event { > struct perf_event_header header; > struct mmap_event mmap; > @@ -419,6 +438,7 @@ union perf_event { > struct context_switch_event context_switch; > struct thread_map_event thread_map; > struct cpu_map_event cpu_map; > + struct stat_config_event stat_config; > }; > > void perf_event__print_totals(void); > diff --git a/tools/perf/util/session.c b/tools/perf/util/session.c > index 5126b18c671c..a550464925ea 100644 > --- a/tools/perf/util/session.c > +++ b/tools/perf/util/session.c > @@ -315,6 +315,15 @@ int process_event_cpu_map_stub(struct perf_tool *tool __maybe_unused, > return 0; > } > > +static > +int process_event_stat_config_stub(struct perf_tool *tool __maybe_unused, > + union perf_event *event __maybe_unused, > + struct perf_session *session __maybe_unused) > +{ > + dump_printf(": unhandled!\n"); > + return 0; > +} > + > void perf_tool__fill_defaults(struct perf_tool *tool) > { > if (tool->sample == NULL) > @@ -369,6 +378,8 @@ void perf_tool__fill_defaults(struct perf_tool *tool) > tool->thread_map = process_event_thread_map_stub; > if (tool->cpu_map == NULL) > tool->cpu_map = process_event_cpu_map_stub; > + if (tool->stat_config == NULL) > + tool->stat_config = process_event_stat_config_stub; > } > > static void swap_sample_id_all(union perf_event *event, void *data) > @@ -686,6 +697,16 @@ static void perf_event__cpu_map_swap(union perf_event *event, > } > } > > +static void perf_event__stat_config_swap(union perf_event *event, > + bool sample_id_all __maybe_unused) > +{ > + u64 size; > + > + size = event->stat_config.nr * sizeof(event->stat_config.data[0]); > + size += 1; /* nr item itself */ > + mem_bswap_64(&event->stat_config.nr, size); > +} > + > typedef void (*perf_event__swap_op)(union perf_event *event, > bool sample_id_all); > > @@ -715,6 +736,7 @@ static perf_event__swap_op perf_event__swap_ops[] = { > [PERF_RECORD_AUXTRACE_ERROR] = perf_event__auxtrace_error_swap, > [PERF_RECORD_THREAD_MAP] = perf_event__thread_map_swap, > [PERF_RECORD_CPU_MAP] = perf_event__cpu_map_swap, > + [PERF_RECORD_STAT_CONFIG] = perf_event__stat_config_swap, > [PERF_RECORD_HEADER_MAX] = NULL, > }; > > @@ -1255,6 +1277,8 @@ static s64 perf_session__process_user_event(struct perf_session *session, > return tool->thread_map(tool, event, session); > case PERF_RECORD_CPU_MAP: > return tool->cpu_map(tool, event, session); > + case PERF_RECORD_STAT_CONFIG: > + return tool->stat_config(tool, event, session); > default: > return -EINVAL; > } > diff --git a/tools/perf/util/tool.h b/tools/perf/util/tool.h > index 9e5925c78519..aa7ae73d76b4 100644 > --- a/tools/perf/util/tool.h > +++ b/tools/perf/util/tool.h > @@ -57,7 +57,8 @@ struct perf_tool { > auxtrace_info, > auxtrace_error, > thread_map, > - cpu_map; > + cpu_map, > + stat_config; > event_op3 auxtrace; > bool ordered_events; > bool ordering_requires_timestamps; > -- > 2.4.3