diff options
author | Bob Moore <robert.moore@intel.com> | 2006-05-12 17:12:00 -0400 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2006-06-14 02:34:48 -0400 |
commit | 958dd242b691f64ab4632b4903dbb1e16fee8269 (patch) | |
tree | ddf4932fb4e3023dd0d1914571f17f2e3b03978d /drivers/acpi/events/evmisc.c | |
parent | b229cf92eee616c7cb5ad8cdb35a19b119f00bc8 (diff) |
ACPI: ACPICA 20060512
Replaced the acpi_os_queue_for_execution() with a new
interface named acpi_os_execute(). The major difference is
that the new interface does not have a Priority parameter,
this appeared to be useless and has been replaced by
a Type parameter. The Type tells the OS what type of
execution is being requested, such as global lock handler,
notify handler, GPE handler, etc. This allows the host
to queue and execute the request as appropriate for the
request type, possibly using different work queues and
different priorities for the various request types. This
enables fixes for multithreading deadlock problems such as
http://bugzilla.kernel.org/show_bug.cgi?id=5534
(Alexey Starikovskiy and Bob Moore)
Fixed a possible memory leak associated with the
support for the so-called "implicit return" ACPI
extension. Reported by FreeBSD (Fiodor Suietov)
http://bugzilla.kernel.org/show_bug.cgi?id=6514
Fixed a problem with the Load() operator where a table
load from an operation region could overwrite an internal
table buffer by up to 7 bytes and cause alignment faults
on IPF systems. (With assistance from Luming Yu)
Signed-off-by: Bob Moore <robert.moore@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/acpi/events/evmisc.c')
-rw-r--r-- | drivers/acpi/events/evmisc.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/drivers/acpi/events/evmisc.c b/drivers/acpi/events/evmisc.c index 97e05481aa7c..24e0b8d36f31 100644 --- a/drivers/acpi/events/evmisc.c +++ b/drivers/acpi/events/evmisc.c | |||
@@ -191,9 +191,8 @@ acpi_ev_queue_notify_request(struct acpi_namespace_node * node, | |||
191 | notify_info->notify.value = (u16) notify_value; | 191 | notify_info->notify.value = (u16) notify_value; |
192 | notify_info->notify.handler_obj = handler_obj; | 192 | notify_info->notify.handler_obj = handler_obj; |
193 | 193 | ||
194 | status = acpi_os_queue_for_execution(OSD_PRIORITY_HIGH, | 194 | status = acpi_os_execute(OSL_NOTIFY_HANDLER, |
195 | acpi_ev_notify_dispatch, | 195 | acpi_ev_notify_dispatch, notify_info); |
196 | notify_info); | ||
197 | if (ACPI_FAILURE(status)) { | 196 | if (ACPI_FAILURE(status)) { |
198 | acpi_ut_delete_generic_state(notify_info); | 197 | acpi_ut_delete_generic_state(notify_info); |
199 | } | 198 | } |
@@ -346,9 +345,8 @@ static u32 acpi_ev_global_lock_handler(void *context) | |||
346 | 345 | ||
347 | /* Run the Global Lock thread which will signal all waiting threads */ | 346 | /* Run the Global Lock thread which will signal all waiting threads */ |
348 | 347 | ||
349 | status = acpi_os_queue_for_execution(OSD_PRIORITY_HIGH, | 348 | status = acpi_os_execute(OSL_GLOBAL_LOCK_HANDLER, |
350 | acpi_ev_global_lock_thread, | 349 | acpi_ev_global_lock_thread, context); |
351 | context); | ||
352 | if (ACPI_FAILURE(status)) { | 350 | if (ACPI_FAILURE(status)) { |
353 | ACPI_EXCEPTION((AE_INFO, status, | 351 | ACPI_EXCEPTION((AE_INFO, status, |
354 | "Could not queue Global Lock thread")); | 352 | "Could not queue Global Lock thread")); |