Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-09-02 Thread Jiri Olsa
On Tue, Sep 02, 2014 at 08:39:56AM +0300, Adrian Hunter wrote: SNIP > >>> -e intel_pt/tsc=1,noretcomp=0/ > >>> > >>> I guess I wanted to hear more elaboration why is this better > >>> than the current way we have by defining an alias, like: > >>> > >>> krava alias: "tsc=1,noretcomp=0" > >>>

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-09-02 Thread Jiri Olsa
On Tue, Sep 02, 2014 at 08:39:56AM +0300, Adrian Hunter wrote: SNIP -e intel_pt/tsc=1,noretcomp=0/ I guess I wanted to hear more elaboration why is this better than the current way we have by defining an alias, like: krava alias: tsc=1,noretcomp=0 -e intel_pt/krava/

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-09-01 Thread Adrian Hunter
On 09/01/2014 10:11 PM, Jiri Olsa wrote: > On Mon, Sep 01, 2014 at 09:27:17AM +0300, Adrian Hunter wrote: >> On 08/30/2014 11:53 AM, Jiri Olsa wrote: >>> On Fri, Aug 29, 2014 at 09:48:51PM +0300, Adrian Hunter wrote: On 16/07/2014 9:22 p.m., Jiri Olsa wrote: > On Wed, Jul 16, 2014 at

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-09-01 Thread Jiri Olsa
On Mon, Sep 01, 2014 at 12:51:42PM -0300, Arnaldo Carvalho de Melo wrote: SNIP > > krava alias: "tsc=1,noretcomp=0" > > > -e intel_pt/krava/ > > > which gives the same result > > So you propose that we start maintaining some table of aliases that > would be installed by default, etc?

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-09-01 Thread Jiri Olsa
On Mon, Sep 01, 2014 at 09:27:17AM +0300, Adrian Hunter wrote: > On 08/30/2014 11:53 AM, Jiri Olsa wrote: > > On Fri, Aug 29, 2014 at 09:48:51PM +0300, Adrian Hunter wrote: > >> On 16/07/2014 9:22 p.m., Jiri Olsa wrote: > >>> On Wed, Jul 16, 2014 at 06:04:44PM +0300, Adrian Hunter wrote: > On

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-09-01 Thread Arnaldo Carvalho de Melo
Em Sat, Aug 30, 2014 at 10:53:10AM +0200, Jiri Olsa escreveu: > On Fri, Aug 29, 2014 at 09:48:51PM +0300, Adrian Hunter wrote: > > On 16/07/2014 9:22 p.m., Jiri Olsa wrote: > > >On Wed, Jul 16, 2014 at 06:04:44PM +0300, Adrian Hunter wrote: > > >>On 16/07/2014 5:25 p.m., Jiri Olsa wrote: > > >>>On

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-09-01 Thread Adrian Hunter
On 08/30/2014 11:53 AM, Jiri Olsa wrote: > On Fri, Aug 29, 2014 at 09:48:51PM +0300, Adrian Hunter wrote: >> On 16/07/2014 9:22 p.m., Jiri Olsa wrote: >>> On Wed, Jul 16, 2014 at 06:04:44PM +0300, Adrian Hunter wrote: On 16/07/2014 5:25 p.m., Jiri Olsa wrote: > On Mon, Jul 14, 2014 at

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-09-01 Thread Adrian Hunter
On 08/30/2014 11:53 AM, Jiri Olsa wrote: On Fri, Aug 29, 2014 at 09:48:51PM +0300, Adrian Hunter wrote: On 16/07/2014 9:22 p.m., Jiri Olsa wrote: On Wed, Jul 16, 2014 at 06:04:44PM +0300, Adrian Hunter wrote: On 16/07/2014 5:25 p.m., Jiri Olsa wrote: On Mon, Jul 14, 2014 at 01:02:44PM +0300,

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-09-01 Thread Arnaldo Carvalho de Melo
Em Sat, Aug 30, 2014 at 10:53:10AM +0200, Jiri Olsa escreveu: On Fri, Aug 29, 2014 at 09:48:51PM +0300, Adrian Hunter wrote: On 16/07/2014 9:22 p.m., Jiri Olsa wrote: On Wed, Jul 16, 2014 at 06:04:44PM +0300, Adrian Hunter wrote: On 16/07/2014 5:25 p.m., Jiri Olsa wrote: On Mon, Jul 14,

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-09-01 Thread Jiri Olsa
On Mon, Sep 01, 2014 at 09:27:17AM +0300, Adrian Hunter wrote: On 08/30/2014 11:53 AM, Jiri Olsa wrote: On Fri, Aug 29, 2014 at 09:48:51PM +0300, Adrian Hunter wrote: On 16/07/2014 9:22 p.m., Jiri Olsa wrote: On Wed, Jul 16, 2014 at 06:04:44PM +0300, Adrian Hunter wrote: On 16/07/2014

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-09-01 Thread Jiri Olsa
On Mon, Sep 01, 2014 at 12:51:42PM -0300, Arnaldo Carvalho de Melo wrote: SNIP krava alias: tsc=1,noretcomp=0 -e intel_pt/krava/ which gives the same result So you propose that we start maintaining some table of aliases that would be installed by default, etc? krava would not

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-09-01 Thread Adrian Hunter
On 09/01/2014 10:11 PM, Jiri Olsa wrote: On Mon, Sep 01, 2014 at 09:27:17AM +0300, Adrian Hunter wrote: On 08/30/2014 11:53 AM, Jiri Olsa wrote: On Fri, Aug 29, 2014 at 09:48:51PM +0300, Adrian Hunter wrote: On 16/07/2014 9:22 p.m., Jiri Olsa wrote: On Wed, Jul 16, 2014 at 06:04:44PM +0300,

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-08-30 Thread Jiri Olsa
On Fri, Aug 29, 2014 at 09:48:51PM +0300, Adrian Hunter wrote: > On 16/07/2014 9:22 p.m., Jiri Olsa wrote: > >On Wed, Jul 16, 2014 at 06:04:44PM +0300, Adrian Hunter wrote: > >>On 16/07/2014 5:25 p.m., Jiri Olsa wrote: > >>>On Mon, Jul 14, 2014 at 01:02:44PM +0300, Adrian Hunter wrote: > This

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-08-30 Thread Jiri Olsa
On Fri, Aug 29, 2014 at 09:48:51PM +0300, Adrian Hunter wrote: On 16/07/2014 9:22 p.m., Jiri Olsa wrote: On Wed, Jul 16, 2014 at 06:04:44PM +0300, Adrian Hunter wrote: On 16/07/2014 5:25 p.m., Jiri Olsa wrote: On Mon, Jul 14, 2014 at 01:02:44PM +0300, Adrian Hunter wrote: This enables a PMU

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-08-29 Thread Adrian Hunter
On 16/07/2014 9:22 p.m., Jiri Olsa wrote: On Wed, Jul 16, 2014 at 06:04:44PM +0300, Adrian Hunter wrote: On 16/07/2014 5:25 p.m., Jiri Olsa wrote: On Mon, Jul 14, 2014 at 01:02:44PM +0300, Adrian Hunter wrote: This enables a PMU event to be specified in the form: pmu// which is

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-08-29 Thread Adrian Hunter
On 16/07/2014 9:22 p.m., Jiri Olsa wrote: On Wed, Jul 16, 2014 at 06:04:44PM +0300, Adrian Hunter wrote: On 16/07/2014 5:25 p.m., Jiri Olsa wrote: On Mon, Jul 14, 2014 at 01:02:44PM +0300, Adrian Hunter wrote: This enables a PMU event to be specified in the form: pmu// which is

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-07-16 Thread Jiri Olsa
On Wed, Jul 16, 2014 at 06:04:44PM +0300, Adrian Hunter wrote: > On 16/07/2014 5:25 p.m., Jiri Olsa wrote: > >On Mon, Jul 14, 2014 at 01:02:44PM +0300, Adrian Hunter wrote: > >>This enables a PMU event to be specified in the form: > >> > >>pmu// > >> > >>which is effectively the same as: > >>

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-07-16 Thread Adrian Hunter
On 16/07/2014 5:25 p.m., Jiri Olsa wrote: On Mon, Jul 14, 2014 at 01:02:44PM +0300, Adrian Hunter wrote: This enables a PMU event to be specified in the form: pmu// which is effectively the same as: pmu/config=0/ This patch is a precursor to defining default config for a

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-07-16 Thread Jiri Olsa
On Mon, Jul 14, 2014 at 01:02:44PM +0300, Adrian Hunter wrote: > This enables a PMU event to be specified in the form: > > pmu// > > which is effectively the same as: > > pmu/config=0/ > > This patch is a precursor to defining > default config for a PMU. I understand the need for

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-07-16 Thread Jiri Olsa
On Mon, Jul 14, 2014 at 01:02:44PM +0300, Adrian Hunter wrote: This enables a PMU event to be specified in the form: pmu// which is effectively the same as: pmu/config=0/ This patch is a precursor to defining default config for a PMU. I understand the need for default

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-07-16 Thread Adrian Hunter
On 16/07/2014 5:25 p.m., Jiri Olsa wrote: On Mon, Jul 14, 2014 at 01:02:44PM +0300, Adrian Hunter wrote: This enables a PMU event to be specified in the form: pmu// which is effectively the same as: pmu/config=0/ This patch is a precursor to defining default config for a

Re: [PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-07-16 Thread Jiri Olsa
On Wed, Jul 16, 2014 at 06:04:44PM +0300, Adrian Hunter wrote: On 16/07/2014 5:25 p.m., Jiri Olsa wrote: On Mon, Jul 14, 2014 at 01:02:44PM +0300, Adrian Hunter wrote: This enables a PMU event to be specified in the form: pmu// which is effectively the same as: pmu/config=0/

[PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-07-14 Thread Adrian Hunter
This enables a PMU event to be specified in the form: pmu// which is effectively the same as: pmu/config=0/ This patch is a precursor to defining default config for a PMU. Signed-off-by: Adrian Hunter --- tools/perf/util/parse-events.c | 6 ++

[PATCH 20/41] perf tools: Let a user specify a PMU event without any config terms

2014-07-14 Thread Adrian Hunter
This enables a PMU event to be specified in the form: pmu// which is effectively the same as: pmu/config=0/ This patch is a precursor to defining default config for a PMU. Signed-off-by: Adrian Hunter adrian.hun...@intel.com --- tools/perf/util/parse-events.c | 6 ++