diff options
| author | Rik van Riel <riel@redhat.com> | 2012-03-24 10:26:21 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-03-24 12:18:32 -0700 |
| commit | 496b919b3bdd957d4b1727df79bfa3751bced1c1 (patch) | |
| tree | 5585608ebdce5a7eca8f5ef334e6b167fcd5aab1 /mm/vmscan.c | |
| parent | 250f6715a4112d6686670c5a62ceb9305da94616 (diff) | |
| download | op-kernel-dev-496b919b3bdd957d4b1727df79bfa3751bced1c1.zip op-kernel-dev-496b919b3bdd957d4b1727df79bfa3751bced1c1.tar.gz | |
Fix potential endless loop in kswapd when compaction is not enabled
We should only test compaction_suitable if the kernel is built with
CONFIG_COMPACTION, otherwise the stub compaction_suitable function will
always return COMPACT_SKIPPED and send kswapd into an infinite loop.
Reported-by: Anton Blanchard <anton@samba.org>
Signed-off-by: Rik van Riel <riel@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/vmscan.c')
| -rw-r--r-- | mm/vmscan.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/mm/vmscan.c b/mm/vmscan.c index 7658fd6..33c332b 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -2946,7 +2946,8 @@ out: continue; /* Would compaction fail due to lack of free memory? */ - if (compaction_suitable(zone, order) == COMPACT_SKIPPED) + if (COMPACTION_BUILD && + compaction_suitable(zone, order) == COMPACT_SKIPPED) goto loop_again; /* Confirm the zone is balanced for order-0 */ |
