From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pg0-f72.google.com (mail-pg0-f72.google.com [74.125.83.72]) by kanga.kvack.org (Postfix) with ESMTP id 87D4D6B0253 for ; Mon, 23 Oct 2017 01:22:11 -0400 (EDT) Received: by mail-pg0-f72.google.com with SMTP id b192so10991269pga.14 for ; Sun, 22 Oct 2017 22:22:11 -0700 (PDT) Received: from lgeamrelo13.lge.com (LGEAMRELO13.lge.com. [156.147.23.53]) by mx.google.com with ESMTP id i73si4406616pgc.409.2017.10.22.22.22.09 for ; Sun, 22 Oct 2017 22:22:10 -0700 (PDT) Date: Mon, 23 Oct 2017 14:26:02 +0900 From: Joonsoo Kim Subject: Re: [PATCH 1/2] mm: drop migrate type checks from has_unmovable_pages Message-ID: <20171023052602.GC23082@js1304-P5Q-DELUXE> References: <20171013115835.zaehapuucuzl2vlv@dhcp22.suse.cz> <20171013120013.698-1-mhocko@kernel.org> <20171019025111.GA3852@js1304-P5Q-DELUXE> <20171019071503.e7w5fo35lsq6ca54@dhcp22.suse.cz> <20171019073355.GA4486@js1304-P5Q-DELUXE> <20171019082041.5zudpqacaxjhe4gw@dhcp22.suse.cz> <20171019122118.y6cndierwl2vnguj@dhcp22.suse.cz> <20171020021329.GB10438@js1304-P5Q-DELUXE> <59E9A426.5070009@huawei.com> <20171020081700.fec53qxieeqouhwi@dhcp22.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171020081700.fec53qxieeqouhwi@dhcp22.suse.cz> Sender: owner-linux-mm@kvack.org List-ID: To: Michal Hocko Cc: Xishi Qiu , linux-mm@kvack.org, Michael Ellerman , Vlastimil Babka , Andrew Morton , KAMEZAWA Hiroyuki , Reza Arbab , Yasuaki Ishimatsu , Igor Mammedov , Vitaly Kuznetsov , LKML On Fri, Oct 20, 2017 at 10:17:00AM +0200, Michal Hocko wrote: > On Fri 20-10-17 15:22:14, Xishi Qiu wrote: > > On 2017/10/20 10:13, Joonsoo Kim wrote: > > > > > On Thu, Oct 19, 2017 at 02:21:18PM +0200, Michal Hocko wrote: > [...] > > >> >From 8cbd811d741f5dd93d1b21bb3ef94482a4d0bd32 Mon Sep 17 00:00:00 2001 > > >> From: Michal Hocko > > >> Date: Thu, 19 Oct 2017 14:14:02 +0200 > > >> Subject: [PATCH] mm: distinguish CMA and MOVABLE isolation in > > >> has_unmovable_pages > > >> > > >> Joonsoo has noticed that "mm: drop migrate type checks from > > >> has_unmovable_pages" would break CMA allocator because it relies on > > >> has_unmovable_pages returning false even for CMA pageblocks which in > > >> fact don't have to be movable: > > >> alloc_contig_range > > >> start_isolate_page_range > > >> set_migratetype_isolate > > >> has_unmovable_pages > > >> > > >> This is a result of the code sharing between CMA and memory hotplug > > >> while each one has a different idea of what has_unmovable_pages should > > >> return. This is unfortunate but fixing it properly would require a lot > > >> of code duplication. > > >> > > >> Fix the issue by introducing the requested migrate type argument > > >> and special case MIGRATE_CMA case where CMA page blocks are handled > > >> properly. This will work for memory hotplug because it requires > > >> MIGRATE_MOVABLE. > > > > > > Unfortunately, alloc_contig_range() can be called with > > > MIGRATE_MOVABLE so this patch cannot perfectly fix the problem. > > > > > > I did a more thinking and found that it's strange to check if there is > > > unmovable page in the pageblock during the set_migratetype_isolate(). > > > set_migratetype_isolate() should be just for setting the migratetype > > > of the pageblock. Checking other things should be done by another > > > place, for example, before calling the start_isolate_page_range() in > > > __offline_pages(). > > > > > > Thanks. > > > > > > > Hi Joonsoo, > > > > How about add a flag to skip or not has_unmovable_pages() in set_migratetype_isolate()? > > Something like the skip_hwpoisoned_pages. > > I believe this is what Joonsoo was proposing actually. I cannot say I Yes, I initially suggested this idea but change my mind. Now, I think that problem is not in has_unmovable_pages() but in set_migratetype_isolate(). So different solution is needed. See my other reply to Michal. 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: email@kvack.org