[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-stable] [PATCH] target-i386: Check for host features before filter

From: Eduardo Habkost
Subject: [Qemu-stable] [PATCH] target-i386: Check for host features before filter_features_for_kvm()
Date: Tue, 2 Apr 2013 17:48:12 -0300

commit 5ec01c2e96910e1588d1a0de8609b9dda7618c7f broke "-cpu ..,enforce",
as it has moved kvm_check_features_against_host() after the
filter_features_for_kvm() call. filter_features_for_kvm() removes all
features not supported by the host, so this effectively made
kvm_check_features_against_host() impossible to fail.

This patch changes the call so we check for host feature support before
filtering the feature bits.

Signed-off-by: Eduardo Habkost <address@hidden>
Cc: Igor Mammedov <address@hidden>
 target-i386/cpu.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index 9ee5b3d..8fe8bc7 100644
--- a/target-i386/cpu.c
+++ b/target-i386/cpu.c
@@ -2132,14 +2132,14 @@ static void x86_cpu_realizefn(DeviceState *dev, Error 
         env->cpuid_ext3_features &= TCG_EXT3_FEATURES;
         env->cpuid_svm_features &= TCG_SVM_FEATURES;
     } else {
-#ifdef CONFIG_KVM
-        filter_features_for_kvm(cpu);
         if (check_cpuid && kvm_check_features_against_host(cpu)
             && enforce_cpuid) {
             error_setg(errp, "Host's CPU doesn't support requested features");
+#ifdef CONFIG_KVM
+        filter_features_for_kvm(cpu);

reply via email to

[Prev in Thread] Current Thread [Next in Thread]