[v6] HVF: Add support for platform vGIC and nested virtualisation

[PATCH v6 09/13] hvf: only call hvf_sync_vtimer() when running without the platform vGIC

Mohamed Mediouni posted 13 patches 4 months, 1 week ago

Diff against v1 v2 v3 v4 v5
Download mbox

Maintainers: Cameron Esfahani <dirty@apple.com>, Roman Bolshakov <rbolshakov@ddn.com>, Phil Dennis-Jordan <phil@philjordan.eu>, Mads Ynddal <mads@ynddal.dk>, "Michael S. Tsirkin" <mst@redhat.com>, Igor Mammedov <imammedo@redhat.com>, Ani Sinha <anisinha@redhat.com>, Peter Maydell <peter.maydell@linaro.org>, Shannon Zhao <shannon.zhaosl@gmail.com>, Paolo Bonzini <pbonzini@redhat.com>, Alexander Graf <agraf@csgraf.de>

Expand all Fold all

[PATCH v6 09/13] hvf: only call hvf_sync_vtimer() when running without the platform vGIC

Posted by Mohamed Mediouni 4 months, 1 week ago

When running with the Apple vGIC, the EL1 vtimer is handled by the platform.

Signed-off-by: Mohamed Mediouni <mohamed@unpredictable.fr>
---
 target/arm/hvf/hvf.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/target/arm/hvf/hvf.c b/target/arm/hvf/hvf.c
index 788d0d41f4..d8741f942d 100644
--- a/target/arm/hvf/hvf.c
+++ b/target/arm/hvf/hvf.c
@@ -2008,7 +2008,8 @@ int hvf_vcpu_exec(CPUState *cpu)
         g_assert_not_reached();
     }
 
-    hvf_sync_vtimer(cpu);
+    if (!hvf_irqchip_in_kernel())
+        hvf_sync_vtimer(cpu);
 
     switch (ec) {
     case EC_SOFTWARESTEP: {
-- 
2.39.5 (Apple Git-154)

Re: [PATCH v6 09/13] hvf: only call hvf_sync_vtimer() when running without the platform vGIC

Posted by Mads Ynddal 3 months, 3 weeks ago

> On 8 Aug 2025, at 09.01, Mohamed Mediouni <mohamed@unpredictable.fr> wrote:
> 
> When running with the Apple vGIC, the EL1 vtimer is handled by the platform.
> 
> Signed-off-by: Mohamed Mediouni <mohamed@unpredictable.fr>
> ---
> target/arm/hvf/hvf.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
> 

Reviewed-by: Mads Ynddal <mads@ynddal.dk>