Bug 479380 - With icc monitor profile enabled there is light patch that follows the cursor
Summary: With icc monitor profile enabled there is light patch that follows the cursor
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: wayland-generic (show other bugs)
Version: 5.91.0
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords: qt6, wayland-only
: 479898 (view as bug list)
Depends on:
Blocks:
 
Reported: 2024-01-04 06:17 UTC by Doug
Modified: 2024-01-16 19:24 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Watch carefully around the cursor as it moves (913.42 KB, video/webm)
2024-01-04 06:17 UTC, Doug
Details
video showing graphical error (3.63 MB, video/mp4)
2024-01-08 19:48 UTC, Doug
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Doug 2024-01-04 06:17:02 UTC
Created attachment 164660 [details]
Watch carefully around the cursor as it moves

SUMMARY
***
NOTE: If you are reporting a crash, please try to attach a backtrace with debug symbols.
See https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports
***


STEPS TO REPRODUCE
1. open system settings
2. add an icc profile for your monitor 
3. click "apply"

OBSERVED RESULT
The mouse cursor is surrounded by box wherein the color is lighter than the rest of the screen

EXPECTED RESULT
No box around the mouse cursor

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: KDE Neon Unstable 
(available in About System)
KDE Plasma Version: 5.91.90
KDE Frameworks Version: 5.248.0
Qt Version: 6.6.1

ADDITIONAL INFORMATION
Monitor is an AOC U3277PWQU, this is attached to a laptop as an external screen by DP
Graphics card is a AMD Radeon WX Pro 3200 Series
Comment 1 Doug 2024-01-04 06:20:03 UTC
Disregard the attachment, the error is not visible on the video. Sorry.
Comment 2 Zamundaaa 2024-01-08 15:42:37 UTC
Can you film the display with a phone to show the issue?
Comment 3 Doug 2024-01-08 19:48:37 UTC
Created attachment 164758 [details]
video showing graphical error
Comment 4 Zamundaaa 2024-01-08 21:59:24 UTC
Hmm, not sure what's going on there. Does it happen with all cursor themes?
Comment 5 Doug 2024-01-08 22:49:04 UTC
(In reply to Zamundaaa from comment #4)
> Hmm, not sure what's going on there. Does it happen with all cursor themes?

I tried both Breeze Light and Dark, and installed a couple others.  It is the same on all of them.
Comment 6 Zamundaaa 2024-01-16 17:10:57 UTC
*** Bug 479898 has been marked as a duplicate of this bug. ***
Comment 7 Zamundaaa 2024-01-16 17:11:27 UTC
Can confirm with the ICC profile attached in bug 479898
Comment 8 Bug Janitor Service 2024-01-16 18:44:12 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/4947
Comment 9 Zamundaaa 2024-01-16 18:45:34 UTC
As side info for you, the profile attached in bug 479898 is not a good one. Zero brightness should ideally always be mapped to zero brightness, but more importantly, it reduces the maximum brightness of the screen, which shouldn't happen with a properly created profile.
Comment 10 Zamundaaa 2024-01-16 19:05:10 UTC
Git commit 6ab8f179a7171a5e5dd41d667dd0cc571765dad6 by Xaver Hugl.
Committed on 16/01/2024 at 19:42.
Pushed by zamundaaa into branch 'master'.

backends/drm: apply the ICC profile without premultiplication

As a side effect, this ensures that when alpha is zero, rgb is zero as well.
This is needed because the ICC profile may contain transformations where zero
brightness gets mapped to a non-zero value.

M  +2    -0    src/backends/drm/icc.frag
M  +2    -0    src/backends/drm/icc_core.frag

https://invent.kde.org/plasma/kwin/-/commit/6ab8f179a7171a5e5dd41d667dd0cc571765dad6
Comment 11 Zamundaaa 2024-01-16 19:24:22 UTC
Git commit 3569eaab15963db93ecee7ccee161492a8b1432e by Xaver Hugl.
Committed on 16/01/2024 at 20:05.
Pushed by zamundaaa into branch 'Plasma/6.0'.

backends/drm: apply the ICC profile without premultiplication

As a side effect, this ensures that when alpha is zero, rgb is zero as well.
This is needed because the ICC profile may contain transformations where zero
brightness gets mapped to a non-zero value.


(cherry picked from commit 6ab8f179a7171a5e5dd41d667dd0cc571765dad6)

M  +2    -0    src/backends/drm/icc.frag
M  +2    -0    src/backends/drm/icc_core.frag

https://invent.kde.org/plasma/kwin/-/commit/3569eaab15963db93ecee7ccee161492a8b1432e