diff options
| author | Thomas Gleixner <[email protected]> | 2020-11-17 13:19:43 +0000 |
|---|---|---|
| committer | Thomas Gleixner <[email protected]> | 2020-11-19 09:48:28 +0000 |
| commit | f73f64d5687192bc8eb7f3d9521ca6256b79f224 (patch) | |
| tree | 9697fa0581b26217f309095d33fe7cd2a82a0f9e /fs/proc/array.c | |
| parent | hrtimer: Fix kernel-doc markups (diff) | |
| download | kernel-f73f64d5687192bc8eb7f3d9521ca6256b79f224.tar.gz kernel-f73f64d5687192bc8eb7f3d9521ca6256b79f224.zip | |
tick/broadcast: Serialize access to tick_next_period
tick_broadcast_setup_oneshot() accesses tick_next_period twice without any
serialization. This is wrong in two aspects:
- Reading it twice might make the broadcast data inconsistent if the
variable is updated concurrently.
- On 32bit systems the access might see an partial update
Protect it with jiffies_lock. That's safe as none of the callchains leading
up to this function can create a lock ordering violation:
timer interrupt
run_local_timers()
hrtimer_run_queues()
hrtimer_switch_to_hres()
tick_init_highres()
tick_switch_to_oneshot()
tick_broadcast_switch_to_oneshot()
or
tick_check_oneshot_change()
tick_nohz_switch_to_nohz()
tick_switch_to_oneshot()
tick_broadcast_switch_to_oneshot()
Signed-off-by: Thomas Gleixner <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Diffstat (limited to 'fs/proc/array.c')
0 files changed, 0 insertions, 0 deletions
