Re: [PATCH v2 2/4] KVM: nSVM: correctly restore nested_run_pending on migration

2021-01-07 Thread Paolo Bonzini
On 07/01/21 21:19, Sean Christopherson wrote: That said, the v1 change had some appeal to it. Which v1 change are you referring to? Moving the to-be-injected event from eventinj to vcpu->arch, and from there to vmcb02 on the next vmentry's inject_pending_event. In the VMX case (if

Re: [PATCH v2 2/4] KVM: nSVM: correctly restore nested_run_pending on migration

2021-01-07 Thread Sean Christopherson
On Thu, Jan 07, 2021, Paolo Bonzini wrote: > On 07/01/21 10:38, Maxim Levitsky wrote: > > The code to store it on the migration exists, but no code was restoring it. > > > > One of the side effects of fixing this is that L1->L2 injected events > > are no longer lost when migration happens with

Re: [PATCH v2 2/4] KVM: nSVM: correctly restore nested_run_pending on migration

2021-01-07 Thread Paolo Bonzini
On 07/01/21 10:38, Maxim Levitsky wrote: The code to store it on the migration exists, but no code was restoring it. One of the side effects of fixing this is that L1->L2 injected events are no longer lost when migration happens with nested run pending. Signed-off-by: Maxim Levitsky ---

[PATCH v2 2/4] KVM: nSVM: correctly restore nested_run_pending on migration

2021-01-07 Thread Maxim Levitsky
The code to store it on the migration exists, but no code was restoring it. One of the side effects of fixing this is that L1->L2 injected events are no longer lost when migration happens with nested run pending. Signed-off-by: Maxim Levitsky --- arch/x86/kvm/svm/nested.c | 4 1 file