Bug #1502896 “CI segfault in mir-mediumtests-runner-mako in clie...” : Bugs : Mir

Alexandros Frantzis (afrantzis) on 2015-10-05

tags:

added: testsfail

Revision history for this message

Alexandros Frantzis (afrantzis) wrote on 2015-10-06:

#1

https://bugs.launchpad.net/mir/+bug/1503281 could be another instance of this problem

Revision history for this message

Alexandros Frantzis (afrantzis) wrote on 2015-10-06:

#2

Another instance https://jenkins.qa.ubuntu.com/job/mir-mediumtests-runner-mako/6901/console

/bin/bash: line 1: 6423 Segmentation fault mir_demo_server --test-client /usr/bin/mir_demo_client_fingerpaint
+ mir_rc=-1

Changed in mir:
importance:	Undecided → High

Revision history for this message

Alan Griffiths (alan-griffiths) wrote on 2015-10-06:

#3

Another one: https://jenkins.qa.ubuntu.com/job/mir-mediumtests-runner-touch/6916/console

Changed in mir:
status:	New → Confirmed

Revision history for this message

Andreas Pokorny (andreas-pokorny) wrote on 2015-10-08:

#4

Download full text (5.3 KiB)

I can reproduce that with krillin here.. and on mx4 I sometimes get it to hang during shutdown..

hread 1 (Thread 0xb6f5e000 (LWP 21998)):
#0 0xb6e59d44 in __libc_do_syscall ()
   from /lib/arm-linux-gnueabihf/libpthread.so.0
No symbol table info available.
#1 0xb6e556b0 in pthread_cond_wait@@GLIBC_2.4 ()
   from /lib/arm-linux-gnueabihf/libpthread.so.0
No symbol table info available.
#2 0xb6ee56e4 in std::condition_variable::wait(std::unique_lock<std::mutex>&)
    () from /usr/lib/arm-linux-gnueabihf/libstdc++.so.6
No symbol table info available.
#3 0xb69d18e2 in std::condition_variable::wait<std::__future_base::_State_baseV2::wait()::{lambda()#1}>(std::unique_lock<std::mutex>&, std::__future_base::_State_baseV2::wait()::{lambda()#1}) (this=0x1d2420, __lock=..., __p=...)
---Type <return> to continue, or q <return> to quit---
    at /usr/arm-linux-gnueabihf/include/c++/4.9.3/condition_variable:98
No locals.
#4 0xb69d0ec6 in std::__future_base::_State_baseV2::wait (this=0x1d2400)
    at /usr/arm-linux-gnueabihf/include/c++/4.9.3/future:323
        __lock = {_M_device = 0x1d2408, _M_owns = true}
#5 0xb69d26bc in std::__basic_future<void>::wait (this=0x1d9a78)
    at /usr/arm-linux-gnueabihf/include/c++/4.9.3/future:596
No locals.
#6 0xb6a5c446 in mir::compositor::MultiThreadedCompositor::destroy_compositing_threads (this=0x1ce5f8)
    at /home/andreas/mir/load-all-platforms/src/server/compositor/multi_threaded_compositor.cpp:341
        f = @0x1d9a78: {<std::__basic_future<void>> = {<std::__future_base> = {<No data fields>},
            _M_state = std::shared_ptr (count 2, weak 0) 0x1d2400}, <No data fields>}
        __for_range = std::vector of length 1, capacity 1 = {
          {<std::__basic_future<void>> = {<std::__future_base> = {<No data fields>}, _M_state = std::shared_ptr (count 2, weak 0) 0x1d2400}, <No data fields>}}
        __for_begin = {<std::__basic_future<void>> = {<std::__future_base> = {<No data fields>},
            _M_state = std::shared_ptr (count 2, weak 0) 0x1d2400}, <No data fields>}
---Type <return> to continue, or q <return> to quit---
        __for_end = {<std::__basic_future<void>> = {<std::__future_base> = {<No data fields>},
            _M_state = <error reading variable: Cannot access memory at address 0x15>}, <No data fields>}
