Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1523612

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

From Jiri Olsa <jolsa@redhat.com>
Newsgroups linux.kernel
Subject Re: [PATCHSET 0/7] perf sched: Introduce timehist command, again (v2)
Date 2016-11-16 16:40 +0100
Message-ID <sEatA-4HA-53@gated-at.bofh.it> (permalink)
References <sE1zX-7AZ-9@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


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@redhat.com>

thanks,
jirka

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCHSET 0/7] perf sched: Introduce timehist command, again (v2) Namhyung Kim <namhyung@kernel.org> - 2016-11-16 07:10 +0100
  [PATCH v2 1/8] perf symbol: Print symbol offsets conditionally Namhyung Kim <namhyung@kernel.org> - 2016-11-16 07:10 +0100
    [tip:perf/core] perf symbols: Print symbol offsets conditionally tip-bot for Namhyung Kim <tipbot@zytor.com> - 2016-11-24 05:20 +0100
  [PATCH v2 4/8] perf sched timehist: Add summary options Namhyung Kim <namhyung@kernel.org> - 2016-11-16 07:10 +0100
    [tip:perf/core] perf sched timehist: Add summary options tip-bot for David Ahern <tipbot@zytor.com> - 2016-11-24 05:30 +0100
  Re: [PATCHSET 0/7] perf sched: Introduce timehist command, again (v2) Ingo Molnar <mingo@kernel.org> - 2016-11-16 09:00 +0100
  Re: [PATCHSET 0/7] perf sched: Introduce timehist command, again (v2) Jiri Olsa <jolsa@redhat.com> - 2016-11-16 16:40 +0100
    Re: [PATCHSET 0/7] perf sched: Introduce timehist command, again (v2) Namhyung Kim <namhyung@kernel.org> - 2016-11-22 08:10 +0100
  Re: [PATCHSET 0/7] perf sched: Introduce timehist command, again (v2) Namhyung Kim <namhyung@kernel.org> - 2016-11-22 08:10 +0100

csiph-web