Messages in this thread | | | Date | Thu, 20 May 2010 15:26:52 -0700 | Subject | Re: [PATCH 1/3] cgroups: Add an API to attach a task to current task's cgroup | From | Paul Menage <> |
| |
On Thu, May 20, 2010 at 3:22 PM, Paul Menage <menage@google.com> wrote: > On Tue, May 18, 2010 at 5:04 PM, Sridhar Samudrala > <samudrala.sridhar@gmail.com> wrote: >> Add a new kernel API to attach a task to current task's cgroup >> in all the active hierarchies. >> >> Signed-off-by: Sridhar Samudrala <sri@us.ibm.com> > > Reviewed-by: Paul Menage <menage@google.com> >
One other thought on this - this would be the first piece of code that's attaching a task to a cgroup without holding the cgroup directory inode i_mutex. I believe that this is probably OK.
Paul
| |