Re: bug in 2.6.9-rc4-mm1 ia64/mm/init.c

Hiroyuki KAMEZAWA
2004-10-13T01:41:42Z

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>