Re: [Qemu-devel] [PATCH] show register info for all cpus

[Anthony Liguori <anthony@codemonkey.ws>]

Glauber Costa wrote:
> Currently, we only do a register dump for mon_cpu().
> Modify do_info_registers() in order to show what's
> going on in all of them.
>   

The currently implemented behaviour is to allow the 'cpu' command to 
determine which CPU the various monitor commands operate with (including 
info registers).  This is pretty consistent throughout the monitor.

Do you disagree with this behaviour or were you just not aware of it?  
It's unclear from the description in your patch.

Regards,

Anthony Liguori

> Signed-off-by: Glauber Costa <gcosta@redhat.com>
> ---
>  qemu/monitor.c |   18 +++++++++++-------
>  1 files changed, 11 insertions(+), 7 deletions(-)
>
> diff --git a/qemu/monitor.c b/qemu/monitor.c
> index 20dcca6..045eecb 100644
> --- a/qemu/monitor.c
> +++ b/qemu/monitor.c
> @@ -292,16 +292,20 @@ static CPUState *mon_get_cpu(void)
>  static void do_info_registers(void)
>  {
>      CPUState *env;
> -    env = mon_get_cpu();
> -    if (!env)
> -        return;
> +
> +    mon_get_cpu();
> +
> +    for(env = first_cpu; env != NULL; env = env->next_cpu) {
> +        term_printf("CPU: %d\n", env->cpu_index);
>  #ifdef TARGET_I386
> -    cpu_dump_state(env, NULL, monitor_fprintf,
> -                   X86_DUMP_FPU);
> +        cpu_dump_state(env, NULL, monitor_fprintf,
> +                       X86_DUMP_FPU);
>  #else
> -    cpu_dump_state(env, NULL, monitor_fprintf,
> -                   0);
> +        cpu_dump_state(env, NULL, monitor_fprintf,
> +                       0);
>  #endif
> +    }
> +    term_printf("\n");
>  }
>  
>  static void do_info_cpus(void)
>