Hi Michal,

On 2021/1/15 18:08, Michal Koutný wrote:

On Fri, Jan 15, 2021 at 09:55:43AM +0800, chenzhou [off-list ref] wrote:

quoted

Yeah, this will select all enabled controllers, but which doesn't the behavior we want.

I see what the issue is now.

quoted

See above. Just the mount behavior isn't what we what.

I agree this a bug and your I find your approach correct

Reviewed-by: Michal Koutný <mkoutny-IBi9RG/b67k@public.gmane.org>

I have sent the v3, which updates the description and add Fixes. 
[v3]: https://lore.kernel.org/patchwork/patch/1365535/
If it is ok for you to add Reviewed-by there.

Thanks,
Chen Zhou

quoted

The behavior was changed since commit f5dfb5315d34 ("cgroup: take
options parsing into ->parse_monolithic()"), will add this as Fixes.

Thanks.

Michal