Re: [PATCH] cgroup_pids: add fork limit

From	Max Kellermann <mk@cm4all.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] cgroup_pids: add fork limit
Date	2015-11-10 18:10 +0100
Message-ID	<qtkAF-7SL-7@gated-at.bofh.it> (permalink)
References	<qthW9-68s-5@gated-at.bofh.it> <qtiSd-6IY-9@gated-at.bofh.it> <qtjbz-6QE-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 2015/11/10 16:44, Tejun Heo <tj@kernel.org> wrote:
> On Tue, Nov 10, 2015 at 04:37:46PM +0100, Max Kellermann wrote:
> > There's "cpu" which changes priority
> 
> The cpu controller can limit both in terms of relative weight and
> absolute CPU cycle bandwidth.

No, Tejun, the "cpu" controller does not do what my feature does: like
I said, it only changes the priority, or let's rephrase (to account
for the "absolute CPU cycle bandwith" thing): it changes the amount of
CPU cycles a process gets every period.

But it does NOT put an upper limit on total consumed CPU cycles!  It
will only slow down a frantic process, but it will not stop it.
Stopping it is what I want.  Once process crosses the limits I
configured, there's no point in keeping it running.

You may disagree that the feature I implemented is useful, and you may
not want it merged, but do not say that I missed a kernel feature,
because that's not true.

The Linux kernel currently does not have a feature that can emulate
the fork limit that I implemented.  Useful or not, it doesn't exist.

Max
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

[PATCH] cgroup_pids: add fork limit Max Kellermann <mk@cm4all.com> - 2015-11-10 15:20 +0100
  Re: [PATCH] cgroup_pids: add fork limit Tejun Heo <tj@kernel.org> - 2015-11-10 16:20 +0100
    Re: [PATCH] cgroup_pids: add fork limit Max Kellermann <mk@cm4all.com> - 2015-11-10 16:40 +0100
      Re: [PATCH] cgroup_pids: add fork limit Tejun Heo <tj@kernel.org> - 2015-11-10 16:50 +0100
      Re: [PATCH] cgroup_pids: add fork limit Max Kellermann <mk@cm4all.com> - 2015-11-10 18:10 +0100
        Re: [PATCH] cgroup_pids: add fork limit Tejun Heo <tj@kernel.org> - 2015-11-10 18:30 +0100
          Re: [PATCH] cgroup_pids: add fork limit Max Kellermann <mk@cm4all.com> - 2015-11-10 19:00 +0100
  Re: [PATCH] cgroup_pids: add fork limit Aleksa Sarai <cyphar@cyphar.com> - 2015-11-10 16:30 +0100
    Re: [PATCH] cgroup_pids: add fork limit Austin S Hemmelgarn <ahferroin7@gmail.com> - 2015-11-10 17:00 +0100
      Re: [PATCH] cgroup_pids: add fork limit Parav Pandit <pandit.parav@gmail.com> - 2015-11-10 17:20 +0100
        Re: [PATCH] cgroup_pids: add fork limit Austin S Hemmelgarn <ahferroin7@gmail.com> - 2015-11-10 21:00 +0100
          Re: [PATCH] cgroup_pids: add fork limit Aleksa Sarai <cyphar@cyphar.com> - 2015-11-15 14:40 +0100
    Re: [PATCH] cgroup_pids: add fork limit Max Kellermann <mk@cm4all.com> - 2015-11-10 17:10 +0100

csiph-web