lkml.org 
[lkml]   [2003]   [Dec]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] another minor bit of cpumask cleanup
At some point in the past, akpm wrote:
>> Please, hang onto it until we get things synced up a bit more.

On Sun, Dec 21, 2003 at 11:19:18PM -0800, Paul Jackson wrote:
> Ok - good idea. I'll resend later on. There is no hurry on this one.

A rereading of this thread reveals the point of the thing was missed.
It's supposed to iterate over online cpus in an given bitmap. It was
meant to replace iterations like:

for_each_cpu(cpu, mask) {
if (!cpu_online(cpu))
continue;
do_something(cpu);
}
with

for_each_online_cpu(cpu, mask)
do_something(cpu);

Using any_online_cpu() as the starting point repairs it, since that
properly ands mask with cpu_online_map and hands back the first cpu,
though it's only a coincidence it hands back the first such cpu. There
isn't a a first_online_cpu() in the API, that's just effectively what
any_online_cpu() does at the moment.

On the other hand, I just don't care anymore, apart from clarifying
intent so as to counter the implication that all I did back then was
crap gibberish all over the tree. I personally have received zero
recognition or other return on my efforts in this area apart from the
mere fact it was merged. In fact, rather the opposite.


-- wli
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:59    [from the cache]
©2003-2011 Jasper Spaans