From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steven Newbury Subject: Re: [PATCH 1/2] ACPI / scan: Make namespace scanning and trimming mutually exclusive Date: Sat, 02 Feb 2013 20:28:01 +0000 Message-ID: <1359836881.2356.2.camel@Nokia-N900> References: <1581701.fnbebHrKch@vostro.rjw.lan> <1359806321.30277.3.camel@infinity> <1750337.zfO6ANFbLh@vostro.rjw.lan> Reply-To: Steven Newbury Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from mtaout01-winn.ispmail.ntl.com ([81.103.221.47]:32647 "EHLO mtaout01-winn.ispmail.ntl.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752172Ab3BBUZR (ORCPT ); Sat, 2 Feb 2013 15:25:17 -0500 In-Reply-To: <1750337.zfO6ANFbLh@vostro.rjw.lan> Content-ID: <1359836880.2356.1.camel@Nokia-N900> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: "Rafael J. Wysocki" Cc: Yinghai Lu , ACPI Devel Maling List , LKML , Bjorn Helgaas , Jiang Liu , Toshi Kani On Sat,=C2=A0 2 Feb 2013, 20:18:28 GMT, Rafael J. Wysocki wrote: > On Saturday, February 02, 2013 11:58:41 AM Steven Newbury wrote: > > On Sat, 2013-01-26 at 15:19 -0800, Yinghai Lu wrote: > > > On Sat, Jan 26, 2013 at 2:41 PM, Rafael J. Wysocki > > > wrote: > > > > From: Rafael J. Wysocki > > > >=20 > > > > There is no guarantee that acpi_bus_scan() and acpi_bus_trim() = will > > > > not be run in parallel for the same scope of the ACPI namespace= , > > > > which may lead to a great deal of confusion, so introduce a new > > > > mutex to prevent that from happening. > > > >=20 > > > > Signed-off-by: Rafael J. Wysocki > > >=20 > > > Acked-by: Yinghai Lu > > >=20 > > > Steven, > > >=20 > > > Can you apply this one to for-pci-res-alloc to check if racing wi= th > > > docking hotplug/eject > > > still happen? > > > or wait one or two days after i rebase that branch. > >=20 > > Tried merging with linux-pm/bleeding-edge, same behaviour: > >=20 > > [ 3589.013578] ACPI: \_SB_.PCI0.PCIE.GDCK: undocking > > [ 3589.585356] vgaarb: device changed decodes: > > PCI:0000:00:02.0,olddecodes=3Dnone,decodes=3Dio+mem:owns=3Dnone [ > > 3589.585422] ACPI: Delete PCI Interrupt Routing Table for 0000:04 [ > > 3589.585426] pci 0000:03:08.0: Oops, 'acpi_handle' corrupt [ > > 3589.585446] pci_bus 0000:04: busn_res: [bus 04] is released > >=20 > > 03:08.0 PCI bridge: PLX Technology, Inc. PEX8112 x1 Lane PCI > > Express-to-PCI Bridge (rev aa) > > 04:00.0 VGA compatible controller: Advanced Micro Devices [AMD] nee= ATI > > Manhattan [Mobility Radeon HD 5430 Series] > > 04:00.1 Audio device: Advanced Micro Devices [AMD] nee ATI Cedar HD= MI > > Audio [Radeon HD 5400/6300 Series] >=20 > That's because of a bug in the dock code I believe. >=20 > If you're willing to test patches, I can try to cook up something to > debug/fix this. Sure, I'm always willing to test patches. -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html