From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f174.google.com (mail-wi0-f174.google.com [209.85.212.174]) by kanga.kvack.org (Postfix) with ESMTP id DB3A86B0038 for ; Fri, 23 Oct 2015 07:32:40 -0400 (EDT) Received: by wikq8 with SMTP id q8so72560246wik.1 for ; Fri, 23 Oct 2015 04:32:40 -0700 (PDT) Received: from mail-wi0-f169.google.com (mail-wi0-f169.google.com. [209.85.212.169]) by mx.google.com with ESMTPS id r6si24315064wjo.168.2015.10.23.04.32.39 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 23 Oct 2015 04:32:39 -0700 (PDT) Received: by wijp11 with SMTP id p11so73076925wij.0 for ; Fri, 23 Oct 2015 04:32:39 -0700 (PDT) Date: Fri, 23 Oct 2015 13:32:38 +0200 From: Michal Hocko Subject: Re: [PATCH 2/8] mm: memcontrol: export root_mem_cgroup Message-ID: <20151023113237.GK2410@dhcp22.suse.cz> References: <1445487696-21545-1-git-send-email-hannes@cmpxchg.org> <1445487696-21545-3-git-send-email-hannes@cmpxchg.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1445487696-21545-3-git-send-email-hannes@cmpxchg.org> Sender: owner-linux-mm@kvack.org List-ID: To: Johannes Weiner Cc: "David S. Miller" , Andrew Morton , Vladimir Davydov , Tejun Heo , netdev@vger.kernel.org, linux-mm@kvack.org, cgroups@vger.kernel.org, linux-kernel@vger.kernel.org On Thu 22-10-15 00:21:30, Johannes Weiner wrote: > A later patch will need this symbol in files other than memcontrol.c, > so export it now and replace mem_cgroup_root_css at the same time. > > Signed-off-by: Johannes Weiner Acked-by: Michal Hocko > --- > include/linux/memcontrol.h | 3 ++- > mm/backing-dev.c | 2 +- > mm/memcontrol.c | 5 ++--- > 3 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/include/linux/memcontrol.h b/include/linux/memcontrol.h > index 805da1f..19ff87b 100644 > --- a/include/linux/memcontrol.h > +++ b/include/linux/memcontrol.h > @@ -275,7 +275,8 @@ struct mem_cgroup { > struct mem_cgroup_per_node *nodeinfo[0]; > /* WARNING: nodeinfo must be the last member here */ > }; > -extern struct cgroup_subsys_state *mem_cgroup_root_css; > + > +extern struct mem_cgroup *root_mem_cgroup; > > /** > * mem_cgroup_events - count memory events against a cgroup > diff --git a/mm/backing-dev.c b/mm/backing-dev.c > index 095b23b..73ab967 100644 > --- a/mm/backing-dev.c > +++ b/mm/backing-dev.c > @@ -702,7 +702,7 @@ static int cgwb_bdi_init(struct backing_dev_info *bdi) > > ret = wb_init(&bdi->wb, bdi, 1, GFP_KERNEL); > if (!ret) { > - bdi->wb.memcg_css = mem_cgroup_root_css; > + bdi->wb.memcg_css = &root_mem_cgroup->css; > bdi->wb.blkcg_css = blkcg_root_css; > } > return ret; > diff --git a/mm/memcontrol.c b/mm/memcontrol.c > index a8ccdbc..e54f434 100644 > --- a/mm/memcontrol.c > +++ b/mm/memcontrol.c > @@ -76,9 +76,9 @@ > struct cgroup_subsys memory_cgrp_subsys __read_mostly; > EXPORT_SYMBOL(memory_cgrp_subsys); > > +struct mem_cgroup *root_mem_cgroup __read_mostly; > + > #define MEM_CGROUP_RECLAIM_RETRIES 5 > -static struct mem_cgroup *root_mem_cgroup __read_mostly; > -struct cgroup_subsys_state *mem_cgroup_root_css __read_mostly; > > /* Whether the swap controller is active */ > #ifdef CONFIG_MEMCG_SWAP > @@ -4213,7 +4213,6 @@ mem_cgroup_css_alloc(struct cgroup_subsys_state *parent_css) > /* root ? */ > if (parent_css == NULL) { > root_mem_cgroup = memcg; > - mem_cgroup_root_css = &memcg->css; > page_counter_init(&memcg->memory, NULL); > memcg->high = PAGE_COUNTER_MAX; > memcg->soft_limit = PAGE_COUNTER_MAX; > -- > 2.6.1 -- Michal Hocko SUSE Labs -- 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