From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?B?U3RlcGhhbiBCw6Ryd29sZg==?= Subject: [PATCH 1/2] KVM: extend "struct x86_emulate_ops" with "get_cpuid" Date: Thu, 12 Jan 2012 16:43:03 +0100 Message-ID: <4F0EFF87.8070402@tu-ilmenau.de> References: <4F0C0EBB.3090506@tu-ilmenau.de> <4F0C1369.9070607@redhat.com> <4F0C2C4E.3000703@tu-ilmenau.de> <4F0C3044.7050307@redhat.com> <4F0C33A0.6080509@tu-ilmenau.de> <4F0C4AA9.6000203@tu-ilmenau.de> <20120111190927.GA13298@amt.cnet> <4F0DEA86.90503@tu-ilmenau.de> <20120111212150.GA18948@amt.cnet> <4F0E0B06.2090708@tu-ilmenau.de> <20120112104756.GB31635@amt.cnet> Reply-To: stephan.baerwolf@tu-ilmenau.de Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="------------040000080803060703090306" Cc: kvm@vger.kernel.org To: Marcelo Tosatti Return-path: Received: from wega.rz.tu-ilmenau.de ([141.24.4.159]:45295 "EHLO wega.rz.tu-ilmenau.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754066Ab2ALPop (ORCPT ); Thu, 12 Jan 2012 10:44:45 -0500 In-Reply-To: <20120112104756.GB31635@amt.cnet> Sender: kvm-owner@vger.kernel.org List-ID: This is a multi-part message in MIME format. --------------040000080803060703090306 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit >>From a8f796f81979094b81cb74535632786ce1ccf9bb Mon Sep 17 00:00:00 2001 From: Stephan Baerwolf Date: Sun, 8 Jan 2012 23:25:59 +0000 Subject: [PATCH 1/2] KVM: extend "struct x86_emulate_ops" with "get_cpuid" In order to be able to proceed checks on CPU-specific properties within the emulator, function "get_cpuid" is introduced. With "get_cpuid" it is possible to virtually call the guests "cpuid"-opcode without changing the VM's context. Cc: Signed-off-by: Stephan Baerwolf --- arch/x86/include/asm/kvm_emulate.h | 4 ++++ arch/x86/kvm/x86.c | 21 +++++++++++++++++++++ 2 files changed, 25 insertions(+), 0 deletions(-) diff --git a/arch/x86/include/asm/kvm_emulate.h b/arch/x86/include/asm/kvm_emulate.h index a026507..b172bf4 100644 --- a/arch/x86/include/asm/kvm_emulate.h +++ b/arch/x86/include/asm/kvm_emulate.h @@ -189,6 +189,10 @@ struct x86_emulate_ops { int (*intercept)(struct x86_emulate_ctxt *ctxt, struct x86_instruction_info *info, enum x86_intercept_stage stage); + + /* retrieve ctxt's vcpu's cpuid */ + bool (*get_cpuid)(struct x86_emulate_ctxt *ctxt, + u32 *eax, u32 *ebx, u32 *ecx, u32 *edx); }; typedef u32 __attribute__((vector_size(16))) sse128_t; diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c index 4c938da..6181783 100644 --- a/arch/x86/kvm/x86.c +++ b/arch/x86/kvm/x86.c @@ -4655,6 +4655,26 @@ static int emulator_intercept(struct x86_emulate_ctxt *ctxt, return kvm_x86_ops->check_intercept(emul_to_vcpu(ctxt), info, stage); } +static bool emulator_get_cpuid(struct x86_emulate_ctxt *ctxt, + u32 *eax, u32 *ebx, u32 *ecx, u32 *edx) +{ + struct kvm_cpuid_entry2 *cpuid = NULL; + + if ((ctxt) && (eax) && (ecx)) { + cpuid = kvm_find_cpuid_entry(emul_to_vcpu(ctxt), (*eax), (*ecx)); + } + + if (cpuid) { + (*eax)=cpuid->eax; + (*ecx)=cpuid->ecx; + if (ebx) (*ebx)=cpuid->ebx; + if (edx) (*edx)=cpuid->edx; + return true; + } + + return false; +} + static struct x86_emulate_ops emulate_ops = { .read_std = kvm_read_guest_virt_system, .write_std = kvm_write_guest_virt_system, @@ -4685,6 +4705,7 @@ static struct x86_emulate_ops emulate_ops = { .get_fpu = emulator_get_fpu, .put_fpu = emulator_put_fpu, .intercept = emulator_intercept, + .get_cpuid = emulator_get_cpuid, }; static void cache_all_regs(struct kvm_vcpu *vcpu) -- 1.7.3.4 --------------040000080803060703090306 Content-Type: text/x-patch; name="0001-KVM-extend-struct-x86_emulate_ops-with-get_cpuid.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename*0="0001-KVM-extend-struct-x86_emulate_ops-with-get_cpuid.patch" >>From a8f796f81979094b81cb74535632786ce1ccf9bb Mon Sep 17 00:00:00 2001 From: Stephan Baerwolf Date: Sun, 8 Jan 2012 23:25:59 +0000 Subject: [PATCH 1/2] KVM: extend "struct x86_emulate_ops" with "get_cpuid" In order to be able to proceed checks on CPU-specific properties within the emulator, function "get_cpuid" is introduced. With "get_cpuid" it is possible to virtually call the guests "cpuid"-opcode without changing the VM's context. Cc: Signed-off-by: Stephan Baerwolf --- arch/x86/include/asm/kvm_emulate.h | 4 ++++ arch/x86/kvm/x86.c | 21 +++++++++++++++++++++ 2 files changed, 25 insertions(+), 0 deletions(-) diff --git a/arch/x86/include/asm/kvm_emulate.h b/arch/x86/include/asm/kvm_emulate.h index a026507..b172bf4 100644 --- a/arch/x86/include/asm/kvm_emulate.h +++ b/arch/x86/include/asm/kvm_emulate.h @@ -189,6 +189,10 @@ struct x86_emulate_ops { int (*intercept)(struct x86_emulate_ctxt *ctxt, struct x86_instruction_info *info, enum x86_intercept_stage stage); + + /* retrieve ctxt's vcpu's cpuid */ + bool (*get_cpuid)(struct x86_emulate_ctxt *ctxt, + u32 *eax, u32 *ebx, u32 *ecx, u32 *edx); }; typedef u32 __attribute__((vector_size(16))) sse128_t; diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c index 4c938da..6181783 100644 --- a/arch/x86/kvm/x86.c +++ b/arch/x86/kvm/x86.c @@ -4655,6 +4655,26 @@ static int emulator_intercept(struct x86_emulate_ctxt *ctxt, return kvm_x86_ops->check_intercept(emul_to_vcpu(ctxt), info, stage); } +static bool emulator_get_cpuid(struct x86_emulate_ctxt *ctxt, + u32 *eax, u32 *ebx, u32 *ecx, u32 *edx) +{ + struct kvm_cpuid_entry2 *cpuid = NULL; + + if ((ctxt) && (eax) && (ecx)) { + cpuid = kvm_find_cpuid_entry(emul_to_vcpu(ctxt), (*eax), (*ecx)); + } + + if (cpuid) { + (*eax)=cpuid->eax; + (*ecx)=cpuid->ecx; + if (ebx) (*ebx)=cpuid->ebx; + if (edx) (*edx)=cpuid->edx; + return true; + } + + return false; +} + static struct x86_emulate_ops emulate_ops = { .read_std = kvm_read_guest_virt_system, .write_std = kvm_write_guest_virt_system, @@ -4685,6 +4705,7 @@ static struct x86_emulate_ops emulate_ops = { .get_fpu = emulator_get_fpu, .put_fpu = emulator_put_fpu, .intercept = emulator_intercept, + .get_cpuid = emulator_get_cpuid, }; static void cache_all_regs(struct kvm_vcpu *vcpu) -- 1.7.3.4 --------------040000080803060703090306--