From: Linus Torvalds <[email protected]>
Date: Wed, 23 Nov 2005 08:54:46 -0800 (PST)
> The way to handle it is to do
>
> static inline int maybe_ignored(int arg, ...)
> {
> return arg;
> }
>
> #define smp_call_function(func,info,retry,wait) \
> maybe_ignored(0, info, retry, wait)
>
> which is a very useful way to say: we don't care about "func", but we want
> to avoid unused warnings for "info", "retry" and "wait", and we want to
> return 0 regardless and compile it all away.
>
> If somebody tests this, puts the "maybe_ignored()" function in some nice
> generic header file, I'll apply it.
I quickly hacked this up and did a UP test build (patch at the end),
but there is another consequence to consider.
With this, we have to either:
1) Mark all IPI functions with ifdef CONFIG_SMP, or
2) Mark them with __attribute__((__unused__))) which is what
the net/core/flow.c case does
Because if we just leave the static IPI functions there without the
CONFIG_SMP ifdef and without the unused attribute, this new
smp_call_function() will generate an unused static function warning.
What we could do is hide that detail behind some kind of
"DEFINE_IPI_FUNC()" macro, and put the gore into a header file.
I'm sure there are other clean ways of handling it.
diff --git a/include/linux/kernel.h b/include/linux/kernel.h
index b1e407a..1876d3c 100644
--- a/include/linux/kernel.h
+++ b/include/linux/kernel.h
@@ -286,6 +286,11 @@ extern void dump_stack(void);
1; \
})
+static inline int maybe_ignored(int arg, ...)
+{
+ return arg;
+}
+
#endif /* __KERNEL__ */
#define SI_LOAD_SHIFT 16
diff --git a/include/linux/smp.h b/include/linux/smp.h
index 9dfa3ee..2a513fe 100644
--- a/include/linux/smp.h
+++ b/include/linux/smp.h
@@ -94,7 +94,8 @@ void smp_prepare_boot_cpu(void);
*/
#define raw_smp_processor_id() 0
#define hard_smp_processor_id() 0
-#define smp_call_function(func,info,retry,wait) ({ 0; })
+#define smp_call_function(func,info,retry,wait) \
+ maybe_ignored(0, info, retry, wait)
#define on_each_cpu(func,info,retry,wait) ({ func(info); 0; })
static inline void smp_send_reschedule(int cpu) { }
#define num_booting_cpus() 1
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
