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 136556B0038 for ; Mon, 26 Dec 2016 19:53:42 -0500 (EST) Received: by mail-pg0-f72.google.com with SMTP id a190so683645348pgc.0 for ; Mon, 26 Dec 2016 16:53:42 -0800 (PST) Received: from mail-pg0-x22a.google.com (mail-pg0-x22a.google.com. [2607:f8b0:400e:c05::22a]) by mx.google.com with ESMTPS id e72si44654273pfl.77.2016.12.26.16.53.40 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 26 Dec 2016 16:53:40 -0800 (PST) Received: by mail-pg0-x22a.google.com with SMTP id y62so86886816pgy.1 for ; Mon, 26 Dec 2016 16:53:40 -0800 (PST) Date: Mon, 26 Dec 2016 16:53:39 -0800 (PST) From: David Rientjes Subject: Re: [patch] mm, thp: always direct reclaim for MADV_HUGEPAGE even when deferred In-Reply-To: <20161226090211.GA11455@dhcp22.suse.cz> Message-ID: References: <20161222100009.GA6055@dhcp22.suse.cz> <20161223085150.GA23109@dhcp22.suse.cz> <20161223111817.GC23109@dhcp22.suse.cz> <20161226090211.GA11455@dhcp22.suse.cz> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-linux-mm@kvack.org List-ID: To: Michal Hocko Cc: Andrew Morton , Jonathan Corbet , "Kirill A. Shutemov" , Vlastimil Babka , Mel Gorman , linux-kernel@vger.kernel.org, linux-mm@kvack.org On Mon, 26 Dec 2016, Michal Hocko wrote: > But my primary argument is that if you tweak "defer" value behavior > then you lose the only "stall free yet allow background compaction" > option. That option is really important. Important to who? What regresses if we kick a background kthread to compact memory for order-9 pageblocks? Why don't we allow userspace to clear __GFP_KSWAPD_RECLAIM if we don't want background reclaim for allocations? > You seem to think that it > is the application which is under the control. And I am not all that > surprised because you are under control of the whole userspace in your > deployments. I have no control over the userspace that runs on my "deployments," I caution you to not make any inferences. > But there are others where the administrator is not under > the control of what application asks for yet he is responsible for the > overal "experience" if you will. The administrator is in charge of an "experience" and wants to avoid background compaction for thp allocations but not background reclaim for any other allocation? (Why am I even replying to this?) If the admin is concerned about anybody doing compaction, they can set defrag to "never". They have had this ability since thp was introduced. > Long stalls during the page faults are > often seen as bugs and users might not really care whether the > application writer really wanted THP or not... > There are no long stalls during page faults introduced by this patch, we are waking up a kthread to do the work. > I definitely _agree_ that this is a very important usecase! I am just > trying to think long term and a more sophisticated background compaction > is something that we definitely lack and _want_ longterm. There are more > high order users than THP. I believe we really want to teach kcompactd > to maintain configurable amount of highorder pages. > We are addressing thp defrag here, not any other use for background compaction for other high-order allocations. I'd prefer that we stay on topic, please. This is only about setting thp defrag to "defer" and if it is possible to kick background compaction and defer direct compaction. We need this patch, Kirill has acked it, and I simply have no more time to talk in circles. > If there is really a need for an immediate solution^Wworkaround then I > think that tweaking the madvise option should be reasonably safe. Admins > are really prepared for stalls because they are explicitly opting in for > madvise behavior and they will get a background compaction on top. This > is a new behavior but I do not see how it would be harmful. If an > excessive compaction is a problem then THP can be reduced to madvise > only vmas. > > But, I really _do_ care about having a stall free option which is not a > complete disable of the background compaction for THP. > This is completely wrong. Before the "defer" option has been introduced, we had "madvise" and should maintain its behavior as much as possible so there are no surprises. We don't change behavior for a tunable out from under existing users because you think you know better. With the new "defer" option, we can make this a stronger variant of "madvise", which Kirill acked, so that existing users of MADV_HUGEPAGE have no change in behavior and we can configure whether we do direct or background compaction for everybody else. If people don't want background compaction, they can set defrag to "madvise". If they want it, they can set it to "defer". It's very simple. That said, I simply don't have the time to continue in circular arguments and would respectfully ask Andrew to apply this acked patch. -- 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