[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 04/13] savevm: add error parameter to qemu_savevm_st
From: |
Pavel Hrdina |
Subject: |
[Qemu-devel] [PATCH 04/13] savevm: add error parameter to qemu_savevm_state_iterate() |
Date: |
Wed, 9 Jan 2013 16:17:58 +0100 |
Signed-off-by: Pavel Hrdina <address@hidden>
---
include/sysemu/sysemu.h | 2 +-
migration.c | 2 +-
savevm.c | 19 ++++++++++++-------
3 files changed, 14 insertions(+), 9 deletions(-)
diff --git a/include/sysemu/sysemu.h b/include/sysemu/sysemu.h
index e50f990..afc1fe6 100644
--- a/include/sysemu/sysemu.h
+++ b/include/sysemu/sysemu.h
@@ -76,7 +76,7 @@ bool qemu_savevm_state_blocked(Error **errp);
int qemu_savevm_state_begin(QEMUFile *f,
const MigrationParams *params,
Error **errp);
-int qemu_savevm_state_iterate(QEMUFile *f);
+int qemu_savevm_state_iterate(QEMUFile *f, Error **errp);
int qemu_savevm_state_complete(QEMUFile *f);
void qemu_savevm_state_cancel(QEMUFile *f);
uint64_t qemu_savevm_state_pending(QEMUFile *f, uint64_t max_size);
diff --git a/migration.c b/migration.c
index cb38867..0f2d0fd 100644
--- a/migration.c
+++ b/migration.c
@@ -691,7 +691,7 @@ static bool migrate_fd_put_ready(MigrationState *s,
uint64_t max_size)
pending_size = qemu_savevm_state_pending(s->file, max_size);
DPRINTF("pending size %lu max %lu\n", pending_size, max_size);
if (pending_size >= max_size) {
- ret = qemu_savevm_state_iterate(s->file);
+ ret = qemu_savevm_state_iterate(s->file, NULL);
if (ret < 0) {
migrate_fd_error(s);
}
diff --git a/savevm.c b/savevm.c
index e2ecb0b..4da2136 100644
--- a/savevm.c
+++ b/savevm.c
@@ -1619,7 +1619,7 @@ int qemu_savevm_state_begin(QEMUFile *f,
* 0 : We haven't finished, caller have to go again
* 1 : We have finished, we can go to complete phase
*/
-int qemu_savevm_state_iterate(QEMUFile *f)
+int qemu_savevm_state_iterate(QEMUFile *f, Error **errp)
{
SaveStateEntry *se;
int ret = 1;
@@ -1652,13 +1652,18 @@ int qemu_savevm_state_iterate(QEMUFile *f)
break;
}
}
- if (ret != 0) {
- return ret;
+
+ if (ret == 0) {
+ ret = qemu_file_get_error(f);
+ if (ret != 0) {
+ qemu_savevm_state_cancel(f);
+ }
}
- ret = qemu_file_get_error(f);
- if (ret != 0) {
- qemu_savevm_state_cancel(f);
+
+ if (ret < 0) {
+ error_setg_errno(errp, errno, "Failed to iterate vmstate save.");
}
+
return ret;
}
@@ -1766,7 +1771,7 @@ static int qemu_savevm_state(QEMUFile *f)
goto out;
do {
- ret = qemu_savevm_state_iterate(f);
+ ret = qemu_savevm_state_iterate(f, NULL);
if (ret < 0)
goto out;
} while (ret == 0);
--
1.8.1
- [Qemu-devel] [PATCH 00/13] convert savevm to use qapi and introduce qmp command, Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 01/13] block: add error parameter to bdrv_snapshot_create() and related functions, Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 03/13] savevm: add error parameter to qemu_savevm_state_begin(), Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 02/13] block: add error parameter to del_existing_snapshots(), Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 04/13] savevm: add error parameter to qemu_savevm_state_iterate(),
Pavel Hrdina <=
- [Qemu-devel] [PATCH 05/13] savevm: add error parameter to qemu_savevm_state_complete(), Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 07/13] qapi: Convert savevm, Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 10/13] savevm: update return value from qemu_savevm_state_begin, Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 08/13] qemu-img: introduce qemu_img_handle_error, Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 09/13] block: update return value from bdrv_snapshot_create, Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 06/13] savevm: add error parameter to qemu_savevm_state(), Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 11/13] savevm: update return value from qemu_savevm_state_complete, Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 12/13] savevm: update return value from qemu_savevm_state, Pavel Hrdina, 2013/01/09