aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJuan Quintela <quintela@redhat.com>2011-10-05 01:02:52 +0200
committerJuan Quintela <quintela@redhat.com>2011-10-20 13:23:52 +0200
commit624b9cc20922fe7b046cd79c839d1f3a5a5fc27e (patch)
treeee5789c0a6966b697caf48b463f95602ecbc94ea
parent3934638539c763ad23026b70483e60be34078c57 (diff)
migration: rename qemu_file_has_error to qemu_file_get_error
Now the function returned errno, so it is better the new name. Signed-off-by: Juan Quintela <quintela@redhat.com> Reviewed-by: Anthony Liguori <aliguori@us.ibm.com>
-rw-r--r--arch_init.c2
-rw-r--r--block-migration.c8
-rw-r--r--buffered_file.c14
-rw-r--r--hw/hw.h2
-rw-r--r--migration.c2
-rw-r--r--savevm.c13
6 files changed, 21 insertions, 20 deletions
diff --git a/arch_init.c b/arch_init.c
index 941d585e0a..9128be0b02 100644
--- a/arch_init.c
+++ b/arch_init.c
@@ -451,7 +451,7 @@ int ram_load(QEMUFile *f, void *opaque, int version_id)
qemu_get_buffer(f, host, TARGET_PAGE_SIZE);
}
- if (qemu_file_has_error(f)) {
+ if (qemu_file_get_error(f)) {
return -EIO;
}
} while (!(flags & RAM_SAVE_FLAG_EOS));
diff --git a/block-migration.c b/block-migration.c
index 325c905307..56907a664c 100644
--- a/block-migration.c
+++ b/block-migration.c
@@ -580,7 +580,7 @@ static int block_save_live(Monitor *mon, QEMUFile *f, int stage, void *opaque)
flush_blks(f);
- if (qemu_file_has_error(f)) {
+ if (qemu_file_get_error(f)) {
blk_mig_cleanup(mon);
return 0;
}
@@ -608,7 +608,7 @@ static int block_save_live(Monitor *mon, QEMUFile *f, int stage, void *opaque)
flush_blks(f);
- if (qemu_file_has_error(f)) {
+ if (qemu_file_get_error(f)) {
blk_mig_cleanup(mon);
return 0;
}
@@ -625,7 +625,7 @@ static int block_save_live(Monitor *mon, QEMUFile *f, int stage, void *opaque)
/* report completion */
qemu_put_be64(f, (100 << BDRV_SECTOR_BITS) | BLK_MIG_FLAG_PROGRESS);
- if (qemu_file_has_error(f)) {
+ if (qemu_file_get_error(f)) {
return 0;
}
@@ -705,7 +705,7 @@ static int block_load(QEMUFile *f, void *opaque, int version_id)
fprintf(stderr, "Unknown flags\n");
return -EINVAL;
}
- if (qemu_file_has_error(f)) {
+ if (qemu_file_get_error(f)) {
return -EIO;
}
} while (!(flags & BLK_MIG_FLAG_EOS));
diff --git a/buffered_file.c b/buffered_file.c
index 94ca8d11f0..41c659cbd8 100644
--- a/buffered_file.c
+++ b/buffered_file.c
@@ -72,7 +72,7 @@ static void buffered_flush(QEMUFileBuffered *s)
{
size_t offset = 0;
- if (qemu_file_has_error(s->file)) {
+ if (qemu_file_get_error(s->file)) {
DPRINTF("flush when error, bailing\n");
return;
}
@@ -113,7 +113,7 @@ static int buffered_put_buffer(void *opaque, const uint8_t *buf, int64_t pos, in
DPRINTF("putting %d bytes at %" PRId64 "\n", size, pos);
- if (qemu_file_has_error(s->file)) {
+ if (qemu_file_get_error(s->file)) {
DPRINTF("flush when error, bailing\n");
return -EINVAL;
}
@@ -172,7 +172,7 @@ static int buffered_close(void *opaque)
DPRINTF("closing\n");
- while (!qemu_file_has_error(s->file) && s->buffer_size) {
+ while (!qemu_file_get_error(s->file) && s->buffer_size) {
buffered_flush(s);
if (s->freeze_output)
s->wait_for_unfreeze(s->opaque);
@@ -198,7 +198,7 @@ static int buffered_rate_limit(void *opaque)
{
QEMUFileBuffered *s = opaque;
- if (qemu_file_has_error(s->file)) {
+ if (qemu_file_get_error(s->file)) {
return -1;
}
if (s->freeze_output)
@@ -213,9 +213,9 @@ static int buffered_rate_limit(void *opaque)
static int64_t buffered_set_rate_limit(void *opaque, int64_t new_rate)
{
QEMUFileBuffered *s = opaque;
- if (qemu_file_has_error(s->file))
+ if (qemu_file_get_error(s->file)) {
goto out;
-
+ }
if (new_rate > SIZE_MAX) {
new_rate = SIZE_MAX;
}
@@ -237,7 +237,7 @@ static void buffered_rate_tick(void *opaque)
{
QEMUFileBuffered *s = opaque;
- if (qemu_file_has_error(s->file)) {
+ if (qemu_file_get_error(s->file)) {
buffered_close(s);
return;
}
diff --git a/hw/hw.h b/hw/hw.h
index 6cf8cd2056..ed20f5a27a 100644
--- a/hw/hw.h
+++ b/hw/hw.h
@@ -85,7 +85,7 @@ uint64_t qemu_get_be64(QEMUFile *f);
int qemu_file_rate_limit(QEMUFile *f);
int64_t qemu_file_set_rate_limit(QEMUFile *f, int64_t new_rate);
int64_t qemu_file_get_rate_limit(QEMUFile *f);
-int qemu_file_has_error(QEMUFile *f);
+int qemu_file_get_error(QEMUFile *f);
void qemu_file_set_error(QEMUFile *f, int error);
/* Try to send any outstanding data. This function is useful when output is
diff --git a/migration.c b/migration.c
index c1d647c102..c233b522ff 100644
--- a/migration.c
+++ b/migration.c
@@ -313,7 +313,7 @@ void migrate_fd_put_notify(void *opaque)
qemu_set_fd_handler2(s->fd, NULL, NULL, NULL, NULL);
qemu_file_put_notify(s->file);
- if (qemu_file_has_error(s->file)) {
+ if (qemu_file_get_error(s->file)) {
migrate_fd_error(s);
}
}
diff --git a/savevm.c b/savevm.c
index b2f22bdc82..819a6f791e 100644
--- a/savevm.c
+++ b/savevm.c
@@ -425,7 +425,7 @@ QEMUFile *qemu_fopen_ops(void *opaque, QEMUFilePutBufferFunc *put_buffer,
return f;
}
-int qemu_file_has_error(QEMUFile *f)
+int qemu_file_get_error(QEMUFile *f)
{
return f->has_error;
}
@@ -1536,7 +1536,7 @@ int qemu_savevm_state_begin(Monitor *mon, QEMUFile *f, int blk_enable,
se->save_live_state(mon, f, QEMU_VM_SECTION_START, se->opaque);
}
- ret = qemu_file_has_error(f);
+ ret = qemu_file_get_error(f);
if (ret != 0) {
qemu_savevm_state_cancel(mon, f);
}
@@ -1576,7 +1576,7 @@ int qemu_savevm_state_iterate(Monitor *mon, QEMUFile *f)
if (ret != 0) {
return ret;
}
- ret = qemu_file_has_error(f);
+ ret = qemu_file_get_error(f);
if (ret != 0) {
qemu_savevm_state_cancel(mon, f);
}
@@ -1623,7 +1623,7 @@ int qemu_savevm_state_complete(Monitor *mon, QEMUFile *f)
qemu_put_byte(f, QEMU_VM_EOF);
- return qemu_file_has_error(f);
+ return qemu_file_get_error(f);
}
void qemu_savevm_state_cancel(Monitor *mon, QEMUFile *f)
@@ -1664,7 +1664,7 @@ static int qemu_savevm_state(Monitor *mon, QEMUFile *f)
out:
if (ret == 0) {
- ret = qemu_file_has_error(f);
+ ret = qemu_file_get_error(f);
}
if (!ret && saved_vm_running)
@@ -1885,8 +1885,9 @@ out:
g_free(le);
}
- if (qemu_file_has_error(f))
+ if (qemu_file_get_error(f)) {
ret = -EIO;
+ }
return ret;
}