Nvidia Wayland sessions sometimes flood the log with "clutter_frame_clock_notify_presented: code should not be reached"

[Daniel van Vugt]

[ Impact ]

Nvidia Wayland sessions sometimes flood the log with "clutter_frame_clock_notify_presented: code should not be reached"

This is almost harmless except that it wastes disk space and the high frequency of messages (one per frame) impacts desktop performance.

[ Test Plan ]

0. Start with either an Nvidia-only desktop providing that bug 2060268 is NOT fixed yet; or an Nvidia desktop which also has an integrated GPU enabled such that Nvidia is configured as the primary (boot) GPU in the BIOS and a monitor is plugged into EACH GPU.

1. Log into a Wayland session.

2. Run:

   journalctl -f /usr/bin/gnome-shell

   and verify there is not a flood of messages with each screen redraw.

[ Where problems could occur ]

Wayland multi-GPU startup is the code affected here so any configuration expecting to use multiple GPUs simultaneously in a Wayland session may be affected.

While bug 2060268 is unresolved even single GPU Nvidia systems will be affected due to the phantom GPU resulting from that bug.

[ Original description ]

Nvidia Wayland sessions sometimes flood the log with:

[ 13.105877] computer gnome-shell[1264]: (../clutter/clutter/clutter-frame-clock.c:332):clutter_frame_clock_notify_presented: code should not be reached
[ 13.106163] computer gnome-shell[1264]: (../clutter/clutter/clutter-frame-clock.c:332):clutter_frame_clock_notify_presented: code should not be reached
[ 13.112490] computer gnome-shell[1264]: (../clutter/clutter/clutter-frame-clock.c:332):clutter_frame_clock_notify_presented: code should not be reached
[ 13.125302] computer gnome-shell[1264]: (../clutter/clutter/clutter-frame-clock.c:332):clutter_frame_clock_notify_presented: code should not be reached
[ 13.142242] computer gnome-shell[1264]: (../clutter/clutter/clutter-frame-clock.c:332):clutter_frame_clock_notify_presented: code should not be reached

I've been seeing the issue for months although it only seems to happen *after* something has gone wrong in mutter. Not by default.

[Daniel van Vugt (vanvugt)]

It appears the main cause of this is presently bug 1959888 when you log in with a secondary monitor on a secondary (Nvidia) GPU. So the log messages stop once you have that fix. Although I would still like to understand it better because that's the second bug I've seen trigger the same messages.

[Daniel van Vugt (vanvugt)]

Fix proposed: https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/2358

[LittleBigBrain (braingateway)]

flooded with ` gnome-shell[3217]: (../clutter/clutter/clutter-frame-clock.c:332):clutter_frame_clock_notify_presented: code should not be reached`

Then flooded with:
> Page flip discarded: drmModeAtomicCommit: Invalid argument
> Failed to post KMS update: drmModeAtomicCommit: Invalid argument

Then external screen freeze but internal LCD works fine.

[LittleBigBrain (braingateway)]

maybe related:
> [drm:__nv_drm_gem_nvkms_map.part.0 [nvidia_drm]] *ERROR* [nvidia-drm] [GPU ID 0x00000100] Failed to map NvKmsKapiMemory Failed to map NvKmsKapiMemory 0x0000000062c80dd4

Mutter 42.5

https://bugs.launchpad.net/ubuntu/+source/mutter/+bug/1999830

[Daniel van Vugt (vanvugt)]

Comment #3 is bug 1968040, not this one.

Bug 1999830 may also help to reproduce the issue.

[Daniel van Vugt (vanvugt)]

The upstream bug https://gitlab.gnome.org/GNOME/mutter/-/issues/2489 also shows there are other ways to trigger this warning.

[Daniel van Vugt (vanvugt)]

This should fix most cases of the log flood:
https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/3115

[zagglez (zagglez)]

Ubuntu 24.04, two monitors, wayland, nvidia 535

[Daniel van Vugt (vanvugt)]

This is happening in 24.04/24.10 now due to bug 2060268, so please see the workarounds in that bug.

[Launchpad Janitor (janitor)]

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

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

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

  [ Daniel van Vugt ]
  * 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)

 -- Daniel van Vugt <email address hidden> Tue, 25 Jun 2024 15:26:37 +0800

[Timo Aaltonen (tjaalton)]

Hello Daniel, 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.

[Daniel van Vugt (vanvugt)]

Verified fixed in 46.2-1ubuntu0.24.04.1 using the test plan in the bug description.

[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)]

The verification of the Stable Release Update for mutter has completed successfully and the package is now being released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.