Re: [PATCH] KVM: SEV: Disable KVM_CAP_VM_COPY_ENC_CONTEXT_FROM for SEV-ES

From: Nathan Tempelman
Date: Thu Sep 16 2021 - 15:06:37 EST


On Thu, Sep 16, 2021 at 11:08 AM Nathan Tempelman <natet@xxxxxxxxxx> wrote:
>
> On Wed, Sep 15, 2021 at 3:33 PM Paolo Bonzini <pbonzini@xxxxxxxxxx> wrote:
> >
> > On 15/09/21 18:10, Peter Gonda wrote:
> > > svm_vm_copy_asid_from() {
> > >
> > > asid = to_kvm_svm(source_kvm)->sev_info.asid;
> > > + handle = to_kvm_svm(source_kvm)->sev_info.handle;
> > > + fd = to_kvm_svm(source_kvm)->sev_info.fd;
> > > + es_active = to_kvm_svm(source_kvm)->sev_info.es_active;
> > >
> > > ...
> > >
> > > /* Set enc_context_owner and copy its encryption context over */
> > > mirror_sev = &to_kvm_svm(kvm)->sev_info;
> > > mirror_sev->enc_context_owner = source_kvm;
> > > mirror_sev->asid = asid;
> > > mirror_sev->active = true;
> > > + mirror_sev->handle = handle;
> > > + mirror_sev->fd = fd;
> > > + mirror_sev->es_active = es_active;
> > >
> > > Paolo would you prefer a patch to enable ES mirroring or continue with
> > > this patch to disable it for now?
> >
> > If it's possible to enable it, it would be better. The above would be a
> > reasonable patch for 5.15-rc.
> >
> > Paolo
> >
>
> +1. We don't have any immediate plans for sev-es, but it would be nice
> to have while we're here. But if you want to make the trivial fix I
> can come along and do it later.

+Steve Rutherford