diff options
author | Paul E. McKenney <paul.mckenney@linaro.org> | 2011-06-21 04:48:03 -0400 |
---|---|---|
committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2011-09-29 00:38:10 -0400 |
commit | 9d68197c05201d8edc70d58bd1d5dad05d8455e8 (patch) | |
tree | c83d9b72ca28debc70fd1912b2e09867e439d87f | |
parent | 8008e129dc90ff4f7a56cb033d6bd042afe3ed52 (diff) |
rcu: Don't destroy rcu_torture_boost() callback until it is done
The rcu_torture_boost() cleanup code destroyed debug-objects state before
waiting for the last RCU callback to be invoked, resulting in rare but
very real debug-objects warnings. Move the destruction to after the
waiting to fix this problem.
Signed-off-by: Paul E. McKenney <paul.mckenney@linaro.org>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
-rw-r--r-- | kernel/rcutorture.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/rcutorture.c b/kernel/rcutorture.c index 75fca518888c..6a30ea3f1d82 100644 --- a/kernel/rcutorture.c +++ b/kernel/rcutorture.c | |||
@@ -796,11 +796,11 @@ checkwait: rcu_stutter_wait("rcu_torture_boost"); | |||
796 | 796 | ||
797 | /* Clean up and exit. */ | 797 | /* Clean up and exit. */ |
798 | VERBOSE_PRINTK_STRING("rcu_torture_boost task stopping"); | 798 | VERBOSE_PRINTK_STRING("rcu_torture_boost task stopping"); |
799 | destroy_rcu_head_on_stack(&rbi.rcu); | ||
800 | rcutorture_shutdown_absorb("rcu_torture_boost"); | 799 | rcutorture_shutdown_absorb("rcu_torture_boost"); |
801 | while (!kthread_should_stop() || rbi.inflight) | 800 | while (!kthread_should_stop() || rbi.inflight) |
802 | schedule_timeout_uninterruptible(1); | 801 | schedule_timeout_uninterruptible(1); |
803 | smp_mb(); /* order accesses to ->inflight before stack-frame death. */ | 802 | smp_mb(); /* order accesses to ->inflight before stack-frame death. */ |
803 | destroy_rcu_head_on_stack(&rbi.rcu); | ||
804 | return 0; | 804 | return 0; |
805 | } | 805 | } |
806 | 806 | ||