diff options
| author | Minchan Kim <minchan@kernel.org> | 2013-05-09 16:21:26 +0900 |
|---|---|---|
| committer | Kyle Yan <kyan@codeaurora.org> | 2016-06-22 14:43:32 -0700 |
| commit | a4e92011d44d60ad33dca31785682ddb82c44e40 (patch) | |
| tree | e6567ff2d985f976c33d9c8721c8cb595a546c04 /mm/vmscan.c | |
| parent | 7d65fdc5d72dbf1e933ca21eb6fddaf480efdb30 (diff) | |
mm: Remove shrink_page
By previous patch, shrink_page_list can handle pages from
multiple zone so let's remove shrink_page.
Change-Id: I3526377aa6ee6142b8f3ec63396e7ada1e442505
Signed-off-by: Minchan Kim <minchan@kernel.org>
Patch-mainline: linux-mm @ 22 Apr 2013 17:45:03
[vinmenon@codeaurora.org: trivial merge conflict fixes]
Signed-off-by: Vinayak Menon <vinmenon@codeaurora.org>
Diffstat (limited to 'mm/vmscan.c')
| -rw-r--r-- | mm/vmscan.c | 45 |
1 files changed, 13 insertions, 32 deletions
diff --git a/mm/vmscan.c b/mm/vmscan.c index b8aa077caf65..fe9c39e6b900 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -1248,6 +1248,13 @@ free_it: * appear not as the counts should be low */ list_add(&page->lru, &free_pages); + /* + * If pagelist are from multiple zones, we should decrease + * NR_ISOLATED_ANON + x on freed pages in here. + */ + if (!zone) + dec_zone_page_state(page, NR_ISOLATED_ANON + + page_is_file_cache(page)); continue; cull_mlocked: @@ -1317,28 +1324,6 @@ unsigned long reclaim_clean_pages_from_list(struct zone *zone, } #ifdef CONFIG_PROCESS_RECLAIM -static unsigned long shrink_page(struct page *page, - struct zone *zone, - struct scan_control *sc, - enum ttu_flags ttu_flags, - unsigned long *ret_nr_dirty, - unsigned long *ret_nr_writeback, - bool force_reclaim, - struct list_head *ret_pages) -{ - int reclaimed; - LIST_HEAD(page_list); - list_add(&page->lru, &page_list); - - reclaimed = shrink_page_list(&page_list, zone, sc, ttu_flags, - ret_nr_dirty, ret_nr_writeback, - force_reclaim); - if (!reclaimed) - list_splice(&page_list, ret_pages); - - return reclaimed; -} - unsigned long reclaim_pages_from_list(struct list_head *page_list) { struct scan_control sc = { @@ -1349,23 +1334,19 @@ unsigned long reclaim_pages_from_list(struct list_head *page_list) .may_swap = 1, }; - LIST_HEAD(ret_pages); + unsigned long nr_reclaimed; struct page *page; unsigned long dummy1, dummy2, dummy3, dummy4, dummy5; - unsigned long nr_reclaimed = 0; - - while (!list_empty(page_list)) { - page = lru_to_page(page_list); - list_del(&page->lru); + list_for_each_entry(page, page_list, lru) ClearPageActive(page); - nr_reclaimed += shrink_page(page, page_zone(page), &sc, + + nr_reclaimed = shrink_page_list(page_list, NULL, &sc, TTU_UNMAP|TTU_IGNORE_ACCESS, &dummy1, &dummy2, &dummy3, &dummy4, &dummy5, true); - } - while (!list_empty(&ret_pages)) { - page = lru_to_page(&ret_pages); + while (!list_empty(page_list)) { + page = lru_to_page(page_list); list_del(&page->lru); dec_zone_page_state(page, NR_ISOLATED_ANON + page_is_file_cache(page)); |
