Hi, Comparing the features tested for in get_para_features() with the kvm_feature_names in target-i386/helper.c, I noticed that we didn't list the cr3_cache feature in the real name table. I presume this is unintentional so here's a patch to correct it. Cheers, Jes
commit 39cb576d15a6ffbbcade3c4f282c2f3e76e3098a Author: Jes Sorensen <Jes.Sorensen@xxxxxxxxxx> Date: Wed Feb 17 18:03:37 2010 +0100 Add kvm_cr3_cache to the list of KVM features. This is to match the features automatically added by target-i386/kvm.c:get_para_features() Signed-off-by: Jes Sorensen <Jes.Sorensen@xxxxxxxxxx> diff --git a/target-i386/helper.c b/target-i386/helper.c index f9d63f6..2cd3dca 100644 --- a/target-i386/helper.c +++ b/target-i386/helper.c @@ -61,7 +61,8 @@ static const char *ext3_feature_name[] = { }; static const char *kvm_feature_name[] = { - "kvmclock", "kvm_nopiodelay", "kvm_mmu", NULL, NULL, NULL, NULL, NULL, + "kvmclock", "kvm_nopiodelay", "kvm_mmu", "kvm_cr3_cache", + NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL,