When inline frame resolution is disabled, a bogus srcline is obtained for hist entries:
~~~~~ $ perf report -s sym,srcline --no-inline --stdio -g none 95.21% 0.00% [.] __libc_start_main __libc_start_main+18446603358170398953 95.21% 0.00% [.] _start _start+18446650082411225129 46.67% 0.00% [.] main main+18446650082411225208 38.75% 0.00% [.] hypot hypot+18446603358164312084 23.75% 0.00% [.] main main+18446650082411225151 20.83% 20.83% [.] std::generate_canonical<double, 53ul, std::linear_congruential_engine<unsigned long, 16807ul, 0ul, 2147483647ul> > random.h:143 18.12% 0.00% [.] main main+18446650082411225165 13.12% 13.12% [.] std::generate_canonical<double, 53ul, std::linear_congruential_engine<unsigned long, 16807ul, 0ul, 2147483647ul> > random.tcc:3330 4.17% 4.17% [.] __hypot_finite __hypot_finite+163 4.17% 4.17% [.] std::generate_canonical<double, 53ul, std::linear_congruential_engine<unsigned long, 16807ul, 0ul, 2147483647ul> > random.tcc:3333 4.17% 0.00% [.] __hypot_finite __hypot_finite+18446603358164312227 4.17% 0.00% [.] std::generate_canonical<double, 53ul, std::linear_congruential_engine<unsigned long, 16807ul, 0ul, 2147483647ul> > std::generate_canonical<double, 53ul, std::line 2.92% 0.00% [.] std::generate_canonical<double, 53ul, std::linear_congruential_engine<unsigned long, 16807ul, 0ul, 2147483647ul> > std::generate_canonical<double, 53ul, std::line 2.50% 2.50% [.] __hypot_finite __hypot_finite+11 2.50% 2.50% [.] __hypot_finite __hypot_finite+24 2.50% 0.00% [.] __hypot_finite __hypot_finite+18446603358164312075 2.50% 0.00% [.] __hypot_finite __hypot_finite+18446603358164312088 ~~~~~ Note how we get very large offsets to main and cannot see any srcline from one of the complex or random headers, even though the instruction pointers actually lie in code inlined from there. This patch fixes the mapping to use map__objdump_2mem instead of map__objdump_2mem in hist_entry__get_srcline. This fixes the srcline values for me when inline resolution is disabled: ~~~~~ $ perf report -s sym,srcline --no-inline --stdio -g none 95.21% 0.00% [.] __libc_start_main __libc_start_main+233 95.21% 0.00% [.] _start _start+41 46.88% 0.00% [.] main complex:589 43.96% 0.00% [.] main random.h:185 38.75% 0.00% [.] hypot hypot+20 20.83% 0.00% [.] std::generate_canonical<double, 53ul, std::linear_congruential_engine<unsigned long, 16807ul, 0ul, 2147483647ul> > random.h:143 13.12% 0.00% [.] std::generate_canonical<double, 53ul, std::linear_congruential_engine<unsigned long, 16807ul, 0ul, 2147483647ul> > random.tcc:3330 4.17% 4.17% [.] __hypot_finite __hypot_finite+140715545239715 4.17% 4.17% [.] std::generate_canonical<double, 53ul, std::linear_congruential_engine<unsigned long, 16807ul, 0ul, 2147483647ul> > std::generate_canonical<double, 53ul, std::line 4.17% 0.00% [.] __hypot_finite __hypot_finite+163 4.17% 0.00% [.] std::generate_canonical<double, 53ul, std::linear_congruential_engine<unsigned long, 16807ul, 0ul, 2147483647ul> > random.tcc:3333 2.92% 2.92% [.] std::generate_canonical<double, 53ul, std::linear_congruential_engine<unsigned long, 16807ul, 0ul, 2147483647ul> > std::generate_canonical<double, 53ul, std::line 2.50% 2.50% [.] __hypot_finite __hypot_finite+140715545239563 2.50% 2.50% [.] __hypot_finite __hypot_finite+140715545239576 2.50% 2.50% [.] std::generate_canonical<double, 53ul, std::linear_congruential_engine<unsigned long, 16807ul, 0ul, 2147483647ul> > std::generate_canonical<double, 53ul, std::line 2.50% 2.50% [.] std::generate_canonical<double, 53ul, std::linear_congruential_engine<unsigned long, 16807ul, 0ul, 2147483647ul> > std::generate_canonical<double, 53ul, std::line 2.50% 0.00% [.] __hypot_finite __hypot_finite+11 ~~~~~ Cc: Arnaldo Carvalho de Melo <a...@redhat.com> Cc: Namhyung Kim <namhy...@kernel.org> Cc: Yao Jin <yao....@linux.intel.com> Cc: Jiri Olsa <jo...@redhat.com> Signed-off-by: Milian Wolff <milian.wo...@kdab.com> Note how most of the large offset values are now gone. Most notably, we get proper srcline resolution for the random.h and complex headers. --- tools/perf/util/sort.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c index 006d10a0dc96..6f3d109078a3 100644 --- a/tools/perf/util/sort.c +++ b/tools/perf/util/sort.c @@ -334,7 +334,7 @@ char *hist_entry__get_srcline(struct hist_entry *he) if (!map) 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); } -- 2.14.2