Comment 0 for bug 1801875

Revision history for this message
AceLan Kao (acelankao) wrote : Suspend to idle power consumption higher than long idle (sk hynix)

[Impact]
On some Dell XPS models, we observed the power consumption raises higher than long idle does 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 to prevent nvme from entering D3, and prevent nvme from being disabled during 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.