Re: [PATCH 0/5] Add multiple event support to perf report V2

Arnaldo Carvalho de Melo
2010-03-05T15:52:29Z

Em Wed, Mar 03, 2010 at 03:38:22PM +0000, Eric B Munson escreveu:
> Currently perf report is unable to handle multiple events even though
> perf record is capable of capturing such a stream. For instance if we run
> 'perf record -e dtlb-misses -e cache-misses sleep 5'
> today and then ask report to show the results we get:
>
> # Samples: 18866
> #
> # Overhead Command Shared Object Symbol
> # ........ ....... ................. ......
> #
> 47.52% sleep [kernel.kallsyms] [k] page_fault
> 32.93% sleep [kernel.kallsyms] [k] locks_remove_posix
> 8.52% sleep [kernel.kallsyms] [k] acl_permission_check
> 3.46% sleep [kernel.kallsyms] [k] __wake_up_bit
> 3.31% sleep [kernel.kallsyms] [k] vfs_read
> 3.23% sleep [kernel.kallsyms] [k] kmem_cache_alloc
> 0.56% sleep [kernel.kallsyms] [k] get_page_from_freelist
> 0.15% sleep [kernel.kallsyms] [k] pipe_read
> 0.13% sleep [kernel.kallsyms] [k] pipe_lock_nested
> 0.13% sleep [kernel.kallsyms] [k] mm_alloc
> 0.06% sleep [kernel.kallsyms] [k] mm_init
> #
> # (For a higher level overview, try: perf report --sort comm,dso)
> #
>
> Worse than only reporting on one of the events, perf report has
> summed the two events. i.e. There is no distinguishing between
> dtlb-misses and cache-misses.
>
> This set of patches addresses this problem and now when perf report
> is run with the same event stream that produced the above report, the
> output is this:

Thanks a lot, pulled and pushed out!

- Arnaldo