Bug 457430

Summary: Display settings in Wayland aren't saved after reboot
Product: [Plasma] KScreen Reporter: kevin <pilotgi>
Component: commonAssignee: kscreen-bugs-null <kscreen-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: nate, pilotgi
Priority: NOR Keywords: wayland
Version: 5.25.3   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed In: 6.0
Sentry Crash Report:
Attachments: attachment-2973908-0.html
attachment-2974748-0.html

Description kevin 2022-08-03 00:42:58 UTC
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. Change display settings in Wayland
2. Reboot
3. 

OBSERVED RESULT
Display settings ( in my case, the scale of a 4K display) aren't saved even though the changed setting is still shown in the display utility

EXPECTED RESULT
Display settings should be saved.

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 5.25.3
KDE Frameworks Version: 5.96.0
Qt Version: 5.15.5

ADDITIONAL INFORMATION
In a Wayland session, my display is always at 4K resolution and 100% scale. If I change the scale, say to 130%, the display will accept that change. If I reboot and start another Wayland session, the scale is back to 100% but the setting still says 130%. Also at the higher scale, some of the small icons in the panel disappear, but return if I log out and log back in.
Comment 1 kevin 2022-08-03 00:46:49 UTC
I'm using nvidia driver 515.65.01
Comment 2 Nate Graham 2022-08-04 19:06:14 UTC
Xaver, is this the DRM blob thing you just fixed today? Or Bug 450068? Or something else?
Comment 3 kevin 2022-11-09 19:57:05 UTC
I installed some updates today. At least the settings utility now shows the scale at 100% when starting a Wayland session. As mentioned, it used to show the previous higher scale I had chosen, even though it had reverted to 100%. Also, the small icons at the right side of the panel didn't disappear when I chose a scale of 135% like they did previously.  

So still not saving the settings I want, but at least settings is showing what the scale actually is in a new session.
Comment 4 kevin 2022-11-09 19:59:44 UTC
Currently using nvidia 520.56.06
Comment 5 Zamundaaa 2023-10-05 19:42:22 UTC
Git commit ae84480fbfdc684b8ee4b0207d3ce679f6fb4cd7 by Xaver Hugl.
Committed on 05/10/2023 at 20:29.
Pushed by zamundaaa into branch 'master'.

outputconfigurationstore: add new config system

Instead of an external service (like KScreen) storing and restoring output configurations,
with this commit KWin takes over that responsibility. This allows it to, among other things,
generate appropriate configs for new sets of outputs immediately, and take KWin-internal information
about outputs into account when generating them.
Related: bug 466208, bug 455082, bug 474021, bug 469653, bug 466342, bug 470863, bug 466556

M  +1    -0    CMakeLists.txt
M  +1    -0    autotests/integration/kwin_wayland_test.cpp
M  +1    -0    src/CMakeLists.txt
M  +5    -0    src/backends/drm/drm_output.cpp
M  +9    -0    src/core/output.cpp
M  +11   -0    src/core/output.h
M  +1    -0    src/core/outputconfiguration.h
M  +7    -0    src/main.cpp
M  +1    -0    src/main.h
M  +0    -4    src/main_wayland.cpp
M  +709  -20   src/outputconfigurationstore.cpp
M  +68   -4    src/outputconfigurationstore.h
M  +1    -0    src/utils/CMakeLists.txt
M  +8    -0    src/utils/edid.cpp
M  +8    -0    src/utils/edid.h
A  +54   -0    src/utils/orientationsensor.cpp     [License: GPL(v2.0+)]
A  +38   -0    src/utils/orientationsensor.h     [License: GPL(v2.0+)]
M  +34   -1    src/wayland/outputdevice_v2.cpp
M  +1    -0    src/wayland/outputdevice_v2.h
M  +12   -1    src/wayland/outputmanagement_v2.cpp
M  +49   -6    src/workspace.cpp
M  +2    -0    src/workspace.h

https://invent.kde.org/plasma/kwin/-/commit/ae84480fbfdc684b8ee4b0207d3ce679f6fb4cd7
Comment 6 kevin 2023-10-05 21:39:31 UTC
Created attachment 162105 [details]
attachment-2973908-0.html

