Re: [PATCH v9 0/8] perf/amd/iommu: Enable multi-IOMMU support

From: Borislav Petkov
Date: Wed Feb 15 2017 - 06:14:03 EST


On Wed, Feb 15, 2017 at 11:44:24AM +0100, Jiri Olsa wrote:
> does it say unsupported when you omit -a?

Yeah, it does dump the event name like it was counting but then it says
<not supported>. For example:

<not supported> amd_iommu_0/...

> [jolsa@krava perf]$ ./perf stat -e 'cpu/cpu-cycles/'
^^^^^

Haha, I chuckle everytime I see this hostname :-)))

> > So, lemme ask perf tool people, can we guys make the -a thing default
> > when detect that we're running only uncore events which all should need
> > -a anyway?
>
> it's possible, can't think of anything being hurt by this now..

Right, so Peter meant something simple like: you parse all events, see
that they're all uncore and enable system_wide if so.

I get the feeling that we want to do that for uncore-only events
anyway...

--
Regards/Gruss,
Boris.

Good mailing practices for 400: avoid top-posting and trim the reply.