cgroup/pids: Avoid spurious event notification
authorXiu Jianfeng <xiujianfeng@huawei.com>
Tue, 30 Jul 2024 03:29:20 +0000 (03:29 +0000)
committerTejun Heo <tj@kernel.org>
Tue, 30 Jul 2024 22:13:19 +0000 (12:13 -1000)
commitd72a00a8485d1cb11ac1a57bf89b02cbd3a405bf
tree6bba6e6e3746edb9350b0b283b7b3d36abc1d567
parentd6326047576266991d88639e1e9739a9a9a20ef4
cgroup/pids: Avoid spurious event notification

Currently when a process in a group forks and fails due to it's
parent's max restriction, all the cgroups from 'pids_forking' to root
will generate event notifications but only the cgroups from
'pids_over_limit' to root will increase the counter of PIDCG_MAX.

Consider this scenario: there are 4 groups A, B, C,and D, the
relationships are as follows, and user is watching on C.pids.events.

root->A->B->C->D

When a process in D forks and fails due to B.max restriction, the
user will get a spurious event notification because when he wakes up
and reads C.pids.events, he will find that the content has not changed.

To address this issue, only the cgroups from 'pids_over_limit' to root
will have their PIDCG_MAX counters increased and event notifications
generated.

Fixes: 385a635cacfe ("cgroup/pids: Make event counters hierarchical")
Signed-off-by: Xiu Jianfeng <xiujianfeng@huawei.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
kernel/cgroup/pids.c