Re: [PATCHSET 0/7] perf sched: Introduce timehist command, again (v2)

From: Jiri Olsa
Date: Wed Nov 16 2016 - 10:34:37 EST


On Wed, Nov 16, 2016 at 03:06:26PM +0900, Namhyung Kim wrote:
> Hello,
>
> This patchset is a rebased version of David's sched timehist work [1].
> I plan to improve perf sched command more and think that having
> timehist command before the work looks good. It seems David is busy
> these days, so I'm retrying it by myself.
>
> * changes in v2)
> - change name 'b/n time' to 'wait time' (Ingo)
> - show arrow between functions in the callchain (Ingo)
> - fix a bug in calculating initial run time

you might want to add some column width cuts.. I've got this:

time cpu task name wait time sch delay run time
[tid/pid] (msec) (msec) (msec)
--------------- ------ -------------------- --------- --------- ---------
482381.387934 [0000] perf[6870] 0.000 0.000 0.000
482381.388838 [0002] <idle> 0.000 0.000 0.000
482381.388845 [0002] plugin-containe[25990/25970] 0.000 0.004 0.007
482381.388897 [0003] <idle> 0.000 0.000 0.000
482381.388919 [0002] <idle> 0.007 0.000 0.074
482381.388925 [0003] Timer[15210/15178] 0.000 0.002 0.027



other than that:

Acked-by: Jiri Olsa <jolsa@xxxxxxxxxx>

thanks,
jirka