From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tejun Heo Subject: [PATCH 06/10] cgroup: convert memcg controller to the new cftype interface Date: Fri, 16 Mar 2012 16:35:59 -0700 Message-ID: <1331940963-15756-7-git-send-email-tj@kernel.org> References: <1331940963-15756-1-git-send-email-tj@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:from:to:cc:subject:date:message-id:x-mailer:in-reply-to :references; bh=cBTVSErr9qGv4t9Ips5t1aTWO4LE5JMoTJwAcYPTLyI=; b=Ua3NhY92T3eTcznG+Ip5Q2xhStxQp6T07Bu6urTO58/UoAr8i4pZFjvd6xOomyqepV ac0iaTH3lB6sp46dLUt3T9NzXh/7zf1xl9OIcW4Pyey3JQrXx8VWaCUAm3s4AHEFcDWT NtJI/jCJjoEwNE/G+pIV/XlaxIKRXpM6x/Yrjwp3hjwGLTNXJtoCoPrrAsRa06WOxLzP zYPmIwGDDuVMloeN7SGYk02cdLGC2J5gowu9GH3MN+Viwwebs/itu7l9HPiAD7s5oN1c qAGkVRppBk1ay+XgLIN1IbT7RXsoRcOtRmVnM0ZydHJfy8kzQRIzYwnsnpFESyP51Bap cCWA== In-Reply-To: <1331940963-15756-1-git-send-email-tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: glommer-bzQdu9zFT3WakBO8gow8eQ@public.gmane.org, lizf-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org, containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org, cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Cc: ctalbott-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org, rni-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org, fweisbec-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, Hugh Dickins , linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Michal Hocko , Johannes Weiner , Tejun Heo Convert memcg to use the new cftype based interface. memcg is rather special in that * memsw_cgroup_files creation is dependent on do_swap_account. * ->populate() is abused for mem_cgroup_sockets_init(). memsw_cgroup_files is converted to use CGROUP_SUBSYS_CFTYPES_COND(), where the condition is really_do_swap_account. ->populate() is preserved for register_kmem_files() invocation but I strongly urge moving the initialization to the tail of ->create(). If there's something which can't be done from ->create(), which isn't clear from the comment, please let me know. Signed-off-by: Tejun Heo Cc: Johannes Weiner Cc: Michal Hocko Cc: Balbir Singh Cc: KAMEZAWA Hiroyuki Cc: Glauber Costa Cc: Hugh Dickins Cc: Greg Thelen --- mm/memcontrol.c | 31 ++++++------------------------- net/ipv4/tcp_memcontrol.c | 8 ++++---- 2 files changed, 10 insertions(+), 29 deletions(-) diff --git a/mm/memcontrol.c b/mm/memcontrol.c index ae2f0a8..f2221ce 100644 --- a/mm/memcontrol.c +++ b/mm/memcontrol.c @@ -4665,7 +4665,9 @@ static struct cftype mem_cgroup_files[] = { .mode = S_IRUGO, }, #endif + { }, /* terminate */ }; +CGROUP_SUBSYS_CFTYPES(mem_cgroup_subsys, mem_cgroup_files); #ifdef CONFIG_CGROUP_MEM_RES_CTLR_SWAP static struct cftype memsw_cgroup_files[] = { @@ -4694,20 +4696,10 @@ static struct cftype memsw_cgroup_files[] = { .trigger = mem_cgroup_reset, .read_u64 = mem_cgroup_read, }, + { } /* terminate */ }; - -static int register_memsw_files(struct cgroup *cont, struct cgroup_subsys *ss) -{ - if (!do_swap_account) - return 0; - return cgroup_add_files(cont, ss, memsw_cgroup_files, - ARRAY_SIZE(memsw_cgroup_files)); -}; -#else -static int register_memsw_files(struct cgroup *cont, struct cgroup_subsys *ss) -{ - return 0; -} +CGROUP_SUBSYS_CFTYPES_COND(mem_cgroup_subsys, memsw_cgroup_files, + really_do_swap_account); #endif static int alloc_mem_cgroup_per_zone_info(struct mem_cgroup *memcg, int node) @@ -4963,18 +4955,7 @@ static void mem_cgroup_destroy(struct cgroup *cont) static int mem_cgroup_populate(struct cgroup_subsys *ss, struct cgroup *cont) { - int ret; - - ret = cgroup_add_files(cont, ss, mem_cgroup_files, - ARRAY_SIZE(mem_cgroup_files)); - - if (!ret) - ret = register_memsw_files(cont, ss); - - if (!ret) - ret = register_kmem_files(cont, ss); - - return ret; + return register_kmem_files(cont, ss); } #ifdef CONFIG_MMU diff --git a/net/ipv4/tcp_memcontrol.c b/net/ipv4/tcp_memcontrol.c index e714c68..21f48f3 100644 --- a/net/ipv4/tcp_memcontrol.c +++ b/net/ipv4/tcp_memcontrol.c @@ -35,7 +35,9 @@ static struct cftype tcp_files[] = { .trigger = tcp_cgroup_reset, .read_u64 = tcp_cgroup_read, }, + { } /* terminate */ }; +CGROUP_SUBSYS_CFTYPES(mem_cgroup_subsys, tcp_files); static inline struct tcp_memcontrol *tcp_from_cgproto(struct cg_proto *cg_proto) { @@ -65,7 +67,7 @@ int tcp_init_cgroup(struct cgroup *cgrp, struct cgroup_subsys *ss) cg_proto = tcp_prot.proto_cgroup(memcg); if (!cg_proto) - goto create_files; + return 0; tcp = tcp_from_cgproto(cg_proto); @@ -88,9 +90,7 @@ int tcp_init_cgroup(struct cgroup *cgrp, struct cgroup_subsys *ss) cg_proto->sockets_allocated = &tcp->tcp_sockets_allocated; cg_proto->memcg = memcg; -create_files: - return cgroup_add_files(cgrp, ss, tcp_files, - ARRAY_SIZE(tcp_files)); + return 0; } EXPORT_SYMBOL(tcp_init_cgroup); -- 1.7.7.3