Mir

[enhancement] need nested mir (in X) server

Bug #1398038 reported by Allison Karlitskaya
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mir
Fix Released
Wishlist
Cemil Azizoglu
Mir 0.15.0
mir (Ubuntu)
New
Undecided
Unassigned

Bug Description

Another bug from the "requested in person a few times but never written down" category:

The number one thing that we could do to improve testability of toolkit ports on mir is to have a nested mir server that could be run under X11 (instead of having to run mir as root on a VT, switch back and forth, chmod the socket, etc.)

Related branches

lp:~cemil-azizoglu/mir/mir-on-x
PS Jenkins bot (community): Approve (continuous-integration)
Kevin DuBois (community): Approve
Alan Griffiths: Approve
Alexandros Frantzis (community): Approve
Robert Carr (community): Approve
Chris Halse Rogers: Approve
Cemil Azizoglu: Pending requested
Diff: 3224 lines (+2433/-85)
56 files modified
CMakeLists.txt (+6/-3)
src/platforms/CMakeLists.txt (+2/-2)
src/platforms/mesa/server/CMakeLists.txt (+3/-0)
src/platforms/mesa/server/common/buffer_allocator.cpp (+105/-18)
src/platforms/mesa/server/common/buffer_allocator.h (+9/-1)
src/platforms/mesa/server/common/display_helpers.cpp (+28/-17)
src/platforms/mesa/server/common/display_helpers.h (+8/-1)
src/platforms/mesa/server/common/ipc_operations.cpp (+4/-5)
src/platforms/mesa/server/common/ipc_operations.h (+2/-2)
src/platforms/mesa/server/kms/guest_platform.cpp (+2/-2)
src/platforms/mesa/server/kms/platform.cpp (+5/-4)
src/platforms/mesa/server/x11/CMakeLists.txt (+61/-0)
src/platforms/mesa/server/x11/display.cpp (+274/-0)
src/platforms/mesa/server/x11/display.h (+131/-0)
src/platforms/mesa/server/x11/display_buffer.cpp (+68/-0)
src/platforms/mesa/server/x11/display_buffer.h (+59/-0)
src/platforms/mesa/server/x11/display_configuration.cpp (+63/-0)
src/platforms/mesa/server/x11/display_configuration.h (+52/-0)
src/platforms/mesa/server/x11/display_group.cpp (+42/-0)
src/platforms/mesa/server/x11/display_group.h (+51/-0)
src/platforms/mesa/server/x11/gl_context.cpp (+45/-0)
src/platforms/mesa/server/x11/gl_context.h (+52/-0)
src/platforms/mesa/server/x11/input/dispatchable.cpp (+132/-0)
src/platforms/mesa/server/x11/input/dispatchable.h (+52/-0)
src/platforms/mesa/server/x11/input/input.cpp (+60/-0)
src/platforms/mesa/server/x11/input/input_device.cpp (+54/-0)
src/platforms/mesa/server/x11/input/input_device.h (+66/-0)
src/platforms/mesa/server/x11/input/input_platform.cpp (+51/-0)
src/platforms/mesa/server/x11/input/input_platform.h (+60/-0)
src/platforms/mesa/server/x11/platform.cpp (+135/-0)
src/platforms/mesa/server/x11/platform.h (+61/-0)
src/platforms/mesa/server/x11/symbols.map (+18/-0)
src/platforms/mesa/server/x11/xserver_connection.h (+47/-0)
tests/CMakeLists.txt (+4/-0)
tests/acceptance-tests/test_client_library.cpp (+10/-10)
tests/include/mir/test/doubles/mock_input_sink.h (+44/-0)
tests/include/mir/test/doubles/mock_x11.h (+78/-0)
tests/mir_test_doubles/CMakeLists.txt (+6/-0)
tests/mir_test_doubles/mock_x11.cpp (+133/-0)
tests/mir_test_doubles/platform_factory.cpp (+9/-1)
tests/mir_test_doubles/stub_buffer.cpp (+3/-3)
tests/mir_test_framework/stubbed_graphics_platform.cpp (+2/-2)
tests/unit-tests/CMakeLists.txt (+8/-2)
tests/unit-tests/client/CMakeLists.txt (+1/-1)
tests/unit-tests/graphics/CMakeLists.txt (+10/-5)
tests/unit-tests/graphics/mesa/CMakeLists.txt (+4/-0)
tests/unit-tests/graphics/mesa/common/CMakeLists.txt (+0/-2)
tests/unit-tests/graphics/mesa/common/test_drm_helper.cpp (+1/-1)
tests/unit-tests/graphics/mesa/kms/CMakeLists.txt (+2/-0)
tests/unit-tests/graphics/mesa/kms/test_buffer_allocator.cpp (+3/-2)
tests/unit-tests/graphics/mesa/kms/test_gbm_buffer.cpp (+1/-1)
tests/unit-tests/graphics/mesa/x11/CMakeLists.txt (+6/-0)
tests/unit-tests/graphics/mesa/x11/test_display.cpp (+101/-0)
tests/unit-tests/graphics/mesa/x11/test_platform.cpp (+125/-0)
tests/unit-tests/input/CMakeLists.txt (+6/-0)
tests/unit-tests/input/test_x11_dispatchable.cpp (+68/-0)
Daniel van Vugt (vanvugt)
Changed in mir:
importance: Undecided → Wishlist
status: New → Triaged
tags: added: enhancement
Revision history for this message
kevin gunn (kgunn72) wrote :

@duflu - seems a common request, could you consider?

Changed in mir:
assignee: nobody → Daniel van Vugt (vanvugt)
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

I think we're waiting for Chris' platform separation stuff before attempting this. Because we'd prefer it if "X" was just another platform Mir can run on. And in the mean time I have significant other work to finish.

Changed in mir:
assignee: Daniel van Vugt (vanvugt) → nobody
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Actually Mir-on-X is very similar to Mir-on-Mir (nested). The only issue is the latter needs to become a first-class platform with its own separate driver binary before we can cleanly copy it and make a Mir-on-X.

kevin gunn (kgunn72)
Changed in mir:
assignee: nobody → Cemil Azizoglu (cemil-azizoglu)
status: Triaged → In Progress
Daniel van Vugt (vanvugt)
Changed in mir:
milestone: none → 0.14.0
summary: - need nested mir (in X) server
+ [enhancement] need nested mir (in X) server
Andreas Pokorny (andreas-pokorny)
Changed in mir:
milestone: 0.14.0 → 0.15.0
Daniel van Vugt (vanvugt)
Changed in mir:
status: In Progress → Fix Committed
Daniel van Vugt (vanvugt)
Changed in mir:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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