target/hppa/cpu.c | 22 ---------------------- 1 file changed, 22 deletions(-)
hppa_cpu_list() is dead code and is never called. Delete it.
Cc: Richard Henderson <rth@twiddle.net>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
---
target/hppa/cpu.c | 22 ----------------------
1 file changed, 22 deletions(-)
diff --git a/target/hppa/cpu.c b/target/hppa/cpu.c
index 00bf444620..b3b1826209 100644
--- a/target/hppa/cpu.c
+++ b/target/hppa/cpu.c
@@ -110,28 +110,6 @@ static void hppa_cpu_realizefn(DeviceState *dev, Error **errp)
#endif
}
-static void hppa_cpu_list_entry(gpointer data, gpointer user_data)
-{
- ObjectClass *oc = data;
- CPUListState *s = user_data;
-
- (*s->cpu_fprintf)(s->file, " %s\n", object_class_get_name(oc));
-}
-
-void hppa_cpu_list(FILE *f, fprintf_function cpu_fprintf)
-{
- CPUListState s = {
- .file = f,
- .cpu_fprintf = cpu_fprintf,
- };
- GSList *list;
-
- list = object_class_get_list_sorted(TYPE_HPPA_CPU, false);
- (*cpu_fprintf)(f, "Available CPUs:\n");
- g_slist_foreach(list, hppa_cpu_list_entry, &s);
- g_slist_free(list);
-}
-
static void hppa_cpu_initfn(Object *obj)
{
CPUState *cs = CPU(obj);
--
2.18.0.rc1.1.g3f1ff2140
On Thu, 18 Apr 2019 01:01:03 -0300
Eduardo Habkost <ehabkost@redhat.com> wrote:
> hppa_cpu_list() is dead code and is never called. Delete it.
>
> Cc: Richard Henderson <rth@twiddle.net>
> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
> ---
> target/hppa/cpu.c | 22 ----------------------
> 1 file changed, 22 deletions(-)
>
> diff --git a/target/hppa/cpu.c b/target/hppa/cpu.c
> index 00bf444620..b3b1826209 100644
> --- a/target/hppa/cpu.c
> +++ b/target/hppa/cpu.c
> @@ -110,28 +110,6 @@ static void hppa_cpu_realizefn(DeviceState *dev, Error **errp)
> #endif
> }
>
> -static void hppa_cpu_list_entry(gpointer data, gpointer user_data)
> -{
> - ObjectClass *oc = data;
> - CPUListState *s = user_data;
> -
> - (*s->cpu_fprintf)(s->file, " %s\n", object_class_get_name(oc));
> -}
> -
> -void hppa_cpu_list(FILE *f, fprintf_function cpu_fprintf)
> -{
> - CPUListState s = {
> - .file = f,
> - .cpu_fprintf = cpu_fprintf,
> - };
> - GSList *list;
> -
> - list = object_class_get_list_sorted(TYPE_HPPA_CPU, false);
> - (*cpu_fprintf)(f, "Available CPUs:\n");
> - g_slist_foreach(list, hppa_cpu_list_entry, &s);
> - g_slist_free(list);
> -}
> -
> static void hppa_cpu_initfn(Object *obj)
> {
> CPUState *cs = CPU(obj);
Hi Eduardo,
On 4/18/19 6:01 AM, Eduardo Habkost wrote:
> hppa_cpu_list() is dead code and is never called. Delete it.
>
> Cc: Richard Henderson <rth@twiddle.net>
> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
> target/hppa/cpu.c | 22 ----------------------
> 1 file changed, 22 deletions(-)
>
> diff --git a/target/hppa/cpu.c b/target/hppa/cpu.c
> index 00bf444620..b3b1826209 100644
> --- a/target/hppa/cpu.c
> +++ b/target/hppa/cpu.c
> @@ -110,28 +110,6 @@ static void hppa_cpu_realizefn(DeviceState *dev, Error **errp)
> #endif
> }
>
> -static void hppa_cpu_list_entry(gpointer data, gpointer user_data)
> -{
> - ObjectClass *oc = data;
> - CPUListState *s = user_data;
> -
> - (*s->cpu_fprintf)(s->file, " %s\n", object_class_get_name(oc));
> -}
> -
> -void hppa_cpu_list(FILE *f, fprintf_function cpu_fprintf)
You forgot to remove the declaration in "target/hppa/cpu.h".
With this header updated:
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> -{
> - CPUListState s = {
> - .file = f,
> - .cpu_fprintf = cpu_fprintf,
> - };
> - GSList *list;
> -
> - list = object_class_get_list_sorted(TYPE_HPPA_CPU, false);
> - (*cpu_fprintf)(f, "Available CPUs:\n");
> - g_slist_foreach(list, hppa_cpu_list_entry, &s);
> - g_slist_free(list);
> -}
> -
> static void hppa_cpu_initfn(Object *obj)
> {
> CPUState *cs = CPU(obj);
>
© 2016 - 2026 Red Hat, Inc.