Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) client-ip=23.128.96.33;
Date:   Fri, 20 Oct 2023 17:26:28 -0700
In-Reply-To: <20230913124227.12574-1-binbin.wu@linux.intel.com>
Mime-Version: 1.0
References: <20230913124227.12574-1-binbin.wu@linux.intel.com>
Message-ID: <ZTMatKliYT5_I0bg@google.com>
Subject: Re: [PATCH v11 00/16] LAM and LASS KVM Enabling
From:   Sean Christopherson <seanjc@google.com>
To:     Binbin Wu <binbin.wu@linux.intel.com>
Cc:     kvm@vger.kernel.org, linux-kernel@vger.kernel.org,
        pbonzini@redhat.com, chao.gao@intel.com, kai.huang@intel.com,
        David.Laight@aculab.com, robert.hu@linux.intel.com,
        guang.zeng@intel.com
Content-Type: text/plain; charset="us-ascii"
Precedence: bulk

On Wed, Sep 13, 2023, Binbin Wu wrote:
> Binbin Wu (10):
>   KVM: x86: Consolidate flags for __linearize()
>   KVM: x86: Use a new flag for branch targets
>   KVM: x86: Add an emulation flag for implicit system access
>   KVM: x86: Add X86EMUL_F_INVLPG and pass it in em_invlpg()
>   KVM: x86/mmu: Drop non-PA bits when getting GFN for guest's PGD
>   KVM: x86: Add & use kvm_vcpu_is_legal_cr3() to check CR3's legality
>   KVM: x86: Remove kvm_vcpu_is_illegal_gpa()
>   KVM: x86: Introduce get_untagged_addr() in kvm_x86_ops and call it in
>     emulator
>   KVM: x86: Untag address for vmexit handlers when LAM applicable
>   KVM: x86: Use KVM-governed feature framework to track "LAM enabled"
> 
> Robert Hoo (3):
>   KVM: x86: Virtualize LAM for supervisor pointer
>   KVM: x86: Virtualize LAM for user pointer
>   KVM: x86: Advertise and enable LAM (user and supervisor)
> 
> Zeng Guang (3):
>   KVM: emulator: Add emulation of LASS violation checks on linear
>     address
>   KVM: VMX: Virtualize LASS
>   KVM: x86: Advertise LASS CPUID to user space

This all looks good!  I have a few minor nits, but nothing I can't tweak when
applying.  Assuming nothing explodes in testing, I'll get this applied for 6.8
next week.

My apologies for not getting to this sooner and missing 6.7 :-(