LinuxPPC-Dev Archive on
 help / color / Atom feed
* [PATCH] powerpc/pseries: correctly track irq state in default idle
@ 2019-09-10 22:52 Nathan Lynch
  2019-09-19 10:25 ` Michael Ellerman
  0 siblings, 1 reply; 2+ messages in thread
From: Nathan Lynch @ 2019-09-10 22:52 UTC (permalink / raw)
  To: linuxppc-dev; +Cc: ego, npiggin

prep_irq_for_idle() is intended to be called before entering
H_CEDE (and it is used by the pseries cpuidle driver). However the
default pseries idle routine does not call it, leading to mismanaged
lazy irq state when the cpuidle driver isn't in use. Manifestations of
this include:

* Dropped IPIs in the time immediately after a cpu comes
  online (before it has installed the cpuidle handler), making the
  online operation block indefinitely waiting for the new cpu to

* Hitting this WARN_ON in arch_local_irq_restore():
	 * We should already be hard disabled here. We had bugs
	 * where that wasn't the case so let's dbl check it and
	 * warn if we are wrong. Only do that when IRQ tracing
	 * is enabled as mfmsr() can be costly.
	if (WARN_ON_ONCE(mfmsr() & MSR_EE))

Call prep_irq_for_idle() from pseries_lpar_idle() and honor its

Fixes: 363edbe2614a ("powerpc: Default arch idle could cede processor on pseries")
Signed-off-by: Nathan Lynch <>
 arch/powerpc/platforms/pseries/setup.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/powerpc/platforms/pseries/setup.c b/arch/powerpc/platforms/pseries/setup.c
index b955d54628ff..f8adcd0e4589 100644
--- a/arch/powerpc/platforms/pseries/setup.c
+++ b/arch/powerpc/platforms/pseries/setup.c
@@ -321,6 +321,9 @@ static void pseries_lpar_idle(void)
 	 * low power mode by ceding processor to hypervisor
+	if (!prep_irq_for_idle())
+		return;
 	/* Indicate to hypervisor that we are idle. */
 	get_lppaca()->idle = 1;

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, back to index

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-10 22:52 [PATCH] powerpc/pseries: correctly track irq state in default idle Nathan Lynch
2019-09-19 10:25 ` Michael Ellerman

LinuxPPC-Dev Archive on

Archives are clonable:
	git clone --mirror linuxppc-dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linuxppc-dev linuxppc-dev/ \
	public-inbox-index linuxppc-dev

Example config snippet for mirrors

Newsgroup available over NNTP:

AGPL code for this site: git clone public-inbox