Russell King wrote: > On Sat, Mar 26, 2005 at 01:06:47PM +1100, Nick Piggin wrote: > >>The reject should be confined to include/asm-ia64, so it will still >>work for you. > > > I guess I should've tried a little harder last night then. Sorry. > Well I only said that just in case you had time to try testing again. I wouldn't expect you to go hunting through rejects and various kernel trees to try to get a working patch ;) > >>But I've put a clean rollup of all Hugh's patches here in case you'd >>like to try it. >> >>http://www.kerneltrap.org/~npiggin/freepgt-2.6.12-rc1.patch > > > This works fine on ARM with high vectors. With low vectors (located in > the 1st page of virtual memory space) I get: > > kernel BUG at mm/mmap.c:1934! > Unable to handle kernel NULL pointer dereference at virtual address 00000000 > pgd = c1e88000 > [00000000] *pgd=c1e86031, *pte=c04440cf, *ppte=c044400e > Internal error: Oops: c1e8981f [#1] > Modules linked in: > CPU: 0 > PC is at __bug+0x40/0x54 > LR is at 0x1 > pc : [] lr : [<00000001>] Not tainted > sp : c1e7bd18 ip : 60000093 fp : c1e7bd28 > r10: c1f4b040 r9 : 00000006 r8 : c1f02ca0 > r7 : 00000000 r6 : 00000000 r5 : c015b8c0 r4 : 00000000 > r3 : 00000000 r2 : 00000000 r1 : 00000d4e r0 : 00000001 > Flags: nZCv IRQs on FIQs on Mode SVC_32 Segment user > Control: C1E8917F Table: C1E8917F DAC: 00000015 > Process init (pid: 235, stack limit = 0xc1e7a194) > Stack: (0xc1e7bd18 to 0xc1e7c000) > ... > Backtrace: > [] (__bug+0x0/0x54) from [] (exit_mmap+0x154/0x168) > r4 = C1E7BD3C > [] (exit_mmap+0x0/0x168) from [] (mmput+0x40/0xc0) > r7 = C015B8C0 r6 = C015B8C0 r5 = 00000000 r4 = C015B8C0 > [] (mmput+0x0/0xc0) from [] (exec_mmap+0xec/0x134) > r4 = C015B6A0 > [] (exec_mmap+0x0/0x134) from [] (flush_old_exec+0x4c8/0x6e4) > r7 = C012B940 r6 = C1E7A000 r5 = C0498A00 r4 = 00000000 > [] (flush_old_exec+0x0/0x6e4) from [] (load_elf_binary+0x5c0/0xdc0) > [] (load_elf_binary+0x0/0xdc0) from [] (search_binary_handler+0xa0/0x244) > [] (search_binary_handler+0x0/0x244) from [] (load_script+0x224/0x22c) > [] (load_script+0x0/0x22c) from [] (search_binary_handler+0xa0/0x244) > r6 = C1E7A000 r5 = C015E400 r4 = C03EC2B4 > [] (search_binary_handler+0x0/0x244) from [] (do_execve+0x134/0x1f8) > [] (do_execve+0x0/0x1f8) from [] (sys_execve+0x3c/0x5c) > [] (sys_execve+0x0/0x5c) from [] (ret_fast_syscall+0x0/0x2c) > r7 = 0000000B r6 = BED6AA74 r5 = BED6AB00 r4 = 0200F008 > Code: 1b0051b8 e59f0014 eb0051b6 e3a03000 (e5833000) > > In this case, we have a page which must be kept mapped at virtual address > 0, which means the first entry in the L1 page table must always exist. > However, user threads start from 0x8000, which is also mapped via the > first entry in the L1 page table. > > At a guess, I'd imagine that we're freeing the first L1 page table entry, > thereby causing mm->nr_ptes to become -1. > > I'll do some debugging and try to work out if that (or exactly what's) > going on. > OK, thanks that would be good. You could well be right in your analysis. May I suggest a possible avenue of investigation: