From: Ralf Baechle <ralf@linux-mips.org>
To: Jason Kwon <jason.kwon@ericsson.com>
Cc: Guenter Roeck <guenter.roeck@ericsson.com>,
David Daney <david.daney@cavium.com>,
"linux-mips@linux-mips.org" <linux-mips@linux-mips.org>
Subject: Re: [PATCH] MIPS: Octeon: Select CONFIG_HOLES_IN_ZONE
Date: Sat, 20 Aug 2011 00:18:30 +0100 [thread overview]
Message-ID: <20110819231830.GA22981@linux-mips.org> (raw)
In-Reply-To: <4E4ED4F5.2050002@ericsson.com>
On Fri, Aug 19, 2011 at 02:26:13PM -0700, Jason Kwon wrote:
> Date: Fri, 19 Aug 2011 14:26:13 -0700
> From: Jason Kwon <jason.kwon@ericsson.com>
> To: Guenter Roeck <guenter.roeck@ericsson.com>
> CC: David Daney <david.daney@cavium.com>, "linux-mips@linux-mips.org"
> <linux-mips@linux-mips.org>, "ralf@linux-mips.org" <ralf@linux-mips.org>
> Subject: Re: [PATCH] MIPS: Octeon: Select CONFIG_HOLES_IN_ZONE
> Content-Type: text/plain; charset="UTF-8"; format=flowed
>
> On 08/19/2011 12:13 PM, Guenter Roeck wrote:
> >On Fri, 2011-08-19 at 14:44 -0400, David Daney wrote:
> >>Current Octeon systems do in fact have holes in their memory zones.
> >>We need to select HOLES_IN_ZONE. If we do not, some memory
> >>configurations will result in crashes at boot time like this:
> >>
> >>.
> >>.
> >>.
> >>CPU 6 Unable to handle kernel paging request at virtual address 0000000000700000, epc == ffffffff8118fe00, ra == ffffffff8118fe9c
> >>Oops[#1]:
> >>Cpu 6
> >>.
> >>.
> >>.
> >> ...
> >>Call Trace:
> >>[<ffffffff8118fe00>] setup_per_zone_wmarks+0x1b0/0x338
> >>[<ffffffff815cd738>] init_per_zone_wmark_min+0x64/0xd0
> >>[<ffffffff81100438>] do_one_initcall+0x38/0x160
> >>.
> >>.
> >>.
> >>
> >>Reported-by: Jason Kwon<jason.kwon@ericsson.com>
> >>Signed-off-by: David Daney<david.daney@cavium.com>
> >>Cc: Jason Kwon<jason.kwon@ericsson.com>
> >>---
> >>Jason, can you test this patch?
> >>
> >>Ralf, if Jason reports that it fixes his problem, it probably is
> >>needed for 3.0 and 3.1.
> >>
> >Your patch fixes the problem for the board with CN38xx and 2GB RAM that
> >crashed previously.
> >
> >Tested-by: Guenter Roeck<guenter.roeck@ericsson.com>
> >
> >Thanks a lot for looking into this.
> >
> >Guenter
> >
> >
>
> I applied the patch to 3.0.3 and was able to boot the CN58XX system
> without any memory restrictions. The same patched kernel booted on
> CN38XX ran into a different problem, which I'm looking into.
Applied. Thanks, folks!
Ralf
prev parent reply other threads:[~2011-08-19 23:18 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-08-19 18:44 [PATCH] MIPS: Octeon: Select CONFIG_HOLES_IN_ZONE David Daney
2011-08-19 19:13 ` Guenter Roeck
2011-08-19 21:26 ` Jason Kwon
2011-08-19 23:18 ` Ralf Baechle [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110819231830.GA22981@linux-mips.org \
--to=ralf@linux-mips.org \
--cc=david.daney@cavium.com \
--cc=guenter.roeck@ericsson.com \
--cc=jason.kwon@ericsson.com \
--cc=linux-mips@linux-mips.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox