[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 09/41] ram: introduce migration_bitmap_set_dirty
From: |
Orit Wasserman |
Subject: |
Re: [Qemu-devel] [PATCH 09/41] ram: introduce migration_bitmap_set_dirty() |
Date: |
Tue, 25 Sep 2012 10:38:35 +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:
> It just marks a region of memory as dirty.
>
> Signed-off-by: Juan Quintela <address@hidden>
> ---
> arch_init.c | 20 +++++++++++++-------
> 1 file changed, 13 insertions(+), 7 deletions(-)
>
> diff --git a/arch_init.c b/arch_init.c
> index 57f7f1a..b2dcc24 100644
> --- a/arch_init.c
> +++ b/arch_init.c
> @@ -332,6 +332,18 @@ static int save_xbzrle_page(QEMUFile *f, uint8_t
> *current_data,
> static RAMBlock *last_block;
> static ram_addr_t last_offset;
>
> +static inline void migration_bitmap_set_dirty(MemoryRegion *mr, int length)
> +{
> + ram_addr_t addr;
> +
> + for (addr = 0; addr < length; addr += TARGET_PAGE_SIZE) {
> + if (!memory_region_get_dirty(mr, addr, TARGET_PAGE_SIZE,
> + DIRTY_MEMORY_MIGRATION)) {
> + memory_region_set_dirty(mr, addr, TARGET_PAGE_SIZE);
> + }
> + }
> +}
> +
> /*
> * ram_save_block: Writes a page of memory to the stream f
> *
> @@ -494,7 +506,6 @@ static void reset_ram_globals(void)
>
> static int ram_save_setup(QEMUFile *f, void *opaque)
> {
> - ram_addr_t addr;
> RAMBlock *block;
>
> memory_global_sync_dirty_bitmap(get_system_memory());
> @@ -516,12 +527,7 @@ static int ram_save_setup(QEMUFile *f, void *opaque)
>
> /* Make sure all dirty bits are set */
> QLIST_FOREACH(block, &ram_list.blocks, next) {
> - for (addr = 0; addr < block->length; addr += TARGET_PAGE_SIZE) {
> - if (!memory_region_get_dirty(block->mr, addr, TARGET_PAGE_SIZE,
> - DIRTY_MEMORY_MIGRATION)) {
> - memory_region_set_dirty(block->mr, addr, TARGET_PAGE_SIZE);
> - }
> - }
> + migration_bitmap_set_dirty(block->mr, block->length);
> }
>
> memory_global_dirty_log_start();
>
Reviewed-by: Orit Wasserman <address@hidden>
- [Qemu-devel] [PATCH 04/41] migration: print total downtime for final phase of migration, (continued)
- [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
- [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
- Re: [Qemu-devel] [PATCH 09/41] ram: introduce migration_bitmap_set_dirty(),
Orit Wasserman <=
- [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
- [Qemu-devel] [PATCH 14/41] Separate migration bitmap, Juan Quintela, 2012/09/21
- [Qemu-devel] [PATCH 17/41] buffered_file: rename opaque to migration_state, Juan Quintela, 2012/09/21
- [Qemu-devel] [PATCH 20/41] buffered_file: unfold migrate_fd_put_ready, Juan Quintela, 2012/09/21
- [Qemu-devel] [PATCH 18/41] buffered_file: opaque is MigrationState, Juan Quintela, 2012/09/21