From: Hiroyuki KAMEZAWA <kamezawa.hiroyu@jp.fujitsu.com>
To: Hiroyuki KAMEZAWA <kamezawa.hiroyu@jp.fujitsu.com>
Cc: akepner@sgi.com, 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 01:46:32 +0000 [thread overview]
Message-ID: <416C88F8.5030109@jp.fujitsu.com> (raw)
In-Reply-To: <416C83EB.1090608@jp.fujitsu.com>
Can you try with this patch ?
It's maybe because page table is not created correctly.
Kame <kamezawa.hiroyu@jp.fujitsu.com>
======== cut from here =========
--- arch/ia64/mm/init.c.org 2004-10-13 10:32:09.969725576 +0900
+++ arch/ia64/mm/init.c 2004-10-13 10:33:25.636222520 +0900
@@ -371,7 +371,8 @@ create_mem_map_page_table (u64 start, u6
pgd_t *pgd;
pmd_t *pmd;
pte_t *pte;
-
+ start = GRANULEROUNDDOWN(start);
+ end = GRANULEROUNDUP(end);
map_start = vmem_map + (__pa(start) >> PAGE_SHIFT);
map_end = vmem_map + (__pa(end) >> PAGE_SHIFT);
Hiroyuki KAMEZAWA wrote:
> 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\x109723728329408&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>
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
--the clue is these footmarks leading to the door.--
KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
WARNING: multiple messages have this Message-ID (diff)
From: Hiroyuki KAMEZAWA <kamezawa.hiroyu@jp.fujitsu.com>
To: Hiroyuki KAMEZAWA <kamezawa.hiroyu@jp.fujitsu.com>
Cc: akepner@sgi.com, 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:46:32 +0900 [thread overview]
Message-ID: <416C88F8.5030109@jp.fujitsu.com> (raw)
In-Reply-To: <416C83EB.1090608@jp.fujitsu.com>
Can you try with this patch ?
It's maybe because page table is not created correctly.
Kame <kamezawa.hiroyu@jp.fujitsu.com>
=============== cut from here ==================
--- arch/ia64/mm/init.c.org 2004-10-13 10:32:09.969725576 +0900
+++ arch/ia64/mm/init.c 2004-10-13 10:33:25.636222520 +0900
@@ -371,7 +371,8 @@ create_mem_map_page_table (u64 start, u6
pgd_t *pgd;
pmd_t *pmd;
pte_t *pte;
-
+ start = GRANULEROUNDDOWN(start);
+ end = GRANULEROUNDUP(end);
map_start = vmem_map + (__pa(start) >> PAGE_SHIFT);
map_end = vmem_map + (__pa(end) >> PAGE_SHIFT);
Hiroyuki KAMEZAWA wrote:
> 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>
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
--the clue is these footmarks leading to the door.--
KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
next prev parent reply other threads:[~2004-10-13 1:46 UTC|newest]
Thread overview: 16+ 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 0:21 ` akepner
2004-10-13 1:24 ` Hiroyuki KAMEZAWA
2004-10-13 1:24 ` Hiroyuki KAMEZAWA
2004-10-13 1:46 ` Hiroyuki KAMEZAWA [this message]
2004-10-13 1:46 ` Hiroyuki KAMEZAWA
2004-10-13 8:04 ` David Mosberger
2004-10-13 8:04 ` David Mosberger
2004-10-13 8:44 ` Hiroyuki KAMEZAWA
2004-10-13 8:44 ` Hiroyuki KAMEZAWA
2004-10-13 8:42 ` David Mosberger
2004-10-13 8:42 ` David Mosberger
2004-10-13 8:56 ` Hiroyuki KAMEZAWA
2004-10-13 9:02 ` Hiroyuki KAMEZAWA
2004-10-14 4:49 ` 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=416C88F8.5030109@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.