Skip to contentSkip to conversation

Move CONFIG_X86 decleration to be x86 specific in configure script

PATCH Applied Jerone Young 2 participants 2 messages Nov 29, 2007 – Nov 30, 2007 KVM

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Move CONFIG_X86 decleration to be x86 specific in configure script

# HG changeset patch
# User Jerone Young <jyoung5-r/[email protected]> # Date 1196356414 21600
# Node ID 6fa44248cb3ad7b8a75ea7c23ee935103547fee4
# Parent eb2a8d4d818eb0b27feec303e028bd9944a28694
Move CONFIG_X86 decleration to be x86 specific in configure script

In the configure script CONFIG_X86 is explicitly declared for qemu. Now while today the support for other archs is not exactly in kvm just yet (it's coming)..this shouldn't be specified for everyone ;-)

configure +2 -1
@@ -99,6 +99,7 @@ fi
 #set parameters compiling
 if [ "$arch" = "i386" -o "$arch" = "x86_64" ]; then
     target_exec="x86_64-softmmu"
+    qemu_cflags+=" -DCONFIG_X86"
 fi
 
 #configure user dir
@@ -109,7 +110,7 @@ fi
 #configure qemu
 (cd qemu; ./configure --target-list=$target_exec \
     --disable-kqemu \
-    --extra-cflags="-I $PWD/../libkvm $qemu_cflags -DCONFIG_X86" \
+    --extra-cflags="-I $PWD/../libkvm $qemu_cflags" \
     --extra-ldflags="-L $PWD/../libkvm $qemu_ldflags" \
     --enable-kvm --kernel-path="$libkvm_kerneldir" \
     ${enable_alsa:+"--enable-alsa"} \

-------------------------------------------------------------------------
SF.Net email is sponsored by: The Future of Linux Business White Paper from Novell. From the desktop to the data center, Linux is going mainstream. Let it simplify your IT future.
http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4

Applied
#1 lore raw
Quoted 7 lines · Jerone Young

# HG changeset patch
# User Jerone Young <jyoung5-r/[email protected]> # Date 1196356414 21600
# Node ID 6fa44248cb3ad7b8a75ea7c23ee935103547fee4
# Parent eb2a8d4d818eb0b27feec303e028bd9944a28694
Move CONFIG_X86 decleration to be x86 specific in configure script

Quoted 6 lines

In the configure script CONFIG_X86 is explicitly declared for qemu. Now while today the support for other archs is not exactly in kvm just yet (it's coming)..this shouldn't be specified for everyone ;-)

Applied, thanks.

Signature · 8 lines
Do not meddle in the internals of kernels, for they are subtle and quick to panic.


-------------------------------------------------------------------------
SF.Net email is sponsored by: The Future of Linux Business White Paper
from Novell.  From the desktop to the data center, Linux is going
mainstream.  Let it simplify your IT future.
http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4