Skip to content

Commit d693bb0

Browse files
fweisbecgregkh
authored andcommitted
xtensa: Add missing RCU idle APIs on idle loop
commit 11ad47a upstream. In the old times, the whole idle task was considered as an RCU quiescent state. But as RCU became more and more successful overtime, some RCU read side critical section have been added even in the code of some architectures idle tasks, for tracing for example. So nowadays, rcu_idle_enter() and rcu_idle_exit() must be called by the architecture to tell RCU about the part in the idle loop that doesn't make use of rcu read side critical sections, typically the part that puts the CPU in low power mode. This is necessary for RCU to find the quiescent states in idle in order to complete grace periods. Add this missing pair of calls in the xtensa's idle loop. Reported-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com> Cc: Chris Zankel <chris@zankel.net> Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Reviewed-by: Josh Triplett <josh@joshtriplett.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent 88ea476 commit d693bb0

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

arch/xtensa/kernel/process.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
#include <linux/mqueue.h>
3232
#include <linux/fs.h>
3333
#include <linux/slab.h>
34+
#include <linux/rcupdate.h>
3435

3536
#include <asm/pgtable.h>
3637
#include <asm/uaccess.h>
@@ -110,8 +111,10 @@ void cpu_idle(void)
110111

111112
/* endless idle loop with no priority at all */
112113
while (1) {
114+
rcu_idle_enter();
113115
while (!need_resched())
114116
platform_idle();
117+
rcu_idle_exit();
115118
schedule_preempt_disabled();
116119
}
117120
}

0 commit comments

Comments
 (0)