From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752443AbcFXC4V (ORCPT ); Thu, 23 Jun 2016 22:56:21 -0400 Received: from mx2.suse.de ([195.135.220.15]:43133 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751905AbcFXC4S (ORCPT ); Thu, 23 Jun 2016 22:56:18 -0400 From: Aleksa Sarai To: Jonathan Corbet , Tejun Heo , Li Zefan , Johannes Weiner , Kenny Yu Cc: linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, cgroups@vger.kernel.org, Aleksa Sarai Subject: [PATCH 1/2] cgroup: pids: show number of failed forks since limit reset Date: Fri, 24 Jun 2016 13:00:48 +1000 Message-Id: <20160624030049.13341-2-asarai@suse.de> X-Mailer: git-send-email 2.8.4 In-Reply-To: <20160624030049.13341-1-asarai@suse.de> References: <20160624030049.13341-1-asarai@suse.de> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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). In addition, I clarified the licensing for this file. Signed-off-by: Aleksa Sarai --- kernel/cgroup_pids.c | 31 ++++++++++++++++++++++--------- 1 file changed, 22 insertions(+), 9 deletions(-) diff --git a/kernel/cgroup_pids.c b/kernel/cgroup_pids.c index 2bd673783f1a..54ec3e4f3b71 100644 --- a/kernel/cgroup_pids.c +++ b/kernel/cgroup_pids.c @@ -26,9 +26,10 @@ * * Copyright (C) 2015 Aleksa Sarai * - * This file is subject to the terms and conditions of version 2 of the GNU - * General Public License. See the file COPYING in the main directory of the - * Linux distribution for more details. + * This program is free software; you can redistribute it and/or modify it + * under the terms of the GNU General Public License as published by the Free + * Software Foundation; either version 2 of the License, or (at your option) + * any later version. */ #include @@ -53,8 +54,11 @@ struct pids_cgroup { /* Handle for "pids.events" */ struct cgroup_file events_file; - /* Number of times fork failed because limit was hit. */ - atomic64_t events_limit; + /* Total number of times fork failed because limit was hit. */ + atomic64_t hits_total; + + /* Number of times fork failed since limit was last set. */ + atomic64_t hits_since; }; static struct pids_cgroup *css_pids(struct cgroup_subsys_state *css) @@ -78,7 +82,8 @@ pids_css_alloc(struct cgroup_subsys_state *parent) pids->limit = PIDS_MAX; atomic64_set(&pids->counter, 0); - atomic64_set(&pids->events_limit, 0); + atomic64_set(&pids->hits_total, 0); + atomic64_set(&pids->hits_since, 0); return &pids->css; } @@ -226,8 +231,12 @@ static int pids_can_fork(struct task_struct *task) pids = css_pids(css); err = pids_try_charge(pids, 1); if (err) { - /* Only log the first time events_limit is incremented. */ - if (atomic64_inc_return(&pids->events_limit) == 1) { + /* + * We only log the first time a fork fails after a limit has + * been set. Resetting the limit will cause us to log again. + */ + atomic64_inc(&pids->hits_total); + if (atomic64_inc_return(&pids->hits_since) == 1) { pr_info("cgroup: fork rejected by pids controller in "); pr_cont_cgroup_path(task_cgroup(current, pids_cgrp_id)); pr_cont("\n"); @@ -281,6 +290,9 @@ set_limit: * critical that any racing fork()s follow the new limit. */ pids->limit = limit; + /* Reset the since counter and notify userspace. */ + atomic64_set(&pids->hits_since, 0); + cgroup_file_notify(&pids->events_file); return nbytes; } @@ -310,7 +322,8 @@ static int pids_events_show(struct seq_file *sf, void *v) { struct pids_cgroup *pids = css_pids(seq_css(sf)); - seq_printf(sf, "max %lld\n", (s64)atomic64_read(&pids->events_limit)); + seq_printf(sf, "since\t%lld\n", (s64)atomic64_read(&pids->hits_since)); + seq_printf(sf, "total\t%lld\n", (s64)atomic64_read(&pids->hits_total)); return 0; } -- 2.8.4