From patchwork Sun Jun 21 17:55:55 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jiang Liu X-Patchwork-Id: 487071 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 D261414018C for ; Mon, 22 Jun 2015 03:56:17 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753260AbbFUR4C (ORCPT ); Sun, 21 Jun 2015 13:56:02 -0400 Received: from mga02.intel.com ([134.134.136.20]:14494 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752729AbbFUR4A (ORCPT ); Sun, 21 Jun 2015 13:56:00 -0400 Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga101.jf.intel.com with ESMTP; 21 Jun 2015 10:55:59 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,655,1427785200"; d="scan'208";a="592108905" Received: from xgeng1-mobl1.ccr.corp.intel.com (HELO [10.255.27.211]) ([10.255.27.211]) by orsmga003.jf.intel.com with ESMTP; 21 Jun 2015 10:55:57 -0700 Message-ID: <5586FAAB.3050303@linux.intel.com> Date: Mon, 22 Jun 2015 01:55:55 +0800 From: Jiang Liu Organization: Intel User-Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Boszormenyi Zoltan , Bjorn Helgaas CC: Andreas Mohr , "Rafael J. Wysocki" , Linux Kernel Mailing List , ACPI Devel Maling List , "linux-pci@vger.kernel.org" Subject: Re: ACPI regression? Was Re: Ethernet chip disappeared from lspci References: <55841815.5000701@pr.hu> <558419B2.7010703@pr.hu> <55841D48.8080809@pr.hu> <12950452.K8inU2UIYe@vostro.rjw.lan> <55869329.4040908@pr.hu> <5586C7E2.9070902@pr.hu> <5586F371.2040100@linux.intel.com> In-Reply-To: <5586F371.2040100@linux.intel.com> Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org On 2015/6/22 1:25, Jiang Liu wrote: [...] >>>> - Memory behind bridge: 80000000-801fffff >>>> - Prefetchable memory behind bridge: 0000000080200000-00000000803fffff >>>> + Memory behind bridge: ff000000-ff1fffff >>>> + Prefetchable memory behind bridge: 00000000ff200000-00000000ff3fffff >>>> >>>> Can't this cause a problem? E.g. programming the bridge with an address range >>>> that the bridge doesn't actually support? >>> This worked in v3.18.16, but not in v4.0.5 or v4.1.0-rc8. You >>> attached a v4.1.0-rc8 dmesg log earlier. Would you mind collecting a >>> v3.18.16 dmesg log, so we can compare them? >> >> I collected all 3 for you to compare them, compressed, attached. >> >> BTW, I browsed git log and found 2ea3d266bab3b497238113b20136f7c3f69ad9c0 >> as suspicious. I will try the 4.0/4.1 kernels with this one reverted. >> >>> >>> These (from the v4.1.0-rc8 dmesg) look wrong, but I'll have to look at >>> the code to see what might be going on: >>> >>> acpi PNP0A08:00: host bridge window expanded to [mem >>> 0x00000000-0xffffffff window]; [mem 0x00000000-0xffffffff window] >>> ignored >>> pci 0000:00:1c.1: can't claim BAR 15 [mem 0xfdf00000-0xfdffffff >>> 64bit pref]: address conflict with PCI Bus 0000:00 [mem >>> 0xf0000000-0xfed8ffff window] >>> >>> Bjorn > Hi Bjorn and Boszormenyi, > From the 3.18 kernel, we got a message: > [ 0.126248] acpi PNP0A08:00: host bridge window > [0x400000000-0xfffffffff] (ignored, not CPU addressable) > And from 4.1.-rc8, we got another message: > [ 0.127051] acpi PNP0A08:00: host bridge window expanded to [mem > 0x00000000-0xffffffff window]; [mem 0x00000000-0xffffffff window] ignored > > That smells like a 32bit overflow or 64bit cut-off issue. Hi Bjorn and Boszormenyi, With v3.18.6, it uses u64 to compare resource ranges. We changed to use resource_size_t with recent changes, and resource_size_t may be u32 or u64 depending on configuration. So resource range [0x400000000-0xfffffffff] may have been cut-off as [0x00000000-0xffffffff], thus cause the trouble. Hi Boszormenyi, Could you please help to try following test patch? against v4.1-rc8? Thanks! Gerry ------------------------------------------------------------------- * For bridges that translate addresses across the bridge, ----------------------------------------------------------------------------- --- To unsubscribe from this list: send the line "unsubscribe linux-pci" in diff --git a/drivers/acpi/resource.c b/drivers/acpi/resource.c index 8244f013f210..d7b8c392c420 100644 --- a/drivers/acpi/resource.c +++ b/drivers/acpi/resource.c @@ -206,6 +206,11 @@ static bool acpi_decode_space(struct resource_win *win, res->start = attr->minimum; res->end = attr->maximum; + if (res->start != attr->minimum || res->end != attr->maximum) { + pr_warn("resource window ([%#llx-%#llx] ignored, not CPU addressable)\n", + attr->minimum, attr->maximum); + return false; + } /*