linux-kernel - Re: [PATCH v11 00/16] LAM and LASS KVM Enabling

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <ZTMatKliYT5_I0bg@google.com>
Date:   Fri, 20 Oct 2023 17:26:28 -0700
From:   Sean Christopherson <seanjc@...gle.com>
To:     Binbin Wu <binbin.wu@...ux.intel.com>
Cc:     kvm@...r.kernel.org, linux-kernel@...r.kernel.org,
        pbonzini@...hat.com, chao.gao@...el.com, kai.huang@...el.com,
        David.Laight@...lab.com, robert.hu@...ux.intel.com,
        guang.zeng@...el.com
Subject: Re: [PATCH v11 00/16] LAM and LASS KVM Enabling

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 :-(