Re: [PATCH 1/3] cgroups: Add an API to attach a task to current task's cgroup

[Paul Menage]

On Thu, May 20, 2010 at 3:22 PM, Paul Menage <menage@xxxxxxxxxx> wrote:
> On Tue, May 18, 2010 at 5:04 PM, Sridhar Samudrala
> <samudrala.sridhar@xxxxxxxxx> 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@xxxxxxxxxx>
>
> Reviewed-by: Paul Menage <menage@xxxxxxxxxx>
>

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