Messages in this thread |  | | Subject | Re: [PATCH] KVM: Enable hardware before doing arch VM initialization | From | Christian Borntraeger <> | Date | Thu, 24 Sep 2020 07:30:20 +0200 |
| |
On 23.09.20 20:57, Sean Christopherson wrote: > Swap the order of hardware_enable_all() and kvm_arch_init_vm() to > accommodate Intel's Trust Domain Extension (TDX), which needs VMX to be > fully enabled during VM init in order to make SEAMCALLs. > > This also provides consistent ordering between kvm_create_vm() and > kvm_destroy_vm() with respect to calling kvm_arch_destroy_vm() and > hardware_disable_all(). > > Cc: Marc Zyngier <maz@kernel.org> > Cc: James Morse <james.morse@arm.com> > Cc: Julien Thierry <julien.thierry.kdev@gmail.com> > Cc: Suzuki K Poulose <suzuki.poulose@arm.com> > Cc: linux-arm-kernel@lists.infradead.org > Cc: Huacai Chen <chenhc@lemote.com> > Cc: Aleksandar Markovic <aleksandar.qemu.devel@gmail.com> > Cc: linux-mips@vger.kernel.org > Cc: Paul Mackerras <paulus@ozlabs.org> > Cc: kvm-ppc@vger.kernel.org > Cc: Christian Borntraeger <borntraeger@de.ibm.com> > Cc: Janosch Frank <frankja@linux.ibm.com> > Cc: David Hildenbrand <david@redhat.com> > Cc: Cornelia Huck <cohuck@redhat.com> > Cc: Claudio Imbrenda <imbrenda@linux.ibm.com> > Cc: Vitaly Kuznetsov <vkuznets@redhat.com> > Cc: Wanpeng Li <wanpengli@tencent.com> > Cc: Jim Mattson <jmattson@google.com> > Cc: Joerg Roedel <joro@8bytes.org> > Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com> > --- > > Obviously not required until the TDX series comes along, but IMO KVM > should be consistent with respect to enabling and disabling virt support > in hardware. > > Tested only on Intel hardware. Unless I missed something, this only > affects x86, Arm and MIPS as hardware enabling is a nop for s390 and PPC.
Yes, looks fine from an s390 perspective.
Reviewed-by: Christian Borntraeger <borntraeger@de.ibm.com>
|  |