Patch Detail

HTTP 200 OK
Allow: GET, PUT, PATCH, HEAD, OPTIONS
Content-Type: application/json
Vary: Accept

{
    "id": 476452,
    "url": "http://patchwork.ozlabs.org/api/patches/476452/?format=api",
    "web_url": "http://patchwork.ozlabs.org/project/linux-pci/patch/1432644564-24746-11-git-send-email-hanjun.guo@linaro.org/",
    "project": {
        "id": 28,
        "url": "http://patchwork.ozlabs.org/api/projects/28/?format=api",
        "name": "Linux PCI development",
        "link_name": "linux-pci",
        "list_id": "linux-pci.vger.kernel.org",
        "list_email": "linux-pci@vger.kernel.org",
        "web_url": null,
        "scm_url": null,
        "webscm_url": null,
        "list_archive_url": "",
        "list_archive_url_format": "",
        "commit_url_format": ""
    },
    "msgid": "<1432644564-24746-11-git-send-email-hanjun.guo@linaro.org>",
    "list_archive_url": null,
    "date": "2015-05-26T12:49:23",
    "name": "[10/11] XEN / PCI: Remove the dependence on arch x86 when PCI_MMCONFIG=y",
    "commit_ref": null,
    "pull_url": null,
    "state": "changes-requested",
    "archived": false,
    "hash": "3a3ed21292c6a70676e27174c49e2006e493ae1a",
    "submitter": {
        "id": 47236,
        "url": "http://patchwork.ozlabs.org/api/people/47236/?format=api",
        "name": "Hanjun Guo",
        "email": "hanjun.guo@linaro.org"
    },
    "delegate": null,
    "mbox": "http://patchwork.ozlabs.org/project/linux-pci/patch/1432644564-24746-11-git-send-email-hanjun.guo@linaro.org/mbox/",
    "series": [],
    "comments": "http://patchwork.ozlabs.org/api/patches/476452/comments/",
    "check": "pending",
    "checks": "http://patchwork.ozlabs.org/api/patches/476452/checks/",
    "tags": {},
    "related": [],
    "headers": {
        "Return-Path": "<linux-pci-owner@vger.kernel.org>",
        "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])\n\tby ozlabs.org (Postfix) with ESMTP id BF27D140129\n\tfor <incoming@patchwork.ozlabs.org>;\n\tTue, 26 May 2015 23:19:36 +1000 (AEST)",
            "(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1753009AbbEZMxB (ORCPT <rfc822;incoming@patchwork.ozlabs.org>);\n\tTue, 26 May 2015 08:53:01 -0400",
            "from mail-pd0-f178.google.com ([209.85.192.178]:36461 \"EHLO\n\tmail-pd0-f178.google.com\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S1752981AbbEZMw7 (ORCPT\n\t<rfc822; linux-pci@vger.kernel.org>); Tue, 26 May 2015 08:52:59 -0400",
            "by pdfh10 with SMTP id h10so90179593pdf.3\n\tfor <linux-pci@vger.kernel.org>; Tue, 26 May 2015 05:50:53 -0700 (PDT)",
            "from localhost ([180.150.153.56]) by mx.google.com with ESMTPSA id\n\tfs16sm13066192pdb.12.2015.05.26.05.50.51\n\t(version=TLSv1.2 cipher=RC4-SHA bits=128/128);\n\tTue, 26 May 2015 05:50:52 -0700 (PDT)"
        ],
        "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20130820;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to\n\t:references;\n\tbh=6ZZehvbpiGvErEF59suz1kivyLM0ApS4D2ix9Xgkmq4=;\n\tb=LQ1ARWoCdnR1P76Fw9an4P7xgstGgKbvmTmyU+gOEQ4/L0K+wN5xBB17LJydSXMA1c\n\t862MKZXpPzDu0SyrY6aNP7I7UNfE10mRzI/lF/+SR6KwNjdhAf5l56Mde6XLTcvHcUaJ\n\tbYwJ+VyHKvFk4vfgyfDScGKVmEGHOEPftzoKH/XRCyH7NsNcQOpIEDFPsTAOYmpt6wop\n\tbTC1UoIkk8EP3KFJ1NJvkktr1yrA+D2ttaXg8XzZzbntLJ3s7wcwlSm2QM8SoZ0PjaGb\n\tMD/cpxIRDjIG8nO4OhuFTGGsEX9/q8oPo/zstGmyT84HDHgT3X7eJbGHTV7ImIEzswjM\n\tqiGA==",
        "X-Gm-Message-State": "ALoCoQmLapOLVWJZQ/VpdYClw9OAqa8tbKNUCwrWb5vh8Z7ifZDoGhiBFoVX8fiObXbpnSt/ylTl",
        "X-Received": "by 10.68.94.193 with SMTP id de1mr48191050pbb.153.1432644653260; \n\tTue, 26 May 2015 05:50:53 -0700 (PDT)",
        "From": "Hanjun Guo <hanjun.guo@linaro.org>",
        "To": "Bjorn Helgaas <bhelgaas@google.com>, Arnd Bergmann <arnd@arndb.de>,\n\tCatalin Marinas <catalin.marinas@arm.com>,\n\tWill Deacon <will.deacon@arm.com>,\n\t\"Rafael J. Wysocki\" <rjw@rjwysocki.net>",
        "Cc": "Jiang Liu <jiang.liu@linux.intel.com>, Liviu Dudau <Liviu.Dudau@arm.com>,\n\tThomas Gleixner <tglx@linutronix.de>,\n\tYijing Wang <wangyijing@huawei.com>,\n\tLorenzo Pieralisi <Lorenzo.Pieralisi@arm.com>,\n\tTomasz Nowicki <tomasz.nowicki@linaro.org>,\n\tSuravee Suthikulpanit <Suravee.Suthikulpanit@amd.com>,\n\tMark Salter <msalter@redhat.com>, linux-pci@vger.kernel.org,\n\tlinux-arm-kernel@lists.infradead.org, linux-acpi@vger.kernel.org,\n\tlinux-kernel@vger.kernel.org, linaro-acpi@lists.linaro.org,\n\tHanjun Guo <hanjun.guo@linaro.org>,\n\tKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>,\n\tBoris Ostrovsky <boris.ostrovsky@oracle.com>",
        "Subject": "[PATCH 10/11] XEN / PCI: Remove the dependence on arch x86 when\n\tPCI_MMCONFIG=y",
        "Date": "Tue, 26 May 2015 20:49:23 +0800",
        "Message-Id": "<1432644564-24746-11-git-send-email-hanjun.guo@linaro.org>",
        "X-Mailer": "git-send-email 1.9.1",
        "In-Reply-To": "<1432644564-24746-1-git-send-email-hanjun.guo@linaro.org>",
        "References": "<1432644564-24746-1-git-send-email-hanjun.guo@linaro.org>",
        "Sender": "linux-pci-owner@vger.kernel.org",
        "Precedence": "bulk",
        "List-ID": "<linux-pci.vger.kernel.org>",
        "X-Mailing-List": "linux-pci@vger.kernel.org"
    },
    "content": "In drivers/xen/pci.c, there are arch x86 dependent codes when\nCONFIG_PCI_MMCONFIG is enabled, since CONFIG_PCI_MMCONFIG\ndepends on ACPI, so this will prevent XEN PCI running on other\narchitectures using ACPI with PCI_MMCONFIG enabled (such as ARM64).\n\nFortunatly, it can be sloved in a simple way. In drivers/xen/pci.c,\nthe only x86 dependent code is if ((pci_probe & PCI_PROBE_MMCONF) == 0),\nand it's defined in asm/pci_x86.h, the code means that\nif the PCI resource is not probed in PCI_PROBE_MMCONF way, just\ningnore the xen mcfg init. Actually this is duplicate, because\nif PCI resource is not probed in PCI_PROBE_MMCONF way, the\npci_mmconfig_list will be empty, and the if (list_empty())\nafter it will do the same job.\n\nSo just remove the arch related code and the head file, this\nwill be no functional change for x86, and also makes xen/pci.c\nusable for other architectures.\n\nSigned-off-by: Hanjun Guo <hanjun.guo@linaro.org>\nCC: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>\nCC: Boris Ostrovsky <boris.ostrovsky@oracle.com>\n---\n drivers/xen/pci.c | 6 ------\n 1 file changed, 6 deletions(-)",
    "diff": "diff --git a/drivers/xen/pci.c b/drivers/xen/pci.c\nindex 6785ebb..9a8dbe3 100644\n--- a/drivers/xen/pci.c\n+++ b/drivers/xen/pci.c\n@@ -28,9 +28,6 @@\n #include <asm/xen/hypervisor.h>\n #include <asm/xen/hypercall.h>\n #include \"../pci/pci.h\"\n-#ifdef CONFIG_PCI_MMCONFIG\n-#include <asm/pci_x86.h>\n-#endif\n \n static bool __read_mostly pci_seg_supported = true;\n \n@@ -222,9 +219,6 @@ static int __init xen_mcfg_late(void)\n \tif (!xen_initial_domain())\n \t\treturn 0;\n \n-\tif ((pci_probe & PCI_PROBE_MMCONF) == 0)\n-\t\treturn 0;\n-\n \tif (list_empty(&pci_mmcfg_list))\n \t\treturn 0;\n \n",
    "prefixes": [
        "10/11"
    ]
}