From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-9.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 61B19C54FD0 for ; Fri, 24 Apr 2020 15:17:21 +0000 (UTC) Received: from mm01.cs.columbia.edu (mm01.cs.columbia.edu [128.59.11.253]) by mail.kernel.org (Postfix) with ESMTP id E5696206D7 for ; Fri, 24 Apr 2020 15:17:20 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E5696206D7 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=kvmarm-bounces@lists.cs.columbia.edu Received: from localhost (localhost [127.0.0.1]) by mm01.cs.columbia.edu (Postfix) with ESMTP id 159C64B27C; Fri, 24 Apr 2020 11:17:20 -0400 (EDT) X-Virus-Scanned: at lists.cs.columbia.edu Received: from mm01.cs.columbia.edu ([127.0.0.1]) by localhost (mm01.cs.columbia.edu [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id g4e+4DI30wUS; Fri, 24 Apr 2020 11:17:18 -0400 (EDT) Received: from mm01.cs.columbia.edu (localhost [127.0.0.1]) by mm01.cs.columbia.edu (Postfix) with ESMTP id C73E14B261; Fri, 24 Apr 2020 11:17:18 -0400 (EDT) Received: from localhost (localhost [127.0.0.1]) by mm01.cs.columbia.edu (Postfix) with ESMTP id 5D8904B255 for ; Fri, 24 Apr 2020 11:17:17 -0400 (EDT) X-Virus-Scanned: at lists.cs.columbia.edu Received: from mm01.cs.columbia.edu ([127.0.0.1]) by localhost (mm01.cs.columbia.edu [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id NVzTE16enKtD for ; Fri, 24 Apr 2020 11:17:16 -0400 (EDT) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mm01.cs.columbia.edu (Postfix) with ESMTP id ECC894B1F8 for ; Fri, 24 Apr 2020 11:17:15 -0400 (EDT) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 79CA931B; Fri, 24 Apr 2020 08:17:15 -0700 (PDT) Received: from localhost.localdomain (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 8258C3F68F; Fri, 24 Apr 2020 08:17:14 -0700 (PDT) From: Andre Przywara To: Will Deacon , Julien Thierry Subject: [PATCH kvmtool v4 6/5] pci: Move legacy IRQ assignment into devices Date: Fri, 24 Apr 2020 16:17:02 +0100 Message-Id: <20200424151702.4750-1-andre.przywara@arm.com> X-Mailer: git-send-email 2.17.1 Cc: kvmarm@lists.cs.columbia.edu, kvm@vger.kernel.org X-BeenThere: kvmarm@lists.cs.columbia.edu X-Mailman-Version: 2.1.14 Precedence: list List-Id: Where KVM/ARM decisions are made List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: kvmarm-bounces@lists.cs.columbia.edu Sender: kvmarm-bounces@lists.cs.columbia.edu So far the (legacy) IRQ line for a PCI device is allocated in devices.c, which should actually not take care of that. Since we allocate all other device specific resources in the actual device emulation code, the IRQ should not be something special. Remove the PCI specific code from devices.c, and move the IRQ line allocation to the PCI code. This drops the IRQ line from the VESA device, since it does not use one. Signed-off-by: Andre Przywara --- Hi Will, this is the patch I mentioned earlier today. Briefly tested on x86 with the VESA device and on a Juno. Cheers, Andre devices.c | 9 --------- include/kvm/pci.h | 2 +- pci.c | 6 +++--- vfio/pci.c | 2 ++ virtio/pci.c | 5 ++--- 5 files changed, 8 insertions(+), 16 deletions(-) diff --git a/devices.c b/devices.c index 2c8b2665..41cffdd7 100644 --- a/devices.c +++ b/devices.c @@ -1,6 +1,5 @@ #include "kvm/devices.h" #include "kvm/kvm.h" -#include "kvm/pci.h" #include #include @@ -28,14 +27,6 @@ int device__register(struct device_header *dev) bus = &device_trees[dev->bus_type]; dev->dev_num = bus->dev_num++; - switch (dev->bus_type) { - case DEVICE_BUS_PCI: - pci__assign_irq(dev); - break; - default: - break; - } - node = &bus->root.rb_node; while (*node) { int num = rb_entry(*node, struct device_header, node)->dev_num; diff --git a/include/kvm/pci.h b/include/kvm/pci.h index ccb155e3..2c29c094 100644 --- a/include/kvm/pci.h +++ b/include/kvm/pci.h @@ -155,7 +155,7 @@ int pci__exit(struct kvm *kvm); struct pci_device_header *pci__find_dev(u8 dev_num); u32 pci_get_mmio_block(u32 size); u16 pci_get_io_port_block(u32 size); -void pci__assign_irq(struct device_header *dev_hdr); +int pci__assign_irq(struct pci_device_header *pci_hdr); void pci__config_wr(struct kvm *kvm, union pci_config_address addr, void *data, int size); void pci__config_rd(struct kvm *kvm, union pci_config_address addr, void *data, int size); diff --git a/pci.c b/pci.c index b6892d97..3ecdd0f9 100644 --- a/pci.c +++ b/pci.c @@ -49,10 +49,8 @@ void *pci_find_cap(struct pci_device_header *hdr, u8 cap_type) return NULL; } -void pci__assign_irq(struct device_header *dev_hdr) +int pci__assign_irq(struct pci_device_header *pci_hdr) { - struct pci_device_header *pci_hdr = dev_hdr->data; - /* * PCI supports only INTA#,B#,C#,D# per device. * @@ -64,6 +62,8 @@ void pci__assign_irq(struct device_header *dev_hdr) if (!pci_hdr->irq_type) pci_hdr->irq_type = IRQ_TYPE_EDGE_RISING; + + return pci_hdr->irq_line; } static void *pci_config_address_ptr(u16 port) diff --git a/vfio/pci.c b/vfio/pci.c index 4412c6d7..7c2ea10c 100644 --- a/vfio/pci.c +++ b/vfio/pci.c @@ -1212,6 +1212,8 @@ static int vfio_pci_configure_dev_irqs(struct kvm *kvm, struct vfio_device *vdev } if (pdev->irq_modes & VFIO_PCI_IRQ_MODE_INTX) { + pci__assign_irq(&vdev->pci.hdr); + ret = vfio_pci_init_intx(kvm, vdev); if (ret) return ret; diff --git a/virtio/pci.c b/virtio/pci.c index 281c3181..c6529493 100644 --- a/virtio/pci.c +++ b/virtio/pci.c @@ -524,13 +524,12 @@ int virtio_pci__init(struct kvm *kvm, void *dev, struct virtio_device *vdev, if (irq__can_signal_msi(kvm)) vpci->features |= VIRTIO_PCI_F_SIGNAL_MSI; + vpci->legacy_irq_line = pci__assign_irq(&vpci->pci_hdr); + r = device__register(&vpci->dev_hdr); if (r < 0) goto free_msix_mmio; - /* save the IRQ that device__register() has allocated */ - vpci->legacy_irq_line = vpci->pci_hdr.irq_line; - return 0; free_msix_mmio: -- 2.17.1 _______________________________________________ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listinfo/kvmarm