On 5/10/2018 1:24 AM, Arnaldo Carvalho de Melo wrote:
Em Wed, May 09, 2018 at 01:42:51PM -0300, Arnaldo Carvalho de Melo escreveu:
Em Wed, May 09, 2018 at 11:57:15PM +0800, Jin Yao escreveu:
When we perform following command lines:
perf record -e "{cycles,branches}" ./div
perf annotate main --stdio
Only shows one event "cycles" and the displaying format is not correct.
Percent | Source code & Disassembly of div for cycles (44550
samples)
-----------------------------------------------------------------------------------
Please indent command output (two spaces before all lines) to avoid
scripts breaking due to mistaking lines starting with --- as the end of
the patch.
Applying.
Also you forgot to add this:
Cc: Jiri Olsa <[email protected]>
Fixes: f681d593d1ce ("perf annotate: Remove disasm__calc_percent() from
disasm_line__print()")
Which I've just added, to help with stable@ processing.
- Arnaldo
Actually I notice this issue during fixing another issue. :)
Another issue is, without using group, the multiple events can't be
supported well in perf annotate.
perf record -e cycles,branches ./div
perf annotate main --stdio
It should show both cycles and branches on the left side but actually it
only show cycles.
I will post patches for fixing this (working on the patch now).
Thanks
Jin Yao
