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
