[regression] Fullscreen software surfaces (like Xmir -sw) can crash the Mir server
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mir |
Fix Released
|
High
|
Daniel van Vugt | ||
mir (Ubuntu) |
Fix Released
|
High
|
Unassigned |
Bug Description
Xmir -sw (software) mode crashes the Mir server. If not immediately on startup then soon after running a few X clients...
/usr/bin/
(gdb) bt
#0 0x00007f8586f64cc0 in gbm_bo_
from /usr/lib/
#1 0x00007f85858212af in ?? ()
from /usr/lib/
#2 0x00007f8585822099 in ?? ()
from /usr/lib/
#3 0x00007f858ac980ac in ?? ()
from /usr/lib/
#4 0x00007f858aca1176 in ?? ()
from /usr/lib/
#5 0x00007f858ad23431 in ?? ()
from /usr/lib/
#6 0x00007f858a910ed0 in ?? () from /usr/lib/
#7 0x00007f858821d6aa in start_thread (arg=0x7f8583ba
at pthread_
#8 0x00007f858a37deed in clone ()
at ../sysdeps/
Related branches
- PS Jenkins bot (community): Approve (continuous-integration)
- Chris Halse Rogers: Approve
- Alan Griffiths: Approve
-
Diff: 144 lines (+71/-13)2 files modifiedsrc/platforms/mesa/server/kms/display_buffer.cpp (+11/-12)
tests/unit-tests/graphics/mesa/kms/test_display_buffer.cpp (+60/-1)
description: | updated |
Changed in xorg-server (Ubuntu): | |
status: | New → Invalid |
Changed in mir: | |
assignee: | nobody → Daniel van Vugt (vanvugt) |
Changed in mir: | |
milestone: | none → 0.17.0 |
status: | New → In Progress |
summary: |
- Xmir -sw (software) mode crashes the Mir server + [regression] Xmir -sw (software) mode crashes the Mir server |
tags: | added: bufferstream-regression regression |
tags: | removed: bufferstream-regression |
affects: | xorg-server (Ubuntu) → mir (Ubuntu) |
Changed in mir (Ubuntu): | |
status: | Invalid → Triaged |
importance: | Undecided → High |
Changed in mir: | |
status: | Fix Committed → Fix Released |
Apparently the GBM bypass logic was quite safe for a couple of years. The offending regression only happened fairly recently in:
------- ------- ------- ------- ------- ------- ------- ------- ----
revno: 2337 [merge]
author: Kevin DuBois <email address hidden>
committer: Tarmac
branch nick: development-branch
timestamp: Mon 2015-02-23 14:50:14 +0000
message:
graphics: move the post() command from the DisplayBuffers to the new DisplayGr
oup interface. This allows platforms to designate which DisplayBuffers must be p
osted in a common function.
Approved by PS Jenkins bot, Alberto Aguirre, Andreas Pokorny, Alan Griffiths, ------- ------- ------- ------- ------- ------- ------- ----
Alexandros Frantzis.
-------