Re: [PATCH] perf trace: Tweak summary output

[Ingo Molnar <mingo@kernel.org>]

* Pekka Enberg <penberg@kernel.org> wrote:

> Tweak the summary output as suggested by Ingo Molnar:
> 
>   [penberg@localhost ~]$ perf trace -a --duration 10000 --summary -- sleep 1
>   ^C
>    Summary of events:
> 
>    Xorg (817), 148 events, 0.0%, 0.000 msec
> 
>      syscall            calls      min       avg       max      stddev
>                                  (msec)    (msec)    (msec)        (%)
>      --------------- -------- --------- --------- ---------     ------
>      read                   7     0.002     0.004     0.011     32.00%
>      rt_sigprocmask        40     0.001     0.001     0.002      1.31%
>      ioctl                  6     0.002     0.003     0.005     19.45%
>      writev                 7     0.004     0.018     0.059     43.76%
>      select                 9     0.000    74.513   507.869     74.61%
>      setitimer              4     0.001     0.002     0.002     10.08%

Acked-by: Ingo Molnar <mingo@kernel.org>

Are you convinced about double-digit precision output for percentages?

I think it suggest a false level of accuracy, but YYMV - typographically 
xx.yy certainly looks nicer than xx.y (it's more symmetric).

Thanks,

	Ingo