Is there an update that needs to happen before this new config system is
effective? I just tried saving a scale setting of 175% in Wayland. Switched
back to X11 and then back to Wayland and the scale had reverted back to
100%. Also, despite the panel height being set to 82 with both X11 and
Wayland, the icons and digital clock are MUCH bigger in Wayland when the
scale is at 175%.

On Thu, Oct 5, 2023 at 2:42 PM Zamundaaa <bugzilla_noreply@kde.org> wrote:

> https://bugs.kde.org/show_bug.cgi?id=457430
>
> Zamundaaa <xaver.hugl@gmail.com> changed:
>
>            What    |Removed                     |Added
>
> ----------------------------------------------------------------------------
>          Resolution|---                         |FIXED
>       Latest Commit|                            |
> https://invent.kde.org/plas
>                    |
> |ma/kwin/-/commit/ae84480fbf
>                    |
> |dc684b8ee4b0207d3ce679f6fb4
>                    |                            |cd7
>              Status|REPORTED                    |RESOLVED
>
> --- Comment #5 from Zamundaaa <xaver.hugl@gmail.com> ---
> Git commit ae84480fbfdc684b8ee4b0207d3ce679f6fb4cd7 by Xaver Hugl.
> Committed on 05/10/2023 at 20:29.
> Pushed by zamundaaa into branch 'master'.
>
> outputconfigurationstore: add new config system
>
> Instead of an external service (like KScreen) storing and restoring output
> configurations,
> with this commit KWin takes over that responsibility. This allows it to,
> among
> other things,
> generate appropriate configs for new sets of outputs immediately, and take
> KWin-internal information
> about outputs into account when generating them.
> Related: bug 466208, bug 455082, bug 474021, bug 469653, bug 466342, bug
> 470863, bug 466556
>
> M  +1    -0    CMakeLists.txt
> M  +1    -0    autotests/integration/kwin_wayland_test.cpp
> M  +1    -0    src/CMakeLists.txt
> M  +5    -0    src/backends/drm/drm_output.cpp
> M  +9    -0    src/core/output.cpp
> M  +11   -0    src/core/output.h
> M  +1    -0    src/core/outputconfiguration.h
> M  +7    -0    src/main.cpp
> M  +1    -0    src/main.h
> M  +0    -4    src/main_wayland.cpp
> M  +709  -20   src/outputconfigurationstore.cpp
> M  +68   -4    src/outputconfigurationstore.h
> M  +1    -0    src/utils/CMakeLists.txt
> M  +8    -0    src/utils/edid.cpp
> M  +8    -0    src/utils/edid.h
> A  +54   -0    src/utils/orientationsensor.cpp     [License: GPL(v2.0+)]
> A  +38   -0    src/utils/orientationsensor.h     [License: GPL(v2.0+)]
> M  +34   -1    src/wayland/outputdevice_v2.cpp
> M  +1    -0    src/wayland/outputdevice_v2.h
> M  +12   -1    src/wayland/outputmanagement_v2.cpp
> M  +49   -6    src/workspace.cpp
> M  +2    -0    src/workspace.h
>
>
> https://invent.kde.org/plasma/kwin/-/commit/ae84480fbfdc684b8ee4b0207d3ce679f6fb4cd7
>
> --
> You are receiving this mail because:
> You reported the bug.
> You are on the CC list for the bug.
Comment 7 Nate Graham 2023-10-05 21:43:01 UTC
This new config system was merged into git master barely two hours ago, so I'm not super surprised that you don't already have it active on your system. :) It will be released with Plasma 6, which is scheduled for a release in early February of next year.
Comment 8 kevin 2023-10-05 21:45:59 UTC
Created attachment 162106 [details]
attachment-2974748-0.html

Thanks for the info. Just a little anxious I guess.

On Thu, Oct 5, 2023 at 4:43 PM Nate Graham <bugzilla_noreply@kde.org> wrote:

> https://bugs.kde.org/show_bug.cgi?id=457430
>
> --- Comment #7 from Nate Graham <nate@kde.org> ---
> This new config system was merged into git master barely two hours ago, so
> I'm
> not super surprised that you don't already have it active on your system.
> :) It
> will be released with Plasma 6, which is scheduled for a release in early
> February of next year.
>
> --
> You are receiving this mail because:
> You reported the bug.
> You are on the CC list for the bug.