Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1545074

Re: Inlined functions in perf report

Show all headers | View raw

On Tue, Dec 20, 2016 at 12:59:54PM +0100, Steinar H. Gunderson wrote:
> Hi Peter,
> 
> I can't find a good point of contact for perf, so I'm contacting you based on
> the MAINTAINERS file; feel free to redirect somewhere if you're not the right
> person.
> 

Cc'ed linux-perf-users@vger.kernel.org

> I'm trying to figure out how to deal with perf report when there are inlined
> functions; they don't generally seem to show up in the call stack, which
> sometimes can make it very hard to figure out what is going, especially in
> a code base one doesn't know too well. As an example, I threw together a
> minimal test program:
> 
>   #include <stdlib.h>
>   
>   inline int foo()
>   {
>           int k = rand();
>           int sum = 1;
>           for (int i = 0; i < 10000000000; ++i)
>           {
>                   sum ^= k;
>                   sum += k;
>           }
>           return sum;
>   }
>   
>   int main(void)
>   {
>           return foo();
>   }
> 
> Compiling with -O2 -g, and running perf record -g yields:
> 
>   # Samples: 6K of event 'cycles:ppp'
>   # Event count (approx.): 5876825543
>   #
>   # Children      Self  Command  Shared Object      Symbol                
>   # ........  ........  .......  .................  ......................
>   #
>       99.98%    99.98%  inline   inline             [.] main
>               |
>               ---0x706258d4c544155
>                  main
>   
>       99.98%     0.00%  inline   [unknown]          [.] 0x0706258d4c544155
>               |
>               ---0x706258d4c544155
>                  main
> 
> Is there a way I can get it to show “foo” in the call graph? (I suppose also
> ideally, “foo” and not “main” should show up in a non-graph run.) Of course,
> this gets even more confusing if foo calls bar, since it now looks like the
> call chain is main -> bar directly.
> 
> I have debug information that should be sufficient in the binary, because if
> I break in gdb, I definitely get the call stack:
> 
>   Program received signal SIGINT, Interrupt.
>   0x0000555555554589 in foo () at inline.c:5
>   5               int k = rand();
>   (gdb) bt
>   #0  0x0000555555554589 in foo () at inline.c:5
>   #1  main () at inline.c:17
>   (gdb) 
> 
> FWIW, this is with perf from 4.10 (git as of a few days ago) and GCC 6.2.1.

OK, so it might be possible with: perf record -g --call-graph dwarf
but that's fairly heavy on the overhead, it will dump the top-of-stack
for each sample (8k default) and unwind using libunwind in userspace.

The default mechanism used for call-graphs is frame-pointers which are
(relatively) simple and fast to traverse from kernel space. The down
side is of course that all your userspace needs to be compiled with
frame pointers enabled and inlined functions, as you noticed, are
'lost'.

There has been talk to attempt to utilize the ELF EH frames which are
mandatory in the x86_64 ABI (even for C) to attempt a kernel based
'DWARF' unwind, but nobody has put forward working code for this yet.
Also, even if the EH stuff is mapped at runtime, it doesn't mean the
pages will actually be loaded (due to demand paging) and available for
use, which also will limit usability. (perf sampling is using
interrupt/NMI context and we cannot page from that, so we're limited to
memory that's present.)

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

Re: Inlined functions in perf report Peter Zijlstra <peterz@infradead.org> - 2016-12-20 13:20 +0100
  Re: Inlined functions in perf report Milian Wolff <milian.wolff@kdab.com> - 2016-12-20 14:40 +0100
    Re: Inlined functions in perf report "Steinar H. Gunderson" <sgunderson@bigfoot.com> - 2016-12-20 14:50 +0100
      Re: Inlined functions in perf report Milian Wolff <milian.wolff@kdab.com> - 2016-12-20 15:10 +0100
    Re: Inlined functions in perf report Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-12-20 15:00 +0100
      Re: Inlined functions in perf report "Steinar H. Gunderson" <sgunderson@bigfoot.com> - 2016-12-20 15:10 +0100
        Re: Inlined functions in perf report Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-12-20 15:50 +0100
          Re: Inlined functions in perf report "Steinar H. Gunderson" <sgunderson@bigfoot.com> - 2016-12-20 18:10 +0100
            Re: Inlined functions in perf report "Jin, Yao" <yao.jin@linux.intel.com> - 2016-12-21 02:00 +0100
              Re: Inlined functions in perf report "Steinar H. Gunderson" <sgunderson@bigfoot.com> - 2016-12-21 11:00 +0100
                Re: Inlined functions in perf report Milian Wolff <milian.wolff@kdab.com> - 2016-12-21 11:10 +0100
                Re: Inlined functions in perf report "Steinar H. Gunderson" <sgunderson@bigfoot.com> - 2016-12-21 11:30 +0100
                Re: Inlined functions in perf report "Jin, Yao" <yao.jin@linux.intel.com> - 2016-12-22 00:00 +0100
                Re: Inlined functions in perf report "Steinar H. Gunderson" <sgunderson@bigfoot.com> - 2016-12-22 00:10 +0100
      Re: Inlined functions in perf report Milian Wolff <milian.wolff@kdab.com> - 2016-12-20 15:10 +0100

csiph-web