Hello, On Fri, Jan 15, 2021 at 09:55:43AM +0800, chenzhou wrote: > Yeah, this will select all enabled controllers, but which doesn't the > behavior we want. > I think the case should return error with information "Disabled controller > xx" rather than > attaching all the other enabled controllers. > > For example, boot with cgroup_no_v1=cpu, and then mount with > "mount -t cgroup -o cpu cpu /sys/fs/cgroup/cpu", then all enabled controllers > will > be attached expect cpu.
Okay, that explanation actually makes sense. Can you please update the description to include what's broken and how it's being fixed? It really isn't clear what the patch is trying to achieve from the current description. Thanks. -- tejun