xen: Small cleanups towards allowing stricter compile warnings.

Signed-off-by: Christoph Egger <Christoph.Egger@amd.com>

diff --git a/xen/common/sched_sedf.c b/xen/common/sched_sedf.c
index 8a1223a03b0f6ebc5fb36904cc2901f50f40b1e4..7b4eafb63212ca8ffe16bde41d62ee59be4daf09 100644 (file)
--- a/xen/common/sched_sedf.c
+++ b/xen/common/sched_sedf.c
@@ -287,18 +287,18 @@ static inline void list_insert_sort(
     list_add(element, cur->prev);
 }
 
-#define DOMAIN_COMPARER(name, field, comp1, comp2)          \
-int name##_comp(struct list_head* el1, struct list_head* el2) \
-{                                                           \
-    struct sedf_vcpu_info *d1, *d2;                     \
-    d1 = list_entry(el1,struct sedf_vcpu_info, field);  \
-    d2 = list_entry(el2,struct sedf_vcpu_info, field);  \
-    if ( (comp1) == (comp2) )                             \
-        return 0;                                   \
-    if ( (comp1) < (comp2) )                              \
-        return -1;                                  \
-    else                                                \
-        return 1;                                   \
+#define DOMAIN_COMPARER(name, field, comp1, comp2)                      \
+static int name##_comp(struct list_head* el1, struct list_head* el2)    \
+{                                                                       \
+    struct sedf_vcpu_info *d1, *d2;                                     \
+    d1 = list_entry(el1,struct sedf_vcpu_info, field);                  \
+    d2 = list_entry(el2,struct sedf_vcpu_info, field);                  \
+    if ( (comp1) == (comp2) )                                           \
+        return 0;                                                       \
+    if ( (comp1) < (comp2) )                                            \
+        return -1;                                                      \
+    else                                                                \
+        return 1;                                                       \
 }
 
 /* adds a domain to the queue of processes which wait for the beginning of the
@@ -1067,7 +1067,7 @@ static inline int should_switch(struct vcpu *cur,
     return 1;
 }
 
-void sedf_wake(struct vcpu *d)
+static void sedf_wake(struct vcpu *d)
 {
     s_time_t              now = NOW();
     struct sedf_vcpu_info* inf = EDOM_INFO(d);

diff --git a/xen/common/stop_machine.c b/xen/common/stop_machine.c
index 1411b9f1979ff6d4c134d002213e4741d6dc6a81..f7d9071b6668434e7a8511090a472860a0184a3b 100644 (file)
--- a/xen/common/stop_machine.c
+++ b/xen/common/stop_machine.c
@@ -25,6 +25,7 @@
 #include <xen/sched.h>
 #include <xen/spinlock.h>
 #include <xen/softirq.h>
+#include <xen/stop_machine.h>
 #include <xen/errno.h>
 #include <xen/smp.h>
 #include <asm/current.h>

diff --git a/xen/common/version.c b/xen/common/version.c
index 6afa128cd024ec99d13de2727af56a770e43a120..b152e27c6882229a42dcdae47aed951d9813dc43 100644 (file)
--- a/xen/common/version.c
+++ b/xen/common/version.c
@@ -1,4 +1,5 @@
 #include <xen/compile.h>
+#include <xen/version.h>
 
 const char *xen_compile_date(void)
 {

diff --git a/xen/drivers/acpi/reboot.c b/xen/drivers/acpi/reboot.c
index ae03526f1754703d9e5d9076d4a994ef54956650..95edc1eb327e736f09b75950f49e738da0c92aa6 100644 (file)
--- a/xen/drivers/acpi/reboot.c
+++ b/xen/drivers/acpi/reboot.c
@@ -1,5 +1,6 @@
 #include <xen/config.h>
 #include <xen/pci.h>
+#include <xen/acpi.h>
 #include <acpi/acpi.h>
 
 void acpi_reboot(void)

diff --git a/xen/drivers/passthrough/amd/iommu_intr.c b/xen/drivers/passthrough/amd/iommu_intr.c
index 5dfdff084e07ab02eed959fc74183ef66316e87a..ef31c5dd6e1dcf2f06d0c119cde9bb2344203c92 100644 (file)
--- a/xen/drivers/passthrough/amd/iommu_intr.c
+++ b/xen/drivers/passthrough/amd/iommu_intr.c
@@ -25,7 +25,7 @@
 DEFINE_SPINLOCK(int_remap_table_lock);
 void *int_remap_table = NULL;
 
-u8* get_intremap_entry(u8 vector, u8 dm)
+static u8 *get_intremap_entry(u8 vector, u8 dm)
 {
     u8 *table;
     int offset = 0;