demo-inprocess-egl doesn't terminate properly
Bug #1173191 reported by
Alan Griffiths
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Mir |
Fix Released
|
Medium
|
Kevin DuBois | ||
Bug Description
Control is not returned to bash after exit. (Switching to X and back to VT appears to fix this, but the program doesn't seem to exit properly.)
I suspect this is because there is nothing to terminate thread_loop() on exit.
Related branches
lp:~kdub/mir/android-internal-client
- PS Jenkins bot (community): Approve (continuous-integration)
- Alan Griffiths: Approve
- Robert Ancell: Approve
-
Diff: 2860 lines (+1466/-454)55 files modifiedexamples/CMakeLists.txt (+1/-4)
examples/demo-inprocess-egl/demo_inprocess_egl.cpp (+1/-0)
examples/demo-inprocess-egl/inprocess_egl_client.cpp (+17/-6)
examples/demo-inprocess-egl/inprocess_egl_client.h (+5/-2)
include/server/mir/graphics/egl/mesa_native_display.h (+0/-42)
include/server/mir/graphics/internal_client.h (+43/-0)
include/server/mir/graphics/platform.h (+6/-5)
include/shared/mir_toolkit/mesa/native_display.h (+9/-0)
include/test/mir_test_doubles/mock_interpreter_resource_cache.h (+37/-0)
include/test/mir_test_doubles/mock_swapper.h (+1/-0)
include/test/mir_test_doubles/stub_platform.h (+60/-0)
src/server/graphics/CMakeLists.txt (+0/-4)
src/server/graphics/android/CMakeLists.txt (+3/-0)
src/server/graphics/android/android_platform.cpp (+5/-3)
src/server/graphics/android/android_platform.h (+2/-3)
src/server/graphics/android/default_framebuffer_factory.cpp (+3/-1)
src/server/graphics/android/internal_client.cpp (+42/-0)
src/server/graphics/android/internal_client.h (+52/-0)
src/server/graphics/android/internal_client_window.cpp (+79/-0)
src/server/graphics/android/internal_client_window.h (+58/-0)
src/server/graphics/android/interpreter_cache.cpp (+41/-0)
src/server/graphics/android/interpreter_cache.h (+45/-0)
src/server/graphics/android/interpreter_resource_cache.h (+50/-0)
src/server/graphics/android/server_render_window.cpp (+6/-12)
src/server/graphics/android/server_render_window.h (+4/-3)
src/server/graphics/egl/CMakeLists.txt (+0/-10)
src/server/graphics/gbm/CMakeLists.txt (+2/-0)
src/server/graphics/gbm/gbm_platform.cpp (+38/-12)
src/server/graphics/gbm/gbm_platform.h (+7/-6)
src/server/graphics/gbm/internal_client.cpp (+40/-0)
src/server/graphics/gbm/internal_client.h (+55/-0)
src/server/graphics/gbm/internal_native_display.cpp (+65/-112)
src/server/graphics/gbm/internal_native_display.h (+56/-0)
tests/integration-tests/graphics/android/CMakeLists.txt (+1/-0)
tests/integration-tests/graphics/android/test_internal_client.cpp (+132/-0)
tests/integration-tests/graphics/gbm/test_buffer_integration.cpp (+2/-2)
tests/integration-tests/test_display_info.cpp (+5/-3)
tests/integration-tests/test_drm_auth_magic.cpp (+2/-2)
tests/integration-tests/test_surfaceloop.cpp (+6/-5)
tests/mir_test_framework/testing_server_options.cpp (+2/-2)
tests/unit-tests/client/gbm/test_gbm_client_platform.cpp (+4/-0)
tests/unit-tests/frontend/test_session_mediator.cpp (+2/-26)
tests/unit-tests/frontend/test_session_mediator_android.cpp (+3/-27)
tests/unit-tests/frontend/test_session_mediator_gbm.cpp (+2/-2)
tests/unit-tests/graphics/CMakeLists.txt (+0/-1)
tests/unit-tests/graphics/android/CMakeLists.txt (+4/-1)
tests/unit-tests/graphics/android/test_internal_client.cpp (+87/-0)
tests/unit-tests/graphics/android/test_internal_client_interpreter.cpp (+149/-0)
tests/unit-tests/graphics/android/test_interpreter_buffer_cache.cpp (+86/-0)
tests/unit-tests/graphics/android/test_server_interpreter.cpp (+24/-74)
tests/unit-tests/graphics/egl/CMakeLists.txt (+0/-6)
tests/unit-tests/graphics/gbm/CMakeLists.txt (+2/-0)
tests/unit-tests/graphics/gbm/test_gbm_platform.cpp (+20/-0)
tests/unit-tests/graphics/gbm/test_internal_client.cpp (+43/-0)
tests/unit-tests/graphics/gbm/test_internal_native_display.cpp (+57/-78)
| Changed in mir: | |
| importance: | Undecided → Medium |
| Changed in mir: | |
| status: | New → Triaged |
| Changed in mir: | |
| assignee: | nobody → Kevin DuBois (kdub) |
| status: | Triaged → Fix Released |
To post a comment you must log in.
