On Nov 24, 2011, at 11:35 PM, Anton Blanchard wrote:
>
> On a 64bit book3s machine I have an oops from a system reset that
> claims the book3e CE bit was set:
>
> MSR: 8000000000021032 <ME,CE,IR,DR> CR: 24004082 XER: 00000010
>
> On a book3s machine system reset sets IBM bit 46 and 47 depending on
> the power saving mode. Separate the definitions by type and for
> completeness add the rest of the bits in.
>
> Signed-off-by: Anton Blanchard <an...@samba.org>
> ---
>
> Index: linux-build/arch/powerpc/kernel/process.c
> ===================================================================
> --- linux-build.orig/arch/powerpc/kernel/process.c 2011-11-25
> 13:22:24.294919094 +1100
> +++ linux-build/arch/powerpc/kernel/process.c 2011-11-25 13:36:23.213834524
> +1100
> @@ -584,16 +584,32 @@ static struct regbit {
> unsigned long bit;
> const char *name;
> } msr_bits[] = {
> +#if defined(CONFIG_PPC64) && !defined(CONFIG_BOOKE)
> + {MSR_SF, "SF"},
> + {MSR_HV, "HV"},
> +#endif
> + {MSR_VEC, "VEC"},
> + {MSR_VSX, "VSX"},
> +#ifdef CONFIG_BOOKE
> + {MSR_CE, "CE"},
> +#endif
> {MSR_EE, "EE"},
> {MSR_PR, "PR"},
> {MSR_FP, "FP"},
> - {MSR_VEC, "VEC"},
> - {MSR_VSX, "VSX"},
> {MSR_ME, "ME"},
> - {MSR_CE, "CE"},
> +#ifdef CONFIG_BOOKE
> {MSR_DE, "DE"},
> +#else
> + {MSR_SE, "SE"},
> + {MSR_BE, "BE"},
> +#endif
> {MSR_IR, "IR"},
> {MSR_DR, "DR"},
> + {MSR_PMM, "PMM"},
> +#ifndef CONFIG_BOOKE
> + {MSR_RI, "RI"},
We have 'RI' on some BOOKE so lets allow for it to be decoded
> + {MSR_LE, "LE"},
> +#endif
> {0, NULL}
> };
Since you're fixing this can you add the following for CONFIG_BOOKE:
MSR_GS, MSR_UCLE, MSR_PMM, MSR_CM
- k
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
