From patchwork Fri Apr 12 22:44:36 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yinghai Lu X-Patchwork-Id: 236270 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 35F882C00A7 for ; Sat, 13 Apr 2013 08:50:24 +1000 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754830Ab3DLWpH (ORCPT ); Fri, 12 Apr 2013 18:45:07 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:47158 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754707Ab3DLWpE (ORCPT ); Fri, 12 Apr 2013 18:45:04 -0400 Received: from acsinet22.oracle.com (acsinet22.oracle.com [141.146.126.238]) by userp1040.oracle.com (Sentrion-MTA-4.3.1/Sentrion-MTA-4.3.1) with ESMTP id r3CMiw8J000327 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Fri, 12 Apr 2013 22:44:59 GMT Received: from userz7022.oracle.com (userz7022.oracle.com [156.151.31.86]) by acsinet22.oracle.com (8.14.4+Sun/8.14.4) with ESMTP id r3CMivGH023037 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=FAIL); Fri, 12 Apr 2013 22:44:58 GMT Received: from abhmt115.oracle.com (abhmt115.oracle.com [141.146.116.67]) by userz7022.oracle.com (8.14.4+Sun/8.14.4) with ESMTP id r3CMivHX010649; Fri, 12 Apr 2013 22:44:57 GMT Received: from linux-siqj.site (/10.132.126.191) by default (Oracle Beehive Gateway v4.0) with ESMTP ; Fri, 12 Apr 2013 15:44:56 -0700 From: Yinghai Lu To: Bjorn Helgaas , Ram Pai Cc: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Yinghai Lu Subject: [PATCH v4 22/29] PCI: Add addon_resource support for pci devices Date: Fri, 12 Apr 2013 15:44:36 -0700 Message-Id: <1365806683-26717-23-git-send-email-yinghai@kernel.org> X-Mailer: git-send-email 1.8.1.4 In-Reply-To: <1365806683-26717-1-git-send-email-yinghai@kernel.org> References: <1365806683-26717-1-git-send-email-yinghai@kernel.org> X-Source-IP: acsinet22.oracle.com [141.146.126.238] Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org Now there is some non normal pci resources other than standard,rom, sriov, bridges. Some could be same as standard reg, but using different position. Some could have own way to read/write to them. Kernel are using different way to hack those resources like abusing pci bridge resource spot on non bridge pci device. Add addon_resources list in pci_dev for those non-standard resources. With this patch, will treat addon-resource like standard resources with special ops. Signed-off-by: Yinghai Lu --- drivers/pci/probe.c | 38 +++++++++++++++++++++++++++++++++++++- include/linux/pci.h | 28 +++++++++++++++++++++++++++- 2 files changed, 64 insertions(+), 2 deletions(-) diff --git a/drivers/pci/probe.c b/drivers/pci/probe.c index 263a575..9e659c7 100644 --- a/drivers/pci/probe.c +++ b/drivers/pci/probe.c @@ -107,22 +107,53 @@ postcore_initcall(pcibus_class_init); struct resource *pci_dev_resource_n(struct pci_dev *dev, int n) { - if (n >= 0 && n < PCI_NUM_RESOURCES) + struct pci_dev_addon_resource *addon_res; + + if (n < 0) + return NULL; + + if (n < PCI_NUM_RESOURCES) return &dev->resource[n]; + n -= PCI_NUM_RESOURCES; + list_for_each_entry(addon_res, &dev->addon_resources, list) { + if (n-- == 0) + return &addon_res->res; + } + return NULL; } EXPORT_SYMBOL(pci_dev_resource_n); int pci_dev_resource_idx(struct pci_dev *dev, struct resource *res) { + struct pci_dev_addon_resource *addon_res; + int n; + if (res >= dev->resource && res <= dev->resource + (PCI_NUM_RESOURCES - 1)) return res - dev->resource; + n = PCI_NUM_RESOURCES; + list_for_each_entry(addon_res, &dev->addon_resources, list) { + if (res == &addon_res->res) + return n; + n++; + } + return -1; } +static void pci_release_dev_addon_resource(struct pci_dev *dev) +{ + struct pci_dev_addon_resource *addon_res, *tmp; + + list_for_each_entry_safe(addon_res, tmp, &dev->addon_resources, list) { + list_del(&addon_res->list); + kfree(addon_res); + } +} + static void __init_res_idx_mask(unsigned long *mask, int flag) { bitmap_zero(mask, PCI_NUM_RESOURCES); @@ -167,6 +198,9 @@ int pci_next_resource_idx(int i, int flag) if (i < PCI_NUM_RESOURCES) return i; + if (flag & PCI_ADDON_RES) + return i; + return -1; } EXPORT_SYMBOL(pci_next_resource_idx); @@ -1199,6 +1233,7 @@ static void pci_release_dev(struct device *dev) pci_dev = to_pci_dev(dev); pci_release_capabilities(pci_dev); pci_release_of_node(pci_dev); + pci_release_dev_addon_resource(pci_dev); kfree(pci_dev); } @@ -1276,6 +1311,7 @@ struct pci_dev *alloc_pci_dev(void) return NULL; INIT_LIST_HEAD(&dev->bus_list); + INIT_LIST_HEAD(&dev->addon_resources); return dev; } diff --git a/include/linux/pci.h b/include/linux/pci.h index 044c474..c337c51 100644 --- a/include/linux/pci.h +++ b/include/linux/pci.h @@ -323,6 +323,7 @@ struct pci_dev { */ unsigned int irq; struct resource resource[DEVICE_COUNT_RESOURCE]; /* I/O and memory regions + expansion ROMs */ + struct list_head addon_resources; /* addon I/O and memory resource */ bool match_driver; /* Skip attaching driver */ /* These fields are used by common fixups */ @@ -376,6 +377,21 @@ struct pci_dev { size_t romlen; /* Length of ROM if it's not from the BAR */ }; +struct resource_ops { + int (*read)(struct pci_dev *dev, struct resource *res, int addr); + int (*write)(struct pci_dev *dev, struct resource *res, int addr); +}; + +struct pci_dev_addon_resource { + struct list_head list; + int reg_addr; + int size; + struct resource res; + struct resource_ops *ops; +}; +#define to_pci_dev_addon_resource(n) \ + container_of(n, struct pci_dev_addon_resource, res) + struct resource *pci_dev_resource_n(struct pci_dev *dev, int n); int pci_dev_resource_idx(struct pci_dev *dev, struct resource *res); @@ -384,8 +400,10 @@ int pci_dev_resource_idx(struct pci_dev *dev, struct resource *res); #define PCI_IOV_RES (1<<2) #define PCI_BRIDGE_RES (1<<3) #define PCI_RES_BLOCK_NUM 4 +/* addon does not need to use bitmap. */ +#define PCI_ADDON_RES (1<<4) -#define PCI_ALL_RES (PCI_STD_RES | PCI_ROM_RES | PCI_BRIDGE_RES | PCI_IOV_RES) +#define PCI_ALL_RES (PCI_STD_RES | PCI_ROM_RES | PCI_BRIDGE_RES | PCI_IOV_RES | PCI_ADDON_RES) #define PCI_ROM_IOV_BRIDGE_RES (PCI_ROM_RES | PCI_BRIDGE_RES | PCI_IOV_RES) #define PCI_STD_ROM_BRIDGE_RES (PCI_STD_RES | PCI_ROM_RES | PCI_BRIDGE_RES) #define PCI_STD_IOV_BRIDGE_RES (PCI_STD_RES | PCI_IOV_RES | PCI_BRIDGE_RES) @@ -393,6 +411,14 @@ int pci_dev_resource_idx(struct pci_dev *dev, struct resource *res); #define PCI_STD_ROM_RES (PCI_STD_RES | PCI_ROM_RES) #define PCI_STD_IOV_RES (PCI_STD_RES | PCI_IOV_RES) #define PCI_STD_ROM_IOV_RES (PCI_STD_RES | PCI_ROM_RES | PCI_IOV_RES) +/* try to treat add on as std */ +#define PCI_STD_ADDON_RES (PCI_STD_RES | PCI_ADDON_RES) +#define PCI_STD_ROM_BRIDGE_ADDON_RES (PCI_STD_RES | PCI_ROM_RES | PCI_BRIDGE_RES | PCI_ADDON_RES) +#define PCI_STD_IOV_BRIDGE_ADDON_RES (PCI_STD_RES | PCI_IOV_RES | PCI_BRIDGE_RES | PCI_ADDON_RES) +#define PCI_STD_ROM_IOV_ADDON_RES (PCI_STD_RES | PCI_ROM_RES | PCI_IOV_RES | PCI_ADDON_RES) +#define PCI_STD_ROM_ADDON_RES (PCI_STD_RES | PCI_ROM_RES | PCI_ADDON_RES) +#define PCI_STD_IOV_ADDON_RES (PCI_STD_RES | PCI_IOV_RES | PCI_ADDON_RES) +#define PCI_STD_ROM_IOV_ADDON_RES (PCI_STD_RES | PCI_ROM_RES | PCI_IOV_RES | PCI_ADDON_RES) int pci_next_resource_idx(int i, int flag);