diff options
author | Juan Quintela <quintela@redhat.com> | 2011-10-19 15:22:18 +0200 |
---|---|---|
committer | Juan Quintela <quintela@redhat.com> | 2011-10-20 13:23:52 +0200 |
commit | 2975725f6b3d634dbe924ea9d9f4d86b8a5b217d (patch) | |
tree | f6a28a67ab95c169cedd0aa237a45cf4e438c28a /arch_init.c | |
parent | 42802d47dd09c6e70763676bb2ba59136427ec6a (diff) | |
download | qemu-2975725f6b3d634dbe924ea9d9f4d86b8a5b217d.zip qemu-2975725f6b3d634dbe924ea9d9f4d86b8a5b217d.tar.gz qemu-2975725f6b3d634dbe924ea9d9f4d86b8a5b217d.tar.bz2 |
migration: make *save_live return errors
Make *save_live() return negative values when there is one error, and
updates all callers to check for the error.
Signed-off-by: Juan Quintela <quintela@redhat.com>
Diffstat (limited to 'arch_init.c')
-rw-r--r-- | arch_init.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/arch_init.c b/arch_init.c index 98daaf3..a411fdf 100644 --- a/arch_init.c +++ b/arch_init.c @@ -256,6 +256,7 @@ int ram_save_live(Monitor *mon, QEMUFile *f, int stage, void *opaque) uint64_t bytes_transferred_last; double bwidth = 0; uint64_t expected_time = 0; + int ret; if (stage < 0) { cpu_physical_memory_set_dirty_tracking(0); @@ -264,7 +265,7 @@ int ram_save_live(Monitor *mon, QEMUFile *f, int stage, void *opaque) if (cpu_physical_sync_dirty_bitmap(0, TARGET_PHYS_ADDR_MAX) != 0) { qemu_file_set_error(f, -EINVAL); - return 0; + return -EINVAL; } if (stage == 1) { @@ -300,7 +301,7 @@ int ram_save_live(Monitor *mon, QEMUFile *f, int stage, void *opaque) bytes_transferred_last = bytes_transferred; bwidth = qemu_get_clock_ns(rt_clock); - while (!qemu_file_rate_limit(f)) { + while ((ret = qemu_file_rate_limit(f)) == 0) { int bytes_sent; bytes_sent = ram_save_block(f); @@ -310,6 +311,10 @@ int ram_save_live(Monitor *mon, QEMUFile *f, int stage, void *opaque) } } + if (ret < 0) { + return ret; + } + bwidth = qemu_get_clock_ns(rt_clock) - bwidth; bwidth = (bytes_transferred - bytes_transferred_last) / bwidth; |