Re: [PATCH 1/4] MIPS: Replace _PAGE_READ with _PAGE_NO_READ

["Jayachandran C." <jayachandranc@netlogicmicro.com>]

On Fri, May 13, 2011 at 09:55:21AM -0700, Kevin Cernekee wrote:
> On Fri, May 13, 2011 at 8:56 AM, Ralf Baechle <ralf@linux-mips.org> wrote:
> > I'm not totally certain with my explanation but it seemed like a good
> > working hypothesis.  Jayachandran C. bisected this morning's linux-queue
> > on his Netlogic XLR which is MIPS64 R1 and found this comment causing
> > the problem.
> 
> Jayachandran, could you please confirm/deny the following:
> 
> Netlogic XLR is a MIPS64 R1 system.
> 
> You are running a 32-bit kernel.
> 
> You are using 64-bit physical addresses.
> 
> You are not enabling RI/XI.
> 
> The commit that caused the regression was "[PATCH 1/4] MIPS: Replace
> _PAGE_READ with _PAGE_NO_READ" (not 2/4, 3/4, or 4/4).
> 
> Do you have a log showing the failure, or any other details of what happened?

Yes, it is a MIPS64R1 system with 64-bit (well 40bit) physical address, we
don't have rixi either on hardware on in kernel overrides. git bisect pointed
the specific patch.

And with your patch it works only on 64 bit compile, 32 bit kernel fails on
init with:
|malloc: subst.c:521: assertion botched
|free: called with already freed block argument

For 32-bit the config is nlm_xlr_defconfig in the source tree. Let me know if
you need any further info.

Regards,
JC.