[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 28/41] migration: use qemu_file_rate_limit consi
From: |
Orit Wasserman |
Subject: |
Re: [Qemu-devel] [PATCH 28/41] migration: use qemu_file_rate_limit consistently |
Date: |
Thu, 21 Feb 2013 09:34:40 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 |
On 02/15/2013 07:47 PM, Paolo Bonzini wrote:
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> migration.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/migration.c b/migration.c
> index 3de7cbf..8d35af5 100644
> --- a/migration.c
> +++ b/migration.c
> @@ -616,7 +616,7 @@ static void *migration_thread(void *opaque)
> int64_t current_time = qemu_get_clock_ms(rt_clock);
> uint64_t pending_size;
>
> - if (s->bytes_xfer < s->xfer_limit) {
> + if (!qemu_file_rate_limit(s->file)) {
> DPRINTF("iterate\n");
> pending_size = qemu_savevm_state_pending(s->file, max_size);
> DPRINTF("pending size %lu max %lu\n", pending_size, max_size);
> @@ -655,7 +655,7 @@ static void *migration_thread(void *opaque)
> s->bytes_xfer = 0;
> initial_time = current_time;
> }
> - if (s->bytes_xfer >= s->xfer_limit) {
> + if (qemu_file_rate_limit(s->file)) {
> /* usleep expects microseconds */
> g_usleep((initial_time + BUFFER_DELAY - current_time)*1000);
> }
>
Reviewed-by: Orit Wasserman <address@hidden>
- [Qemu-devel] [PATCH 34/41] migration: use QEMUFile for migration channel lifetime, (continued)
- [Qemu-devel] [PATCH 34/41] migration: use QEMUFile for migration channel lifetime, Paolo Bonzini, 2013/02/15
- [Qemu-devel] [PATCH 29/41] migration: merge qemu_popen_cmd with qemu_popen, Paolo Bonzini, 2013/02/15
- [Qemu-devel] [PATCH 24/41] qemu-file: make qemu_fflush and qemu_file_set_error private again, Paolo Bonzini, 2013/02/15
- [Qemu-devel] [PATCH 28/41] migration: use qemu_file_rate_limit consistently, Paolo Bonzini, 2013/02/15
- Re: [Qemu-devel] [PATCH 00/41] Migration cleanups and latency improvements, Chegu Vinod, 2013/02/19