Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753454AbaB1SKk (ORCPT ); Fri, 28 Feb 2014 13:10:40 -0500 Received: from smtp.citrix.com ([66.165.176.89]:37997 "EHLO SMTP.CITRIX.COM" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752096AbaB1SKf (ORCPT ); Fri, 28 Feb 2014 13:10:35 -0500 X-IronPort-AV: E=Sophos;i="4.97,563,1389744000"; d="scan'208";a="106702611" Message-ID: <5310D118.2090302@citrix.com> Date: Fri, 28 Feb 2014 19:10:32 +0100 From: =?UTF-8?B?Um9nZXIgUGF1IE1vbm7DqQ==?= User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:24.0) Gecko/20100101 Thunderbird/24.3.0 MIME-Version: 1.0 To: Boris Ostrovsky CC: , , David Vrabel Subject: Re: [PATCH v2] xen: add support for MSI message groups References: <530F68BE.2070505@oracle.com> <1393524935-4216-1-git-send-email-roger.pau@citrix.com> <530F87CC.8090000@oracle.com> <5310C54A.1080906@oracle.com> <5310CB6F.1070706@citrix.com> <5310CEB6.4010604@oracle.com> In-Reply-To: <5310CEB6.4010604@oracle.com> X-Enigmail-Version: 1.6 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8bit X-DLP: MIA2 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 28/02/14 19:00, Boris Ostrovsky wrote: > On 02/28/2014 12:46 PM, Roger Pau Monné wrote: >> On 28/02/14 18:20, Boris Ostrovsky wrote: >>> On 02/27/2014 01:45 PM, Boris Ostrovsky wrote: >>>> On 02/27/2014 01:15 PM, Roger Pau Monne wrote: >>>>> Add support for MSI message groups for Xen Dom0 using the >>>>> MAP_PIRQ_TYPE_MULTI_MSI pirq map type. >>>>> >>>>> In order to keep track of which pirq is the first one in the group all >>>>> pirqs in the MSI group except for the first one have the newly >>>>> introduced PIRQ_MSI_GROUP flag set. This prevents calling >>>>> PHYSDEVOP_unmap_pirq on them, since the unmap must be done with the >>>>> first pirq in the group. >>>> Reviewed-by: Boris Ostrovsky >>>> >>>> >>> >>> I was just looking at xen_setup_msi_irqs() (for a different reason) and >>> I am no longer sure this patch does anything: >>> >>> static int xen_setup_msi_irqs(struct pci_dev *dev, int nvec, int type) >>> { >>> int irq, ret, i; >>> struct msi_desc *msidesc; >>> int *v; >>> >>> if (type == PCI_CAP_ID_MSI && nvec > 1) >>> return 1; >>> .... >>> >>> Same thing for xen_hvm_setup_msi_irqs(). >> As said in the commit message this is only for Dom0, so the function >> modified is xen_initdom_setup_msi_irqs (were this check is removed). > > Then what is the reason for these changes: > > diff --git a/arch/x86/pci/xen.c b/arch/x86/pci/xen.c > index 103e702..905956f 100644 > --- a/arch/x86/pci/xen.c > +++ b/arch/x86/pci/xen.c > @@ -178,6 +178,7 @@ static int xen_setup_msi_irqs(struct pci_dev *dev, > int nvec, int type) > i = 0; > list_for_each_entry(msidesc, &dev->msi_list, list) { > irq = xen_bind_pirq_msi_to_irq(dev, msidesc, v[i], > + (type == PCI_CAP_ID_MSI) ? nvec : 1, > (type == PCI_CAP_ID_MSIX) ? > "pcifront-msi-x" : > "pcifront-msi", > @@ -245,6 +246,7 @@ static int xen_hvm_setup_msi_irqs(struct pci_dev > *dev, int nvec, int type) > "xen: msi already bound to pirq=%d\n", pirq); > } > irq = xen_bind_pirq_msi_to_irq(dev, msidesc, pirq, > + (type == PCI_CAP_ID_MSI) ? nvec : 1, > (type == PCI_CAP_ID_MSIX) ? > "msi-x" : "msi", > DOMID_SELF); > > Should you simply pass 1? Yes, but then if we implement MSI message groups for those cases we will need to modify this line again, this way it's already correctly setup. If you think it's best to hardcode it to 1, I can change it (I was also in doubt about which way was better when modifying those lines). Roger. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/