linux-kernel - Re: [PATCH 0/2] KVM: nVMX: Skip IPBP on nested VMCS switch

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] [day] [month] [year] [list]

Message-ID: <0cc08cc0-252c-378e-15b3-6d9639ee5552@redhat.com>
Date:   Thu, 7 May 2020 13:58:34 +0200
From:   Paolo Bonzini <pbonzini@...hat.com>
To:     Sean Christopherson <sean.j.christopherson@...el.com>
Cc:     Vitaly Kuznetsov <vkuznets@...hat.com>,
        Wanpeng Li <wanpengli@...cent.com>,
        Jim Mattson <jmattson@...gle.com>,
        Joerg Roedel <joro@...tes.org>, kvm@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/2] KVM: nVMX: Skip IPBP on nested VMCS switch

On 07/05/20 01:58, Sean Christopherson wrote:
> Or as Jim would say, "Really skip IPBP on nested VMCS switch" :-D
> 
> Patch 1 is the delta between kvm/queue and v3 of the original patch[*],
> i.e. I just cherry-picked v3 and fixed the conflicts.
> 
> Patch 2 applies the "no IPBP" logic to copy_vmcs02_to_vmcs12_rare().
> 
> Feel free to sqaush both of these to commit 7407a52f23732 ("KVM: nVMX:
> Skip IBPB when switching between vmcs01 and vmcs02") if you so desire.

I squashed patch 1 and applied patch 2 separately.  Thanks!

Paolo

> [*] https://lkml.kernel.org/r/20200505044644.16563-1-sean.j.christopherson@intel.com
> 
> Sean Christopherson (2):
>   KVM: nVMX: Refactor IBPB handling on VMCS switch to genericize code
>   KVM: nVMX: Skip IPBP when switching between vmcs01 and vmcs02, redux
> 
>  arch/x86/kvm/vmx/nested.c | 13 +++----------
>  arch/x86/kvm/vmx/vmx.c    | 19 ++++++++++++++-----
>  arch/x86/kvm/vmx/vmx.h    |  4 ++--
>  3 files changed, 19 insertions(+), 17 deletions(-)
>