From patchwork Mon May 6 13:44:16 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Gavin Shan X-Patchwork-Id: 2524201 X-Patchwork-Delegate: bhelgaas@google.com Return-Path: X-Original-To: patchwork-linux-pci@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id A3C503FD85 for ; Mon, 6 May 2013 13:44:50 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753215Ab3EFNop (ORCPT ); Mon, 6 May 2013 09:44:45 -0400 Received: from e7.ny.us.ibm.com ([32.97.182.137]:54463 "EHLO e7.ny.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753417Ab3EFNoc (ORCPT ); Mon, 6 May 2013 09:44:32 -0400 Received: from /spool/local by e7.ny.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Mon, 6 May 2013 09:44:31 -0400 Received: from d01dlp02.pok.ibm.com (9.56.250.167) by e7.ny.us.ibm.com (192.168.1.107) with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted; Mon, 6 May 2013 09:44:30 -0400 Received: from d01relay05.pok.ibm.com (d01relay05.pok.ibm.com [9.56.227.237]) by d01dlp02.pok.ibm.com (Postfix) with ESMTP id 143F36E8059 for ; Mon, 6 May 2013 09:44:26 -0400 (EDT) Received: from d01av04.pok.ibm.com (d01av04.pok.ibm.com [9.56.224.64]) by d01relay05.pok.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id r46DiTrJ333810 for ; Mon, 6 May 2013 09:44:29 -0400 Received: from d01av04.pok.ibm.com (loopback [127.0.0.1]) by d01av04.pok.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id r46DiN6g018077 for ; Mon, 6 May 2013 09:44:28 -0400 Received: from shangw ([9.125.31.68]) by d01av04.pok.ibm.com (8.14.4/8.13.1/NCO v10.0 AVin) with ESMTP id r46DiL5a017790; Mon, 6 May 2013 09:44:22 -0400 Received: by shangw (Postfix, from userid 1000) id 42CC83019AD; Mon, 6 May 2013 21:44:19 +0800 (CST) From: Gavin Shan To: linuxppc-dev@lists.ozlabs.org, linux-pci@vger.kernel.org Cc: benh@kernel.crashing.org, yinghai@kernel.org, bhelgaas@google.com, Gavin Shan Subject: [PATCH 1/3] PCI: PCI_BUS_FLAGS_NO_IO flag for PCI bus Date: Mon, 6 May 2013 21:44:16 +0800 Message-Id: <1367847858-6506-1-git-send-email-shangw@linux.vnet.ibm.com> X-Mailer: git-send-email 1.7.5.4 X-TM-AS-MML: No X-Content-Scanned: Fidelis XPS MAILER x-cbid: 13050613-5806-0000-0000-00002103D63D Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org There has some PCI hosts (e.g. PHB3) that don't support IO. So the patch intends to introduce the flag to indicate the special case. In turn, we won't do IO resource assignment and enable that. Signed-off-by: Gavin Shan --- drivers/pci/pci.c | 3 +++ drivers/pci/setup-bus.c | 25 ++++++++++++++++--------- include/linux/pci.h | 1 + 3 files changed, 20 insertions(+), 9 deletions(-) diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c index a899d8b..2548bad 100644 --- a/drivers/pci/pci.c +++ b/drivers/pci/pci.c @@ -1218,6 +1218,9 @@ int pci_enable_device_mem(struct pci_dev *dev) */ int pci_enable_device(struct pci_dev *dev) { + if (dev->bus->bus_flags & PCI_BUS_FLAGS_NO_IO) + return pci_enable_device_flags(dev, IORESOURCE_MEM); + return pci_enable_device_flags(dev, IORESOURCE_MEM | IORESOURCE_IO); } diff --git a/drivers/pci/setup-bus.c b/drivers/pci/setup-bus.c index 16abaaa..c8293e90 100644 --- a/drivers/pci/setup-bus.c +++ b/drivers/pci/setup-bus.c @@ -133,6 +133,10 @@ static void pdev_sort_resources(struct pci_dev *dev, struct list_head *head) r = &dev->resource[i]; + if ((dev->bus->bus_flags & PCI_BUS_FLAGS_NO_IO) && + (r->flags & IORESOURCE_IO)) + continue; + if (r->flags & IORESOURCE_PCI_FIXED) continue; @@ -1083,17 +1087,20 @@ static void __ref __pci_bus_size_bridges(struct pci_bus *bus, additional_io_size = pci_hotplug_io_size; additional_mem_size = pci_hotplug_mem_size; } + + /* Follow thru */ + default: + if (!(bus->bus_flags & PCI_BUS_FLAGS_NO_IO)) + pbus_size_io(bus, + realloc_head ? 0 : additional_io_size, + additional_io_size, realloc_head); /* - * Follow thru + * If the bridge supports prefetchable range, size it + * separately. If it doesn't, or its prefetchable window + * has already been allocated by arch code, try + * non-prefetchable range for both types of PCI memory + * resources. */ - default: - pbus_size_io(bus, realloc_head ? 0 : additional_io_size, - additional_io_size, realloc_head); - /* If the bridge supports prefetchable range, size it - separately. If it doesn't, or its prefetchable window - has already been allocated by arch code, try - non-prefetchable range for both types of PCI memory - resources. */ mask = IORESOURCE_MEM; prefmask = IORESOURCE_MEM | IORESOURCE_PREFETCH; if (pbus_size_mem(bus, prefmask, prefmask, diff --git a/include/linux/pci.h b/include/linux/pci.h index 3a24e4f..53b595a 100644 --- a/include/linux/pci.h +++ b/include/linux/pci.h @@ -181,6 +181,7 @@ typedef unsigned short __bitwise pci_bus_flags_t; enum pci_bus_flags { PCI_BUS_FLAGS_NO_MSI = (__force pci_bus_flags_t) 1, PCI_BUS_FLAGS_NO_MMRBC = (__force pci_bus_flags_t) 2, + PCI_BUS_FLAGS_NO_IO = (__force pci_bus_flags_t) 4 }; /* Based on the PCI Hotplug Spec, but some values are made up by us */