Re: [PATCH 1/2] Tracking cgroup-level niced CPU time
From: Joshua Hahn
Date: Thu Aug 29 2024 - 15:26:59 EST
Hello, thank you for reviewing the patch.
> > Cgroup-level CPU statistics currently include time spent on
> > user/system processes, but do not include niced CPU time (despite
> > already being tracked). This patch exposes niced CPU time to the
> > userspace, allowing users to get a better understanding of their
> > hardware limits and can facilitate better load-balancing.
>
> You aren't talking about the in-kernel scheduler's load balancer, right? If
> so, can you please update the description? This is a bit too confusing for a
> commit message for a kernel commit.
Thank you for pointing this out -- I'll edit the commit message to the
following in a v2:
Cgroup-level CPU statistics currently include time spent on
user/system processes, but do not include niced CPU time (despite
already being tracked). This patch exposes niced CPU time to the
userspace, allowing users to get a better understanding of their
hardware limits and can facilitate more informed workload distribution.
Thanks,
Joshua