Re: [RFC PATCH v2 08/17] cgroup: Move debug cgroup to its own file

From: Tejun Heo
Date: Wed May 17 2017 - 17:36:56 EST


Hello, Waiman.

On Mon, May 15, 2017 at 09:34:07AM -0400, Waiman Long wrote:
> The debug cgroup currently resides within cgroup-v1.c and is enabled
> only for v1 cgroup. To enable the debug cgroup also for v2, it
> makes sense to put the code into its own file as it will no longer
> be v1 specific. The only change in this patch is the expansion of
> cgroup_task_count() within the debug_taskcount_read() function.
>
> Signed-off-by: Waiman Long <longman@xxxxxxxxxx>

I don't mind enabling the debug controller for v2 but let's please
hide it behind an unwieldy boot param / controller name so that it's
clear that its interface isn't expected to be stable.

Thanks.

--
tejun