#7 0xb6a5c15e in mir::compositor::MultiThreadedCompositor::stop (
    this=0x1ce5f8)
    at /home/andreas/mir/load-all-platforms/src/server/compositor/multi_threaded_compositor.cpp:307
        started = mir::compositor::CompositorState::started
        cleanup_if_unwinding = {unwind = {__this = 0x1ce5f8}}
#8 0xb6967c72 in mir::DisplayServer::run (this=0xbebc41e8)
    at /home/andreas/mir/load-all-platforms/src/server/display_server.cpp:206
        server = @0x16aa50: {emergency_cleanup =
    std::shared_ptr (count 2, weak 2) 0x16a284, graphics_platform =
    std::shared_ptr (count 1, weak 1) 0x16df64,
          display = std::shared_ptr (count 6, weak 1) 0x16e074,
          input_dispatcher = std::shared_ptr (count 2, weak 1) 0x1b8410,
          compositor = std::shared_ptr (count 2, weak 1) 0x1ce5f8, connector =
    std::shared_ptr (count 1, weak 1) 0x1cee2c,
          p...

I can reproduce that with krillin here.. and on mx4 I sometimes get it to hang during shutdown..

hread 1 (Thread 0xb6f5e000 (LWP 21998)):
#0  0xb6e59d44 in __libc_do_syscall ()
   from /lib/arm-linux-gnueabihf/libpthread.so.0
No symbol table info available.
#1  0xb6e556b0 in pthread_cond_wait@@GLIBC_2.4 ()
   from /lib/arm-linux-gnueabihf/libpthread.so.0
No symbol table info available.
#2  0xb6ee56e4 in std::condition_variable::wait(std::unique_lock<std::mutex>&)
    () from /usr/lib/arm-linux-gnueabihf/libstdc++.so.6
No symbol table info available.
#3  0xb69d18e2 in std::condition_variable::wait<std::__future_base::_State_baseV2::wait()::{lambda()#1}>(std::unique_lock<std::mutex>&, std::__future_base::_State_baseV2::wait()::{lambda()#1}) (this=0x1d2420, __lock=..., __p=...)
---Type <return> to continue, or q <return> to quit---
    at /usr/arm-linux-gnueabihf/include/c++/4.9.3/condition_variable:98
No locals.
#4  0xb69d0ec6 in std::__future_base::_State_baseV2::wait (this=0x1d2400)
    at /usr/arm-linux-gnueabihf/include/c++/4.9.3/future:323
        __lock = {_M_device = 0x1d2408, _M_owns = true}
#5  0xb69d26bc in std::__basic_future<void>::wait (this=0x1d9a78)
    at /usr/arm-linux-gnueabihf/include/c++/4.9.3/future:596
No locals.
#6  0xb6a5c446 in mir::compositor::MultiThreadedCompositor::destroy_compositing_threads (this=0x1ce5f8)
    at /home/andreas/mir/load-all-platforms/src/server/compositor/multi_threaded_compositor.cpp:341
        f = @0x1d9a78: {<std::__basic_future<void>> = {<std::__future_base> = {<No data fields>}, 
            _M_state = std::shared_ptr (count 2, weak 0) 0x1d2400}, <No data fields>}
        __for_range = std::vector of length 1, capacity 1 = {
          {<std::__basic_future<void>> = {<std::__future_base> = {<No data fields>}, _M_state = std::shared_ptr (count 2, weak 0) 0x1d2400}, <No data fields>}}
        __for_begin = {<std::__basic_future<void>> = {<std::__future_base> = {<No data fields>}, 
            _M_state = std::shared_ptr (count 2, weak 0) 0x1d2400}, <No data fields>}
---Type <return> to continue, or q <return> to quit---
        __for_end = {<std::__basic_future<void>> = {<std::__future_base> = {<No data fields>}, 
            _M_state = <error reading variable: Cannot access memory at address 0x15>}, <No data fields>}
#7  0xb6a5c15e in mir::compositor::MultiThreadedCompositor::stop (
    this=0x1ce5f8)
    at /home/andreas/mir/load-all-platforms/src/server/compositor/multi_threaded_compositor.cpp:307
        started = mir::compositor::CompositorState::started
        cleanup_if_unwinding = {unwind = {__this = 0x1ce5f8}}
