lkml.org 
[lkml]   [2014]   [Jul]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 2/2] perf, x86: Don't mark DataLA addresses as store
    On Tue, Jul 15, 2014 at 12:49:43AM +0200, Stephane Eranian wrote:
    > On Tue, Jul 15, 2014 at 12:39 AM, Andi Kleen <ak@linux.intel.com> wrote:
    > >> I have a problem with this patch.
    > >>
    > >> It makes: perf mem -t store rec record OP_NA for the store.
    > >> It was recording OP_STORE before.
    > >>
    > >> I think we need to keep LD/ST info. This is useful for analysis
    > >> especially if we collect loads/stores simultaneously.
    > >>
    > >> Was working before for the mem-loads, mem-stores events.
    > >
    > > Ok. Would it be enough if it only worked for "mem-stores" and not
    > > all PEBS events?
    > >
    > Ok, do that at a minimum.

    I fixed it now. However it turned out that "perf mem report"
    actually not report mem_op, only mem_lvl.

    You may want to fix that separately.

    -Andi


    \
     
     \ /
      Last update: 2014-07-19 04:21    [W:2.523 / U:0.140 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site