Re: [Spice-devel] [qxl-win PATCH 2/2] display: fix deadlock when dbg_level = 15

2013-06-24 Thread Yonit Halperin
On 06/23/2013 06:30 AM, Uri Lublin wrote: On 06/21/2013 04:50 PM, Yonit Halperin wrote: DebugPrintV first locks print_sem, and then locks io_sem. async_io, locks io_sem. In ordr to avoid a deadlock, DebugPrintV MUSTN'T be called when io_sem is locked. Also notice, that locking io_sem during

Re: [Spice-devel] [qxl-win PATCH 2/2] display: fix deadlock when dbg_level = 15

2013-06-23 Thread Uri Lublin
On 06/21/2013 04:50 PM, Yonit Halperin wrote: DebugPrintV first locks print_sem, and then locks io_sem. async_io, locks io_sem. In ordr to avoid a deadlock, DebugPrintV MUSTN'T be called when io_sem is locked. Also notice, that locking io_sem during DebugPrintV limits our ability to use the

[Spice-devel] [qxl-win PATCH 2/2] display: fix deadlock when dbg_level = 15

2013-06-21 Thread Yonit Halperin
DebugPrintV first locks print_sem, and then locks io_sem. async_io, locks io_sem. In ordr to avoid a deadlock, DebugPrintV MUSTN'T be called when io_sem is locked. Also notice, that locking io_sem during DebugPrintV limits our ability to use the log_port for debugging concurrency problems related