diff options
| author | Joel Fernandes (Google) <joel@joelfernandes.org> | 2018-10-06 01:18:11 +0200 |
|---|---|---|
| committer | Paul E. McKenney <paulmck@linux.ibm.com> | 2018-11-12 17:56:25 +0100 |
| commit | 090c1685fd628a8c191d77b5267a7dc226246a5b (patch) | |
| tree | 922090d60ed455677ab938eba1f8f1358cae3a4d /Documentation/RCU | |
| parent | doc: rcu: Update core and full API in whatisRCU (diff) | |
| download | linux-090c1685fd628a8c191d77b5267a7dc226246a5b.tar.xz linux-090c1685fd628a8c191d77b5267a7dc226246a5b.zip | |
doc: rcu: Add more rationale for using rcu_read_lock_sched in checklist
This commit explains why rcu_read_lock_sched is better than using
preempt_disable.
Signed-off-by: Joel Fernandes (Google) <joel@joelfernandes.org>
Signed-off-by: Paul E. McKenney <paulmck@linux.ibm.com>
Diffstat (limited to 'Documentation/RCU')
| -rw-r--r-- | Documentation/RCU/checklist.txt | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Documentation/RCU/checklist.txt b/Documentation/RCU/checklist.txt index 49747717d905..8860ab2a897a 100644 --- a/Documentation/RCU/checklist.txt +++ b/Documentation/RCU/checklist.txt @@ -63,7 +63,7 @@ over a rather long period of time, but improvements are always welcome! pointer must be covered by rcu_read_lock(), rcu_read_lock_bh(), rcu_read_lock_sched(), or by the appropriate update-side lock. Disabling of preemption can serve as rcu_read_lock_sched(), but - is less readable. + is less readable and prevents lockdep from detecting locking issues. Letting RCU-protected pointers "leak" out of an RCU read-side critical section is every bid as bad as letting them leak out |