Re: [PATCH -V6 07/14] memcg: Add HugeTLB extension

From: David Rientjes
Date: Sat Jun 09 2012 - 21:56:14 EST


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@xxxxxxxxxxxxxx )
>

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.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/