Re: [PATCH v2 01/10] blk-throttle: correct stale comment in throtl_pd_init

From: Tejun Heo
Date: Wed Nov 30 2022 - 16:08:20 EST


On Tue, Nov 29, 2022 at 11:01:38AM +0800, Kemeng Shi wrote:
> On the default hierarchy (cgroup2), the throttle interface files don't
> exist in the root cgroup, so the ablity to limit the whole system
> by configuring root group is not existing anymore. In general, cgroup
> doesn't wanna be in the business of restricting resources at the
> system level, so correct the stale comment that we can limit whole
> system to we can only limit subtree.
>
> Signed-off-by: Kemeng Shi <shikemeng@xxxxxxxxxx>

Acked-by: Tejun Heo <tj@xxxxxxxxxx>

> + * read_bps limit is set on a "parent" group, summary bps of
> + * "parent" group and its subtree groups can't exceed 16M for the

but why the quotes around parent?

--
tejun