From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751220AbdJWFWL (ORCPT ); Mon, 23 Oct 2017 01:22:11 -0400 Received: from LGEAMRELO13.lge.com ([156.147.23.53]:58030 "EHLO lgeamrelo13.lge.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750745AbdJWFWK (ORCPT ); Mon, 23 Oct 2017 01:22:10 -0400 X-Original-SENDERIP: 156.147.1.127 X-Original-MAILFROM: iamjoonsoo.kim@lge.com X-Original-SENDERIP: 10.177.222.138 X-Original-MAILFROM: iamjoonsoo.kim@lge.com Date: Mon, 23 Oct 2017 14:26:02 +0900 From: Joonsoo Kim 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 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> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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.