aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMel Gorman <mgorman@suse.de>2013-10-07 06:29:06 -0400
committerIngo Molnar <mingo@kernel.org>2013-10-09 06:40:33 -0400
commit073b5beea735c7e1970686c94ff1f3aaac790a2a (patch)
treeba44188b0a4b52fddd1d0a7b9a6a214ea6212def
parent1bc115d87dffd1c43bdc3c9c9d1e3a51c195d18e (diff)
sched/numa: Remove check that skips small VMAs
task_numa_work skips small VMAs. At the time the logic was to reduce the scanning overhead which was considerable. It is a dubious hack at best. It would make much more sense to cache where faults have been observed and only rescan those regions during subsequent PTE scans. Remove this hack as motivation to do it properly in the future. Signed-off-by: Mel Gorman <mgorman@suse.de> Reviewed-by: Rik van Riel <riel@redhat.com> Cc: Andrea Arcangeli <aarcange@redhat.com> Cc: Johannes Weiner <hannes@cmpxchg.org> Cc: Srikar Dronamraju <srikar@linux.vnet.ibm.com> Signed-off-by: Peter Zijlstra <peterz@infradead.org> Link: http://lkml.kernel.org/r/1381141781-10992-29-git-send-email-mgorman@suse.de Signed-off-by: Ingo Molnar <mingo@kernel.org>
-rw-r--r--kernel/sched/fair.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index 3383079b1508..862d20d02e5c 100644
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -1127,10 +1127,6 @@ void task_numa_work(struct callback_head *work)
1127 if (!vma_migratable(vma)) 1127 if (!vma_migratable(vma))
1128 continue; 1128 continue;
1129 1129
1130 /* Skip small VMAs. They are not likely to be of relevance */
1131 if (vma->vm_end - vma->vm_start < HPAGE_SIZE)
1132 continue;
1133
1134 do { 1130 do {
1135 start = max(start, vma->vm_start); 1131 start = max(start, vma->vm_start);
1136 end = ALIGN(start + (pages << PAGE_SHIFT), HPAGE_SIZE); 1132 end = ALIGN(start + (pages << PAGE_SHIFT), HPAGE_SIZE);