On Wed, Nov 04, 2015 at 04:21:35PM +0200, Kirill A. Shutemov wrote: > On Wed, Nov 04, 2015 at 12:20:19AM +0900, Minchan Kim wrote: > > On Tue, Nov 03, 2015 at 04:33:29PM +0900, Minchan Kim wrote: > > > On Tue, Nov 03, 2015 at 09:16:50AM +0200, Kirill A. Shutemov wrote: > > > > On Tue, Nov 03, 2015 at 12:02:58PM +0900, Minchan Kim wrote: > > > > > Hello Kirill, > > > > > > > > > > On Mon, Nov 02, 2015 at 02:57:49PM +0200, Kirill A. Shutemov wrote: > > > > > > On Fri, Oct 30, 2015 at 04:03:50PM +0900, Minchan Kim wrote: > > > > > > > On Thu, Oct 29, 2015 at 11:52:06AM +0200, Kirill A. Shutemov wrote: > > > > > > > > On Thu, Oct 29, 2015 at 04:58:29PM +0900, Minchan Kim wrote: > > > > > > > > > On Thu, Oct 29, 2015 at 02:25:24AM +0200, Kirill A. Shutemov wrote: > > > > > > > > > > On Thu, Oct 22, 2015 at 06:00:51PM +0900, Minchan Kim wrote: > > > > > > > > > > > On Thu, Oct 22, 2015 at 10:21:36AM +0900, Minchan Kim wrote: > > > > > > > > > > > > Hello Hugh, > > > > > > > > > > > > > > > > > > > > > > > > On Wed, Oct 21, 2015 at 05:59:59PM -0700, Hugh Dickins wrote: > > > > > > > > > > > > > On Thu, 22 Oct 2015, Minchan Kim wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > I added the code to check it and queued it again but I had another oops > > > > > > > > > > > > > > in this time but symptom is related to anon_vma, too. > > > > > > > > > > > > > > (kernel is based on recent mmotm + unconditional mkdirty for bug fix) > > > > > > > > > > > > > > It seems page_get_anon_vma returns NULL since the page was not page_mapped > > > > > > > > > > > > > > at that time but second check of page_mapped right before try_to_unmap seems > > > > > > > > > > > > > > to be true. > > > > > > > > > > > > > > > > > > > > > > > > > > > > Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS > > > > > > > > > > > > > > Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS > > > > > > > > > > > > > > page:ffffea0001cfbfc0 count:3 mapcount:1 mapping:ffff88007f1b5f51 index:0x600000aff > > > > > > > > > > > > > > flags: 0x4000000000048019(locked|uptodate|dirty|swapcache|swapbacked) > > > > > > > > > > > > > > page dumped because: VM_BUG_ON_PAGE(PageAnon(page) && !PageKsm(page) && !anon_vma) > > > > > > > > > > > > > > > > > > > > > > > > > > That's interesting, that's one I added in my page migration series. > > > > > > > > > > > > > Let me think on it, but it could well relate to the one you got before. > > > > > > > > > > > > > > > > > > > > > > > > I will roll back to mm/madv_free-v4.3-rc5-mmotm-2015-10-15-15-20 > > > > > > > > > > > > instead of next-20151021 to remove noise from your migration cleanup > > > > > > > > > > > > series and will test it again. > > > > > > > > > > > > If it is fixed, I will test again with your migration patchset, then. > > > > > > > > > > > > > > > > > > > > > > I tested mmotm-2015-10-15-15-20 with test program I attach for a long time. > > > > > > > > > > > Therefore, there is no patchset from Hugh's migration patch in there. > > > > > > > > > > > And I added below debug code with request from Kirill to all test kernels. > > > > > > > > > > > > > > > > > > > > It took too long time (and a lot of printk()), but I think I track it down > > > > > > > > > > finally. > > > > > > > > > > > > > > > > > > > > The patch below seems fixes issue for me. It's not yet properly tested, but > > > > > > > > > > looks like it works. > > > > > > > > > > > > > > > > > > > > The problem was my wrong assumption on how migration works: I thought that > > > > > > > > > > kernel would wait migration to finish on before deconstruction mapping. > > > > > > > > > > > > > > > > > > > > But turn out that's not true. > > > > > > > > > > > > > > > > > > > > As result if zap_pte_range() races with split_huge_page(), we can end up > > > > > > > > > > with page which is not mapped anymore but has _count and _mapcount > > > > > > > > > > elevated. The page is on LRU too. So it's still reachable by vmscan and by > > > > > > > > > > pfn scanners (Sasha showed few similar traces from compaction too). > > > > > > > > > > It's likely that page->mapping in this case would point to freed anon_vma. > > > > > > > > > > > > > > > > > > > > BOOM! > > > > > > > > > > > > > > > > > > > > The patch modify freeze/unfreeze_page() code to match normal migration > > > > > > > > > > entries logic: on setup we remove page from rmap and drop pin, on removing > > > > > > > > > > we get pin back and put page on rmap. This way even if migration entry > > > > > > > > > > will be removed under us we don't corrupt page's state. > > > > > > > > > > > > > > > > > > > > Please, test. > > > > > > > > > > > > > > > > > > > > > > > > > > > > kernel: On mmotm-2015-10-15-15-20 + pte_mkdirty patch + your new patch, I tested > > > > > > > > > one I sent to you(ie, oops.c + memcg_test.sh) > > > > > > > > > > > > > > > > > > page:ffffea00016a0000 count:3 mapcount:0 mapping:ffff88007f49d001 index:0x600001800 compound_mapcount: 0 > > > > > > > > > flags: 0x4000000000044009(locked|uptodate|head|swapbacked) > > > > > > > > > page dumped because: VM_BUG_ON_PAGE(!page_mapcount(page)) > > > > > > > > > page->mem_cgroup:ffff88007f613c00 > > > > > > > > > > > > > > > > Ignore my previous answer. Still sleeping. > > > > > > > > > > > > > > > > The right way to fix I think is something like: > > > > > > > > > > > > > > > > diff --git a/mm/rmap.c b/mm/rmap.c > > > > > > > > index 35643176bc15..f2d46792a554 100644 > > > > > > > > --- a/mm/rmap.c > > > > > > > > +++ b/mm/rmap.c > > > > > > > > @@ -1173,20 +1173,12 @@ void do_page_add_anon_rmap(struct page *page, > > > > > > > > bool compound = flags & RMAP_COMPOUND; > > > > > > > > bool first; > > > > > > > > > > > > > > > > - if (PageTransCompound(page)) { > > > > > > > > + if (PageTransCompound(page) && compound) { > > > > > > > > + atomic_t *mapcount; > > > > > > > > VM_BUG_ON_PAGE(!PageLocked(page), page); > > > > > > > > - if (compound) { > > > > > > > > - atomic_t *mapcount; > > > > > > > > - > > > > > > > > - VM_BUG_ON_PAGE(!PageTransHuge(page), page); > > > > > > > > - mapcount = compound_mapcount_ptr(page); > > > > > > > > - first = atomic_inc_and_test(mapcount); > > > > > > > > - } else { > > > > > > > > - /* Anon THP always mapped first with PMD */ > > > > > > > > - first = 0; > > > > > > > > - VM_BUG_ON_PAGE(!page_mapcount(page), page); > > > > > > > > - atomic_inc(&page->_mapcount); > > > > > > > > - } > > > > > > > > + VM_BUG_ON_PAGE(!PageTransHuge(page), page); > > > > > > > > + mapcount = compound_mapcount_ptr(page); > > > > > > > > + first = atomic_inc_and_test(mapcount); > > > > > > > > } else { > > > > > > > > VM_BUG_ON_PAGE(compound, page); > > > > > > > > first = atomic_inc_and_test(&page->_mapcount); > > > > > > > > -- > > > > > > > > > > > > > > kernel: On mmotm-2015-10-15-15-20 + pte_mkdirty patch + freeze/unfreeze patch + above patch, > > > > > > > > > > > > > > Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS > > > > > > > Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS > > > > > > > Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS > > > > > > > Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS > > > > > > > Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS > > > > > > > BUG: Bad rss-counter state mm:ffff880058d2e580 idx:1 val:512 > > > > > > > Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS > > > > > > > Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS > > > > > > > > > > > > > > <SNIP> > > > > > > > > > > > > > > Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS > > > > > > > Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS > > > > > > > Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS > > > > > > > BUG: Bad rss-counter state mm:ffff880046980700 idx:1 val:511 > > > > > > > BUG: Bad rss-counter state mm:ffff880046980700 idx:2 val:1 > > > > > > > > > > > > Hm. I was not able to trigger this and don't see anything obviuous what can > > > > > > lead to this kind of missmatch :-/ > > > > > > > > I managed to trigger this when switched back from MADV_DONTNEED to > > > > MADV_FREE. Hm.. > > > > > > Hmm,, > > > What version of MADV_FREE do you test on? > > > Old MADV_FREE(ie, before posting MADV_FREE refactoring and fix KSM page) > > > had a bug. > > > > > > I tried your patches on top of recent my MADV_FREE patches. > > > But when I try it with old THP refcount redesign, I couldn't find > > > any problem so far. However, I'm not saying it's your fault. > > > > > > I will give it a shot with MADV_DONTNEED to reproduce the problem. > > > But one thing I could say is MADV_DONTNEED is more hard to hit > > > compared to MADV_FREE because memory pressure of MADV_DONTNEED test > > > wouldn't be heavy. > > > > I reproduced this on the kernel which has no code related to MADV_FREE: > > > > mmotm-2015-10-15-15-20-no-madvise_free, IOW it means git head for > > 54bad5da4834 arm64: add pmd_[dirty|mkclean] for THP so there is no > > MADV_FREE code in there > > + pte_mkdirty patch > > + freeze/unfreeze patch > > + do_page_add_anon_rmap patch > > > > Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS > > BUG: Bad rss-counter state mm:ffff88007fdd5b00 idx:1 val:511 > > BUG: Bad rss-counter state mm:ffff88007fdd5b00 idx:2 val:1 > > I have one idea why it could happen, but not sure yet.. > > Could you check if it makes any difference for you? > > diff --git a/include/linux/huge_mm.h b/include/linux/huge_mm.h > index 5c7b00e88236..194f7f8b8c66 100644 > --- a/include/linux/huge_mm.h > +++ b/include/linux/huge_mm.h > @@ -103,12 +103,7 @@ void deferred_split_huge_page(struct page *page); > void __split_huge_pmd(struct vm_area_struct *vma, pmd_t *pmd, > unsigned long address); > > -#define split_huge_pmd(__vma, __pmd, __address) \ > - do { \ > - pmd_t *____pmd = (__pmd); \ > - if (pmd_trans_huge(*____pmd)) \ > - __split_huge_pmd(__vma, __pmd, __address); \ > - } while (0) > +#define split_huge_pmd(__vma, __pmd, __address) __split_huge_pmd(__vma, __pmd, __address) mmotm-2015-10-15-15-20-no-madvise_free, IOW it means git head for 54bad5da4834 arm64: add pmd_[dirty|mkclean] for THP so there is no MADV_FREE code in there + pte_mkdirty patch + freeze/unfreeze patch + do_page_add_anon_rmap patch + above split_huge_pmd Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS BUG: Bad rss-counter state mm:ffff88007fa3bb80 idx:1 val:512 Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS Adding 4191228k swap on /dev/vda5. Priority:-1 extents:1 across:4191228k FS BUG: unable to handle kernel NULL pointer dereference at 0000000000000008 IP: [<ffffffff810782a9>] down_read_trylock+0x9/0x30 PGD 0 Oops: 0000 [#1] SMP Dumping ftrace buffer: (ftrace buffer empty) Modules linked in: CPU: 11 PID: 59 Comm: khugepaged Not tainted 4.3.0-rc5-mm1-no-madv-free+ #2 Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/2011 task: ffff8800b9851a40 ti: ffff8800b985c000 task.ti: ffff8800b985c000 RIP: 0010:[<ffffffff810782a9>] [<ffffffff810782a9>] down_read_trylock+0x9/0x30 RSP: 0018:ffff8800b985f778 EFLAGS: 00010202 RAX: 0000000000000001 RBX: ffffea0000154b80 RCX: ffff8800b985f918 RDX: 0000000000000000 RSI: ffff8800b985f818 RDI: 0000000000000008 RBP: ffff8800b985f778 R08: ffffffff818446a0 R09: ffff8800b903cff8 R10: ffff8800b903d168 R11: ffff8800b985f7b8 R12: ffff88007ef6c731 R13: ffff88007ef6c730 R14: 0000000000000008 R15: 0000000000000001 FS: 0000000000000000(0000) GS:ffff8800bfb60000(0000) knlGS:0000000000000000 CS: 0010 DS: 0000 ES: 0000 CR0: 000000008005003b CR2: 0000000000000008 CR3: 0000000001808000 CR4: 00000000000006a0 Stack: ffff8800b985f7a8 ffffffff81124f20 ffffea0000154b80 ffff8800b985f818 ffff88007ff4dc00 0000000000000000 ffff8800b985f7f0 ffffffff81125663 0000000000000000 ffffffff818446a0 ffffea0000154b80 ffff8800b985f918 Call Trace: [<ffffffff81124f20>] page_lock_anon_vma_read+0x60/0x180 [<ffffffff81125663>] rmap_walk+0x1b3/0x3f0 [<ffffffff81125a43>] page_referenced+0x1a3/0x220 [<ffffffff81123e20>] ? __page_check_address+0x1a0/0x1a0 [<ffffffff81124ec0>] ? page_get_anon_vma+0xd0/0xd0 [<ffffffff81123810>] ? anon_vma_ctor+0x40/0x40 [<ffffffff8110086b>] shrink_page_list+0x5ab/0xde0 [<ffffffff8110173c>] shrink_inactive_list+0x18c/0x4b0 [<ffffffff811023ad>] shrink_lruvec+0x59d/0x740 [<ffffffff811025e0>] shrink_zone+0x90/0x250 [<ffffffff811028cd>] do_try_to_free_pages+0x12d/0x3b0 [<ffffffff81102d2d>] try_to_free_mem_cgroup_pages+0x9d/0x120 [<ffffffff81149949>] try_charge+0x1f9/0x670 [<ffffffff810fb030>] ? lru_cache_add_file+0x40/0x40 [<ffffffff8114d0a6>] mem_cgroup_try_charge+0x86/0x120 [<ffffffff811433bc>] khugepaged+0x7cc/0x1ac0 [<ffffffff81064f01>] ? __clear_sched_clock_stable+0x11/0x20 [<ffffffff81072430>] ? prepare_to_wait_event+0xf0/0xf0 [<ffffffff81142bf0>] ? __split_huge_pmd_locked+0x4a0/0x4a0 [<ffffffff81056cd9>] kthread+0xc9/0xe0 [<ffffffff81056c10>] ? kthread_park+0x60/0x60 [<ffffffff8142066f>] ret_from_fork+0x3f/0x70 [<ffffffff81056c10>] ? kthread_park+0x60/0x60 Code: 6e 7b 3a 00 48 83 c4 08 5b 5d c3 48 89 45 f0 e8 ab 63 3a 00 48 8b 45 f0 eb df 0f 1f 84 00 00 00 00 00 0f 1f 44 00 00 55 48 89 e5 <48> 8b 07 48 89 c2 48 83 c2 01 7e 07 f0 48 0f b1 17 75 f0 48 f7 -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>