If user press the suspend hotkey many times quickly, system will keep entering S3 after wake up as many times as user pressed. We do not expect the system enter S3 again after wake up, even if we press the hotkey many times. This issue comes from the acpi proc event will queue the events that didn't process yet, and then report the events one by one when available, so that system will enter S3 after wake up. I think it's safe not to generate the proc event while pressing the suspend hotkey, since "/proc/acpi/event" is deprecated. And system can enter S3 correctly without it. Signed-off-by: AceLan Kao <acelan.kao@xxxxxxxxxxxxx> --- drivers/acpi/button.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/acpi/button.c b/drivers/acpi/button.c index d27d072..e35bd92 100644 --- a/drivers/acpi/button.c +++ b/drivers/acpi/button.c @@ -298,8 +298,6 @@ static void acpi_button_notify(struct acpi_device *device, u32 event) pm_wakeup_event(&device->dev, 0); } - - acpi_bus_generate_proc_event(device, event, ++button->pushed); break; default: ACPI_DEBUG_PRINT((ACPI_DB_INFO, -- 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
