Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1033475AbbKEQQP (ORCPT ); Thu, 5 Nov 2015 11:16:15 -0500 Received: from shards.monkeyblade.net ([149.20.54.216]:36296 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1033092AbbKEQQM (ORCPT ); Thu, 5 Nov 2015 11:16:12 -0500 Date: Thu, 05 Nov 2015 11:16:09 -0500 (EST) Message-Id: <20151105.111609.1695015438589063316.davem@davemloft.net> To: mhocko@kernel.org Cc: hannes@cmpxchg.org, akpm@linux-foundation.org, vdavydov@virtuozzo.com, tj@kernel.org, netdev@vger.kernel.org, linux-mm@kvack.org, cgroups@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 5/8] mm: memcontrol: account socket memory on unified hierarchy From: David Miller In-Reply-To: <20151105144002.GB15111@dhcp22.suse.cz> References: <20151104104239.GG29607@dhcp22.suse.cz> <20151104195037.GA6872@cmpxchg.org> <20151105144002.GB15111@dhcp22.suse.cz> X-Mailer: Mew version 6.6 on Emacs 24.5 / Mule 6.0 (HANACHIRUSATO) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.12 (shards.monkeyblade.net [149.20.54.216]); Thu, 05 Nov 2015 08:16:12 -0800 (PST) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1250 Lines: 28 From: Michal Hocko Date: Thu, 5 Nov 2015 15:40:02 +0100 > On Wed 04-11-15 14:50:37, Johannes Weiner wrote: > [...] >> Because it goes without saying that once the cgroupv2 interface is >> released, and people use it in production, there is no way we can then >> *add* dentry cache, inode cache, and others to memory.current. That >> would be an unacceptable change in interface behavior. > > They would still have to _enable_ the config option _explicitly_. make > oldconfig wouldn't change it silently for them. I do not think > it is an unacceptable change of behavior if the config is changed > explicitly. Every user is going to get this config option when they update their distibution kernel or whatever. Then they will all wonder why their networking performance went down. This is why I do not want the networking accounting bits on by default even if the kconfig option is enabled. They must be off by default and guarded by a static branch so the cost is exactly zero. -- 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/