Messages in this thread | | | From | Peter Gonda <> | Date | Wed, 17 Nov 2021 10:47:32 -0700 | Subject | Re: [PATCH 3/4] KVM: SEV: cleanup locking for KVM_CAP_VM_MOVE_ENC_CONTEXT_FROM |
| |
On Wed, Nov 17, 2021 at 9:38 AM Paolo Bonzini <pbonzini@redhat.com> wrote: > > Encapsulate the handling of the migration_in_progress flag for both VMs in > two functions sev_lock_two_vms and sev_unlock_two_vms. It does not matter > if KVM_CAP_VM_MOVE_ENC_CONTEXT_FROM locks the source struct kvm a bit > earlier, and this change 1) keeps the cleanup chain of labels smaller 2) > makes it possible for KVM_CAP_VM_COPY_ENC_CONTEXT_FROM to reuse the logic. > > Cc: Peter Gonda <pgonda@google.com> > Cc: Sean Christopherson <seanjc@google.com> > Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Acked-by: Peter Gonda <pgonda@google.com>
I like the cleanup thanks Paolo.
| |