target-unicore32: Detect attempt to instantiate non-CPU type in cpu_init()

Consolidate model checking into a new uc32_cpu_class_by_name().

If the name matches an existing type, also check whether that type is
actually (a sub-type of) TYPE_UNICORE32_CPU.

This fixes, e.g., -cpu puv3_dma asserting.

Cc: qemu-stable@nongnu.org
Signed-off-by: Andreas Färber <afaerber@suse.de>
This commit is contained in:
Andreas Färber 2013-01-23 12:07:17 +01:00
parent bd039ce009
commit d89e12188d
2 changed files with 27 additions and 2 deletions

View File

@ -22,6 +22,21 @@ static inline void set_feature(CPUUniCore32State *env, int feature)
/* CPU models */ /* CPU models */
static ObjectClass *uc32_cpu_class_by_name(const char *cpu_model)
{
ObjectClass *oc;
if (cpu_model == NULL) {
return NULL;
}
oc = object_class_by_name(cpu_model);
if (oc != NULL && !object_class_dynamic_cast(oc, TYPE_UNICORE32_CPU)) {
oc = NULL;
}
return oc;
}
typedef struct UniCore32CPUInfo { typedef struct UniCore32CPUInfo {
const char *name; const char *name;
void (*instance_init)(Object *obj); void (*instance_init)(Object *obj);
@ -80,6 +95,13 @@ static void uc32_cpu_initfn(Object *obj)
tlb_flush(env, 1); tlb_flush(env, 1);
} }
static void uc32_cpu_class_init(ObjectClass *oc, void *data)
{
CPUClass *cc = CPU_CLASS(oc);
cc->class_by_name = uc32_cpu_class_by_name;
}
static void uc32_register_cpu_type(const UniCore32CPUInfo *info) static void uc32_register_cpu_type(const UniCore32CPUInfo *info)
{ {
TypeInfo type_info = { TypeInfo type_info = {
@ -98,6 +120,7 @@ static const TypeInfo uc32_cpu_type_info = {
.instance_init = uc32_cpu_initfn, .instance_init = uc32_cpu_initfn,
.abstract = true, .abstract = true,
.class_size = sizeof(UniCore32CPUClass), .class_size = sizeof(UniCore32CPUClass),
.class_init = uc32_cpu_class_init,
}; };
static void uc32_cpu_register_types(void) static void uc32_cpu_register_types(void)

View File

@ -29,12 +29,14 @@ CPUUniCore32State *uc32_cpu_init(const char *cpu_model)
{ {
UniCore32CPU *cpu; UniCore32CPU *cpu;
CPUUniCore32State *env; CPUUniCore32State *env;
ObjectClass *oc;
static int inited = 1; static int inited = 1;
if (object_class_by_name(cpu_model) == NULL) { oc = cpu_class_by_name(TYPE_UNICORE32_CPU, cpu_model);
if (oc == NULL) {
return NULL; return NULL;
} }
cpu = UNICORE32_CPU(object_new(cpu_model)); cpu = UNICORE32_CPU(object_new(object_class_get_name(oc)));
env = &cpu->env; env = &cpu->env;
if (inited) { if (inited) {