tree 109f343f1e5207c3c6a2fc4c339465f4a1e81dc6
parent 418ba9e5d6849ef2e8512d8853628ce4bf37937a
author Corentin Chary <corentin.chary@gmail.com> 1331708327 +0100
committer Anthony Liguori <aliguori@us.ibm.com> 1331760166 -0500

vnc: don't mess up with iohandlers in the vnc thread

The threaded VNC servers messed up with QEMU fd handlers without
any kind of locking, and that can cause some nasty race conditions.

Using qemu_mutex_lock_iothread() won't work because vnc_dpy_cpy(),
which will wait for the current job queue to finish, can be called with
the iothread lock held.

Instead, we now store the data in a temporary buffer, and use a bottom
half to notify the main thread that new data is available.

vnc_[un]lock_ouput() is still needed to access VncState members like
abort, csock or jobs_buffer.

Signed-off-by: Corentin Chary <corentin.chary@gmail.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>