> From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Wednesday, July 11, 2018 9:25 PM > > From: Suravee Suthikulpanit <suravee.suthikulpa...@amd.com> > > This patch modifies the hvm_funcs.virtual_intr_delivery_enabled() > to become a bool variable as VMX does and SVM will simply return a > static value. > > Signed-off-by: Suravee Suthikulpanit <suravee.suthikulpa...@amd.com> > Signed-off-by: Jan Beulich <jbeul...@suse.com> > --- > Extracted from an SVM/AVIC series patch. > > --- a/xen/arch/x86/hvm/vlapic.c > +++ b/xen/arch/x86/hvm/vlapic.c > @@ -1258,14 +1258,6 @@ void vlapic_adjust_i8259_target(struct d > pt_adjust_global_vcpu_target(v); > } > > -int vlapic_virtual_intr_delivery_enabled(void) > -{ > - if ( hvm_funcs.virtual_intr_delivery_enabled ) > - return hvm_funcs.virtual_intr_delivery_enabled(); > - else > - return 0; > -} > - > int vlapic_has_pending_irq(struct vcpu *v) > { > struct vlapic *vlapic = vcpu_vlapic(v); > @@ -1278,7 +1270,7 @@ int vlapic_has_pending_irq(struct vcpu * > if ( irr == -1 ) > return -1; > > - if ( vlapic_virtual_intr_delivery_enabled() && > + if ( hvm_funcs.virtual_intr_delivery_enabled && > !nestedhvm_vcpu_in_guestmode(v) ) > return irr; >
above is another functional change. Can you explain and if making sense then put the fix in separate patch? Thanks Kevin _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel