From: Michal Hocko <mhocko@suse.cz> To: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com> Cc: linux-mm@kvack.org, mgorman@suse.de, kamezawa.hiroyu@jp.fujitsu.com, dhillf@gmail.com, aarcange@redhat.com, akpm@linux-foundation.org, hannes@cmpxchg.org, linux-kernel@vger.kernel.org, cgroups@vger.kernel.org Subject: Re: [PATCH -V4 04/10] memcg: Add HugeTLB extension Date: Wed, 28 Mar 2012 15:40:20 +0200 [thread overview] Message-ID: <20120328134020.GG20949@tiehlicka.suse.cz> (raw) In-Reply-To: <1331919570-2264-5-git-send-email-aneesh.kumar@linux.vnet.ibm.com> On Fri 16-03-12 23:09:24, Aneesh Kumar K.V wrote: [...] > diff --git a/mm/memcontrol.c b/mm/memcontrol.c > index 6728a7a..4b36c5e 100644 > --- a/mm/memcontrol.c > +++ b/mm/memcontrol.c [...] > @@ -4887,6 +5013,7 @@ err_cleanup: > static struct cgroup_subsys_state * __ref > mem_cgroup_create(struct cgroup_subsys *ss, struct cgroup *cont) > { > + int idx; > struct mem_cgroup *memcg, *parent; > long error = -ENOMEM; > int node; > @@ -4929,9 +5056,14 @@ mem_cgroup_create(struct cgroup_subsys *ss, struct cgroup *cont) > * mem_cgroup(see mem_cgroup_put). > */ > mem_cgroup_get(parent); > + for (idx = 0; idx < HUGE_MAX_HSTATE; idx++) > + res_counter_init(&memcg->hugepage[idx], > + &parent->hugepage[idx]); Hmm, I do not think we want to make groups deeper in the hierarchy unlimited as we cannot reclaim. Shouldn't we copy the limit from the parent? Still not ideal but slightly more expected behavior IMO. The hierarchy setups are still interesting and the limitations should be described in the documentation... > } else { > res_counter_init(&memcg->res, NULL); > res_counter_init(&memcg->memsw, NULL); > + for (idx = 0; idx < HUGE_MAX_HSTATE; idx++) > + res_counter_init(&memcg->hugepage[idx], NULL); > } > memcg->last_scanned_node = MAX_NUMNODES; > INIT_LIST_HEAD(&memcg->oom_notify); -- Michal Hocko SUSE Labs SUSE LINUX s.r.o. Lihovarska 1060/12 190 00 Praha 9 Czech Republic
WARNING: multiple messages have this Message-ID (diff)
From: Michal Hocko <mhocko@suse.cz> To: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com> Cc: linux-mm@kvack.org, mgorman@suse.de, kamezawa.hiroyu@jp.fujitsu.com, dhillf@gmail.com, aarcange@redhat.com, akpm@linux-foundation.org, hannes@cmpxchg.org, linux-kernel@vger.kernel.org, cgroups@vger.kernel.org Subject: Re: [PATCH -V4 04/10] memcg: Add HugeTLB extension Date: Wed, 28 Mar 2012 15:40:20 +0200 [thread overview] Message-ID: <20120328134020.GG20949@tiehlicka.suse.cz> (raw) In-Reply-To: <1331919570-2264-5-git-send-email-aneesh.kumar@linux.vnet.ibm.com> On Fri 16-03-12 23:09:24, Aneesh Kumar K.V wrote: [...] > diff --git a/mm/memcontrol.c b/mm/memcontrol.c > index 6728a7a..4b36c5e 100644 > --- a/mm/memcontrol.c > +++ b/mm/memcontrol.c [...] > @@ -4887,6 +5013,7 @@ err_cleanup: > static struct cgroup_subsys_state * __ref > mem_cgroup_create(struct cgroup_subsys *ss, struct cgroup *cont) > { > + int idx; > struct mem_cgroup *memcg, *parent; > long error = -ENOMEM; > int node; > @@ -4929,9 +5056,14 @@ mem_cgroup_create(struct cgroup_subsys *ss, struct cgroup *cont) > * mem_cgroup(see mem_cgroup_put). > */ > mem_cgroup_get(parent); > + for (idx = 0; idx < HUGE_MAX_HSTATE; idx++) > + res_counter_init(&memcg->hugepage[idx], > + &parent->hugepage[idx]); Hmm, I do not think we want to make groups deeper in the hierarchy unlimited as we cannot reclaim. Shouldn't we copy the limit from the parent? Still not ideal but slightly more expected behavior IMO. The hierarchy setups are still interesting and the limitations should be described in the documentation... > } else { > res_counter_init(&memcg->res, NULL); > res_counter_init(&memcg->memsw, NULL); > + for (idx = 0; idx < HUGE_MAX_HSTATE; idx++) > + res_counter_init(&memcg->hugepage[idx], NULL); > } > memcg->last_scanned_node = MAX_NUMNODES; > INIT_LIST_HEAD(&memcg->oom_notify); -- Michal Hocko SUSE Labs SUSE LINUX s.r.o. Lihovarska 1060/12 190 00 Praha 9 Czech Republic -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2012-03-28 13:40 UTC|newest] Thread overview: 130+ messages / expand[flat|nested] mbox.gz Atom feed top 2012-03-16 17:39 [PATCH -V4 00/10] memcg: Add memcg extension to control HugeTLB allocation Aneesh Kumar K.V 2012-03-16 17:39 ` Aneesh Kumar K.V 2012-03-16 17:39 ` [PATCH -V4 01/10] hugetlb: rename max_hstate to hugetlb_max_hstate Aneesh Kumar K.V 2012-03-16 17:39 ` Aneesh Kumar K.V 2012-03-19 2:07 ` KAMEZAWA Hiroyuki 2012-03-19 2:07 ` KAMEZAWA Hiroyuki 2012-03-28 9:18 ` Michal Hocko 2012-03-28 9:18 ` Michal Hocko 2012-03-28 9:18 ` Michal Hocko 2012-03-16 17:39 ` [PATCH -V4 02/10] hugetlbfs: don't use ERR_PTR with VM_FAULT* values Aneesh Kumar K.V 2012-03-16 17:39 ` Aneesh Kumar K.V 2012-03-19 2:11 ` KAMEZAWA Hiroyuki 2012-03-19 2:11 ` KAMEZAWA Hiroyuki 2012-03-19 2:11 ` KAMEZAWA Hiroyuki 2012-03-19 6:37 ` Aneesh Kumar K.V 2012-03-19 6:37 ` Aneesh Kumar K.V 2012-03-28 9:25 ` Michal Hocko 2012-03-28 9:25 ` Michal Hocko 2012-03-28 11:35 ` Aneesh Kumar K.V 2012-03-28 11:35 ` Aneesh Kumar K.V 2012-03-28 11:35 ` Aneesh Kumar K.V 2012-03-16 17:39 ` [PATCH -V4 03/10] hugetlbfs: Add an inline helper for finding hstate index Aneesh Kumar K.V 2012-03-16 17:39 ` Aneesh Kumar K.V 2012-03-19 2:15 ` KAMEZAWA Hiroyuki 2012-03-19 2:15 ` KAMEZAWA Hiroyuki 2012-03-28 9:41 ` Michal Hocko 2012-03-28 9:41 ` Michal Hocko 2012-03-16 17:39 ` [PATCH -V4 04/10] memcg: Add HugeTLB extension Aneesh Kumar K.V 2012-03-16 17:39 ` Aneesh Kumar K.V 2012-03-19 2:38 ` KAMEZAWA Hiroyuki 2012-03-19 2:38 ` KAMEZAWA Hiroyuki 2012-03-19 2:38 ` KAMEZAWA Hiroyuki 2012-03-19 6:52 ` Aneesh Kumar K.V 2012-03-19 6:52 ` Aneesh Kumar K.V 2012-03-19 6:52 ` Aneesh Kumar K.V 2012-03-19 7:00 ` KAMEZAWA Hiroyuki 2012-03-19 7:00 ` KAMEZAWA Hiroyuki 2012-03-19 7:00 ` KAMEZAWA Hiroyuki 2012-03-19 11:39 ` Glauber Costa 2012-03-19 12:07 ` KAMEZAWA Hiroyuki 2012-03-19 12:07 ` KAMEZAWA Hiroyuki 2012-03-21 4:48 ` Aneesh Kumar K.V 2012-03-21 4:48 ` Aneesh Kumar K.V 2012-03-21 4:48 ` Aneesh Kumar K.V 2012-03-21 5:22 ` KAMEZAWA Hiroyuki 2012-03-21 5:22 ` KAMEZAWA Hiroyuki 2012-03-21 5:22 ` KAMEZAWA Hiroyuki 2012-03-28 11:33 ` Michal Hocko 2012-03-28 11:33 ` Michal Hocko 2012-03-28 11:33 ` Michal Hocko 2012-03-28 13:40 ` Aneesh Kumar K.V 2012-03-28 13:40 ` Aneesh Kumar K.V 2012-03-28 15:44 ` Michal Hocko 2012-03-28 15:44 ` Michal Hocko 2012-03-28 13:40 ` Michal Hocko [this message] 2012-03-28 13:40 ` Michal Hocko 2012-03-28 17:37 ` Aneesh Kumar K.V 2012-03-28 17:37 ` Aneesh Kumar K.V 2012-03-28 17:37 ` Aneesh Kumar K.V 2012-03-29 0:18 ` KAMEZAWA Hiroyuki 2012-03-29 0:18 ` KAMEZAWA Hiroyuki 2012-03-29 0:18 ` KAMEZAWA Hiroyuki 2012-03-29 7:57 ` Michal Hocko 2012-03-29 7:57 ` Michal Hocko 2012-03-29 7:57 ` Michal Hocko 2012-03-16 17:39 ` [PATCH -V4 05/10] hugetlb: add charge/uncharge calls for HugeTLB alloc/free Aneesh Kumar K.V 2012-03-16 17:39 ` Aneesh Kumar K.V 2012-03-19 2:41 ` KAMEZAWA Hiroyuki 2012-03-19 2:41 ` KAMEZAWA Hiroyuki 2012-03-19 2:41 ` KAMEZAWA Hiroyuki 2012-03-19 7:01 ` Aneesh Kumar K.V 2012-03-19 7:01 ` Aneesh Kumar K.V 2012-03-28 13:17 ` Michal Hocko 2012-03-28 13:17 ` Michal Hocko 2012-03-28 13:17 ` Michal Hocko 2012-03-28 17:39 ` Aneesh Kumar K.V 2012-03-28 17:39 ` Aneesh Kumar K.V 2012-03-28 17:39 ` Aneesh Kumar K.V 2012-03-29 8:10 ` Michal Hocko 2012-03-29 8:10 ` Michal Hocko 2012-03-30 10:40 ` Aneesh Kumar K.V 2012-03-30 10:40 ` Aneesh Kumar K.V 2012-03-30 10:46 ` Michal Hocko 2012-03-30 10:46 ` Michal Hocko 2012-03-30 10:46 ` Michal Hocko 2012-03-16 17:39 ` [PATCH -V4 06/10] memcg: track resource index in cftype private Aneesh Kumar K.V 2012-03-16 17:39 ` Aneesh Kumar K.V 2012-03-19 2:43 ` KAMEZAWA Hiroyuki 2012-03-19 2:43 ` KAMEZAWA Hiroyuki 2012-03-19 2:43 ` KAMEZAWA Hiroyuki 2012-03-16 17:39 ` [PATCH -V4 07/10] hugetlbfs: Add memcg control files for hugetlbfs Aneesh Kumar K.V 2012-03-16 17:39 ` Aneesh Kumar K.V 2012-03-19 2:56 ` KAMEZAWA Hiroyuki 2012-03-19 2:56 ` KAMEZAWA Hiroyuki 2012-03-19 7:14 ` Aneesh Kumar K.V 2012-03-19 7:14 ` Aneesh Kumar K.V 2012-03-19 7:14 ` Aneesh Kumar K.V 2012-03-19 7:34 ` KAMEZAWA Hiroyuki 2012-03-19 7:34 ` KAMEZAWA Hiroyuki 2012-03-19 7:34 ` KAMEZAWA Hiroyuki 2012-03-20 9:22 ` Aneesh Kumar K.V 2012-03-20 9:22 ` Aneesh Kumar K.V 2012-03-16 17:39 ` [PATCH -V4 08/10] hugetlbfs: Add a list for tracking in-use HugeTLB pages Aneesh Kumar K.V 2012-03-16 17:39 ` Aneesh Kumar K.V 2012-03-19 3:00 ` KAMEZAWA Hiroyuki 2012-03-19 3:00 ` KAMEZAWA Hiroyuki 2012-03-19 3:00 ` KAMEZAWA Hiroyuki 2012-03-19 8:59 ` Aneesh Kumar K.V 2012-03-19 8:59 ` Aneesh Kumar K.V 2012-03-28 13:58 ` Michal Hocko 2012-03-28 13:58 ` Michal Hocko 2012-03-28 17:38 ` Aneesh Kumar K.V 2012-03-28 17:38 ` Aneesh Kumar K.V 2012-03-29 8:11 ` Michal Hocko 2012-03-29 8:11 ` Michal Hocko 2012-03-16 17:39 ` [PATCH -V4 09/10] memcg: move HugeTLB resource count to parent cgroup on memcg removal Aneesh Kumar K.V 2012-03-16 17:39 ` Aneesh Kumar K.V 2012-03-19 3:04 ` KAMEZAWA Hiroyuki 2012-03-19 3:04 ` KAMEZAWA Hiroyuki 2012-03-19 3:04 ` KAMEZAWA Hiroyuki 2012-03-19 9:00 ` Aneesh Kumar K.V 2012-03-19 9:00 ` Aneesh Kumar K.V 2012-03-28 14:07 ` Michal Hocko 2012-03-28 14:07 ` Michal Hocko 2012-03-28 14:07 ` Michal Hocko 2012-03-16 17:39 ` [PATCH -V4 10/10] memcg: Add memory controller documentation for hugetlb management Aneesh Kumar K.V 2012-03-16 17:39 ` Aneesh Kumar K.V 2012-03-28 14:36 ` Michal Hocko 2012-03-28 14:36 ` Michal Hocko 2012-03-28 14:36 ` Michal Hocko
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20120328134020.GG20949@tiehlicka.suse.cz \ --to=mhocko@suse.cz \ --cc=aarcange@redhat.com \ --cc=akpm@linux-foundation.org \ --cc=aneesh.kumar@linux.vnet.ibm.com \ --cc=cgroups@vger.kernel.org \ --cc=dhillf@gmail.com \ --cc=hannes@cmpxchg.org \ --cc=kamezawa.hiroyu@jp.fujitsu.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=mgorman@suse.de \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.