From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f198.google.com (mail-wr0-f198.google.com [209.85.128.198]) by kanga.kvack.org (Postfix) with ESMTP id 5F86A6B0390 for ; Tue, 11 Apr 2017 06:47:16 -0400 (EDT) Received: by mail-wr0-f198.google.com with SMTP id z62so6653625wrc.0 for ; Tue, 11 Apr 2017 03:47:16 -0700 (PDT) Received: from mx2.suse.de (mx2.suse.de. [195.135.220.15]) by mx.google.com with ESMTPS id k12si25528736wrk.201.2017.04.11.03.47.14 for (version=TLS1 cipher=AES128-SHA bits=128/128); Tue, 11 Apr 2017 03:47:15 -0700 (PDT) Date: Tue, 11 Apr 2017 12:47:10 +0200 From: Michal Hocko Subject: Re: [PATCH -v2 0/9] mm: make movable onlining suck less Message-ID: <20170411104710.GF6729@dhcp22.suse.cz> References: <20170410110351.12215-1-mhocko@kernel.org> <20170410162749.7d7f31c1@nial.brq.redhat.com> <20170410145639.GE4618@dhcp22.suse.cz> <20170411100152.6b4be896@nial.brq.redhat.com> <20170411084142.GB6729@dhcp22.suse.cz> <20170411115322.669c435d@nial.brq.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170411115322.669c435d@nial.brq.redhat.com> Sender: owner-linux-mm@kvack.org List-ID: To: Igor Mammedov Cc: linux-mm@kvack.org, Andrew Morton , Mel Gorman , Vlastimil Babka , Andrea Arcangeli , Jerome Glisse , Reza Arbab , Yasuaki Ishimatsu , qiuxishi@huawei.com, Kani Toshimitsu , slaoub@gmail.com, Joonsoo Kim , Andi Kleen , David Rientjes , Daniel Kiper , Vitaly Kuznetsov , LKML , Dan Williams , Heiko Carstens , Lai Jiangshan , Martin Schwidefsky , Tobias Regnery On Tue 11-04-17 11:53:22, Igor Mammedov wrote: > On Tue, 11 Apr 2017 10:41:42 +0200 > Michal Hocko wrote: > > > On Tue 11-04-17 10:01:52, Igor Mammedov wrote: > > > On Mon, 10 Apr 2017 16:56:39 +0200 > > > Michal Hocko wrote: > > [...] > > > > > #echo online_kernel > memory32/state > > > > > write error: Invalid argument > > > > > // that's not what's expected > > > > > > > > this is proper behavior with the current implementation. Does anything > > > > depend on the zone reusing? > > > if we didn't have zone imbalance issue in design, > > > the it wouldn't matter but as it stands it's not > > > minore issue. > > > > > > Consider following, > > > one hotplugs some memory and onlines it as movable, > > > then one needs to hotplug some more but to do so > > > one one needs more memory from zone NORMAL and to keep > > > zone balance some memory in MOVABLE should be reonlined > > > as NORMAL > > > > Is this something that we absolutely have to have right _now_? Or are you > > OK if I address this in follow up series? Because it will make the > > current code slightly more complex and to be honest I would rather like > > to see this "core" merge and build more on top. > > It's fine by me to do it on top. OK, I will document this in the changelog of the patch 6. " Please note that this patch also changes the original behavior when offlining a memory block adjacent to another zone (Normal vs. Movable) used to allow to change its movable type. This will be handled later. " -- Michal Hocko SUSE Labs -- 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