Activity log for bug #1801875

Date Who What changed Old value New value Message
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