Nested servers don't apply their display configuration at startup
Bug #1492269 reported by
Alexandros Frantzis
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Mir |
Fix Released
|
High
|
Alan Griffiths | ||
| 0.17 |
Fix Released
|
Undecided
|
Unassigned | ||
| mir (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
A nested server has its own display configuration policy, which is applied to the display configuration it gets from the host at startup. However, that change is not propagate to the host server, leaving the two servers out of sync.
Note that subsequent configuration changes at the nested server are propagated to the host, it's only the initial config that is the problem.
Related branches
lp://staging/~alan-griffiths/mir/nested-server-applies-display-config-policy
- PS Jenkins bot (community): Approve (continuous-integration)
- Kevin DuBois (community): Approve
- Alexandros Frantzis (community): Approve
- Andreas Pokorny (community): Approve
-
Diff: 225 lines (+119/-17)4 files modifiedinclude/platform/mir/graphics/display_configuration.h (+3/-0)
src/platform/graphics/display_configuration.cpp (+23/-0)
src/server/graphics/nested/display.cpp (+9/-2)
tests/acceptance-tests/test_nested_mir.cpp (+84/-15)
lp://staging/mir/0.17
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
- Mir development team: Pending requested
-
Diff: 865 lines (+427/-65)23 files modifiedCMakeLists.txt (+1/-1)
debian/changelog (+20/-0)
debian/control (+1/-0)
debian/mir-client-platform-mesa-dev.install (+1/-0)
include/platform/mir/graphics/display_configuration.h (+3/-0)
src/platform/graphics/display_configuration.cpp (+23/-0)
src/platforms/android/client/android_client_platform.cpp (+3/-14)
src/platforms/common/client/mir/CMakeLists.txt (+1/-0)
src/platforms/common/client/mir/weak_egl.cpp (+65/-0)
src/platforms/common/client/mir/weak_egl.h (+47/-0)
src/platforms/mesa/CMakeLists.txt (+10/-1)
src/platforms/mesa/client/client_platform.cpp (+6/-17)
src/platforms/mesa/mir-client-platform-mesa-dev.pc.in (+1/-1)
src/platforms/mesa/mir-client-platform-mesa.pc.in (+7/-0)
src/server/graphics/nested/display.cpp (+39/-6)
src/server/graphics/nested/display.h (+6/-0)
src/server/input/display_input_region.cpp (+4/-1)
src/server/scene/rendering_tracker.cpp (+12/-0)
src/server/scene/rendering_tracker.h (+2/-1)
src/server/scene/surface_stack.cpp (+11/-8)
tests/acceptance-tests/test_nested_mir.cpp (+109/-15)
tests/unit-tests/input/test_display_input_region.cpp (+11/-0)
tests/unit-tests/scene/test_surface_stack.cpp (+44/-0)
| Changed in mir: | |
| assignee: | Alexandros Frantzis (afrantzis) → Alan Griffiths (alan-griffiths) |
| status: | New → In Progress |
| importance: | Undecided → High |
| milestone: | none → 0.18.0 |
Revision history for this message
| PS Jenkins bot (ps-jenkins) wrote | #1 |
| Changed in mir: | |
| status: | In Progress → Fix Committed |
| Changed in mir (Ubuntu): | |
| status: | New → Fix Released |
| Changed in mir: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fix committed into lp:mir at revision None, scheduled for release in mir, milestone 0.18.0