[Spice-devel] [PATCH 00/10] split off main channel, start reusing red_worker

Alon Levy alevy at redhat.com
Thu Jan 13 11:48:28 PST 2011


On Thu, Jan 13, 2011 at 08:36:46AM +0100, Hans de Goede wrote:
> Hi,
> 
> On 01/13/2011 06:01 AM, Alon Levy wrote:
> >Patches for 0.7
> 
> I hope / think you mean 0.9 / master, as we don't want to do
> anything potentially de-stabilizing to 0.7 now. So this is all
> for master, right ?

Right, yes - sorry. I'm still living in make believe land where
I get this into 0.8 :/

> 
> Regards,
> 
> Hans
> 
> 
> , towards using red_channel in all channels (except
> >audio which is a little different, so a little harder to justify / do).
> >
> >Patches basically:
> >  * split red channel from reds.c (introduce main_channel.{c,h})
> >  * a bunch of small fixes
> >  * reuse red_channel.h in main_channel.c
> >
> >Please Review,
> >Alon
> >
> >Alon Levy (10):
> >   server: split main_channel from reds
> >   server/reds:
> >     s/reds_push_migrate_data_item/reds_marshall_migrate_data_item/
> >   server/reds: fix possible segfault when accessing vdagent from
> >     reds_update_mouse_mode after vdagent set to NULL
> >   server/reds: don't call close on NULL channel on atexit callback
> >   server/reds: protect reds_update_mouse_mode when main_channel is
> >     disconnected
> >   server/reds: don't remove agent if it's not connected
> >   server/red_channel: go marshaller for outgoing (copied from
> >     red_worker)
> >   server/inputs_channel: use outgoing marshaller in
> >     red_channel/RedChannel
> >   server/red_channel: no need for extra loop
> >   server/main_channel: use red_channel (most code is pipe send/marshall
> >     separation)
> >
> >  server/Makefile.am      |    1 +
> >  server/inputs_channel.c |  125 ++++---
> >  server/main_channel.c   |  849 +++++++++++++++++++++++++++++++++++++++++++
> >  server/main_channel.h   |   78 ++++
> >  server/red_channel.c    |  105 ++----
> >  server/red_channel.h    |   11 +-
> >  server/reds.c           |  926 +++++++++++------------------------------------
> >  server/reds.h           |   22 ++
> >  8 files changed, 1263 insertions(+), 854 deletions(-)
> >  create mode 100644 server/main_channel.c
> >  create mode 100644 server/main_channel.h
> >


More information about the Spice-devel mailing list