| Summary: | Built-In Color Profile turns screen black in Wayland session | ||
|---|---|---|---|
| Product: | [Plasma] kwin | Reporter: | Danie de Jager <danie.dejager> |
| Component: | colour-management | Assignee: | KWin default assignee <kwin-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | nate, xaver.hugl |
| Priority: | NOR | ||
| Version First Reported In: | 6.2.5 | ||
| Target Milestone: | --- | ||
| Platform: | Neon | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/kwin/-/commit/d02b39b2a8a4a824c0c67a68afada31f51fa5b30 | Version Fixed In: | 6.3.1 |
| Sentry Crash Report: | |||
| Attachments: |
Display setting where "Color Profile" can be found.
drm-info output |
||
|
Description
Danie de Jager
2025-01-13 17:17:36 UTC
Created attachment 177323 [details]
Display setting where "Color Profile" can be found.
Please attach the output of > edid-decode /sys/class/drm/card1-eDP-1/edid the card number and maybe connector name might need adjusting to be about a valid file Most likely, the color information in the profile is unusable. We should detect that situation and hide the option entirely when that's the case. > I thought there is no timeout revert happening so I forced a reboot which permanently set the screen's new value. I'd expect the new value to only set once I confirmed it as I could see to click a confirm button. Please open a separate bug report about that; this should be doable but is rather complicated. > The timeout could be a little shorter? Unfortunately some screens take a long time to switch modes, making the timeout shorter would make it impossible to use those. A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/7016 Git commit 037d710b3d656e873a001ed479695c8b8463a33e by Xaver Hugl. Committed on 16/01/2025 at 10:30. Pushed by zamundaaa into branch 'master'. utils/edid: check if colorimetry could possibly be correct With invalid colorimetry, selecting the "built-in" color profile could otherwise cause a black screen or glitches. M +14 -6 src/utils/edid.cpp https://invent.kde.org/plasma/kwin/-/commit/037d710b3d656e873a001ed479695c8b8463a33e (In reply to Zamundaaa from comment #2) > Please attach the output of > > edid-decode /sys/class/drm/card1-eDP-1/edid > the card number and maybe connector name might need adjusting to be about a > valid file > > Most likely, the color information in the profile is unusable. We should > detect that situation and hide the option entirely when that's the case. > > > I thought there is no timeout revert happening so I forced a reboot which permanently set the screen's new value. I'd expect the new value to only set once I confirmed it as I could see to click a confirm button. > Please open a separate bug report about that; this should be doable but is > rather complicated. > > > The timeout could be a little shorter? > Unfortunately some screens take a long time to switch modes, making the > timeout shorter would make it impossible to use those. Theses are from my Macbook: ``` edid-decode /sys/class/drm/card1- card1-DP-1/ card1-DP-2/ card1-DP-3/ card1-HDMI-A-1/ card1-HDMI-A-2/ card1-HDMI-A-3/ card1-LVDS-1/ card1-VGA-1/ ``` I don't have a `/sys/class/drm/card1-eDP-1/edid` Git commit d02b39b2a8a4a824c0c67a68afada31f51fa5b30 by Vlad Zahorodnii, on behalf of Xaver Hugl. Committed on 16/01/2025 at 16:21. Pushed by vladz into branch 'Plasma/6.3'. utils/edid: check if colorimetry could possibly be correct With invalid colorimetry, selecting the "built-in" color profile could otherwise cause a black screen or glitches. (cherry picked from commit 037d710b3d656e873a001ed479695c8b8463a33e) M +14 -6 src/utils/edid.cpp https://invent.kde.org/plasma/kwin/-/commit/d02b39b2a8a4a824c0c67a68afada31f51fa5b30 (In reply to Danie de Jager from comment #5) > Theses are from my Macbook: > ``` > edid-decode /sys/class/drm/card1- > card1-DP-1/ card1-DP-2/ card1-DP-3/ card1-HDMI-A-1/ > card1-HDMI-A-2/ card1-HDMI-A-3/ card1-LVDS-1/ card1-VGA-1/ > ``` > I don't have a `/sys/class/drm/card1-eDP-1/edid` In that case, you need to use > edid-decode /sys/class/drm/card1-LVDS-1 (In reply to Zamundaaa from comment #7) > (In reply to Danie de Jager from comment #5) > > Theses are from my Macbook: > > ``` > > edid-decode /sys/class/drm/card1- > > card1-DP-1/ card1-DP-2/ card1-DP-3/ card1-HDMI-A-1/ > > card1-HDMI-A-2/ card1-HDMI-A-3/ card1-LVDS-1/ card1-VGA-1/ > > ``` > > I don't have a `/sys/class/drm/card1-eDP-1/edid` > In that case, you need to use > > edid-decode /sys/class/drm/card1-LVDS-1 That was Empty: > $ edid-decode /sys/class/drm/card1-LVDS-1 > EDID of '/sys/class/drm/card1-LVDS-1' was empty. huh, that's unexpected. What's the output of drm_info? Created attachment 177504 [details]
drm-info output
(In reply to Danie de Jager from comment #10) > Created attachment 177504 [details] > drm-info output I attached json output. (In reply to Zamundaaa from comment #9) > huh, that's unexpected. What's the output of drm_info? Attached. Oh, I only now noticed that I got the command wrong, sorry! It was supposed to be
> edid-decode /sys/class/drm/card1-LVDS-1/edid
Also, does it still happen for you in Plasma 6.3? After the release of 6.3 I did no longer experience the problem. Consider it resolved. Thanks a lot! |