On Tue, Mar 03, 2015 at 10:34:37PM +1100, Dave Chinner wrote: > On Mon, Mar 02, 2015 at 10:56:14PM -0800, Linus Torvalds wrote: > > On Mon, Mar 2, 2015 at 9:20 PM, Dave Chinner <david@xxxxxxxxxxxxx> wrote: > > >> > > >> But are those migrate-page calls really common enough to make these > > >> things happen often enough on the same pages for this all to matter? > > > > > > It's looking like that's a possibility. > > > > Hmm. Looking closer, commit 10c1045f28e8 already should have > > re-introduced the "pte was already NUMA" case. > > > > So that's not it either, afaik. Plus your numbers seem to say that > > it's really "migrate_pages()" that is done more. So it feels like the > > numa balancing isn't working right. > > So that should show up in the vmstats, right? Oh, and there's a > tracepoint in migrate_pages, too. Same 6x10s samples in phase 3: > The stats indicate both more updates and more faults. Can you try this please? It's against 4.0-rc1. ---8<--- mm: numa: Reduce amount of IPI traffic due to automatic NUMA balancing Dave Chinner reported the following on https://lkml.org/lkml/2015/3/1/226 Across the board the 4.0-rc1 numbers are much slower, and the degradation is far worse when using the large memory footprint configs. Perf points straight at the cause - this is from 4.0-rc1 on the "-o bhash=101073" config: - 56.07% 56.07% [kernel] [k] default_send_IPI_mask_sequence_phys - default_send_IPI_mask_sequence_phys - 99.99% physflat_send_IPI_mask - 99.37% native_send_call_func_ipi smp_call_function_many - native_flush_tlb_others - 99.85% flush_tlb_page ptep_clear_flush try_to_unmap_one rmap_walk try_to_unmap migrate_pages migrate_misplaced_page - handle_mm_fault - 99.73% __do_page_fault trace_do_page_fault do_async_page_fault + async_page_fault 0.63% native_send_call_func_single_ipi generic_exec_single smp_call_function_single This was bisected to commit 4d94246699 ("mm: convert p[te|md]_mknonnuma and remaining page table manipulations") but I expect the full issue is related series up to and including that patch. There are two important changes that might be relevant here. The first is marking huge PMDs to trap a hinting fault potentially sends an IPI to flush TLBs. This did not show up in Dave's report and it almost certainly is not a factor but it would affect IPI counts for other users. The second is that the PTE protection update now clears the PTE leaving a window where parallel faults can be trapped resulting in more overhead from faults. Higher faults, even if correct can result in higher scan rates indirectly and may explain what Dave is saying. This is not signed off or tested. --- mm/huge_memory.c | 11 +++++++++-- mm/mprotect.c | 17 +++++++++++++++-- 2 files changed, 24 insertions(+), 4 deletions(-) diff --git a/mm/huge_memory.c b/mm/huge_memory.c index fc00c8cb5a82..7fc4732c77d7 100644 --- a/mm/huge_memory.c +++ b/mm/huge_memory.c @@ -1494,8 +1494,15 @@ int change_huge_pmd(struct vm_area_struct *vma, pmd_t *pmd, } if (!prot_numa || !pmd_protnone(*pmd)) { - ret = 1; - entry = pmdp_get_and_clear_notify(mm, addr, pmd); + /* + * NUMA hinting update can avoid a clear and flush as + * it is not a functional correctness issue if access + * occurs after the update + */ + if (prot_numa) + entry = *pmd; + else + entry = pmdp_get_and_clear_notify(mm, addr, pmd); entry = pmd_modify(entry, newprot); ret = HPAGE_PMD_NR; set_pmd_at(mm, addr, pmd, entry); diff --git a/mm/mprotect.c b/mm/mprotect.c index 44727811bf4c..1efd03ffa0d8 100644 --- a/mm/mprotect.c +++ b/mm/mprotect.c @@ -77,19 +77,32 @@ static unsigned long change_pte_range(struct vm_area_struct *vma, pmd_t *pmd, pte_t ptent; /* - * Avoid trapping faults against the zero or KSM - * pages. See similar comment in change_huge_pmd. + * prot_numa does not clear the pte during protection + * update as asynchronous hardware updates are not + * a concern but unnecessary faults while the PTE is + * cleared is overhead. */ if (prot_numa) { struct page *page; page = vm_normal_page(vma, addr, oldpte); + + /* + * Avoid trapping faults against the zero or KSM + * pages. See similar comment in change_huge_pmd. + */ if (!page || PageKsm(page)) continue; /* Avoid TLB flush if possible */ if (pte_protnone(oldpte)) continue; + + ptent = *pte; + ptent = pte_modify(ptent, newprot); + set_pte_at(mm, addr, pte, ptent); + pages++; + continue; } ptent = ptep_modify_prot_start(mm, addr, pte); _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
