[lkml]   [2008]   [Feb]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [RFC/PATCH] cpuset: cpuset irq affinities
    Peter wrote:
    > IRQ are hardware notifiers and
    > do all kinds of things depending on the hardware.

    So some of these irqs might have very particular node affinity then,
    right? If my thingamajig board is attached to node 72, then I might
    want its interrupts going to a CPU on node 72, right?

    In which case, putting that irq in my boot cpuset that only has nodes
    0-3 would be harmful to the performance of my thingamajig board, right?

    I'm suspecting here that you don't want a small 'boot' cpuset
    (usually a small cpuset running legacy Unix stuff) holding the irqs,
    but rather you want a big 'system' cpuset, which has -all-but- a few
    nodes dedicated to hard real time or other isolated (there's that
    word again) purposes.

    That way, most irqs can go to most CPUs, depending on their specific

    Unfortunately, I don't think the cpuset hierarchy and conventions admit
    of both a big 'system' cpuset (all but a few isolated nodes) and a small
    overlapping 'boot' cpuset.

    > We don't support mv on cgroups, right? To easily create
    > common parents...

    The only mv supported is simple rename, preserving parentage.

    And if one could and did a tree reshaping mv near the top of the
    hierarchy, it would confuse the heck out of existing uses and users.

    > I might just be new-fangled, but I have a /cgroup mount.
    > but I guess that's just different mount-point of cgroup, right?

    All cgroups mount beneath /cgroup. For backwards compatibility,
    one can also "mount -t cpuset cpuset /dev/cpuset", and just get
    the cpuset interface, with a couple of legacy hooks to make it behave
    just like good old fashioned cpusets, rather than new fangled cgroups.

    > I'm fine with whatever, I saw a ',' in the bitmap stuff, not really sure
    > how that ended up being a ' ' in the patch I send out... :-)

    Yes - that's another commonly supported form. If that's a better
    presentation, then you'd probably want to rework your code, to take
    in and display the entire vector of irq numbers in one line, using
    a comma-separated list of irqs and ranges of irqs.

    See further bitmap_scnprintf(), bitmap_parse_user(),
    bitmap_scnlistprintf() and bitmap_parselist(), in bitmap.c.

    Given that you don't have an pre-existing bitmap of irqs (that I know
    of) and that you might have a distinct error code for each irq that you
    try to attach to a different cpuset, I'm guessing you want to stick
    with the single irq per write on input, single irq per line on output,
    paradigm, similar to what the 'tasks' file uses for task pids.

    I won't rest till it's the best ...
    Programmer, Linux Scalability
    Paul Jackson <> 1.940.382.4214

     \ /
      Last update: 2008-02-29 22:57    [W:0.032 / U:12.592 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site