On 2013-11-17 03:12, Christoffer Dall wrote: > Initialize the cntvoff at kvm_init_vm time, not before running the > VCPUs > at the first time because that will overwrite any potentially > restored > values from user space. > > Cc: Marc Zynger <marc.zyngier@xxxxxxx> > Cc: Andre Przywara <andre.przywara@xxxxxxxxxx> > Signed-off-by: Christoffer Dall <christoffer.dall@xxxxxxxxxx> Acked-by: Marc Zyngier <marc.zyngier@xxxxxxx> > --- > > Changelog[v2]: > - Call kvm_timer_init from kvm_arch_init_vm instead of moving the > cntvoff to > vcpu initialization, which would be redundant. > > arch/arm/kvm/arm.c | 2 ++ > virt/kvm/arm/vgic.c | 1 - > 2 files changed, 2 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/kvm/arm.c b/arch/arm/kvm/arm.c > index e312e4a..2bc2ec4 100644 > --- a/arch/arm/kvm/arm.c > +++ b/arch/arm/kvm/arm.c > @@ -137,6 +137,8 @@ int kvm_arch_init_vm(struct kvm *kvm, unsigned > long type) > if (ret) > goto out_free_stage2_pgd; > > + kvm_timer_init(kvm); > + > /* Mark the initial VMID generation invalid */ > kvm->arch.vmid_gen = 0; > > diff --git a/virt/kvm/arm/vgic.c b/virt/kvm/arm/vgic.c > index 685fc72..81e9481 100644 > --- a/virt/kvm/arm/vgic.c > +++ b/virt/kvm/arm/vgic.c > @@ -1409,7 +1409,6 @@ int kvm_vgic_init(struct kvm *kvm) > for (i = VGIC_NR_PRIVATE_IRQS; i < VGIC_NR_IRQS; i += 4) > vgic_set_target_reg(kvm, 0, i); > > - kvm_timer_init(kvm); > kvm->arch.vgic.ready = true; > out: > mutex_unlock(&kvm->lock); -- Fast, cheap, reliable. Pick two. _______________________________________________ kvmarm mailing list kvmarm@xxxxxxxxxxxxxxxxxxxxx https://lists.cs.columbia.edu/cucslists/listinfo/kvmarm
