From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sheng Yang Subject: Re: [PATCH 08/10] KVM: Add assigned_device_msi_dispatch() Date: Thu, 30 Oct 2008 16:56:58 +0800 Message-ID: <200810301656.59139.sheng@linux.intel.com> References: <1225354818-16963-1-git-send-email-sheng@linux.intel.com> <1225354818-16963-9-git-send-email-sheng@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Cc: Avi Kivity To: kvm@vger.kernel.org Return-path: Received: from mga09.intel.com ([134.134.136.24]:38894 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753216AbYJ3I7N (ORCPT ); Thu, 30 Oct 2008 04:59:13 -0400 In-Reply-To: <1225354818-16963-9-git-send-email-sheng@linux.intel.com> Content-Disposition: inline Sender: kvm-owner@vger.kernel.org List-ID: On Thursday 30 October 2008 16:20:16 Sheng Yang wrote: > The function is used to dispatch MSI to lapic according to MSI message > address and message data. > > (Any way to replace macro GET_VAL_FROM_SHIFT()?) > > Signed-off-by: Sheng Yang > --- > virt/kvm/kvm_main.c | 45 +++++++++++++++++++++++++++++++++++++++++++++ > 1 files changed, 45 insertions(+), 0 deletions(-) > > diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c > index ddfd7ac..be0f943 100644 > --- a/virt/kvm/kvm_main.c > +++ b/virt/kvm/kvm_main.c > @@ -46,6 +46,7 @@ > #include > #include > #include > +#include > > #ifdef KVM_COALESCED_MMIO_PAGE_OFFSET > #include "coalesced_mmio.h" > @@ -78,6 +79,50 @@ static long kvm_vcpu_ioctl(struct file *file, unsigned > int ioctl, bool kvm_rebooting; > > #ifdef KVM_CAP_DEVICE_ASSIGNMENT > +static void assigned_device_msi_dispatch(struct kvm_assigned_dev_kernel > *dev) +{ > +#define GET_VAL_FROM_SHIFT(v, s) (((v) & (1 << s)) >> s) > + int vcpu_id; > + struct kvm_vcpu *vcpu; > + struct kvm_ioapic *ioapic = ioapic_irqchip(dev->kvm); > + u8 dest_id = MSI_ADDR_DEST_ID(dev->guest_msi_addr); > + u8 vector = MSI_DATA_VECTOR(dev->guest_msi_data); Oops... These two macros are wrong used, would be fixed in next version. -- regards Yang, Sheng > + u8 dest_mode = GET_VAL_FROM_SHIFT(dev->guest_msi_addr, > + MSI_ADDR_DEST_MODE_SHIFT); > + u8 trig_mode = GET_VAL_FROM_SHIFT(dev->guest_msi_data, > + MSI_DATA_TRIGGER_SHIFT); > + u8 delivery_mode = GET_VAL_FROM_SHIFT(dev->guest_msi_data, > + MSI_DATA_DELIVERY_MODE_SHIFT); > + u32 deliver_bitmask; > + > + BUG_ON(!ioapic); > + > + deliver_bitmask = ioapic_get_delivery_bitmask(ioapic, > + dest_id, dest_mode); > + switch (delivery_mode) { > + case MSI_DATA_DELIVERY_LOWPRI_VAL: > + vcpu = kvm_get_lowest_prio_vcpu(ioapic->kvm, vector, > + deliver_bitmask); > + if (vcpu != NULL) > + kvm_apic_set_irq(vcpu, vector, trig_mode); > + else > + printk(KERN_INFO "kvm: null lowest priority vcpu!\n"); > + break; > + case MSI_DATA_DELIVERY_FIXED_VAL: > + for (vcpu_id = 0; deliver_bitmask != 0; vcpu_id++) { > + if (!(deliver_bitmask & (1 << vcpu_id))) > + continue; > + deliver_bitmask &= ~(1 << vcpu_id); > + vcpu = ioapic->kvm->vcpus[vcpu_id]; > + if (vcpu) > + kvm_apic_set_irq(vcpu, vector, trig_mode); > + } > + break; > + default: > + printk(KERN_INFO "kvm: unsupported MSI delivery mode\n"); > + } > +} > + > static struct kvm_assigned_dev_kernel *kvm_find_assigned_dev(struct > list_head *head, int assigned_dev_id) > {