Re: [PATCH] perf sched latency: prettify printed table
On Sat, Mar 15, 2014 at 12:17:38PM -0400, Ramkumar Ramachandra wrote: > Cc: Frederic Weisbecker > Cc: David Ahern > Cc: Jiri Olsa > Cc: Arnaldo Carvalho de Melo > Signed-off-by: Ramkumar Ramachandra > --- > tools/perf/builtin-sched.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) ENOCHANGELOG ;-) please provide changelog details how's the output prettified (before/after example) thanks, jirka -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] perf sched latency: prettify printed table
On Sat, Mar 15, 2014 at 12:17:38PM -0400, Ramkumar Ramachandra wrote: Cc: Frederic Weisbecker fweis...@gmail.com Cc: David Ahern dsah...@gmail.com Cc: Jiri Olsa jo...@redhat.com Cc: Arnaldo Carvalho de Melo a...@redhat.com Signed-off-by: Ramkumar Ramachandra artag...@gmail.com --- tools/perf/builtin-sched.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) ENOCHANGELOG ;-) please provide changelog details how's the output prettified (before/after example) thanks, jirka -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] perf sched latency: prettify printed table
Cc: Frederic Weisbecker
Cc: David Ahern
Cc: Jiri Olsa
Cc: Arnaldo Carvalho de Melo
Signed-off-by: Ramkumar Ramachandra
---
tools/perf/builtin-sched.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c
index 6a76a07..0c41a2b 100644
--- a/tools/perf/builtin-sched.c
+++ b/tools/perf/builtin-sched.c
@@ -1124,7 +1124,7 @@ static void output_lat_thread(struct perf_sched *sched,
struct work_atoms *work_
avg = work_list->total_lat / work_list->nb_atoms;
- printf("|%11.3f ms |%9" PRIu64 " | avg:%9.3f ms | max:%9.3f ms | max
at: %9.6f s\n",
+ printf("|%11.3f ms |%9" PRIu64 " | avg:%9.3f ms | max:%9.3f ms | max
at: %12.6f s\n",
(double)work_list->total_runtime / 1e6,
work_list->nb_atoms, (double)avg / 1e6,
(double)work_list->max_lat / 1e6,
@@ -1527,9 +1527,9 @@ static int perf_sched__lat(struct perf_sched *sched)
perf_sched__sort_lat(sched);
- printf("\n
---\n");
- printf(" Task | Runtime ms | Switches | Average
delay ms | Maximum delay ms | Maximum delay at |\n");
- printf("
---\n");
+ printf("\n
-\n");
+ printf(" Task | Runtime ms | Switches | Average
delay ms | Maximum delay ms | Maximum delay at |\n");
+ printf("
-\n");
next = rb_first(>sorted_atom_root);
@@ -1541,7 +1541,7 @@ static int perf_sched__lat(struct perf_sched *sched)
next = rb_next(next);
}
- printf("
-\n");
+ printf("
-\n");
printf(" TOTAL:|%11.3f ms |%9" PRIu64 " |\n",
(double)sched->all_runtime / 1e6, sched->all_count);
--
1.9.rc0.1.g9d22d25
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[PATCH] perf sched latency: prettify printed table
Cc: Frederic Weisbecker fweis...@gmail.com Cc: David Ahern dsah...@gmail.com Cc: Jiri Olsa jo...@redhat.com Cc: Arnaldo Carvalho de Melo a...@redhat.com Signed-off-by: Ramkumar Ramachandra artag...@gmail.com --- tools/perf/builtin-sched.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c index 6a76a07..0c41a2b 100644 --- a/tools/perf/builtin-sched.c +++ b/tools/perf/builtin-sched.c @@ -1124,7 +1124,7 @@ static void output_lat_thread(struct perf_sched *sched, struct work_atoms *work_ avg = work_list-total_lat / work_list-nb_atoms; - printf(|%11.3f ms |%9 PRIu64 | avg:%9.3f ms | max:%9.3f ms | max at: %9.6f s\n, + printf(|%11.3f ms |%9 PRIu64 | avg:%9.3f ms | max:%9.3f ms | max at: %12.6f s\n, (double)work_list-total_runtime / 1e6, work_list-nb_atoms, (double)avg / 1e6, (double)work_list-max_lat / 1e6, @@ -1527,9 +1527,9 @@ static int perf_sched__lat(struct perf_sched *sched) perf_sched__sort_lat(sched); - printf(\n ---\n); - printf( Task | Runtime ms | Switches | Average delay ms | Maximum delay ms | Maximum delay at |\n); - printf( ---\n); + printf(\n -\n); + printf( Task | Runtime ms | Switches | Average delay ms | Maximum delay ms | Maximum delay at |\n); + printf( -\n); next = rb_first(sched-sorted_atom_root); @@ -1541,7 +1541,7 @@ static int perf_sched__lat(struct perf_sched *sched) next = rb_next(next); } - printf( -\n); + printf( -\n); printf( TOTAL:|%11.3f ms |%9 PRIu64 |\n, (double)sched-all_runtime / 1e6, sched-all_count); -- 1.9.rc0.1.g9d22d25 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
