Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932421Ab3CLHkw (ORCPT ); Tue, 12 Mar 2013 03:40:52 -0400 Received: from LGEMRELSE6Q.lge.com ([156.147.1.121]:50656 "EHLO LGEMRELSE6Q.lge.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932287Ab3CLHiz (ORCPT ); Tue, 12 Mar 2013 03:38:55 -0400 X-AuditID: 9c930179-b7c78ae000000e4b-45-513edb8c5aaa From: Minchan Kim To: linux-mm@kvack.org Cc: linux-kernel@vger.kernel.org, Michael Kerrisk , Arun Sharma , John Stultz , Mel Gorman , Hugh Dickins , Dave Hansen , Rik van Riel , Neil Brown , Mike Hommey , Taras Glek , KOSAKI Motohiro , KAMEZAWA Hiroyuki , Jason Evans , sanjay@google.com, Paul Turner , Johannes Weiner , Michel Lespinasse , Andrew Morton , Minchan Kim Subject: [RFC v7 05/11] Add purge operation Date: Tue, 12 Mar 2013 16:38:29 +0900 Message-Id: <1363073915-25000-6-git-send-email-minchan@kernel.org> X-Mailer: git-send-email 1.8.1.1 In-Reply-To: <1363073915-25000-1-git-send-email-minchan@kernel.org> References: <1363073915-25000-1-git-send-email-minchan@kernel.org> X-Brightmail-Tracker: AAAAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 16018 Lines: 588 This patch adds discarding function part. Logic is following as. 1. Memory pressure happens 2. VM start to reclaim anonymous page if system has a swap device 3. Check the page is in volatile range. 4. If so, zap the page from the process's page table. (By sematic vrange(2), we should mark it with another one to make page fault when you try to access the address. It will be introduced later patch) 5. If page is unmapped from all processes, discard it instead of swapping. Signed-off-by: Minchan Kim --- include/linux/rmap.h | 12 ++- include/linux/swap.h | 1 + include/linux/vrange.h | 9 ++ mm/ksm.c | 2 +- mm/rmap.c | 23 +++-- mm/swapfile.c | 36 ++++++++ mm/vmscan.c | 16 +++- mm/vrange.c | 235 +++++++++++++++++++++++++++++++++++++++++++++++++ 8 files changed, 320 insertions(+), 14 deletions(-) diff --git a/include/linux/rmap.h b/include/linux/rmap.h index 6dacb93..6432dfb 100644 --- a/include/linux/rmap.h +++ b/include/linux/rmap.h @@ -83,6 +83,8 @@ enum ttu_flags { }; #ifdef CONFIG_MMU +unsigned long vma_address(struct page *page, struct vm_area_struct *vma); + static inline void get_anon_vma(struct anon_vma *anon_vma) { atomic_inc(&anon_vma->refcount); @@ -182,9 +184,11 @@ static inline void page_dup_rmap(struct page *page) * Called from mm/vmscan.c to handle paging out */ int page_referenced(struct page *, int is_locked, - struct mem_cgroup *memcg, unsigned long *vm_flags); + struct mem_cgroup *memcg, unsigned long *vm_flags, + int *is_vrange); int page_referenced_one(struct page *, struct vm_area_struct *, - unsigned long address, unsigned int *mapcount, unsigned long *vm_flags); + unsigned long address, unsigned int *mapcount, unsigned long *vm_flags, + int *is_vrange); #define TTU_ACTION(x) ((x) & TTU_ACTION_MASK) @@ -249,9 +253,11 @@ int rmap_walk(struct page *page, int (*rmap_one)(struct page *, static inline int page_referenced(struct page *page, int is_locked, struct mem_cgroup *memcg, - unsigned long *vm_flags) + unsigned long *vm_flags, + int *is_vrange) { *vm_flags = 0; + *is_vrange = 0; return 0; } diff --git a/include/linux/swap.h b/include/linux/swap.h index 2818a12..bf56fb4 100644 --- a/include/linux/swap.h +++ b/include/linux/swap.h @@ -379,6 +379,7 @@ extern int swap_duplicate(swp_entry_t); extern int swapcache_prepare(swp_entry_t); extern void swap_free(swp_entry_t); extern void swapcache_free(swp_entry_t, struct page *page); +extern int __free_swap_and_cache(swp_entry_t); extern int free_swap_and_cache(swp_entry_t); extern int swap_type_of(dev_t, sector_t, struct block_device **); extern unsigned int count_swap_pages(int, int); diff --git a/include/linux/vrange.h b/include/linux/vrange.h index 74b5e37..eb3f941 100644 --- a/include/linux/vrange.h +++ b/include/linux/vrange.h @@ -5,6 +5,9 @@ #include #include +/* To protect race with forker */ +static DECLARE_RWSEM(vrange_fork_lock); + struct vrange { struct interval_tree_node node; bool purged; @@ -34,6 +37,9 @@ static inline void vrange_unlock(struct mm_struct *mm) extern void exit_vrange(struct mm_struct *mm); void vrange_init(void); +int discard_vpage(struct page *page); +bool vrange_address(struct mm_struct *mm, unsigned long start, + unsigned long end); #else @@ -41,5 +47,8 @@ static inline void vrange_init(void) {}; static inline void mm_init_vrange(struct mm_struct *mm) {}; static inline void exit_vrange(struct mm_struct *mm); +static inline bool vrange_address(struct mm_struct *mm, unsigned long start, + unsigned long end) { return false; }; +static inline int discard_vpage(struct page *page) { return 0 }; #endif #endif /* _LINIUX_VRANGE_H */ diff --git a/mm/ksm.c b/mm/ksm.c index b6afe0c..debc20c 100644 --- a/mm/ksm.c +++ b/mm/ksm.c @@ -1932,7 +1932,7 @@ again: continue; referenced += page_referenced_one(page, vma, - rmap_item->address, &mapcount, vm_flags); + rmap_item->address, &mapcount, vm_flags, NULL); if (!search_new_forks || !mapcount) break; } diff --git a/mm/rmap.c b/mm/rmap.c index 807c96b..90cf51c 100644 --- a/mm/rmap.c +++ b/mm/rmap.c @@ -57,6 +57,8 @@ #include #include #include +#include +#include #include @@ -523,8 +525,7 @@ __vma_address(struct page *page, struct vm_area_struct *vma) return vma->vm_start + ((pgoff - vma->vm_pgoff) << PAGE_SHIFT); } -inline unsigned long -vma_address(struct page *page, struct vm_area_struct *vma) +unsigned long vma_address(struct page *page, struct vm_area_struct *vma) { unsigned long address = __vma_address(page, vma); @@ -662,7 +663,7 @@ int page_mapped_in_vma(struct page *page, struct vm_area_struct *vma) */ int page_referenced_one(struct page *page, struct vm_area_struct *vma, unsigned long address, unsigned int *mapcount, - unsigned long *vm_flags) + unsigned long *vm_flags, int *is_vrange) { struct mm_struct *mm = vma->vm_mm; int referenced = 0; @@ -724,6 +725,9 @@ int page_referenced_one(struct page *page, struct vm_area_struct *vma, referenced++; } pte_unmap_unlock(pte, ptl); + if (is_vrange && + vrange_address(mm, address, address + PAGE_SIZE -1)) + *is_vrange = 1; } (*mapcount)--; @@ -736,7 +740,8 @@ out: static int page_referenced_anon(struct page *page, struct mem_cgroup *memcg, - unsigned long *vm_flags) + unsigned long *vm_flags, + int *is_vrange) { unsigned int mapcount; struct anon_vma *anon_vma; @@ -761,7 +766,7 @@ static int page_referenced_anon(struct page *page, if (memcg && !mm_match_cgroup(vma->vm_mm, memcg)) continue; referenced += page_referenced_one(page, vma, address, - &mapcount, vm_flags); + &mapcount, vm_flags, is_vrange); if (!mapcount) break; } @@ -826,7 +831,7 @@ static int page_referenced_file(struct page *page, if (memcg && !mm_match_cgroup(vma->vm_mm, memcg)) continue; referenced += page_referenced_one(page, vma, address, - &mapcount, vm_flags); + &mapcount, vm_flags, NULL); if (!mapcount) break; } @@ -841,6 +846,7 @@ static int page_referenced_file(struct page *page, * @is_locked: caller holds lock on the page * @memcg: target memory cgroup * @vm_flags: collect encountered vma->vm_flags who actually referenced the page + * @is_vrange: the page in vrange of some process * * Quick test_and_clear_referenced for all mappings to a page, * returns the number of ptes which referenced the page. @@ -848,7 +854,8 @@ static int page_referenced_file(struct page *page, int page_referenced(struct page *page, int is_locked, struct mem_cgroup *memcg, - unsigned long *vm_flags) + unsigned long *vm_flags, + int *is_vrange) { int referenced = 0; int we_locked = 0; @@ -867,7 +874,7 @@ int page_referenced(struct page *page, vm_flags); else if (PageAnon(page)) referenced += page_referenced_anon(page, memcg, - vm_flags); + vm_flags, is_vrange); else if (page->mapping) referenced += page_referenced_file(page, memcg, vm_flags); diff --git a/mm/swapfile.c b/mm/swapfile.c index a1f7772..962024c 100644 --- a/mm/swapfile.c +++ b/mm/swapfile.c @@ -734,6 +734,42 @@ int try_to_free_swap(struct page *page) } /* + * It's almost same with free_swap_and_cache except page is already + * locked. + */ +int __free_swap_and_cache(swp_entry_t entry) +{ + struct swap_info_struct *p; + struct page *page = NULL; + + if (non_swap_entry(entry)) + return 1; + + p = swap_info_get(entry); + if (p) { + if (swap_entry_free(p, entry, 1) == SWAP_HAS_CACHE) { + page = find_get_page(swap_address_space(entry), + entry.val); + } + spin_unlock(&swap_lock); + } + + if (page) { + /* + * Not mapped elsewhere, or swap space full? Free it! + * Also recheck PageSwapCache now page is locked (above). + */ + if (PageSwapCache(page) && !PageWriteback(page) && + (!page_mapped(page) || vm_swap_full())) { + delete_from_swap_cache(page); + SetPageDirty(page); + } + page_cache_release(page); + } + return p != NULL; +} + +/* * Free the swap entry like above, but also try to * free the page cache entry if it is the last user. */ diff --git a/mm/vmscan.c b/mm/vmscan.c index 88c5fed..6ba4e8ea 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -42,6 +42,7 @@ #include #include #include +#include #include #include @@ -610,6 +611,7 @@ enum page_references { PAGEREF_RECLAIM, PAGEREF_RECLAIM_CLEAN, PAGEREF_KEEP, + PAGEREF_DISCARD, PAGEREF_ACTIVATE, }; @@ -618,9 +620,10 @@ static enum page_references page_check_references(struct page *page, { int referenced_ptes, referenced_page; unsigned long vm_flags; + int is_vrange = 0; referenced_ptes = page_referenced(page, 1, sc->target_mem_cgroup, - &vm_flags); + &vm_flags, &is_vrange); referenced_page = TestClearPageReferenced(page); /* @@ -630,6 +633,12 @@ static enum page_references page_check_references(struct page *page, if (vm_flags & VM_LOCKED) return PAGEREF_RECLAIM; + /* + * Bail out if the page is in vrange and try to discard. + */ + if (is_vrange) + return PAGEREF_DISCARD; + if (referenced_ptes) { if (PageSwapBacked(page)) return PAGEREF_ACTIVATE; @@ -768,6 +777,9 @@ static unsigned long shrink_page_list(struct list_head *page_list, goto activate_locked; case PAGEREF_KEEP: goto keep_locked; + case PAGEREF_DISCARD: + if (discard_vpage(page)) + goto free_it; case PAGEREF_RECLAIM: case PAGEREF_RECLAIM_CLEAN: ; /* try to reclaim the page below */ @@ -1496,7 +1508,7 @@ static void shrink_active_list(unsigned long nr_to_scan, } if (page_referenced(page, 0, sc->target_mem_cgroup, - &vm_flags)) { + &vm_flags, NULL)) { nr_rotated += hpage_nr_pages(page); /* * Identify referenced, file-backed active pages and diff --git a/mm/vrange.c b/mm/vrange.c index f8b6f0e..78aa252 100644 --- a/mm/vrange.c +++ b/mm/vrange.c @@ -6,6 +6,13 @@ #include #include #include +#include +#include +#include +#include "internal.h" +#include +#include +#include static struct kmem_cache *vrange_cachep; @@ -213,3 +220,231 @@ SYSCALL_DEFINE4(vrange, unsigned long, start, out: return ret; } + +bool __vrange_address(struct mm_struct *mm, + unsigned long start, unsigned long end) +{ + struct rb_root *root = &mm->v_rb; + struct interval_tree_node *node; + + node = interval_tree_iter_first(root, start, end); + return node ? true : false; +} + +bool vrange_address(struct mm_struct *mm, + unsigned long start, unsigned long end) +{ + bool ret; + + vrange_lock(mm); + ret = __vrange_address(mm, start, end); + vrange_unlock(mm); + return ret; +} + +static pte_t *__vpage_check_address(struct page *page, + struct mm_struct *mm, unsigned long address, spinlock_t **ptlp) +{ + pmd_t *pmd; + pte_t *pte; + spinlock_t *ptl; + bool present; + + /* TODO : look into tlbfs */ + if (unlikely(PageHuge(page))) + return NULL; + + pmd = mm_find_pmd(mm, address); + if (!pmd) + return NULL; + /* + * TODO : Support THP + */ + if (pmd_trans_huge(*pmd)) + return NULL; + + pte = pte_offset_map_lock(mm, pmd, address, &ptl); + if (pte_none(*pte)) + goto out; + + present = pte_present(*pte); + if (present && page_to_pfn(page) != pte_pfn(*pte)) + goto out; + else if (present) { + *ptlp = ptl; + return pte; + } else { + swp_entry_t entry = { .val = page_private(page) }; + + VM_BUG_ON(non_swap_entry(entry)); + if (entry.val != pte_to_swp_entry(*pte).val) + goto out; + *ptlp = ptl; + return pte; + } +out: + pte_unmap_unlock(pte, ptl); + return NULL; +} + +/* + * This functions checks @page is matched with pte's encoded one + * which could be a page or swap slot. + */ +static inline pte_t *vpage_check_address(struct page *page, + struct mm_struct *mm, unsigned long address, + spinlock_t **ptlp) +{ + pte_t *ptep; + __cond_lock(*ptlp, ptep = __vpage_check_address(page, + mm, address, ptlp)); + return ptep; +} + +static void __vrange_purge(struct mm_struct *mm, + unsigned long start, unsigned long end) +{ + struct rb_root *root = &mm->v_rb; + struct vrange *range; + struct interval_tree_node *node; + + node = interval_tree_iter_first(root, start, end); + while (node) { + range = container_of(node, struct vrange, node); + range->purged = true; + node = interval_tree_iter_next(node, start, end); + } +} + +int try_to_discard_one(struct page *page, struct vm_area_struct *vma, + unsigned long address) +{ + struct mm_struct *mm = vma->vm_mm; + pte_t *pte; + pte_t pteval; + spinlock_t *ptl; + int ret = 0; + bool present; + + VM_BUG_ON(!PageLocked(page)); + + vrange_lock(mm); + pte = vpage_check_address(page, mm, address, &ptl); + if (!pte) { + vrange_unlock(mm); + goto out; + } + + if (vma->vm_flags & VM_LOCKED) { + pte_unmap_unlock(pte, ptl); + vrange_unlock(mm); + return 0; + } + + present = pte_present(*pte); + flush_cache_page(vma, address, page_to_pfn(page)); + pteval = ptep_clear_flush(vma, address, pte); + + update_hiwater_rss(mm); + dec_mm_counter(mm, MM_ANONPAGES); + + page_remove_rmap(page); + page_cache_release(page); + if (!present) { + swp_entry_t entry = pte_to_swp_entry(*pte); + dec_mm_counter(mm, MM_SWAPENTS); + if (unlikely(!__free_swap_and_cache(entry))) + BUG_ON(1); + } + + pte_unmap_unlock(pte, ptl); + mmu_notifier_invalidate_page(mm, address); + ret = 1; + __vrange_purge(mm, address, address + PAGE_SIZE -1); +out: + return ret; +} + +static int try_to_discard_vpage(struct page *page) +{ + struct anon_vma *anon_vma; + struct anon_vma_chain *avc; + pgoff_t pgoff; + struct vm_area_struct *vma; + struct mm_struct *mm; + unsigned long address; + bool ret = 0; + + anon_vma = page_lock_anon_vma_read(page); + if (!anon_vma) + return ret; + + pgoff = page->index << (PAGE_CACHE_SHIFT - PAGE_SHIFT); + anon_vma_interval_tree_foreach(avc, &anon_vma->rb_root, pgoff, pgoff) { + pte_t *pte; + spinlock_t *ptl; + + vma = avc->vma; + mm = vma->vm_mm; + address = vma_address(page, vma); + + vrange_lock(mm); + /* + * We can't use page_check_address because it doesn't check + * swap entry of the page table. We need the check because + * we have to make sure atomicity of shared vrange. + * It means all vranges which are shared a page should be + * purged if a page in a process is purged. + */ + pte = vpage_check_address(page, mm, address, &ptl); + if (!pte) { + vrange_unlock(mm); + continue; + } + + if (vma->vm_flags & VM_LOCKED) { + pte_unmap_unlock(pte, ptl); + vrange_unlock(mm); + goto out; + } + + pte_unmap_unlock(pte, ptl); + if (!__vrange_address(mm, address, + address + PAGE_SIZE - 1)) { + vrange_unlock(mm); + goto out; + } + + vrange_unlock(mm); + } + + anon_vma_interval_tree_foreach(avc, &anon_vma->rb_root, pgoff, pgoff) { + vma = avc->vma; + address = vma_address(page, vma); + if (!try_to_discard_one(page, vma, address)) + goto out; + } + + ret = 1; +out: + page_unlock_anon_vma_read(anon_vma); + return ret; +} + +int discard_vpage(struct page *page) +{ + VM_BUG_ON(!PageLocked(page)); + VM_BUG_ON(PageLRU(page)); + + if (try_to_discard_vpage(page)) { + if (PageSwapCache(page)) + try_to_free_swap(page); + + if (page_freeze_refs(page, 1)) { + unlock_page(page); + return 1; + } + } + + return 0; +} -- 1.8.1.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/