diff options
| author | Markus Armbruster <armbru@redhat.com> | 2015-12-17 17:35:09 +0100 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2016-01-13 11:58:58 +0100 |
| commit | c525436e69bb7e74ca96982a40b8ead037186049 (patch) | |
| tree | deba652f822c68c7eff9dd92af502a1848a1e8e8 /hw/m68k | |
| parent | 6231a6da9f40c3a33589402c9c57557e3a4f05ad (diff) | |
| download | qemu-c525436e69bb7e74ca96982a40b8ead037186049.zip qemu-c525436e69bb7e74ca96982a40b8ead037186049.tar.gz qemu-c525436e69bb7e74ca96982a40b8ead037186049.tar.bz2 | |
hw: Don't use hw_error() for machine initialization errors
Printing CPU registers is not helpful during machine initialization.
Moreover, these are straightforward configuration or "can get
resources" errors, so dumping core isn't appropriate either. Replace
hw_error() by error_report(); exit(1). Matches how we report these
errors in other machine initializations.
Cc: Richard Henderson <rth@twiddle.net>
Cc: qemu-arm@nongnu.org
Cc: qemu-ppc@nongnu.org
Cc: Guan Xuetao <gxt@mprc.pku.edu.cn>
Signed-off-by: Markus Armbruster <armbru@pond.sub.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-Id: <1450370121-5768-2-git-send-email-armbru@redhat.com>
Reviewed-by: Richard Henderson <rth@twiddle.net>
Diffstat (limited to 'hw/m68k')
| -rw-r--r-- | hw/m68k/an5206.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/hw/m68k/an5206.c b/hw/m68k/an5206.c index c1dea17..8d9ccaa 100644 --- a/hw/m68k/an5206.c +++ b/hw/m68k/an5206.c @@ -12,6 +12,7 @@ #include "hw/loader.h" #include "elf.h" #include "exec/address-spaces.h" +#include "qemu/error-report.h" #include "sysemu/qtest.h" #define KERNEL_LOAD_ADDR 0x10000 @@ -39,7 +40,8 @@ static void an5206_init(MachineState *machine) } cpu = cpu_m68k_init(cpu_model); if (!cpu) { - hw_error("Unable to find m68k CPU definition\n"); + error_report("Unable to find m68k CPU definition"); + exit(1); } env = &cpu->env; |