[PATCH 0/2] perf evsel: add support for inlined function in callchains

From: Jonas Rabenstein
Date: Tue Feb 19 2019 - 13:38:33 EST


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.

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