Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.31 as permitted sender) client-ip=23.128.96.31;
Date:   Wed, 29 Nov 2023 17:42:04 -0800
In-Reply-To: <20231025152406.1879274-1-vkuznets@redhat.com>
Mime-Version: 1.0
References: <20231025152406.1879274-1-vkuznets@redhat.com>
Message-ID: <ZWfobPuhnXZYaAVj@google.com>
Subject: Re: [PATCH 00/14] KVM: x86: Make Hyper-V emulation optional
From:   Sean Christopherson <seanjc@google.com>
To:     Vitaly Kuznetsov <vkuznets@redhat.com>
Cc:     kvm@vger.kernel.org, Paolo Bonzini <pbonzini@redhat.com>,
        Maxim Levitsky <mlevitsk@redhat.com>,
        linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="us-ascii"
Precedence: bulk

On Wed, Oct 25, 2023, Vitaly Kuznetsov wrote:
> Vitaly Kuznetsov (14):
>   KVM: x86: xen: Remove unneeded xen context from struct kvm_arch when
>     !CONFIG_KVM_XEN
>   KVM: x86: hyper-v: Move Hyper-V partition assist page out of Hyper-V
>     emulation context
>   KVM: VMX: Split off vmx_onhyperv.{ch} from hyperv.{ch}
>   KVM: x86: hyper-v: Introduce kvm_hv_synic_auto_eoi_set()
>   KVM: x86: hyper-v: Introduce kvm_hv_synic_has_vector()
>   KVM: VMX: Split off hyperv_evmcs.{ch}
>   KVM: x86: hyper-v: Introduce kvm_hv_nested_transtion_tlb_flush()
>     helper
>   KVM: selftests: Make all Hyper-V tests explicitly dependent on Hyper-V
>     emulation support in KVM
>   KVM: selftests: Fix vmxon_pa == vmcs12_pa == -1ull
>     vmx_set_nested_state_test for !eVMCS case
>   KVM: x86: Make Hyper-V emulation optional
>   KVM: nVMX: hyper-v: Introduce nested_vmx_evmptr12() and
>     nested_vmx_is_evmptr12_valid() helpers
>   KVM: nVMX: hyper-v: Introduce nested_vmx_evmcs() accessor
>   KVM: nVMX: hyper-v: Hide more stuff under CONFIG_KVM_HYPERV
>   KVM: nSVM: hyper-v: Hide more stuff under
>     CONFIG_KVM_HYPERV/CONFIG_HYPERV

No major complaints.  If it hadn't been for the build failure, I likely wouldn't
have even bothered with most of my nits ;-)

I'll wait for v2 though, trying to fixup as I go will be a bit risky.