https://bugzilla.kernel.org/show_bug.cgi?id=206553

--- Comment #20 from Jan Engelhardt (ej+...@inai.de) ---
Your patch is faulty;

1. AE_TIME is emitted from the function, which is incorrect. We want to execute
a similar pattern such as the "Make sure that a global lock actually exists."
branch a few lines above.

1b. The caller at exfield.c:394 (see comment 2) ignores the AE_TIME return
value, which causes some more problems down the line because things pretend the
lock was taken when it's not. Eventually the system hangs again at some point.

2. Due to "break;", the acpi_os_acquire_lock call is skipped but not the
acpi_os_release_lock after the loop. This leads to a release of a mutex that is
not locked.

New patch follows.

-- 
You are receiving this mail because:
You are watching the assignee of the bug.

_______________________________________________
acpi-bugzilla mailing list
acpi-bugzilla@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla

Reply via email to