2018-11-06 07:45:25 |
AceLan Kao |
bug |
|
|
added bug |
2018-11-06 07:46:59 |
AceLan Kao |
attachment added |
|
0001-pci-prevent-sk-hynix-nvme-from-entering-D3.patch https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1801875/+attachment/5209576/+files/0001-pci-prevent-sk-hynix-nvme-from-entering-D3.patch |
|
2018-11-06 07:47:16 |
AceLan Kao |
attachment added |
|
0002-nvme-add-quirk-to-not-call-disable-function-when-sus.patch https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1801875/+attachment/5209577/+files/0002-nvme-add-quirk-to-not-call-disable-function-when-sus.patch |
|
2018-11-06 07:48:37 |
AceLan Kao |
description |
[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. |
[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 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. |
|
2018-11-06 07:49:08 |
AceLan Kao |
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 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. |
[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 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. |
|
2018-11-06 07:49:45 |
AceLan Kao |
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 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. |
[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 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. |
|
2018-11-06 07:50:17 |
AceLan Kao |
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 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. |
[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. |
|
2018-11-06 07:59:50 |
AceLan Kao |
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. |
[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_via_s2idle() function from below commit which is still under reviewing.
https://lists.ubuntu.com/archives/kernel-team/2018-October/096141.html |
|
2018-11-06 08:09:47 |
AceLan Kao |
summary |
Suspend to idle power consumption higher than long idle (sk hynix) |
Power consumption during s2idle is higher than long idle(sk hynix) |
|
2018-11-06 12:23:45 |
Ubuntu Foundations Team Bug Bot |
tags |
|
patch |
|
2018-11-06 12:23:45 |
Ubuntu Foundations Team Bug Bot |
bug |
|
|
added subscriber Joseph Salisbury |
2018-11-06 16:50:02 |
Kleber Sacilotto de Souza |
nominated for series |
|
Ubuntu Bionic |
|
2018-11-06 16:50:02 |
Kleber Sacilotto de Souza |
bug task added |
|
linux (Ubuntu Bionic) |
|
2018-11-06 16:50:02 |
Kleber Sacilotto de Souza |
nominated for series |
|
Ubuntu Cosmic |
|
2018-11-06 16:50:02 |
Kleber Sacilotto de Souza |
bug task added |
|
linux (Ubuntu Cosmic) |
|
2018-11-07 02:43:01 |
AceLan Kao |
linux (Ubuntu): status |
In Progress |
Fix Committed |
|
2018-11-08 05:06:11 |
Khaled El Mously |
linux (Ubuntu Bionic): status |
New |
Fix Committed |
|
2018-11-08 05:06:15 |
Khaled El Mously |
linux (Ubuntu Cosmic): status |
New |
Fix Committed |
|
2018-11-15 11:04:02 |
Brad Figg |
tags |
patch |
patch verification-needed-cosmic |
|
2018-11-15 11:34:50 |
Brad Figg |
tags |
patch verification-needed-cosmic |
patch verification-needed-bionic verification-needed-cosmic |
|
2018-11-19 09:04:26 |
AceLan Kao |
tags |
patch verification-needed-bionic verification-needed-cosmic |
patch verification-done-bionic verification-needed-cosmic |
|
2018-11-19 09:04:48 |
AceLan Kao |
tags |
patch verification-done-bionic verification-needed-cosmic |
patch verification-done-bionic verification-done-cosmic |
|
2018-12-03 08:49:32 |
Launchpad Janitor |
linux (Ubuntu Cosmic): status |
Fix Committed |
Fix Released |
|
2018-12-03 08:49:32 |
Launchpad Janitor |
cve linked |
|
2018-18653 |
|
2018-12-03 08:49:32 |
Launchpad Janitor |
cve linked |
|
2018-18955 |
|
2018-12-03 08:49:32 |
Launchpad Janitor |
cve linked |
|
2018-6559 |
|
2018-12-03 14:01:15 |
Launchpad Janitor |
linux (Ubuntu Bionic): status |
Fix Committed |
Fix Released |
|
2019-02-04 14:46:37 |
Launchpad Janitor |
linux (Ubuntu): status |
Fix Committed |
Fix Released |
|
2019-07-24 21:29:40 |
Brad Figg |
tags |
patch verification-done-bionic verification-done-cosmic |
cscc patch verification-done-bionic verification-done-cosmic |
|