[v2,0/2] KVM: nSVM: few fixes for the nested migration

Message

Maxim Levitsky May 4, 2021, 2:39 p.m. UTC

Those are few fixes for issues I uncovered by doing variants of a
synthetic migration test I just created:

I modified the qemu, such that on each vm pause/resume cycle,
just prior to resuming a vCPU, qemu reads its KVM state,
then (optionaly) resets this state by uploading a
dummy reset state to KVM, and then it uploads back to KVM,
the state that this vCPU had before.

V2: those are only last 2 patches from V1,
updated with review feedback from Paolo (Thanks!).

Best regards,
	Maxim Levitsky

Maxim Levitsky (2):
  KVM: nSVM: always restore the L1's GIF on migration
  KVM: nSVM: remove a warning about vmcb01 VM exit reason

 arch/x86/kvm/svm/nested.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Paolo Bonzini May 4, 2021, 4:19 p.m. UTC | #1

On 04/05/21 16:39, Maxim Levitsky wrote:
> Those are few fixes for issues I uncovered by doing variants of a
> synthetic migration test I just created:
> 
> I modified the qemu, such that on each vm pause/resume cycle,
> just prior to resuming a vCPU, qemu reads its KVM state,
> then (optionaly) resets this state by uploading a
> dummy reset state to KVM, and then it uploads back to KVM,
> the state that this vCPU had before.
> 
> V2: those are only last 2 patches from V1,
> updated with review feedback from Paolo (Thanks!).

Queued, thanks.

Paolo

> Best regards,
> 	Maxim Levitsky
> 
> Maxim Levitsky (2):
>    KVM: nSVM: always restore the L1's GIF on migration
>    KVM: nSVM: remove a warning about vmcb01 VM exit reason
> 
>   arch/x86/kvm/svm/nested.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>