On Sun, 30 Sep 2007 05:09:28 +1000 Nick Piggin <[EMAIL PROTECTED]> wrote:
> On Sunday 30 September 2007 05:20, Andrew Morton wrote: > > On Sat, 29 Sep 2007 06:19:33 +1000 Nick Piggin <[EMAIL PROTECTED]> > wrote: > > > On Saturday 29 September 2007 19:27, Andrew Morton wrote: > > > > On Sat, 29 Sep 2007 11:14:02 +0200 Peter Zijlstra > > > > <[EMAIL PROTECTED]> > > > > > > wrote: > > > > > > oom-killings, or page allocation failures? The latter, one hopes. > > > > > > > > > > Linux version 2.6.23-rc4-mm1-dirty ([EMAIL PROTECTED]) (gcc version > > > > > 4.1.2 > > > > > (Ubuntu 4.1.2-0ubuntu4)) #27 Tue Sep 18 15:40:35 CEST 2007 > > > > > > > > > > ... > > > > > > > > > > > > > > > mm_tester invoked oom-killer: gfp_mask=0x40d0, order=2, oomkilladj=0 > > > > > Call Trace: > > > > > 611b3878: [<6002dd28>] printk_ratelimit+0x15/0x17 > > > > > 611b3888: [<60052ed4>] out_of_memory+0x80/0x100 > > > > > 611b38c8: [<60054b0c>] __alloc_pages+0x1ed/0x280 > > > > > 611b3948: [<6006c608>] allocate_slab+0x5b/0xb0 > > > > > 611b3968: [<6006c705>] new_slab+0x7e/0x183 > > > > > 611b39a8: [<6006cbae>] __slab_alloc+0xc9/0x14b > > > > > 611b39b0: [<6011f89f>] radix_tree_preload+0x70/0xbf > > > > > 611b39b8: [<600980f2>] do_mpage_readpage+0x3b3/0x472 > > > > > 611b39e0: [<6011f89f>] radix_tree_preload+0x70/0xbf > > > > > 611b39f8: [<6006cc81>] kmem_cache_alloc+0x51/0x98 > > > > > 611b3a38: [<6011f89f>] radix_tree_preload+0x70/0xbf > > > > > 611b3a58: [<6004f8e2>] add_to_page_cache+0x22/0xf7 > > > > > 611b3a98: [<6004f9c6>] add_to_page_cache_lru+0xf/0x24 > > > > > 611b3ab8: [<6009821e>] mpage_readpages+0x6d/0x109 > > > > > 611b3ac0: [<600d59f0>] ext3_get_block+0x0/0xf2 > > > > > 611b3b08: [<6005483d>] get_page_from_freelist+0x8d/0xc1 > > > > > 611b3b88: [<600d6937>] ext3_readpages+0x18/0x1a > > > > > 611b3b98: [<60056f00>] read_pages+0x37/0x9b > > > > > 611b3bd8: [<60057064>] __do_page_cache_readahead+0x100/0x157 > > > > > 611b3c48: [<60057196>] do_page_cache_readahead+0x52/0x5f > > > > > 611b3c78: [<60050ab4>] filemap_fault+0x145/0x278 > > > > > 611b3ca8: [<60022b61>] run_syscall_stub+0xd1/0xdd > > > > > 611b3ce8: [<6005eae3>] __do_fault+0x7e/0x3ca > > > > > 611b3d68: [<6005ee60>] do_linear_fault+0x31/0x33 > > > > > 611b3d88: [<6005f149>] handle_mm_fault+0x14e/0x246 > > > > > 611b3da8: [<60120a7b>] __up_read+0x73/0x7b > > > > > 611b3de8: [<60013177>] handle_page_fault+0x11f/0x23b > > > > > 611b3e48: [<60013419>] segv+0xac/0x297 > > > > > 611b3f28: [<60013367>] segv_handler+0x68/0x6e > > > > > 611b3f48: [<600232ad>] get_skas_faultinfo+0x9c/0xa1 > > > > > 611b3f68: [<60023853>] userspace+0x13a/0x19d > > > > > 611b3fc8: [<60010d58>] fork_handler+0x86/0x8d > > > > > > > > OK, that's different. Someone broke the vm - order-2 GFP_KERNEL > > > > allocations aren't supposed to fail. > > > > > > > > I'm suspecting that did_some_progress thing. > > > > > > The allocation didn't fail -- it invoked the OOM killer because the > > > kernel ran out of unfragmented memory. > > > > We can't "run out of unfragmented memory" for an order-2 GFP_KERNEL > > allocation in this workload. We go and synchronously free stuff up to make > > it work. > > > > How did this get broken? > > Either no more order-2 pages could be freed, or the ones that were being > freed were being used by something else (eg. other order-2 slab allocations). No. The current design of reclaim (for better or for worse) is that for order 0,1,2 and 3 allocations we just keep on trying until it works. That got broken and I think it got broken at a design level when that did_some_progress logic went in. Perhaps something else we did later worsened things. > > > > Probably because higher order > > > allocations are the new vogue in -mm at the moment ;) > > > > That's a different bug. > > > > bug 1: We shouldn't be doing higher-order allocations in slub because of > > the considerable damage this does to atomic allocations. > > > > bug 2: order-2 GFP_KERNEL allocations shouldn't fail like this. > > I think one causes 2 as well -- it isn't just considerable damage to atomic > allocations but to GFP_KERNEL allocations too. Well sure, because we already broke GFP_KERNEL allocations. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/