> -----Original Message----- > From: Zhang, Yang Z [mailto:yang.z.zhang@xxxxxxxxx] > Sent: Friday, April 26, 2013 1:10 PM > To: Yangminqiang; kvm@xxxxxxxxxxxxxxx > Cc: gleb@xxxxxxxxxx; mtosatti@xxxxxxxxxx; Zhang, Xiantao; Nakajima, Jun; > Luohao (brian); Haofeng > Subject: RE: [PATCH v10 7/7] KVM: VMX: Use posted interrupt to deliver virtual > interrupt > > Yangminqiang wrote on 2013-04-26: > > Hi Yang Zhang, > > > > Could you please let me know your CPU model or the CPU models which > > supports apic-v which your patch requires()? So that I could try you > > patches. > > > > Intel Software Developer's Manualm, Volume 3C, > > System Programming Guide, Part 3. Ch29, > > APIC VIRTUALIZATION AND VIRTUAL INTERRUPTS > > Or how can I know whether my hardware support those features listed in the > > manual above? > Ivytown or newer platform supported it. Ivytown? Do you mean Ivy Bridge? > > > Thanks, > > Steven > > > > kvm-owner@xxxxxxxxxxxxxxx wrote on 2013-04-11: > >> Subject: [PATCH v10 7/7] KVM: VMX: Use posted interrupt to deliver virtual > >> interrupt > >> > >> From: Yang Zhang <yang.z.zhang@xxxxxxxxx> > >> > >> If posted interrupt is avaliable, then uses it to inject virtual > >> interrupt to guest. > >> > >> Signed-off-by: Yang Zhang <yang.z.zhang@xxxxxxxxx> > >> --- > >> arch/x86/kvm/lapic.c | 30 +++++++++++++++++++----------- > >> arch/x86/kvm/vmx.c | 2 +- > >> arch/x86/kvm/x86.c | 1 + > >> 3 files changed, 21 insertions(+), 12 deletions(-) > >> diff --git a/arch/x86/kvm/lapic.c b/arch/x86/kvm/lapic.c > >> index dbf74c9..e29883c 100644 > >> --- a/arch/x86/kvm/lapic.c > >> +++ b/arch/x86/kvm/lapic.c > >> @@ -353,6 +353,7 @@ static inline int apic_find_highest_irr(struct > kvm_lapic > >> *apic) > >> if (!apic->irr_pending) > >> return -1; > >> + kvm_x86_ops->sync_pir_to_irr(apic->vcpu); > >> result = apic_search_irr(apic); > >> ASSERT(result == -1 || result >= 16); > >> @@ -683,18 +684,25 @@ static int __apic_accept_irq(struct kvm_lapic > *apic, > >> int delivery_mode, > >> if (dest_map) > >> __set_bit(vcpu->vcpu_id, dest_map); > >> - result = !apic_test_and_set_irr(vector, apic); > >> - trace_kvm_apic_accept_irq(vcpu->vcpu_id, delivery_mode, > >> - trig_mode, vector, !result); > >> - if (!result) { > >> - if (trig_mode) > >> - apic_debug("level trig mode repeatedly for " > >> - "vector %d", vector); > >> - break; > >> - } > >> + if (kvm_x86_ops->deliver_posted_interrupt) { > >> + result = 1; > >> + kvm_x86_ops->deliver_posted_interrupt(vcpu, vector); > >> + } else { > >> + result = !apic_test_and_set_irr(vector, apic); > >> > >> - kvm_make_request(KVM_REQ_EVENT, vcpu); > >> - kvm_vcpu_kick(vcpu); > >> + if (!result) { > >> + if (trig_mode) > >> + apic_debug("level trig mode repeatedly " > >> + "for vector %d", vector); > >> + goto out; > >> + } > >> + > >> + kvm_make_request(KVM_REQ_EVENT, vcpu); > >> + kvm_vcpu_kick(vcpu); > >> + } > >> +out: > >> + trace_kvm_apic_accept_irq(vcpu->vcpu_id, delivery_mode, > >> + trig_mode, vector, !result); > >> break; > >> > >> case APIC_DM_REMRD: > >> diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c > >> index 314b2ed..52b21da 100644 > >> --- a/arch/x86/kvm/vmx.c > >> +++ b/arch/x86/kvm/vmx.c > >> @@ -84,7 +84,7 @@ module_param(vmm_exclusive, bool, S_IRUGO); > >> static bool __read_mostly fasteoi = 1; > >> module_param(fasteoi, bool, S_IRUGO); > >> -static bool __read_mostly enable_apicv; > >> +static bool __read_mostly enable_apicv = 1; > >> module_param(enable_apicv, bool, S_IRUGO); > >> > >> /* > >> diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c > >> index 6147d24..628582f 100644 > >> --- a/arch/x86/kvm/x86.c > >> +++ b/arch/x86/kvm/x86.c > >> @@ -2685,6 +2685,7 @@ void kvm_arch_vcpu_put(struct kvm_vcpu *vcpu) > >> static int kvm_vcpu_ioctl_get_lapic(struct kvm_vcpu *vcpu, > > >> struct kvm_lapic_state *s) { + kvm_x86_ops->sync_pir_to_irr(vcpu); > >> memcpy(s->regs, vcpu->arch.apic->regs, sizeof *s); > >> > >> return 0; > >> -- > >> 1.7.1 > >> > >> -- > >> To unsubscribe from this list: send the line "unsubscribe kvm" in > >> the body of a message to majordomo@xxxxxxxxxxxxxxx > >> More majordomo info at http://vger.kernel.org/majordomo-info.html > > > Best regards, > Yang > ��.n��������+%������w��{.n�����o�^n�r������&��z�ޗ�zf���h���~����������_��+v���)ߣ�
