Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:4 as permitted sender) client-ip=2620:137:e000::3:4;
Date:   Fri, 20 Oct 2023 17:34:11 -0700
In-Reply-To: <ZTMatKliYT5_I0bg@google.com>
Mime-Version: 1.0
References: <20230913124227.12574-1-binbin.wu@linux.intel.com> <ZTMatKliYT5_I0bg@google.com>
Message-ID: <ZTMcg__FPmRVqec9@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 Fri, Oct 20, 2023, Sean Christopherson wrote:
> 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.

Gah, by "this" I meant the LAM parts.  LASS is going to have to wait until the
kernel support lands.