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

[Hiroyuki KAMEZAWA]

Can you try with this patch ?
It's maybe because page table is not created correctly.

Kame <kamezawa.hiroyu@xxxxxxxxxxxxxx>
=============== 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@xxxxxxx 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@xxxxxxxxxxxxxx>
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


--
--the clue is these footmarks leading to the door.--
KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx>

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/