Vlastimil pointed out that a check for isolation is redundant in
__free_one_page as compaction_capture checks for it.

This is a fix for the mmotm patch
mm-compaction-capture-a-page-under-direct-compaction.patch

Signed-off-by: Mel Gorman <[email protected]>
---
 mm/page_alloc.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index d61174bb0333..b2eee9b71986 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -887,8 +887,7 @@ static inline void __free_one_page(struct page *page,
 continue_merging:
        while (order < max_order - 1) {
                if (compaction_capture(capc, page, order, migratetype)) {
-                       if (likely(!is_migrate_isolate(migratetype)))
-                               __mod_zone_freepage_state(zone, -(1 << order),
+                       __mod_zone_freepage_state(zone, -(1 << order),
                                                                migratetype);
                        return;
                }

Reply via email to