diff mbox

[3/3] KVM: PPC: Book3S HV: Fix soft lockups in KVM on HMI for time base errors

Message ID 20160114031528.1287.7326.stgit@mars
State Superseded
Headers show

Commit Message

Mahesh J Salgaonkar Jan. 14, 2016, 3:15 a.m. UTC
From: Mahesh Salgaonkar <mahesh@linux.vnet.ibm.com>

When secondaries are napping in kvm_unsplit_nap() with hwthread_req = 1,
the HMI goes ignored even though subcores are already exited the guest.
Hence HMI keeps waking up secondaries from nap in a loop and secondaries
always go back to nap since no vcore is assigned to them. This makes
impossible for primary thread to get hold of secondary threads resulting
into a soft lockup in KVM path.

This patch fixes this by adding a HMI check just before the thread goes
to unsplit nap.

Signed-off-by: Mahesh Salgaonkar <mahesh@linux.vnet.ibm.com>
 arch/powerpc/kvm/book3s_hv_rmhandlers.S |    6 ++++++
 1 file changed, 6 insertions(+)

To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox


diff --git a/arch/powerpc/kvm/book3s_hv_rmhandlers.S b/arch/powerpc/kvm/book3s_hv_rmhandlers.S
index e8a456e..c5d43b9 100644
--- a/arch/powerpc/kvm/book3s_hv_rmhandlers.S
+++ b/arch/powerpc/kvm/book3s_hv_rmhandlers.S
@@ -440,6 +440,12 @@  kvm_no_guest:
  * whole-core mode, so we need to nap.
+	/* Before we head down to nap, check if HMI is pending and handle it */
+	bne	55f
+	li	r3, 0			/* NULL argument */
+	bl	hmi_exception_realmode
 	 * Ensure that secondary doesn't nap when it has
 	 * its vcore pointer set.