[Spice-devel] [PATCH v2 1/4] rename red_dispatcher_ functions to red_qxl_

Frediano Ziglio fziglio at redhat.com
Fri Mar 4 09:26:17 UTC 2016


> 
> > 
> > On Thu, 2016-03-03 at 16:28 +0000, Frediano Ziglio wrote:
> > > RedDispatcher is basically implementing QXLInstance.
> > > After some internal discussion in order to avoid rewriting too
> > > many time the patch we came out with RedQXL name.
> > 
> > I guess this message will need to be updated slightly if we decide not to
> > use
> > RedQXL as an actual type name.
> > 
> > Other than that, I think this patch is fine
> 
> What about:
> 
> "RedDispatcher is basically implementing QXLInstance.
> After some internal discussion we decided to use QXLState structure
> and red_qxl_ as function prefix."
> 
> Frediano

Changed comment to:


    rename red_dispatcher_ functions to red_qxl_
    
    RedDispatcher is basically implementing QXLInstance.
    After some internal discussion we decided to use QXLState structure
    and red_qxl_ as function prefix.
    This is the first of 3 patches that rename functions, structure
    and files.


Frediano
> 
> > 
> > Acked-by: Jonathon Jongsma <jjongsma at redhat.com>
> > 
> > > This is the first of 3 patches that rename functions, structure
> > > and files.
> > > 
> > > Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> > > ---
> > >  server/display-channel.c |   2 +-
> > >  server/red-dispatcher.c  | 286
> > >  +++++++++++++++++++++++-----------------------
> > > -
> > >  server/red-dispatcher.h  |  36 +++---
> > >  server/red-worker.c      |  10 +-
> > >  server/reds.c            |  32 +++---
> > >  5 files changed, 183 insertions(+), 183 deletions(-)
> > > 
> > > diff --git a/server/display-channel.c b/server/display-channel.c
> > > index 3b5e169..175ac7e 100644
> > > --- a/server/display-channel.c
> > > +++ b/server/display-channel.c
> > > @@ -2164,7 +2164,7 @@ static void set_gl_draw_async_count(DisplayChannel
> > > *display, int num)
> > >      if (num == 0) {
> > >          struct AsyncCommand *async = qxl->st->gl_draw_async;
> > >          qxl->st->gl_draw_async = NULL;
> > > -        red_dispatcher_async_complete(qxl->st->dispatcher, async);
> > > +        red_qxl_async_complete(qxl->st->dispatcher, async);
> > >      }
> > >  }
> > >  
> > > diff --git a/server/red-dispatcher.c b/server/red-dispatcher.c
> > > index c2ca6b6..cfc113d 100644
> > > --- a/server/red-dispatcher.c
> > > +++ b/server/red-dispatcher.c
> > > @@ -58,7 +58,7 @@ struct RedDispatcher {
> > >      unsigned int max_monitors;
> > >  };
> > >  
> > > -static int red_dispatcher_check_qxl_version(RedDispatcher *rd, int
> > > major,
> > > int
> > > minor)
> > > +static int red_qxl_check_qxl_version(RedDispatcher *rd, int major, int
> > > minor)
> > >  {
> > >      int qxl_major = rd->qxl->st->qif->base.major_version;
> > >      int qxl_minor = rd->qxl->st->qif->base.minor_version;
> > > @@ -67,10 +67,10 @@ static int
> > > red_dispatcher_check_qxl_version(RedDispatcher
> > > *rd, int major, int mi
> > >              ((qxl_major == major) && (qxl_minor >= minor)));
> > >  }
> > >  
> > > -static void red_dispatcher_set_display_peer(RedChannel *channel,
> > > RedClient
> > > *client,
> > > -                                            RedsStream *stream, int
> > > migration,
> > > -                                            int num_common_caps,
> > > uint32_t
> > > *common_caps, int num_caps,
> > > -                                            uint32_t *caps)
> > > +static void red_qxl_set_display_peer(RedChannel *channel, RedClient
> > > *client,
> > > +                                     RedsStream *stream, int migration,
> > > +                                     int num_common_caps, uint32_t
> > > *common_caps, int num_caps,
> > > +                                     uint32_t *caps)
> > >  {
> > >      RedWorkerMessageDisplayConnect payload = {0,};
> > >      RedDispatcher *dispatcher;
> > > @@ -93,7 +93,7 @@ static void red_dispatcher_set_display_peer(RedChannel
> > > *channel, RedClient *clie
> > >                              &payload);
> > >  }
> > >  
> > > -static void red_dispatcher_disconnect_display_peer(RedChannelClient
> > > *rcc)
> > > +static void red_qxl_disconnect_display_peer(RedChannelClient *rcc)
> > >  {
> > >      RedWorkerMessageDisplayDisconnect payload;
> > >      RedDispatcher *dispatcher;
> > > @@ -114,7 +114,7 @@ static void
> > > red_dispatcher_disconnect_display_peer(RedChannelClient *rcc)
> > >                              &payload);
> > >  }
> > >  
> > > -static void red_dispatcher_display_migrate(RedChannelClient *rcc)
> > > +static void red_qxl_display_migrate(RedChannelClient *rcc)
> > >  {
> > >      RedWorkerMessageDisplayMigrate payload;
> > >      RedDispatcher *dispatcher;
> > > @@ -129,10 +129,10 @@ static void
> > > red_dispatcher_display_migrate(RedChannelClient *rcc)
> > >                              &payload);
> > >  }
> > >  
> > > -static void red_dispatcher_set_cursor_peer(RedChannel *channel,
> > > RedClient
> > > *client, RedsStream *stream,
> > > -                                           int migration, int
> > > num_common_caps,
> > > -                                           uint32_t *common_caps, int
> > > num_caps,
> > > -                                           uint32_t *caps)
> > > +static void red_qxl_set_cursor_peer(RedChannel *channel, RedClient
> > > *client,
> > > RedsStream *stream,
> > > +                                    int migration, int num_common_caps,
> > > +                                    uint32_t *common_caps, int num_caps,
> > > +                                    uint32_t *caps)
> > >  {
> > >      RedWorkerMessageCursorConnect payload = {0,};
> > >      RedDispatcher *dispatcher = (RedDispatcher *)channel->data;
> > > @@ -153,7 +153,7 @@ static void red_dispatcher_set_cursor_peer(RedChannel
> > > *channel, RedClient *clien
> > >                              &payload);
> > >  }
> > >  
> > > -static void red_dispatcher_disconnect_cursor_peer(RedChannelClient *rcc)
> > > +static void red_qxl_disconnect_cursor_peer(RedChannelClient *rcc)
> > >  {
> > >      RedWorkerMessageCursorDisconnect payload;
> > >      RedDispatcher *dispatcher;
> > > @@ -171,7 +171,7 @@ static void
> > > red_dispatcher_disconnect_cursor_peer(RedChannelClient *rcc)
> > >                              &payload);
> > >  }
> > >  
> > > -static void red_dispatcher_cursor_migrate(RedChannelClient *rcc)
> > > +static void red_qxl_cursor_migrate(RedChannelClient *rcc)
> > >  {
> > >      RedWorkerMessageCursorMigrate payload;
> > >      RedDispatcher *dispatcher;
> > > @@ -187,9 +187,9 @@ static void
> > > red_dispatcher_cursor_migrate(RedChannelClient
> > > *rcc)
> > >                              &payload);
> > >  }
> > >  
> > > -static void red_dispatcher_update_area(RedDispatcher *dispatcher,
> > > uint32_t
> > > surface_id,
> > > -                                   QXLRect *qxl_area, QXLRect
> > > *qxl_dirty_rects,
> > > -                                   uint32_t num_dirty_rects, uint32_t
> > > clear_dirty_region)
> > > +static void red_qxl_update_area(RedDispatcher *dispatcher, uint32_t
> > > surface_id,
> > > +                                QXLRect *qxl_area, QXLRect
> > > *qxl_dirty_rects,
> > > +                                uint32_t num_dirty_rects, uint32_t
> > > clear_dirty_region)
> > >  {
> > >      RedWorkerMessageUpdate payload = {0,};
> > >  
> > > @@ -203,15 +203,15 @@ static void
> > > red_dispatcher_update_area(RedDispatcher
> > > *dispatcher, uint32_t surfa
> > >                              &payload);
> > >  }
> > >  
> > > -gboolean red_dispatcher_use_client_monitors_config(RedDispatcher
> > > *dispatcher)
> > > +gboolean red_qxl_use_client_monitors_config(RedDispatcher *dispatcher)
> > >  {
> > > -    return (red_dispatcher_check_qxl_version(dispatcher, 3, 3) &&
> > > +    return (red_qxl_check_qxl_version(dispatcher, 3, 3) &&
> > >          dispatcher->qxl->st->qif->client_monitors_config &&
> > >          dispatcher->qxl->st->qif->client_monitors_config(dispatcher->qxl,
> > > NULL));
> > >  }
> > >  
> > > -gboolean red_dispatcher_client_monitors_config(RedDispatcher
> > > *dispatcher,
> > > -                                               VDAgentMonitorsConfig
> > > *monitors_config)
> > > +gboolean red_qxl_client_monitors_config(RedDispatcher *dispatcher,
> > > +                                        VDAgentMonitorsConfig
> > > *monitors_config)
> > >  {
> > >      return (dispatcher->qxl->st->qif->client_monitors_config &&
> > >          dispatcher->qxl->st->qif->client_monitors_config(dispatcher->qxl,
> > > @@ -231,11 +231,11 @@ static AsyncCommand
> > > *async_command_alloc(RedDispatcher
> > > *dispatcher,
> > >      return async_command;
> > >  }
> > >  
> > > -static void red_dispatcher_update_area_async(RedDispatcher *dispatcher,
> > > -                                         uint32_t surface_id,
> > > -                                         QXLRect *qxl_area,
> > > -                                         uint32_t clear_dirty_region,
> > > -                                         uint64_t cookie)
> > > +static void red_qxl_update_area_async(RedDispatcher *dispatcher,
> > > +                                      uint32_t surface_id,
> > > +                                      QXLRect *qxl_area,
> > > +                                      uint32_t clear_dirty_region,
> > > +                                      uint64_t cookie)
> > >  {
> > >      RedWorkerMessage message = RED_WORKER_MESSAGE_UPDATE_ASYNC;
> > >      RedWorkerMessageUpdateAsync payload;
> > > @@ -253,11 +253,11 @@ static void qxl_worker_update_area(QXLWorker
> > > *qxl_worker, uint32_t surface_id,
> > >                                     QXLRect *qxl_area, QXLRect
> > > *qxl_dirty_rects,
> > >                                     uint32_t num_dirty_rects, uint32_t
> > > clear_dirty_region)
> > >  {
> > > -    red_dispatcher_update_area((RedDispatcher*)qxl_worker, surface_id,
> > > qxl_area,
> > > -                               qxl_dirty_rects, num_dirty_rects,
> > > clear_dirty_region);
> > > +    red_qxl_update_area((RedDispatcher*)qxl_worker, surface_id,
> > > qxl_area,
> > > +                        qxl_dirty_rects, num_dirty_rects,
> > > clear_dirty_region);
> > >  }
> > >  
> > > -static void red_dispatcher_add_memslot(RedDispatcher *dispatcher,
> > > QXLDevMemSlot *mem_slot)
> > > +static void red_qxl_add_memslot(RedDispatcher *dispatcher, QXLDevMemSlot
> > > *mem_slot)
> > >  {
> > >      RedWorkerMessageAddMemslot payload;
> > >  
> > > @@ -269,10 +269,10 @@ static void
> > > red_dispatcher_add_memslot(RedDispatcher
> > > *dispatcher, QXLDevMemSlot
> > >  
> > >  static void qxl_worker_add_memslot(QXLWorker *qxl_worker, QXLDevMemSlot
> > > *mem_slot)
> > >  {
> > > -    red_dispatcher_add_memslot((RedDispatcher*)qxl_worker, mem_slot);
> > > +    red_qxl_add_memslot((RedDispatcher*)qxl_worker, mem_slot);
> > >  }
> > >  
> > > -static void red_dispatcher_add_memslot_async(RedDispatcher *dispatcher,
> > > QXLDevMemSlot *mem_slot, uint64_t cookie)
> > > +static void red_qxl_add_memslot_async(RedDispatcher *dispatcher,
> > > QXLDevMemSlot *mem_slot, uint64_t cookie)
> > >  {
> > >      RedWorkerMessageAddMemslotAsync payload;
> > >      RedWorkerMessage message = RED_WORKER_MESSAGE_ADD_MEMSLOT_ASYNC;
> > > @@ -282,7 +282,7 @@ static void
> > > red_dispatcher_add_memslot_async(RedDispatcher
> > > *dispatcher, QXLDevMe
> > >      dispatcher_send_message(&dispatcher->dispatcher, message, &payload);
> > >  }
> > >  
> > > -static void red_dispatcher_del_memslot(RedDispatcher *dispatcher,
> > > uint32_t
> > > slot_group_id, uint32_t slot_id)
> > > +static void red_qxl_del_memslot(RedDispatcher *dispatcher, uint32_t
> > > slot_group_id, uint32_t slot_id)
> > >  {
> > >      RedWorkerMessageDelMemslot payload;
> > >      RedWorkerMessage message = RED_WORKER_MESSAGE_DEL_MEMSLOT;
> > > @@ -294,10 +294,10 @@ static void
> > > red_dispatcher_del_memslot(RedDispatcher
> > > *dispatcher, uint32_t slot_
> > >  
> > >  static void qxl_worker_del_memslot(QXLWorker *qxl_worker, uint32_t
> > > slot_group_id, uint32_t slot_id)
> > >  {
> > > -    red_dispatcher_del_memslot((RedDispatcher*)qxl_worker,
> > > slot_group_id,
> > > slot_id);
> > > +    red_qxl_del_memslot((RedDispatcher*)qxl_worker, slot_group_id,
> > > slot_id);
> > >  }
> > >  
> > > -static void red_dispatcher_destroy_surfaces(RedDispatcher *dispatcher)
> > > +static void red_qxl_destroy_surfaces(RedDispatcher *dispatcher)
> > >  {
> > >      RedWorkerMessageDestroySurfaces payload;
> > >  
> > > @@ -308,10 +308,10 @@ static void
> > > red_dispatcher_destroy_surfaces(RedDispatcher *dispatcher)
> > >  
> > >  static void qxl_worker_destroy_surfaces(QXLWorker *qxl_worker)
> > >  {
> > > -    red_dispatcher_destroy_surfaces((RedDispatcher*)qxl_worker);
> > > +    red_qxl_destroy_surfaces((RedDispatcher*)qxl_worker);
> > >  }
> > >  
> > > -static void red_dispatcher_destroy_surfaces_async(RedDispatcher
> > > *dispatcher,
> > > uint64_t cookie)
> > > +static void red_qxl_destroy_surfaces_async(RedDispatcher *dispatcher,
> > > uint64_t cookie)
> > >  {
> > >      RedWorkerMessageDestroySurfacesAsync payload;
> > >      RedWorkerMessage message =
> > >      RED_WORKER_MESSAGE_DESTROY_SURFACES_ASYNC;
> > > @@ -320,7 +320,7 @@ static void
> > > red_dispatcher_destroy_surfaces_async(RedDispatcher *dispatcher, uin
> > >      dispatcher_send_message(&dispatcher->dispatcher, message, &payload);
> > >  }
> > >  
> > > -static void
> > > red_dispatcher_destroy_primary_surface_complete(RedDispatcher
> > > *dispatcher)
> > > +static void red_qxl_destroy_primary_surface_complete(RedDispatcher
> > > *dispatcher)
> > >  {
> > >      dispatcher->x_res = 0;
> > >      dispatcher->y_res = 0;
> > > @@ -331,20 +331,20 @@ static void
> > > red_dispatcher_destroy_primary_surface_complete(RedDispatcher *dispa
> > >  }
> > >  
> > >  static void
> > > -red_dispatcher_destroy_primary_surface_sync(RedDispatcher *dispatcher,
> > > -                                            uint32_t surface_id)
> > > +red_qxl_destroy_primary_surface_sync(RedDispatcher *dispatcher,
> > > +                                     uint32_t surface_id)
> > >  {
> > >      RedWorkerMessageDestroyPrimarySurface payload;
> > >      payload.surface_id = surface_id;
> > >      dispatcher_send_message(&dispatcher->dispatcher,
> > >                              RED_WORKER_MESSAGE_DESTROY_PRIMARY_SURFACE,
> > >                              &payload);
> > > -    red_dispatcher_destroy_primary_surface_complete(dispatcher);
> > > +    red_qxl_destroy_primary_surface_complete(dispatcher);
> > >  }
> > >  
> > >  static void
> > > -red_dispatcher_destroy_primary_surface_async(RedDispatcher *dispatcher,
> > > -                                             uint32_t surface_id,
> > > uint64_t
> > > cookie)
> > > +red_qxl_destroy_primary_surface_async(RedDispatcher *dispatcher,
> > > +                                      uint32_t surface_id, uint64_t
> > > cookie)
> > >  {
> > >      RedWorkerMessageDestroyPrimarySurfaceAsync payload;
> > >      RedWorkerMessage message =
> > > RED_WORKER_MESSAGE_DESTROY_PRIMARY_SURFACE_ASYNC;
> > > @@ -355,22 +355,22 @@
> > > red_dispatcher_destroy_primary_surface_async(RedDispatcher *dispatcher,
> > >  }
> > >  
> > >  static void
> > > -red_dispatcher_destroy_primary_surface(RedDispatcher *dispatcher,
> > > -                                       uint32_t surface_id, int async,
> > > uint64_t cookie)
> > > +red_qxl_destroy_primary_surface(RedDispatcher *dispatcher,
> > > +                                uint32_t surface_id, int async, uint64_t
> > > cookie)
> > >  {
> > >      if (async) {
> > > -        red_dispatcher_destroy_primary_surface_async(dispatcher,
> > > surface_id,
> > > cookie);
> > > +        red_qxl_destroy_primary_surface_async(dispatcher, surface_id,
> > > cookie);
> > >      } else {
> > > -        red_dispatcher_destroy_primary_surface_sync(dispatcher,
> > > surface_id);
> > > +        red_qxl_destroy_primary_surface_sync(dispatcher, surface_id);
> > >      }
> > >  }
> > >  
> > >  static void qxl_worker_destroy_primary_surface(QXLWorker *qxl_worker,
> > > uint32_t surface_id)
> > >  {
> > > -    red_dispatcher_destroy_primary_surface((RedDispatcher*)qxl_worker,
> > > surface_id, 0, 0);
> > > +    red_qxl_destroy_primary_surface((RedDispatcher*)qxl_worker,
> > > surface_id,
> > > 0, 0);
> > >  }
> > >  
> > > -static void red_dispatcher_create_primary_surface_complete(RedDispatcher
> > > *dispatcher)
> > > +static void red_qxl_create_primary_surface_complete(RedDispatcher
> > > *dispatcher)
> > >  {
> > >      QXLDevSurfaceCreate *surface = &dispatcher->surface_create;
> > >  
> > > @@ -384,8 +384,8 @@ static void
> > > red_dispatcher_create_primary_surface_complete(RedDispatcher *dispat
> > >  }
> > >  
> > >  static void
> > > -red_dispatcher_create_primary_surface_async(RedDispatcher *dispatcher,
> > > uint32_t surface_id,
> > > -                                            QXLDevSurfaceCreate
> > > *surface,
> > > uint64_t cookie)
> > > +red_qxl_create_primary_surface_async(RedDispatcher *dispatcher, uint32_t
> > > surface_id,
> > > +                                     QXLDevSurfaceCreate *surface,
> > > uint64_t
> > > cookie)
> > >  {
> > >      RedWorkerMessageCreatePrimarySurfaceAsync payload;
> > >      RedWorkerMessage message =
> > > RED_WORKER_MESSAGE_CREATE_PRIMARY_SURFACE_ASYNC;
> > > @@ -398,8 +398,8 @@
> > > red_dispatcher_create_primary_surface_async(RedDispatcher
> > > *dispatcher, uint32_t
> > >  }
> > >  
> > >  static void
> > > -red_dispatcher_create_primary_surface_sync(RedDispatcher *dispatcher,
> > > uint32_t surface_id,
> > > -                                           QXLDevSurfaceCreate *surface)
> > > +red_qxl_create_primary_surface_sync(RedDispatcher *dispatcher, uint32_t
> > > surface_id,
> > > +                                    QXLDevSurfaceCreate *surface)
> > >  {
> > >      RedWorkerMessageCreatePrimarySurface payload = {0,};
> > >  
> > > @@ -409,27 +409,27 @@
> > > red_dispatcher_create_primary_surface_sync(RedDispatcher
> > > *dispatcher, uint32_t s
> > >      dispatcher_send_message(&dispatcher->dispatcher,
> > >                              RED_WORKER_MESSAGE_CREATE_PRIMARY_SURFACE,
> > >                              &payload);
> > > -    red_dispatcher_create_primary_surface_complete(dispatcher);
> > > +    red_qxl_create_primary_surface_complete(dispatcher);
> > >  }
> > >  
> > >  static void
> > > -red_dispatcher_create_primary_surface(RedDispatcher *dispatcher,
> > > uint32_t
> > > surface_id,
> > > -                                      QXLDevSurfaceCreate *surface, int
> > > async, uint64_t cookie)
> > > +red_qxl_create_primary_surface(RedDispatcher *dispatcher, uint32_t
> > > surface_id,
> > > +                               QXLDevSurfaceCreate *surface, int async,
> > > uint64_t cookie)
> > >  {
> > >      if (async) {
> > > -        red_dispatcher_create_primary_surface_async(dispatcher,
> > > surface_id,
> > > surface, cookie);
> > > +        red_qxl_create_primary_surface_async(dispatcher, surface_id,
> > > surface,
> > > cookie);
> > >      } else {
> > > -        red_dispatcher_create_primary_surface_sync(dispatcher,
> > > surface_id,
> > > surface);
> > > +        red_qxl_create_primary_surface_sync(dispatcher, surface_id,
> > > surface);
> > >      }
> > >  }
> > >  
> > >  static void qxl_worker_create_primary_surface(QXLWorker *qxl_worker,
> > >  uint32_t
> > > surface_id,
> > >                                        QXLDevSurfaceCreate *surface)
> > >  {
> > > -    red_dispatcher_create_primary_surface((RedDispatcher*)qxl_worker,
> > > surface_id, surface, 0, 0);
> > > +    red_qxl_create_primary_surface((RedDispatcher*)qxl_worker,
> > > surface_id,
> > > surface, 0, 0);
> > >  }
> > >  
> > > -static void red_dispatcher_reset_image_cache(RedDispatcher *dispatcher)
> > > +static void red_qxl_reset_image_cache(RedDispatcher *dispatcher)
> > >  {
> > >      RedWorkerMessageResetImageCache payload;
> > >  
> > > @@ -440,10 +440,10 @@ static void
> > > red_dispatcher_reset_image_cache(RedDispatcher *dispatcher)
> > >  
> > >  static void qxl_worker_reset_image_cache(QXLWorker *qxl_worker)
> > >  {
> > > -    red_dispatcher_reset_image_cache((RedDispatcher*)qxl_worker);
> > > +    red_qxl_reset_image_cache((RedDispatcher*)qxl_worker);
> > >  }
> > >  
> > > -static void red_dispatcher_reset_cursor(RedDispatcher *dispatcher)
> > > +static void red_qxl_reset_cursor(RedDispatcher *dispatcher)
> > >  {
> > >      RedWorkerMessageResetCursor payload;
> > >  
> > > @@ -454,11 +454,11 @@ static void
> > > red_dispatcher_reset_cursor(RedDispatcher
> > > *dispatcher)
> > >  
> > >  static void qxl_worker_reset_cursor(QXLWorker *qxl_worker)
> > >  {
> > > -    red_dispatcher_reset_cursor((RedDispatcher*)qxl_worker);
> > > +    red_qxl_reset_cursor((RedDispatcher*)qxl_worker);
> > >  }
> > >  
> > > -static void red_dispatcher_destroy_surface_wait_sync(RedDispatcher
> > > *dispatcher,
> > > -                                                     uint32_t
> > > surface_id)
> > > +static void red_qxl_destroy_surface_wait_sync(RedDispatcher *dispatcher,
> > > +                                              uint32_t surface_id)
> > >  {
> > >      RedWorkerMessageDestroySurfaceWait payload;
> > >  
> > > @@ -468,9 +468,9 @@ static void
> > > red_dispatcher_destroy_surface_wait_sync(RedDispatcher *dispatcher,
> > >                              &payload);
> > >  }
> > >  
> > > -static void red_dispatcher_destroy_surface_wait_async(RedDispatcher
> > > *dispatcher,
> > > -                                                      uint32_t
> > > surface_id,
> > > -                                                      uint64_t cookie)
> > > +static void red_qxl_destroy_surface_wait_async(RedDispatcher
> > > *dispatcher,
> > > +                                               uint32_t surface_id,
> > > +                                               uint64_t cookie)
> > >  {
> > >      RedWorkerMessageDestroySurfaceWaitAsync payload;
> > >      RedWorkerMessage message =
> > >      RED_WORKER_MESSAGE_DESTROY_SURFACE_WAIT_ASYNC;
> > > @@ -480,23 +480,23 @@ static void
> > > red_dispatcher_destroy_surface_wait_async(RedDispatcher *dispatcher,
> > >      dispatcher_send_message(&dispatcher->dispatcher, message, &payload);
> > >  }
> > >  
> > > -static void red_dispatcher_destroy_surface_wait(RedDispatcher
> > > *dispatcher,
> > > -                                                uint32_t surface_id,
> > > -                                                int async, uint64_t
> > > cookie)
> > > +static void red_qxl_destroy_surface_wait(RedDispatcher *dispatcher,
> > > +                                         uint32_t surface_id,
> > > +                                         int async, uint64_t cookie)
> > >  {
> > >      if (async) {
> > > -        red_dispatcher_destroy_surface_wait_async(dispatcher,
> > > surface_id,
> > > cookie);
> > > +        red_qxl_destroy_surface_wait_async(dispatcher, surface_id,
> > > cookie);
> > >      } else {
> > > -        red_dispatcher_destroy_surface_wait_sync(dispatcher,
> > > surface_id);
> > > +        red_qxl_destroy_surface_wait_sync(dispatcher, surface_id);
> > >      }
> > >  }
> > >  
> > >  static void qxl_worker_destroy_surface_wait(QXLWorker *qxl_worker,
> > >  uint32_t
> > > surface_id)
> > >  {
> > > -    red_dispatcher_destroy_surface_wait((RedDispatcher*)qxl_worker,
> > > surface_id, 0, 0);
> > > +    red_qxl_destroy_surface_wait((RedDispatcher*)qxl_worker, surface_id,
> > > 0,
> > > 0);
> > >  }
> > >  
> > > -static void red_dispatcher_reset_memslots(RedDispatcher *dispatcher)
> > > +static void red_qxl_reset_memslots(RedDispatcher *dispatcher)
> > >  {
> > >      RedWorkerMessageResetMemslots payload;
> > >  
> > > @@ -507,10 +507,10 @@ static void
> > > red_dispatcher_reset_memslots(RedDispatcher
> > > *dispatcher)
> > >  
> > >  static void qxl_worker_reset_memslots(QXLWorker *qxl_worker)
> > >  {
> > > -    red_dispatcher_reset_memslots((RedDispatcher*)qxl_worker);
> > > +    red_qxl_reset_memslots((RedDispatcher*)qxl_worker);
> > >  }
> > >  
> > > -static bool red_dispatcher_set_pending(RedDispatcher *dispatcher, int
> > > pending)
> > > +static bool red_qxl_set_pending(RedDispatcher *dispatcher, int pending)
> > >  {
> > >      // this is not atomic but is not an issue
> > >      if (test_bit(pending, dispatcher->pending)) {
> > > @@ -521,11 +521,11 @@ static bool
> > > red_dispatcher_set_pending(RedDispatcher
> > > *dispatcher, int pending)
> > >      return FALSE;
> > >  }
> > >  
> > > -static void red_dispatcher_wakeup(RedDispatcher *dispatcher)
> > > +static void red_qxl_wakeup(RedDispatcher *dispatcher)
> > >  {
> > >      RedWorkerMessageWakeup payload;
> > >  
> > > -    if (red_dispatcher_set_pending(dispatcher,
> > > RED_DISPATCHER_PENDING_WAKEUP))
> > > +    if (red_qxl_set_pending(dispatcher, RED_DISPATCHER_PENDING_WAKEUP))
> > >          return;
> > >  
> > >      dispatcher_send_message(&dispatcher->dispatcher,
> > > @@ -535,14 +535,14 @@ static void red_dispatcher_wakeup(RedDispatcher
> > > *dispatcher)
> > >  
> > >  static void qxl_worker_wakeup(QXLWorker *qxl_worker)
> > >  {
> > > -    red_dispatcher_wakeup((RedDispatcher*)qxl_worker);
> > > +    red_qxl_wakeup((RedDispatcher*)qxl_worker);
> > >  }
> > >  
> > > -static void red_dispatcher_oom(RedDispatcher *dispatcher)
> > > +static void red_qxl_oom(RedDispatcher *dispatcher)
> > >  {
> > >      RedWorkerMessageOom payload;
> > >  
> > > -    if (red_dispatcher_set_pending(dispatcher,
> > > RED_DISPATCHER_PENDING_OOM))
> > > +    if (red_qxl_set_pending(dispatcher, RED_DISPATCHER_PENDING_OOM))
> > >          return;
> > >  
> > >      dispatcher_send_message(&dispatcher->dispatcher,
> > > @@ -552,10 +552,10 @@ static void red_dispatcher_oom(RedDispatcher
> > > *dispatcher)
> > >  
> > >  static void qxl_worker_oom(QXLWorker *qxl_worker)
> > >  {
> > > -    red_dispatcher_oom((RedDispatcher*)qxl_worker);
> > > +    red_qxl_oom((RedDispatcher*)qxl_worker);
> > >  }
> > >  
> > > -void red_dispatcher_start(RedDispatcher *dispatcher)
> > > +void red_qxl_start(RedDispatcher *dispatcher)
> > >  {
> > >      RedWorkerMessageStart payload;
> > >  
> > > @@ -566,10 +566,10 @@ void red_dispatcher_start(RedDispatcher
> > > *dispatcher)
> > >  
> > >  static void qxl_worker_start(QXLWorker *qxl_worker)
> > >  {
> > > -    red_dispatcher_start((RedDispatcher*)qxl_worker);
> > > +    red_qxl_start((RedDispatcher*)qxl_worker);
> > >  }
> > >  
> > > -static void red_dispatcher_flush_surfaces_async(RedDispatcher
> > > *dispatcher,
> > > uint64_t cookie)
> > > +static void red_qxl_flush_surfaces_async(RedDispatcher *dispatcher,
> > > uint64_t
> > > cookie)
> > >  {
> > >      RedWorkerMessageFlushSurfacesAsync payload;
> > >      RedWorkerMessage message = RED_WORKER_MESSAGE_FLUSH_SURFACES_ASYNC;
> > > @@ -578,10 +578,10 @@ static void
> > > red_dispatcher_flush_surfaces_async(RedDispatcher *dispatcher, uint6
> > >      dispatcher_send_message(&dispatcher->dispatcher, message, &payload);
> > >  }
> > >  
> > > -static void red_dispatcher_monitors_config_async(RedDispatcher
> > > *dispatcher,
> > > -                                                 QXLPHYSICAL
> > > monitors_config,
> > > -                                                 int group_id,
> > > -                                                 uint64_t cookie)
> > > +static void red_qxl_monitors_config_async(RedDispatcher *dispatcher,
> > > +                                          QXLPHYSICAL monitors_config,
> > > +                                          int group_id,
> > > +                                          uint64_t cookie)
> > >  {
> > >      RedWorkerMessageMonitorsConfigAsync payload;
> > >      RedWorkerMessage message = RED_WORKER_MESSAGE_MONITORS_CONFIG_ASYNC;
> > > @@ -594,7 +594,7 @@ static void
> > > red_dispatcher_monitors_config_async(RedDispatcher *dispatcher,
> > >      dispatcher_send_message(&dispatcher->dispatcher, message, &payload);
> > >  }
> > >  
> > > -static void red_dispatcher_driver_unload(RedDispatcher *dispatcher)
> > > +static void red_qxl_driver_unload(RedDispatcher *dispatcher)
> > >  {
> > >      RedWorkerMessageDriverUnload payload;
> > >  
> > > @@ -603,7 +603,7 @@ static void
> > > red_dispatcher_driver_unload(RedDispatcher
> > > *dispatcher)
> > >                              &payload);
> > >  }
> > >  
> > > -void red_dispatcher_stop(RedDispatcher *dispatcher)
> > > +void red_qxl_stop(RedDispatcher *dispatcher)
> > >  {
> > >      RedWorkerMessageStop payload;
> > >  
> > > @@ -614,12 +614,12 @@ void red_dispatcher_stop(RedDispatcher *dispatcher)
> > >  
> > >  static void qxl_worker_stop(QXLWorker *qxl_worker)
> > >  {
> > > -    red_dispatcher_stop((RedDispatcher*)qxl_worker);
> > > +    red_qxl_stop((RedDispatcher*)qxl_worker);
> > >  }
> > >  
> > > -static void red_dispatcher_loadvm_commands(RedDispatcher *dispatcher,
> > > -                                           struct QXLCommandExt *ext,
> > > -                                           uint32_t count)
> > > +static void red_qxl_loadvm_commands(RedDispatcher *dispatcher,
> > > +                                    struct QXLCommandExt *ext,
> > > +                                    uint32_t count)
> > >  {
> > >      RedWorkerMessageLoadvmCommands payload;
> > >  
> > > @@ -635,26 +635,26 @@ static void qxl_worker_loadvm_commands(QXLWorker
> > > *qxl_worker,
> > >                                         struct QXLCommandExt *ext,
> > >                                         uint32_t count)
> > >  {
> > > -    red_dispatcher_loadvm_commands((RedDispatcher*)qxl_worker, ext,
> > > count);
> > > +    red_qxl_loadvm_commands((RedDispatcher*)qxl_worker, ext, count);
> > >  }
> > >  
> > > -void red_dispatcher_set_mm_time(RedDispatcher *dispatcher, uint32_t
> > > mm_time)
> > > +void red_qxl_set_mm_time(RedDispatcher *dispatcher, uint32_t mm_time)
> > >  {
> > >      dispatcher->qxl->st->qif->set_mm_time(dispatcher->qxl, mm_time);
> > >  }
> > >  
> > > -void red_dispatcher_attach_worker(RedDispatcher *dispatcher)
> > > +void red_qxl_attach_worker(RedDispatcher *dispatcher)
> > >  {
> > >      QXLInstance *qxl = dispatcher->qxl;
> > >      qxl->st->qif->attache_worker(qxl, &dispatcher->base);
> > >  }
> > >  
> > > -void red_dispatcher_set_compression_level(RedDispatcher *dispatcher, int
> > > level)
> > > +void red_qxl_set_compression_level(RedDispatcher *dispatcher, int level)
> > >  {
> > >      dispatcher->qxl->st->qif->set_compression_level(dispatcher->qxl,
> > >      level);
> > >  }
> > >  
> > > -uint32_t red_dispatcher_qxl_ram_size(RedDispatcher *dispatcher)
> > > +uint32_t red_qxl_get_ram_size(RedDispatcher *dispatcher)
> > >  {
> > >      QXLDevInitInfo qxl_info;
> > >      dispatcher->qxl->st->qif->get_init_info(dispatcher->qxl, &qxl_info);
> > > @@ -664,25 +664,25 @@ uint32_t red_dispatcher_qxl_ram_size(RedDispatcher
> > > *dispatcher)
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_wakeup(QXLInstance *instance)
> > >  {
> > > -    red_dispatcher_wakeup(instance->st->dispatcher);
> > > +    red_qxl_wakeup(instance->st->dispatcher);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_oom(QXLInstance *instance)
> > >  {
> > > -    red_dispatcher_oom(instance->st->dispatcher);
> > > +    red_qxl_oom(instance->st->dispatcher);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_start(QXLInstance *instance)
> > >  {
> > > -    red_dispatcher_start(instance->st->dispatcher);
> > > +    red_qxl_start(instance->st->dispatcher);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_stop(QXLInstance *instance)
> > >  {
> > > -    red_dispatcher_stop(instance->st->dispatcher);
> > > +    red_qxl_stop(instance->st->dispatcher);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > > @@ -690,121 +690,121 @@ void spice_qxl_update_area(QXLInstance *instance,
> > > uint32_t surface_id,
> > >                      struct QXLRect *area, struct QXLRect *dirty_rects,
> > >                      uint32_t num_dirty_rects, uint32_t
> > >                      clear_dirty_region)
> > >  {
> > > -    red_dispatcher_update_area(instance->st->dispatcher, surface_id,
> > > area,
> > > dirty_rects,
> > > -                               num_dirty_rects, clear_dirty_region);
> > > +    red_qxl_update_area(instance->st->dispatcher, surface_id, area,
> > > dirty_rects,
> > > +                        num_dirty_rects, clear_dirty_region);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_add_memslot(QXLInstance *instance, QXLDevMemSlot *slot)
> > >  {
> > > -    red_dispatcher_add_memslot(instance->st->dispatcher, slot);
> > > +    red_qxl_add_memslot(instance->st->dispatcher, slot);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_del_memslot(QXLInstance *instance, uint32_t
> > >  slot_group_id,
> > > uint32_t slot_id)
> > >  {
> > > -    red_dispatcher_del_memslot(instance->st->dispatcher, slot_group_id,
> > > slot_id);
> > > +    red_qxl_del_memslot(instance->st->dispatcher, slot_group_id,
> > > slot_id);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_reset_memslots(QXLInstance *instance)
> > >  {
> > > -    red_dispatcher_reset_memslots(instance->st->dispatcher);
> > > +    red_qxl_reset_memslots(instance->st->dispatcher);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_destroy_surfaces(QXLInstance *instance)
> > >  {
> > > -    red_dispatcher_destroy_surfaces(instance->st->dispatcher);
> > > +    red_qxl_destroy_surfaces(instance->st->dispatcher);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_destroy_primary_surface(QXLInstance *instance, uint32_t
> > > surface_id)
> > >  {
> > > -    red_dispatcher_destroy_primary_surface(instance->st->dispatcher,
> > > surface_id, 0, 0);
> > > +    red_qxl_destroy_primary_surface(instance->st->dispatcher,
> > > surface_id,
> > > 0,
> > > 0);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_create_primary_surface(QXLInstance *instance, uint32_t
> > > surface_id,
> > >                                  QXLDevSurfaceCreate *surface)
> > >  {
> > > -    red_dispatcher_create_primary_surface(instance->st->dispatcher,
> > > surface_id, surface, 0, 0);
> > > +    red_qxl_create_primary_surface(instance->st->dispatcher, surface_id,
> > > surface, 0, 0);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_reset_image_cache(QXLInstance *instance)
> > >  {
> > > -    red_dispatcher_reset_image_cache(instance->st->dispatcher);
> > > +    red_qxl_reset_image_cache(instance->st->dispatcher);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_reset_cursor(QXLInstance *instance)
> > >  {
> > > -    red_dispatcher_reset_cursor(instance->st->dispatcher);
> > > +    red_qxl_reset_cursor(instance->st->dispatcher);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_destroy_surface_wait(QXLInstance *instance, uint32_t
> > > surface_id)
> > >  {
> > > -    red_dispatcher_destroy_surface_wait(instance->st->dispatcher,
> > > surface_id,
> > > 0, 0);
> > > +    red_qxl_destroy_surface_wait(instance->st->dispatcher, surface_id,
> > > 0,
> > > 0);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_loadvm_commands(QXLInstance *instance, struct
> > >  QXLCommandExt
> > > *ext, uint32_t count)
> > >  {
> > > -    red_dispatcher_loadvm_commands(instance->st->dispatcher, ext,
> > > count);
> > > +    red_qxl_loadvm_commands(instance->st->dispatcher, ext, count);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_update_area_async(QXLInstance *instance, uint32_t
> > >  surface_id,
> > > QXLRect *qxl_area,
> > >                                   uint32_t clear_dirty_region, uint64_t
> > > cookie)
> > >  {
> > > -    red_dispatcher_update_area_async(instance->st->dispatcher,
> > > surface_id,
> > > qxl_area,
> > > +    red_qxl_update_area_async(instance->st->dispatcher, surface_id,
> > > qxl_area,
> > >                                       clear_dirty_region, cookie);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_add_memslot_async(QXLInstance *instance, QXLDevMemSlot
> > >  *slot,
> > > uint64_t cookie)
> > >  {
> > > -    red_dispatcher_add_memslot_async(instance->st->dispatcher, slot,
> > > cookie);
> > > +    red_qxl_add_memslot_async(instance->st->dispatcher, slot, cookie);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_destroy_surfaces_async(QXLInstance *instance, uint64_t
> > >  cookie)
> > >  {
> > > -    red_dispatcher_destroy_surfaces_async(instance->st->dispatcher,
> > > cookie);
> > > +    red_qxl_destroy_surfaces_async(instance->st->dispatcher, cookie);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_destroy_primary_surface_async(QXLInstance *instance,
> > >  uint32_t
> > > surface_id, uint64_t cookie)
> > >  {
> > > -    red_dispatcher_destroy_primary_surface(instance->st->dispatcher,
> > > surface_id, 1, cookie);
> > > +    red_qxl_destroy_primary_surface(instance->st->dispatcher,
> > > surface_id,
> > > 1,
> > > cookie);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_create_primary_surface_async(QXLInstance *instance,
> > >  uint32_t
> > > surface_id,
> > >                                  QXLDevSurfaceCreate *surface, uint64_t
> > > cookie)
> > >  {
> > > -    red_dispatcher_create_primary_surface(instance->st->dispatcher,
> > > surface_id, surface, 1, cookie);
> > > +    red_qxl_create_primary_surface(instance->st->dispatcher, surface_id,
> > > surface, 1, cookie);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_destroy_surface_async(QXLInstance *instance, uint32_t
> > > surface_id, uint64_t cookie)
> > >  {
> > > -    red_dispatcher_destroy_surface_wait(instance->st->dispatcher,
> > > surface_id,
> > > 1, cookie);
> > > +    red_qxl_destroy_surface_wait(instance->st->dispatcher, surface_id,
> > > 1,
> > > cookie);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_flush_surfaces_async(QXLInstance *instance, uint64_t
> > >  cookie)
> > >  {
> > > -    red_dispatcher_flush_surfaces_async(instance->st->dispatcher,
> > > cookie);
> > > +    red_qxl_flush_surfaces_async(instance->st->dispatcher, cookie);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_monitors_config_async(QXLInstance *instance, QXLPHYSICAL
> > > monitors_config,
> > >                                       int group_id, uint64_t cookie)
> > >  {
> > > -    red_dispatcher_monitors_config_async(instance->st->dispatcher,
> > > monitors_config, group_id, cookie);
> > > +    red_qxl_monitors_config_async(instance->st->dispatcher,
> > > monitors_config,
> > > group_id, cookie);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > > @@ -816,7 +816,7 @@ void spice_qxl_set_max_monitors(QXLInstance
> > > *instance,
> > > unsigned int max_monitors
> > >  SPICE_GNUC_VISIBLE
> > >  void spice_qxl_driver_unload(QXLInstance *instance)
> > >  {
> > > -    red_dispatcher_driver_unload(instance->st->dispatcher);
> > > +    red_qxl_driver_unload(instance->st->dispatcher);
> > >  }
> > >  
> > >  SPICE_GNUC_VISIBLE
> > > @@ -875,8 +875,8 @@ void spice_qxl_gl_draw_async(QXLInstance *qxl,
> > >      dispatcher_send_message(&dispatcher->dispatcher, message, &draw);
> > >  }
> > >  
> > > -void red_dispatcher_async_complete(struct RedDispatcher *dispatcher,
> > > -                                   AsyncCommand *async_command)
> > > +void red_qxl_async_complete(struct RedDispatcher *dispatcher,
> > > +                            AsyncCommand *async_command)
> > >  {
> > >      spice_debug("%p: cookie %" PRId64, async_command,
> > >      async_command->cookie);
> > >      switch (async_command->message) {
> > > @@ -889,10 +889,10 @@ void red_dispatcher_async_complete(struct
> > > RedDispatcher
> > > *dispatcher,
> > >      case RED_WORKER_MESSAGE_GL_DRAW_ASYNC:
> > >          break;
> > >      case RED_WORKER_MESSAGE_CREATE_PRIMARY_SURFACE_ASYNC:
> > > -        red_dispatcher_create_primary_surface_complete(dispatcher);
> > > +        red_qxl_create_primary_surface_complete(dispatcher);
> > >          break;
> > >      case RED_WORKER_MESSAGE_DESTROY_PRIMARY_SURFACE_ASYNC:
> > > -        red_dispatcher_destroy_primary_surface_complete(dispatcher);
> > > +        red_qxl_destroy_primary_surface_complete(dispatcher);
> > >          break;
> > >      default:
> > >          spice_warning("unexpected message %d", async_command->message);
> > > @@ -902,7 +902,7 @@ void red_dispatcher_async_complete(struct
> > > RedDispatcher
> > > *dispatcher,
> > >      free(async_command);
> > >  }
> > >  
> > > -void red_dispatcher_init(QXLInstance *qxl)
> > > +void red_qxl_init(QXLInstance *qxl)
> > >  {
> > >      RedDispatcher *red_dispatcher;
> > >      RedChannel *channel;
> > > @@ -947,17 +947,17 @@ void red_dispatcher_init(QXLInstance *qxl)
> > >  
> > >      // TODO: move to their respective channel files
> > >      channel = red_worker_get_cursor_channel(worker);
> > > -    client_cbs.connect = red_dispatcher_set_cursor_peer;
> > > -    client_cbs.disconnect = red_dispatcher_disconnect_cursor_peer;
> > > -    client_cbs.migrate = red_dispatcher_cursor_migrate;
> > > +    client_cbs.connect = red_qxl_set_cursor_peer;
> > > +    client_cbs.disconnect = red_qxl_disconnect_cursor_peer;
> > > +    client_cbs.migrate = red_qxl_cursor_migrate;
> > >      red_channel_register_client_cbs(channel, &client_cbs);
> > >      red_channel_set_data(channel, red_dispatcher);
> > >      reds_register_channel(reds, channel);
> > >  
> > >      channel = red_worker_get_display_channel(worker);
> > > -    client_cbs.connect = red_dispatcher_set_display_peer;
> > > -    client_cbs.disconnect = red_dispatcher_disconnect_display_peer;
> > > -    client_cbs.migrate = red_dispatcher_display_migrate;
> > > +    client_cbs.connect = red_qxl_set_display_peer;
> > > +    client_cbs.disconnect = red_qxl_disconnect_display_peer;
> > > +    client_cbs.migrate = red_qxl_display_migrate;
> > >      red_channel_register_client_cbs(channel, &client_cbs);
> > >      red_channel_set_data(channel, red_dispatcher);
> > >      red_channel_set_cap(channel, SPICE_DISPLAY_CAP_MONITORS_CONFIG);
> > > @@ -970,30 +970,30 @@ void red_dispatcher_init(QXLInstance *qxl)
> > >      qxl->st->dispatcher = red_dispatcher;
> > >  }
> > >  
> > > -struct Dispatcher *red_dispatcher_get_dispatcher(RedDispatcher
> > > *red_dispatcher)
> > > +struct Dispatcher *red_qxl_get_dispatcher(RedDispatcher *red_dispatcher)
> > >  {
> > >      return &red_dispatcher->dispatcher;
> > >  }
> > >  
> > > -void red_dispatcher_set_dispatcher_opaque(RedDispatcher *red_dispatcher,
> > > +void red_qxl_set_dispatcher_opaque(RedDispatcher *red_dispatcher,
> > >                                            void *opaque)
> > >  {
> > >      dispatcher_set_opaque(&red_dispatcher->dispatcher, opaque);
> > >  }
> > >  
> > > -void red_dispatcher_clear_pending(RedDispatcher *red_dispatcher, int
> > > pending)
> > > +void red_qxl_clear_pending(RedDispatcher *red_dispatcher, int pending)
> > >  {
> > >      spice_return_if_fail(red_dispatcher != NULL);
> > >  
> > >      clear_bit(pending, &red_dispatcher->pending);
> > >  }
> > >  
> > > -gboolean red_dispatcher_get_primary_active(RedDispatcher *dispatcher)
> > > +gboolean red_qxl_get_primary_active(RedDispatcher *dispatcher)
> > >  {
> > >      return dispatcher->primary_active;
> > >  }
> > >  
> > > -gboolean red_dispatcher_get_allow_client_mouse(RedDispatcher
> > > *dispatcher,
> > > gint *x_res, gint *y_res)
> > > +gboolean red_qxl_get_allow_client_mouse(RedDispatcher *dispatcher, gint
> > > *x_res, gint *y_res)
> > >  {
> > >      if (dispatcher->use_hardware_cursor) {
> > >          if (x_res)
> > > @@ -1004,7 +1004,7 @@ gboolean
> > > red_dispatcher_get_allow_client_mouse(RedDispatcher *dispatcher, gint *
> > >      return dispatcher->use_hardware_cursor;
> > >  }
> > >  
> > > -void red_dispatcher_on_ic_change(RedDispatcher *dispatcher,
> > > SpiceImageCompression ic)
> > > +void red_qxl_on_ic_change(RedDispatcher *dispatcher,
> > > SpiceImageCompression
> > > ic)
> > >  {
> > >      RedWorkerMessageSetCompression payload;
> > >      payload.image_compression = ic;
> > > @@ -1013,7 +1013,7 @@ void red_dispatcher_on_ic_change(RedDispatcher
> > > *dispatcher, SpiceImageCompressio
> > >                              &payload);
> > >  }
> > >  
> > > -void red_dispatcher_on_sv_change(RedDispatcher *dispatcher, int sv)
> > > +void red_qxl_on_sv_change(RedDispatcher *dispatcher, int sv)
> > >  {
> > >      RedWorkerMessageSetStreamingVideo payload;
> > >      payload.streaming_video = sv;
> > > @@ -1022,7 +1022,7 @@ void red_dispatcher_on_sv_change(RedDispatcher
> > > *dispatcher, int sv)
> > >                              &payload);
> > >  }
> > >  
> > > -void red_dispatcher_set_mouse_mode(RedDispatcher *dispatcher, uint32_t
> > > mode)
> > > +void red_qxl_set_mouse_mode(RedDispatcher *dispatcher, uint32_t mode)
> > >  {
> > >      RedWorkerMessageSetMouseMode payload;
> > >      payload.mode = mode;
> > > diff --git a/server/red-dispatcher.h b/server/red-dispatcher.h
> > > index 1aa63c2..1348e6c 100644
> > > --- a/server/red-dispatcher.h
> > > +++ b/server/red-dispatcher.h
> > > @@ -24,23 +24,23 @@ typedef struct RedDispatcher RedDispatcher;
> > >  
> > >  typedef struct AsyncCommand AsyncCommand;
> > >  
> > > -void red_dispatcher_init(QXLInstance *qxl);
> > > -
> > > -void red_dispatcher_set_mm_time(RedDispatcher *dispatcher, uint32_t);
> > > -void red_dispatcher_on_ic_change(RedDispatcher *dispatcher,
> > > SpiceImageCompression ic);
> > > -void red_dispatcher_on_sv_change(RedDispatcher *dispatcher, int sv);
> > > -void red_dispatcher_set_mouse_mode(RedDispatcher *dispatcher, uint32_t
> > > mode);
> > > -void red_dispatcher_attach_worker(RedDispatcher *dispatcher);
> > > -void red_dispatcher_set_compression_level(RedDispatcher *dispatcher, int
> > > level);
> > > -void red_dispatcher_stop(RedDispatcher *dispatcher);
> > > -void red_dispatcher_start(RedDispatcher *dispatcher);
> > > -uint32_t red_dispatcher_qxl_ram_size(RedDispatcher *dispatcher);
> > > -void red_dispatcher_async_complete(struct RedDispatcher *, AsyncCommand
> > > *);
> > > -struct Dispatcher *red_dispatcher_get_dispatcher(struct RedDispatcher
> > > *);
> > > -gboolean red_dispatcher_use_client_monitors_config(RedDispatcher
> > > *dispatcher);
> > > -gboolean red_dispatcher_client_monitors_config(RedDispatcher
> > > *dispatcher,
> > > VDAgentMonitorsConfig *monitors_config);
> > > -gboolean red_dispatcher_get_primary_active(RedDispatcher *dispatcher);
> > > -gboolean red_dispatcher_get_allow_client_mouse(RedDispatcher
> > > *dispatcher,
> > > gint *x_res, gint *y_res);
> > > +void red_qxl_init(QXLInstance *qxl);
> > > +
> > > +void red_qxl_set_mm_time(RedDispatcher *dispatcher, uint32_t);
> > > +void red_qxl_on_ic_change(RedDispatcher *dispatcher,
> > > SpiceImageCompression
> > > ic);
> > > +void red_qxl_on_sv_change(RedDispatcher *dispatcher, int sv);
> > > +void red_qxl_set_mouse_mode(RedDispatcher *dispatcher, uint32_t mode);
> > > +void red_qxl_attach_worker(RedDispatcher *dispatcher);
> > > +void red_qxl_set_compression_level(RedDispatcher *dispatcher, int
> > > level);
> > > +void red_qxl_stop(RedDispatcher *dispatcher);
> > > +void red_qxl_start(RedDispatcher *dispatcher);
> > > +uint32_t red_qxl_get_ram_size(RedDispatcher *dispatcher);
> > > +void red_qxl_async_complete(struct RedDispatcher *, AsyncCommand *);
> > > +struct Dispatcher *red_qxl_get_dispatcher(struct RedDispatcher *);
> > > +gboolean red_qxl_use_client_monitors_config(RedDispatcher *dispatcher);
> > > +gboolean red_qxl_client_monitors_config(RedDispatcher *dispatcher,
> > > VDAgentMonitorsConfig *monitors_config);
> > > +gboolean red_qxl_get_primary_active(RedDispatcher *dispatcher);
> > > +gboolean red_qxl_get_allow_client_mouse(RedDispatcher *dispatcher, gint
> > > *x_res, gint *y_res);
> > >  
> > >  typedef uint32_t RedWorkerMessage;
> > >  
> > > @@ -267,6 +267,6 @@ enum {
> > >      RED_DISPATCHER_PENDING_OOM,
> > >  };
> > >  
> > > -void red_dispatcher_clear_pending(RedDispatcher *red_dispatcher, int
> > > pending);
> > > +void red_qxl_clear_pending(RedDispatcher *red_dispatcher, int pending);
> > >  
> > >  #endif
> > > diff --git a/server/red-worker.c b/server/red-worker.c
> > > index 771e1eb..b8ab555 100644
> > > --- a/server/red-worker.c
> > > +++ b/server/red-worker.c
> > > @@ -823,7 +823,7 @@ static void handle_dev_wakeup(void *opaque, void
> > > *payload)
> > >      RedWorker *worker = opaque;
> > >  
> > >      stat_inc_counter(reds, worker->wakeup_counter, 1);
> > > -    red_dispatcher_clear_pending(worker->red_dispatcher,
> > > RED_DISPATCHER_PENDING_WAKEUP);
> > > +    red_qxl_clear_pending(worker->red_dispatcher,
> > > RED_DISPATCHER_PENDING_WAKEUP);
> > >  }
> > >  
> > >  static void handle_dev_oom(void *opaque, void *payload)
> > > @@ -853,7 +853,7 @@ static void handle_dev_oom(void *opaque, void
> > > *payload)
> > >                  display->glz_drawable_count,
> > >                  display->current_size,
> > >                  red_channel_sum_pipes_size(display_red_channel));
> > > -    red_dispatcher_clear_pending(worker->red_dispatcher,
> > > RED_DISPATCHER_PENDING_OOM);
> > > +    red_qxl_clear_pending(worker->red_dispatcher,
> > > RED_DISPATCHER_PENDING_OOM);
> > >  }
> > >  
> > >  static void handle_dev_reset_cursor(void *opaque, void *payload)
> > > @@ -1194,7 +1194,7 @@ static void
> > > worker_handle_dispatcher_async_done(void
> > > *opaque,
> > >      RedWorkerMessageAsync *msg_async = payload;
> > >  
> > >      spice_debug(NULL);
> > > -    red_dispatcher_async_complete(worker->red_dispatcher,
> > > msg_async->cmd);
> > > +    red_qxl_async_complete(worker->red_dispatcher, msg_async->cmd);
> > >  }
> > >  
> > >  static void worker_dispatcher_record(void *opaque, uint32_t
> > >  message_type,
> > > void *payload)
> > > @@ -1394,7 +1394,7 @@ static void handle_dev_input(int fd, int event,
> > > void
> > > *opaque)
> > >  {
> > >      RedWorker *worker = opaque;
> > >  
> > > -    dispatcher_handle_recv_read(red_dispatcher_get_dispatcher(worker
> > > ->red_dispatcher));
> > > +    dispatcher_handle_recv_read(red_qxl_get_dispatcher(worker
> > > ->red_dispatcher));
> > >  }
> > >  
> > >  typedef struct RedWorkerSource {
> > > @@ -1487,7 +1487,7 @@ RedWorker* red_worker_new(QXLInstance *qxl,
> > > RedDispatcher *red_dispatcher)
> > >              spice_error("failed to write replay header");
> > >          }
> > >      }
> > > -    dispatcher = red_dispatcher_get_dispatcher(red_dispatcher);
> > > +    dispatcher = red_qxl_get_dispatcher(red_dispatcher);
> > >      dispatcher_set_opaque(dispatcher, worker);
> > >  
> > >      worker->red_dispatcher = red_dispatcher;
> > > diff --git a/server/reds.c b/server/reds.c
> > > index e58cec5..decf846 100644
> > > --- a/server/reds.c
> > > +++ b/server/reds.c
> > > @@ -571,7 +571,7 @@ static void reds_set_mouse_mode(RedsState *reds,
> > > uint32_t
> > > mode)
> > >      reds->mouse_mode = mode;
> > >  
> > >      for (l = reds->dispatchers; l != NULL; l = l->next)
> > > -        red_dispatcher_set_mouse_mode(l->data, mode);
> > > +        red_qxl_set_mouse_mode(l->data, mode);
> > >  
> > >      main_channel_push_mouse_mode(reds->main_channel, reds->mouse_mode,
> > >      reds
> > > ->is_client_mouse_allowed);
> > >  }
> > > @@ -3209,7 +3209,7 @@ SPICE_GNUC_VISIBLE int
> > > spice_server_add_interface(SpiceServer *s,
> > >          pthread_mutex_init(&qxl->st->scanout_mutex, NULL);
> > >          qxl->st->scanout.drm_dma_buf_fd = -1;
> > >          qxl->st->qif = SPICE_CONTAINEROF(interface, QXLInterface, base);
> > > -        red_dispatcher_init(qxl);
> > > +        red_qxl_init(qxl);
> > >          dispatcher = qxl->st->dispatcher;
> > >          reds->dispatchers = g_list_prepend(reds->dispatchers,
> > >          dispatcher);
> > >  
> > > @@ -3218,8 +3218,8 @@ SPICE_GNUC_VISIBLE int
> > > spice_server_add_interface(SpiceServer *s,
> > >           * this callback is called. This as clients assume they can
> > >           start
> > >           the
> > >           * dispatchers. Also note that this should be the first callback
> > >           to
> > >           * be called. */
> > > -        red_dispatcher_attach_worker(dispatcher);
> > > -        red_dispatcher_set_compression_level(dispatcher,
> > > calc_compression_level(reds));
> > > +        red_qxl_attach_worker(dispatcher);
> > > +        red_qxl_set_compression_level(dispatcher,
> > > calc_compression_level(reds));
> > >      } else if (strcmp(interface->type, SPICE_INTERFACE_TABLET) == 0) {
> > >          SpiceTabletInstance *tablet = SPICE_CONTAINEROF(sin,
> > > SpiceTabletInstance, base);
> > >          spice_info("SPICE_INTERFACE_TABLET");
> > > @@ -4078,8 +4078,8 @@ void reds_update_client_mouse_allowed(RedsState
> > > *reds)
> > >          for (l = reds->dispatchers; l != NULL && allow_now; l = l->next)
> > >          {
> > >  
> > >              RedDispatcher *now = l->data;
> > > -            if (red_dispatcher_get_primary_active(now)) {
> > > -                allow_now = red_dispatcher_get_allow_client_mouse(now,
> > > &x_res, &y_res);
> > > +            if (red_qxl_get_primary_active(now)) {
> > > +                allow_now = red_qxl_get_allow_client_mouse(now, &x_res,
> > > &y_res);
> > >                  break;
> > >              }
> > >          }
> > > @@ -4102,7 +4102,7 @@ gboolean reds_use_client_monitors_config(RedsState
> > > *reds)
> > >      for (l = reds->dispatchers; l != NULL ; l = l->next) {
> > >          RedDispatcher *now = l->data;
> > >  
> > > -        if (!red_dispatcher_use_client_monitors_config(now))
> > > +        if (!red_qxl_use_client_monitors_config(now))
> > >              return FALSE;
> > >      }
> > >      return TRUE;
> > > @@ -4114,7 +4114,7 @@ void reds_client_monitors_config(RedsState *reds,
> > > VDAgentMonitorsConfig *monitor
> > >  
> > >      for (l = reds->dispatchers; l != NULL; l = l->next) {
> > >          RedDispatcher *now = l->data;
> > > -        if (!red_dispatcher_client_monitors_config(now,
> > > monitors_config))
> > > {
> > > +        if (!red_qxl_client_monitors_config(now, monitors_config)) {
> > >              /* this is a normal condition, some qemu devices might not
> > > implement it */
> > >              spice_debug("QXLInterface::client_monitors_config
> > >              failed\n");
> > >          }
> > > @@ -4127,7 +4127,7 @@ void reds_set_mm_time(RedsState *reds, uint32_t
> > > mm_time)
> > >  
> > >      for (l = reds->dispatchers; l != NULL; l = l->next) {
> > >          RedDispatcher *now = l->data;
> > > -        red_dispatcher_set_mm_time(now, mm_time);
> > > +        red_qxl_set_mm_time(now, mm_time);
> > >      }
> > >  }
> > >  
> > > @@ -4150,8 +4150,8 @@ void reds_on_ic_change(RedsState *reds)
> > >  
> > >      for (l = reds->dispatchers; l != NULL; l = l->next) {
> > >          RedDispatcher *d = l->data;
> > > -        red_dispatcher_set_compression_level(d, compression_level);
> > > -        red_dispatcher_on_ic_change(d,
> > > spice_server_get_image_compression(reds));
> > > +        red_qxl_set_compression_level(d, compression_level);
> > > +        red_qxl_on_ic_change(d,
> > > spice_server_get_image_compression(reds));
> > >      }
> > >  }
> > >  
> > > @@ -4162,8 +4162,8 @@ void reds_on_sv_change(RedsState *reds)
> > >  
> > >      for (l = reds->dispatchers; l != NULL; l = l->next) {
> > >          RedDispatcher *d = l->data;
> > > -        red_dispatcher_set_compression_level(d, compression_level);
> > > -        red_dispatcher_on_sv_change(d, reds_get_streaming_video(reds));
> > > +        red_qxl_set_compression_level(d, compression_level);
> > > +        red_qxl_on_sv_change(d, reds_get_streaming_video(reds));
> > >      }
> > >  }
> > >  
> > > @@ -4172,7 +4172,7 @@ void reds_on_vm_stop(RedsState *reds)
> > >      GList *l;
> > >  
> > >      for (l = reds->dispatchers; l != NULL; l = l->next)
> > > -        red_dispatcher_stop(l->data);
> > > +        red_qxl_stop(l->data);
> > >  }
> > >  
> > >  void reds_on_vm_start(RedsState *reds)
> > > @@ -4180,7 +4180,7 @@ void reds_on_vm_start(RedsState *reds)
> > >      GList *l;
> > >  
> > >      for (l = reds->dispatchers; l != NULL; l = l->next)
> > > -        red_dispatcher_start(l->data);
> > > +        red_qxl_start(l->data);
> > >  }
> > >  
> > >  uint32_t reds_qxl_ram_size(RedsState *reds)
> > > @@ -4191,7 +4191,7 @@ uint32_t reds_qxl_ram_size(RedsState *reds)
> > >      }
> > >  
> > >      first = reds->dispatchers->data;
> > > -    return red_dispatcher_qxl_ram_size(first);
> > > +    return red_qxl_get_ram_size(first);
> > >  }
> > >  
> > >  MainDispatcher* reds_get_main_dispatcher(RedsState *reds)
> > 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/spice-devel
> 


More information about the Spice-devel mailing list