APPLIED[artful]: [PATCH] [Zesty] powerpc/powernv: Fix boot on Power8 bare metal due to opal_configure_cores()

Seth Forshee seth.forshee at canonical.com
Tue Jul 18 12:24:51 UTC 2017


On Tue, Jul 18, 2017 at 09:42:59AM +0200, Kleber Souza wrote:
> On 07/18/17 09:11, Stefan Bader wrote:
> > On 18.07.2017 03:05, Michael Ellerman wrote:
> >> Breno Leitao <leitao at debian.org> writes:
> >>
> >>> diff --git a/arch/powerpc/platforms/powernv/opal.c b/arch/powerpc/platforms/powernv/opal.c
> >>> index e0f856bfbfe8..a3b6de84f49c 100644
> >>> --- a/arch/powerpc/platforms/powernv/opal.c
> >>> +++ b/arch/powerpc/platforms/powernv/opal.c
> >>> @@ -66,11 +68,24 @@ void opal_configure_cores(void)
> >>>  	 * but it might clobber a bunch.
> >>>  	 */
> >>>  #ifdef __BIG_ENDIAN__
> >>> -	opal_reinit_cpus(OPAL_REINIT_CPUS_HILE_BE);
> >>> +	reinit_flags |= OPAL_REINIT_CPUS_HILE_BE;
> >>>  #else
> >>> -	opal_reinit_cpus(OPAL_REINIT_CPUS_HILE_LE);
> >>> +	reinit_flags |= OPAL_REINIT_CPUS_HILE_LE;
> >>>  #endif
> >>>  
> >>> +	/*
> >>> +	 * POWER9 always support running hash:
> >>> +	 *  ie. Host hash  supports  hash guests
> >>> +	 *      Host radix supports  hash/radix guests
> >>> +	 */
> >>> +	if (cpu_has_feature(CPU_FTR_ARCH_300)) {
> >>
> >> This is wrong. It can break booting on bare-metal Power8 machines with
> >> some versions of firmware.
> >>
> >> Fix here:
> >>
> >> https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git/commit/?h=fixes&id=a70b487b07cf4201bc6702e7f646fa593b23009f
> >>
> >> http://patchwork.ozlabs.org/patch/789381/
> > 
> > Ok, that is in linux-next by now (see attached).
> > 
> > -Stefan
> 
> Attached patch (powerpc/powernv: Fix boot on Power8 bare metal due to
> opal_configure_cores()) acked and applied to zesty/master-next branch.

Also applied to artful/master-next and unstable/master.




More information about the kernel-team mailing list