From patchwork Sat Jul 2 16:47:12 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bjorn Helgaas X-Patchwork-Id: 940442 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter2.kernel.org (8.14.4/8.14.4) with ESMTP id p62GlOn2030686 for ; Sat, 2 Jul 2011 16:47:24 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752455Ab1GBQrX (ORCPT ); Sat, 2 Jul 2011 12:47:23 -0400 Received: from smtp-out.google.com ([74.125.121.67]:47261 "EHLO smtp-out.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752308Ab1GBQrX (ORCPT ); Sat, 2 Jul 2011 12:47:23 -0400 Received: from kpbe12.cbf.corp.google.com (kpbe12.cbf.corp.google.com [172.25.105.76]) by smtp-out.google.com with ESMTP id p62GlE45006204; Sat, 2 Jul 2011 09:47:15 -0700 DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=google.com; s=beta; t=1309625235; bh=kItvQBkwqEWrHQb1UZEObD7Ixy8=; h=Subject:To:From:Cc:Date:Message-ID:MIME-Version:Content-Type: Content-Transfer-Encoding; b=bMKJnIsp5o3QZk4EzSxNJxb2IU2X50+e+554O0VER5riCKM/51b4P07jAePhrSCwt D06jvXM+s+piMIOO/LpoQ== DomainKey-Signature: a=rsa-sha1; s=beta; d=google.com; c=nofws; q=dns; h=subject:to:from:cc:date:message-id:user-agent: mime-version:content-type:content-transfer-encoding; b=VAo7YfUh0aVuicM8kqXpYLnXuAPSGzqoh9SJ7pOfM067cF2IB/oOC1QRpM4pkt5wE p06+OWrzVDZWtsz2vzUzg== Received: from bhelgaas.mtv.corp.google.com (bhelgaas.mtv.corp.google.com [172.18.96.155]) by kpbe12.cbf.corp.google.com with ESMTP id p62GlCZk016534; Sat, 2 Jul 2011 09:47:13 -0700 Received: from bhelgaas.mtv.corp.google.com (unknown [IPv6:::1]) by bhelgaas.mtv.corp.google.com (Postfix) with ESMTP id A27CD20CDB; Sat, 2 Jul 2011 09:47:12 -0700 (PDT) Subject: [PATCH] x86/PCI: reduce severity of host bridge window conflict warnings To: Jesse Barnes From: Bjorn Helgaas Cc: linux-pci@vger.kernel.org, Das , Jools Wills Date: Sat, 02 Jul 2011 10:47:12 -0600 Message-ID: <20110702164712.21109.66511.stgit@bhelgaas.mtv.corp.google.com> User-Agent: StGit/0.15 MIME-Version: 1.0 Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter2.kernel.org [140.211.167.43]); Sat, 02 Jul 2011 16:47:25 +0000 (UTC) Host bridge windows are top-level resources, so if we find a host bridge window conflict, it's probably with a hard-coded legacy reservation. Moving host bridge windows is theoretically possible, but we don't support it; we just ignore windows with conflicts, and it's not worth making this a user-visible error. Reported-and-tested-by: Jools Wills References: https://bugzilla.kernel.org/show_bug.cgi?id=38522 Reported-by: Das References: https://bugzilla.kernel.org/show_bug.cgi?id=16497 Signed-off-by: Bjorn Helgaas --- arch/x86/pci/acpi.c | 7 +++---- 1 files changed, 3 insertions(+), 4 deletions(-) -- 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/acpi.c b/arch/x86/pci/acpi.c index 68c3c13..ae3cb23 100644 --- a/arch/x86/pci/acpi.c +++ b/arch/x86/pci/acpi.c @@ -246,10 +246,9 @@ static void add_resources(struct pci_root_info *info) conflict = insert_resource_conflict(root, res); if (conflict) - dev_err(&info->bridge->dev, - "address space collision: host bridge window %pR " - "conflicts with %s %pR\n", - res, conflict->name, conflict); + dev_info(&info->bridge->dev, + "ignoring host bridge window %pR (conflicts with %s %pR)\n", + res, conflict->name, conflict); else pci_bus_add_resource(info->bus, res, 0); }