swapin_walk_pmd_entry() is defined as pmd_entry(), but it has no code about pmd handling (except pmd_none_or_trans_huge_or_clear_bad, but the same check are now done in core page table walk code). So let's move this function on pte_entry() as swapin_walk_pte_entry(). Signed-off-by: Naoya Horiguchi <n-horiguchi@xxxxxxxxxxxxx> --- mm/madvise.c | 43 +++++++++++++------------------------------ 1 file changed, 13 insertions(+), 30 deletions(-) diff --git v3.14-rc2.orig/mm/madvise.c v3.14-rc2/mm/madvise.c index 539eeb96b323..5e957b984c14 100644 --- v3.14-rc2.orig/mm/madvise.c +++ v3.14-rc2/mm/madvise.c @@ -135,38 +135,22 @@ static long madvise_behavior(struct vm_area_struct *vma, } #ifdef CONFIG_SWAP -static int swapin_walk_pmd_entry(pmd_t *pmd, unsigned long start, +static int swapin_walk_pte_entry(pte_t *pte, unsigned long start, unsigned long end, struct mm_walk *walk) { - pte_t *orig_pte; - struct vm_area_struct *vma = walk->private; - unsigned long index; + swp_entry_t entry; + struct page *page; + struct vm_area_struct *vma = walk->vma; - if (pmd_none_or_trans_huge_or_clear_bad(pmd)) + if (pte_present(*pte) || pte_none(*pte) || pte_file(*pte)) return 0; - - for (index = start; index != end; index += PAGE_SIZE) { - pte_t pte; - swp_entry_t entry; - struct page *page; - spinlock_t *ptl; - - orig_pte = pte_offset_map_lock(vma->vm_mm, pmd, start, &ptl); - pte = *(orig_pte + ((index - start) / PAGE_SIZE)); - pte_unmap_unlock(orig_pte, ptl); - - if (pte_present(pte) || pte_none(pte) || pte_file(pte)) - continue; - entry = pte_to_swp_entry(pte); - if (unlikely(non_swap_entry(entry))) - continue; - - page = read_swap_cache_async(entry, GFP_HIGHUSER_MOVABLE, - vma, index); - if (page) - page_cache_release(page); - } - + entry = pte_to_swp_entry(*pte); + if (unlikely(non_swap_entry(entry))) + return 0; + page = read_swap_cache_async(entry, GFP_HIGHUSER_MOVABLE, + vma, start); + if (page) + page_cache_release(page); return 0; } @@ -175,8 +159,7 @@ static void force_swapin_readahead(struct vm_area_struct *vma, { struct mm_walk walk = { .mm = vma->vm_mm, - .pmd_entry = swapin_walk_pmd_entry, - .private = vma, + .pte_entry = swapin_walk_pte_entry, }; walk_page_range(start, end, &walk); -- 1.8.5.3 -- 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>