lkml.org 
[lkml]   [2003]   [Dec]   [21]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateSun, 21 Dec 2003 18:14:18 -0800
FromWilliam Lee Irwin III <>
SubjectRe: [PATCH] another minor bit of cpumask cleanup
On Sun, Dec 21, 2003 at 06:00:44PM -0800, Paul Jackson wrote:
> Looking further, I see this macro is never used, and its subordinate
> inline macro next_online_cpu() used no where else.  What's more, it's
> redundant.  Calling it with a map of "cpu_online_map" (which you have to
> do, given it's broken thus) is just as good as calling the macro right
> above, "for_each_cpu()", with that same "cpu_online_map". Indeed the
> only uses of "for_each_cpu()", in arch/i386/mach-voyager/voyager_smp.c,
> do pass "cpu_online_map" explicitly, in 5 of 6 calls there from.

Callers couldn't be converted without risking a "cleanup factor". It's
not terribly surprising some issue might appear since it wasn't used.

I don't honestly care if it lives or dies; it appeared to make more
sense as a generic macro than a voyager-specific macro at the time.


-- 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 12:59    [from the cache]
©2003-2008