Suspend to idle: Open lid didn't resume

Bug #1771542 reported by Woodrow Shen on 2018-05-16
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
HWE Next
Undecided
Unassigned
linux (Ubuntu)
Medium
Woodrow Shen
Artful
Medium
Unassigned
Bionic
Medium
Unassigned

Bug Description

[Impact]
Some platforms are set suspend-to-idle (s2idle) as default suspend mode, and then they can't resume the system from opening the lid.

[Test Case]
1. echo s2idle | sudo tee /sys/power/mem_sleep
2. Close the lid
3. Open the lid

[Fix]
According a patch sent to 4.17-rc1, which was not be included before 4.15 EOL, so we cherry pick the patch to Ubuntu 4.15 kernel.

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=147a7d9d25ca2551aab15071cb1f048ecd9b7953

[Regression Potential]
Per commit, this change also allows Dells XPS13 9360 blacklisted by commit 71630b7a832f to use the power button for waking up from suspend-to-idle, no regression could be happened

description: updated
description: updated

This bug is missing log files that will aid in diagnosing the problem. While running an Ubuntu kernel (not a mainline or third-party kernel) please enter the following command in a terminal window:

apport-collect 1771542

and then change the status of the bug to 'Confirmed'.

If, due to the nature of the issue you have encountered, you are unable to run this command, please add a comment stating that fact and change the bug status to 'Confirmed'.

This change has been made by an automated script, maintained by the Ubuntu Kernel Team.

Changed in linux (Ubuntu):
status: New → Incomplete
Changed in linux (Ubuntu):
status: Incomplete → Confirmed
Changed in linux (Ubuntu):
importance: Undecided → Medium
status: Confirmed → Triaged
Changed in linux (Ubuntu Artful):
status: New → Triaged
Changed in linux (Ubuntu Bionic):
status: New → Triaged
Changed in linux (Ubuntu Artful):
importance: Undecided → Medium
Changed in linux (Ubuntu Bionic):
importance: Undecided → Medium
description: updated
affects: linux (Ubuntu) → linux-oem (Ubuntu)
Changed in linux-oem (Ubuntu):
status: Triaged → New
Changed in linux-oem (Ubuntu Artful):
status: Triaged → New
Changed in linux-oem (Ubuntu Bionic):
status: Triaged → New
Changed in linux (Ubuntu):
status: New → Confirmed
Changed in linux-oem (Ubuntu):
status: New → Confirmed
no longer affects: linux (Ubuntu)
no longer affects: linux (Ubuntu Artful)
no longer affects: linux (Ubuntu Bionic)
affects: linux-oem (Ubuntu) → linux (Ubuntu)
Changed in linux (Ubuntu):
assignee: nobody → Woodrow Shen (woodrow-shen)
Changed in linux-oem (Ubuntu):
status: New → Confirmed
no longer affects: linux-oem (Ubuntu)
Changed in linux (Ubuntu Artful):
status: New → Confirmed
status: Confirmed → New
Changed in linux-oem (Ubuntu):
status: New → Confirmed
Woodrow Shen (woodrow-shen) wrote :

@jsalisbury,

Sorry for changing the status. Please let the bug follow your policy, thanks.

description: updated
Changed in linux (Ubuntu Artful):
status: New → In Progress
Changed in linux (Ubuntu Bionic):
status: New → In Progress
Changed in linux (Ubuntu Artful):
status: In Progress → Fix Committed
Changed in linux (Ubuntu Bionic):
status: In Progress → Fix Committed
Changed in linux (Ubuntu):
status: Confirmed → Fix Committed
Po-Hsu Lin (cypressyew) wrote :

Hi Woodrow,

I saw you have opened a task for linux-oem, but the patch was only targeted for A/B.
Do you have plan to sent a patch for OEM kernel as well?

Thanks

Po-Hsu Lin (cypressyew) wrote :

Hi Woodrow,

I have contacted Timo about this, he said he will pick this up with rebase once it got applied on Bionic master branch.
There is no need to resend this. Let's wait for this and change the bug status later.

Thanks

Woodrow Shen (woodrow-shen) wrote :

Ok, thanks. Do I need to remove linux-oem? or you can help me to do it :)

Anthony Wong (anthonywong) wrote :

@woodrow
Either works, I'll remove it for you :)

no longer affects: linux-oem (Ubuntu)
Brad Figg (brad-figg) wrote :

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-bionic' to 'verification-done-bionic'. If the problem still exists, change the tag 'verification-needed-bionic' to 'verification-failed-bionic'.

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!

tags: added: verification-needed-bionic
Brad Figg (brad-figg) wrote :

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-artful' to 'verification-done-artful'. If the problem still exists, change the tag 'verification-needed-artful' to 'verification-failed-artful'.

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!

tags: added: verification-needed-artful
Woodrow Shen (woodrow-shen) wrote :

I verified both bionic-proposed and artful-proposed kernel on Dell Inspiron 5767 laptop, and they work fine. So I change the tag to verification-done-artful/verification-done-bionic. Thanks.

tags: added: verification-done-artful verification-done-bionic
removed: verification-needed-artful verification-needed-bionic
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers