diff mbox

[POWERPC] display cpu id dump state

Message ID 1424857165-26277-1-git-send-email-chouteau@adacore.com
State New
Headers show

Commit Message

Fabien Chouteau Feb. 25, 2015, 9:39 a.m. UTC
From: Tristan Gingold <gingold@adacore.com>


Signed-off-by: Fabien Chouteau <chouteau@adacore.com>
---
 target-ppc/translate.c |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Alexander Graf Feb. 26, 2015, 12:17 p.m. UTC | #1
On 25.02.15 10:39, Fabien Chouteau wrote:
> From: Tristan Gingold <gingold@adacore.com>
> 
> 
> Signed-off-by: Fabien Chouteau <chouteau@adacore.com>

Same nit here - please always provide a commit message stating your
rationale. I can guess (debugging SMP guests), but it's nicer when you
can look through the logs and there's no need to guess.


Alex

> ---
>  target-ppc/translate.c |    5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/target-ppc/translate.c b/target-ppc/translate.c
> index 88c18e3..2a78e99 100644
> --- a/target-ppc/translate.c
> +++ b/target-ppc/translate.c
> @@ -11214,8 +11214,9 @@ void ppc_cpu_dump_state(CPUState *cs, FILE *f, fprintf_function cpu_fprintf,
>      int i;
>  
>      cpu_fprintf(f, "NIP " TARGET_FMT_lx "   LR " TARGET_FMT_lx " CTR "
> -                TARGET_FMT_lx " XER " TARGET_FMT_lx "\n",
> -                env->nip, env->lr, env->ctr, cpu_read_xer(env));
> +                TARGET_FMT_lx " XER " TARGET_FMT_lx " CPU#%d\n",
> +                env->nip, env->lr, env->ctr, cpu_read_xer(env),
> +                cs->cpu_index);
>      cpu_fprintf(f, "MSR " TARGET_FMT_lx " HID0 " TARGET_FMT_lx "  HF "
>                  TARGET_FMT_lx " idx %d\n", env->msr, env->spr[SPR_HID0],
>                  env->hflags, env->mmu_idx);
>
Alexander Graf Feb. 26, 2015, 12:18 p.m. UTC | #2
On 25.02.15 10:39, Fabien Chouteau wrote:
> From: Tristan Gingold <gingold@adacore.com>
> 
> 
> Signed-off-by: Fabien Chouteau <chouteau@adacore.com>

That said, applied to ppc-next nevertheless.


Alex
diff mbox

Patch

diff --git a/target-ppc/translate.c b/target-ppc/translate.c
index 88c18e3..2a78e99 100644
--- a/target-ppc/translate.c
+++ b/target-ppc/translate.c
@@ -11214,8 +11214,9 @@  void ppc_cpu_dump_state(CPUState *cs, FILE *f, fprintf_function cpu_fprintf,
     int i;
 
     cpu_fprintf(f, "NIP " TARGET_FMT_lx "   LR " TARGET_FMT_lx " CTR "
-                TARGET_FMT_lx " XER " TARGET_FMT_lx "\n",
-                env->nip, env->lr, env->ctr, cpu_read_xer(env));
+                TARGET_FMT_lx " XER " TARGET_FMT_lx " CPU#%d\n",
+                env->nip, env->lr, env->ctr, cpu_read_xer(env),
+                cs->cpu_index);
     cpu_fprintf(f, "MSR " TARGET_FMT_lx " HID0 " TARGET_FMT_lx "  HF "
                 TARGET_FMT_lx " idx %d\n", env->msr, env->spr[SPR_HID0],
                 env->hflags, env->mmu_idx);