[PATCH] Revert "perf: Require exclude_guest to use PEBS - kernelside enforcement"

From: Ingo Molnar
Date: Thu Dec 13 2012 - 02:48:31 EST



* David Ahern <dsahern@xxxxxxxxx> wrote:

> On 12/12/12 8:34 PM, Linus Torvalds wrote:
> >On Wed, Dec 12, 2012 at 7:25 PM, David Ahern <dsahern@xxxxxxxxx> wrote:
> >>
> >>Are you running an older perf binary on the 3.8 kernel?
> >
> >I am.. I don't tend to rebuild 'perf'..
> >
> >>Does this work: perf record -e cycles:ppH ...
> >
> >Yes it does. What is 'H' and why should anybody care? Especially since
> >I'm not running virtualized.
> >
> > That whole "exclude_guest" test is insane when there isn't
> > any virtualization going on. Very annoying.
>
> you know what's worse? [...]

No, nothing can be worse than breaking 99% of our installed
base...

I'm wondering where this broke - is it:

20b279ddb38c perf: Require exclude_guest to use PEBS - kernel side enforcement

Linus, does the straight revert below fix everything for you -
or do we need to do more?

( The VM problem needs a different fix: a new include_guest bit
should be introduced, which would naturally default to 'off'
on older binaries, and the old bit should be phased out. Then
new perf binaries can turn on that bit safely. Or PEBS should
be fixed for guests. Or something along these lines - but
it should *not* by fixed by regressing existing binaries ... )

Thanks,

Ingo

----------------->