From: Arnaldo Carvalho de Melo <acme@kernel.org>
To: Namhyung Kim <namhyung@kernel.org>
Cc: David Ahern <dsahern@gmail.com>, Ingo Molnar <mingo@kernel.org>,
Peter Zijlstra <a.p.zijlstra@chello.nl>,
Jiri Olsa <jolsa@kernel.org>, LKML <linux-kernel@vger.kernel.org>,
Stephane Eranian <eranian@google.com>,
Andi Kleen <andi@firstfloor.org>
Subject: Re: [PATCH v2 7/8] perf sched timehist: Add -V/--cpu-visual option
Date: Tue, 22 Nov 2016 15:33:26 -0300 [thread overview]
Message-ID: <20161122183326.GD5390@kernel.org> (raw)
In-Reply-To: <20161116060634.28477-8-namhyung@kernel.org>
Em Wed, Nov 16, 2016 at 03:06:33PM +0900, Namhyung Kim escreveu:
> From: David Ahern <dsahern@gmail.com>
>
> The -V option provides a visual aid for sched switches by cpu:
>
> $ perf sched timehist -V
> time cpu 0123456789abc task name b/n time sch delay run time
> [tid/pid] (msec) (msec) (msec)
> --------------- ------ ------------- -------------------- --------- --------- ---------
> ...
> 2412598.429696 [0009] i <idle> 0.000 0.000 0.000
> 2412598.429767 [0002] s perf[7219] 0.000 0.000 0.000
> 2412598.429783 [0009] s perf[7220] 0.000 0.006 0.087
> 2412598.429794 [0010] i <idle> 0.000 0.000 0.000
> 2412598.429795 [0009] s migration/9[53] 0.000 0.003 0.011
> 2412598.430370 [0010] s sleep[7220] 0.011 0.000 0.576
> 2412598.432584 [0003] i <idle> 0.000 0.000 0.000
> ...
Forgot to add docs, will do.
- Arnaldo
> Signed-off-by: David Ahern <dsahern@gmail.com>
> Signed-off-by: Namhyung Kim <namhyung@kernel.org>
> ---
> tools/perf/builtin-sched.c | 44 ++++++++++++++++++++++++++++++++++++++++++--
> 1 file changed, 42 insertions(+), 2 deletions(-)
>
> diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c
> index 1f8731640809..829468defa07 100644
> --- a/tools/perf/builtin-sched.c
> +++ b/tools/perf/builtin-sched.c
> @@ -201,6 +201,7 @@ struct perf_sched {
> bool summary_only;
> bool show_callchain;
> unsigned int max_stack;
> + bool show_cpu_visual;
> bool show_wakeups;
> u64 skipped_samples;
> };
> @@ -1783,10 +1784,23 @@ static char *timehist_get_commstr(struct thread *thread)
> return str;
> }
>
> -static void timehist_header(void)
> +static void timehist_header(struct perf_sched *sched)
> {
> + u32 ncpus = sched->max_cpu + 1;
> + u32 i, j;
> +
> printf("%15s %6s ", "time", "cpu");
>
> + if (sched->show_cpu_visual) {
> + printf(" ");
> + for (i = 0, j = 0; i < ncpus; ++i) {
> + printf("%x", j++);
> + if (j > 15)
> + j = 0;
> + }
> + printf(" ");
> + }
> +
> printf(" %-20s %9s %9s %9s",
> "task name", "wait time", "sch delay", "run time");
>
> @@ -1797,6 +1811,9 @@ static void timehist_header(void)
> */
> printf("%15s %-6s ", "", "");
>
> + if (sched->show_cpu_visual)
> + printf(" %*s ", ncpus, "");
> +
> printf(" %-20s %9s %9s %9s\n", "[tid/pid]", "(msec)", "(msec)", "(msec)");
>
> /*
> @@ -1804,6 +1821,9 @@ static void timehist_header(void)
> */
> printf("%.15s %.6s ", graph_dotted_line, graph_dotted_line);
>
> + if (sched->show_cpu_visual)
> + printf(" %.*s ", ncpus, graph_dotted_line);
> +
> printf(" %.20s %.9s %.9s %.9s",
> graph_dotted_line, graph_dotted_line, graph_dotted_line,
> graph_dotted_line);
> @@ -1817,11 +1837,28 @@ static void timehist_print_sample(struct perf_sched *sched,
> struct thread *thread)
> {
> struct thread_runtime *tr = thread__priv(thread);
> + u32 max_cpus = sched->max_cpu + 1;
> char tstr[64];
>
> timestamp__scnprintf_usec(sample->time, tstr, sizeof(tstr));
> printf("%15s [%04d] ", tstr, sample->cpu);
>
> + if (sched->show_cpu_visual) {
> + u32 i;
> + char c;
> +
> + printf(" ");
> + for (i = 0; i < max_cpus; ++i) {
> + /* flag idle times with 'i'; others are sched events */
> + if (i == sample->cpu)
> + c = (thread->tid == 0) ? 'i' : 's';
> + else
> + c = ' ';
> + printf("%c", c);
> + }
> + printf(" ");
> + }
> +
> printf(" %-*s ", comm_width, timehist_get_commstr(thread));
>
> print_sched_time(tr->dt_wait, 6);
> @@ -2095,6 +2132,8 @@ static void timehist_print_wakeup_event(struct perf_sched *sched,
>
> timestamp__scnprintf_usec(sample->time, tstr, sizeof(tstr));
> printf("%15s [%04d] ", tstr, sample->cpu);
> + if (sched->show_cpu_visual)
> + printf(" %*s ", sched->max_cpu + 1, "");
>
> printf(" %-*s ", comm_width, timehist_get_commstr(thread));
>
> @@ -2458,7 +2497,7 @@ static int perf_sched__timehist(struct perf_sched *sched)
> sched->summary = sched->summary_only;
>
> if (!sched->summary_only)
> - timehist_header();
> + timehist_header(sched);
>
> err = perf_session__process_events(session);
> if (err) {
> @@ -2842,6 +2881,7 @@ int cmd_sched(int argc, const char **argv, const char *prefix __maybe_unused)
> OPT_BOOLEAN('S', "with-summary", &sched.summary,
> "Show all syscalls and summary with statistics"),
> OPT_BOOLEAN('w', "wakeups", &sched.show_wakeups, "Show wakeup events"),
> + OPT_BOOLEAN('V', "cpu-visual", &sched.show_cpu_visual, "Add CPU visual"),
> OPT_PARENT(sched_options)
> };
>
> --
> 2.10.1
next prev parent reply other threads:[~2016-11-22 18:33 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-16 6:06 [PATCHSET 0/7] perf sched: Introduce timehist command, again (v2) Namhyung Kim
2016-11-16 6:06 ` [PATCH v2 1/8] perf symbol: Print symbol offsets conditionally Namhyung Kim
2016-11-24 4:18 ` [tip:perf/core] perf symbols: " tip-bot for Namhyung Kim
2016-11-16 6:06 ` [PATCH v2 2/8] perf tools: Support printing callchains with arrows Namhyung Kim
2016-11-24 4:18 ` [tip:perf/core] perf evsel: " tip-bot for Namhyung Kim
2016-11-16 6:06 ` [PATCH v2 3/8] perf sched timehist: Introduce timehist command Namhyung Kim
2016-11-24 4:19 ` [tip:perf/core] " tip-bot for David Ahern
2016-11-16 6:06 ` [PATCH v2 4/8] perf sched timehist: Add summary options Namhyung Kim
2016-11-24 4:19 ` [tip:perf/core] " tip-bot for David Ahern
2016-11-16 6:06 ` [PATCH v2 5/8] perf sched timehist: Add -w/--wakeups option Namhyung Kim
2016-11-24 4:20 ` [tip:perf/core] " tip-bot for David Ahern
2016-11-16 6:06 ` [PATCH v2 6/8] perf sched timehist: Add call graph options Namhyung Kim
2016-11-24 4:20 ` [tip:perf/core] " tip-bot for David Ahern
2016-11-16 6:06 ` [PATCH v2 7/8] perf sched timehist: Add -V/--cpu-visual option Namhyung Kim
2016-11-22 18:33 ` Arnaldo Carvalho de Melo [this message]
2016-11-23 5:34 ` Namhyung Kim
2016-11-23 13:36 ` Arnaldo Carvalho de Melo
2016-11-24 4:21 ` [tip:perf/core] " tip-bot for David Ahern
2016-11-16 6:06 ` [PATCH v2 8/8] perf sched: Add documentation for timehist options Namhyung Kim
2016-11-16 7:52 ` [PATCHSET 0/7] perf sched: Introduce timehist command, again (v2) Ingo Molnar
2016-11-16 15:34 ` Jiri Olsa
2016-11-22 6:59 ` Namhyung Kim
2016-11-22 7:00 ` Namhyung Kim
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20161122183326.GD5390@kernel.org \
--to=acme@kernel.org \
--cc=a.p.zijlstra@chello.nl \
--cc=andi@firstfloor.org \
--cc=dsahern@gmail.com \
--cc=eranian@google.com \
--cc=jolsa@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@kernel.org \
--cc=namhyung@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).