diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2005-10-16 00:17:33 -0700 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2005-10-16 00:17:33 -0700 |
| commit | 688ce17b8599abc548b406c00e4d18ae0dec954f (patch) | |
| tree | c64c78c72bf9582c2dcc5a3455d6fe561c6e7085 | |
| parent | e6850cce8f0fcb0e16b981f13cb9c69618bbdaf1 (diff) | |
| download | lwn-688ce17b8599abc548b406c00e4d18ae0dec954f.tar.gz lwn-688ce17b8599abc548b406c00e4d18ae0dec954f.zip | |
[PATCH]: highest_possible_processor_id() has to be a macro
... otherwise, things like alpha and sparc64 break and break
badly. They define cpu_possible_map to something else in smp.h
*AFTER* having included cpumask.h.
If that puppy is a macro, expansion will happen at the actual
caller, when we'd already seen #define cpu_possible_map ... and we will
get the right thing used.
As an inline helper it will be tokenized before we get to that
define and that's it; no matter what we define later, it won't affect
anything. We get modules with dependency on cpu_possible_map instead
of the right symbol (phys_cpu_present_map in case of sparc64), or outright
link errors if they are built-in.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | include/linux/cpumask.h | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/include/linux/cpumask.h b/include/linux/cpumask.h index fe9778301d07..9bdba8169b41 100644 --- a/include/linux/cpumask.h +++ b/include/linux/cpumask.h @@ -393,15 +393,13 @@ extern cpumask_t cpu_present_map; #define for_each_present_cpu(cpu) for_each_cpu_mask((cpu), cpu_present_map) /* Find the highest possible smp_processor_id() */ -static inline unsigned int highest_possible_processor_id(void) -{ - unsigned int cpu, highest = 0; - - for_each_cpu_mask(cpu, cpu_possible_map) - highest = cpu; - - return highest; -} +#define highest_possible_processor_id() \ +({ \ + unsigned int cpu, highest = 0; \ + for_each_cpu_mask(cpu, cpu_possible_map) \ + highest = cpu; \ + highest; \ +}) #endif /* __LINUX_CPUMASK_H */ |