From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f72.google.com (mail-wm0-f72.google.com [74.125.82.72]) by kanga.kvack.org (Postfix) with ESMTP id B04D46B0390 for ; Tue, 11 Apr 2017 04:41:48 -0400 (EDT) Received: by mail-wm0-f72.google.com with SMTP id 187so4074300wmn.5 for ; Tue, 11 Apr 2017 01:41:48 -0700 (PDT) Received: from mx2.suse.de (mx2.suse.de. [195.135.220.15]) by mx.google.com with ESMTPS id 29si25043983wrt.302.2017.04.11.01.41.47 for (version=TLS1 cipher=AES128-SHA bits=128/128); Tue, 11 Apr 2017 01:41:47 -0700 (PDT) Date: Tue, 11 Apr 2017 10:41:42 +0200 From: Michal Hocko Subject: Re: [PATCH -v2 0/9] mm: make movable onlining suck less Message-ID: <20170411084142.GB6729@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> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170411100152.6b4be896@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 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. -- 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