Hi Rindong,
Chen Ridong <chenridong@xxxxxxxxxxxxxxx> writes:
On 2025/7/17 20:56, Michal Koutný wrote:
Hello Tiffany.
On Sun, Jul 13, 2025 at 10:00:09PM -0700, Tiffany Yang
<ynaffit@xxxxxxxxxx> wrote:
--- a/Documentation/admin-guide/cgroup-v2.rst
+++ b/Documentation/admin-guide/cgroup-v2.rst
@@ -1018,6 +1018,14 @@ All cgroup core files are prefixed with "cgroup."
it's possible to delete a frozen (and empty) cgroup, as well as
create new sub-cgroups.
+ cgroup.freeze.stat
With the given implementation (and use scenario), this'd better exposed
in
cgroup.freeze.stat.local
Would it be possible to add this field to either cgroup.event or
cgroup.stat?
Since the frozen status is already tracked in cgroup.event, this
placement would maintain better
cohesion with existing metrics.
This is just a suggestion.
Best regards,
Ridong
Thanks for taking a look!
I don't think this would *quite* fit in cgroup.event because we're
measuring when the cgroup begins freezing instead of when it reaches the
frozen state. I also worry that having the value so close to
cgroup.frozen would cause additional confusion about its
meaning. cgroup.stat seems reasonable, but the values inside appear to
be accounted for hierarchically, which wouldn't suit our use case.
--
Tiffany Y. Yang
