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





--- Comment #25 from Zhang Rui <rui.zh...@intel.com>  2010-05-19 08:27:09 ---
acpi_ev_asynch_enable_gpe enables the GPE if gpe_register_info->enable_for_run
is not cleared, so I guess we can not disable the GPE storm because it's
enabled by the deferred GPE handler.

IMO, this sounds like a problem in the gpe reference counting patch.
Say, we don't update the runtime_count/wakeup_count when invoking acpi_set_gpe.
so in order to make acpi_set_gpe work properly, we can not use
runtime_count/wakeup_count to update the
gpe_register_info->enable_for_run/enable_for_wake flag, as we do in
acpi_ev_update_gpe_enable_masks currently.

But I'm still wondering why patch-1 breaks patch-2...

-- 
Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

------------------------------------------------------------------------------

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

Reply via email to