Re: [PATCHv4 0/8] perf, tool: Allow to use hw events in PMU syntax

[Stephane Eranian]

On Wed, Oct 10, 2012 at 2:53 PM, Jiri Olsa <jolsa@xxxxxxxxxx> wrote:
> hi,
> here's the change to make following syntax available:
>   perf stat -e cpu/event=instructions/u ls
>   perf stat -e cpu/cycles/u ls
>
Unless, I am mistaken but I did not see in your patch where you adjust the list
of generic Hw events (and encodings) you expose via sysfs based on CPU model
number. Some generic HW PMU may not have actual physical mappings. Look
at IvyBridge, for instance.


> this is identical to:
>   perf stat -e instructions:u ls
>   perf stat -e cycles:u ls
>
> v4 changes:
>   - kernel change is now structured differently allowing
>     to pick up cpu differences
>   - added support for intel/p6/amd PMUs - intel and amd tested,
>     did not get p6 server, but should be ok.. feel free to omit
>
> Attached patches:
>   1/8 perf x86: Making hardware events translations available in sysfs
>   2/8 perf x86: Filter out undefined events from sysfs events attribute
>   3/8 perf x86: Adding hardware events translations for intel cpus
>   4/8 perf x86: Adding hardware events translations for amd cpus
>   5/8 perf x86: Adding hardware events translations for p6 cpus
>   6/8 perf tools: Fix pmu object alias initialization
>   7/8 perf tools: Add support to specify hw event as pmu event term
>   8/8 perf test: Add automated tests for pmu sysfs translated events
>
> jirka
>
> Suggested-by: Peter Zijlstra <a.p.zijlstra@xxxxxxxxx>
> Signed-off-by: Jiri Olsa <jolsa@xxxxxxxxxx>
> Cc: Arnaldo Carvalho de Melo <acme@xxxxxxxxxxxxxxxxxx>
> Cc: Peter Zijlstra <a.p.zijlstra@xxxxxxxxx>
> Cc: Ingo Molnar <mingo@xxxxxxx>
> Cc: Paul Mackerras <paulus@xxxxxxxxx>
> Cc: Corey Ashford <cjashfor@xxxxxxxxxxxxxxxxxx>
> Cc: Frederic Weisbecker <fweisbec@xxxxxxxxx>
> Cc: Stephane Eranian <eranian@xxxxxxxxxx>
> ---
>  arch/x86/kernel/cpu/perf_event.c       | 121 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  arch/x86/kernel/cpu/perf_event.h       |   2 ++
>  arch/x86/kernel/cpu/perf_event_amd.c   |   9 +++++++
>  arch/x86/kernel/cpu/perf_event_intel.c |   9 +++++++
>  arch/x86/kernel/cpu/perf_event_p6.c    |   2 ++
>  include/linux/perf_event.h             |   3 +++
>  tools/perf/util/parse-events-test.c    |  68 ++++++++++++++++++++++++++++++++++++++++++++++++++
>  tools/perf/util/parse-events.c         |  18 ++++++++++++++
>  tools/perf/util/parse-events.h         |   2 ++
>  tools/perf/util/parse-events.y         |  18 ++++++++++++++
>  tools/perf/util/pmu.c                  |   7 +++---
>  11 files changed, 256 insertions(+), 3 deletions(-)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/