On Wed, Aug 4, 2021 at 3:17 AM Andrew Jones [off-list ref] wrote:

quoted

diff --git a/arch/arm64/kvm/arch_timer.c b/arch/arm64/kvm/arch_timer.c
index a8815b09da3e..f15058612994 100644
--- a/arch/arm64/kvm/arch_timer.c
+++ b/arch/arm64/kvm/arch_timer.c

@@ -85,11 +85,15 @@ u64 timer_get_cval(struct arch_timer_context *ctxt)
 static u64 timer_get_offset(struct arch_timer_context *ctxt)
 {
      struct kvm_vcpu *vcpu = ctxt->vcpu;
+     struct arch_timer_cpu *timer = vcpu_timer(vcpu);

This new timer variable doesn't appear to get used.

Ooops, this is stale. Thanks for catching that.

Reviewed-by: Andrew Jones <redacted>

Thanks for the quick review!

--
Oliver