From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-f46.google.com (mail-pa0-f46.google.com [209.85.220.46]) by kanga.kvack.org (Postfix) with ESMTP id E7FB16B003A for ; Wed, 6 Aug 2014 03:11:21 -0400 (EDT) Received: by mail-pa0-f46.google.com with SMTP id lj1so2939847pab.33 for ; Wed, 06 Aug 2014 00:11:21 -0700 (PDT) Received: from lgemrelse6q.lge.com (LGEMRELSE6Q.lge.com. [156.147.1.121]) by mx.google.com with ESMTP id rz4si89813pab.184.2014.08.06.00.11.18 for ; Wed, 06 Aug 2014 00:11:19 -0700 (PDT) From: Joonsoo Kim Subject: [PATCH v2 3/8] mm/isolation: remove unstable check for isolated page Date: Wed, 6 Aug 2014 16:18:31 +0900 Message-Id: <1407309517-3270-6-git-send-email-iamjoonsoo.kim@lge.com> In-Reply-To: <1407309517-3270-1-git-send-email-iamjoonsoo.kim@lge.com> References: <1407309517-3270-1-git-send-email-iamjoonsoo.kim@lge.com> Sender: owner-linux-mm@kvack.org List-ID: To: Andrew Morton Cc: "Kirill A. Shutemov" , Rik van Riel , Mel Gorman , Johannes Weiner , Minchan Kim , Yasuaki Ishimatsu , Zhang Yanfei , "Srivatsa S. Bhat" , Tang Chen , Naoya Horiguchi , Bartlomiej Zolnierkiewicz , Wen Congyang , Marek Szyprowski , Michal Nazarewicz , Laura Abbott , Heesub Shin , "Aneesh Kumar K.V" , Ritesh Harjani , t.stanislaws@samsung.com, Gioh Kim , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Joonsoo Kim The check '!PageBuddy(page) && page_count(page) == 0 && migratetype == MIGRATE_ISOLATE' would mean the page on free processing. Although it could go into buddy allocator within a short time, futher operation such as isolate_freepages_range() in CMA, called after test_page_isolated_in_pageblock(), could be failed due to this unstability since it requires that the page is on buddy. I think that removing this unstability is good thing. And, following patch makes isolated freepage has new status matched with this condition and this check is the obstacle to that change. So remove it. Signed-off-by: Joonsoo Kim --- mm/page_isolation.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/mm/page_isolation.c b/mm/page_isolation.c index d1473b2..3100f98 100644 --- a/mm/page_isolation.c +++ b/mm/page_isolation.c @@ -198,11 +198,7 @@ __test_page_isolated_in_pageblock(unsigned long pfn, unsigned long end_pfn, MIGRATE_ISOLATE); } pfn += 1 << page_order(page); - } - else if (page_count(page) == 0 && - get_freepage_migratetype(page) == MIGRATE_ISOLATE) - pfn += 1; - else if (skip_hwpoisoned_pages && PageHWPoison(page)) { + } else if (skip_hwpoisoned_pages && PageHWPoison(page)) { /* * The HWPoisoned page may be not in buddy * system, and page_count() is not 0. -- 1.7.9.5 -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org