[Spice-devel] [PATCH 05/18] Call public RedClient API instead of poking internals

Frediano Ziglio fziglio at redhat.com
Thu Apr 28 17:11:27 UTC 2016


> 
> Use red_client_during_migrate_at_target() instead of checking
> rcc->client->during_target_migrate.
> ---
>  server/red-channel.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/server/red-channel.c b/server/red-channel.c
> index 2bcf919..a7c4e5b 100644
> --- a/server/red-channel.c
> +++ b/server/red-channel.c
> @@ -839,7 +839,7 @@ void
> red_channel_client_start_connectivity_monitoring(RedChannelClient *rcc, uin
>      if (rcc->latency_monitor.timer == NULL) {
>          rcc->latency_monitor.timer = rcc->channel->core->timer_add(
>              rcc->channel->core, red_channel_client_ping_timer, rcc);
> -        if (!rcc->client->during_target_migrate) {
> +        if (!red_client_during_migrate_at_target(rcc->client)) {
>              red_channel_client_start_ping_timer(rcc,
>              PING_TEST_IDLE_NET_TIMEOUT_MS);
>          }
>          rcc->latency_monitor.roundtrip = -1;
> @@ -849,7 +849,7 @@ void
> red_channel_client_start_connectivity_monitoring(RedChannelClient *rcc, uin
>          rcc->connectivity_monitor.timer = rcc->channel->core->timer_add(
>              rcc->channel->core, red_channel_client_connectivity_timer, rcc);
>          rcc->connectivity_monitor.timeout = timeout_ms;
> -        if (!rcc->client->during_target_migrate) {
> +        if (!red_client_during_migrate_at_target(rcc->client)) {
>             rcc->channel->core->timer_start(rcc->connectivity_monitor.timer,
>                                             rcc->connectivity_monitor.timeout);
>          }

Acked-by: Frediano Ziglio <fziglio at redhat.com>

Frediano


More information about the Spice-devel mailing list