Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1430769

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

Show all headers | View raw

Hello,

On Fri, Jun 24, 2016 at 01:00:48PM +1000, Aleksa Sarai wrote:
> This allows users to dynamically adjust their limits based on how many
> failed forks happened since they last reset their limits, otherwise they
> would have to track (in a racy way) how many limit failures there were
> since the last limit change manually. In addition, we log the first
> failure since the limit was reset (which was the original semantics of
> the patchset).

Isn't that trivially achievable by reading the counter and then
calculating the diff?  I don't think it matters all that much whether
the log message is printed once per cgroup or per config-change.  It's
just a hint for the admin to avoid setting her off on a wild goose
chase.

Thanks.

-- 
tejun

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

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