>From: Arnaldo Carvalho de Melo [mailto:a...@kernel.org]
>Em Tue, Nov 24, 2015 at 11:54:24AM -0300, Arnaldo Carvalho de Melo escreveu: >> Em Tue, Nov 24, 2015 at 12:18:22PM +0100, Michael Petlan escreveu: >> > I have been playing with perf-probe tool and I found out that some bogus >> > values of a function argument are obtained by perf-record. > >> > How to reproduce: > >> > gcc -O0 -g -o dummy dummy.c >> > perf probe -x ./dummy --add 'isprime a' >> > perf record -e probe_dummy:isprime ./dummy >> > perf script > >> Checking this here, but as non root I get: > >> [acme@zoo ~]$ gcc -O0 -g -o isprime isprime.c >> [acme@zoo ~]$ perf probe -x ./isprime --add 'isprime a' >> Mapping for the register number 4113 missing on this architecture. >> Sorry, we don't support this variable location yet. >> Error: Failed to add events. >> [acme@zoo ~]$ > >Bisected it down to: > >[acme@zoo linux]$ git bisect good >05c8d802fa52ef17dbcce21c38b72b4a313eb036 is the first bad commit >commit 05c8d802fa52ef17dbcce21c38b72b4a313eb036 >Author: Masami Hiramatsu <masami.hiramatsu...@hitachi.com> >Date: Wed Nov 18 15:40:12 2015 +0900 > > perf probe: Fix to free temporal Dwarf_Frame > > Since dwarf_cfi_addrframe returns malloc'd Dwarf_Frame object, it has to > be freed after it is used. > > Signed-off-by: Masami Hiramatsu <masami.hiramatsu...@hitachi.com> > Cc: Adrian Hunter <adrian.hun...@intel.com> > Cc: Jiri Olsa <jo...@redhat.com> > Cc: Namhyung Kim <namhy...@kernel.org> > Cc: Peter Zijlstra <a.p.zijls...@chello.nl> > Link: > http://lkml.kernel.org/r/20151118064011.30709.65674.stgit@localhost.localdomain > Signed-off-by: Arnaldo Carvalho de Melo <a...@redhat.com> > Ah, OK. I miss understood. The dwarf_frame_cfa can return a part of Dwarf_Frame(afaics libdw source code), so it should be freed right after the Dwarf_Op (pf->fb_ops) is used. OK, I'll fix that. Thanks! -- To unsubscribe from this list: send the line "unsubscribe linux-perf-users" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
