From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pf0-f197.google.com (mail-pf0-f197.google.com [209.85.192.197]) by kanga.kvack.org (Postfix) with ESMTP id 0545B6B0006 for ; Mon, 16 Jul 2018 16:05:28 -0400 (EDT) Received: by mail-pf0-f197.google.com with SMTP id u18-v6so25741571pfh.21 for ; Mon, 16 Jul 2018 13:05:27 -0700 (PDT) Received: from mx1.suse.de (mx2.suse.de. [195.135.220.15]) by mx.google.com with ESMTPS id r138-v6si36384774pfc.202.2018.07.16.13.05.26 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 16 Jul 2018 13:05:26 -0700 (PDT) Date: Mon, 16 Jul 2018 22:05:17 +0200 From: Michal Hocko Subject: Re: [PATCH v1 00/10] mm: online/offline 4MB chunks controlled by device driver Message-ID: <20180716200517.GA16803@dhcp22.suse.cz> References: <20180524093121.GZ20441@dhcp22.suse.cz> <20180524120341.GF20441@dhcp22.suse.cz> <1a03ac4e-9185-ce8e-a672-c747c3e40ff2@redhat.com> <20180524142241.GJ20441@dhcp22.suse.cz> <819e45c5-6ae3-1dff-3f1d-c0411b6e2e1d@redhat.com> <3748f033-f349-6d88-d189-d77c76565981@redhat.com> <20180611115641.GL13364@dhcp22.suse.cz> <71bd1b65-2a88-5de7-9789-bf4fac26507d@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org List-ID: To: David Hildenbrand Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, Alexander Potapenko , Andrew Morton , Andrey Ryabinin , Balbir Singh , Baoquan He , Benjamin Herrenschmidt , Boris Ostrovsky , Dan Williams , Dave Young , Dmitry Vyukov , Greg Kroah-Hartman , Hari Bathini , Huang Ying , Hugh Dickins , Ingo Molnar , Jaewon Kim , Jan Kara , =?iso-8859-1?B?Suly9G1l?= Glisse , Joonsoo Kim , Juergen Gross , Kate Stewart , "Kirill A. Shutemov" , Matthew Wilcox , Mel Gorman , Michael Ellerman , Miles Chen , Oscar Salvador , Paul Mackerras , Pavel Tatashin , Philippe Ombredanne , Rashmica Gupta , Reza Arbab , Souptick Joarder , Tetsuo Handa , Thomas Gleixner , Vlastimil Babka On Mon 16-07-18 21:48:59, David Hildenbrand wrote: > On 11.06.2018 14:33, David Hildenbrand wrote: > > On 11.06.2018 13:56, Michal Hocko wrote: > >> On Mon 11-06-18 13:53:49, David Hildenbrand wrote: > >>> On 24.05.2018 23:07, David Hildenbrand wrote: > >>>> On 24.05.2018 16:22, Michal Hocko wrote: > >>>>> I will go over the rest of the email later I just wanted to make this > >>>>> point clear because I suspect we are talking past each other. > >>>> > >>>> It sounds like we are now talking about how to solve the problem. I like > >>>> that :) > >>>> > >>> > >>> Hi Michal, > >>> > >>> did you have time to think about the details of your proposed idea? > >> > >> Not really. Sorry about that. It's been busy time. I am planning to > >> revisit after merge window closes. > >> > > > > Sure no worries, I still have a bunch of other things to work on. But it > > would be nice to clarify soon in which direction I have to head to get > > this implemented and upstream (e.g. what I proposed, what you proposed > > or maybe something different). > > > I would really like to make progress here. > > I pointed out basic problems/questions with the proposed alternative. I > think I answered all your questions. But you also said that you are not > going to accept the current approach. So some decision has to be made. > > Although it's very demotivating and frustrating (I hope not all work in > the MM area will be like this), if there is no guidance on how to > proceed, I'll have to switch to adding/removing/onlining/offlining whole > segments. This is not what I want, but maybe this has a higher chance of > getting reviews/acks. > > Understanding that you are busy, please if you make suggestions, follow > up on responses. I plan to get back to this. It's busy time with too many things happening both upstream and on my work table as well. Sorry about that. I do understand your frustration but there is only that much time I have. There are not that many people to review this code unfortunately. In principle though, I still maintain my position that the memory hotplug code is way too subtle to add more on top. Maybe the code can be reworked to be less section oriented but that will be a lot of work. If you _really_ need a smaller granularity I do not have a better suggestion than to emulate that on top of sections. I still have to go back to your last emails though. -- Michal Hocko SUSE Labs