[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 14/36] migration: use qemu_file_get_error() return v
From: |
Juan Quintela |
Subject: |
[Qemu-devel] [PATCH 14/36] migration: use qemu_file_get_error() return value when possible |
Date: |
Tue, 11 Oct 2011 12:00:34 +0200 |
Signed-off-by: Juan Quintela <address@hidden>
---
arch_init.c | 6 ++++--
block-migration.c | 7 ++++---
buffered_file.c | 13 ++++++++-----
savevm.c | 4 ++--
4 files changed, 18 insertions(+), 12 deletions(-)
diff --git a/arch_init.c b/arch_init.c
index 9128be0..98daaf3 100644
--- a/arch_init.c
+++ b/arch_init.c
@@ -371,6 +371,7 @@ int ram_load(QEMUFile *f, void *opaque, int version_id)
{
ram_addr_t addr;
int flags;
+ int error;
if (version_id < 3 || version_id > 4) {
return -EINVAL;
@@ -451,8 +452,9 @@ int ram_load(QEMUFile *f, void *opaque, int version_id)
qemu_get_buffer(f, host, TARGET_PAGE_SIZE);
}
- if (qemu_file_get_error(f)) {
- return -EIO;
+ error = qemu_file_get_error(f);
+ if (error) {
+ return error;
}
} while (!(flags & RAM_SAVE_FLAG_EOS));
diff --git a/block-migration.c b/block-migration.c
index 8308753..e36f2e3 100644
--- a/block-migration.c
+++ b/block-migration.c
@@ -646,6 +646,7 @@ static int block_load(QEMUFile *f, void *opaque, int
version_id)
uint8_t *buf;
int64_t total_sectors = 0;
int nr_sectors;
+ int ret;
do {
addr = qemu_get_be64(f);
@@ -654,7 +655,6 @@ static int block_load(QEMUFile *f, void *opaque, int
version_id)
addr >>= BDRV_SECTOR_BITS;
if (flags & BLK_MIG_FLAG_DEVICE_BLOCK) {
- int ret;
/* get device name */
len = qemu_get_byte(f);
qemu_get_buffer(f, (uint8_t *)device_name, len);
@@ -704,8 +704,9 @@ static int block_load(QEMUFile *f, void *opaque, int
version_id)
fprintf(stderr, "Unknown flags\n");
return -EINVAL;
}
- if (qemu_file_get_error(f)) {
- return -EIO;
+ ret = qemu_file_get_error(f);
+ if (ret != 0) {
+ return ret;
}
} while (!(flags & BLK_MIG_FLAG_EOS));
diff --git a/buffered_file.c b/buffered_file.c
index 82f4001..42bbbd3 100644
--- a/buffered_file.c
+++ b/buffered_file.c
@@ -71,9 +71,11 @@ static void buffered_append(QEMUFileBuffered *s,
static void buffered_flush(QEMUFileBuffered *s)
{
size_t offset = 0;
+ int error;
- if (qemu_file_get_error(s->file)) {
- DPRINTF("flush when error, bailing\n");
+ error = qemu_file_get_error(s->file);
+ if (error != 0) {
+ DPRINTF("flush when error, bailing: %s\n", strerror(-error));
return;
}
@@ -108,13 +110,14 @@ static void buffered_flush(QEMUFileBuffered *s)
static int buffered_put_buffer(void *opaque, const uint8_t *buf, int64_t pos,
int size)
{
QEMUFileBuffered *s = opaque;
- int offset = 0;
+ int offset = 0, error;
ssize_t ret;
DPRINTF("putting %d bytes at %" PRId64 "\n", size, pos);
- if (qemu_file_get_error(s->file)) {
- DPRINTF("flush when error, bailing\n");
+ error = qemu_file_get_error(s->file);
+ if (error) {
+ DPRINTF("flush when error, bailing: %s\n", strerror(-error));
return -EINVAL;
}
diff --git a/savevm.c b/savevm.c
index 69a2ccd..3042ae5 100644
--- a/savevm.c
+++ b/savevm.c
@@ -1835,8 +1835,8 @@ out:
g_free(le);
}
- if (qemu_file_get_error(f)) {
- ret = -EIO;
+ if (ret == 0) {
+ ret = qemu_file_get_error(f);
}
return ret;
--
1.7.6.4
- [Qemu-devel] [PATCH 07/36] buffered_file: Use right "opaque", (continued)
- [Qemu-devel] [PATCH 07/36] buffered_file: Use right "opaque", Juan Quintela, 2011/10/11
- [Qemu-devel] [PATCH 08/36] buffered_file: reuse QEMUFile has_error field, Juan Quintela, 2011/10/11
- [Qemu-devel] [PATCH 11/36] migration: change has_error to contain errno values, Juan Quintela, 2011/10/11
- [Qemu-devel] [PATCH 12/36] migration: rename qemu_file_has_error to qemu_file_get_error, Juan Quintela, 2011/10/11
- [Qemu-devel] [PATCH 13/36] savevm: Rename has_error to last_error field, Juan Quintela, 2011/10/11
- [Qemu-devel] [PATCH 14/36] migration: use qemu_file_get_error() return value when possible,
Juan Quintela <=
- [Qemu-devel] [PATCH 15/36] migration: Make *start_outgoing_migration return FdMigrationState, Juan Quintela, 2011/10/11
- [Qemu-devel] [PATCH 20/36] migration: Make all posible migration functions static, Juan Quintela, 2011/10/11
- [Qemu-devel] [PATCH 21/36] migration: move migrate_new to do_migrate, Juan Quintela, 2011/10/11
- [Qemu-devel] [PATCH 19/36] migration: Refactor MigrationState creation, Juan Quintela, 2011/10/11
- [Qemu-devel] [PATCH 22/36] migration: Introduce MIG_STATE_SETUP, Juan Quintela, 2011/10/11