From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:42174) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TLfDe-0002Eo-St for qemu-devel@nongnu.org; Tue, 09 Oct 2012 15:12:00 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TLfDV-0004XH-9S for qemu-devel@nongnu.org; Tue, 09 Oct 2012 15:11:54 -0400 Received: from mx1.redhat.com ([209.132.183.28]:24873) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TLfDV-0004Wk-17 for qemu-devel@nongnu.org; Tue, 09 Oct 2012 15:11:45 -0400 Date: Tue, 9 Oct 2012 16:01:48 -0300 From: Marcelo Tosatti Message-ID: <20121009190148.GA2927@amt.cnet> References: <1348497138-2516-1-git-send-email-Don@CloudSwitch.com> <1348497138-2516-7-git-send-email-Don@CloudSwitch.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1348497138-2516-7-git-send-email-Don@CloudSwitch.com> Subject: Re: [Qemu-devel] [PATCH v6 06/16] target-i386: Use Hypervisor level in -machine pc, accel=tcg. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Don Slutz Cc: peter.maydell@linaro.org, ehabkost@redhat.com, kvm@vger.kernel.org, qemu-devel@nongnu.org, avi@redhat.com, anthony@codemonkey.ws, imammedo@redhat.com, afaerber@suse.de On Mon, Sep 24, 2012 at 10:32:08AM -0400, Don Slutz wrote: > Also known as Paravirtualization level. > > This change is based on: > > Microsoft Hypervisor CPUID Leaves: > http://msdn.microsoft.com/en-us/library/windows/hardware/ff542428%28v=vs.85%29.aspx > > Linux kernel change starts with: > http://fixunix.com/kernel/538707-use-cpuid-communicate-hypervisor.html > Also: > http://lkml.indiana.edu/hypermail/linux/kernel/1205.0/00100.html > > VMware documention on CPUIDs (Mechanisms to determine if software is > running in a VMware virtual machine): > http://kb.vmware.com/selfservice/microsites/search.do?language=en_US&cmd=displayKC&externalId=1009458 > > QEMU knows this as KVM_CPUID_SIGNATURE (0x40000000) in kvm on linux. > > This does not provide vendor support in tcg yet. > > Signed-off-by: Don Slutz > --- > target-i386/cpu.c | 27 +++++++++++++++++++++++++++ > 1 files changed, 27 insertions(+), 0 deletions(-) > > diff --git a/target-i386/cpu.c b/target-i386/cpu.c > index 48bdaf9..920278b 100644 > --- a/target-i386/cpu.c > +++ b/target-i386/cpu.c > @@ -1651,6 +1651,21 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count, > index = env->cpuid_xlevel; > } > } > + } else if (index & 0x40000000) { > + if (env->cpuid_hv_level_set) { > + uint32_t real_level = env->cpuid_hv_level; > + > + /* Handle Hypervisor CPUIDs */ > + if (real_level < 0x40000000) { > + real_level = 0x40000000; > + } > + if (index > real_level) { > + index = real_level; > + } > + } else { > + if (index > env->cpuid_level) > + index = env->cpuid_level; > + } Whats the purpose of this checks? Please provide changelogs for each patch. > } else { > if (index > env->cpuid_level) > index = env->cpuid_level; > @@ -1789,6 +1804,18 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index, uint32_t count, > *edx = 0; > } > break; > + case 0x40000000: > + *eax = env->cpuid_hv_level; > + *ebx = 0; > + *ecx = 0; > + *edx = 0; > + break; > + case 0x40000001: > + *eax = env->cpuid_kvm_features; > + *ebx = 0; > + *ecx = 0; > + *edx = 0; > + break; > case 0x80000000: > *eax = env->cpuid_xlevel; > *ebx = env->cpuid_vendor1; > -- > 1.7.1 > > -- > To unsubscribe from this list: send the line "unsubscribe kvm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html