Re: [PATCH v3 2/2] KVM: x86/xen: Stop Xen timer before changing IRQ

From: David Woodhouse
Date: Tue Aug 09 2022 - 10:52:58 EST


On Tue, 2022-08-09 at 14:40 +0000, Sean Christopherson wrote:
> I wanted to separate the "init" from the "stop+start", e.g. if there were a more
> appropriate place for invoking kvm_xen_init_timer() I would have suggested moving
> the call outside of KVM_XEN_VCPU_ATTR_TYPE_TIMER entirely.

Yeah, there's nowhere sensible that would apply to only Xen guests. I
looked at kvm_xen_init_vcpu() but that's unconditional.

I do note that we're now calling kvm_xen_init_timer() even when an
*unset* timer is being restored after live update/live migration. But I
think that's OK.

Attachment: smime.p7s
Description: S/MIME cryptographic signature