Re: [PATCH] cgroup_pids: add fork limit

Path	csiph.com!eternal-september.org!feeder.eternal-september.org!aioe.org!bofh.it!news.nic.it!robomod
From	Max Kellermann <mk@cm4all.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] cgroup_pids: add fork limit
Date	Tue, 10 Nov 2015 18:10:01 +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>
X-Original-To	Tejun Heo <tj@kernel.org>, cgroups@vger.kernel.org, linux-kernel@vger.kernel.org
Mail-Followup-To	Tejun Heo <tj@kernel.org>, cgroups@vger.kernel.org, linux-kernel@vger.kernel.org
MIME-Version	1.0
Content-Type	text/plain; charset=us-ascii
Content-Disposition	inline
User-Agent	Mutt/1.5.24 (2015-08-30)
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	30
Organization	linux.* mail to news gateway
X-Original-Date	Tue, 10 Nov 2015 18:06:12 +0100
X-Original-Message-ID	<20151110170612.GA21582@rabbit.intern.cm-ag>
X-Original-References	<144716440621.20175.1000688899886388119.stgit@rabbit.intern.cm-ag> <20151110151223.GA17938@mtj.duckdns.org> <20151110153746.GA20758@rabbit.intern.cm-ag>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1266648

Show key headers only | 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