Re: [PATCH v3 -next 2/2] blk-cgroup: synchoronize blkg creation against policy deactivation

From: Tejun Heo
Date: Tue Oct 19 2021 - 13:03:19 EST


Hello,

Some nitpicks.

On Tue, Oct 19, 2021 at 10:41:32AM +0800, Yu Kuai wrote:
...
> Because blkcg_deactivate_policy() require queue to be freezed, thus grab
^ ^ ^
requires frozen we can grab

> q_usage_counter to synchoronize blkg_conf_prep() against
> blkcg_deactivate_policy().
...
> + /*
> + * blkcg_deactivate_policy() require queue to be freezed, thus grab
> + * q_usage_counter to prevent concurrent with blkcg_deactivate_policy().
> + */

Same here.

Looks good otherwise. Please feel free to add

Acked-by: Tejun Heo <tj@xxxxxxxxxx>

Thanks.

--
tejun