Messages in this thread | | | Date | Sat, 9 Jun 2012 18:55:30 -0700 (PDT) | From | David Rientjes <> | Subject | Re: [PATCH -V6 07/14] memcg: Add HugeTLB extension |
| |
On Sat, 9 Jun 2012, Aneesh Kumar K.V wrote:
> David Rientjes didn't like HugetTLB limit to be a memcg extension and > wanted this to be a separate controller. I posted a v7 version that did > HugeTLB limit as a separate controller and used page cgroup to track > HugeTLB cgroup. Kamezawa Hiroyuki didn't like the usage of page_cgroup > in HugeTLB controller( http://mid.gmane.org/4FCD648E.90709@jp.fujitsu.com ) >
Yes, and thank you very much for working on v8 to remove the dependency on page_cgroup and to seperate this out. I think it will benefit users who don't want to enable all of memcg but still want to account and restrict hugetlb page usage, and I think the code seperation is much cleaner internally.
I'll review that patchset and suggest that the old hugetlb extension in -mm be dropped in the interim.
| |