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

[Jiri Olsa]

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