Re: [PATCH spice-server 05/10] main-dispatcher: Use dispatcher_send_message_generic

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



There should be some justification in the commit log about why this is
an improvement. 

Jonathon

On Wed, 2019-03-20 at 09:59 +0000, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
>  server/main-dispatcher.c | 38 ++++++++------------------------------
>  1 file changed, 8 insertions(+), 30 deletions(-)
> 
> diff --git a/server/main-dispatcher.c b/server/main-dispatcher.c
> index 839e7242..904579a8 100644
> --- a/server/main-dispatcher.c
> +++ b/server/main-dispatcher.c
> @@ -121,15 +121,6 @@ main_dispatcher_init(MainDispatcher *self)
>      self->priv = main_dispatcher_get_instance_private(self);
>  }
>  
> -enum {
> -    MAIN_DISPATCHER_CHANNEL_EVENT = 0,
> -    MAIN_DISPATCHER_MIGRATE_SEAMLESS_DST_COMPLETE,
> -    MAIN_DISPATCHER_SET_MM_TIME_LATENCY,
> -    MAIN_DISPATCHER_CLIENT_DISCONNECT,
> -
> -    MAIN_DISPATCHER_NUM_MESSAGES
> -};
> -
>  typedef struct MainDispatcherChannelEventMessage {
>      int event;
>      SpiceChannelEventInfo *info;
> @@ -168,8 +159,8 @@ void main_dispatcher_channel_event(MainDispatcher
> *self, int event, SpiceChannel
>      }
>      msg.event = event;
>      msg.info = info;
> -    dispatcher_send_message(DISPATCHER(self),
> MAIN_DISPATCHER_CHANNEL_EVENT,
> -                            &msg);
> +    dispatcher_send_message_generic(DISPATCHER(self),
> main_dispatcher_handle_channel_event,
> +                                    &msg, sizeof(msg), false);
>  }
>  
>  
> @@ -214,8 +205,8 @@ void
> main_dispatcher_seamless_migrate_dst_complete(MainDispatcher *self,
>      }
>  
>      msg.client = g_object_ref(client);
> -    dispatcher_send_message(DISPATCHER(self),
> MAIN_DISPATCHER_MIGRATE_SEAMLESS_DST_COMPLETE,
> -                            &msg);
> +    dispatcher_send_message_generic(DISPATCHER(self),
> main_dispatcher_handle_migrate_complete,
> +                                    &msg, sizeof(msg), false);
>  }
>  
>  void main_dispatcher_set_mm_time_latency(MainDispatcher *self,
> RedClient *client, uint32_t latency)
> @@ -229,8 +220,8 @@ void
> main_dispatcher_set_mm_time_latency(MainDispatcher *self, RedClient
> *client
>  
>      msg.client = g_object_ref(client);
>      msg.latency = latency;
> -    dispatcher_send_message(DISPATCHER(self),
> MAIN_DISPATCHER_SET_MM_TIME_LATENCY,
> -                            &msg);
> +    dispatcher_send_message_generic(DISPATCHER(self),
> main_dispatcher_handle_mm_time_latency,
> +                                    &msg, sizeof(msg), false);
>  }
>  
>  void main_dispatcher_client_disconnect(MainDispatcher *self,
> RedClient *client)
> @@ -240,8 +231,8 @@ void
> main_dispatcher_client_disconnect(MainDispatcher *self, RedClient
> *client)
>      if (!red_client_is_disconnecting(client)) {
>          spice_debug("client %p", client);
>          msg.client = g_object_ref(client);
> -        dispatcher_send_message(DISPATCHER(self),
> MAIN_DISPATCHER_CLIENT_DISCONNECT,
> -                                &msg);
> +        dispatcher_send_message_generic(DISPATCHER(self),
> main_dispatcher_handle_client_disconnect,
> +                                        &msg, sizeof(msg), false);
>      } else {
>          spice_debug("client %p already during disconnection",
> client);
>      }
> @@ -263,7 +254,6 @@ MainDispatcher* main_dispatcher_new(RedsState
> *reds)
>  {
>      MainDispatcher *self = g_object_new(TYPE_MAIN_DISPATCHER,
>                                          "spice-server", reds,
> -                                        "max-message-type",
> MAIN_DISPATCHER_NUM_MESSAGES,
>                                          NULL);
>      return self;
>  }
> @@ -280,18 +270,6 @@ void main_dispatcher_constructed(GObject
> *object)
>                              dispatcher_get_recv_fd(DISPATCHER(self))
> ,
>                              SPICE_WATCH_EVENT_READ,
> dispatcher_handle_read,
>                              DISPATCHER(self));
> -    dispatcher_register_handler(DISPATCHER(self),
> MAIN_DISPATCHER_CHANNEL_EVENT,
> -                                main_dispatcher_handle_channel_event
> ,
> -                                sizeof(MainDispatcherChannelEventMes
> sage), false);
> -    dispatcher_register_handler(DISPATCHER(self),
> MAIN_DISPATCHER_MIGRATE_SEAMLESS_DST_COMPLETE,
> -                                main_dispatcher_handle_migrate_compl
> ete,
> -                                sizeof(MainDispatcherMigrateSeamless
> DstCompleteMessage), false);
> -    dispatcher_register_handler(DISPATCHER(self),
> MAIN_DISPATCHER_SET_MM_TIME_LATENCY,
> -                                main_dispatcher_handle_mm_time_laten
> cy,
> -                                sizeof(MainDispatcherMmTimeLatencyMe
> ssage), false);
> -    dispatcher_register_handler(DISPATCHER(self),
> MAIN_DISPATCHER_CLIENT_DISCONNECT,
> -                                main_dispatcher_handle_client_discon
> nect,
> -                                sizeof(MainDispatcherClientDisconnec
> tMessage), false);
>  }
>  
>  static void main_dispatcher_finalize(GObject *object)

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]