[Spice-devel] [PATCH spice-server 1/2] red_channel: stop sending ping messages after migration has completed

[Hans de Goede]

Hi,

Looks good, ack series.

Regards,

Hans


On 05/01/2013 08:19 PM, Yonit Halperin wrote:
> We mustn't send any msg to the client, besides MSG_MIGRATE_DATA, after
> we send MSG_MIGRATE.
> ---
>   server/red_channel.c | 5 +++++
>   1 file changed, 5 insertions(+)
>
> diff --git a/server/red_channel.c b/server/red_channel.c
> index 9047762..4e03d64 100644
> --- a/server/red_channel.c
> +++ b/server/red_channel.c
> @@ -873,6 +873,11 @@ static void red_channel_client_default_disconnect(RedChannelClient *base)
>
>   void red_channel_client_default_migrate(RedChannelClient *rcc)
>   {
> +    if (rcc->latency_monitor.timer) {
> +        red_channel_client_cancel_ping_timer(rcc);
> +        rcc->channel->core->timer_remove(rcc->latency_monitor.timer);
> +        rcc->latency_monitor.timer = NULL;
> +    }
>       red_channel_client_pipe_add_type(rcc, PIPE_ITEM_TYPE_MIGRATE);
>   }
>
>