mbox series

[v3,0/2] KVM: VMX: Unionize vcpu_vmx.exit_reason

Message ID 20200421075328.14458-1-sean.j.christopherson@intel.com (mailing list archive)
Headers show
Series KVM: VMX: Unionize vcpu_vmx.exit_reason | expand

Message

Sean Christopherson April 21, 2020, 7:53 a.m. UTC
Minor fixup patch for a mishandled conflict between the vmcs.INTR_INFO
caching series and the union series, plus the actual unionization patch
rebased onto kvm/queue, commit 604e8bba0dc5 ("KVM: Remove redundant ...").

Sean Christopherson (2):
  KVM: nVMX: Drop a redundant call to vmx_get_intr_info()
  KVM: VMX: Convert vcpu_vmx.exit_reason to a union

 arch/x86/kvm/vmx/nested.c | 39 ++++++++++++++---------
 arch/x86/kvm/vmx/vmx.c    | 65 ++++++++++++++++++++-------------------
 arch/x86/kvm/vmx/vmx.h    | 25 ++++++++++++++-
 3 files changed, 83 insertions(+), 46 deletions(-)

Comments

Paolo Bonzini April 21, 2020, 4:19 p.m. UTC | #1
On 21/04/20 09:53, Sean Christopherson wrote:
> Minor fixup patch for a mishandled conflict between the vmcs.INTR_INFO
> caching series and the union series, plus the actual unionization patch
> rebased onto kvm/queue, commit 604e8bba0dc5 ("KVM: Remove redundant ...").
> 
> Sean Christopherson (2):
>   KVM: nVMX: Drop a redundant call to vmx_get_intr_info()
>   KVM: VMX: Convert vcpu_vmx.exit_reason to a union
> 
>  arch/x86/kvm/vmx/nested.c | 39 ++++++++++++++---------
>  arch/x86/kvm/vmx/vmx.c    | 65 ++++++++++++++++++++-------------------
>  arch/x86/kvm/vmx/vmx.h    | 25 ++++++++++++++-
>  3 files changed, 83 insertions(+), 46 deletions(-)
> 

Thanks, I queued patch 1.  I am not too enthusiastic about patch 2, but
when SGX comes around it may be a better idea.

Paolo
Sean Christopherson April 21, 2020, 5:01 p.m. UTC | #2
On Tue, Apr 21, 2020 at 06:19:15PM +0200, Paolo Bonzini wrote:
> On 21/04/20 09:53, Sean Christopherson wrote:
> > Minor fixup patch for a mishandled conflict between the vmcs.INTR_INFO
> > caching series and the union series, plus the actual unionization patch
> > rebased onto kvm/queue, commit 604e8bba0dc5 ("KVM: Remove redundant ...").
> > 
> > Sean Christopherson (2):
> >   KVM: nVMX: Drop a redundant call to vmx_get_intr_info()
> >   KVM: VMX: Convert vcpu_vmx.exit_reason to a union
> > 
> >  arch/x86/kvm/vmx/nested.c | 39 ++++++++++++++---------
> >  arch/x86/kvm/vmx/vmx.c    | 65 ++++++++++++++++++++-------------------
> >  arch/x86/kvm/vmx/vmx.h    | 25 ++++++++++++++-
> >  3 files changed, 83 insertions(+), 46 deletions(-)
> > 
> 
> Thanks, I queued patch 1.  I am not too enthusiastic about patch 2, but
> when SGX comes around it may be a better idea.

And maybe it'll grow on you by the time I figure out how to send a pull
request ;-).