Subject: + mm-fix-tlb-flush-race-between-migration-and-change_protection_range-fix.patch added to -mm tree To: mgorman@xxxxxxx,athorlton@xxxxxxx From: akpm@xxxxxxxxxxxxxxxxxxxx Date: Wed, 11 Dec 2013 13:24:45 -0800 The patch titled Subject: mm-fix-tlb-flush-race-between-migration-and-change_protection_range-fix has been added to the -mm tree. Its filename is mm-fix-tlb-flush-race-between-migration-and-change_protection_range-fix.patch This patch should soon appear at http://ozlabs.org/~akpm/mmots/broken-out/mm-fix-tlb-flush-race-between-migration-and-change_protection_range-fix.patch and later at http://ozlabs.org/~akpm/mmotm/broken-out/mm-fix-tlb-flush-race-between-migration-and-change_protection_range-fix.patch Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/SubmitChecklist when testing your code *** The -mm tree is included into linux-next and is updated there every 3-4 working days ------------------------------------------------------ From: Mel Gorman <mgorman@xxxxxxx> Subject: mm-fix-tlb-flush-race-between-migration-and-change_protection_range-fix The following build error was reported by the 0-day build checker. >> arch/arm/mm/context.c:51:18: error: 'tlb_flush_pending' redeclared as different kind of symbol include/linux/mm_types.h:477:91: note: previous definition of 'tlb_flush_pending' was here This patch renames tlb_flush_pending to mm_tlb_flush_pending. This is a fix for the -mm patch mm-fix-tlb-flush-race-between-migration-and-change_protection_range.patch Note that when slotted into place that it will cause a conflict with mm-numa-defer-tlb-flush-for-thp-migration-as-long-as-possible.patch . The resolution is to delete the call from huge_memory.c and make sure the tlb_flush_pending call in mm/migrate.c is renamed appropriately. Signed-off-by: Mel Gorman <mgorman@xxxxxxx> Cc: Mel Gorman <mgorman@xxxxxxx> Cc: Alex Thorlton <athorlton@xxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- arch/x86/include/asm/pgtable.h | 2 +- include/linux/mm_types.h | 4 ++-- mm/huge_memory.c | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff -puN arch/x86/include/asm/pgtable.h~mm-fix-tlb-flush-race-between-migration-and-change_protection_range-fix arch/x86/include/asm/pgtable.h --- a/arch/x86/include/asm/pgtable.h~mm-fix-tlb-flush-race-between-migration-and-change_protection_range-fix +++ a/arch/x86/include/asm/pgtable.h @@ -458,7 +458,7 @@ static inline bool pte_accessible(struct return true; if ((pte_flags(a) & (_PAGE_PROTNONE | _PAGE_NUMA)) && - tlb_flush_pending(mm)) + mm_tlb_flush_pending(mm)) return true; return false; diff -puN include/linux/mm_types.h~mm-fix-tlb-flush-race-between-migration-and-change_protection_range-fix include/linux/mm_types.h --- a/include/linux/mm_types.h~mm-fix-tlb-flush-race-between-migration-and-change_protection_range-fix +++ a/include/linux/mm_types.h @@ -474,7 +474,7 @@ static inline cpumask_t *mm_cpumask(stru * The barriers below prevent the compiler from re-ordering the instructions * around the memory barriers that are already present in the code. */ -static inline bool tlb_flush_pending(struct mm_struct *mm) +static inline bool mm_tlb_flush_pending(struct mm_struct *mm) { barrier(); return mm->tlb_flush_pending; @@ -491,7 +491,7 @@ static inline void clear_tlb_flush_pendi mm->tlb_flush_pending = false; } #else -static inline bool tlb_flush_pending(struct mm_struct *mm) +static inline bool mm_tlb_flush_pending(struct mm_struct *mm) { return false; } diff -puN mm/huge_memory.c~mm-fix-tlb-flush-race-between-migration-and-change_protection_range-fix mm/huge_memory.c --- a/mm/huge_memory.c~mm-fix-tlb-flush-race-between-migration-and-change_protection_range-fix +++ a/mm/huge_memory.c @@ -1380,7 +1380,7 @@ int do_huge_pmd_numa_page(struct mm_stru * The page_table_lock above provides a memory barrier * with change_protection_range. */ - if (tlb_flush_pending(mm)) + if (mm_tlb_flush_pending(mm)) flush_tlb_range(vma, haddr, haddr + HPAGE_PMD_SIZE); /* _ Patches currently in -mm which might be from mgorman@xxxxxxx are mm-numa-serialise-parallel-get_user_page-against-thp-migration.patch mm-numa-call-mmu-notifiers-on-thp-migration.patch mm-clear-pmd_numa-before-invalidating.patch mm-numa-do-not-clear-pmd-during-pte-update-scan.patch mm-numa-do-not-clear-pte-for-pte_numa-update.patch mm-numa-ensure-anon_vma-is-locked-to-prevent-parallel-thp-splits.patch mm-numa-avoid-unnecessary-work-on-the-failure-path.patch sched-numa-skip-inaccessible-vmas.patch mm-numa-clear-numa-hinting-information-on-mprotect.patch mm-numa-avoid-unnecessary-disruption-of-numa-hinting-during-migration.patch mm-fix-tlb-flush-race-between-migration-and-change_protection_range.patch mm-fix-tlb-flush-race-between-migration-and-change_protection_range-fix.patch mm-numa-defer-tlb-flush-for-thp-migration-as-long-as-possible.patch mm-hugetlbfs-add-some-vm_bug_ons-to-catch-non-hugetlbfs-pages.patch mm-hugetlb-use-get_page_foll-in-follow_hugetlb_page.patch mm-hugetlbfs-move-the-put-get_page-slab-and-hugetlbfs-optimization-in-a-faster-path.patch mm-thp-optimize-compound_trans_huge.patch mm-tail-page-refcounting-optimization-for-slab-and-hugetlbfs.patch mm-hugetlbfs-use-__compound_tail_refcounted-in-__get_page_tail-too.patch mm-hugetlbc-simplify-pageheadhuge-and-pagehuge.patch mm-swapc-reorganize-put_compound_page.patch mm-hugetlbc-defer-pageheadhuge-symbol-export.patch mm-get-rid-of-unnecessary-pageblock-scanning-in-setup_zone_migrate_reserve.patch mm-get-rid-of-unnecessary-pageblock-scanning-in-setup_zone_migrate_reserve-fix.patch mm-call-mmu-notifiers-when-copying-a-hugetlb-page-range.patch mm-show_mem-remove-show_mem_filter_page_count.patch x86-get-pg_data_ts-memory-from-other-node.patch memblock-numa-introduce-flags-field-into-memblock.patch memblock-mem_hotplug-introduce-memblock_hotplug-flag-to-mark-hotpluggable-regions.patch memblock-make-memblock_set_node-support-different-memblock_type.patch acpi-numa-mem_hotplug-mark-hotpluggable-memory-in-memblock.patch acpi-numa-mem_hotplug-mark-all-nodes-the-kernel-resides-un-hotpluggable.patch memblock-mem_hotplug-make-memblock-skip-hotpluggable-regions-if-needed.patch x86-numa-acpi-memory-hotplug-make-movable_node-have-higher-priority.patch mm-rmap-recompute-pgoff-for-huge-page.patch mm-rmap-factor-nonlinear-handling-out-of-try_to_unmap_file.patch mm-rmap-factor-lock-function-out-of-rmap_walk_anon.patch mm-rmap-make-rmap_walk-to-get-the-rmap_walk_control-argument.patch mm-rmap-extend-rmap_walk_xxx-to-cope-with-different-cases.patch mm-rmap-use-rmap_walk-in-try_to_unmap.patch mm-rmap-use-rmap_walk-in-try_to_munlock.patch mm-rmap-use-rmap_walk-in-page_referenced.patch mm-rmap-use-rmap_walk-in-page_mkclean.patch mm-page_alloc-allow-__gfp_nofail-to-allocate-below-watermarks-after-reclaim.patch mm-numa-make-numa-migrate-related-functions-static.patch mm-numa-limit-scope-of-lock-for-numa-migrate-rate-limiting.patch mm-numa-trace-tasks-that-fail-migration-due-to-rate-limiting.patch mm-numa-do-not-automatically-migrate-ksm-pages.patch sched-add-tracepoints-related-to-numa-task-migration.patch sched-add-tracepoints-related-to-numa-task-migration-fix.patch linux-next.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html