From owner-freebsd-acpi@FreeBSD.ORG Tue Dec 4 08:50:09 2012 Return-Path: Delivered-To: freebsd-acpi@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 9D4BCF1D; Tue, 4 Dec 2012 08:50:09 +0000 (UTC) (envelope-from stefan@fafoe.narf.at) Received: from fep19.mx.upcmail.net (fep19.mx.upcmail.net [62.179.121.39]) by mx1.freebsd.org (Postfix) with ESMTP id 9CBDE8FC0C; Tue, 4 Dec 2012 08:50:08 +0000 (UTC) Received: from edge03.upcmail.net ([192.168.13.238]) by viefep19-int.chello.at (InterMail vM.8.01.05.05 201-2260-151-110-20120111) with ESMTP id <20121204085001.GLSB26940.viefep19-int.chello.at@edge03.upcmail.net>; Tue, 4 Dec 2012 09:50:01 +0100 Received: from mole.fafoe.narf.at ([80.109.55.137]) by edge03.upcmail.net with edge id XLq11k00t2xdvHc03Lq1yj; Tue, 04 Dec 2012 09:50:01 +0100 X-SourceIP: 80.109.55.137 Received: by mole.fafoe.narf.at (Postfix, from userid 1001) id 0CF346D462; Tue, 4 Dec 2012 09:50:01 +0100 (CET) Date: Tue, 4 Dec 2012 09:50:00 +0100 From: Stefan Farfeleder To: Andriy Gapon Subject: Re: ACPI panic Message-ID: <20121204085000.GA1445@mole.fafoe.narf.at> References: <20121125140008.GA1497@mole.fafoe.narf.at> <50B244A1.1040800@FreeBSD.org> <20121126091101.GA1469@mole.fafoe.narf.at> <50B33693.2060000@FreeBSD.org> <20121126093704.GB1469@mole.fafoe.narf.at> <50B34484.1090807@FreeBSD.org> <20121126104737.GC1469@mole.fafoe.narf.at> <50B34EEA.4000209@FreeBSD.org> <20121129084627.GA1450@mole.fafoe.narf.at> <50B7D717.4030402@FreeBSD.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <50B7D717.4030402@FreeBSD.org> User-Agent: Mutt/1.5.21 (2010-09-15) Cc: freebsd-acpi@FreeBSD.org X-BeenThere: freebsd-acpi@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: ACPI and power management development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 04 Dec 2012 08:50:09 -0000 On Thu, Nov 29, 2012 at 11:43:51PM +0200, Andriy Gapon wrote: > Hmmm... > Another bug-catching patch before I start questioning my ability to understand > the code. > > index 59ecf21..1687c75b 100644 > --- a/sys/contrib/dev/acpica/components/utilities/utcache.c > +++ b/sys/contrib/dev/acpica/components/utilities/utcache.c > @@ -238,6 +238,8 @@ AcpiOsReleaseObject ( > > else > { > + if (AcpiGbl_MutexInfo[ACPI_MTX_CACHES].ThreadId == AcpiOsGetThreadId ()) > + panic("ACPI_MTX_CACHES acquired recursively"); > Status = AcpiUtAcquireMutex (ACPI_MTX_CACHES); > if (ACPI_FAILURE (Status)) > { > @@ -311,6 +313,8 @@ AcpiOsAcquireObject ( > return (NULL); > } > > + if (AcpiGbl_MutexInfo[ACPI_MTX_CACHES].ThreadId == AcpiOsGetThreadId ()) > + panic("ACPI_MTX_CACHES acquired recursively"); > Status = AcpiUtAcquireMutex (ACPI_MTX_CACHES); > if (ACPI_FAILURE (Status)) > { These checks have not triggered additional panics. I still get the cycle one. Stefan