diff options
author | Charan Teja Reddy <charante@codeaurora.org> | 2021-05-04 18:36:51 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2021-05-05 11:27:24 -0700 |
commit | 06dac2f467fe9269a433aa5056dd2ee1d20475e9 (patch) | |
tree | 95f275981034d644d9a329e8bfe00dd8c2dc39ff /mm/migrate.c | |
parent | ef4984384172e93cc95e0e8cd102536d67e8a787 (diff) | |
download | linux-06dac2f467fe9269a433aa5056dd2ee1d20475e9.tar.gz linux-06dac2f467fe9269a433aa5056dd2ee1d20475e9.tar.bz2 linux-06dac2f467fe9269a433aa5056dd2ee1d20475e9.zip |
mm: compaction: update the COMPACT[STALL|FAIL] events properly
By definition, COMPACT[STALL|FAIL] events needs to be counted when there
is 'At least in one zone compaction wasn't deferred or skipped from the
direct compaction'. And when compaction is skipped or deferred,
COMPACT_SKIPPED will be returned but it will still go and update these
compaction events which is wrong in the sense that COMPACT[STALL|FAIL]
is counted without even trying the compaction.
Correct this by skipping the counting of these events when
COMPACT_SKIPPED is returned for compaction. This indirectly also avoid
the unnecessary try into the get_page_from_freelist() when compaction is
not even tried.
There is a corner case where compaction is skipped but still count
COMPACTSTALL event, which is that IRQ came and freed the page and the
same is captured in capture_control.
Link: https://lkml.kernel.org/r/1613151184-21213-1-git-send-email-charante@codeaurora.org
Signed-off-by: Charan Teja Reddy <charante@codeaurora.org>
Acked-by: Vlastimil Babka <vbabka@suse.cz>
Acked-by: David Rientjes <rientjes@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/migrate.c')
0 files changed, 0 insertions, 0 deletions