From: Joonsoo Kim <iamjoonsoo.kim@lge.com> To: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com> Cc: Andrew Morton <akpm@linux-foundation.org>, "Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>, Rik van Riel <riel@redhat.com>, Mel Gorman <mgorman@suse.de>, Johannes Weiner <hannes@cmpxchg.org>, Minchan Kim <minchan@kernel.org>, Yasuaki Ishimatsu <isimatu.yasuaki@jp.fujitsu.com>, Zhang Yanfei <zhangyanfei@cn.fujitsu.com>, "Srivatsa S. Bhat" <srivatsa.bhat@linux.vnet.ibm.com>, Tang Chen <tangchen@cn.fujitsu.com>, Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>, Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>, Wen Congyang <wency@cn.fujitsu.com>, Marek Szyprowski <m.szyprowski@samsung.com>, Michal Nazarewicz <mina86@mina86.com>, Laura Abbott <lauraa@codeaurora.org>, Heesub Shin <heesub.shin@samsung.com>, Ritesh Harjani <ritesh.list@gmail.com>, t.stanislaws@samsung.com, Gioh Kim <gioh.kim@lge.com>, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 2/8] mm/isolation: remove unstable check for isolated page Date: Wed, 13 Aug 2014 17:19:21 +0900 [thread overview] Message-ID: <20140813081921.GC30451@js1304-P5Q-DELUXE> (raw) In-Reply-To: <87a97b5qi0.fsf@linux.vnet.ibm.com> On Mon, Aug 11, 2014 at 02:53:35PM +0530, Aneesh Kumar K.V wrote: > Joonsoo Kim <iamjoonsoo.kim@lge.com> writes: > > > 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. > > Is that true in case of check_pages_isolated_cb ? Does that require > PageBuddy to be true ? I think so. > > > > > 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. > > Can you quote the patch summary in the above case ? ie, something like > > And the followiing patch "mm/....." makes isolate freepage. > Okay. "mm/isolation: change pageblock isolation logic to fix freepage counting bugs" introduce PageIsolated() and mark freepages PageIsolated() during isolation. Those pages are !PageBuddy() and page_count() == 0. Thanks.
WARNING: multiple messages have this Message-ID (diff)
From: Joonsoo Kim <iamjoonsoo.kim@lge.com> To: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com> Cc: Andrew Morton <akpm@linux-foundation.org>, "Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>, Rik van Riel <riel@redhat.com>, Mel Gorman <mgorman@suse.de>, Johannes Weiner <hannes@cmpxchg.org>, Minchan Kim <minchan@kernel.org>, Yasuaki Ishimatsu <isimatu.yasuaki@jp.fujitsu.com>, Zhang Yanfei <zhangyanfei@cn.fujitsu.com>, "Srivatsa S. Bhat" <srivatsa.bhat@linux.vnet.ibm.com>, Tang Chen <tangchen@cn.fujitsu.com>, Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>, Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>, Wen Congyang <wency@cn.fujitsu.com>, Marek Szyprowski <m.szyprowski@samsung.com>, Michal Nazarewicz <mina86@mina86.com>, Laura Abbott <lauraa@codeaurora.org>, Heesub Shin <heesub.shin@samsung.com>, Ritesh Harjani <ritesh.list@gmail.com>, t.stanislaws@samsung.com, Gioh Kim <gioh.kim@lge.com>, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 2/8] mm/isolation: remove unstable check for isolated page Date: Wed, 13 Aug 2014 17:19:21 +0900 [thread overview] Message-ID: <20140813081921.GC30451@js1304-P5Q-DELUXE> (raw) In-Reply-To: <87a97b5qi0.fsf@linux.vnet.ibm.com> On Mon, Aug 11, 2014 at 02:53:35PM +0530, Aneesh Kumar K.V wrote: > Joonsoo Kim <iamjoonsoo.kim@lge.com> writes: > > > 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. > > Is that true in case of check_pages_isolated_cb ? Does that require > PageBuddy to be true ? I think so. > > > > > 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. > > Can you quote the patch summary in the above case ? ie, something like > > And the followiing patch "mm/....." makes isolate freepage. > Okay. "mm/isolation: change pageblock isolation logic to fix freepage counting bugs" introduce PageIsolated() and mark freepages PageIsolated() during isolation. Those pages are !PageBuddy() and page_count() == 0. Thanks. -- 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: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2014-08-13 8:19 UTC|newest] Thread overview: 84+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-08-06 7:18 [PATCH v2 0/8] fix freepage count problems in memory isolation Joonsoo Kim 2014-08-06 7:18 ` Joonsoo Kim 2014-08-06 7:18 ` [PATCH v2 1/8] mm/page_alloc: correct to clear guard attribute in DEBUG_PAGEALLOC Joonsoo Kim 2014-08-06 7:18 ` Joonsoo Kim 2014-08-07 1:46 ` Zhang Yanfei 2014-08-07 1:46 ` Zhang Yanfei 2014-08-06 7:18 ` [PATCH v2 1/8] mm/page_alloc: fix pcp high, batch management Joonsoo Kim 2014-08-06 7:18 ` Joonsoo Kim 2014-08-12 1:24 ` Minchan Kim 2014-08-12 1:24 ` Minchan Kim 2014-08-13 8:13 ` Joonsoo Kim 2014-08-13 8:13 ` Joonsoo Kim 2014-08-06 7:18 ` [PATCH v2 2/8] mm/isolation: remove unstable check for isolated page Joonsoo Kim 2014-08-06 7:18 ` Joonsoo Kim 2014-08-07 13:49 ` Vlastimil Babka 2014-08-07 13:49 ` Vlastimil Babka 2014-08-08 6:22 ` Joonsoo Kim 2014-08-08 6:22 ` Joonsoo Kim 2014-08-11 9:23 ` Aneesh Kumar K.V 2014-08-11 9:23 ` Aneesh Kumar K.V 2014-08-13 8:19 ` Joonsoo Kim [this message] 2014-08-13 8:19 ` Joonsoo Kim 2014-08-06 7:18 ` [PATCH v2 2/8] mm/page_alloc: correct to clear guard attribute in DEBUG_PAGEALLOC Joonsoo Kim 2014-08-06 7:18 ` Joonsoo Kim 2014-08-12 1:45 ` Minchan Kim 2014-08-12 1:45 ` Minchan Kim 2014-08-13 8:20 ` Joonsoo Kim 2014-08-13 8:20 ` Joonsoo Kim 2014-08-06 7:18 ` [PATCH v2 3/8] mm/isolation: remove unstable check for isolated page Joonsoo Kim 2014-08-06 7:18 ` Joonsoo Kim 2014-08-06 7:18 ` [PATCH v2 3/8] mm/page_alloc: fix pcp high, batch management Joonsoo Kim 2014-08-06 7:18 ` Joonsoo Kim 2014-08-07 2:11 ` Zhang Yanfei 2014-08-07 2:11 ` Zhang Yanfei 2014-08-07 8:23 ` Joonsoo Kim 2014-08-07 8:23 ` Joonsoo Kim 2014-08-06 7:18 ` [PATCH v2 4/8] mm/isolation: close the two race problems related to pageblock isolation Joonsoo Kim 2014-08-06 7:18 ` Joonsoo Kim 2014-08-07 14:34 ` Vlastimil Babka 2014-08-07 14:34 ` Vlastimil Babka 2014-08-08 6:30 ` Joonsoo Kim 2014-08-08 6:30 ` Joonsoo Kim 2014-08-12 5:17 ` Minchan Kim 2014-08-12 5:17 ` Minchan Kim 2014-08-12 9:45 ` Vlastimil Babka 2014-08-12 9:45 ` Vlastimil Babka 2014-08-13 8:09 ` Joonsoo Kim 2014-08-13 8:09 ` Joonsoo Kim 2014-08-13 8:29 ` Joonsoo Kim 2014-08-13 8:29 ` Joonsoo Kim 2014-08-06 7:18 ` [PATCH v2 5/8] mm/isolation: change pageblock isolation logic to fix freepage counting bugs Joonsoo Kim 2014-08-06 7:18 ` Joonsoo Kim 2014-08-06 15:12 ` Vlastimil Babka 2014-08-06 15:12 ` Vlastimil Babka 2014-08-07 8:19 ` Joonsoo Kim 2014-08-07 8:19 ` Joonsoo Kim 2014-08-07 8:53 ` Vlastimil Babka 2014-08-07 8:53 ` Vlastimil Babka 2014-08-07 12:26 ` Joonsoo Kim 2014-08-07 12:26 ` Joonsoo Kim 2014-08-07 13:04 ` Vlastimil Babka 2014-08-07 13:04 ` Vlastimil Babka 2014-08-07 13:35 ` Joonsoo Kim 2014-08-07 13:35 ` Joonsoo Kim 2014-08-07 15:15 ` Vlastimil Babka 2014-08-07 15:15 ` Vlastimil Babka 2014-08-08 6:45 ` Joonsoo Kim 2014-08-08 6:45 ` Joonsoo Kim 2014-08-12 6:43 ` Minchan Kim 2014-08-12 6:43 ` Minchan Kim 2014-08-12 10:58 ` Vlastimil Babka 2014-08-12 10:58 ` Vlastimil Babka 2014-08-06 7:18 ` [PATCH v2 6/8] mm/isolation: factor out pre/post logic on set/unset_migratetype_isolate() Joonsoo Kim 2014-08-06 7:18 ` Joonsoo Kim 2014-08-06 7:18 ` [PATCH v2 7/8] mm/isolation: fix freepage counting bug on start/undo_isolat_page_range() Joonsoo Kim 2014-08-06 7:18 ` Joonsoo Kim 2014-08-06 7:18 ` [PATCH v2 8/8] mm/isolation: remove useless race handling related to pageblock isolation Joonsoo Kim 2014-08-06 7:18 ` Joonsoo Kim 2014-08-06 7:25 ` [PATCH v2 0/8] fix freepage count problems in memory isolation Joonsoo Kim 2014-08-06 7:25 ` Joonsoo Kim 2014-08-07 0:49 ` Zhang Yanfei 2014-08-07 0:49 ` Zhang Yanfei 2014-08-07 8:20 ` Joonsoo Kim 2014-08-07 8:20 ` Joonsoo Kim
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20140813081921.GC30451@js1304-P5Q-DELUXE \ --to=iamjoonsoo.kim@lge.com \ --cc=akpm@linux-foundation.org \ --cc=aneesh.kumar@linux.vnet.ibm.com \ --cc=b.zolnierkie@samsung.com \ --cc=gioh.kim@lge.com \ --cc=hannes@cmpxchg.org \ --cc=heesub.shin@samsung.com \ --cc=isimatu.yasuaki@jp.fujitsu.com \ --cc=kirill.shutemov@linux.intel.com \ --cc=lauraa@codeaurora.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=m.szyprowski@samsung.com \ --cc=mgorman@suse.de \ --cc=mina86@mina86.com \ --cc=minchan@kernel.org \ --cc=n-horiguchi@ah.jp.nec.com \ --cc=riel@redhat.com \ --cc=ritesh.list@gmail.com \ --cc=srivatsa.bhat@linux.vnet.ibm.com \ --cc=t.stanislaws@samsung.com \ --cc=tangchen@cn.fujitsu.com \ --cc=wency@cn.fujitsu.com \ --cc=zhangyanfei@cn.fujitsu.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.