Re: [PATCH 01/04]Create x86 directory to hold x86-specific files.

[Avi Kivity <avi@qumranet.com>]

Zhang, Xiantao wrote:
> From 03ac444d1ab4446c587e8180ceaba60b9e75b28d Mon Sep 17 00:00:00 2001
> From: Xiantao Zhang <xiantao.zhang@intel.com>
> Date: Fri, 11 Jul 2008 10:13:08 +0800
> Subject: [PATCH] KVM: external module: Moving x86-speicif files to x86
> directory.
>
> Create x86 directory to hold x86-specific files.
> Signed-off-by: Xiantao Zhang <xiantao.zhang@intel.com>
> ---
>  kernel/{ => x86}/anon_inodes.c            |    0 
>   

This isn't really x86 specific.  It's just kernel version dependent.  
The problem is that it is built unconditionally, even if the kernel has 
anon_inodes support.

Please send a patch that wraps the entire file in #ifdef so that we use 
the host kernel's anon_inodes if it is available.

>  kernel/{ => x86}/external-module-compat.c |    0 
>  kernel/{ => x86}/external-module-compat.h |    0 
>   

Parts of this are generic, for example the mutex code.  Please move only 
the x86 specifc parts (even if ia64 doesn't need everything in the 
generic code).

-- 
error compiling committee.c: too many arguments to function