Re: [PATCH 1/2] cgroup: pids: show number of failed forks since limit reset

From	Tejun Heo <tj@kernel.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH 1/2] cgroup: pids: show number of failed forks since limit reset
Date	2016-06-27 01:50 +0200
Message-ID	<rOrYl-We-3@gated-at.bofh.it> (permalink)
References	<rNpvA-2g4-11@gated-at.bofh.it> <rNpvA-2g4-13@gated-at.bofh.it> <rNBn4-1op-31@gated-at.bofh.it> <rOgzU-2mY-11@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hello,

On Sun, Jun 26, 2016 at 09:34:41PM +1000, Aleksa Sarai wrote:
> If a user has a setup where they wait for notifications on changes to
> pids.event, and then auto-adjust the cgroup limits based on the number of
> failures you have a race condition between reading the pids.event file and
> then setting the new limit. Then, upon getting notified again there may have
> been many failed forks with the old limit set, so you might decide to bump
> up the limit again.
> 
> It's not a huge deal, I just though it could be useful to alleviate problems
> like the above.

This is something which can easily be avoided from userland.  I don't
think we need to add extra facilities for this.

Thanks.

-- 
tejun

Thread

[PATCH 1/2] cgroup: pids: show number of failed forks since limit reset Aleksa Sarai <asarai@suse.de> - 2016-06-24 05:00 +0200
  Re: [PATCH 1/2] cgroup: pids: show number of failed forks since  limit reset Tejun Heo <tj@kernel.org> - 2016-06-24 17:40 +0200
    Re: [PATCH 1/2] cgroup: pids: show number of failed forks since limit  reset Aleksa Sarai <asarai@suse.de> - 2016-06-26 13:40 +0200
      Re: [PATCH 1/2] cgroup: pids: show number of failed forks since  limit reset Tejun Heo <tj@kernel.org> - 2016-06-27 01:50 +0200
        Re: [PATCH 1/2] cgroup: pids: show number of failed forks since limit  reset Aleksa Sarai <asarai@suse.de> - 2016-06-29 07:10 +0200
          Re: [PATCH 1/2] cgroup: pids: show number of failed forks since  limit reset Tejun Heo <tj@kernel.org> - 2016-06-29 17:40 +0200

csiph-web