]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
mm: page_alloc: fix zone allocation fairness on UP
authorJohannes Weiner <hannes@cmpxchg.org>
Thu, 2 Oct 2014 23:21:10 +0000 (16:21 -0700)
committerJiri Slaby <jslaby@suse.cz>
Mon, 3 Nov 2014 09:50:11 +0000 (10:50 +0100)
commit28f04bcc068a44c5641c727883947960fb8dcbd5
tree81ad5c4b869e11df64846cf1b0ac3cd9a5d2e828
parent6eb2b7e3bd16f6e63fd028f6213d352281d9663e
mm: page_alloc: fix zone allocation fairness on UP

commit abe5f972912d086c080be4bde67750630b6fb38b upstream.

The zone allocation batches can easily underflow due to higher-order
allocations or spills to remote nodes.  On SMP that's fine, because
underflows are expected from concurrency and dealt with by returning 0.
But on UP, zone_page_state will just return a wrapped unsigned long,
which will get past the <= 0 check and then consider the zone eligible
until its watermarks are hit.

Commit 3a025760fc15 ("mm: page_alloc: spill to remote nodes before
waking kswapd") already made the counter-resetting use
atomic_long_read() to accomodate underflows from remote spills, but it
didn't go all the way with it.

Make it clear that these batches are expected to go negative regardless
of concurrency, and use atomic_long_read() everywhere.

Fixes: 81c0a2bb515f ("mm: page_alloc: fair zone allocator policy")
Reported-by: Vlastimil Babka <vbabka@suse.cz>
Reported-by: Leon Romanovsky <leon@leon.nu>
Signed-off-by: Johannes Weiner <hannes@cmpxchg.org>
Acked-by: Mel Gorman <mgorman@suse.de>
Cc: <stable@vger.kernel.org> [3.12+]
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
mm/page_alloc.c