Night Light occasionally gets stuck

[Kevin]

[ Impact ]

Since installing Ubuntu 24.04, every now and then Night Light gets stuck when my screen is locked at night and I unlock it in the morning. The screen is then very orange (more so than normally is the case with Night Light), and disabling it with the button in in the top bar menu does nothing.

The only ways to restore the screen to normal are disabling and enabling the colour profile or running `gdbus call -e -o /org/gnome/SettingsDaemon/Color -d org.gnome.SettingsDaemon.Color -m org.gnome.SettingsDaemon.Color.NightLightPreview 1`

[ Test Plan ]

Unknown. Requires OP verification.

[ Where problems could occur ]

Night light and colour profiles are one in the same feature so touching one always risks affecting the other. Since all such gamma changes get committed along with other screen updates, there is the risk (particularly with atomic mode setting) that a mistake in any one change will cause the entire screen update to fail, resulting in black frames.

[ Other Info ]

I have never been able to find anything remotely relevant in any system logs regarding this issue. It also does not happen every day, but probably once every seven to ten days or so.

My PC has an AMD RX550 GPU using the amdgpu driver. I'm not sure if it makes a difference, but so far I've not seen this problem on my other machines with Intel IGPs.

The upstream bug report can be found here: https://gitlab.gnome.org/GNOME/mutter/-/issues/3439

[Launchpad Janitor (janitor)]

This bug was fixed in the package mutter - 46.2-1ubuntu1

---------------
mutter (46.2-1ubuntu1) oracular; urgency=medium

  * Merge with Debian. Remaining changes:
    - Add x11-Add-support-for-fractional-scaling-using-Randr.patch
    - Add window-Add-ability-to-override-the-edge-constraints.patch
      + Make possible for extensions (such as Tiling Assistant) to override
        window constraints
    - Ignore test results on armhf & ppc64el

