qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH qom-cpu 09/59] cpus: Simplify hw_error()


From: Andreas Färber
Subject: Re: [Qemu-devel] [PATCH qom-cpu 09/59] cpus: Simplify hw_error()
Date: Mon, 10 Jun 2013 23:48:39 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130510 Thunderbird/17.0.6

Am 09.06.2013 21:12, schrieb Andreas Färber:
> Use new qemu_for_each_cpu().
> 
> Signed-off-by: Andreas Färber <address@hidden>
> ---
>  cpus.c | 15 ++++++++-------
>  1 file changed, 8 insertions(+), 7 deletions(-)
> 
> diff --git a/cpus.c b/cpus.c
> index 1975c5c..9195bc3 100644
> --- a/cpus.c
> +++ b/cpus.c
> @@ -386,21 +386,22 @@ void configure_icount(const char *option)
>  }
>  
>  /***********************************************************/
> +
> +static void hw_one_error(CPUState *cpu, void *data)
> +{
> +    fprintf(stderr, "CPU #%d:\n", cpu->cpu_index);
> +    cpu_dump_state(cpu->env_ptr, stderr, fprintf, CPU_DUMP_FPU);

This env_ptr usage is dangerous since as void* it bypasses all argument
type checks - luckily 18/59 does convert it.

If we want to go with qemu_for_each_cpu(), I better add a temporary
CPUArchState *env variable.

Andreas

> +}
> +
>  void hw_error(const char *fmt, ...)
>  {
>      va_list ap;
> -    CPUArchState *env;
> -    CPUState *cpu;
>  
>      va_start(ap, fmt);
>      fprintf(stderr, "qemu: hardware error: ");
>      vfprintf(stderr, fmt, ap);
>      fprintf(stderr, "\n");
> -    for (env = first_cpu; env != NULL; env = env->next_cpu) {
> -        cpu = ENV_GET_CPU(env);
> -        fprintf(stderr, "CPU #%d:\n", cpu->cpu_index);
> -        cpu_dump_state(env, stderr, fprintf, CPU_DUMP_FPU);
> -    }
> +    qemu_for_each_cpu(hw_one_error, NULL);
>      va_end(ap);
>      abort();
>  }
> 


-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg



reply via email to

[Prev in Thread] Current Thread [Next in Thread]