Re: [PATCH] perf trace: Simplify '--summary' output

2013-11-13 Thread Ingo Molnar
* Pekka Enberg wrote: > On 11/12/13 11:40 PM, Ingo Molnar wrote: > >So if you prefer unit-less lines that's defensible, perhaps output the > >unit somewhere else: > > > > syscallcalls min avg max stddev > >(msec) (msec) (msec)

Re: [PATCH] perf trace: Simplify '--summary' output

2013-11-12 Thread Pekka Enberg
On 11/12/13 11:40 PM, Ingo Molnar wrote: So if you prefer unit-less lines that's defensible, perhaps output the unit somewhere else: syscallcalls min avg max stddev (msec) (msec) (msec)(%) --- -

Re: [PATCH] perf trace: Simplify '--summary' output

2013-11-12 Thread Ingo Molnar
* Pekka Enberg wrote: > > %. That was dropped in the recent output change. > > Sorry about that. Why do we show them in percentages, btw? Standard > deviation is usually represented in the same unit as the data to make it > readable. So, the problem with output in the same unit is that it's

Re: [PATCH] perf trace: Simplify '--summary' output

2013-11-12 Thread David Ahern
On 11/12/13, 2:38 PM, Pekka Enberg wrote: Sorry about that. Why do we show them in percentages, btw? Standard deviation is usually represented in the same unit as the data to make it readable. Consistency. perf-stat shows relative stddev, for example. David -- To unsubscribe from this list:

Re: [PATCH] perf trace: Simplify '--summary' output

2013-11-12 Thread Ingo Molnar
* David Ahern wrote: > On 11/12/13, 2:33 PM, Ingo Molnar wrote: > >>> Summary of events: > >>> > >>> dbus-daemon (555), 10 events, 0.0%, 0.000 msec > >>> > >>> msec/call > >>>syscallcalls min avg max stddev > >>

Re: [PATCH] perf trace: Simplify '--summary' output

2013-11-12 Thread Pekka Enberg
On 11/12/13 11:36 PM, David Ahern wrote: On 11/12/13, 2:33 PM, Ingo Molnar wrote: > Summary of events: > > dbus-daemon (555), 10 events, 0.0%, 0.000 msec > > msec/call >syscallcalls min avg max stddev >--- -

Re: [PATCH] perf trace: Simplify '--summary' output

2013-11-12 Thread David Ahern
On 11/12/13, 2:33 PM, Ingo Molnar wrote: > Summary of events: > > dbus-daemon (555), 10 events, 0.0%, 0.000 msec > > msec/call >syscallcalls min avg max stddev >---

Re: [PATCH] perf trace: Simplify '--summary' output

2013-11-12 Thread Ingo Molnar
* Pekka Enberg wrote: > The output of 'perf trace --summary' tries to be too cute with > formatting and makes it very hard to read. Simplify it in the spirit of > "strace -c": > > [penberg@localhost libtrading]$ perf trace -a --duration 1 --summary -- > sleep 1 > ^C Btw., just a very sma

[PATCH] perf trace: Simplify '--summary' output

2013-11-12 Thread Pekka Enberg
The output of 'perf trace --summary' tries to be too cute with formatting and makes it very hard to read. Simplify it in the spirit of "strace -c": [penberg@localhost libtrading]$ perf trace -a --duration 1 --summary -- sleep 1 ^C Summary of events: dbus-daemon (555), 10 events, 0.0%, 0.0