Re: [Qemu-devel] [PATCH 2/4] migration: calculate end time after we have sent the data

[Orit Wasserman]

On 02/01/2013 02:32 PM, Juan Quintela wrote:
> Signed-off-by: Juan Quintela <address@hidden>
> ---
>  migration.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/migration.c b/migration.c
> index d86946e..67abd12 100644
> --- a/migration.c
> +++ b/migration.c
> @@ -681,7 +681,7 @@ static void *buffered_file_thread(void *opaque)
>      qemu_mutex_unlock_iothread();
> 
>      while (true) {
> -        int64_t current_time = qemu_get_clock_ms(rt_clock);
> +        int64_t current_time;
>          uint64_t pending_size;
> 
>          qemu_mutex_lock_iothread();
> @@ -735,6 +735,7 @@ static void *buffered_file_thread(void *opaque)
>              }
>          }
>          qemu_mutex_unlock_iothread();
> +        current_time = qemu_get_clock_ms(rt_clock);
>          if (current_time >= initial_time + BUFFER_DELAY) {
>              uint64_t transferred_bytes = s->bytes_xfer;
>              uint64_t time_spent = current_time - initial_time;
> 
Reviewed-by: Orit Wasserman <address@hidden>