sched/fair: Add READ_ONCE() and use existing helper function to access ->avg_irq
authorShrikanth Hegde <sshegde@linux.vnet.ibm.com>
Mon, 1 Jan 2024 15:46:24 +0000 (21:16 +0530)
committerIngo Molnar <mingo@kernel.org>
Wed, 28 Feb 2024 14:11:15 +0000 (15:11 +0100)
commita6965b31888501f889261a6783f0de6afff84f8d
treee53367e0254a49d64b0f9828e01c28d14f829e56
parent8b936fc1d84f7d70009ea34d66bbf6c54c09fae7
sched/fair: Add READ_ONCE() and use existing helper function to access ->avg_irq

Use existing helper function cpu_util_irq() instead of open-coding
access to ->avg_irq.

During review it was noted that ->avg_irq could be updated by a
different CPU than the one which is trying to access it.

->avg_irq is updated with WRITE_ONCE(), use READ_ONCE to access it
in order to avoid any compiler optimizations.

Signed-off-by: Shrikanth Hegde <sshegde@linux.vnet.ibm.com>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Reviewed-by: Vincent Guittot <vincent.guittot@linaro.org>
Link: https://lore.kernel.org/r/20240101154624.100981-3-sshegde@linux.vnet.ibm.com
kernel/sched/fair.c
kernel/sched/sched.h