[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC] migration: make sure to run iterate precopy
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH RFC] migration: make sure to run iterate precopy during the bulk stage |
Date: |
Tue, 4 Sep 2018 10:09:28 +0100 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
* Quan Xu (address@hidden) wrote:
> From 8dbf7370e7ea1caab0b769d0d4dcdd072d14d421 Mon Sep 17 00:00:00 2001
> From: Quan Xu <address@hidden>
> Date: Wed, 29 Aug 2018 21:33:14 +0800
> Subject: [PATCH RFC] migration: make sure to run iterate precopy during the
> bulk stage
>
> Since the bulk stage assumes in (migration_bitmap_find_dirty) that every
> page is dirty, return a rough total ram as pending size to make sure that
> migration thread continues to run iterate precopy during the bulk stage.
>
> Otherwise the downtime grows unpredictably, as migration thread needs to
> send both the rest of pages and dirty pages during complete precopy.
>
> Signed-off-by: Quan Xu <address@hidden>
Hi,
Can you explain a bit more about the problem you're seeing?
I think you're saying it's exiting the iteration near the end of
the bulk stage, before it's done the first sync?
> ---
> migration/ram.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/migration/ram.c b/migration/ram.c
> index 79c8942..cfa304c 100644
> --- a/migration/ram.c
> +++ b/migration/ram.c
> @@ -3308,7 +3308,8 @@ static void ram_save_pending(QEMUFile *f, void
> *opaque, uint64_t max_size,
> /* We can do postcopy, and all the data is postcopiable */
> *res_compatible += remaining_size;
> } else {
> - *res_precopy_only += remaining_size;
> + *res_precopy_only += (rs->ram_bulk_stage ?
> + ram_bytes_total() : remaining_size);
So that's assuming that the whole of RAM is dirty, even if we're
near the end of the bulk stage?
Dave
> }
> }
>
> --
> 1.8.3.1
>
>
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- Re: [Qemu-devel] [PATCH RFC] migration: make sure to run iterate precopy during the bulk stage,
Dr. David Alan Gilbert <=