[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250502121930.4008251-1-max.kellermann@ionos.com>
Date: Fri, 2 May 2025 14:19:30 +0200
From: Max Kellermann <max.kellermann@...os.com>
To: tj@...nel.org,
hannes@...xchg.org,
mkoutny@...e.com,
cgroups@...r.kernel.org.com,
linux-kernel@...r.kernel.org
Cc: Max Kellermann <max.kellermann@...os.com>
Subject: [PATCH] kernel/cgroup/pids: add "pids.forks" counter
Counts the number of fork()/clone() calls, similar to the "processes"
row in /proc/stat, but per cgroup. This helps with analyzing who was
responsible for peaks in the global "processes" counter.
Signed-off-by: Max Kellermann <max.kellermann@...os.com>
---
Documentation/admin-guide/cgroup-v2.rst | 5 +++++
kernel/cgroup/pids.c | 18 ++++++++++++++++++
2 files changed, 23 insertions(+)
diff --git a/Documentation/admin-guide/cgroup-v2.rst b/Documentation/admin-guide/cgroup-v2.rst
index 1a16ce68a4d7..88f996e083e2 100644
--- a/Documentation/admin-guide/cgroup-v2.rst
+++ b/Documentation/admin-guide/cgroup-v2.rst
@@ -2294,6 +2294,11 @@ PID Interface Files
The maximum value that the number of processes in the cgroup and its
descendants has ever reached.
+ pids.forks
+ A read-only single value file which exists on non-root cgroups.
+
+ The number of fork()/clone() calls (whether successful or not).
+
pids.events
A read-only flat-keyed file which exists on non-root cgroups. Unless
specified otherwise, a value change in this file generates a file
diff --git a/kernel/cgroup/pids.c b/kernel/cgroup/pids.c
index 8f61114c36dd..fb18741f85ba 100644
--- a/kernel/cgroup/pids.c
+++ b/kernel/cgroup/pids.c
@@ -49,6 +49,9 @@ enum pidcg_event {
struct pids_cgroup {
struct cgroup_subsys_state css;
+ /* the "pids.forks" counter */
+ atomic64_t forks;
+
/*
* Use 64-bit types so that we can safely represent "max" as
* %PIDS_MAX = (%PID_MAX_LIMIT + 1).
@@ -147,6 +150,7 @@ static void pids_charge(struct pids_cgroup *pids, int num)
struct pids_cgroup *p;
for (p = pids; parent_pids(p); p = parent_pids(p)) {
+ atomic64_add(num, &p->forks);
int64_t new = atomic64_add_return(num, &p->counter);
pids_update_watermark(p, new);
@@ -168,6 +172,7 @@ static int pids_try_charge(struct pids_cgroup *pids, int num, struct pids_cgroup
struct pids_cgroup *p, *q;
for (p = pids; parent_pids(p); p = parent_pids(p)) {
+ atomic64_add(num, &p->forks);
int64_t new = atomic64_add_return(num, &p->counter);
int64_t limit = atomic64_read(&p->limit);
@@ -342,6 +347,14 @@ static int pids_max_show(struct seq_file *sf, void *v)
return 0;
}
+static s64 pids_forks_read(struct cgroup_subsys_state *css,
+ struct cftype *cft)
+{
+ struct pids_cgroup *pids = css_pids(css);
+
+ return atomic64_read(&pids->forks);
+}
+
static s64 pids_current_read(struct cgroup_subsys_state *css,
struct cftype *cft)
{
@@ -404,6 +417,11 @@ static struct cftype pids_files[] = {
.flags = CFTYPE_NOT_ON_ROOT,
.read_s64 = pids_peak_read,
},
+ {
+ .name = "forks",
+ .read_s64 = pids_forks_read,
+ .flags = CFTYPE_NOT_ON_ROOT,
+ },
{
.name = "events",
.seq_show = pids_events_show,
--
2.47.2
Powered by blists - more mailing lists