Re: [PATCH v2 00/15] KVM x86/VMX cleanups

From: David Hildenbrand <david@redhat.com>
To: kvm@vger.kernel.org
Cc: "Paolo Bonzini" <pbonzini@redhat.com>,
	"Radim Krčmář" <rkrcmar@redhat.com>
Subject: Re: [PATCH v2 00/15] KVM x86/VMX cleanups
Date: Tue, 19 Sep 2017 14:42:07 +0200	[thread overview]
Message-ID: <89c38a89-5934-8786-a37e-244de3016739@redhat.com> (raw)
In-Reply-To: <20170824185137.6253-1-david@redhat.com>

On 24.08.2017 20:51, David Hildenbrand wrote:

Ping :)

> Some cleanups discovered while digging through the code.
> 
> v1 -> v2:
> - dropped "KVM: x86: mmu: use for_each_shadow_entry_lockless()"
> - added "KVM: VMX: rename RDSEED and RDRAND vmx ctrls to reflect exiting"
> - added "KVM: nVMX: no need to set ept/vpid caps to 0"
> - added "KVM: VMX: drop enable_ept check from ept_sync_context()"
> 
> David Hildenbrand (15):
>   KVM: x86: mmu: returning void in a void function is strange
>   KVM: x86: mmu: free_page can handle NULL
>   KVM: x86: drop BUG_ON(vcpu->kvm)
>   KVM: VMX: vmx_vcpu_setup() cannot fail
>   KVM: x86: no need to inititalize vcpu members to 0
>   KVM: VMX: drop enable_ept check from ept_sync_context()
>   KVM: VMX: call ept_sync_global() with enable_ept only
>   KVM: VMX: require INVEPT GLOBAL for EPT
>   KVM: VMX: drop unnecessary function declarations
>   KVM: nVMX: no need to set vcpu->cpu when switching vmcs
>   KVM: nVMX: no need to set ept/vpid caps to 0
>   KVM: VMX: cleanup init_rmode_identity_map()
>   KVM: x86: document special identity map address value
>   KVM: x86: allow setting identity map addr with no vcpus only
>   KVM: VMX: rename RDSEED and RDRAND vmx ctrls to reflect exiting
> 
>  Documentation/virtual/kvm/api.txt |  4 ++
>  arch/x86/include/asm/vmx.h        |  4 +-
>  arch/x86/kvm/mmu.c                |  5 +-
>  arch/x86/kvm/vmx.c                | 96 ++++++++++++++-------------------------
>  arch/x86/kvm/x86.c                | 21 ++++-----
>  5 files changed, 51 insertions(+), 79 deletions(-)
> 


-- 

Thanks,

David