From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751347AbcBHFqi (ORCPT ); Mon, 8 Feb 2016 00:46:38 -0500 Received: from gum.cmpxchg.org ([85.214.110.215]:57320 "EHLO gum.cmpxchg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750751AbcBHFqh (ORCPT ); Mon, 8 Feb 2016 00:46:37 -0500 Date: Mon, 8 Feb 2016 00:46:16 -0500 From: Johannes Weiner To: Vladimir Davydov Cc: Andrew Morton , Michal Hocko , linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/5] mm: memcontrol: enable kmem accounting for all cgroups in the legacy hierarchy Message-ID: <20160208054616.GA22202@cmpxchg.org> References: <5e6c9361f901fbfae84fe51ad1d27694d2377bd3.1454864628.git.vdavydov@virtuozzo.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5e6c9361f901fbfae84fe51ad1d27694d2377bd3.1454864628.git.vdavydov@virtuozzo.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, Feb 07, 2016 at 08:27:31PM +0300, Vladimir Davydov wrote: > Currently, in the legacy hierarchy kmem accounting is off for all > cgroups by default and must be enabled explicitly by writing something > to memory.kmem.limit_in_bytes. Since we don't support reclaim on hitting > kmem limit, nor do we have any plans to implement it, this is likely to > be -1, just to enable kmem accounting and limit kernel memory > consumption by the memory.limit_in_bytes along with user memory. > > This user API was introduced when the implementation of kmem accounting > lacked slab shrinker support and hence was useless in practice. Things > have changed since then - slab shrinkers were made memcg aware, the > accounting overhead seems to be negligible, and a failure to charge a > kmem allocation should not have critical consequences, because we only > account those kernel objects that should be safe to fail. That's why > kmem accounting is enabled by default for all cgroups in the default > hierarchy, which will eventually replace the legacy one. > > The ability to enable kmem accounting for some cgroups while keeping it > disabled for others is getting difficult to maintain. E.g. to make > shadow node shrinker memcg aware (see mm/workingset.c), we need to know > the relationship between the number of shadow nodes allocated for a > cgroup and the size of its lru list. If kmem accounting is enabled for > all cgroups there is no problem, but what should we do if kmem > accounting is enabled only for half of cgroups? We've no other choice > but use global lru stats while scanning root cgroup's shadow nodes, but > that would be wrong if kmem accounting was enabled for all cgroups > (which is the case if the unified hierarchy is used), in which case we > should use lru stats of the root cgroup's lruvec. > > That being said, let's enable kmem accounting for all memory cgroups by > default. If one finds it unstable or too costly, it can always be > disabled system-wide by passing cgroup.memory=nokmem to the kernel at > boot time. > > Signed-off-by: Vladimir Davydov Acked-by: Johannes Weiner A little bolder than I would have preferred for legacy memcg, but I don't think we have another choice here. And you're right, accounting costs are a far cry from what they once were. So I'm okay with this.