[lkml]   [2004]   [Apr]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] mask ADT: new mask.h file [2/22]
    On Tue, 2004-04-06 at 16:06, Paul Jackson wrote:
    > > You'll have covered about 300 of them. I don't think a complete
    > > abstraction is actually required or desirable:
    > I suspect we've hit on our first area of actual disagreement here.
    > You observe that providing inline wrappers for the 5 most commonly
    > used cpumask macros would cover 300 of the 420 uses. The other 23
    > or so macros are less commonly used. Sounds about right ...
    > I prefer to provide all 28 macros. I don't see a cost, but do see
    > a gain.

    Because I believe one should *always* resist the urge to write
    infrastructure. Wait until the users of your functionality gather out
    the front of your house with torches because they're all sick of the
    burden of using existing infrastructure.


    I don't even want to learn 28 bitops primitives. I certainly don't want
    to learn 28 nodemask and 28 cpumask primitives.

    I prefer a single set of operators, but preempting complaints means
    figuring out what people want. I'd be happy with the obviously
    cpu-specific ones, myself:


    > The gain is that someone coding some operations on a cpumask doesn't
    > have to go fishing around in multiple places to find out what ops
    > are supported

    Agreed. That's a big benefit of cutting it out altogether.

    > Just to be specific, a typical implementation for such an operator would look like:
    > typedef struct { DECLARE_BITMAP(bits, NR_CPUS); } cpumask_t;
    > static inline void cpus_or(cpumask_t d, const cpumask_t s1, const cpumask_t s2)
    > {
    > bitmap_or(d.bits, s1.bits, s2.bits, NR_CPUS);
    > }

    That'd be a noop, I think.

    Anyone who quotes me in their signature is an idiot -- Rusty Russell

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 14:02    [W:0.031 / U:237.956 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site