diff options
author | Igor Mammedov | 2017-09-13 18:04:54 +0200 |
---|---|---|
committer | Eduardo Habkost | 2017-09-19 14:09:32 +0200 |
commit | 4482e05cbbb7e50e476f6a9500cf0b38913bd939 (patch) | |
tree | c2135cfd57c8f7fe0c50ddf19e43e98479383c08 /hw/m68k | |
parent | qom: cpus: split cpu_generic_init() on feature parsing and cpu creation parts (diff) | |
download | qemu-4482e05cbbb7e50e476f6a9500cf0b38913bd939.tar.gz qemu-4482e05cbbb7e50e476f6a9500cf0b38913bd939.tar.xz qemu-4482e05cbbb7e50e476f6a9500cf0b38913bd939.zip |
cpu: make cpu_generic_init() abort QEMU on error
Almost every user of cpu_generic_init() checks for
returned NULL and then reports failure in a custom way
and aborts process.
Some users assume that call can't fail and don't check
for failure, though they should have checked for it.
In either cases cpu_generic_init() failure is fatal,
so instead of checking for failure and reporting
it various ways, make cpu_generic_init() report
errors in consistent way and terminate QEMU on failure.
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <1505318697-77161-3-git-send-email-imammedo@redhat.com>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Diffstat (limited to 'hw/m68k')
-rw-r--r-- | hw/m68k/an5206.c | 4 | ||||
-rw-r--r-- | hw/m68k/mcf5208.c | 4 |
2 files changed, 0 insertions, 8 deletions
diff --git a/hw/m68k/an5206.c b/hw/m68k/an5206.c index 7b9b15d6c4..9002c460e5 100644 --- a/hw/m68k/an5206.c +++ b/hw/m68k/an5206.c @@ -43,10 +43,6 @@ static void an5206_init(MachineState *machine) cpu_model = "m5206"; } cpu = M68K_CPU(cpu_generic_init(TYPE_M68K_CPU, cpu_model)); - if (!cpu) { - error_report("Unable to find m68k CPU definition"); - exit(1); - } env = &cpu->env; /* Initialize CPU registers. */ diff --git a/hw/m68k/mcf5208.c b/hw/m68k/mcf5208.c index 1a0f18073a..b9dde75106 100644 --- a/hw/m68k/mcf5208.c +++ b/hw/m68k/mcf5208.c @@ -233,10 +233,6 @@ static void mcf5208evb_init(MachineState *machine) cpu_model = "m5208"; } cpu = M68K_CPU(cpu_generic_init(TYPE_M68K_CPU, cpu_model)); - if (!cpu) { - fprintf(stderr, "Unable to find m68k CPU definition\n"); - exit(1); - } env = &cpu->env; /* Initialize CPU registers. */ |