diff options
author | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2015-07-02 15:27:31 -0400 |
---|---|---|
committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2015-07-17 17:59:05 -0400 |
commit | 32bb1c79996069ef9e4e53b428050749f9841c3f (patch) | |
tree | 6e3d28b5807d8f77f215d27b0a59f25809bc975a /kernel/rcu/tree.c | |
parent | b9a425cfcb3c473b4ca2f3dfaeaf13848f4a7976 (diff) |
rcu: Rename RCU_GP_DONE_FQS to RCU_GP_DOING_FQS
The grace-period kthread sleeps waiting to do a force-quiescent-state
scan, and when awakened sets rsp->gp_state to RCU_GP_DONE_FQS.
However, this is confusing because the kthread has not done the
force-quiescent-state, but is instead just starting to do it. This commit
therefore renames RCU_GP_DONE_FQS to RCU_GP_DOING_FQS in order to make
things a bit easier on reviewers.
Reported-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Diffstat (limited to 'kernel/rcu/tree.c')
-rw-r--r-- | kernel/rcu/tree.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/rcu/tree.c b/kernel/rcu/tree.c index b2803730ac13..f66f6e7730bc 100644 --- a/kernel/rcu/tree.c +++ b/kernel/rcu/tree.c | |||
@@ -2088,7 +2088,7 @@ static int __noreturn rcu_gp_kthread(void *arg) | |||
2088 | rsp->gp_state = RCU_GP_WAIT_FQS; | 2088 | rsp->gp_state = RCU_GP_WAIT_FQS; |
2089 | ret = wait_event_interruptible_timeout(rsp->gp_wq, | 2089 | ret = wait_event_interruptible_timeout(rsp->gp_wq, |
2090 | rcu_gp_fqs_check_wake(rsp, &gf), j); | 2090 | rcu_gp_fqs_check_wake(rsp, &gf), j); |
2091 | rsp->gp_state = RCU_GP_DONE_FQS; | 2091 | rsp->gp_state = RCU_GP_DOING_FQS; |
2092 | /* Locking provides needed memory barriers. */ | 2092 | /* Locking provides needed memory barriers. */ |
2093 | /* If grace period done, leave loop. */ | 2093 | /* If grace period done, leave loop. */ |
2094 | if (!READ_ONCE(rnp->qsmask) && | 2094 | if (!READ_ONCE(rnp->qsmask) && |