Re: [PATCH 1/2] sched/fair: Add cfs bandwidth burst statistics
From: Tejun Heo
Date: Mon Aug 23 2021 - 15:03:05 EST
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