linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Suleiman Souhlal <suleiman@google.com>
To: Glauber Costa <glommer@parallels.com>
Cc: Suleiman Souhlal <ssouhlal@freebsd.org>,
	gthelen@google.com, yinghan@google.com,
	kamezawa.hiroyu@jp.fujitsu.com, jbottomley@parallels.com,
	linux-mm@kvack.org
Subject: Re: [RFC] [PATCH 4/4] memcg: Document kernel memory accounting.
Date: Mon, 17 Oct 2011 10:19:35 -0700	[thread overview]
Message-ID: <CABCjUKAYmzZPn8N1bcinQCR63SD2P7rDL9xWo81fBf-PZ4BJNQ@mail.gmail.com> (raw)
In-Reply-To: <4E9BEDA9.6000908@parallels.com>

Hello Glauber,

On Mon, Oct 17, 2011 at 1:56 AM, Glauber Costa <glommer@parallels.com> wrote:
> On 10/15/2011 04:38 AM, Suleiman Souhlal wrote:
> 3) Easier to do per-cache tuning if we ever want to.

What kind of tuning are you thinking about?

> About, on-demand creation, I think it is a nice idea. But it may impact
> allocation latency on caches that we are sure to be used, like the dentry
> cache. So that gives us:

I don't think this is really a problem, as only the first allocation
of that type is impacted.
But if it turns out it is, we can just always create them
asynchronously (which we already do if we have GFP_NOWAIT).

>> +When a kmem_cache gets migrated to the root cgroup, "dead" is appended to
>> +its name, to indicated that it is not going to be used for new
>> allocations.
>
> Why not just remove it?

Because there are still objects allocated from it.

> * We still need the ability to restrict kernel memory usage separately from
> user memory, dependent on a selectable, as we already discussed here.

This should not be difficult to add.
My main concern is when and what to reclaim, when there is a kernel
memory limit.

Thanks for the comments,
-- Suleiman

--
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:[~2011-10-17 17:19 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-15  0:38 [RFC] [PATCH 0/4] memcg: Kernel " Suleiman Souhlal
2011-10-15  0:38 ` [RFC] [PATCH 1/4] memcg: Kernel memory accounting infrastructure Suleiman Souhlal
2011-10-15  0:38   ` [RFC] [PATCH 2/4] memcg: Introduce __GFP_NOACCOUNT Suleiman Souhlal
2011-10-15  0:38     ` [RFC] [PATCH 3/4] memcg: Slab accounting Suleiman Souhlal
2011-10-15  0:38       ` [RFC] [PATCH 4/4] memcg: Document kernel memory accounting Suleiman Souhlal
2011-10-17  8:56         ` Glauber Costa
2011-10-17 17:19           ` Suleiman Souhlal [this message]
2011-10-17  0:32 ` [RFC] [PATCH 0/4] memcg: Kernel " KAMEZAWA Hiroyuki

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=CABCjUKAYmzZPn8N1bcinQCR63SD2P7rDL9xWo81fBf-PZ4BJNQ@mail.gmail.com \
    --to=suleiman@google.com \
    --cc=glommer@parallels.com \
    --cc=gthelen@google.com \
    --cc=jbottomley@parallels.com \
    --cc=kamezawa.hiroyu@jp.fujitsu.com \
    --cc=linux-mm@kvack.org \
    --cc=ssouhlal@freebsd.org \
    --cc=yinghan@google.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox