diff options
| author | Vlastimil Babka <[email protected]> | 2017-05-08 15:54:33 -0700 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2017-05-08 17:15:09 -0700 |
| commit | 228d7e33903040a0b9dd9a5ee9b3a49c538c0613 (patch) | |
| tree | 17eb03bb7f86959fba7506b304e43b3c53ea56c8 /tools/perf/scripts/python/bin/stackcollapse-record | |
| parent | f25ba6dccc3bfe7e1524f4498a171be038507c45 (diff) | |
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/[email protected]
Signed-off-by: Vlastimil Babka <[email protected]>
Acked-by: Mel Gorman <[email protected]>
Acked-by: Johannes Weiner <[email protected]>
Cc: Joonsoo Kim <[email protected]>
Cc: David Rientjes <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/bin/stackcollapse-record')
0 files changed, 0 insertions, 0 deletions