lkml.org 
[lkml]   [2012]   [Dec]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[PATCH] Revert "perf: Require exclude_guest to use PEBS - kernel side enforcement"

    * David Ahern <dsahern@gmail.com> wrote:

    > On 12/12/12 8:34 PM, Linus Torvalds wrote:
    > >On Wed, Dec 12, 2012 at 7:25 PM, David Ahern <dsahern@gmail.com> 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

    ----------------->
    From 581ba4671bf1d1095e9ecf843be61904e4c97e91 Mon Sep 17 00:00:00 2001
    From: Ingo Molnar <mingo@kernel.org>
    Date: Thu, 13 Dec 2012 08:41:40 +0100
    Subject: [PATCH] Revert "perf: Require exclude_guest to use PEBS - kernel
    side enforcement"

    This reverts commit 20b279ddb38ca42f8863cec07b4d45ec24589f13.

    Signed-off-by: Ingo Molnar <mingo@kernel.org>
    ---
    arch/x86/kernel/cpu/perf_event.c | 6 ------
    1 file changed, 6 deletions(-)

    diff --git a/arch/x86/kernel/cpu/perf_event.c b/arch/x86/kernel/cpu/perf_event.c
    index 4428fd1..6774c17 100644
    --- a/arch/x86/kernel/cpu/perf_event.c
    +++ b/arch/x86/kernel/cpu/perf_event.c
    @@ -340,9 +340,6 @@ int x86_setup_perfctr(struct perf_event *event)
    /* BTS is currently only allowed for user-mode. */
    if (!attr->exclude_kernel)
    return -EOPNOTSUPP;
    -
    - if (!attr->exclude_guest)
    - return -EOPNOTSUPP;
    }

    hwc->config |= config;
    @@ -385,9 +382,6 @@ int x86_pmu_hw_config(struct perf_event *event)
    if (event->attr.precise_ip) {
    int precise = 0;

    - if (!event->attr.exclude_guest)
    - return -EOPNOTSUPP;
    -
    /* Support for constant skid */
    if (x86_pmu.pebs_active && !x86_pmu.pebs_broken) {
    precise++;

    \
     
     \ /
      Last update: 2012-12-13 09:01    [W:2.445 / U:0.024 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site