Messages in this thread | | | Date | Mon, 23 Aug 2021 09:03:00 -1000 | From | Tejun Heo <> | Subject | Re: [PATCH 1/2] sched/fair: Add cfs bandwidth burst statistics |
| |
On Mon, Aug 23, 2021 at 02:32:10PM -0400, Daniel Jordan wrote: > burst_time is nsec, not usec. > > Just "burst_time" seems the way to go for consistency with > throttled_time in the same file, though "burst_usec" nicely has units > and would have the same name between cgroup1 and 2.
The cgroup2 interface rule is that time units are in usecs. The right thing to do is keeping the _usec postfix (to avoid confusion due to the unit change from cgroup1) and converting it to usecs for printing.
Thanks.
-- tejun
| |