[lkml]   [2011]   [Nov]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] new cgroup controller "fork"
On 11/03/2011 04:30 PM, Max Kellermann wrote:
> On 2011/11/03 18:50, Glauber Costa<> wrote:
>> That still seems to be up to admin. If no processes are removed from
>> the cgroup or included in the cgroup, the only action/verb the
>> counter
>> is concerned about is to fork. Under this circumstance, both seem
>> equivalent from my PoV.
> I'm confused. One of us misunderstands the whole thing.
> Examples of both controllers:
> task_counter: task.limit=2. Let's say the only process in that group
> forks, then you have two processes. Forking is disallowed from now
> on. The child process exits, and there's only one left - which is
> allowed to fork! The group may bounce between 0 and 2 processes
> forever.

Ah, I see. I assumed you were decrementing the counter when a process

> cgroup_fork: fork.remaining=2. Now let's say we have one thousand
> processes in that group! One of those forks (allowed). And it forks
> again (allowed). And tries again - blocked because "fork.remaining"
> has reached zero. We have 1002 processes; when 1001 of those
> processes exit, one remains, but it is still disallowed to fork,
> because "fork.remaining" is still zero. It will remaing zero until
> somebody with write permissions raises it again.
> Did I get it wrong? To me, that is not look equivalent at all.
No, I was not careful enough when reading the patch, and as already
stated, I made an assumption that seemed reasonable.

Which raises the question: If you don't decrement on exit, and only
count how many forks happened in the past, what is your use case for this?

Please note that both of you seem to target the same goal: prevent fork
bombs. If a child exits, I see no reason to not open up room for a new
process inside the group.

 \ /
  Last update: 2011-11-03 19:37    [W:0.047 / U:1.724 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site