diff options
author | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2014-03-03 19:51:08 -0500 |
---|---|---|
committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2014-05-14 12:45:58 -0400 |
commit | 589a8f59509dc4c68f9c1e2522c5b0b556009221 (patch) | |
tree | bfd54dddc6ffaf7500b73c7f46a192d0e87634bf /kernel/rcu/rcutorture.c | |
parent | 14d9d84c9e9b54e02ea69ab5cb6e2114f3639160 (diff) |
rcutorture: Print negatives for SRCU counter wraparound
The srcu_torture_stats() function prints SRCU's per-CPU c[] array with
an unsigned format, which means that the number one less than zero is
a very large number. This commit therefore prints this array with a
signed format in order to improve readability of the rcutorture output.
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Reviewed-by: Josh Triplett <josh@joshtriplett.org>
Diffstat (limited to 'kernel/rcu/rcutorture.c')
-rw-r--r-- | kernel/rcu/rcutorture.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/kernel/rcu/rcutorture.c b/kernel/rcu/rcutorture.c index 0bc5f0a4c1ab..80d2d2440210 100644 --- a/kernel/rcu/rcutorture.c +++ b/kernel/rcu/rcutorture.c | |||
@@ -492,9 +492,11 @@ static void srcu_torture_stats(char *page) | |||
492 | page += sprintf(page, "%s%s per-CPU(idx=%d):", | 492 | page += sprintf(page, "%s%s per-CPU(idx=%d):", |
493 | torture_type, TORTURE_FLAG, idx); | 493 | torture_type, TORTURE_FLAG, idx); |
494 | for_each_possible_cpu(cpu) { | 494 | for_each_possible_cpu(cpu) { |
495 | page += sprintf(page, " %d(%lu,%lu)", cpu, | 495 | long c0, c1; |
496 | per_cpu_ptr(srcu_ctl.per_cpu_ref, cpu)->c[!idx], | 496 | |
497 | per_cpu_ptr(srcu_ctl.per_cpu_ref, cpu)->c[idx]); | 497 | c0 = (long)per_cpu_ptr(srcu_ctl.per_cpu_ref, cpu)->c[!idx]; |
498 | c1 = (long)per_cpu_ptr(srcu_ctl.per_cpu_ref, cpu)->c[idx]; | ||
499 | page += sprintf(page, " %d(%ld,%ld)", cpu, c0, c1); | ||
498 | } | 500 | } |
499 | sprintf(page, "\n"); | 501 | sprintf(page, "\n"); |
500 | } | 502 | } |