All of lore.kernel.org
 help / color / mirror / Atom feed
From: Glauber Costa <glommer@parallels.com>
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, mhocko@suse.cz, akpm@linux-foundation.org,
	hannes@cmpxchg.org, linux-kernel@vger.kernel.org,
	cgroups@vger.kernel.org
Subject: Re: [PATCH -V3 2/8] memcg: Add HugeTLB extension
Date: Tue, 13 Mar 2012 17:28:09 +0400	[thread overview]
Message-ID: <4F5F4B69.806@parallels.com> (raw)
In-Reply-To: <1331622432-24683-3-git-send-email-aneesh.kumar@linux.vnet.ibm.com>

On 03/13/2012 11:07 AM, Aneesh Kumar K.V wrote:
> @@ -4951,6 +5083,12 @@ static int mem_cgroup_pre_destroy(struct cgroup_subsys *ss,
>   					struct cgroup *cont)
>   {
>   	struct mem_cgroup *memcg = mem_cgroup_from_cont(cont);
> +	/*
> +	 * Don't allow memcg removal if we have HugeTLB resource
> +	 * usage.
> +	 */
> +	if (mem_cgroup_hugetlb_usage(memcg)>  0)
> +		return -EBUSY;
>
>   	return mem_cgroup_force_empty(memcg, false);

Why can't you move the charges like everyone else?

--
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>

  reply	other threads:[~2012-03-13 13:28 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-13  7:07 [PATCH -V3 0/8] memcg: Add memcg extension to control HugeTLB allocation Aneesh Kumar K.V
2012-03-13  7:07 ` Aneesh Kumar K.V
2012-03-13  7:07 ` [PATCH -V3 1/8] hugetlb: rename max_hstate to hugetlb_max_hstate Aneesh Kumar K.V
2012-03-13  7:07   ` Aneesh Kumar K.V
     [not found]   ` <1331622432-24683-2-git-send-email-aneesh.kumar-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org>
2012-03-13 13:13     ` Hillf Danton
2012-03-13 13:13       ` Hillf Danton
2012-03-13 13:13       ` Hillf Danton
2012-03-13  7:07 ` [PATCH -V3 2/8] memcg: Add HugeTLB extension Aneesh Kumar K.V
2012-03-13  7:07   ` Aneesh Kumar K.V
2012-03-13 13:28   ` Glauber Costa [this message]
2012-03-13 13:42     ` Glauber Costa
2012-03-13 21:33   ` Andrew Morton
2012-03-13 21:33     ` Andrew Morton
2012-03-14 10:21     ` Aneesh Kumar K.V
2012-03-14 10:21       ` Aneesh Kumar K.V
     [not found]       ` <87zkbj8ou9.fsf-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org>
2012-03-14 23:43         ` Andrew Morton
2012-03-14 23:43           ` Andrew Morton
2012-03-14 23:43           ` Andrew Morton
2012-03-13  7:07 ` [PATCH -V3 3/8] hugetlb: add charge/uncharge calls for HugeTLB alloc/free Aneesh Kumar K.V
2012-03-13  7:07   ` Aneesh Kumar K.V
2012-03-13 13:20   ` Hillf Danton
2012-03-13 13:20     ` Hillf Danton
     [not found]     ` <CAJd=RBAHYi-BOXBO+u0u9-C=35Lu=ow=L77w2WSsndUBxVKf9w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-03-14 10:22       ` Aneesh Kumar K.V
2012-03-14 10:22         ` Aneesh Kumar K.V
2012-03-14 10:22         ` Aneesh Kumar K.V
2012-03-13 13:31   ` Glauber Costa
     [not found]     ` <4F5F4C48.8050001-bzQdu9zFT3WakBO8gow8eQ@public.gmane.org>
2012-03-14 10:41       ` Aneesh Kumar K.V
2012-03-14 10:41         ` Aneesh Kumar K.V
2012-03-14 10:41         ` Aneesh Kumar K.V
     [not found]   ` <1331622432-24683-4-git-send-email-aneesh.kumar-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org>
2012-03-13 21:36     ` Andrew Morton
2012-03-13 21:36       ` Andrew Morton
2012-03-13 21:36       ` Andrew Morton
     [not found]       ` <20120313143654.5dd1243d.akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>
2012-03-14 10:47         ` Aneesh Kumar K.V
2012-03-14 10:47           ` Aneesh Kumar K.V
2012-03-14 10:47           ` Aneesh Kumar K.V
2012-03-13  7:07 ` [PATCH -V3 4/8] memcg: track resource index in cftype private Aneesh Kumar K.V
2012-03-13  7:07   ` Aneesh Kumar K.V
2012-03-13 13:34   ` Glauber Costa
     [not found]     ` <4F5F4CD0.3080207-bzQdu9zFT3WakBO8gow8eQ@public.gmane.org>
2012-03-14 10:48       ` Aneesh Kumar K.V
2012-03-14 10:48         ` Aneesh Kumar K.V
2012-03-14 10:48         ` Aneesh Kumar K.V
2012-03-13  7:07 ` [PATCH -V3 5/8] hugetlbfs: Add memcg control files for hugetlbfs Aneesh Kumar K.V
2012-03-13  7:07   ` Aneesh Kumar K.V
     [not found]   ` <1331622432-24683-6-git-send-email-aneesh.kumar-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org>
2012-03-13 21:42     ` Andrew Morton
2012-03-13 21:42       ` Andrew Morton
2012-03-13 21:42       ` Andrew Morton
2012-03-14 11:10       ` Aneesh Kumar K.V
2012-03-14 11:10         ` Aneesh Kumar K.V
2012-03-14 11:35         ` Andrew Morton
2012-03-14 11:35           ` Andrew Morton
     [not found]           ` <20120314043530.d6f3d424.akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>
2012-03-14 15:57             ` Aneesh Kumar K.V
2012-03-14 15:57               ` Aneesh Kumar K.V
2012-03-14 15:57               ` Aneesh Kumar K.V
2012-03-13  7:07 ` [PATCH -V3 6/8] hugetlbfs: Add a list for tracking in-use HugeTLB pages Aneesh Kumar K.V
2012-03-13  7:07   ` Aneesh Kumar K.V
2012-03-13  7:07 ` [PATCH -V3 7/8] memcg: move HugeTLB resource count to parent cgroup on memcg removal Aneesh Kumar K.V
2012-03-13  7:07   ` Aneesh Kumar K.V
     [not found]   ` <1331622432-24683-8-git-send-email-aneesh.kumar-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org>
2012-03-13 21:47     ` Andrew Morton
2012-03-13 21:47       ` Andrew Morton
2012-03-13 21:47       ` Andrew Morton
2012-03-14 12:22       ` Aneesh Kumar K.V
2012-03-14 12:22         ` Aneesh Kumar K.V
2012-03-13  7:07 ` [PATCH -V3 8/8] memcg: Add memory controller documentation for hugetlb management Aneesh Kumar K.V
2012-03-13  7:07   ` Aneesh Kumar K.V
     [not found] ` <1331622432-24683-1-git-send-email-aneesh.kumar-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org>
2012-03-13 21:49   ` [PATCH -V3 0/8] memcg: Add memcg extension to control HugeTLB allocation Andrew Morton
2012-03-13 21:49     ` Andrew Morton
2012-03-13 21:49     ` Andrew Morton
2012-03-14 13:01     ` Aneesh Kumar K.V
2012-03-14 13:01       ` Aneesh Kumar K.V

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=4F5F4B69.806@parallels.com \
    --to=glommer@parallels.com \
    --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 \
    --cc=mhocko@suse.cz \
    /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: link
Be 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.