* [PATCHv2] kvm: fix cpuid eax
@ 2012-05-02 14:55 Michael S. Tsirkin
2012-05-06 12:52 ` Avi Kivity
0 siblings, 1 reply; 2+ messages in thread
From: Michael S. Tsirkin @ 2012-05-02 14:55 UTC (permalink / raw)
To: kvm
Cc: Rob Landley, Avi Kivity, Marcelo Tosatti, Thomas Gleixner,
Ingo Molnar, H. Peter Anvin, x86, gleb, Randy Dunlap, linux-doc,
linux-kernel
cpuid eax should return the max leaf so that
guests can find out the valid range.
This matches Xen et al.
Update documentation to match.
Tested with -cpu host.
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
---
Changes from v1:
Updated documentation.
Documentation/virtual/kvm/cpuid.txt | 6 +++++-
arch/x86/kvm/cpuid.c | 2 +-
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/Documentation/virtual/kvm/cpuid.txt b/Documentation/virtual/kvm/cpuid.txt
index 8820685..83afe65 100644
--- a/Documentation/virtual/kvm/cpuid.txt
+++ b/Documentation/virtual/kvm/cpuid.txt
@@ -10,11 +10,15 @@ a guest.
KVM cpuid functions are:
function: KVM_CPUID_SIGNATURE (0x40000000)
-returns : eax = 0,
+returns : eax = 0x40000001,
ebx = 0x4b4d564b,
ecx = 0x564b4d56,
edx = 0x4d.
Note that this value in ebx, ecx and edx corresponds to the string "KVMKVMKVM".
+The value in eax corresponds to the maximum cpuid function present in this leaf,
+and will be updated if more functions are added in the future.
+Note also that old hosts set eax value to 0x0. This should
+be interpreted as if the value was 0x40000001.
This function queries the presence of KVM cpuid leafs.
diff --git a/arch/x86/kvm/cpuid.c b/arch/x86/kvm/cpuid.c
index 9fed5be..eba8345 100644
--- a/arch/x86/kvm/cpuid.c
+++ b/arch/x86/kvm/cpuid.c
@@ -397,7 +397,7 @@ static int do_cpuid_ent(struct kvm_cpuid_entry2 *entry, u32 function,
case KVM_CPUID_SIGNATURE: {
char signature[12] = "KVMKVMKVM\0\0";
u32 *sigptr = (u32 *)signature;
- entry->eax = 0;
+ entry->eax = KVM_CPUID_FEATURES;
entry->ebx = sigptr[0];
entry->ecx = sigptr[1];
entry->edx = sigptr[2];
--
MST
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCHv2] kvm: fix cpuid eax
2012-05-02 14:55 [PATCHv2] kvm: fix cpuid eax Michael S. Tsirkin
@ 2012-05-06 12:52 ` Avi Kivity
0 siblings, 0 replies; 2+ messages in thread
From: Avi Kivity @ 2012-05-06 12:52 UTC (permalink / raw)
To: Michael S. Tsirkin
Cc: kvm, Rob Landley, Marcelo Tosatti, Thomas Gleixner, Ingo Molnar,
H. Peter Anvin, x86, gleb, Randy Dunlap, linux-doc, linux-kernel
On 05/02/2012 05:55 PM, Michael S. Tsirkin wrote:
> cpuid eax should return the max leaf so that
> guests can find out the valid range.
> This matches Xen et al.
> Update documentation to match.
>
> Tested with -cpu host.
>
>
Thanks, applied.
--
error compiling committee.c: too many arguments to function
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-05-06 12:52 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-05-02 14:55 [PATCHv2] kvm: fix cpuid eax Michael S. Tsirkin
2012-05-06 12:52 ` Avi Kivity
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).