#8  0xb6967c72 in mir::DisplayServer::run (this=0xbebc41e8)
    at /home/andreas/mir/load-all-platforms/src/server/display_server.cpp:206
        server = @0x16aa50: {emergency_cleanup = 
    std::shared_ptr (count 2, weak 2) 0x16a284, graphics_platform = 
    std::shared_ptr (count 1, weak 1) 0x16df64, 
          display = std::shared_ptr (count 6, weak 1) 0x16e074, 
          input_dispatcher = std::shared_ptr (count 2, weak 1) 0x1b8410, 
          compositor = std::shared_ptr (count 2, weak 1) 0x1ce5f8, connector = 
    std::shared_ptr (count 1, weak 1) 0x1cee2c, 
          prompt_connector = std::shared_ptr (count 1, weak 1) 0x1cf20c, 
          input_manager = std::shared_ptr (count 1, weak 1) 0x1d1fac, 
          main_loop = std::shared_ptr (count 7, weak 1) 0x16a2d0, 
          server_status_listener = std::shared_ptr (count 1, weak 1) 0x1d99a4, 
---Type <return> to continue, or q <return> to quit---
          display_changer = std::shared_ptr (count 3, weak 1) 0x1ce990}
#9  0xb6963cd2 in mir::run_mir(mir::ServerConfiguration&, std::function<void (mir::DisplayServer&)>, std::function<void (int)> const&) (config=..., init=..., 
    terminator=...)
    at /home/andreas/mir/load-all-platforms/src/server/run_mir.cpp:110
        server_ptr = 0xbebc41e8
        main_loop = std::shared_ptr (count 7, weak 1) 0x16a2d0
        fatal_error_strategy = {old_fatal_error_handler = 0xb67d9f31
     <mir::fatal_error_except(char const*, ...)>}
        server = {p = {_M_b = {_M_p = 0x16aa50}}}
        concurrent_calls = {<std::__atomic_base<unsigned int>> = {
            _M_i = 1}, <No data fields>}
        raii = {deleter = {<No data fields>}, owner = true}
#10 0xb698a732 in mir::Server::run (this=0xbebc4288)
    at /home/andreas/mir/load-all-platforms/src/server/server.cpp:404
        emergency_cleanup = std::shared_ptr (count 2, weak 2) 0x16a284
#11 0xb6c7070a in main (argc=3, argv=0xbebc4454)
    at /home/andreas/mir/load-all-platforms/examples/server_example.cpp:108
        context = {
          client_kill_action = std::unique_ptr<mir::time::Alarm> containing 0x1d4c80, 
          server_stop_action = std::unique_ptr<mir::time::Alarm> containing 0x1d5270, test_failed = {<std::atomic_bool> = {_M_base = {
---Type <return> to continue, or q <return> to quit---
                _M_i = false}}, <No data fields>}}
        screen_rotation_filter = std::shared_ptr (count 2, weak 0) 0x1597ec
        server = {self = std::shared_ptr (count 2, weak 0) 0x1584e4}
        quit_filter = std::shared_ptr (count 2, weak 1) 0x159574
        printing_filter = std::shared_ptr (count 2, weak 0) 0x159adc
#12 0x00010f26 in main (argc=3, argv=0xbebc4454)
    at /home/andreas/mir/load-all-platforms/examples/mir_demo_server_loader.cpp:40
        so = 0x14b058
        loaded_main = 0xb6c705f9 <main(int, char const**)>

Changed in mir:
assignee:	nobody → Andreas Pokorny (andreas-pokorny)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2015-10-08:

#5

Fix committed into lp:mir at revision 3006, scheduled for release in mir, milestone 0.18.0

Changed in mir:
status:	Confirmed → Fix Committed

Daniel van Vugt (vanvugt) on 2015-10-09

Changed in mir:
milestone:	none → 0.17.0

Revision history for this message

Daniel van Vugt (vanvugt) wrote on 2015-10-09:

#6

Just missed the branching of 0.17

Changed in mir:
milestone:	0.17.0 → 0.18.0

Revision history for this message

Andreas Pokorny (andreas-pokorny) wrote on 2015-10-10:

#7

The problem still exists. It just seems to happen less frequently:
https://jenkins.qa.ubuntu.com/job/mir-mediumtests-runner-touch/6986/console

Changed in mir:
status:	Fix Committed → Confirmed

Daniel van Vugt (vanvugt) on 2015-10-12

Changed in mir:
assignee:	Andreas Pokorny (andreas-pokorny) → nobody
tags:	added: mako

Revision history for this message

Andreas Pokorny (andreas-pokorny) wrote on 2015-10-14:

#8

Another recent example now with the mir_client_basic: https://jenkins.qa.ubuntu.com/job/mir-mediumtests-runner-touch/7015/console

Revision history for this message

Andreas Pokorny (andreas-pokorny) wrote on 2015-10-15:

#9

set to commited tsince we do have two other bugs to track similar behavior.

Changed in mir:
status:	Confirmed → Fix Committed

Revision history for this message

Andreas Pokorny (andreas-pokorny) wrote on 2015-10-15:

#10

Look for lp:1506137 which also mentions the above scenario with mir_client_basic.

Daniel van Vugt (vanvugt) on 2015-12-04

Changed in mir:
assignee:	nobody → Alan Griffiths (alan-griffiths)

Revision history for this message

Launchpad Janitor (janitor) wrote on 2015-12-22:

#11

Download full text (6.3 KiB)

This bug was fixed in the package mir - 0.18.0+16.04.20151216.1-0ubuntu1

