[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/4] spice: fix locking
From: |
Alon Levy |
Subject: |
Re: [Qemu-devel] [PATCH 0/4] spice: fix locking |
Date: |
Fri, 29 Apr 2011 14:20:09 +0300 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Fri, Apr 29, 2011 at 11:38:28AM +0200, Gerd Hoffmann wrote:
> Hi,
>
> This patch series moves a bunch of work spice has to do from spice
> server thread context to iothread context, which in turn allows to
> drop the current locking mess as we don't touch qemu internals from
> spice server thread any more.
>
> A long-standing warning fix from Jes is included too.
>
Tested with winxp, works well with changes between vga and qxl (tested
by setting the debug to 2 in spice-display.c). Just a single comment about
the debug=1 change in the second patch.
Alon
> cheers,
> Gerd
>
> The following changes since commit 430a3c18064fd3c007048d757e8bd0fff45fcc99:
>
> configure: reenable opengl by default (2011-04-26 23:26:49 +0200)
>
> are available in the git repository at:
> git://anongit.freedesktop.org/spice/qemu spice.v34
>
> Gerd Hoffmann (3):
> spice: don't create updates in spice server context.
> spice: don't call displaystate callbacks from spice server context.
> spice: drop obsolete iothread locking
>
> Jes Sorensen (1):
> Make spice dummy functions inline to fix calls not checking return
> values
>
> hw/qxl-render.c | 25 ++++++++++----------
> hw/qxl.c | 27 ++++++++++-----------
> ui/qemu-spice.h | 12 ++++++++-
> ui/spice-display.c | 63
> +++++++++++++++++++++++++++++++++-------------------
> ui/spice-display.h | 24 +++++++++++++++----
> 5 files changed, 94 insertions(+), 57 deletions(-)
>