Re: [PATCH v3] cpuset: Enable cpuset controller in default hierarchy

From: Tejun Heo
Date: Mon Nov 27 2017 - 16:42:41 EST


Hello, Waiman.

On Mon, Nov 27, 2017 at 04:19:57PM -0500, Waiman Long wrote:
> > Let's start just with [e]cpus and [e]mems. The flags interface looks
> > fine but the implementations of these features are really bad and
> > cgroup2 doesn't migrate resources for other controllers either anyway.
>
> That is added because the mem_migrate feature is used in libvirt, I
> think. I am thinking of add a "[EXPERIMENTAL]" tag to the flags to
> indicate that it is subject to change.

I see. Do you happen to know what it's used for and why that's
necessary just so that we can evaluate it better? I'm not quite sure
what adding [EXPERIMENTAL] tag would achieve. If we expose the
feature and people use it, we just have to keep it anyway.

Thanks.

--
tejun