smp_call_function/on_selected_cpus/on_each_cpu all return void.

None of them can fail, so a return code is pointless.

Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

1 files changed, 3 insertions, 4 deletions

diff --git a/xen/arch/x86/smp.c b/xen/arch/x86/smp.c
index 0483ac5f10..650835ff1f 100644
--- a/xen/arch/x86/smp.c
+++ b/xen/arch/x86/smp.c
@@ -274,17 +274,17 @@ static struct call_data_struct {
     cpumask_t selected;
 } call_data;
 
-int smp_call_function(
+void smp_call_function(
     void (*func) (void *info),
     void *info,
     int wait)
 {
     cpumask_t allbutself = cpu_online_map;
     cpu_clear(smp_processor_id(), allbutself);
-    return on_selected_cpus(&allbutself, func, info, wait);
+    on_selected_cpus(&allbutself, func, info, wait);
 }
 
-int on_selected_cpus(
+void on_selected_cpus(
     const cpumask_t *selected,
     void (*func) (void *info),
     void *info,
@@ -323,7 +323,6 @@ int on_selected_cpus(
 
  out:
     spin_unlock(&call_lock);
-    return 0;
 }
 
 static void __stop_this_cpu(void)