diff options
| author | Paul E. McKenney <[email protected]> | 2020-01-03 19:42:05 +0000 |
|---|---|---|
| committer | Paul E. McKenney <[email protected]> | 2020-02-21 00:01:11 +0000 |
| commit | 710426068dc60f2d2e139478d6185710802cdc0a (patch) | |
| tree | 00005fee34c298dea42e655f5e06a42392cba95d /tools/perf/scripts/python/event_analyzing_sample.py | |
| parent | srcu: Fix process_srcu()/srcu_batches_completed() datarace (diff) | |
| download | kernel-710426068dc60f2d2e139478d6185710802cdc0a.tar.gz kernel-710426068dc60f2d2e139478d6185710802cdc0a.zip | |
srcu: Hold srcu_struct ->lock when updating ->srcu_gp_seq
A read of the srcu_struct structure's ->srcu_gp_seq field should not
need READ_ONCE() when that structure's ->lock is held. Except that this
lock is not always held when updating this field. This commit therefore
acquires the lock around updates and removes a now-unneeded READ_ONCE().
This data race was reported by KCSAN.
Signed-off-by: Paul E. McKenney <[email protected]>
[ paulmck: Switch from READ_ONCE() to lock per Peter Zilstra question. ]
Acked-by: Peter Zijlstra (Intel) <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/event_analyzing_sample.py')
0 files changed, 0 insertions, 0 deletions