2002-10-29 23:59:31

by Matthew Dobson

[permalink] [raw]
Subject: Re: [Linux-ia64] Re: [PATCH] topology for ia64

William Lee Irwin III wrote:
> On Tue, Oct 29, 2002 at 02:19:25PM -0800, Matthew Dobson wrote:
> +/*
> + * Returns the number of the first CPU on Node 'node'.
> + * Slow in the current implementation.
> + * Who needs this?
> + */
> +/* #define __node_to_first_cpu(node) pool_cpus[pool_ptr[node]] */
> +static inline int __node_to_first_cpu(int node)
>
> So far so safe... though no obvious use of it.

Yep...


> On Tue, Oct 29, 2002 at 02:19:25PM -0800, Matthew Dobson wrote:
>
>>No one is using it now. I think that I will probably deprecate this
>>function in the near future as it is pretty useless. Anyone looking for
>>that functionality can just do an __ffs(__node_to_cpu_mask(node))
>>instead, and hope that there is a reasonably quick implementation of
>>__node_to_cpu_mask.
>
>
> This assumes the value returned by __node_to_cpu_mask() is a single word.

Which is the case right now. When (not if) that changes, we'll come up
with more flexible ffs macros, or a better way to count variable length
bitmasks... especially as there will be a TON of them.

Cheers!

-Matt