Bug 201575 - power button sometimes wakeup tablet only after holding it for a few seconds - HP 1013
Summary: power button sometimes wakeup tablet only after holding it for a few seconds ...
Status: CLOSED INSUFFICIENT_DATA
Alias: None
Product: ACPI
Classification: Unclassified
Component: EC (show other bugs)
Hardware: Intel Linux
: P1 normal
Assignee: superm1
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-10-31 06:03 UTC by RussianNeuroMancer
Modified: 2021-06-24 12:52 UTC (History)
5 users (show)

See Also:
Kernel Version: 4.19.0
Subsystem:
Regression: No
Bisected commit-id:


Attachments
Dell Latitude 7285 acpidump (1.06 MB, text/plain)
2018-10-31 06:04 UTC, RussianNeuroMancer
Details
Dell Latitude 7285 dsdt.dsl (166.80 KB, application/octet-stream)
2018-10-31 06:04 UTC, RussianNeuroMancer
Details
HP Elite x2 1013 G3 acpidump (1.58 MB, text/plain)
2018-10-31 06:05 UTC, RussianNeuroMancer
Details
HP Elite x2 1013 G3 dsdt.dsl (1.04 MB, text/plain)
2018-10-31 06:05 UTC, RussianNeuroMancer
Details
HP Elite x2 1013 G3 dmesg Linux 4.19.0 (90.46 KB, text/plain)
2018-10-31 06:06 UTC, RussianNeuroMancer
Details

Description RussianNeuroMancer 2018-10-31 06:03:32 UTC
I find that on HP Elite x2 1013 G3 and Dell Latitude 7285 power button sometimes wakeup it tablet after short press, and sometimes power button wakeup tablet only after holding it for a few seconds. 

Dell Latitude 7285: 
Initially with Linux 4.13-4.15 (earlier releases wasn't tested) it was requirement to hold power button for five seconds to wakeup from suspend (freeze).  Later, in Linux 4.16 or Linux 4.17 power button started to wakeup tablet after short press, except it did so most of the time, not always. Currently sometimes tablet doesn't wakeup from lid event and power button press, so in such cases it's still necessary to hold power button for five seconds for wakeup it. 

Tested BIOS: from 1.0.5 to 1.2.0.
acpidump and dsdt.dsl is attached

HP Elite x2 1013 G3:
Tablet was tested only with Linux 4.19.0. As with Dell Latitude 7285, sometimes tablet doesn't wakeup from lid event or power button press, and it's still necessary to hold power button for three seconds for wakeup. It also happens with suspend freeze, not suspend to ram which I didn't test yet.

Tested BIOS: Q87 01.01.03 Rev.A
acpidump, dsdt.dsl and dmesg from suspend->wakeup cycle, when issue is reproduced, is attached. 

Please let me know if any additional debug information is required.
Comment 1 RussianNeuroMancer 2018-10-31 06:04:08 UTC
Created attachment 279261 [details]
Dell Latitude 7285 acpidump
Comment 2 RussianNeuroMancer 2018-10-31 06:04:31 UTC
Created attachment 279263 [details]
Dell Latitude 7285 dsdt.dsl
Comment 3 RussianNeuroMancer 2018-10-31 06:05:11 UTC
Created attachment 279265 [details]
HP Elite x2 1013 G3 acpidump
Comment 4 RussianNeuroMancer 2018-10-31 06:05:49 UTC
Created attachment 279267 [details]
HP Elite x2 1013 G3 dsdt.dsl
Comment 5 RussianNeuroMancer 2018-10-31 06:06:14 UTC
Created attachment 279269 [details]
HP Elite x2 1013 G3 dmesg Linux 4.19.0
Comment 6 RussianNeuroMancer 2018-10-31 06:34:46 UTC
I want to note that on HP Elite x2 1013 G3, when wakeup from short power button press doesn't work, wakeup from lid also doesn't work. However, when tablet in this state (no wakeup from lid events or short power button press)  wakeup from Bluetooth (HP ActivePen G2 top button) and USB (any key on keyboard) still works well.
Comment 7 RussianNeuroMancer 2018-11-28 08:53:36 UTC
This issue is reproducible only with suspend freeze.

On HP Elite x2 1013 G3 there is BIOS option that allow to use S3, and when this option (or "mem_sleep_default=deep" kernel option) is enabled, tablet always wakeup from short power button press. Unfortunately, for Dell Latitude 7285 S3 is not an option, and fix of this issue is still required.
Comment 8 Zhang Rui 2019-07-01 06:21:41 UTC
CC Mario.
Comment 9 RussianNeuroMancer 2019-08-27 14:37:25 UTC
@Zhang Rui

On HP Elite x2 1013 G3 with latest BIOS and latest kernel issue is worse now. 

Tablet no longer wakeup from suspend freeze by power button at all. 
Until this is fixed I propose to change default for this tablet from S0ix to S3.
Comment 10 Zhang Rui 2019-09-03 06:48:59 UTC
let's focus on the Dell issue here, as we have Mario in the thread.

so the problem is that,
with latest kernel, power button short press can not wake the system from s2idle in some cases, right?
Comment 11 RussianNeuroMancer 2019-09-03 07:11:03 UTC
I have to return two Latitude 7285 to Dell due to unsolvable hardware issues with device (unrelated to power button). I don't know current status of this issue on this device.

What is next steps with debugging HP 1013?
Comment 13 Zhang Rui 2020-06-29 07:45:56 UTC
does the problem still exist in the latest upstream kernel?
We have quite some EC changes, and hopefully this can fix the problem for you.
Comment 14 RussianNeuroMancer 2020-06-29 11:14:52 UTC
Answered in bug 202519
Comment 15 RussianNeuroMancer 2020-09-26 20:49:02 UTC
On Linux 5.9rc6:
If I suspend tablet for a few minutes, then short press is sufficient to wakeup tablet.
If I suspend tablet for a half-hour, then issue is still reproducible and holding power button for a few second is required to wakeup tablet.
Comment 16 Len Brown 2021-05-27 14:28:19 UTC
Can you update us on the status of this issue with the current upstream Linux kernel?

It is interesting that different version of Linux do seem to impact this latency, though the impact will be indirect, since Linux isn't actually running when this wakeup decision is made.  We may or may not be able to impact how the EC is running here with other functional constraints.
Comment 17 Zhang Rui 2021-06-24 12:49:45 UTC
Bug closed as this is no update from the bug reporter.
Please feel free to reopen it if the problem still exists in the latest upstream kernel.

Note You need to log in before you can comment on or make changes to this bug.