Bug 442846 – app-emulation/qemu - Do check for KVM option processor specific

• Status: RESOLVED WONTFIX
• Product: Gentoo Linux

[Description Justin Lecher (RETIRED) 2012-11-12 09:14:53 UTC]

I always an ERROR during pkg_pretend because the check is for AMD and INTEL KVM simultaniously.
Please find attach a patch to solve this issue

[Comment 1 Justin Lecher (RETIRED) 2012-11-12 09:15:30 UTC]

Index: qemu-1.2.0.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/app-emulation/qemu/qemu-1.2.0.ebuild,v
retrieving revision 1.8
diff -u -B -r1.8 qemu-1.2.0.ebuild
--- qemu-1.2.0.ebuild	5 Nov 2012 05:32:44 -0000	1.8
+++ qemu-1.2.0.ebuild	12 Nov 2012 09:15:14 -0000
@@ -156,7 +156,17 @@
 			ERROR_VHOST_NET+=" support"
 
 			if use amd64 || use x86 || use amd64-linux || use x86-linux; then
-				CONFIG_CHECK+=" ~KVM_AMD ~KVM_INTEL"
+				if [[ -e /proc/cpuinfo ]]; then
+					if [[ $(grep -q -i intel /proc/cpuinfo ]]; then
+						CONFIG_CHECK+=" ~KVM_INTEL"
+					elif [[ $(grep -q -i amd /proc/cpuinfo ]]; then
+						CONFIG_CHECK+=" ~KVM_AMD"
+					else
+						CONFIG_CHECK+=" ~KVM_AMD ~KVM_INTEL"
+					fi
+				else
+					CONFIG_CHECK+=" ~KVM_AMD ~KVM_INTEL"
+				fi
 			fi
 
 			use python && CONFIG_CHECK+=" ~DEBUG_FS"

[Comment 2 Justin Lecher (RETIRED) 2012-11-12 09:19:55 UTC]

Of course the patch needs to be like this:

Index: qemu-1.2.0.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/app-emulation/qemu/qemu-1.2.0.ebuild,v
retrieving revision 1.8
diff -u -B -r1.8 qemu-1.2.0.ebuild
--- qemu-1.2.0.ebuild	5 Nov 2012 05:32:44 -0000	1.8
+++ qemu-1.2.0.ebuild	12 Nov 2012 09:19:39 -0000
@@ -156,7 +156,17 @@
 			ERROR_VHOST_NET+=" support"
 
 			if use amd64 || use x86 || use amd64-linux || use x86-linux; then
-				CONFIG_CHECK+=" ~KVM_AMD ~KVM_INTEL"
+				if [[ -e /proc/cpuinfo ]]; then
+					if grep -q -i intel /proc/cpuinfo; then
+						CONFIG_CHECK+=" ~KVM_INTEL"
+					elif grep -q -i amd /proc/cpuinfo; then
+						CONFIG_CHECK+=" ~KVM_AMD"
+					else
+						CONFIG_CHECK+=" ~KVM_AMD ~KVM_INTEL"
+					fi
+				else
+					CONFIG_CHECK+=" ~KVM_AMD ~KVM_INTEL"
+				fi
 			fi
 
 			use python && CONFIG_CHECK+=" ~DEBUG_FS"

[Comment 3 Doug Goldstein (RETIRED) 2012-11-12 09:28:39 UTC]

No. The check is the way it is due to people building binary packages.