lkml.org 
[lkml]   [2011]   [May]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [BUG] perf: bogus correlation of kernel symbols
    Hi Pekka,

    On Thu, May 12, 2011 at 10:58:53PM +0300, Pekka Enberg wrote:
    > On Thu, May 12, 2011 at 10:01 PM, David Miller <davem@davemloft.net> wrote:
    > > From: Dave Jones <davej@redhat.com>
    > > Date: Thu, 12 May 2011 14:37:41 -0400
    > >
    > >> On Thu, May 12, 2011 at 02:06:30PM -0400, David Miller wrote:
    > >>  > I hate this too, and I think it's absolutely rediculous.
    > >>  >
    > >>  > Also, like you, I lost an entire afternoon trying to figure out why
    > >>  > this started happening.
    > >>  >
    > >>  > I wish we could revert this change.
    > >>
    > >> At least it can be permanently disabled..
    > >>
    > >> echo kernel.kptr_restrict = 0 >> /etc/sysctl.conf
    > >
    > > Regardless, what to do about all of the "perf is broken" reports?
    >
    > Lets revert the commit 9f36e2c448007b54851e7e4fa48da97d1477a175
    > ("printk: use %pK for /proc/kallsyms and /proc/modules"), please! I
    > too have been wondering what's going on with perf reporting insane
    > symbols and this should definitely not be enabled by default.

    No, reverting that is not the answer. If perf has a problem with the
    kptr_restrict feature, it should just disable it in /proc/sys when it
    runs and restore it when finished. Since our defaults should be secure
    for the average user (who does not use perf), it's fine the way it
    is. Anyone using perf can adjust this for their use-case (that is why
    there is a /proc/sys tunable).

    -Kees

    --
    Kees Cook
    Ubuntu Security Team
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2011-05-13 08:17    [W:0.023 / U:0.212 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site