Re: [Xen-devel] [PATCH] Fix ia64 tools build

["KUWAMURA Shin'ya" <kuwa@xxxxxxxxxxxxxx>]

Hi Keir and Ian,

>>>>> On Fri, 29 Oct 2010 08:55:31 +0100
>>>>> keir@xxxxxxx(Keir Fraser)  said:
> 
> I think it would ne neater to have your own
> tools/libxc/ia64/ac_ia64_tools.h, and then conditionally include that from
> Xen's acenv.h (e.g., dependent on __XEN_TOOLS__).

I created a patch for Keir's first advice.

This patch introduces ac_ia64_tools.h and it only defines identifiers
in order to avoid compilation errors.

Because all identifiers can be easily led from the error message, 
it is trivial.

Any comments?

Best regards,
-- 
  KUWAMURA Shin'ya

diff -r bbffb400f507 tools/libxc/ia64/ac_ia64_tools.h
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/tools/libxc/ia64/ac_ia64_tools.h  Mon Nov 01 17:15:10 2010 +0900
@@ -0,0 +1,17 @@
+#ifndef AC_IA64_TOOLS_H
+#define AC_IA64_TOOLS_H
+
+#define ACPI_MACHINE_WIDTH 64
+#define COMPILER_DEPENDENT_UINT64 unsigned long long
+#define COMPILER_DEPENDENT_INT64 long long
+typedef unsigned long long u64;
+typedef long long s64;
+typedef unsigned u32;
+typedef int s32;
+typedef unsigned char u8;
+typedef unsigned short u16;
+#define __iomem
+#define asmlinkage
+#define CONFIG_ACPI_BOOT
+
+#endif /* AC_IA64_TOOLS_H */
diff -r bbffb400f507 xen/include/acpi/platform/acenv.h
--- a/xen/include/acpi/platform/acenv.h Mon Nov 01 15:35:03 2010 +0900
+++ b/xen/include/acpi/platform/acenv.h Mon Nov 01 17:15:10 2010 +0900
@@ -136,7 +136,9 @@
 
 /*! [Begin] no source code translation */
 
-#if 1/*defined(_LINUX) || defined(__linux__)*/
+#if defined(__XEN_TOOLS__) && defined(__ia64__)
+#include "ac_ia64_tools.h"
+#elif 1 /*defined(_LINUX) || defined(__linux__)*/
 #include "aclinux.h"
 
 #elif defined(_AED_EFI)

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel