[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-ppc] [PATCH for-2.11 4/6] ppc: replace inter-function cyclic depen
From: |
Igor Mammedov |
Subject: |
[Qemu-ppc] [PATCH for-2.11 4/6] ppc: replace inter-function cyclic dependency/recurssion with 2 simple lookups |
Date: |
Thu, 24 Aug 2017 10:21:49 +0200 |
previous patches cleaned up cpu model/alias naming which
allows to simplify cpu model/alias to cpu type lookup a bit
byt removing recurssion and dependency of ppc_cpu_class_by_name() /
ppc_cpu_class_by_alias() on each other.
Besides of simplifying code it reduces it by ~15LOC.
Signed-off-by: Igor Mammedov <address@hidden>
---
target/ppc/translate_init.c | 43 +++++++++++++------------------------------
1 file changed, 13 insertions(+), 30 deletions(-)
diff --git a/target/ppc/translate_init.c b/target/ppc/translate_init.c
index 0325226..f1a559d 100644
--- a/target/ppc/translate_init.c
+++ b/target/ppc/translate_init.c
@@ -10176,22 +10176,6 @@ PowerPCCPUClass *ppc_cpu_class_by_pvr_mask(uint32_t
pvr)
return pcc;
}
-static gint ppc_cpu_compare_class_name(gconstpointer a, gconstpointer b)
-{
- ObjectClass *oc = (ObjectClass *)a;
- const char *name = b;
- PowerPCCPUClass *pcc = POWERPC_CPU_CLASS(oc);
-
- if (strncasecmp(name, object_class_get_name(oc), strlen(name)) == 0 &&
- ppc_cpu_is_valid(pcc) &&
- strcmp(object_class_get_name(oc) + strlen(name),
- POWERPC_CPU_TYPE_SUFFIX) == 0) {
- return 0;
- }
- return -1;
-}
-
-
static ObjectClass *ppc_cpu_class_by_name(const char *name);
static ObjectClass *ppc_cpu_class_by_alias(PowerPCCPUAlias *alias)
@@ -10216,8 +10200,8 @@ static ObjectClass
*ppc_cpu_class_by_alias(PowerPCCPUAlias *alias)
static ObjectClass *ppc_cpu_class_by_name(const char *name)
{
- GSList *list, *item;
- ObjectClass *ret = NULL;
+ char *cpu_model, *typename;
+ ObjectClass *oc;
const char *p;
int i, len;
@@ -10238,21 +10222,20 @@ static ObjectClass *ppc_cpu_class_by_name(const char
*name)
}
}
- list = object_class_get_list(TYPE_POWERPC_CPU, false);
- item = g_slist_find_custom(list, name, ppc_cpu_compare_class_name);
- if (item != NULL) {
- ret = OBJECT_CLASS(item->data);
+ cpu_model = g_ascii_strup(name, -1);
+ p = ppc_cpu_lookup_alias(cpu_model);
+ if (p) {
+ g_free(cpu_model);
+ cpu_model = g_strdup(p);
}
- g_slist_free(list);
- if (ret) {
- return ret;
- }
+ typename = g_strdup_printf("%s" POWERPC_CPU_TYPE_SUFFIX, cpu_model);
+ oc = object_class_by_name(typename);
+ g_free(typename);
+ g_free(cpu_model);
- for (i = 0; ppc_cpu_aliases[i].alias != NULL; i++) {
- if (strcasecmp(ppc_cpu_aliases[i].alias, name) == 0) {
- return ppc_cpu_class_by_alias(&ppc_cpu_aliases[i]);
- }
+ if (oc && ppc_cpu_is_valid(POWERPC_CPU_CLASS(oc))) {
+ return oc;
}
return NULL;
--
2.7.4
- Re: [Qemu-ppc] [Qemu-devel] [PATCH for-2.11 1/6] ppc: use macros to make cpu type name from string literal, (continued)
- [Qemu-ppc] [PATCH for-2.11 3/6] ppc: make cpu alias point only to real cpu models, Igor Mammedov, 2017/08/24
- [Qemu-ppc] [PATCH for-2.11 5/6] ppc: simplify cpu model lookup by PVR, Igor Mammedov, 2017/08/24
- Re: [Qemu-ppc] [PATCH for-2.11 5/6] ppc: simplify cpu model lookup by PVR, David Gibson, 2017/08/25
- Re: [Qemu-ppc] [Qemu-devel] [PATCH for-2.11 5/6] ppc: simplify cpu model lookup by PVR, Igor Mammedov, 2017/08/25
- Re: [Qemu-ppc] [Qemu-devel] [PATCH for-2.11 5/6] ppc: simplify cpu model lookup by PVR, David Gibson, 2017/08/25
- Re: [Qemu-ppc] [Qemu-devel] [PATCH for-2.11 5/6] ppc: simplify cpu model lookup by PVR, Igor Mammedov, 2017/08/25
- Re: [Qemu-ppc] [Qemu-devel] [PATCH for-2.11 5/6] ppc: simplify cpu model lookup by PVR, Igor Mammedov, 2017/08/30
- Re: [Qemu-ppc] [Qemu-devel] [PATCH for-2.11 5/6] ppc: simplify cpu model lookup by PVR, David Gibson, 2017/08/31
[Qemu-ppc] [PATCH for-2.11 4/6] ppc: replace inter-function cyclic dependency/recurssion with 2 simple lookups,
Igor Mammedov <=
[Qemu-ppc] [PATCH for-2.11 6/6] ppc: drop caching ObjectClass from PowerPCCPUAlias, Igor Mammedov, 2017/08/24
[Qemu-ppc] [PATCH for-2.11 2/6] ppc: make cpu_model translation to type consistent, Igor Mammedov, 2017/08/24