From mboxrd@z Thu Jan 1 00:00:00 1970 From: Glauber Costa Subject: Re: [PATCH 0/2] page_cgroup cleanups Date: Fri, 5 Apr 2013 15:32:27 +0400 Message-ID: <515EB64B.8010104@parallels.com> References: <1365156072-24100-1-git-send-email-glommer@parallels.com> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1365156072-24100-1-git-send-email-glommer-bzQdu9zFT3WakBO8gow8eQ@public.gmane.org> Sender: cgroups-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: Content-Type: text/plain; charset="us-ascii" To: cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Cc: linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org, kamezawa.hiroyu-+CUm20s59erQFUHtdCDX3A@public.gmane.org, Johannes Weiner , Michal Hocko On 04/05/2013 02:01 PM, Glauber Costa wrote: > Hi, > > Last time I sent the mem cgroup bypass patches, Kame and Michal pointed out > that some of it was a bit of cleanup, specifically at the page_cgroup side. > I've decided to separate those patches and send them separately. After these > patches are applied, page_cgroup will be initialized together with the root > cgroup, instead of init/main.c > > When we move cgroup initialization to the first non-root cgroup created, all > we'll have to do from the page_cgroup side would be to move the initialization > that now happens at root, to the first child. > > Glauber Costa (2): > memcg: consistently use vmalloc for page_cgroup allocations > memcg: defer page_cgroup initialization > > include/linux/page_cgroup.h | 21 +------------------ > init/main.c | 2 -- > mm/memcontrol.c | 2 ++ > mm/page_cgroup.c | 51 +++++++++++++++------------------------------ > 4 files changed, 20 insertions(+), 56 deletions(-) > FYI: There are kbuild warnings with this. I wanted to send it earlier to see what people think. If there is no changes requested, please let me know I will send a new version with just the kbuild fixes folded.