diff options
| author | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2018-05-14 16:52:27 -0400 | 
|---|---|---|
| committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2018-07-12 18:39:15 -0400 | 
| commit | 6f56f714db067056c80f5d71510118f82872e34c (patch) | |
| tree | 831ac61392c65e088ed1ae8856b5f42de30fcd8d /kernel/rcu/tree.c | |
| parent | a7538352da722fae5cc95ae6656ea2013f5b8b21 (diff) | |
rcu: Improve RCU-tasks naming and comments
The naming and comments associated with some RCU-tasks code make
the faulty assumption that context switches due to cond_resched()
are voluntary.  As several people pointed out, this is not the case.
This commit therefore updates function names and comments to better
reflect current reality.
Reported-by: Byungchul Park <byungchul.park@lge.com>
Reported-by: Joel Fernandes <joel@joelfernandes.org>
Reported-by: Steven Rostedt <rostedt@goodmis.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 6f2922168216..ccc061acf887 100644 --- a/kernel/rcu/tree.c +++ b/kernel/rcu/tree.c  | |||
| @@ -457,7 +457,7 @@ void rcu_note_context_switch(bool preempt) | |||
| 457 | rcu_momentary_dyntick_idle(); | 457 | rcu_momentary_dyntick_idle(); | 
| 458 | this_cpu_inc(rcu_dynticks.rcu_qs_ctr); | 458 | this_cpu_inc(rcu_dynticks.rcu_qs_ctr); | 
| 459 | if (!preempt) | 459 | if (!preempt) | 
| 460 | rcu_note_voluntary_context_switch_lite(current); | 460 | rcu_tasks_qs(current); | 
| 461 | out: | 461 | out: | 
| 462 | trace_rcu_utilization(TPS("End context switch")); | 462 | trace_rcu_utilization(TPS("End context switch")); | 
| 463 | barrier(); /* Avoid RCU read-side critical sections leaking up. */ | 463 | barrier(); /* Avoid RCU read-side critical sections leaking up. */ | 
