9ae3a8
From a52fe59a941e8cfa7b94ecc743ffcfcf0e1bcda7 Mon Sep 17 00:00:00 2001
9ae3a8
From: Laszlo Ersek <lersek@redhat.com>
9ae3a8
Date: Wed, 28 Aug 2013 15:50:45 +0200
9ae3a8
Subject: [PATCH 28/28] Convert stderr message calling error_get_pretty() to error_report()
9ae3a8
9ae3a8
RH-Author: Laszlo Ersek <lersek@redhat.com>
9ae3a8
Message-id: <1377705045-1054-3-git-send-email-lersek@redhat.com>
9ae3a8
Patchwork-id: 53887
9ae3a8
O-Subject: [RHEL-7 qemu-kvm PATCH v3 2/2] Convert stderr message calling error_get_pretty() to error_report()
9ae3a8
Bugzilla: 906937
9ae3a8
RH-Acked-by: Michal Novotny <minovotn@redhat.com>
9ae3a8
RH-Acked-by: Markus Armbruster <armbru@redhat.com>
9ae3a8
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
9ae3a8
9ae3a8
Convert stderr messages calling error_get_pretty()
9ae3a8
to error_report().
9ae3a8
9ae3a8
Timestamp is prepended by -msg timstamp option with it.
9ae3a8
9ae3a8
Per Markus's comment below, A conversion from fprintf() to
9ae3a8
error_report() is always an improvement, regardless of
9ae3a8
error_get_pretty().
9ae3a8
9ae3a8
http://marc.info/?l=qemu-devel&m=137513283408601&w=2
9ae3a8
9ae3a8
But, it is not reasonable to convert them at one time
9ae3a8
because fprintf() is used everwhere in qemu.
9ae3a8
9ae3a8
So, it should be done step by step with avoiding regression.
9ae3a8
9ae3a8
Signed-off-by: Seiji Aguchi <seiji.aguchi@hds.com>
9ae3a8
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
9ae3a8
Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
9ae3a8
(cherry picked from commit 4a44d85e28bd282f53ccf0fa933dd71b8744a229)
9ae3a8
9ae3a8
Conflicts:
9ae3a8
9ae3a8
	hw/char/serial.c
9ae3a8
9ae3a8
RHEL-7 notes:
9ae3a8
9ae3a8
- I dropped the "hw/char/serial.c" hunks because they depend on
9ae3a8
  upstream commit db895a1e ("isa: Use realizefn for ISADevice"):
9ae3a8
9ae3a8
  -    serial_init_core(s);
