lkml.org 
[lkml]   [2017]   [Oct]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v5 16/16] perf util: use correct IP mapping to find srcline for hist entry
On Thu, Oct 12, 2017 at 08:22:58PM +0200, Milian Wolff wrote:

SNIP

> > > return SRCLINE_UNKNOWN;
> > >
> > > - return get_srcline(map->dso, map__rip_2objdump(map, he->ip),
> > > + return get_srcline(map->dso, map__objdump_2mem(map, he->ip),
> > >
> > > he->ms.sym, true, true);
> >
> > I'm not sure this is right. IIRC hist_entry->ip is a relative IP so
> > it needs to be changed for objdump use. Maybe there's some bug on
> > translating the address but it seems that the original code is
> > correct. And this change breaks srcline for my test program (which is
> > a PIE).
>
> Odd, I'll have to look at that. But this is imo unrelated to the rest of the
> patch series. So maybe we skip this one and apply the others. Assuming those
> other patches are OK by now?
>
> Jiri, you also reviewed it before, is there anything else missing?

I plan to check on it this week.. sry for delay

jirka

\
 
 \ /
  Last update: 2017-10-12 20:53    [W:0.094 / U:25.964 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site