[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-ppc] [PATCHv3 9/9] pseries: Clean up error reporting in htab migra
From: |
David Gibson |
Subject: |
[Qemu-ppc] [PATCHv3 9/9] pseries: Clean up error reporting in htab migration functions |
Date: |
Mon, 18 Jan 2016 15:24:43 +1100 |
The functions for migrating the hash page table on pseries machine type
(htab_save_setup() and htab_load()) can report some errors with an
explicit fprintf() before returning an appropriate error code. Change these
to use error_report() instead.
Signed-off-by: David Gibson <address@hidden>
Reviewed-by: Thomas Huth <address@hidden>
---
hw/ppc/spapr.c | 17 +++++++++--------
1 file changed, 9 insertions(+), 8 deletions(-)
diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
index 58f26cd..ba0bfdf 100644
--- a/hw/ppc/spapr.c
+++ b/hw/ppc/spapr.c
@@ -1317,8 +1317,9 @@ static int htab_save_setup(QEMUFile *f, void *opaque)
spapr->htab_fd = kvmppc_get_htab_fd(false);
spapr->htab_fd_stale = false;
if (spapr->htab_fd < 0) {
- fprintf(stderr, "Unable to open fd for reading hash table from
KVM: %s\n",
- strerror(errno));
+ error_report(
+ "Unable to open fd for reading hash table from KVM: %s",
+ strerror(errno));
return -1;
}
}
@@ -1534,7 +1535,7 @@ static int htab_load(QEMUFile *f, void *opaque, int
version_id)
int fd = -1;
if (version_id < 1 || version_id > 1) {
- fprintf(stderr, "htab_load() bad version\n");
+ error_report("htab_load() bad version");
return -EINVAL;
}
@@ -1555,8 +1556,8 @@ static int htab_load(QEMUFile *f, void *opaque, int
version_id)
fd = kvmppc_get_htab_fd(true);
if (fd < 0) {
- fprintf(stderr, "Unable to open fd to restore KVM hash table:
%s\n",
- strerror(errno));
+ error_report("Unable to open fd to restore KVM hash table: %s",
+ strerror(errno));
}
}
@@ -1576,9 +1577,9 @@ static int htab_load(QEMUFile *f, void *opaque, int
version_id)
if ((index + n_valid + n_invalid) >
(HTAB_SIZE(spapr) / HASH_PTE_SIZE_64)) {
/* Bad index in stream */
- fprintf(stderr, "htab_load() bad index %d (%hd+%hd entries) "
- "in htab stream (htab_shift=%d)\n", index, n_valid,
n_invalid,
- spapr->htab_shift);
+ error_report(
+ "htab_load() bad index %d (%hd+%hd entries) in htab stream
(htab_shift=%d)",
+ index, n_valid, n_invalid, spapr->htab_shift);
return -EINVAL;
}
--
2.5.0
- [Qemu-ppc] [PATCHv3 0/9] Cleanups to error reporting on ppc and spapr, David Gibson, 2016/01/17
- [Qemu-ppc] [PATCHv3 9/9] pseries: Clean up error reporting in htab migration functions,
David Gibson <=
- [Qemu-ppc] [PATCHv3 4/9] pseries: Clean up error handling in spapr_validate_node_memory(), David Gibson, 2016/01/17
- [Qemu-ppc] [PATCHv3 8/9] pseries: Clean up error reporting in ppc_spapr_init(), David Gibson, 2016/01/17
- [Qemu-ppc] [PATCHv3 5/9] pseries: Cleanup error handling in spapr_vga_init(), David Gibson, 2016/01/17
- [Qemu-ppc] [PATCHv3 2/9] pseries: Cleanup error handling of spapr_cpu_init(), David Gibson, 2016/01/17
- [Qemu-ppc] [PATCHv3 1/9] ppc: Cleanup error handling in ppc_set_compat(), David Gibson, 2016/01/17