[0/3] KVM: x86: Cleanups

Message

Michal Luczaj Aug. 14, 2023, 10:08 p.m. UTC

As discussed[*], some minor clean-ups. Plus an attempt to smuggle a fix for
a typo in API documentation.

[*] https://lore.kernel.org/kvm/[e-mail]/

Michal Luczaj (3):
  KVM: x86: Remove redundant vcpu->arch.cr0 assignments
  KVM: x86: Force TLB flush on changes to special registers
  KVM: Correct kvm_vcpu_event(s) typo in KVM API documentation

 Documentation/virt/kvm/api.rst | 34 +++++++++++++++++-----------------
 arch/x86/kvm/smm.c             |  1 -
 arch/x86/kvm/x86.c             |  9 ++++++---
 3 files changed, 23 insertions(+), 21 deletions(-)

Comments

Sean Christopherson Sept. 28, 2023, 4:41 p.m. UTC | #1

On Tue, 15 Aug 2023 00:08:34 +0200, Michal Luczaj wrote:
> As discussed[*], some minor clean-ups. Plus an attempt to smuggle a fix for
> a typo in API documentation.
> 
> [*] https://lore.kernel.org/kvm/[e-mail]/
> 
> Michal Luczaj (3):
>   KVM: x86: Remove redundant vcpu->arch.cr0 assignments
>   KVM: x86: Force TLB flush on changes to special registers
>   KVM: Correct kvm_vcpu_event(s) typo in KVM API documentation
> 
> [...]

Applied 1 and 2 to kvm-x86 misc, and 3 to docs.  I massaged the changelog for
patch 2 to make it clear that flushing whenever the MMU context is reset is
overkill.  I had to go back to the original discussion between you and Paolo to
understand why Paolo suggested blasting a guest TLB flush.

Thanks!

[1/3] KVM: x86: Remove redundant vcpu->arch.cr0 assignments
      https://github.com/kvm-x86/linux/commit/9dbb029b9c44
[2/3] KVM: x86: Force TLB flush on changes to special registers
      https://github.com/kvm-x86/linux/commit/4346db6e6e7a
[3/3] KVM: Correct kvm_vcpu_event(s) typo in KVM API documentation
      https://github.com/kvm-x86/linux/commit/57f33f1a8756

--
https://github.com/kvm-x86/linux/tree/next