[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-ppc] [PATCH ppc-next v2 45/52] target-ppc: Register all types for
From: |
Andreas Färber |
Subject: |
[Qemu-ppc] [PATCH ppc-next v2 45/52] target-ppc: Register all types for TARGET_PPCEMB |
Date: |
Mon, 18 Feb 2013 10:16:44 +0100 |
Don't attempt to suppress registration of CPU types, since the criteria
is actually a property of the class and should thus become a field.
Since we can't check a field set in a class_init function before
registering the type that leads to execution of that function, guard the
-cpu class lookup instead and suppress exposing these classes in -cpu ?
and in QMP.
In case someone tries to hot-add an incompatible CPU via device_add,
error out in realize.
Signed-off-by: Andreas Färber <address@hidden>
---
target-ppc/translate_init.c | 43 ++++++++++++++++++++++++++++++++++---------
1 Datei geändert, 34 Zeilen hinzugefügt(+), 9 Zeilen entfernt(-)
diff --git a/target-ppc/translate_init.c b/target-ppc/translate_init.c
index 9fb9d1a..521e4ed 100644
--- a/target-ppc/translate_init.c
+++ b/target-ppc/translate_init.c
@@ -7876,14 +7876,6 @@ enum {
/* PowerPC CPU definitions */
#define POWERPC_DEF_PREFIX(pvr, svr, type) \
glue(glue(glue(glue(pvr, _), svr), _), type)
-#if defined(TARGET_PPCEMB)
-#define POWERPC_DEF_CONDITION(type) \
- if (glue(POWERPC_MMU_, type) != POWERPC_MMU_BOOKE) { \
- return; \
- }
-#else
-#define POWERPC_DEF_CONDITION(type)
-#endif
#define POWERPC_DEF_SVR(_name, _pvr, _svr, _type) \
static void \
glue(POWERPC_DEF_PREFIX(_pvr, _svr, _type), _cpu_class_init) \
@@ -7912,7 +7904,6 @@ enum {
static void \
glue(POWERPC_DEF_PREFIX(_pvr, _svr, _type), _cpu_register_types)(void) \
{ \
- POWERPC_DEF_CONDITION(_type) \
type_register_static( \
&glue(POWERPC_DEF_PREFIX(_pvr, _svr, _type), _cpu_type_info)); \
} \
@@ -10040,6 +10031,15 @@ static void ppc_cpu_realize(Object *obj, Error **errp)
}
}
+#if defined(TARGET_PPCEMB)
+ if (pcc->mmu_model != POWERPC_MMU_BOOKE) {
+ error_setg(errp, "CPU does not possess a BookE MMU. "
+ "Please use qemu-system-ppc or qemu-system-ppc64 instead "
+ "or choose another CPU model.");
+ return;
+ }
+#endif
+
create_ppc_opcodes(cpu, &local_err);
if (local_err != NULL) {
error_propagate(errp, local_err);
@@ -10237,6 +10237,12 @@ static gint ppc_cpu_compare_class_pvr(gconstpointer a,
gconstpointer b)
return -1;
}
+#if defined(TARGET_PPCEMB)
+ if (pcc->mmu_model != POWERPC_MMU_BOOKE) {
+ return -1;
+ }
+#endif
+
return pcc->pvr == pvr ? 0 : -1;
}
@@ -10259,8 +10265,14 @@ static gint ppc_cpu_compare_class_name(gconstpointer
a, gconstpointer b)
{
ObjectClass *oc = (ObjectClass *)a;
const char *name = b;
+#if defined(TARGET_PPCEMB)
+ PowerPCCPUClass *pcc = POWERPC_CPU_CLASS(oc);
+#endif
if (strncasecmp(name, object_class_get_name(oc), strlen(name)) == 0 &&
+#if defined(TARGET_PPCEMB)
+ pcc->mmu_model == POWERPC_MMU_BOOKE &&
+#endif
strcmp(object_class_get_name(oc) + strlen(name),
"-" TYPE_POWERPC_CPU) == 0) {
return 0;
@@ -10384,6 +10396,12 @@ static void ppc_cpu_list_entry(gpointer data, gpointer
user_data)
const char *typename = object_class_get_name(oc);
char *name;
+#if defined(TARGET_PPCEMB)
+ if (pcc->mmu_model != POWERPC_MMU_BOOKE) {
+ return;
+ }
+#endif
+
name = g_strndup(typename,
strlen(typename) - strlen("-" TYPE_POWERPC_CPU));
(*s->cpu_fprintf)(s->file, "PowerPC %-16s PVR %08x\n",
@@ -10424,6 +10442,13 @@ static void ppc_cpu_defs_entry(gpointer data, gpointer
user_data)
const char *typename;
CpuDefinitionInfoList *entry;
CpuDefinitionInfo *info;
+#if defined(TARGET_PPCEMB)
+ PowerPCCPUClass *pcc = POWERPC_CPU_CLASS(oc);
+
+ if (pcc->mmu_model != POWERPC_MMU_BOOKE) {
+ return;
+ }
+#endif
typename = object_class_get_name(oc);
info = g_malloc0(sizeof(*info));
--
1.7.10.4
- [Qemu-ppc] [PATCH ppc-next v2 12/52] target-ppc: Extract 74x7[A] aliases, (continued)
- [Qemu-ppc] [PATCH ppc-next v2 12/52] target-ppc: Extract 74x7[A] aliases, Andreas Färber, 2013/02/18
- [Qemu-ppc] [PATCH ppc-next v2 14/52] target-ppc: Extract 74x1 aliases, Andreas Färber, 2013/02/18
- [Qemu-ppc] [PATCH ppc-next v2 20/52] target-ppc: Extract 750 aliases, Andreas Färber, 2013/02/18
- [Qemu-ppc] [PATCH ppc-next v2 24/52] target-ppc: Extract 603r alias, Andreas Färber, 2013/02/18
- [Qemu-ppc] [PATCH ppc-next v2 27/52] target-ppc: Extract e500v1/e500v2 aliases, Andreas Färber, 2013/02/18
- [Qemu-ppc] [PATCH ppc-next v2 21/52] target-ppc: Extract 740/750 aliases, Andreas Färber, 2013/02/18
- [Qemu-ppc] [PATCH ppc-next v2 28/52] target-ppc: Extract MPC83xx aliases, Andreas Färber, 2013/02/18
- [Qemu-ppc] [PATCH ppc-next v2 31/52] target-ppc: Extract MPC82xx alias, Andreas Färber, 2013/02/18
- [Qemu-ppc] [PATCH ppc-next v2 34/52] target-ppc: Extract MPC82xx_HiP{3, 4} aliases, Andreas Färber, 2013/02/18
- [Qemu-ppc] [PATCH ppc-next v2 36/52] target-ppc: Extract MPC5200/MPC5200B aliases, Andreas Färber, 2013/02/18
- [Qemu-ppc] [PATCH ppc-next v2 45/52] target-ppc: Register all types for TARGET_PPCEMB,
Andreas Färber <=
- [Qemu-ppc] [PATCH ppc-next v2 51/52] target-ppc: Fix remaining microcontroller typos among models, Andreas Färber, 2013/02/18
- [Qemu-ppc] [PATCH ppc-next v2 49/52] target-ppc: Update Coding Style for CPU models, Andreas Färber, 2013/02/18
- [Qemu-ppc] [PATCH ppc-next v2 44/52] target-ppc: Set instruction flags on CPU family classes, Andreas Färber, 2013/02/18
[Qemu-ppc] [PATCH ppc-next v2 50/52] target-ppc: Split model definitions out of translate_init.c, Andreas Färber, 2013/02/18
[Qemu-ppc] [PATCH ppc-next v2 33/52] target-ppc: Extract MPC82xx aliases to *_HiP4, Andreas Färber, 2013/02/18
[Qemu-ppc] [PATCH ppc-next v2 25/52] target-ppc: Extract 601/601v aliases, Andreas Färber, 2013/02/18