All of lore.kernel.org
 help / color / mirror / Atom feed
From: Serge Hallyn <serge.hallyn-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>
To: Gao feng <gaofeng-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org>
Cc: cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org,
	tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	lizefan-hv44wF8Li93QT0dZR+AlfA@public.gmane.org,
	ebiederm-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org,
	glommer-bzQdu9zFT3WakBO8gow8eQ@public.gmane.org
Subject: Re: [RFC PATCH 0/9] Add container support for cgroup
Date: Mon, 17 Dec 2012 07:16:08 -0600	[thread overview]
Message-ID: <20121217131608.GA5163@sergelap> (raw)
In-Reply-To: <1355726615-15224-1-git-send-email-gaofeng-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org>

Quoting Gao feng (gaofeng-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org):
> Right now,if we mount cgroup in the container,we will get
> host's cgroup informations and even we can change host's
> cgroup in container.
> 
> So the resource controller of the container will lose
> effectiveness.
> 
> This patchset try to add contianer support for cgroup.
> the main idea is allocateing cgroup super-block for each
> cgroup mounted in different pid namespace.
> 
> The top cgroup of container will share css with host.
> When the cgroup being mounted in contianer,the tasks in
> this container will be attached to this new mounted
> hierarchy's top cgroup, And when unmounting cgroup in
> container,these tasks will be attached back to host's cgroup.
> 
> Since the container can change the shared css through it's
> cgroup subsystem files. patch 7/8 disable the write permission
> of container's top cgroup files. In my TODO list, container
> will have it's own css, this problem will disappear.
> 
> 
> This patchset is sent as RFC,any comments are welcome.
> Maybe this isn't the best solution, if you have better
> solution,Please let me know.

Sounds very interesting, thanks.  I'm out (and mostly AFK) but
will take a look on wed or thu.

-serge

  parent reply	other threads:[~2012-12-17 13:16 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-17  6:43 [RFC PATCH 0/9] Add container support for cgroup Gao feng
     [not found] ` <1355726615-15224-1-git-send-email-gaofeng-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org>
2012-12-17  6:43   ` [RFC PATCH 1/9] cgroup: introduce cgroupfs_root flag ROOT_NAMESPACE Gao feng
2012-12-17  6:43   ` Gao feng
2012-12-17  6:43   ` [RFC PATCH 2/9] cgroup: introduce the top root Gao feng
2012-12-17  6:43   ` Gao feng
2012-12-17  6:43   ` [RFC PATCH 3/9] cgroup: use root->top_root instead of root Gao feng
2012-12-17  6:43   ` Gao feng
2012-12-17  6:43   ` [RFC PATCH 4/9] introduce helper function cgroup_in_root Gao feng
2012-12-17  6:43   ` Gao feng
2012-12-17  6:43   ` [RFC PATCH 5/9] cgroup: add container support for cgroup Gao feng
2012-12-17  6:43   ` Gao feng
2012-12-17  6:43   ` [RFC PATCH 6/9] pidns: move next_tgid to kernel/pid.c Gao feng
2012-12-17  6:43   ` Gao feng
2012-12-17  6:43   ` [RFC PATCH 7/9] cgroup: attach container's tasks to proper cgroup Gao feng
2012-12-17  6:43   ` Gao feng
2012-12-17  6:43   ` [RFC PATCH 8/9] cgroup: disallow container to change top cgroup's subsys files Gao feng
2012-12-17  6:43   ` Gao feng
2012-12-17  6:43   ` [RFC PATCH 9/9] cgroup: rework cgroup_path Gao feng
2012-12-17  6:43   ` Gao feng
2012-12-17  8:08   ` [RFC PATCH 0/9] Add container support for cgroup Glauber Costa
     [not found]     ` <50CED2FD.1040509-bzQdu9zFT3WakBO8gow8eQ@public.gmane.org>
2012-12-17  8:54       ` Gao feng
2012-12-17  8:54       ` Gao feng
2012-12-19 21:39       ` Serge Hallyn
2012-12-19 21:39       ` Serge Hallyn
2012-12-17  8:08   ` Glauber Costa
2012-12-17 13:16   ` Serge Hallyn [this message]
2012-12-17 13:16   ` Serge Hallyn
2012-12-17 23:48   ` Tejun Heo
     [not found]     ` <20121217234816.GA10220-9pTldWuhBndy/B6EtB590w@public.gmane.org>
2012-12-17 23:54       ` Eric W. Biederman
     [not found]         ` <87obhsgrq7.fsf-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org>
2012-12-17 23:56           ` Tejun Heo
2012-12-18  5:37       ` Gao feng
2012-12-18  5:37       ` Gao feng
  -- strict thread matches above, loose matches on Subject: below --
2012-12-17  6:43 Gao feng

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=20121217131608.GA5163@sergelap \
    --to=serge.hallyn-z7wlfzj8ewms+fvcfc7uqw@public.gmane.org \
    --cc=cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org \
    --cc=ebiederm-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org \
    --cc=gaofeng-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org \
    --cc=glommer-bzQdu9zFT3WakBO8gow8eQ@public.gmane.org \
    --cc=lizefan-hv44wF8Li93QT0dZR+AlfA@public.gmane.org \
    --cc=tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    /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.