** Also affects: mir (Ubuntu)
   Importance: Undecided
       Status: New

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to mir in Ubuntu.
https://bugs.launchpad.net/bugs/1464690

Title:
  [regression] titlebar in "canonical" example WM + KeyRepeatDispatcher
  causes deadlock

Status in Mir:
  Fix Committed
Status in mir package in Ubuntu:
  New

Bug description:
  Introducing the KeyRepeatDispatcher means that input events can now
  occur on two threads: the "Input dispatch" thread and the GLibMainLoop
  thread. This leads to lock contention in EventFilterChainDispatcher
  which isn't necessarily fatal - but I think we'd be better using a
  single thread for dispatching input (to impose an order on events).

  What is fatal is that this breaks the (dubious) assumption in
  CanonicalWindowManagerPolicyCopy that we can wait for swap_buffers()
  while handling an input event. This leads to a deadlock if an input
  event is on the GLibMainLoop thread (in the original description this
  is in EventFilterChainDispatcher, but that just prevents it occurring
  in BasicWindowManagerCopy).

  Original text:

  I caused this while testing a branch (lp:~alan-
  griffiths/mir/msh_SystemCompositorWindowManager), but I don't think
  the problem originates there:

  $ sudo bin/mir_demo_server --file /tmp/mir_socket --vt 2 --arw-file
  --window-manager system-compositor

  $ bin/mir_demo_server --host /tmp/mir_socket --window-manager
  canonical

  $ bin/mir_demo_client_multiwin & bin/mir_demo_client_eglplasma &

  After a few Alt+Tab focus switches things lock up in the nested
  server. Attaching with GDB gets the following stack traces:

  The event loop (via) KeyRepeatDispatcher is waiting for a lock in
  EventFilterChainDispatcher::handle().

  #0  __lll_lock_wait () at ../sysdeps/unix/sysv/linux/x86_64/lowlevellock.S:135
  #1  0x00007ffff6960cfd in __GI___pthread_mutex_lock (mutex=0x7ca060) at 
../nptl/pthread_mutex_lock.c:80
  #2  0x00007ffff76e7815 in __gthread_mutex_lock (__mutex=0x7ca060)
      at /usr/include/x86_64-linux-gnu/c++/4.9/bits/gthr-default.h:748
  #3  0x00007ffff76e8738 in std::mutex::lock (this=0x7ca060) at 
/usr/include/c++/4.9/mutex:135
  #4  0x00007ffff76e8b2e in std::lock_guard<std::mutex>::lock_guard 
(this=0x7fffffffdbb0, __m=...)
      at /usr/include/c++/4.9/mutex:377
  #5  0x00007ffff775d1db in mir::input::EventFilterChainDispatcher::handle 
(this=0x7ca050, event=...)
      at 
/home/alan/display_server/mir3/src/server/input/event_filter_chain_dispatcher.cpp:34
  #6  0x00007ffff775d510 in mir::input::EventFilterChainDispatcher::dispatch 
(this=0x7ca050, event=...)
      at 
/home/alan/display_server/mir3/src/server/input/event_filter_chain_dispatcher.cpp:67
  #7  0x00007ffff7796e2e in 
mir::input::KeyRepeatDispatcher::<lambda()>::operator()(void) 
(__closure=0x7fffe000a0a0)
      at 
/home/alan/display_server/mir3/src/server/input/key_repeat_dispatcher.cpp:117
  #8  0x00007ffff7797512 in std::_Function_handler<void(), 
mir::input::KeyRepeatDispatcher::handle_key_input(MirInputDeviceId, const 
MirKeyboardEvent*)::<lambda()> >::_M_invoke(const std::_Any_data &) 
(__functor=...) at /usr/include/c++/4.9/functional:2039
  #9  0x00007ffff76ee54c in std::function<void ()>::operator()() const 
(this=0x7fffe0000e68)
      at /usr/include/c++/4.9/functional:2439
  #10 0x00007ffff773b626 in mir::BasicCallback::operator() (this=0x7fffe0000e60)
      at /home/alan/display_server/mir3/src/server/basic_callback.cpp:28
  #11 0x00007ffff773b111 in mir::LockableCallbackWrapper::operator() 
(this=0x7fffe00036f0)
      at 
/home/alan/display_server/mir3/src/server/lockable_callback_wrapper.cpp:41
  #12 0x00007ffff770face in mir::detail::TimerGSource::dispatch 
(source=0x949320)
      at 
