From patchwork Fri Aug 31 16:44:20 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yinghai Lu X-Patchwork-Id: 180996 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 72F3F2C0374 for ; Sat, 1 Sep 2012 02:44:23 +1000 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754323Ab2HaQoW (ORCPT ); Fri, 31 Aug 2012 12:44:22 -0400 Received: from mail-vc0-f174.google.com ([209.85.220.174]:48857 "EHLO mail-vc0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754203Ab2HaQoV (ORCPT ); Fri, 31 Aug 2012 12:44:21 -0400 Received: by vcbfk26 with SMTP id fk26so3500670vcb.19 for ; Fri, 31 Aug 2012 09:44:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type; bh=U5Kc3wBBbe2WWbuuIjcvz5o+cY8NK5L2mPzP2egwfRM=; b=Fz8dHQFgRrR5fuLGxtjuikuz/NwjTj1BDNherk0D4r2xBpZ17nypaFiJTRy8oLZdS+ TpJoSLwG9u6VK7g/+JDLRHo9T34/R4W+QHHkCAjzKgtfCLGqxwbTBo9AIDWluKuJtc5u oEBzmLKwwS5kMKWUq8Hb9Lt6X6KRfVBePdD+D7C2JRmxC4AgaFzjPzNO3NDV7dkmuOY+ PIBczsQOORqHUAv/zaRk2elBJ9dVkQlF87wjsbsYXyQlyu6jjzqWpiCCzgOBYZiA31y2 6FIxfGYSYKXJ1agN0xSfQfVMUKqrUyGYLCPrwKjBjzAR41cEnkmk8JAOpAIE4UCKGzyu hWgg== MIME-Version: 1.0 Received: by 10.52.21.82 with SMTP id t18mr5067021vde.66.1346431460431; Fri, 31 Aug 2012 09:44:20 -0700 (PDT) Received: by 10.52.170.73 with HTTP; Fri, 31 Aug 2012 09:44:20 -0700 (PDT) In-Reply-To: References: <20120810150955.e4ab3c7f.izumi.taku@jp.fujitsu.com> <50400D6E.1050108@huawei.com> <5040494E.8080501@huawei.com> Date: Fri, 31 Aug 2012 09:44:20 -0700 X-Google-Sender-Auth: OLIRJOgOxy8kWg4fJAn4Ekqalss Message-ID: Subject: Re: [PATCH 0/7][RESEND] acpi, pci: hostbridge hotplug support From: Yinghai Lu To: Bjorn Helgaas Cc: Jiang Liu , Taku Izumi , linux-pci@vger.kernel.org, linux-acpi@vger.kernel.org, kaneshige.kenji@jp.fujitsu.com Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org On Thu, Aug 30, 2012 at 10:42 PM, Bjorn Helgaas wrote: > > That's true. I think we should blindly probe for devices only when > (1) the system does not have ACPI or (2) the DMI BIOS date is older > than some date, e.g., 1 Jan 2013. For new systems with ACPI, we > should only look at the host bridges described in ACPI. if those buses do not get scanned, then it should be enough. how about just removing pcibios_last_bus probing code? also Nehalem *EX* and Westmere *EX* does not support EDAC. so if the user with Nehalem/Westmere Desktop version, want to use to EDAC, they could just append pci=last_bus=255 to get those buses scanned. Yinghai --- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/arch/x86/pci/mmconfig-shared.c b/arch/x86/pci/mmconfig-shared.c index 937bcec..3f3c460 100644 --- a/arch/x86/pci/mmconfig-shared.c +++ b/arch/x86/pci/mmconfig-shared.c @@ -616,16 +616,6 @@ static void __init __pci_mmcfg_init(int early) if (list_empty(&pci_mmcfg_list)) return; - if (pcibios_last_bus < 0) { - const struct pci_mmcfg_region *cfg; - - list_for_each_entry(cfg, &pci_mmcfg_list, list) { - if (cfg->segment) - break; - pcibios_last_bus = cfg->end_bus; - } - } - if (pci_mmcfg_arch_init()) pci_probe = (pci_probe & ~PCI_PROBE_MASK) | PCI_PROBE_MMCONF; else {