[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 22/40] sh4: simplify superh_cpu_class_by_name
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v2 22/40] sh4: simplify superh_cpu_class_by_name() |
Date: |
Thu, 5 Oct 2017 12:16:34 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 10/05/2017 10:50 AM, Igor Mammedov wrote:
> currently for sh4 cpu_model argument for '-cpu' option
> could be either 'cpu model' name or cpu_typename.
>
> however typically '-cpu' takes 'cpu model' name and
> cpu type for sh4 target isn't advertised publicly
> ('-cpu help' prints only 'cpu model' names) so we
> shouldn't care about this use case (it's more of a bug).
>
> 1. Drop '-cpu cpu_typename' to align with the rest of
> targets.
> 2. Compose searched for typename from cpu model and use
> it with object_class_by_name() directly instead of
> over-complicated
> object_class_get_list()
> g_slist_find_custom() + superh_cpu_name_compare()
>
> With #1 droped, #2 could be used for both lookups which
> simplifies superh_cpu_class_by_name() quite a bit.
>
> Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> v2:
> - there is no need to NULL-initialize 'oc' in superh_cpu_class_by_name()
> also drop extra ';' (Philippe Mathieu-Daudé)
>
> CC: address@hidden
> ---
> target/sh4/cpu.c | 34 ++++++++++++----------------------
> 1 file changed, 12 insertions(+), 22 deletions(-)
>
> diff --git a/target/sh4/cpu.c b/target/sh4/cpu.c
> index 72f1dec..e715560 100644
> --- a/target/sh4/cpu.c
> +++ b/target/sh4/cpu.c
> @@ -120,36 +120,26 @@ void sh4_cpu_list(FILE *f, fprintf_function cpu_fprintf)
> g_slist_free(list);
> }
>
> -static gint superh_cpu_name_compare(gconstpointer a, gconstpointer b)
> -{
> - const SuperHCPUClass *scc = SUPERH_CPU_CLASS(a);
> - const char *name = b;
> -
> - return strcasecmp(scc->name, name);
> -}
> -
> static ObjectClass *superh_cpu_class_by_name(const char *cpu_model)
> {
> ObjectClass *oc;
> - GSList *list, *item;
> + char *s, *typename = NULL;
>
> - if (strcasecmp(cpu_model, "any") == 0) {
> - return object_class_by_name(TYPE_SH7750R_CPU);
> + s = g_ascii_strdown(cpu_model, -1);
> + if (strcmp(s, "any") == 0) {
> + oc = object_class_by_name(TYPE_SH7750R_CPU);
> + goto out;
> }
>
> - oc = object_class_by_name(cpu_model);
> - if (oc != NULL && object_class_dynamic_cast(oc, TYPE_SUPERH_CPU) != NULL
> - && !object_class_is_abstract(oc)) {
> - return oc;
> + typename = g_strdup_printf(SUPERH_CPU_TYPE_NAME("%s"), s);
> + oc = object_class_by_name(typename);
> + if (oc != NULL && object_class_is_abstract(oc)) {
> + oc = NULL;
> }
>
> - oc = NULL;
> - list = object_class_get_list(TYPE_SUPERH_CPU, false);
> - item = g_slist_find_custom(list, cpu_model, superh_cpu_name_compare);
> - if (item != NULL) {
> - oc = item->data;
> - }
> - g_slist_free(list);
> +out:
> + g_free(s);
> + g_free(typename);
> return oc;
> }
>
>
- [Qemu-devel] [PATCH v2 11/40] m68k: cleanup cpu type name composition, (continued)
- [Qemu-devel] [PATCH v2 11/40] m68k: cleanup cpu type name composition, Igor Mammedov, 2017/10/05
- [Qemu-devel] [PATCH v2 15/40] moxie: cleanup cpu type name composition, Igor Mammedov, 2017/10/05
- [Qemu-devel] [PATCH v2 18/40] openrisc: use generic cpu_model parsing, Igor Mammedov, 2017/10/05
- [Qemu-devel] [PATCH v2 14/40] moxie: fix qemu-system-moxie failing to start with CLI "-cpu MoxieLite", Igor Mammedov, 2017/10/05
- [Qemu-devel] [PATCH v2 19/40] sh4: r2d: use generic cpu_model parsing, Igor Mammedov, 2017/10/05
- [Qemu-devel] [PATCH v2 16/40] moxie: use generic cpu_model parsing, Igor Mammedov, 2017/10/05
- [Qemu-devel] [PATCH v2 17/40] openrisc: cleanup cpu type name composition, Igor Mammedov, 2017/10/05
- [Qemu-devel] [PATCH v2 22/40] sh4: simplify superh_cpu_class_by_name(), Igor Mammedov, 2017/10/05
- [Qemu-devel] [PATCH v2 23/40] sh4: remove SuperHCPUClass::name field, Igor Mammedov, 2017/10/05
- [Qemu-devel] [PATCH v2 20/40] sh4: shix: use generic cpu_model parsing, Igor Mammedov, 2017/10/05
- [Qemu-devel] [PATCH v2 24/40] xtensa: cleanup cpu type name composition, Igor Mammedov, 2017/10/05
- [Qemu-devel] [PATCH v2 21/40] sh4: cleanup cpu type name composition, Igor Mammedov, 2017/10/05
- [Qemu-devel] [PATCH v2 25/40] xtensa: sim: use generic cpu_model parsing, Igor Mammedov, 2017/10/05
- [Qemu-devel] [PATCH v2 26/40] xtensa: lx60/lx200/ml605/kc705: use generic cpu_model parsing, Igor Mammedov, 2017/10/05
- [Qemu-devel] [PATCH v2 29/40] tricore: cleanup cpu type name composition, Igor Mammedov, 2017/10/05
- [Qemu-devel] [PATCH v2 28/40] unicore32: use generic cpu_model parsing, Igor Mammedov, 2017/10/05