Gnome-shell crash after turn display off in mutter 3.36.6-ubuntu0.20.04.1 (NVIDIA)

Bug #1896818 reported by Cristiano Fraga G. Nunes
30
This bug affects 2 people
Affects Status Importance Assigned to Milestone
mutter (Ubuntu)
Fix Released
High
Marco Trevisan (Treviño)
Focal
Fix Released
High
Marco Trevisan (Treviño)

Bug Description

In Ubuntu 20.04.1 [focal-proposed]:

[ Impact ]
GNOME Shell crash after updating mutter to version 3.36.6-ubuntu0.20.04.1.
With the old version (3.36.4-0ubuntu0.20.04.1) and (3.36.4-0ubuntu0.20.04.2), all works fine.

[ Test case ]
Steps to reproduce the error:
1) Lock screen;
2) Turn off the display;
3) Turn on the display;

And the Gnome Shell crash and show us a blank screen with the message:
"Oh no! Something has gone wrong."

[ Regression potential ]
This very fix was already handled as part of bug #1889090 and we had no regression report,.

However, one possible is that the screen size is not set properly for the session, or that panning is enabled.

[ Workaround ]
Downgrade the libmutter package temporarily.

ProblemType: BugDistroRelease: Ubuntu 20.04
Package: mutter 3.36.6-1ubuntu0.20.04.1
ProcVersionSignature: Ubuntu 5.4.0-49.53-generic 5.4.65
Uname: Linux 5.4.0-49-generic x86_64
NonfreeKernelModules: nvidia_modeset nvidia
ApportVersion: 2.20.11-0ubuntu27.9
Architecture: amd64
CasperMD5CheckResult: skip
CurrentDesktop: ubuntu:GNOME
Date: Wed Sep 23 15:57:17 2020
InstallationDate: Installed on 2020-08-15 (38 days ago)
InstallationMedia: Ubuntu 20.04.1 2020.08.15 LTS "Custom Focal Fossa" (20200815)SourcePackage: mutter
UpgradeStatus: No upgrade log present (probably fresh install)
modified.conffile..etc.default.apport:
 # set this to 0 to disable apport, or to 1 to enable it
 # you can temporarily override this with
 # sudo service apport start force_start=1
 enabled=0
mtime.conffile..etc.default.apport: 2020-08-15T17:44:36

Revision history for this message
Cristiano Fraga G. Nunes (cfgnunes) wrote :
Revision history for this message
Cristiano Fraga G. Nunes (cfgnunes) wrote :
Revision history for this message
Cristiano Fraga G. Nunes (cfgnunes) wrote :
Revision history for this message
Cristiano Fraga G. Nunes (cfgnunes) wrote :
Revision history for this message
Cristiano Fraga G. Nunes (cfgnunes) wrote :
Revision history for this message
Cristiano Fraga G. Nunes (cfgnunes) wrote :
Revision history for this message
Cristiano Fraga G. Nunes (cfgnunes) wrote :
Revision history for this message
Cristiano Fraga G. Nunes (cfgnunes) wrote :
description: updated
Revision history for this message
Cristiano Fraga G. Nunes (cfgnunes) wrote :

In my syslog:

gnome-shell[3407]: mutter:ERROR:../src/backends/x11/meta-monitor-manager-xrandr.c:211:meta_monitor_manager_xrandr_update_screen_size: assertion failed: (width > 0 && height > 0 && scale > 0)

gnome-shell[3407]: Bail out! mutter:ERROR:../src/backends/x11/meta-monitor-manager-xrandr.c:211:meta_monitor_manager_xrandr_update_screen_size: assertion failed: (width > 0 && height > 0 && scale > 0)

description: updated
description: updated
summary: Gnome-shell crash after turn display off in mutter
- 3.36.6-ubuntu0.20.04.1 (mostly on Nvidia)
+ 3.36.6-ubuntu0.20.04.1 (NVIDIA)
Revision history for this message
Cristiano Fraga G. Nunes (cfgnunes) wrote :

I've tested in a fresh install also.

description: updated
description: updated
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

Oh, crazy... This was supposed to be fixed already, looks like something broke when adapting the patch?!

Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

I mean, this looks like bug #1889090 is back..

Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

Thanks for making the bug SRU compliant, I've uploaded a new version.

In the mean time you can probably test it using the ppa at
 https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/4267

description: updated
Changed in mutter (Ubuntu):
status: New → In Progress
importance: Undecided → High
assignee: nobody → Marco Trevisan (Treviño) (3v1n0)
status: In Progress → Fix Released
Changed in mutter (Ubuntu Focal):
assignee: nobody → Marco Trevisan (Treviño) (3v1n0)
importance: Undecided → High
status: New → In Progress
Revision history for this message
Cristiano Fraga G. Nunes (cfgnunes) wrote :

