diff mbox

Only compile KVM support for matching CPU types

Message ID 1245883217-27455-1-git-send-email-aliguori@us.ibm.com (mailing list archive)
State New, archived
Headers show

Commit Message

Anthony Liguori June 24, 2009, 10:40 p.m. UTC
I don't know why this is disabled for qemu-kvm.git.

Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
---
 configure |    5 +----
 1 files changed, 1 insertions(+), 4 deletions(-)

Comments

Dustin Kirkland June 24, 2009, 11:09 p.m. UTC | #1
On Wed, 2009-06-24 at 17:40 -0500, Anthony Liguori wrote:
> I don't know why this is disabled for qemu-kvm.git.
> 
> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
> ---
>  configure |    5 +----
>  1 files changed, 1 insertions(+), 4 deletions(-)
> 
> diff --git a/configure b/configure
> index 4bad3c3..1b73eaf 100755
> --- a/configure
> +++ b/configure
> @@ -2117,13 +2117,12 @@ configure_kvm() {
>    fi
>  }
>  
> -if [ use_upstream_kvm = yes ]; then
> -
>  # Make sure the target and host cpus are compatible
>  if test ! \( "$target_cpu" = "$cpu" -o \
>    \( "$target_cpu" = "ppcemb" -a "$cpu" = "ppc" \) -o \
>    \( "$target_cpu" = "x86_64" -a "$cpu" = "i386"   \) -o \
>    \( "$target_cpu" = "i386"   -a "$cpu" = "x86_64" \) \) ; then
> +  \( "$target_cpu" = "ia64"   -a "$cpu" = "ia64" \) \) ; then

Almost...

-  \( "$target_cpu" = "i386"   -a "$cpu" = "x86_64" \) \) ; then
+  \( "$target_cpu" = "i386"   -a "$cpu" = "x86_64" \) \) -o \
+  \( "$target_cpu" = "ia64"   -a "$cpu" = "ia64" \) \) ; then

:-Dustin
Anthony Liguori June 24, 2009, 11:29 p.m. UTC | #2
Dustin Kirkland wrote:
> On Wed, 2009-06-24 at 17:40 -0500, Anthony Liguori wrote:
>   
>> I don't know why this is disabled for qemu-kvm.git.
>>
>> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
>> ---
>>  configure |    5 +----
>>  1 files changed, 1 insertions(+), 4 deletions(-)
>>
>> diff --git a/configure b/configure
>> index 4bad3c3..1b73eaf 100755
>> --- a/configure
>> +++ b/configure
>> @@ -2117,13 +2117,12 @@ configure_kvm() {
>>    fi
>>  }
>>  
>> -if [ use_upstream_kvm = yes ]; then
>> -
>>  # Make sure the target and host cpus are compatible
>>  if test ! \( "$target_cpu" = "$cpu" -o \
>>    \( "$target_cpu" = "ppcemb" -a "$cpu" = "ppc" \) -o \
>>    \( "$target_cpu" = "x86_64" -a "$cpu" = "i386"   \) -o \
>>    \( "$target_cpu" = "i386"   -a "$cpu" = "x86_64" \) \) ; then
>> +  \( "$target_cpu" = "ia64"   -a "$cpu" = "ia64" \) \) ; then
>>     
>
> Almost...
>
> -  \( "$target_cpu" = "i386"   -a "$cpu" = "x86_64" \) \) ; then
> +  \( "$target_cpu" = "i386"   -a "$cpu" = "x86_64" \) \) -o \
> +  \( "$target_cpu" = "ia64"   -a "$cpu" = "ia64" \) \) ; then
>   

Yeah, I have that locally but I forgot to commit --amend.  doh!

Regards,

Anthony Liguori

> :-Dustin
>
Avi Kivity June 28, 2009, 2:03 p.m. UTC | #3
On 06/25/2009 01:40 AM, Anthony Liguori wrote:

Applied, thanks.

> I don't know why this is disabled for qemu-kvm.git.
>    

It was disabled before qemu.git got reworked here, IIRC.
diff mbox

Patch

diff --git a/configure b/configure
index 4bad3c3..1b73eaf 100755
--- a/configure
+++ b/configure
@@ -2117,13 +2117,12 @@  configure_kvm() {
   fi
 }
 
-if [ use_upstream_kvm = yes ]; then
-
 # Make sure the target and host cpus are compatible
 if test ! \( "$target_cpu" = "$cpu" -o \
   \( "$target_cpu" = "ppcemb" -a "$cpu" = "ppc" \) -o \
   \( "$target_cpu" = "x86_64" -a "$cpu" = "i386"   \) -o \
   \( "$target_cpu" = "i386"   -a "$cpu" = "x86_64" \) \) ; then
+  \( "$target_cpu" = "ia64"   -a "$cpu" = "ia64" \) \) ; then
   target_kvm="no"
 fi
 # Disable KVM for linux-user
@@ -2131,8 +2130,6 @@  if test "$target_softmmu" = "no" ; then
   target_kvm="no"
 fi
 
-fi
-
 case "$target_cpu" in
   i386)
     echo "TARGET_ARCH=i386" >> $config_mak