Thread (22 messages) 22 messages, 6 authors, 2025-03-27

Re: [PATCH 1/7] KVM: x86: Free vCPUs before freeing VM state

From: Sean Christopherson <seanjc@google.com>
Date: 2025-02-26 00:27:33
Also in: kvm, kvm-riscv, kvmarm, linux-arm-kernel, linux-mips, linux-riscv, lkml, loongarch

On Wed, Feb 26, 2025, Paolo Bonzini wrote:
On 2/25/25 00:55, Sean Christopherson wrote:
quoted
Free vCPUs before freeing any VM state, as both SVM and VMX may access
VM state when "freeing" a vCPU that is currently "in" L2, i.e. that needs
to be kicked out of nested guest mode.

Commit 6fcee03df6a1 ("KVM: x86: avoid loading a vCPU after .vm_destroy was
called") partially fixed the issue, but for unknown reasons only moved the
MMU unloading before VM destruction.  Complete the change, and free all
vCPU state prior to destroying VM state, as nVMX accesses even more state
than nSVM.
I applied this to kvm-coco-queue, I will place it in kvm/master too unless
you shout.
Depends on what "this" is :-)

My plan/hope is to land patches 1 and 2 in 6.14, i.e. in kvm/master, but the
rest are firmly 6.15 IMO.  And based on Yan's feedback, I'm planning on adding a
few more cleanups (though I think they're fully additive, i.e. can go on top).
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help