Thanks, Trevinho! ;)

Revision history for this message
Cristiano Fraga G. Nunes (cfgnunes) wrote :

I've tested the mutter 3.36.6-1ubuntu0.20.04.2~wip1 (in the PPA).

Now, all works fine!

Thanks for your attention and work.

Revision history for this message
Brian Murray (brian-murray) wrote : Please test proposed package

Hello Cristiano, or anyone else affected,

Accepted mutter into focal-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/mutter/3.36.6-1ubuntu0.20.04.2 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-focal to verification-done-focal. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-focal. 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.

Changed in mutter (Ubuntu Focal):
status: In Progress → Fix Committed
tags: added: verification-needed verification-needed-focal
Revision history for this message
Cristiano Fraga G. Nunes (cfgnunes) wrote :

Hi, Brian and Marco,

I've tested the mutter 3.36.6-1ubuntu0.20.04.2 (from "focal-proposed" repository).

Now, all works fine! Without problems.
Thanks for your attention and work! :)

tags: added: verification-done verification-done-focal
removed: verification-needed verification-needed-focal
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package mutter - 3.36.6-1ubuntu0.20.04.2

---------------
mutter (3.36.6-1ubuntu0.20.04.2) focal; urgency=medium

  * xrandr-scaling: Never try to set invalid screen sizes (again!)
    Looks like this change (introduced to fix LP:1889090) got lost while
    rebasing previous fixes, gbp has been a bad guy this time! (LP: #1896818)

mutter (3.36.6-1ubuntu0.20.04.1) focal; urgency=medium

  * Merge with debian, containing new stable release (LP: #1896332)
  * debian/patches:
    - Refresh patches and drop applied ones
    - Don't set the font-dpi on clutter-backend-x11 (LP: #1892440)
    - Cherry-pick upstream night-switch fixes for 3.36 branch
    - Cherry-pick upstream fixes for handling X11 devices removal
      (LP: #1558886)
  * xrandr-scaling: Don't update UI scale factor on xrandr-manager creation
    (LP: #1892440)
  * debian/control: Revert pipewire version bumping
  * Remaining changes with debian:
    - debian/control:
      + Update VCS flags to point to ubuntu salsa branch
    - debian/gbp.conf: update branch to point to ubuntu/master
    - debian/patches/x11-Add-support-for-fractional-scaling-using-Randr.patch:
      + X11: Add support for fractional scaling using Randr

mutter (3.36.6-1) unstable; urgency=medium

  * Team upload
  * New upstream release
  * Update symbols file
  * d/control.in, d/rules: Enable Pipewire on Debian (but not Ubuntu).
    Now that we have Pipewire 0.3, we can enable screencasting and remote
    desktop support again.

mutter (3.36.5-1) unstable; urgency=medium

  * Team upload
  * New upstream release
    - Screencast fixes and improvements
    - Fix glitches when subsurfaces extend outside the toplevel
    - Improve background display in overview workspace switcher
    - Fix wine copy & paste
    - Plug memory leaks
  * Drop patches that were applied upstream

mutter (3.36.4-1) unstable; urgency=medium

  * New upstream stable release (LP: #1887998)
    - Fix crash on area screenshots with fractional scaling
    - Do not paint textures of fully obscured windows
    - Turn off CRTCs as well when enabling DPMS
    - Improve selection support
    - Use a more appropriate combine function on opaque areas
    - Fix remote desktop being broken without screencast session
    - Fix popovers disappearing on wayland and HiDPI
    - Fixed crashes (LP: #1870867, LP: #1857947)
    - Plugged memory leaks
  * d/p/screen-cast-Let-the-reason-for-recording-determine-what-t.patch,
    d/p/screen-cast-src-Add-flag-to-maybe_record.patch,
    d/p/screen-cast-src-Fix-signedness-of-timestamp-field.patch,
    d/p/screen-cast-src-Make-record-functions-return-an-error-whe.patch,
    d/p/screen-cast-src-Make-the-two-record-vfuncs-more-similarly.patch,
    d/p/screen-cast-src-Record-follow-up-frame-after-timeout.patch,
    d/p/screen-cast-src-Remove-follow-up-timeout-source-on-disabl.patch,
    d/p/screen-cast-src-Use-G_USEC_PER_SEC-instead-of-1000000.patch,
    d/p/screen-cast-window-stream-src-Fix-indentation.patch:
    - Import more fixes for screencasting and remote desktop

 -- Marco Trevisan (Treviño) <email address hidden> Wed, 23 Sep 2020 22:03:53 +0200

Changed in mutter (Ubuntu Focal):
status: Fix Committed → Fix Released
Revision history for this message
Brian Murray (brian-murray) wrote : Update Released

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.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Related questions

Remote bug watches

Bug watches keep track of this bug in other bug trackers.