Messages in this thread | | | Date | Sat, 19 Jul 2014 02:49:41 +0200 | From | Andi Kleen <> | Subject | Re: [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
| |