qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v3 00/17] Allow changing of Hypervisor CPUIDs.


From: Eduardo Habkost
Subject: Re: [Qemu-devel] [PATCH v3 00/17] Allow changing of Hypervisor CPUIDs.
Date: Mon, 17 Sep 2012 16:27:06 -0300
User-agent: Mutt/1.5.21 (2010-09-15)

On Mon, Sep 17, 2012 at 10:00:50AM -0400, Don Slutz wrote:
> Resend with new id so the backup files are not included.
> 
> Also known as Paravirtualization CPUIDs.
> 
> This is primarily done so that the guest will think it is running
> under vmware when hypervisor-vendor=vmware is specified as a
> property of a cpu.
> 
> 
> This depends on:
> 
> http://lists.gnu.org/archive/html/qemu-devel/2012-09/msg01400.html
> 
> As far as I know it is #4. It depends on (1) and (2) and (3).

Correct.

I have removed v2 and added this version to my cpu-queue[1] branch.

[1] https://github.com/ehabkost/qemu/commits/cpu-queue
    My branch is now based on Andreas's qom-cpu branch from
    https://github.com/afaerber/qemu-cpu/commits/qom-cpu


> 
> 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
> 
> Changes from v1 to v2:
> 
> 1) Added 1/4 from 
> http://lists.gnu.org/archive/html/qemu-devel/2012-08/msg05153.html
> 
>    Because Fred is changing jobs and so will not be pushing to get
>    this in. It needed to be rebased, And I needed it to complete the
>    testing of this change.
> 
> 2) Added 2/4 because of the re-work I needed a way to clear all KVM bits,
> 
> 3) The rework of v1.  Make it fit into the object model re-work of cpu.c for 
> x86.
> 
> 4) Added 3/4 -- The split out of the code that is not needed for accel=kvm.
> 
> Changes from v2 to v3:
> 
> Marcelo Tosatti:
>   Its one big patch, better split in logically correlated patches
>   (with better changelog). This would help reviewers.
> 
> So split 3 and 4 into 3 to 17.  More info in change log.
> No code change.
> 
> Don Slutz (17):
>   target-i386: Allow tsc-frequency to be larger then 2.147G
>   target-i386: Add missing kvm bits.
>   target-i386: Add Hypervisor level.
>   target-i386: Add cpu object access routines for Hypervisor level.
>   target-i386: Add x86_set_hyperv.
>   target-i386: Use Hypervisor level in -machine pc,accel=kvm.
>   target-i386: Use Hypervisor level in -machine pc,accel=tcg.
>   target-i386: Add Hypervisor vendor.
>   target-i386: Add cpu object access routines for Hypervisor vendor.
>   target-i386: Use Hypervisor vendor in -machine pc,accel=kvm.
>   target-i386: Use Hypervisor vendor in -machine pc,accel=tcg.
>   target-i386: Add some known names to Hypervisor vendor.
>   target-i386: Add optional Hypervisor leaf extra.
>   target-i386: Add cpu object access routines for Hypervisor leaf
>     extra.
>   target-i386: Add setting of Hypervisor leaf extra for known vmare4.
>   target-i386: Use Hypervisor leaf extra in -machine pc,accel=kvm.
>   target-i386: Use Hypervisor leaf extra in -machine pc,accel=tcg.
> 
>  target-i386/cpu.c |  261 
> ++++++++++++++++++++++++++++++++++++++++++++++++++++-
>  target-i386/cpu.h |   21 +++++
>  target-i386/kvm.c |   33 ++++++--
>  3 files changed, 304 insertions(+), 11 deletions(-)
> 
> 

-- 
Eduardo



reply via email to

[Prev in Thread] Current Thread [Next in Thread]