summaryrefslogtreecommitdiffstats
path: root/tools/perf
diff options
context:
space:
mode:
authorFrederic Weisbecker <fweisbec@gmail.com>2010-05-06 04:55:22 +0200
committerFrederic Weisbecker <fweisbec@gmail.com>2010-05-09 13:45:29 +0200
commit5efe08cf685f33f562566dc68b6077b6f6a4f706 (patch)
treeae792d0ba092caed622825a39fec84eb6c67174c /tools/perf
parent84c7a21791eb2e962a27e19bab5b77d5d9e13a34 (diff)
downloadop-kernel-dev-5efe08cf685f33f562566dc68b6077b6f6a4f706.zip
op-kernel-dev-5efe08cf685f33f562566dc68b6077b6f6a4f706.tar.gz
perf: Fix perf lock bad rate
Fix the cast made to get the bad rate. It is made in the result instead of the operands. We need the operands to be cast in double, otherwise the result will always be zero. Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com> Cc: Ingo Molnar <mingo@elte.hu> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Cc: Arnaldo Carvalho de Melo <acme@redhat.com> Cc: Paul Mackerras <paulus@samba.org> Cc: Hitoshi Mitake <mitake@dcl.info.waseda.ac.jp>
Diffstat (limited to 'tools/perf')
-rw-r--r--tools/perf/builtin-lock.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/perf/builtin-lock.c b/tools/perf/builtin-lock.c
index 3b304ed..d7dde9c 100644
--- a/tools/perf/builtin-lock.c
+++ b/tools/perf/builtin-lock.c
@@ -733,8 +733,8 @@ static void print_bad_events(int bad, int total)
{ "acquire", "acquired", "contended", "release" };
pr_info("\n=== output for debug===\n\n");
- pr_info("bad:%d, total:%d\n", bad, total);
- pr_info("bad rate:%f\n", (double)(bad / total));
+ pr_info("bad: %d, total: %d\n", bad, total);
+ pr_info("bad rate: %f %%\n", (double)bad / (double)total * 100);
pr_info("histogram of events caused bad sequence\n");
for (i = 0; i < BROKEN_MAX; i++)
pr_info(" %10s: %d\n", name[i], bad_hist[i]);
OpenPOWER on IntegriCloud