mutter (46.2-1) experimental; urgency=medium

  [ Daniel van Vugt ]
  * New upstream release 46.2 (LP: #2068598)
    - Fixed mouse clicks falling through to the window behind (LP: #2026194)
    - Fixed a crash in meta_wayland_transaction_commit (LP: #2061739)
    - Fixed night light getting stuck on (LP: #2066902)
  * Update Support-Dynamic-triple-double-buffering.patch:
    - Removed bogus changes introduced in 46.1-2 (unrelated files)
    - Fixed debugging mode "MUTTER_DEBUG_TRIPLE_BUFFERING=always" which had
      temporarily stopped working in 46.1 but also nobody should have been
      using it.
    - Updated commit message to mention the upstream source is maintained
      in 'triple-buffering-v4-46'.
  * Refreshed patches:
    - meson-Do-not-mark-CI-test-tools-as-required.patch
    - Mark-several-additional-tests-as-flaky.patch
  * Add clutter/conform gesture tests to the list of known flaky ones:
    it is a new test in mutter 46
  * Add patch to compile gsettings schemas for use in build tests

  [ Jeremy Bícha ]
  * Stop compiling gsettings schemas in debian/rules
  * Opt into Salsa CI

 -- Jeremy Bícha <email address hidden> Wed, 12 Jun 2024 16:30:58 -0400

[Timo Aaltonen (tjaalton)]

Hello Kevin, or anyone else affected,

Accepted mutter into noble-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/mutter/46.2-1ubuntu0.24.04.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-noble to verification-done-noble. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-noble. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

[Kevin (the7thstranger)]

I have installed the mutter package from noble-proposed and rebooted, so I'll be testing it from now on.

kevin@arcadia:~$ mutter --version
mutter 46.2

Sadly I don't know of any way to manually trigger this bug. Sometimes it doesn't happen for days, and then all of a sudden the colour temperature is stuck again when I unlock my computer in the morning. I haven't discovered any pattern or causality.

So I guess the only way to test this is to just wait and see if it doesn't happen any more.

[Daniel van Vugt (vanvugt)]

Please ensure you have the new version of the libmutter-14-0 package installed as a minimum.

[Daniel van Vugt (vanvugt)]

Depending on which mutter packages you already have installed, download enough to replace them, at least:

1. https://launchpad.net/ubuntu/+source/mutter/46.2-1ubuntu0.24.04.1/+build/28630670/+files/gir1.2-mutter-14_46.2-1ubuntu0.24.04.1_amd64.deb

2. https://launchpad.net/ubuntu/+source/mutter/46.2-1ubuntu0.24.04.1/+build/28630670/+files/libmutter-14-0_46.2-1ubuntu0.24.04.1_amd64.deb

3. https://launchpad.net/ubuntu/+source/mutter/46.2-1ubuntu0.24.04.1/+build/28630670/+files/mutter-common-bin_46.2-1ubuntu0.24.04.1_amd64.deb

4. https://launchpad.net/ubuntu/+source/mutter/46.2-1ubuntu0.24.04.1/+build/28630670/+files/mutter-common_46.2-1ubuntu0.24.04.1_all.deb

Then install them as one:

  sudo dpkg -i *.deb

and reboot.

[Kevin (the7thstranger)]

I've been testing all these packages since last Monday, and this morning night light was stuck again when I unlocked my screen. So I'm afraid this doesn't fix the issue.

I also haven't seen any activity in the upstream ticket of people actually confirming the issue was fixed.

[Daniel van Vugt (vanvugt)]

Thanks for testing that.

To the SRU team:
I don't think this is important enough to block the update. And retesting all the other bugs for days/weeks again wouldn't make sense if we were only going to remove the bug reference from the changelog. So we should just reopen this after release.

[Launchpad Janitor (janitor)]

This bug was fixed in the package mutter - 46.2-1ubuntu0.24.04.1

---------------
mutter (46.2-1ubuntu0.24.04.1) noble; urgency=medium

  [ Daniel van Vugt ]
  * New upstream release 46.2 (LP: #2068598)
    - Fixed mouse clicks falling through to the window behind (LP: #2026194)
    - Fixed a crash in meta_wayland_transaction_commit (LP: #2061739)
    - Fixed night light getting stuck on (LP: #2066902)
  * New upstream release 46.1 (LP: #2064735):
    - Fixed blank screen on hybrid GPU machines using nouveau (LP: #2066931)
    - Fixed a monitor hotplug crash (LP: #2050865)
    - Improved performance on secondary GPU monitors (LP: #2038801)
  * Refreshed patches:
    - Support-Dynamic-triple-double-buffering.patch (LP: #2070437, LP: #2070438)
    - meson-Do-not-mark-CI-test-tools-as-required.patch
    - Mark-several-additional-tests-as-flaky.patch
  * Add clutter/conform gesture tests to the list of known flaky ones:
    it's a new test in Mutter 46 and not yet reliable.
  * Add build-Compile-schemas-locally-for-use-in-build-tests.patch to compile
    gsettings schemas for use in build tests. Debian CI requires this now and
    it makes rebuilds more resilient against infrastructure changes.
  * Add wayland-Disable-linux-drm-syncobj-v1.patch to avoid enabling major
    new features in Noble updates, as detailed in:
    https://lists.ubuntu.com/archives/ubuntu-release/2024-May/006087.html
  * Add backends-Handle-null-views-instead-of-creating-dummy-view.patch
    to prevent "clutter_frame_clock_notify_presented: code should not be
    reached" log spam in some Nvidia Wayland sessions (LP: #1967707)
  * Add backends-Restore-support-for-X11-cursor-themes.patch to restore
    support for X11 cursor themes, which was lost in Mutter 46.0
    (LP: #2062377, LP: #2063869)

  [ Jeremy Bícha ]
  * Stop compiling gsettings schemas in debian/rules
  * Opt into Salsa CI
  * Drop packaging references to Hurd since this is Linux-only
  * Update libmutter-14-0.symbols
  * Update Breaks
  * Refreshed patches:
    - window-Add-ability-to-override-the-edge-constraints.patch
    - x11-Add-support-for-fractional-scaling-using-Randr.patch
  * Drop upstreamed patches:
    - Nvidia-secondary-GPU-copy-acceleration.patch
    - compositor-sync-ring-Allow-the-gpu_fence-to-be-moved.patch

  [ Marco Trevisan ]
  * Add place-Always-center-initial-setup.patch (LP: #2016013)

 -- Daniel van Vugt <email address hidden> Fri, 28 Jun 2024 17:35:12 +0800

[Chris Halse Rogers (raof)]

Re-opening this bug as the SRU did not, in fact, fix it, but we don't want to have to re-verify all the other fixes in this upload.

[Kevin (the7thstranger)]

Perhaps related to this issue, I have also noticed that sometimes when I (re)boot my computer in the evening when Night Light is in an early stage, it remains stuck in that initial level of orange set after boot, and it doesn't gradually increase like it should do. Disabling and enabling Night Light then makes it work again.

As with the issue of Night Light being stuck after unlocking my machine in the morning, there is no way to manually trigger the behaviour, and it also doesn't happen all the time, which makes it very hard to debug.

[Daniel van Vugt (vanvugt)]

Keep an eye on bug 2025006 because it will definitely solve some night light reliability issues.

[Kevin (the7thstranger)]

I have been testing those packages (described in bug 2025006) from noble-proposed for a week now, and so far I haven't seen the issue any more.

Usually it happened at least once a week, so I am carefully optimistic. I'd have to not see the issue for a bit longer to be completely convinced it's fixed.

[Daniel van Vugt (vanvugt)]

Let's assume it's fixed in line with bug 2025006, and if you experience the problem again then we can reopen this.