From: Hiroyuki KAMEZAWA <kamezawa.hiroyu@jp.fujitsu.com>
To: akepner@sgi.com
Cc: linux-kernel@vger.kernel.org, linux-ia64@vger.kernel.org,
akpm@osdl.org, jbarnes@sgi.com
Subject: Re: bug in 2.6.9-rc4-mm1 ia64/mm/init.c
Date: Wed, 13 Oct 2004 10:24:59 +0900 [thread overview]
Message-ID: <416C83EB.1090608@jp.fujitsu.com> (raw)
In-Reply-To: <Pine.LNX.4.33.0410121705510.31839-100000@localhost.localdomain>
Hi,
akepner@sgi.com wrote:
> Hi Folks;
>
> Tried a kernel built with akpm's 2.6.9-rc4-mm1 patch today (using
> a default sn2 .config file.) It crashes on boot with:
>
> ....
> SGI SAL version 3.40
> Virtual mem_map starts at 0xa0007ffe85938000
> Unable to handle kernel paging request at virtual address a0007ffeaf970000
> swapper[0]: Oops 8813272891392 [1]
> Modules linked in:
<snip>
> I traced this down to a recent patch (see
> http://marc.theaimsgroup.com/?l=linux-mm&m=109723728329408&w=2)
> which contains:
>
> diff -puN arch/ia64/mm/init.c~ia64_fix arch/ia64/mm/init.c
> --- test-kernel/arch/ia64/mm/init.c~ia64_fix 2004-10-08
> 18:29:20.510992392 +0900
> +++ test-kernel-kamezawa/arch/ia64/mm/init.c 2004-10-08
> 18:29:20.515991632 +0900
> @@ -410,7 +410,8 @@ virtual_memmap_init (u64 start, u64 end,
> struct page *map_start, *map_end;
>
> args = (struct memmap_init_callback_data *) arg;
> -
> + start = GRANULEROUNDDOWN(start);
> + end = GRANULEROUNDUP(end);
> map_start = vmem_map + (__pa(start) >> PAGE_SHIFT);
> map_end = vmem_map + (__pa(end) >> PAGE_SHIFT);
>
>
> Merely commenting out the new lines containting GRANULEROUNDDOWN, and
> GRANULEROUNDUP allowed the system to boot and me to finish the testing
> I needed to do.
>
> Looks like the above patch needs to be revised. I could test it if
> necessary. Please email me directly as I'm not subscribed to lkml
> or linux-ia64.
>
Hmm.. I added above 2 lines for making vmemmap to be aligned with ia64's GRANULE.
But it looks troublesome here, revising it will be needed currently.
I'd like to check my codes again and fix it.
Kame <kamezawa.hiroyu@jp.fujitsu.com>
next prev parent reply other threads:[~2004-10-13 1:19 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-10-13 0:21 bug in 2.6.9-rc4-mm1 ia64/mm/init.c akepner
2004-10-13 1:24 ` Hiroyuki KAMEZAWA [this message]
2004-10-13 1:46 ` Hiroyuki KAMEZAWA
2004-10-13 8:04 ` David Mosberger
2004-10-13 8:44 ` Hiroyuki KAMEZAWA
2004-10-13 8:42 ` David Mosberger
2004-10-13 9:02 ` Hiroyuki KAMEZAWA
2004-10-14 4:49 ` Hiroyuki KAMEZAWA
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=416C83EB.1090608@jp.fujitsu.com \
--to=kamezawa.hiroyu@jp.fujitsu.com \
--cc=akepner@sgi.com \
--cc=akpm@osdl.org \
--cc=jbarnes@sgi.com \
--cc=linux-ia64@vger.kernel.org \
--cc=linux-kernel@vger.kernel.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