Patchwork [27/35] ram: Use memory_region_test_and_clear_dirty

login
register
mail settings
Submitter Juan Quintela
Date Dec. 11, 2012, 12:47 p.m.
Message ID <1355230031-28233-28-git-send-email-quintela@redhat.com>
Download mbox | patch
Permalink /patch/205220/
State New
Headers show

Comments

Juan Quintela - Dec. 11, 2012, 12:47 p.m.
This avoids having to do two walks over the dirty bitmap, once reading
the dirty bits, and anthoer cleaning them.

Signed-off-by: Juan Quintela <quintela@redhat.com>
---
 arch_init.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

Patch

diff --git a/arch_init.c b/arch_init.c
index 2d566d6..b204602 100644
--- a/arch_init.c
+++ b/arch_init.c
@@ -390,13 +390,12 @@  static void migration_bitmap_sync(void)

     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)) {
+            if (memory_region_test_and_clear_dirty(block->mr,
+                                                   addr, TARGET_PAGE_SIZE,
+                                                   DIRTY_MEMORY_MIGRATION)) {
                 migration_bitmap_set_dirty(block->mr, addr);
             }
         }
-        memory_region_reset_dirty(block->mr, 0, block->length,
-                                  DIRTY_MEMORY_MIGRATION);
     }
     trace_migration_bitmap_sync_end(migration_dirty_pages
                                     - num_dirty_pages_init);