lkml.org 
[lkml]   [2018]   [Oct]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v13 00/11] cpuset: Enable cpuset controller in default hierarchy
    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 more consistent. e.g. sth like cpuset.type which
    takes "normal" or "partition" and shows one of "normal", "partition"
    and "partition invalid". I'd be perfectly happy with that being a
    follow-up patch.

    Peter, does the patchset look good to you? It does everything we want
    it to in terms of sched domain partitioning and is compliant all
    cgroup2 conventions. If so, I'll apply the series to the cgroup devel
    branch.

    Thank you very much for the work and persistence!

    --
    tejun

    \
     
     \ /
      Last update: 2018-10-15 18:37    [W:2.919 / U:0.076 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site