Power consumption during s2idle is higher than long idle(sk hynix)
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
linux (Ubuntu) |
Fix Released
|
Undecided
|
AceLan Kao | ||
Bionic |
Fix Released
|
Undecided
|
Unassigned | ||
Cosmic |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
[Impact]
On some Dell XPS models, we observed the power consumption raises higher than long idle does during s2idle with sk hynix nvme.
C2:
Short idle: 4
Long idle: 1
S2I: 3.7
S5: 0.19
C3:
Short idle: 7.2
Long idle: 4.5
S2I: 6.22
S5: 0.18
C5:
Short idle: 6.5
Long idle: 1
S2I: 2.88
S5: 0.18
[Fix]
From SK hynix FE, MS Windows doesn't put nvme to D3, and uses its own APST feature to do the power management. To leverage its APST feature during s2idle, we can't disable nvme device while suspending, too.
So, here is what we did on the driver, 1. prevent nvme from entering D3, 2. prevent nvme from being disabled when suspending.
[Test]
Verified on different XPS machines with different sk hynix nvme disks, it fixes the power consumption issue with no regression. And the power consumption drops to 0.77W during s2idle.
[Regression Potential]
Low, the patches only applied to specific nvme module, and from our test, the system is still stable.
[Misc]
This issue should be fixed by the firmware, and we're pushing sk hynix to fix it. But before sk hynix find out how to solve it, we have to preserve these commits in our kernel for a while.
BTW, the patches use pm_suspend_
https:/
CVE References
description: | updated |
summary: |
- Suspend to idle power consumption higher than long idle (sk hynix) + Power consumption during s2idle is higher than long idle(sk hynix) |
tags: | added: patch |
Changed in linux (Ubuntu): | |
status: | In Progress → Fix Committed |
Changed in linux (Ubuntu Bionic): | |
status: | New → Fix Committed |
Changed in linux (Ubuntu Cosmic): | |
status: | New → Fix Committed |
tags: |
added: verification-done-bionic removed: verification-needed-bionic |
tags: |
added: verification-done-cosmic removed: verification-needed-cosmic |
tags: | added: cscc |
This bug is awaiting verification that the kernel in -proposed solves the problem. Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification- needed- cosmic' to 'verification- done-cosmic' . If the problem still exists, change the tag 'verification- needed- cosmic' to 'verification- failed- cosmic' .
If verification is not done by 5 working days from today, this fix will be dropped from the source code, and this bug will be closed.
See https:/ /wiki.ubuntu. com/Testing/ EnableProposed for documentation how to enable and use -proposed. Thank you!