[Xen-ia64-devel] Re: PATCH: implements DOM0_DOMAIN_SETUP

[Alex Williamson <alex.williamson@xxxxxx>]

On Mon, 2006-06-26 at 16:01 +0200, Tristan Gingold wrote:
> Hi,
> 
> because DOM0_DOMAIN_SETUP has been added in common code, I repost my patch 
> (slightly updated).
> This patch doesn't add features for user: s/he just have to recompile the 
> tools, the hypervisor and the kernel :-)
> 
> The main change is cleanups wrt domain creation.  This is also a first step 
> for save & restore.

Tristan,

   Tools won't build for me with this patch:

gcc  -O2 -fomit-frame-pointer -DNDEBUG -Wall -Wstrict-prototypes 
-Wdeclaration-after-statement  -D__XEN_TOOLS__ -Werror -fno-strict-aliasing  
-I. -Wp,-MD,.xc_linux_build.o.d -c -o xc_linux_build.o xc_linux_build.c
In file included from /usr/include/asm/pal.h:84,
                 from /usr/include/asm/system.h:19,
                 from xc_linux_build.c:19:
/usr/include/asm/fpu.h:57: error: redefinition of `struct ia64_fpreg'
In file included from /usr/include/asm/system.h:19,
                 from xc_linux_build.c:19:
/usr/include/asm/pal.h:92: error: syntax error before "pal_status_t"
/usr/include/asm/pal.h:105: error: syntax error before "pal_cache_level_t"
/usr/include/asm/pal.h:113: error: syntax error before "pal_cache_type_t"
/usr/include/asm/pal.h:126: error: syntax error before "pal_cache_line_state_t"
/usr/include/asm/pal.h:133: error: syntax error before "u64"
...

It's making me a little uncomfortable to include <asm/system.h> in that
file given that there are no other asm includes.  Thanks,

        Alex

-- 
Alex Williamson                             HP Open Source & Linux Org.


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