Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751953AbbG2AdW (ORCPT ); Tue, 28 Jul 2015 20:33:22 -0400 Received: from mail-pd0-f177.google.com ([209.85.192.177]:34538 "EHLO mail-pd0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750791AbbG2AdV (ORCPT ); Tue, 28 Jul 2015 20:33:21 -0400 Date: Tue, 28 Jul 2015 17:33:18 -0700 (PDT) From: David Rientjes X-X-Sender: rientjes@chino.kir.corp.google.com To: Vlastimil Babka cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, Andrew Morton , Hugh Dickins , Andrea Arcangeli , "Kirill A. Shutemov" , Rik van Riel , Mel Gorman , Joonsoo Kim Subject: Re: [RFC 1/4] mm, compaction: introduce kcompactd In-Reply-To: <55B1DF11.8070100@suse.cz> Message-ID: References: <1435826795-13777-1-git-send-email-vbabka@suse.cz> <1435826795-13777-2-git-send-email-vbabka@suse.cz> <55AE0AFE.8070200@suse.cz> <55AFB569.90702@suse.cz> <55B0B175.9090306@suse.cz> <55B1DF11.8070100@suse.cz> User-Agent: Alpine 2.10 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3986 Lines: 83 On Fri, 24 Jul 2015, Vlastimil Babka wrote: > > Two issues I want to bring up: > > > > (1) do non-thp configs benefit from periodic compaction? > > > > In my experience, no, but perhaps there are other use cases where > > this has been a pain. The primary candidates, in my opinion, > > would be the networking stack and slub. Joonsoo reports having to > > workaround issues with high-order slub allocations being too > > expensive. I'm not sure that would be better served by periodic > > compaction, but it seems like a candidate for background compaction. > > Yes hopefully a proactive background compaction would serve them enough. > > > This is why my rfc tied periodic compaction to khugepaged, and we > > have strong evidence that this helps thp and cpu utilization. For > > periodic compaction to be possible outside of thp, we'd need a use > > case for it. > > > > (2) does kcompactd have to be per-node? > > > > I don't see the immediate benefit since direct compaction can > > already scan remote memory and migrate it, khugepaged can do the > > It can work remotely, but it's slower. > > > same. Is there evidence that suggests that a per-node kcompactd > > is significantly better than a single kthread? I think others > > would be more receptive of a single kthread addition. > > I think it's simpler design wrt waking up the kthread for the desired node, > and self-tuning any sleeping depending on per-node pressure. It also matches > the design of kswapd. And IMHO machines with many memory nodes should > naturally have also many CPU's to cope with the threads, so it should all > scale well. > I see your "proactive background compaction" as my "periodic compaction" :) And I agree with your comment that we should be careful about defining the API so it can be easily extended in the future. I see the two mechanisms different enough that they need to be defined separately: periodic compaction that would be done at certain intervals regardless of fragmentation or allocation failures to keep fragmentation low, and background compaction that would be done when a zone reaches a certain fragmentation index for high orders, similar to extfrag_threshold, or an allocation failure. Per-node kcompactd threads we agree would be optimal, so let's try to see if we can make that work. What do you think about the following? - add vm.compact_period_secs to define the number of seconds between full compactions on each node. This compaction would reset the pageblock skip heuristic and be synchronous. It would default to 900 based only on our evidence that 15m period compaction helps increase our cpu utilization for khugepaged; it is arbitrary and I'd happily change it if someone has a better suggestion. Changing it to 0 would disable periodic compaction (we don't anticipate anybody will ever want kcompactd threads will take 100% of cpu on each node). We can stagger this over all nodes to avoid all kcompactd threads working at the same time. - add vm.compact_background_extfrag_threshold to define the extfrag threshold when kcompactd should start doing sync_light migration in the background without resetting the pageblock skip heuristic. The threshold is defined at PAGE_ALLOC_COSTLY_ORDER and is halved for each order higher so that very high order allocations don't trigger it. To reduce overhead, this can be checked only in the slowpath. I'd also like to talk about compacting of mlocked memory and limit it to only periodic compaction so that we aren't constantly incurring minor faults when not expected. How does this sound? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/