From patchwork Wed Dec 6 11:35:44 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: tip-bot for Kobe Wu X-Patchwork-Id: 10095667 X-Patchwork-Delegate: bhelgaas@google.com Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 5A4BA60327 for ; Wed, 6 Dec 2017 11:38:17 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 4D7742983B for ; Wed, 6 Dec 2017 11:38:17 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 4233F29CC4; Wed, 6 Dec 2017 11:38:17 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=2.0 tests=BAYES_00,RCVD_IN_DNSWL_HI autolearn=unavailable version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id CF7C92983B for ; Wed, 6 Dec 2017 11:38:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751680AbdLFLiC (ORCPT ); Wed, 6 Dec 2017 06:38:02 -0500 Received: from terminus.zytor.com ([65.50.211.136]:45697 "EHLO terminus.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752282AbdLFLh5 (ORCPT ); Wed, 6 Dec 2017 06:37:57 -0500 Received: from terminus.zytor.com (localhost [127.0.0.1]) by terminus.zytor.com (8.15.2/8.15.2) with ESMTP id vB6BZkNA026084; Wed, 6 Dec 2017 03:35:46 -0800 Received: (from tipbot@localhost) by terminus.zytor.com (8.15.2/8.15.2/Submit) id vB6BZidI026070; Wed, 6 Dec 2017 03:35:44 -0800 Date: Wed, 6 Dec 2017 03:35:44 -0800 X-Authentication-Warning: terminus.zytor.com: tipbot set sender to tipbot@zytor.com using -f From: "tip-bot for Rafael J. Wysocki" Message-ID: Cc: linux-kernel@vger.kernel.org, dave.hansen@linux.intel.com, linux-pci@vger.kernel.org, rafael.j.wysocki@intel.com, mingo@kernel.org, hpa@zytor.com, bhelgaas@google.com, tglx@linutronix.de Reply-To: tglx@linutronix.de, bhelgaas@google.com, hpa@zytor.com, mingo@kernel.org, rafael.j.wysocki@intel.com, linux-pci@vger.kernel.org, dave.hansen@linux.intel.com, linux-kernel@vger.kernel.org In-Reply-To: <3186627.pxZj1QbYNg@aspire.rjw.lan> References: <3186627.pxZj1QbYNg@aspire.rjw.lan> To: linux-tip-commits@vger.kernel.org Subject: [tip:x86/urgent] x86/PCI: Make broadcom_postcore_init() check acpi_disabled Git-Commit-ID: ddec3bdee05b06f1dda20ded003c3e10e4184cab X-Mailer: tip-git-log-daemon Robot-ID: Robot-Unsubscribe: Contact to get blacklisted from these emails MIME-Version: 1.0 Content-Disposition: inline Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Commit-ID: ddec3bdee05b06f1dda20ded003c3e10e4184cab Gitweb: https://git.kernel.org/tip/ddec3bdee05b06f1dda20ded003c3e10e4184cab Author: Rafael J. Wysocki AuthorDate: Fri, 1 Dec 2017 15:08:12 +0100 Committer: Ingo Molnar CommitDate: Wed, 6 Dec 2017 12:27:47 +0100 x86/PCI: Make broadcom_postcore_init() check acpi_disabled acpi_os_get_root_pointer() may return a valid address even if acpi_disabled is set, but the host bridge information from the ACPI tables is not going to be used in that case and the Broadcom host bridge initialization should not be skipped then, So make broadcom_postcore_init() check acpi_disabled too to avoid this issue. Fixes: 6361d72b04d1 (x86/PCI: read Broadcom CNB20LE host bridge info before PCI scan) Reported-by: Dave Hansen Signed-off-by: Rafael J. Wysocki Signed-off-by: Thomas Gleixner Cc: Bjorn Helgaas Cc: Linux PCI Cc: stable@vger.kernel.org Link: https://lkml.kernel.org/r/3186627.pxZj1QbYNg@aspire.rjw.lan Signed-off-by: Ingo Molnar --- arch/x86/pci/broadcom_bus.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/pci/broadcom_bus.c b/arch/x86/pci/broadcom_bus.c index bb461cf..526536c 100644 --- a/arch/x86/pci/broadcom_bus.c +++ b/arch/x86/pci/broadcom_bus.c @@ -97,7 +97,7 @@ static int __init broadcom_postcore_init(void) * We should get host bridge information from ACPI unless the BIOS * doesn't support it. */ - if (acpi_os_get_root_pointer()) + if (!acpi_disabled && acpi_os_get_root_pointer()) return 0; #endif