---------------
mir (0.18.0+16.04.20151216.1-0ubuntu1) xenial; urgency=medium

  [ Kevin DuBois ]
  * New upstream release 0.18.0 (https://launchpad.net/mir/+milestone/0.18.0)
    - ABI summary: Only servers need rebuilding;
      . Mirclient ABI unchanged at 9
      . Mirserver ABI bumped to 36
      . Mircommon ABI unchanged at 5
      . Mirplatform ABI unchanged at 11
      . Mirprotobuf ABI unchanged at 3
      . Mirplatformgraphics ABI bumped to 7
      . Mirclientplatform ABI unchanged at 3
      . Mirinputplatform ABI added. Current version is 4
    - Enhancements:
      . Use libinput by default, and remove the android input stack
      . Add x11 input probing
      . Add alternative buffer swapping mechanism internally, available with
        --nbuffers 0
      . Automatic searching and selection of input platforms
      . Better support for themed cursors
      . Add demo client that uses multiple buffer streams in one surface
      . Improve fingerpaint demo to use touch pressure
      . Allow for configuring cursor acceleration, scroll speed and left or
        right handed mice
      . Allow for setting a base display configuration via client api
      . Various nested server multimonitor fixes and stability improvements
      . Remove DepthId from the SurfaceStack
    - Bug fixes:
      . Unit test failures in Display.* on Android (LP: #1519276)
      . Build failure due to missing dependency of client rpc code on mir
        protobuf (LP: #1518372)
      . Test failure in
        NestedServer.display_configuration_reset_when_application_exits
        (LP: #1517990)
      . CI test failures in various NesterServer tests (LP: #1517781)
      . FTBFS with -DMIR_PLATFORM=android (LP: #1517532)
      . Nesting Mir servers with assorted display configs causes lockup
        (LP: #1516670)
      . [testsfail] RaiseSurfaces.motion_events_dont_prevent_raise
        (LP: #1515931)
      . CI test failures in GLMark2Test (LP: #1515660)
      . Shells that inject user input events need to agree with the system
        compositor on the clock to use (LP: #1515515)
      . mircookie-dev is missing nettle-dev dependency (LP: #1514391)
      . Segmentation fault on server shutdown with mesa-kms (LP: #1513901)
      . mircookie requires nettle but libmircookie-dev doesn't depend on it
        (LP: #1513792)
      . libmircookie1 package does not list libnettle as dependency
        (LP: #1513225)
      . display configuration not reset when application exits (LP: #1511798)
      . unplugging external monitor causes nested server to throttle client
        (LP: #1511723)
      . 1/2 screen on external monitor (LP: #1511538)
      . unity-system-compositor crash, no interaction on windowed mode
        (LP: #1511095)
      . [regression] arm64/powerpc cross compile doesn't build any more
        (LP: #1510778)
      . mir_connection_get_egl_pixel_format() crashes if libEGL is loaded
        RTLD_LAZY (LP: #1510218)
      . [multimonitor] nested server surface positioning incorrect
        (LP: #1506846)
      . unity-system-compositor fails to build against lp:mir r3027
   ...

This bug was fixed in the package mir - 0.18.0+16.04.20151216.1-0ubuntu1

---------------
mir (0.18.0+16.04.20151216.1-0ubuntu1) xenial; urgency=medium

[ Kevin DuBois ]
  * New upstream release 0.18.0 (https://launchpad.net/mir/+milestone/0.18.0)
    - ABI summary: Only servers need rebuilding;
      . Mirclient ABI unchanged at 9
      . Mirserver ABI bumped to 36
      . Mircommon ABI unchanged at 5
      . Mirplatform ABI unchanged at 11
      . Mirprotobuf ABI unchanged at 3
      . Mirplatformgraphics ABI bumped to 7
      . Mirclientplatform ABI unchanged at 3
      . Mirinputplatform ABI added. Current version is 4
    - Enhancements:
      . Use libinput by default, and remove the android input stack
      . Add x11 input probing
      . Add alternative buffer swapping mechanism internally, available with
        --nbuffers 0
      . Automatic searching and selection of input platforms
      . Better support for themed cursors
      . Add demo client that uses multiple buffer streams in one surface
      . Improve fingerpaint demo to use touch pressure
      . Allow for configuring cursor acceleration, scroll speed and left or
        right handed mice
      . Allow for setting a base display configuration via client api
      . Various nested server multimonitor fixes and stability improvements
      . Remove DepthId from the SurfaceStack
    - Bug fixes:
      . Unit test failures in Display.* on Android (LP: #1519276)
      . Build failure due to missing dependency of client rpc code on mir
        protobuf (LP: #1518372)
      . Test failure in
        NestedServer.display_configuration_reset_when_application_exits
        (LP: #1517990)
      . CI test failures in various NesterServer tests (LP: #1517781)
      . FTBFS with -DMIR_PLATFORM=android (LP: #1517532)
      . Nesting Mir servers with assorted display configs causes lockup
        (LP: #1516670)
      . [testsfail] RaiseSurfaces.motion_events_dont_prevent_raise
        (LP: #1515931)
      . CI test failures in GLMark2Test (LP: #1515660)
      . Shells that inject user input events need to agree with the system
        compositor on the clock to use (LP: #1515515)
      . mircookie-dev is missing nettle-dev dependency (LP: #1514391)
      . Segmentation fault on server shutdown with mesa-kms (LP: #1513901)
      . mircookie requires nettle but libmircookie-dev doesn't depend on it
        (LP: #1513792)
      . libmircookie1 package does not list libnettle as dependency
        (LP: #1513225)
      . display configuration not reset when application exits (LP: #1511798)
      . unplugging external monitor causes nested server to throttle client
        (LP: #1511723)
      . 1/2 screen on external monitor (LP: #1511538)
      . unity-system-compositor crash, no interaction on windowed mode
        (LP: #1511095)
      . [regression] arm64/powerpc cross compile doesn't build any more
        (LP: #1510778)
      . mir_connection_get_egl_pixel_format() crashes if libEGL is loaded
        RTLD_LAZY (LP: #1510218)
      . [multimonitor] nested server surface positioning incorrect
        (LP: #1506846)
      . unity-system-compositor fails to build against lp:mir r3027
        (LP: #1506537)
      . [regression] Resampled touch events are slightly less regular than they
        used to be (LP: #1506331)
      . [testsfail] Segmentation fault mir_demo_server --test-client
        /usr/bin/mir_demo_client_basic (LP: #1506137)
      . [regression] Can't move/resize windows any more using Alt+left/middle
        buttons (LP: #1505114)
      . CI failure on some 32-bit targets in
        ServerSignal.terminate_handler_is_called_for_SIGTERM and others
        (LP: #1504579)
      . multibufferstreams use surface size instead of stream size
        (LP: #1503317)
      . CI segfault in mir-mediumtests-runner-mako in client tests
        (LP: #1502896)
      . CI segfault in mir-mediumtests-runner-mako after AndroidHardwareSanity
        tests (LP: #1502782)
      . compositing never stops when external monitor is connected
        (LP: #1499039)
      . mir_demo_server crashes with ERROR: Dynamic exception type:
        std::out_of_range - std::exception::what: map::at when an Xmir rootless
        app is closed by its File menu. (LP: #1497128)
      . Nested servers don't apply their display configuration at startup
        (LP: #1492269)
      . mir fails to open the right mouse device in kvm/qemu (LP: #1489522)
      . nexus4 & nexus7 hotplug survival (LP: #1488863)
      . Android uses mir_display_output_invalid_id (0) as an output id value
        (LP: #1463873)
      . DemoRenderer: GL_BLEND is really always enabled even when not required
        (also causes RGBX discolouration in mir_proving_server) (LP: #1423462)
      . Severe graphical corruption (mostly horizontal streaks/lines) running
        software clients (including Xmir) on android (LP: #1406725)
      . [enhancement] Ubuntu needs a way to set mouse scrolling speed
        (LP: #124440)
      . input-evdev: always send all active contacts.
      . fix backwards frame jumps on the FB context on some devices
        (LP: #1270245)
      . fix race in test (NestedServer suite) (LP: #1514884)
      . fix signal callbacks in buffer stream creation (LP: #1519998 and
        LP: #1519999)
      . allow client to retrieve error messages when stream creation fails
        (LP: #1520005)
      . [regression] Ctrl/Alt modifiers not handled correctly (LP: #1521225)
      . snappy packaging fixes
      . CI test failures due to memory errors during GLibMainLoop shutdown
        (LP: #1520217)
      . fix UBSan build (LP: #1521808)
      . SessionMediator::exchange_buffer can create references to null pointers
        (LP: #1522093)
      . After exiting Mir (by Ctrl+Alt+BkSp) VT switching by Ctrl+Alt+Fn is
        broken (LP: #1521651)
      . notify when cursor images are removed (LP: #1521795)
      . fix arm64, powerpc, ppc64el builds (LP: #1522122)
      . fix locking in ms::SurfaceStack (LP: #1522105)
      . [regression] Mouse wheel events are now unusably over-sensitive and
        backwards (LP: #1522673)
      . Cross compiling Mir from a xenial host fails (dpkg-checkbuilddeps:
        error: Unmet build dependencies: ...) (LP: #1523801)

[ CI Train Bot ]
  * No-change rebuild.

-- Kevin DuBois <kevin.dubois@canonical.com>  Wed, 16 Dec 2015 18:24:40 +0000

Changed in mir (Ubuntu):
status:	New → Fix Released

Kevin DuBois (kdub) on 2015-12-22

Changed in mir:
status:	Fix Committed → Fix Released

Mir

CI segfault in mir-mediumtests-runner-mako in client tests

Bug Description

Related branches

Other bug subscribers

Remote bug watches

Affects		Status	Importance	Assigned to	Milestone
	Mir	Fix Released	High	Alan Griffiths	Mir 0.18.0
	mir (Ubuntu)	Fix Released	Undecided	Unassigned