Commit 5e26aa93 authored by Peter Zijlstra's avatar Peter Zijlstra Committed by Ingo Molnar
Browse files

cpuidle/poll: Ensure IRQs stay disabled after cpuidle_state::enter() calls 



Make cpuidle_state::enter() methods IRQ state invariant on exit.

Additionally make sure to use raw_local_irq_*() methods since this
cpuidle callback will be called with RCU already disabled.

Signed-off-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
Signed-off-by: default avatarIngo Molnar <mingo@kernel.org>
Tested-by: default avatarTony Lindgren <tony@atomide.com>
Tested-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
Reviewed-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
Reviewed-by: default avatarFrederic Weisbecker <frederic@kernel.org>
Link: https://lore.kernel.org/r/20230112195539.515253662@infradead.org
parent aaa3896b

drivers/cpuidle/poll_state.c

+3 −1
Original line number Diff line number Diff line
@@ -17,7 +17,7 @@ static int __cpuidle poll_idle(struct cpuidle_device *dev, 


	dev->poll_time_limit = false;



	local_irq_enable();

	raw_local_irq_enable();

	if (!current_set_polling_and_test()) {

		unsigned int loop_count = 0;

		u64 limit;
@@ -36,6 +36,8 @@ static int __cpuidle poll_idle(struct cpuidle_device *dev, 
			}

		}

	}

	raw_local_irq_disable();



	current_clr_polling();



	return index;