This message generated a parse failure. Raw output follows here. Please use 'back' to navigate. From devnull@lkml.org Fri Apr 19 17:44:49 2024 Envelope-to: j@jasper.es Delivery-date: Thu, 20 Mar 2014 13:28:58 +0100 Received: from localhost ([127.0.0.1] helo=squeeze.vs19.net) by squeeze.vs19.net with esmtp (Exim 4.80) (envelope-from ) id 1WQc5h-0005Ij-CS for j@jasper.es; Thu, 20 Mar 2014 13:28:58 +0100 Original-Recipient: rfc822;jasper@telfort.nl Received: from pop3.telfort.nl [213.75.3.52] by squeeze.vs19.net with POP3 (fetchmail-6.3.21) for (single-drop); Thu, 20 Mar 2014 13:28:57 +0100 (CET) Received: from cpxmta-msg11.kpnxchange.com (10.94.114.32) by cpxmbs-msg01.support.local (8.6.060.31) id 53186C40009F4AD9 for jasper@telfort.nl; Thu, 20 Mar 2014 13:14:10 +0100 Received: from cpsmtpb-ews08.kpnxchange.com (213.75.39.13) by cpxmta-msg11.kpnxchange.com (8.6.060.14) id 53168AEB01F4EDF8 for jasper@telfort.nl; Thu, 20 Mar 2014 13:14:10 +0100 Received: from cpsps-ews30.kpnxchange.com ([10.94.84.196]) by cpsmtpb-ews08.kpnxchange.com with Microsoft SMTPSVC(7.5.7601.17514); Thu, 20 Mar 2014 13:14:10 +0100 Received: from vger.kernel.org ([209.132.180.67]) by cpsps-ews30.kpnxchange.com with Microsoft SMTPSVC(7.5.7601.17514); Thu, 20 Mar 2014 13:14:10 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932977AbaCTMOH convert rfc822-to-quoted-printable (ORCPT ); Thu, 20 Mar 2014 08:14:07 -0400 Received: from v094114.home.net.pl ([79.96.170.134]:64352 "HELO v094114.home.net.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S932681AbaCTMN6 convert rfc822-to-8bit (ORCPT ); Thu, 20 Mar 2014 08:13:58 -0400 Received: from afkh37.neoplus.adsl.tpnet.pl [178.42.7.37] (HELO vostro.rjw.lan) by serwer1319399.home.pl [79.96.170.134] with SMTP (IdeaSmtpServer v0.80) id 41a8e19b223ea915; Thu, 20 Mar 2014 13:13:56 +0100 From: "Rafael J. Wysocki" To: "Zhang, Rui" Cc: "Lu, Aaron" , Borislav Petkov , lkml , "x86@kernel.org" , Bjorn Helgaas , Linux PCI , ACPI Devel Maling List User-Agent: KMail/4.11.5 (Linux/3.14.0-rc6+; KDE/4.11.5; x86_64; ; ) In-Reply-To: <744357E9AAD1214791ACBA4B0B9092630121F201@SHSMSX101.ccr.corp.intel.com> References: <20140224162400.GE16457@pd.tnic> <532A514C.5010106@intel.com> <744357E9AAD1214791ACBA4B0B9092630121F201@SHSMSX101.ccr.corp.intel.com> Mime-Version: 1.0 Content-Transfer-Encoding: QUOTED-PRINTABLE Content-Type: text/plain; charset=utf-8 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-Id: X-Mailing-List: linux-kernel@vger.kernel.org X-OriginalArrivalTime: 20 Mar 2014 12:14:10.0500 (UTC) FILETIME=[E6502040:01CF4435] X-RcptDomain: telfort.nl On Thursday, March 20, 2014 03:03:45 AM Zhang, Rui wrote: >=20 > > -----Original Message----- > > From: Lu, Aaron > > Sent: Thursday, March 20, 2014 10:24 AM > > To: Rafael J. Wysocki; Borislav Petkov > > Cc: lkml; x86@kernel.org; Bjorn Helgaas; Linux PCI; ACPI Devel Mali= ng > > List; Zhang, Rui; Yinghai Lu; H. Peter Anvin; Stephane Eranian > > Subject: Re: Info: mapping multiple BARs. Your kernel is fine. > > Importance: High > >=20 > > On 03/15/2014 10:15 PM, Rafael J. Wysocki wrote: > > > [CC list rearranged] > > > > > > On Monday, February 24, 2014 05:24:00 PM Borislav Petkov wrote: > > >> This started happening this morning after booting -rc4+tip, let'= s > > add > > >> *everybody* to CC :-) > > >> > > >> We have intel_uncore_init, snb_uncore_imc_init_box, uncore_pci_p= robe > > >> and other goodies on the stack. > > > > > > I've just gone throught this. > > > > > > So the problem is that we have the PNP "system" driver whose only > > > purpose seems to be to reserve system resources so that the PCI l= ayer > > > doesn't assign them to new devices on hotplug (disclaimer: I didn= 't > > > invent it, I only read the code and comments in there). > >=20 > > And to PCI devices which have uninitialized BARs. > >=20 > > > > > > It does that for ACPI device objects having the "PNP0C02" and > > "PNP0C01" IDs. > > > > > > Apparently, snb_uncore_imc_init_box() steps on a range already > > > reserved by that driver on your box. And this doesn't seem to be= a > > > coincidence, because the ACPI device object in question probably > > > *does* correspond to the memory controller that the uncore driver > > attempts to use. > > > > > > I'm not sure how to address that right now to be honest. Arguabl= y, > > > the PNP "system" driver should be replaced with something saner, = but > > > still the resources it claims need to be kept out of reach of the > > > PCI's resource allocation code. > >=20 > > The quirk_system_pci_resources is meant to disable PNP devices' > > resource if they collide with any known PCI device's BAR. I'm not s= ure > > why it doesn't work here, perhaps the uncore PCI device doesn't hav= e a > > BAR that falls in the PNP device's resource window? > > > I've talked with Yan Zheng, and I was told that this resource "0xfed1= 0000 - 0xfed15fff" > is got from PCI device register directly, which is not in its BAR ran= ge. > Thus IMO, it is impossible for PNP layer to be aware of this resource= =2E >=20 > BTW, about drivers/pnp/system.c, if its ONLY purpose is to prevent th= ose > resources from being allocated to uninitialized PCI devices, then IMO= , > the best way to do this is make PCI bus handle those PNP0C01/PNP0C02 > resources directly, probably via a platform callback, say, > 1. make drivers/pnp/system.c a no-op for PNPACPI, by checking pnp_dev= ->protocol. Then we can drop drivers/pnp/system.c entirely I think. > 2. introduce acpi_check_reserved_resource() to parsing PNP0C01/PNP0C0= 2 resources. > 3. in PCI bus, invoke acpi_check_reserved_resource() when assigning > resources to PCI devices. Well, sounds reasonable. > > > > > >> ... > > >> [ 0.488998] software IO TLB [mem 0xcac30000-0xcec30000] (64MB= ) > > mapped at [ffff8800cac30000-ffff8800cec2ffff] > > >> [ 0.489975] resource map sanity check conflict: 0xfed10000 > > 0xfed15fff 0xfed10000 0xfed13fff pnp 00:01 > > >> [ 0.490079] ------------[ cut here ]------------ > > >> [ 0.490204] WARNING: CPU: 2 PID: 1 at arch/x86/mm/ioremap.c:1= 71 > > __ioremap_caller+0x372/0x380() > > >> [ 0.490306] Info: mapping multiple BARs. Your kernel is fine. > > >> [ 0.490371] Modules linked in: > > >> [ 0.490558] CPU: 2 PID: 1 Comm: swapper/0 Not tainted 3.14.0-= rc4+ > > #1 > > >> [ 0.490642] Hardware name: LENOVO 2320CTO/2320CTO, BIOS G2ET8= 6WW > > (2.06 ) 11/13/2012 > > >> [ 0.490742] 00000000000000ab ffff880213d01ad8 ffffffff816112= e3 > > 0000000000000006 > > >> [ 0.491032] ffff880213d01b28 ffff880213d01b18 ffffffff8104e9= bc > > ffff880213d01b08 > > >> [ 0.491343] ffffc90000c58000 00000000fed10000 00000000fed100= 00 > > 0000000000006000 > > >> [ 0.491631] Call Trace: > > >> [ 0.493337] [] dump_stack+0x4f/0x7c > > >> [ 0.493420] [] warn_slowpath_common+0x8c/0= xc0 > > >> [ 0.493503] [] warn_slowpath_fmt+0x46/0x50 > > >> [ 0.493588] [] __ioremap_caller+0x372/0x38= 0 > > >> [ 0.493674] [] ? > > snb_uncore_imc_init_box+0x62/0x90 > > >> [ 0.493761] [] ioremap_nocache+0x17/0x20 > > >> [ 0.493846] [] > > snb_uncore_imc_init_box+0x62/0x90 > > >> [ 0.493933] [] uncore_pci_probe+0xe5/0x1e0 > > >> [ 0.494020] [] local_pci_probe+0x4e/0xa0 > > >> [ 0.494104] [] ? get_device+0x19/0x20 > > >> [ 0.494213] [] pci_device_probe+0xe1/0x130 > > >> [ 0.494300] [] driver_probe_device+0x7b/0x= 240 > > >> [ 0.494385] [] __driver_attach+0xab/0xb0 > > >> [ 0.494469] [] ? > > driver_probe_device+0x240/0x240 > > >> [ 0.494551] [] bus_for_each_dev+0x5e/0x90 > > >> [ 0.494634] [] driver_attach+0x1e/0x20 > > >> [ 0.494718] [] bus_add_driver+0x117/0x230 > > >> [ 0.494802] [] driver_register+0x64/0xf0 > > >> [ 0.494884] [] __pci_register_driver+0x64/= 0x70 > > >> [ 0.494972] [] ? uncore_types_init+0x19c/0= x19c > > >> [ 0.495056] [] intel_uncore_init+0x177/0x4= 1c > > >> [ 0.495155] [] ? uncore_types_init+0x19c/0= x19c > > >> [ 0.495242] [] do_one_initcall+0x4e/0x170 > > >> [ 0.495326] [] ? parse_args+0x60/0x360 > > >> [ 0.495411] [] > > kernel_init_freeable+0x106/0x19a > > >> [ 0.495497] [] ? do_early_param+0x86/0x86 > > >> [ 0.495582] [] ? rest_init+0xd0/0xd0 > > >> [ 0.495666] [] kernel_init+0xe/0xf0 > > >> [ 0.495749] [] ret_from_fork+0x7c/0xb0 > > >> [ 0.495831] [] ? rest_init+0xd0/0xd0 > > >> [ 0.495921] ---[ end trace 428f365c054d9a01 ]--- > > >> [ 0.496196] RAPL PMU detected, hw unit 2^-16 Joules, API unit= is > > 2^-32 Joules, 3 fixed counters 163840 ms ovfl timer > > >> [ 0.498598] futex hash table entries: 1024 (order: 5, 131072 > > bytes) > > >> [ 0.498833] audit: initializing netlink subsys (disabled) > > >> [ 0.499024] audit: type=3D2000 audit(1393259866.477:1): initi= alized > > >> ... > > >> > > >> > > > >=20 > N=EF=BF=BD=EF=BF=BD=EF=BF=BD=EF=BF=BD=EF=BF=BDr=EF=BF=BD=EF=BF=BDy=EF= =BF=BD=EF=BF=BD=EF=BF=BD=EF=BF=BDb=EF=BF=BDX=EF=BF=BD=EF=BF=BD=C7=A7v=EF= =BF=BD^=EF=BF=BD)=DE=BA{.n=EF=BF=BD+=EF=BF=BD=EF=BF=BD=EF=BF=BD=EF=BF=BD= {=EF=BF=BDi=EF=BF=BDb=EF=BF=BD{ay=EF=BF=BD=1D=CA=87=DA=99=EF=BF=BD,j=07= =EF=BF=BD=EF=BF=BDf=EF=BF=BD=EF=BF=BD=EF=BF=BDh=EF=BF=BD=EF=BF=BD=EF=BF= =BDz=EF=BF=BD=1E=EF=BF=BDw=EF=BF=BD=EF=BF=BD=EF=BF=BD=0C=EF=BF=BD=EF=BF= =BD=EF=BF=BDj:+v=EF=BF=BD=EF=BF=BD=EF=BF=BDw=EF=BF=BDj=EF=BF=BDm=EF=BF=BD= =EF=BF=BD=EF=BF=BD=EF=BF=BD=07=EF=BF=BD=EF=BF=BD=EF=BF=BD=EF=BF=BDzZ+=EF= =BF=BD=EF=BF=BD=EF=BF=BD=EF=BF=BD=EF=BF=BD=DD=A2j"=EF=BF=BD=EF=BF=BD!=EF= =BF=BDir --=20 I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel"= in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/