9ae3a8
  +    serial_realize_core(s, &err;;
9ae3a8
9ae3a8
  Upstream commit db895a1e was first tagged in v1.6.0-rc0 and we don't
9ae3a8
  have it.
9ae3a8
9ae3a8
  Instead, I replaced the fprintf(stderr, ...) call in serial_init_core()
9ae3a8
  with a matching error_report() call, following Stefan's review of v2
9ae3a8
  2/2.
9ae3a8
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
9ae3a8
---
9ae3a8
 arch_init.c                 |    4 ++--
9ae3a8
 hw/char/serial.c            |    3 ++-
9ae3a8
 hw/i386/pc.c                |    6 +++---
9ae3a8
 qemu-char.c                 |    2 +-
9ae3a8
 target-i386/cpu.c           |    2 +-
9ae3a8
 target-ppc/translate_init.c |    3 ++-
9ae3a8
 vl.c                        |    9 +++++----
9ae3a8
 7 files changed, 16 insertions(+), 13 deletions(-)
9ae3a8
9ae3a8
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
9ae3a8
---
9ae3a8
 arch_init.c                 |    4 ++--
9ae3a8
 hw/char/serial.c            |    3 ++-
9ae3a8
 hw/i386/pc.c                |    6 +++---
9ae3a8
 qemu-char.c                 |    2 +-
9ae3a8
 target-i386/cpu.c           |    2 +-
9ae3a8
 target-ppc/translate_init.c |    3 ++-
9ae3a8
 vl.c                        |    9 +++++----
9ae3a8
 7 files changed, 16 insertions(+), 13 deletions(-)
9ae3a8
9ae3a8
diff --git a/arch_init.c b/arch_init.c
9ae3a8
index 30edd5f..cd2bb5a 100644
9ae3a8
--- a/arch_init.c
9ae3a8
+++ b/arch_init.c
9ae3a8
@@ -1069,8 +1069,8 @@ void do_acpitable_option(const QemuOpts *opts)
9ae3a8
 
9ae3a8
     acpi_table_add(opts, &err;;
9ae3a8
     if (err) {
9ae3a8
-        fprintf(stderr, "Wrong acpi table provided: %s\n",
9ae3a8
-                error_get_pretty(err));
9ae3a8
+        error_report("Wrong acpi table provided: %s",
9ae3a8
+                     error_get_pretty(err));
9ae3a8
         error_free(err);
9ae3a8
         exit(1);
9ae3a8
     }
9ae3a8
diff --git a/hw/char/serial.c b/hw/char/serial.c
9ae3a8
index 66b6348..0c472e2 100644
9ae3a8
--- a/hw/char/serial.c
9ae3a8
+++ b/hw/char/serial.c
9ae3a8
@@ -27,6 +27,7 @@
9ae3a8
 #include "sysemu/char.h"
9ae3a8
 #include "qemu/timer.h"
9ae3a8
 #include "exec/address-spaces.h"
9ae3a8
+#include "qemu/error-report.h"
9ae3a8
 
9ae3a8
 //#define DEBUG_SERIAL
9ae3a8
 
9ae3a8
@@ -673,7 +674,7 @@ static void serial_reset(void *opaque)
9ae3a8
 void serial_init_core(SerialState *s)
9ae3a8
 {
9ae3a8
     if (!s->chr) {
9ae3a8
-        fprintf(stderr, "Can't create serial device, empty char device\n");
9ae3a8
+        error_report("Can't create serial device, empty char device");
9ae3a8
 	exit(1);
9ae3a8
     }
9ae3a8
 
9ae3a8
diff --git a/hw/i386/pc.c b/hw/i386/pc.c
9ae3a8
index a7ecfb0..71b7a26 100644
9ae3a8
--- a/hw/i386/pc.c
9ae3a8
+++ b/hw/i386/pc.c
9ae3a8
@@ -968,7 +968,7 @@ void pc_cpus_init(const char *cpu_model, DeviceState *icc_bridge)
9ae3a8
         cpu = pc_new_cpu(cpu_model, x86_cpu_apic_id_from_index(i),
9ae3a8
                          icc_bridge, &error);
9ae3a8
         if (error) {
9ae3a8
-            fprintf(stderr, "%s\n", error_get_pretty(error));
9ae3a8
+            error_report("%s", error_get_pretty(error));
9ae3a8
             error_free(error);
9ae3a8
             exit(1);
9ae3a8
         }
9ae3a8
@@ -1007,8 +1007,8 @@ void pc_acpi_init(const char *default_dsdt)
9ae3a8
 
9ae3a8
         acpi_table_add(opts, &err;;
9ae3a8
         if (err) {
9ae3a8
-            fprintf(stderr, "WARNING: failed to load %s: %s\n", filename,
9ae3a8
-                    error_get_pretty(err));
9ae3a8
+            error_report("WARNING: failed to load %s: %s", filename,
9ae3a8
+                         error_get_pretty(err));
9ae3a8
             error_free(err);
9ae3a8
         }
9ae3a8
         g_free(arg);
9ae3a8
diff --git a/qemu-char.c b/qemu-char.c
9ae3a8
index ccc9912..566267a 100644
9ae3a8
--- a/qemu-char.c
9ae3a8
+++ b/qemu-char.c
9ae3a8
@@ -3334,7 +3334,7 @@ CharDriverState *qemu_chr_new(const char *label, const char *filename, void (*in
9ae3a8
 
9ae3a8
     chr = qemu_chr_new_from_opts(opts, init, &err;;
9ae3a8
     if (error_is_set(&err)) {
9ae3a8
-        fprintf(stderr, "%s\n", error_get_pretty(err));
9ae3a8
+        error_report("%s", error_get_pretty(err));
9ae3a8
         error_free(err);
9ae3a8
     }
9ae3a8
     if (chr && qemu_opt_get_bool(opts, "mux", 0)) {
9ae3a8
diff --git a/target-i386/cpu.c b/target-i386/cpu.c
9ae3a8
index 0ad8259..355375b 100644
9ae3a8
--- a/target-i386/cpu.c
9ae3a8
+++ b/target-i386/cpu.c
9ae3a8
@@ -1847,7 +1847,7 @@ X86CPU *cpu_x86_init(const char *cpu_model)
9ae3a8
 
9ae3a8
 out:
9ae3a8
     if (error) {
9ae3a8
-        fprintf(stderr, "%s\n", error_get_pretty(error));
9ae3a8
+        error_report("%s", error_get_pretty(error));
9ae3a8
         error_free(error);
9ae3a8
         if (cpu != NULL) {
9ae3a8
             object_unref(OBJECT(cpu));
9ae3a8
diff --git a/target-ppc/translate_init.c b/target-ppc/translate_init.c
9ae3a8
index 2e2bffa..5a12c0e 100644
9ae3a8
--- a/target-ppc/translate_init.c
9ae3a8
+++ b/target-ppc/translate_init.c
9ae3a8
@@ -27,6 +27,7 @@
9ae3a8
 #include "cpu-models.h"
9ae3a8
 #include "mmu-hash32.h"
9ae3a8
 #include "mmu-hash64.h"
9ae3a8
+#include "qemu/error-report.h"
9ae3a8
 
9ae3a8
 //#define PPC_DUMP_CPU
9ae3a8
 //#define PPC_DEBUG_SPR
9ae3a8
@@ -8027,7 +8028,7 @@ PowerPCCPU *cpu_ppc_init(const char *cpu_model)
9ae3a8
 
9ae3a8
     object_property_set_bool(OBJECT(cpu), true, "realized", &err;;
9ae3a8
     if (err != NULL) {
9ae3a8
-        fprintf(stderr, "%s\n", error_get_pretty(err));
9ae3a8
+        error_report("%s", error_get_pretty(err));
9ae3a8
         error_free(err);
9ae3a8
         object_unref(OBJECT(cpu));
9ae3a8
         return NULL;
9ae3a8
diff --git a/vl.c b/vl.c
9ae3a8
index 0ce554d..51c04e7 100644
9ae3a8
--- a/vl.c
9ae3a8
+++ b/vl.c
9ae3a8
@@ -2363,7 +2363,7 @@ static int chardev_init_func(QemuOpts *opts, void *opaque)
9ae3a8
 
9ae3a8
     qemu_chr_new_from_opts(opts, NULL, &local_err);
9ae3a8
     if (error_is_set(&local_err)) {
9ae3a8
-        fprintf(stderr, "%s\n", error_get_pretty(local_err));
9ae3a8
+        error_report("%s", error_get_pretty(local_err));
9ae3a8
         error_free(local_err);
9ae3a8
         return -1;
9ae3a8
     }
9ae3a8
@@ -4393,8 +4393,8 @@ int main(int argc, char **argv, char **envp)
9ae3a8
         vnc_display_init(ds);
9ae3a8
         vnc_display_open(ds, vnc_display, &local_err);
9ae3a8
         if (local_err != NULL) {
9ae3a8
-            fprintf(stderr, "Failed to start VNC server on `%s': %s\n",
9ae3a8
-                    vnc_display, error_get_pretty(local_err));
9ae3a8
+            error_report("Failed to start VNC server on `%s': %s",
9ae3a8
+                         vnc_display, error_get_pretty(local_err));
9ae3a8
             error_free(local_err);
9ae3a8
             exit(1);
9ae3a8
         }
9ae3a8
@@ -4437,7 +4437,8 @@ int main(int argc, char **argv, char **envp)
9ae3a8
         Error *local_err = NULL;
9ae3a8
         qemu_start_incoming_migration(incoming, &local_err);
9ae3a8
         if (local_err) {
9ae3a8
-            fprintf(stderr, "-incoming %s: %s\n", incoming, error_get_pretty(local_err));
9ae3a8
+            error_report("-incoming %s: %s", incoming,
9ae3a8
+                         error_get_pretty(local_err));
9ae3a8
             error_free(local_err);
9ae3a8
             exit(1);
9ae3a8
         }
9ae3a8
-- 
9ae3a8
1.7.1
9ae3a8