Bug 428424 - Open Laptop Lid doesn't turn on Display (Wayland)
Summary: Open Laptop Lid doesn't turn on Display (Wayland)
Status: RESOLVED WORKSFORME
Alias: None
Product: kscreenlocker
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords: wayland
: 428525 435311 (view as bug list)
Depends on:
Blocks:
 
Reported: 2020-10-29 10:45 UTC by Rainer Finke
Modified: 2023-01-17 17:15 UTC (History)
7 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Rainer Finke 2020-10-29 10:45:26 UTC
If I open the laptop lid after my laptop with Plasma Wayland was in standby, the display is not turned on until I press a key.

Sounds similar to the closed bug https://bugs.kde.org/show_bug.cgi?id=403634

STEPS TO REPRODUCE
1. Open Plasma Wayland on a laptop
2. Wait for standby
3. Open laptop lid

OBSERVED RESULT
- Black screen

EXPECTED RESULT
- Screenlocker is visible immediately

Operating System: Arch Linux
KDE Plasma Version: 5.20.2
KDE Frameworks Version: 5.75.0
Qt Version: 5.15.1
Kernel Version: 5.9.1-arch1-1
OS Type: 64-bit
Processors: 4 × Intel® Core™ i7-5600U CPU @ 2.60GHz
Memory: 7.7 GiB of RAM
Graphics Processor: Mesa Intel® HD Graphics 5500
Comment 1 Nate Graham 2020-10-29 19:33:29 UTC
Aleix, is this the thing you just fixed recently?
Comment 2 Aleix Pol 2020-10-30 02:15:24 UTC
What I did was at a KScreen level. I assume he is wondering why the laptop doesn't turn on when the lid opens. Maybe we need to deal with dpms when the lid opens?

Does moving the mouse also turn the screen on?
Comment 3 Patrick Silva 2020-11-01 02:01:40 UTC
*** Bug 428525 has been marked as a duplicate of this bug. ***
Comment 4 Bug Janitor Service 2020-11-02 01:35:37 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/412
Comment 5 Aleix Pol 2020-11-02 01:36:42 UTC
Note I haven't been able to reproduce as it works for me, can somebody test?
Comment 6 Rainer Finke 2020-11-02 18:41:41 UTC
You are probably not able to reproduce this issue, if you close the laptop lid before the display is dimmed and standby kicks in.

If I use the touchpad the display is turned on as well, not only with the keyboard. Touchscreen needs multiple touches and takes time.
Comment 7 Aleix Pol 2020-11-03 03:15:28 UTC
I was able to test now, managed to address the fix, which is much simpler too. Thanks Rainer!
Comment 8 Aleix Pol 2020-11-03 17:59:28 UTC
Git commit 69eb8789ba8bb240c7723bd1e404c14d0a4a7c11 by Aleix Pol Gonzalez, on behalf of Aleix Pol.
Committed on 03/11/2020 at 17:59.
Pushed by apol into branch 'master'.

drm: Make sure the screens are turned on when we come from suspend

Listen to logind for resume notification and turn the outputs on when it
happens, much like we do when pressing a key.

This way laptops come back on when the lid opens.

M  +7    -0    logind.cpp
M  +1    -0    logind.h
M  +5    -0    plugins/platforms/drm/drm_backend.cpp

https://invent.kde.org/plasma/kwin/commit/69eb8789ba8bb240c7723bd1e404c14d0a4a7c11
Comment 9 Aleix Pol 2020-11-03 18:02:48 UTC
Git commit ce044ec3bea0198ba94c91872a3c1ba0f0da9b23 by Aleix Pol Gonzalez, on behalf of Aleix Pol.
Committed on 03/11/2020 at 18:02.
Pushed by apol into branch 'cherry-pick-69eb8789'.

drm: Make sure the screens are turned on when we come from suspend

Listen to logind for resume notification and turn the outputs on when it
happens, much like we do when pressing a key.

This way laptops come back on when the lid opens.


(cherry picked from commit 69eb8789ba8bb240c7723bd1e404c14d0a4a7c11)

M  +7    -0    logind.cpp
M  +1    -0    logind.h
M  +5    -0    plugins/platforms/drm/drm_backend.cpp

https://invent.kde.org/plasma/kwin/commit/ce044ec3bea0198ba94c91872a3c1ba0f0da9b23
Comment 10 Nate Graham 2020-11-03 18:36:09 UTC
Git commit 191bef73ac00cdb3907ca4be33be6049ac40fa7c by Nate Graham, on behalf of Aleix Pol.
Committed on 03/11/2020 at 18:36.
Pushed by ngraham into branch 'Plasma/5.20'.

drm: Make sure the screens are turned on when we come from suspend

Listen to logind for resume notification and turn the outputs on when it
happens, much like we do when pressing a key.

This way laptops come back on when the lid opens.


(cherry picked from commit 69eb8789ba8bb240c7723bd1e404c14d0a4a7c11)

M  +7    -0    logind.cpp
M  +1    -0    logind.h
M  +5    -0    plugins/platforms/drm/drm_backend.cpp

https://invent.kde.org/plasma/kwin/commit/191bef73ac00cdb3907ca4be33be6049ac40fa7c
Comment 11 Ahmed 2020-11-15 23:46:38 UTC
It is not fixed. I use KDE Plasma 5.20.3, and I still have to move the mouse, or press any key to turn on the screen.
Comment 12 Fushan Wen 2021-04-03 11:54:15 UTC
It's not fixed, and I have opened a new bug.
Comment 13 Nate Graham 2021-04-05 17:32:02 UTC
*** Bug 435311 has been marked as a duplicate of this bug. ***
Comment 14 Nate Graham 2021-04-07 22:19:43 UTC
Works for me git a git master Plasma Wayland session. Could you test again in 5.21 (or even git master, if possible)? Thanks!
Comment 15 Fushan Wen 2021-04-15 15:10:26 UTC
(In reply to Nate Graham from comment #14)
> Works for me git a git master Plasma Wayland session. Could you test again
> in 5.21 (or even git master, if possible)? Thanks!

It still doesn't work on my laptop. Have to wait for screen timeout (Screen Energy Saving) to reproduce the bug.

Operating System: Kubuntu 21.04
KDE Plasma Version: 5.21.4
KDE Frameworks Version: 5.80.0
Qt Version: 5.15.2
Comment 16 Bug Janitor Service 2021-04-30 04:33:37 UTC
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least
15 days. Please provide the requested information as soon as
possible and set the bug status as REPORTED. Due to regular bug
tracker maintenance, if the bug is still in NEEDSINFO status with
no change in 30 days the bug will be closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please
mark the bug as REPORTED so that the KDE team knows that the bug is
ready to be confirmed.

Thank you for helping us make KDE software even better for everyone!
Comment 17 Bug Janitor Service 2021-05-15 04:33:39 UTC
This bug has been in NEEDSINFO status with no change for at least
30 days. The bug is now closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

Thank you for helping us make KDE software even better for everyone!
Comment 18 Hector Martin 2023-01-16 15:44:11 UTC
Are we conflating two issues here? What I see on Apple Silicon machines is that KDE is configured to *suspend* on lid close (or you manually suspend it prior to closing the lid) then indeed it wakes up and turns the screen on when you open the lid. However, if it is configured to *turn off the screen* on lid close, the screen doesn't turn on when you open it until you press a key.
Comment 19 Nate Graham 2023-01-17 16:52:09 UTC
Hmm, that seems different from what's being described here.