diff options
author | Marek Szyprowski <m.szyprowski@samsung.com> | 2012-12-12 16:51:19 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-12-12 20:38:32 -0500 |
commit | be49a6e13537f32f85bd8c4ba04317ca36ca60ff (patch) | |
tree | 00f629b7be8203ce1fef100ab8875e291033dc5d /mm | |
parent | c8bf2d8ba4fbc093de7c0d192fe5d2531f14b8b9 (diff) |
mm: use migrate_prep() instead of migrate_prep_local()
__alloc_contig_migrate_range() should use all possible ways to get all the
pages migrated from the given memory range, so pruning per-cpu lru lists
for all CPUs is required, regadless the cost of such operation. Otherwise
some pages which got stuck at per-cpu lru list might get missed by
migration procedure causing the contiguous allocation to fail.
Reported-by: SeongHwan Yoon <sunghwan.yun@samsung.com>
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Acked-by: Michal Nazarewicz <mina86@mina86.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm')
-rw-r--r-- | mm/page_alloc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/page_alloc.c b/mm/page_alloc.c index 5a8d339d282a..4171cd4f8257 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c | |||
@@ -5727,7 +5727,7 @@ static int __alloc_contig_migrate_range(struct compact_control *cc, | |||
5727 | unsigned int tries = 0; | 5727 | unsigned int tries = 0; |
5728 | int ret = 0; | 5728 | int ret = 0; |
5729 | 5729 | ||
5730 | migrate_prep_local(); | 5730 | migrate_prep(); |
5731 | 5731 | ||
5732 | while (pfn < end || !list_empty(&cc->migratepages)) { | 5732 | while (pfn < end || !list_empty(&cc->migratepages)) { |
5733 | if (fatal_signal_pending(current)) { | 5733 | if (fatal_signal_pending(current)) { |