Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 4 Dec 2012 09:50:00 +0100
From:      Stefan Farfeleder <stefanf@FreeBSD.org>
To:        Andriy Gapon <avg@FreeBSD.org>
Cc:        freebsd-acpi@FreeBSD.org
Subject:   Re: ACPI panic
Message-ID:  <20121204085000.GA1445@mole.fafoe.narf.at>
In-Reply-To: <50B7D717.4030402@FreeBSD.org>
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>

next in thread | previous in thread | raw e-mail | index | archive | help
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



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20121204085000.GA1445>