[PATCH v10 53/59] KVM: arm64: nv: Publish emulated timer interrupt state in the in-memory state

From: Marc Zyngier <maz@kernel•org>
To: kvmarm@lists•linux.dev, kvm@vger•kernel.org,
	linux-arm-kernel@lists•infradead.org
Cc: Alexandru Elisei <alexandru.elisei@arm•com>,
	Andre Przywara <andre.przywara@arm•com>,
	Chase Conklin <chase.conklin@arm•com>,
	Christoffer Dall <christoffer.dall@arm•com>,
	Ganapatrao Kulkarni <gankulkarni@os•amperecomputing.com>,
	Darren Hart <darren@os•amperecomputing.com>,
	Jintack Lim <jintack@cs•columbia.edu>,
	Russell King <rmk+kernel@armlinux•org.uk>,
	Miguel Luis <miguel.luis@oracle•com>,
	James Morse <james.morse@arm•com>,
	Suzuki K Poulose <suzuki.poulose@arm•com>,
	Oliver Upton <oliver.upton@linux•dev>,
	Zenghui Yu <yuzenghui@huawei•com>
Subject: [PATCH v10 53/59] KVM: arm64: nv: Publish emulated timer interrupt state in the in-memory state
Date: Mon, 15 May 2023 18:30:57 +0100	[thread overview]
Message-ID: <20230515173103.1017669-54-maz@kernel.org> (raw)
In-Reply-To: <20230515173103.1017669-1-maz@kernel.org>

With FEAT_NV2, the EL0 timer state is entirely stored in memory,
meaning that the hypervisor can only provide a very poor emulation.

The only thing we can really do is to publish the interrupt state
in the guest view of CNT{P,V}_CTL_EL0, and defer everything else
to the next exit.

Only FEAT_ECV will allow us to fix it, at the cost of extra trapping.

Suggested-by: Chase Conklin <chase.conklin@arm•com>
Suggested-by: Ganapatrao Kulkarni <gankulkarni@os•amperecomputing.com>
Signed-off-by: Marc Zyngier <maz@kernel•org>
---
 arch/arm64/kvm/arch_timer.c | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/arch/arm64/kvm/arch_timer.c b/arch/arm64/kvm/arch_timer.c
index 294ab5013e40..f551aa4ac321 100644
--- a/arch/arm64/kvm/arch_timer.c
+++ b/arch/arm64/kvm/arch_timer.c
@@ -453,6 +453,25 @@ static void kvm_timer_update_irq(struct kvm_vcpu *vcpu, bool new_level,
 {
 	int ret;
 
+	/*
+	 * Paper over NV2 brokenness by publishing the interrupt status
+	 * bit. This still results in a poor quality of emulation (guest
+	 * writes will have no effect until the next exit).
+	 *
+	 * But hey, it's fast, right?
+	 */
+	if (vcpu_has_nv2(vcpu) && is_hyp_ctxt(vcpu) &&
+	    (timer_ctx == vcpu_vtimer(vcpu) || timer_ctx == vcpu_ptimer(vcpu))) {
+		u32 ctl = timer_get_ctl(timer_ctx);
+
+		if (new_level)
+			ctl |= ARCH_TIMER_CTRL_IT_STAT;
+		else
+			ctl &= ~ARCH_TIMER_CTRL_IT_STAT;
+
+		timer_set_ctl(timer_ctx, ctl);
+	}
+
 	timer_ctx->irq.level = new_level;
 	trace_kvm_timer_update_irq(vcpu->vcpu_id, timer_irq(timer_ctx),
 				   timer_ctx->irq.level);
-- 
2.34.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists•infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
