[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 8/8] target-i386: kvm_check_features_against_host():
From: |
Eduardo Habkost |
Subject: |
[Qemu-devel] [PATCH 8/8] target-i386: kvm_check_features_against_host(): Don't use kvm_cpu_fill_host() |
Date: |
Sun, 24 Nov 2013 17:55:40 -0200 |
As now kvm_cpu_fill_host() will simply fill the x86_def_t.features array
with the results of kvm_arch_get_supported_cpuid() calls, we can simply
call kvm_arch_get_supported_cpuid() directly.
Note how kvm_check_features_against_host() and filter_features_for_kvm()
are getting very similar. Eventually they should become a single
function.
Signed-off-by: Eduardo Habkost <address@hidden>
---
target-i386/cpu.c | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index 6c540c6..048d87f 100644
--- a/target-i386/cpu.c
+++ b/target-i386/cpu.c
@@ -1200,20 +1200,20 @@ static int unavailable_host_feature(FeatureWordInfo *f,
uint32_t mask)
*
* This function may be called only if KVM is enabled.
*/
-static int kvm_check_features_against_host(X86CPU *cpu)
+static int kvm_check_features_against_host(KVMState *s, X86CPU *cpu)
{
CPUX86State *env = &cpu->env;
- x86_def_t host_def;
int rv = 0;
FeatureWord w;
assert(kvm_enabled());
- kvm_cpu_fill_host(&host_def);
for (w = 0; w < FEATURE_WORDS; w++) {
FeatureWordInfo *wi = &feature_word_info[w];
uint32_t guest_feat = env->features[w];
- uint32_t host_feat = host_def.features[w];
+ uint32_t host_feat = kvm_arch_get_supported_cpuid(s, wi->cpuid_eax,
+ wi->cpuid_ecx,
+ wi->cpuid_reg);
uint32_t mask;
for (mask = 1; mask; mask <<= 1) {
if (guest_feat & mask && !(host_feat & mask)) {
@@ -2516,7 +2516,8 @@ static void x86_cpu_realizefn(DeviceState *dev, Error
**errp)
env->features[FEAT_8000_0001_ECX] &= TCG_EXT3_FEATURES;
env->features[FEAT_SVM] &= TCG_SVM_FEATURES;
} else {
- if (check_cpuid && kvm_check_features_against_host(cpu)
+ KVMState *s = kvm_state;
+ if (check_cpuid && kvm_check_features_against_host(s, cpu)
&& enforce_cpuid) {
error_setg(&local_err,
"Host's CPU doesn't support requested features");
--
1.8.3.1
- [Qemu-devel] [PATCH 0/8] target-i386: Simplify kvm_cpu_fill_host() and kvm_check_features_against_host(), Eduardo Habkost, 2013/11/24
- [Qemu-devel] [PATCH 1/8] target-i386: kvm_cpu_fill_host(): Kill unused code, Eduardo Habkost, 2013/11/24
- [Qemu-devel] [PATCH 2/8] target-i386: kvm_cpu_fill_host(): No need to check level, Eduardo Habkost, 2013/11/24
- [Qemu-devel] [PATCH 3/8] target-i386: kvm_cpu_fill_host(): No need to check CPU vendor, Eduardo Habkost, 2013/11/24
- [Qemu-devel] [PATCH 4/8] target-i386: kvm_cpu_fill_host(): No need to check xlevel2, Eduardo Habkost, 2013/11/24
- [Qemu-devel] [PATCH 5/8] target-i386: kvm_cpu_fill_host(): Set all feature words at end of function, Eduardo Habkost, 2013/11/24
- [Qemu-devel] [PATCH 6/8] target-i386: kvm_cpu_fill_host(): Fill feature words in a loop, Eduardo Habkost, 2013/11/24
- [Qemu-devel] [PATCH 7/8] target-i386: kvm_check_features_against_host(): Kill feature word array, Eduardo Habkost, 2013/11/24
- [Qemu-devel] [PATCH 8/8] target-i386: kvm_check_features_against_host(): Don't use kvm_cpu_fill_host(),
Eduardo Habkost <=