lkml.org 
[lkml]   [2011]   [Dec]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH -mm] cgroup: Fix task counter common ancestor logic
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 11/24/2011 08:24 AM, Frederic Weisbecker wrote:
> On Tue, Nov 08, 2011 at 01:51:11PM -0800, Andrew Morton wrote:
>> On Tue, 8 Nov 2011 16:21:10 +0100 Frederic Weisbecker
>> <fweisbec@gmail.com> wrote:
>>
>>> To solve this, keep the original cgroup of each thread in the
>>> thread group cached in the flex array and pass it to
>>> can_attach_task()/attach_task() and cancel_attach_task() so
>>> that the correct common ancestor between the old and new cgroup
>>> can be safely retrieved for each task.
>>
>> OK, thanks.
>>
>> We need to work out what to do with this patchset. ie: should we
>> merge it. I'm not sure that the case has been made?
>>
>> Let's please drag this thing onto the table and poke at it for a
>> while. Probably everyone has forgotten everything so we'll need
>> to start again, sorry. Perhaps you can (re)start proceedings by
>> telling us why it's useful to our users and why we should merge
>> it?
>
> Right, so the main purpose is to have a suitable forkbomb
> protection in the lxc containers. It seems that these days, using
> NR_PROC rlimit is the prime choice to protect against forkbombs.
> But we can't use this for containers because if they run under the
> same user, they can starve each others by generating high number of
> processes. So we need the limit on number of processes to be per
> container.
>
> The basic requirement is to be able to run untrustee process inside
> a container while protecting against attacks from there without
> impacting the rest of the system.
>
> I'm adding in Cc some Lxc people who could perhaps provide more
> details and testify we really need this.
>
>> Some mental notes:
>>
>> Tim says it would be useful for the things he's doing but
>> doesn't appear to have confirmed/tested that.
>
> Yeah, I'm waiting for more details from him. Tim?
>
>> Kay has said that it would not be useful for his plumber's
>> wishlist item, which is a shame.
>
> Indeed. I mean it would work but this cgroup subsystem is too much
> overhead to be used by an init process (and then all other
> processes).
>
>> I seem to recall complaining that it doesn't address the forkbomb
>> issue for non-cgroups setups, so the forkbomb issue remains
>> unaddressed.
>
> Right. Now if we can find a generic solution to protect against all
> forkbombs, something deterministic that can react soon enough so
> that it doesn't impact the rest of system, in order to avoid
> running into some DDOS, then we will consider it.
>
> Thanks.


I have heard that this is being held up because of a lack of
justification. Here is my attempt. I would like to use the ability
to prevent a fork bomb and guaranteed killall for three different
tools that I work on.

1. Kiosk/xguest user. Currently we tell people to log out of a
system on the kiosk and the pam stack attempts to kill all processes
running in the kiosk user session. The kiosk user is not allowed to
login to the system if a process is still running as the user. The
reason for this is we want to guarantee no previous user left a trojan
process waiting to track your activity. If the login program can not
kill all user processes at logout, then the next user will not be able
to login.

2. Sandbox Desktop tools are used to write one or more processes
within a lockeddown environment. The idea here is to run untrusted
code and potentially this code could trigger a fork bomb, you want to
be able to kill the session.

3. Secure containers, similarly to sandboxes allow users to run
multiple Linux Containers in a locked down manner. We would like to
be guaranteed that we can kill all processes within a secure
containers at exit.



-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iEYEARECAAYFAk7nuSYACgkQrlYvE4MpobMY1ACgmZ/P8LbVTHBrOQUsh6xL5Qy2
scsAoK641lrpZkX8DB1JETpKT6jfWe5i
=E93Z
-----END PGP SIGNATURE-----

\
 
 \ /
  Last update: 2011-12-13 21:47    [from the cache]
©2003-2011 Jasper Spaans