* [merged] mm-compaction-remove-redundant-watermark-check-in-compact_finished.patch removed from -mm tree
@ 2017-05-09 18:00 akpm
0 siblings, 0 replies; only message in thread
From: akpm @ 2017-05-09 18:00 UTC (permalink / raw)
To: hannes, iamjoonsoo.kim, mgorman, mm-commits, rientjes, vbabka
The patch titled
Subject: mm, compaction: remove redundant watermark check in compact_finished()
has been removed from the -mm tree. Its filename was
mm-compaction-remove-redundant-watermark-check-in-compact_finished.patch
This patch was dropped because it was merged into mainline or a subsystem tree
------------------------------------------------------
From: Vlastimil Babka <vbabka@suse.cz>
Subject: mm, compaction: remove redundant watermark check in compact_finished()
When detecting whether compaction has succeeded in forming a high-order
page, __compact_finished() employs a watermark check, followed by an own
search for a suitable page in the freelists. This is not ideal for two
reasons:
- The watermark check also searches high-order freelists, but has a less
strict criteria wrt fallback. It's therefore redundant and waste of
cycles. This was different in the past when high-order watermark check
attempted to apply reserves to high-order pages.
- The watermark check might actually fail due to lack of order-0 pages.
Compaction can't help with that, so there's no point in continuing
because of that. It's possible that high-order page still exists and it
terminates.
This patch therefore removes the watermark check. This should save some
cycles and terminate compaction sooner in some cases.
Link: http://lkml.kernel.org/r/20170307131545.28577-3-vbabka@suse.cz
Signed-off-by: Vlastimil Babka <vbabka@suse.cz>
Acked-by: Mel Gorman <mgorman@techsingularity.net>
Acked-by: Johannes Weiner <hannes@cmpxchg.org>
Cc: Joonsoo Kim <iamjoonsoo.kim@lge.com>
Cc: David Rientjes <rientjes@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---
mm/compaction.c | 8 --------
1 file changed, 8 deletions(-)
diff -puN mm/compaction.c~mm-compaction-remove-redundant-watermark-check-in-compact_finished mm/compaction.c
--- a/mm/compaction.c~mm-compaction-remove-redundant-watermark-check-in-compact_finished
+++ a/mm/compaction.c
@@ -1280,7 +1280,6 @@ static enum compact_result __compact_fin
const int migratetype)
{
unsigned int order;
- unsigned long watermark;
if (cc->contended || fatal_signal_pending(current))
return COMPACT_CONTENDED;
@@ -1308,13 +1307,6 @@ static enum compact_result __compact_fin
if (is_via_compact_memory(cc->order))
return COMPACT_CONTINUE;
- /* Compaction run is not finished if the watermark is not met */
- watermark = zone->watermark[cc->alloc_flags & ALLOC_WMARK_MASK];
-
- if (!zone_watermark_ok(zone, cc->order, watermark, cc->classzone_idx,
- cc->alloc_flags))
- return COMPACT_CONTINUE;
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2017-05-09 18:00 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-09 18:00 [merged] mm-compaction-remove-redundant-watermark-check-in-compact_finished.patch removed from -mm tree akpm
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).