lkml.org 
[lkml]   [2019]   [Feb]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 0/2] perf evsel: add support for inlined function in callchains
Em Tue, Feb 19, 2019 at 07:38:08PM +0100, Jonas Rabenstein escreveu:
> Hi,
> sample__fprintf_callchain currently did not use the already available
> code to get the symbols of an inlined function if such information is
> available in a dso. This patchset adds the required logic to add
> appropriate lines.

You forgot to add the person that did lots of work in this area more
recently, Milian, which I'm adding here so that he have the opportunity
to comment on your work,

Thanks for working on this!

- Arnaldo

> As I am quite new to the code base of perf I am not sure how to test
> that changeset in a correct way. At least the codes builds and the tools
> that make use of sample__fprintf_callchain (perf-script, perf-trace and
> perf-sched as far as I can see) did not fail to run - also I did not get
> into the details of perf-trace and perf-sched as I have never used them
> before.
>
> Another thing I am not sure how to deal with are some warnings of
> checkpatch.pl due to the 80 character line limit. Due to the long
> function names in use the current implementation already exceeded that
> limit in the same spots by even more characters as I have taken the
> inner loop and put it into a separate function.
>
> I hope to expand my work to perf-report but thought it might be usefull
> to get already early feedback on those patches.
>
> Thank you,
> Jonas
>
> Jonas Rabenstein (2):
> perf evsel: split sample__fprintf_callchain in output and iteration
> perf evsel: add inline functions to sample callchain output
>
> tools/perf/util/evsel_fprintf.c | 157 ++++++++++++++++++++++----------
> 1 file changed, 109 insertions(+), 48 deletions(-)
>
> --
> 2.19.2

--

- Arnaldo

\
 
 \ /
  Last update: 2019-02-19 20:39    [W:0.081 / U:30.064 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site