Re: [PATCH v13 00/11] cpuset: Enable cpuset controller in default hierarchy

2018-10-15 Thread Waiman Long
On 10/15/2018 12:35 PM, Tejun Heo wrote: > Hello, Waiman. > > This looks great to me. I have only one small nit in terms of > interface. Currently, cpuset.partition file uses -1, 0, 1; however, > given that this is consistent with how cgroup.type behaves (something > can be set but may be invalid

Re: [PATCH v13 00/11] cpuset: Enable cpuset controller in default hierarchy

2018-10-15 Thread Tejun Heo
Hello, Waiman. This looks great to me. I have only one small nit in terms of interface. Currently, cpuset.partition file uses -1, 0, 1; however, given that this is consistent with how cgroup.type behaves (something can be set but may be invalid), I wonder whether using a similar syntax would be

[PATCH v13 00/11] cpuset: Enable cpuset controller in default hierarchy

2018-10-12 Thread Waiman Long
v13: - A major rewrite of the partition code so that there will be no auto-turning off anymore. Instead, the partition root can enter into an error state that can be restored back to a partition root later on. - Patches 1 and 9 are the same as previous version, the rests are either new o