[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 08/41] savevm: Factorize ram globals reset in it
From: |
Orit Wasserman |
Subject: |
Re: [Qemu-devel] [PATCH 08/41] savevm: Factorize ram globals reset in its own function |
Date: |
Tue, 25 Sep 2012 10:37:58 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20120827 Thunderbird/15.0 |
On 09/21/2012 11:47 AM, Juan Quintela wrote:
> Signed-off-by: Juan Quintela <address@hidden>
> ---
> arch_init.c | 12 +++++++++---
> 1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/arch_init.c b/arch_init.c
> index 52ccc7b..57f7f1a 100644
> --- a/arch_init.c
> +++ b/arch_init.c
> @@ -482,6 +482,14 @@ static void ram_migration_cancel(void *opaque)
> migration_end();
> }
>
> +
> +static void reset_ram_globals(void)
> +{
> + last_block = NULL;
> + last_offset = 0;
> + sort_ram_list();
> +}
> +
> #define MAX_WAIT 50 /* ms, half buffered_file limit */
>
> static int ram_save_setup(QEMUFile *f, void *opaque)
> @@ -491,9 +499,7 @@ static int ram_save_setup(QEMUFile *f, void *opaque)
>
> memory_global_sync_dirty_bitmap(get_system_memory());
> bytes_transferred = 0;
> - last_block = NULL;
> - last_offset = 0;
> - sort_ram_list();
> + reset_ram_globals();
>
> if (migrate_use_xbzrle()) {
> XBZRLE.cache = cache_init(migrate_xbzrle_cache_size() /
>
Reviewed-by: Orit Wasserman <address@hidden>
- [Qemu-devel] [PATCH 05/41] migration: rename expected_time to expected_downtime, (continued)
- [Qemu-devel] [PATCH 05/41] migration: rename expected_time to expected_downtime, Juan Quintela, 2012/09/21
- [Qemu-devel] [PATCH 03/41] migration: store end_time in a local variable, Juan Quintela, 2012/09/21
- [Qemu-devel] [PATCH 04/41] migration: print total downtime for final phase of migration, Juan Quintela, 2012/09/21
- [Qemu-devel] [PATCH 08/41] savevm: Factorize ram globals reset in its own function, Juan Quintela, 2012/09/21
- Re: [Qemu-devel] [PATCH 08/41] savevm: Factorize ram globals reset in its own function,
Orit Wasserman <=
- [Qemu-devel] [PATCH 07/41] migration: print expected downtime in info migrate, Juan Quintela, 2012/09/21
- [Qemu-devel] [PATCH 06/41] migration: export migrate_get_current(), Juan Quintela, 2012/09/21
- [Qemu-devel] [PATCH 11/41] ram: Export last_ram_offset(), Juan Quintela, 2012/09/21
- [Qemu-devel] [PATCH 09/41] ram: introduce migration_bitmap_set_dirty(), Juan Quintela, 2012/09/21
- [Qemu-devel] [PATCH 16/41] BufferedFile: append, then flush, Juan Quintela, 2012/09/21
- [Qemu-devel] [PATCH 12/41] ram: introduce migration_bitmap_sync(), Juan Quintela, 2012/09/21
- [Qemu-devel] [PATCH 10/41] ram: Introduce migration_bitmap_test_and_reset_dirty(), Juan Quintela, 2012/09/21