/home/alan/display_server/mir3/src/server/glib_main_loop_sources.cpp:299
  #13 0x00007ffff5443c3d in g_main_context_dispatch () from 
/lib/x86_64-linux-gnu/libglib-2.0.so.0
  #14 0x00007ffff5443f20 in ?? () from /lib/x86_64-linux-gnu/libglib-2.0.so.0
  #15 0x00007ffff5443fcc in g_main_context_iteration () from 
/lib/x86_64-linux-gnu/libglib-2.0.so.0
  #16 0x00007ffff7705f91 in mir::GLibMainLoop::run (this=0x6f6e70)
  ---Type <return> to continue, or q <return> to quit---

  The lock is held by the window manager (which is trying to paint a
  titlebar).

  #0  pthread_cond_wait@@GLIBC_2.3.2 () at 
../sysdeps/unix/sysv/linux/x86_64/pthread_cond_wait.S:185
  #1  0x00007ffff64f7d1c in 
std::condition_variable::wait(std::unique_lock<std::mutex>&) ()
     from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
  #2  0x00000000004b270b in std::condition_variable::wait<(anonymous 
namespace)::swap_buffers(const std::shared_ptr<mir::scene::Surface>&, 
mir::graphics::Buffer*&)::<lambda()> >(std::unique_lock<std::mutex> &, 
(anonymous namespace)::<lambda()>) (
      this=0x7fffee2f21b0, __lock=..., __p=...) at 
/usr/include/c++/4.9/condition_variable:98
  #3  0x00000000004aeaa4 in (anonymous namespace)::swap_buffers 
(surface=std::shared_ptr (count 4, weak 4) 0x7fffc4006750,
      surface_buffer=@0x7fffc4006fc0: 0x7fffc40113f0)
      at 
/home/alan/display_server/mir3/examples/server_example_canonical_window_manager.cpp:277
  #4  0x00000000004aecea in (anonymous namespace)::paint_titlebar 
(titlebar=std::shared_ptr (count 4, weak 4) 0x7fffc4006750,
      titlebar_info=..., intensity=255) at 
/home/alan/display_server/mir3/examples/server_example_canonical_window_manager.cpp:297
  #5  0x00000000004b0f45 in 
mir::examples::CanonicalWindowManagerPolicyCopy::select_active_surface 
(this=0x944cd8,
      surface=std::shared_ptr (count 3, weak 5) 0x7fffc4004fd0)
      at 
/home/alan/display_server/mir3/examples/server_example_canonical_window_manager.cpp:742
  #6  0x00000000004b042a in 
mir::examples::CanonicalWindowManagerPolicyCopy::handle_keyboard_event 
(this=0x944cd8,
      event=0x7fffe0000910) at 
/home/alan/display_server/mir3/examples/server_example_canonical_window_manager.cpp:577
  #7  0x00000000004a0c5e in 
mir::examples::BasicWindowManagerCopy<mir::examples::CanonicalWindowManagerPolicyCopy,
 mir::examples::CanonicalSessionInfoCopy, 
mir::examples::CanonicalSurfaceInfoCopy>::handle_keyboard_event (this=0x944cc0, 
event=0x7fffe0000910)
      at 
/home/alan/display_server/mir3/examples/server_example_basic_window_manager.h:190
  #8  0x00007ffff78beccb in mir::shell::AbstractShell::handle (this=0x944bf0, 
event=...)
      at /home/alan/display_server/mir3/src/server/shell/abstract_shell.cpp:245
  #9  0x00007ffff775d27c in mir::input::EventFilterChainDispatcher::handle 
(this=0x7ca050, event=...)
      at 
/home/alan/display_server/mir3/src/server/input/event_filter_chain_dispatcher.cpp:45
  #10 0x00007ffff775d510 in mir::input::EventFilterChainDispatcher::dispatch 
(this=0x7ca050, event=...)
      at 
/home/alan/display_server/mir3/src/server/input/event_filter_chain_dispatcher.cpp:67
  #11 0x00007ffff7796d22 in mir::input::KeyRepeatDispatcher::dispatch 
(this=0x7ca0d0, event=...)
      at 
/home/alan/display_server/mir3/src/server/input/key_repeat_dispatcher.cpp:63
  ---Type <return> to continue, or q <return> to quit---

  I guess the titlebar paint doesn't complete because the event loop is
  waiting.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mir/+bug/1464690/+subscriptions

-- 
Mailing list: https://launchpad.net/~touch-packages
Post to     : touch-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~touch-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to