Re: [PATCH] ARM: avoid mis-detecting some V7 cores in the decompressor

Nicolas Pitre
2013-06-06T04:21:48Z

On Wed, 5 Jun 2013, Stephen Boyd wrote:

> On 06/04, Nicolas Pitre wrote:
> >
> > The LC0 area should be considered read-only as it may be located in
> > flash.
> >
> > Here's what I came with instead:
> >
> > From: Nicolas Pitre <nicolas.pitre@linaro.org>
> > Date: Tue, 4 Jun 2013 17:01:30 -0400
> > Subject: [PATCH] ARM: zImage: don't overwrite ourself with a page table
> >
> > When zImage is loaded into RAM at a low address but TEXT_OFFSET
> > is set higher, we risk overwriting ourself with the page table
> > needed to turn on the cache as it is located relative to the relocation
> > address. Let's defer the cache setup after relocation in that case.
> >
> > Signed-off-by: Nicolas Pitre <nico@linaro.org>
>
> Reported-by: Stephen Boyd <sboyd@codeurora.org>
> Tested-by: Stephen Boyd <sboyd@codeurora.org>
>
> This one passes testing on my two platforms with and without the
> 2Mb reservation at the beginning of ram. Seems like a good enough
> compromise for me.

Good! Queued here:

http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=7751/1

Nicolas