svn commit: r184558 - head/sys/dev/acpica/Osd

Jung-uk Kim jkim at
Mon Nov 3 07:51:09 PST 2008

On Sunday 02 November 2008 07:50 am, Alexander Motin wrote:
> Author: mav
> Date: Sun Nov  2 12:50:16 2008
> New Revision: 184558
> URL:
> Log:
>   As soon as we have several threads per process now, it is not
> correct to use process ID as ACPI thread ID. Concurrent requests
> with equal thread IDs broke ACPI mutexes operation causing
> unpredictable errors including AE_AML_MUTEX_NOT_ACQUIRED that I
> have seen.
>   Use kernel thread ID instead of process ID for ACPI thread.

Sorry but this patch is incorrect, i.e., td_tid is not unique.  You 
have to use curthread or (p_pid, td_tid) pair.  Unfortunately, even 
if you correct this problem, you also have to correct ACPI_THREAD_ID 
definition, which is in the vendor code.  That's why it wasn't done 
yet and it is more complicated than you think, i.e., ACPI-CA assumes 
sizeof(ACPI_THREAD_ID) == sizeof(int), etc.  Please see the related 
ACPI-CA bugs:

In fact, I have been maintaining patchsets with the fix here:<ACPI_RELEASE>.diff.gz

Please revert this commit until we resolve these issues with the 
vendor first.


Jung-uk Kim

More information about the svn-src-head mailing list