On Mon, Nov 02, 2015 at 11:21:43AM +1100, Sam Mendoza-Jonas wrote:
> On Mon, Nov 02, 2015 at 09:55:53AM +1100, Sam Mendoza-Jonas wrote:
> > On Tue, Oct 06, 2015 at 05:55:49PM -0500, Scott Wood wrote:
> > > Commit a304e2d82a8c3 ("ppc64: purgatory: Reset primary cpu endian to
> > > big-endian) changed bctr to rfid. rfid is book3s-only and will cause a
> > > fatal exception on book3e.
> > >
> > > Purgatory is an isolated environment which makes importing information
> > > about the subarch awkward, so instead rely on the fact that MSR_LE
> > > should never be set on book3e, and the rfid is only needed if MSR_LE is
> > > set (and thus needs to be cleared). In theory that MSR bit is reserved
> > > on book3e, rather than zero, but in practice I have not seen it set.
> > >
> > > Signed-off-by: Scott Wood <scottwood at freescale.com>
> > > Cc: Samuel Mendoza-Jonas <sam.mj at au1.ibm.com>
> > > ---
> > > v2: new patch
> >
> > Looks good to me, thanks for catching this!
> > I'll ping Michael too if you'd like his reviewed-by.
> >
> > Reviewed-by: Samuel Mendoza-Jonas <sam.mj at au1.ibm.com>
>
> Following up, also add my
>
> Tested-by: Samuel Mendoza-Jonas <sam.mj at au1.ibm.com>
Thanks, I have queued this up for v2.0.11.
