linux-kernel - Re: [PATCH v3 00/22] https://www.spinics.net/lists/kvm/msg267878.html

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <2939c0cb-8e0c-7de4-7143-2df303bbb542@redhat.com>
Date:   Thu, 14 Apr 2022 09:47:31 +0200
From:   Paolo Bonzini <pbonzini@...hat.com>
To:     linux-kernel@...r.kernel.org, kvm@...r.kernel.org
Cc:     seanjc@...gle.com
Subject: Re: [PATCH v3 00/22] https://www.spinics.net/lists/kvm/msg267878.html

Uh-oh, wrong subject.  Should be "KVM MMU refactoring part 2: role changes".

Supersedes: <20220221162243.683208-1-pbonzini@...hat.com>

Paolo

On 4/14/22 09:39, Paolo Bonzini wrote:
> Right now the "MMU role" is a messy mix of the shadow page table format
> and the CPU paging mode (CR0/CR4/EFER, SMM, guest mode, etc).  Whenever
> something is different between the MMU and the CPU, it is stored as an
> extra field in struct kvm_mmu; for extra bonus complication, sometimes
> the same thing is stored in both the role and an extra field.
> 
> This series cleans up things by putting the two in separate fields,
> so that the "MMU role" represents exactly the role of the root page.
> This in turn makes it possible to eliminate various fields that are
> now redundant with either the CPU or te MMU role.
> 
> These patches have mostly been posted and reviewed already[1], and I
> have now retested them on top of kvm/next.
> 
> Paolo
> 
> [1] https://patchew.org/linux/20220221162243.683208-1-pbonzini@redhat.com/
> 
> Paolo Bonzini (21):
>    KVM: x86/mmu: nested EPT cannot be used in SMM
>    KVM: x86/mmu: constify uses of struct kvm_mmu_role_regs
>    KVM: x86/mmu: pull computation of kvm_mmu_role_regs to kvm_init_mmu
>    KVM: x86/mmu: rephrase unclear comment
>    KVM: x86/mmu: remove "bool base_only" arguments
>    KVM: x86/mmu: split cpu_role from mmu_role
>    KVM: x86/mmu: do not recompute root level from kvm_mmu_role_regs
>    KVM: x86/mmu: remove ept_ad field
>    KVM: x86/mmu: remove kvm_calc_shadow_root_page_role_common
>    KVM: x86/mmu: cleanup computation of MMU roles for two-dimensional
>      paging
>    KVM: x86/mmu: cleanup computation of MMU roles for shadow paging
>    KVM: x86/mmu: store shadow EFER.NX in the MMU role
>    KVM: x86/mmu: remove extended bits from mmu_role, rename field
>    KVM: x86/mmu: rename kvm_mmu_role union
>    KVM: x86/mmu: remove redundant bits from extended role
>    KVM: x86/mmu: remove valid from extended role
>    KVM: x86/mmu: simplify and/or inline computation of shadow MMU roles
>    KVM: x86/mmu: pull CPU mode computation to kvm_init_mmu
>    KVM: x86/mmu: replace shadow_root_level with root_role.level
>    KVM: x86/mmu: replace root_level with cpu_role.base.level
>    KVM: x86/mmu: replace direct_map with root_role.direct
> 
> Sean Christopherson (1):
>    KVM: x86: Clean up and document nested #PF workaround
> 
>   arch/x86/include/asm/kvm_host.h |  19 +-
>   arch/x86/kvm/mmu.h              |   2 +-
>   arch/x86/kvm/mmu/mmu.c          | 376 ++++++++++++++------------------
>   arch/x86/kvm/mmu/paging_tmpl.h  |  14 +-
>   arch/x86/kvm/mmu/tdp_mmu.c      |   4 +-
>   arch/x86/kvm/svm/nested.c       |  18 +-
>   arch/x86/kvm/svm/svm.c          |   2 +-
>   arch/x86/kvm/vmx/nested.c       |  15 +-
>   arch/x86/kvm/vmx/vmx.c          |   2 +-
>   arch/x86/kvm/x86.c              |  33 ++-
>   10 files changed, 219 insertions(+), 266 deletions(-)
>