[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 4/5] migration: code clean up
From: |
Juan Quintela |
Subject: |
[Qemu-devel] [PULL 4/5] migration: code clean up |
Date: |
Wed, 4 Nov 2015 13:48:56 +0100 |
From: Liang Li <address@hidden>
Just clean up code, no behavior change.
Signed-off-by: Liang Li <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>al3
Reviewed-by: Amit Shah <address@hidden>al3
Signed-off-by: Juan Quintela <address@hidden>al3
---
migration/block.c | 9 ++-------
migration/ram.c | 9 ++-------
2 files changed, 4 insertions(+), 14 deletions(-)
diff --git a/migration/block.c b/migration/block.c
index ecfe005..cf9d9f8 100644
--- a/migration/block.c
+++ b/migration/block.c
@@ -591,7 +591,7 @@ static int64_t get_remaining_dirty(void)
/* Called with iothread lock taken. */
-static void blk_mig_cleanup(void)
+static void block_migration_cleanup(void *opaque)
{
BlkMigDevState *bmds;
BlkMigBlock *blk;
@@ -618,11 +618,6 @@ static void blk_mig_cleanup(void)
blk_mig_unlock();
}
-static void block_migration_cancel(void *opaque)
-{
- blk_mig_cleanup();
-}
-
static int block_save_setup(QEMUFile *f, void *opaque)
{
int ret;
@@ -884,7 +879,7 @@ static SaveVMHandlers savevm_block_handlers = {
.save_live_complete = block_save_complete,
.save_live_pending = block_save_pending,
.load_state = block_load,
- .cleanup = block_migration_cancel,
+ .cleanup = block_migration_cleanup,
.is_active = block_is_active,
};
diff --git a/migration/ram.c b/migration/ram.c
index 0a51473..df3df9e 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -1100,7 +1100,7 @@ static void migration_bitmap_free(struct BitmapRcu *bmap)
g_free(bmap);
}
-static void migration_end(void)
+static void ram_migration_cleanup(void *opaque)
{
/* caller have hold iothread lock or is in a bh, so there is
* no writing race against this migration_bitmap
@@ -1124,11 +1124,6 @@ static void migration_end(void)
XBZRLE_cache_unlock();
}
-static void ram_migration_cancel(void *opaque)
-{
- migration_end();
-}
-
static void reset_ram_globals(void)
{
last_seen_block = NULL;
@@ -1685,7 +1680,7 @@ static SaveVMHandlers savevm_ram_handlers = {
.save_live_complete = ram_save_complete,
.save_live_pending = ram_save_pending,
.load_state = ram_load,
- .cleanup = ram_migration_cancel,
+ .cleanup = ram_migration_cleanup,
};
void ram_mig_init(void)
--
2.4.3
- [Qemu-devel] [PULL 0/5] Migration pull request, Juan Quintela, 2015/11/04
- [Qemu-devel] [PULL 1/5] migration: defer migration_end & blk_mig_cleanup, Juan Quintela, 2015/11/04
- [Qemu-devel] [PULL 3/5] migration: rename cancel to cleanup in SaveVMHandles, Juan Quintela, 2015/11/04
- [Qemu-devel] [PULL 2/5] migration: rename qemu_savevm_state_cancel, Juan Quintela, 2015/11/04
- [Qemu-devel] [PULL 4/5] migration: code clean up,
Juan Quintela <=
- [Qemu-devel] [PULL 5/5] migration: fix analyze-migration.py script, Juan Quintela, 2015/11/04
- Re: [Qemu-devel] [PULL 0/5] Migration pull request, Peter Maydell, 2015/11/05