[libvirt] [PATCH v5 0/2] Add new HyperV features

[Ján Tomko]

On 06/21/2013 04:03 PM, Paolo Bonzini wrote:
> Il 21/06/2013 15:56, Daniel P. Berrange ha scritto:
>> On Fri, Jun 21, 2013 at 03:51:37PM +0200, Paolo Bonzini wrote:
>>> Il 21/06/2013 12:41, Ján Tomko ha scritto:
>>>>
>>>> Ján Tomko (2):
>>>>   conf: add vapic and spinlocks to hyperv features
>>>>   qemu: add hv_vapic and hv_spinlocks support
>>>>
>>>>  docs/formatdomain.html.in                       | 17 ++++++-
>>>>  docs/schemas/domaincommon.rng                   | 13 +++++
>>>>  src/conf/domain_conf.c                          | 64 ++++++++++++++++++++++++-
>>>>  src/conf/domain_conf.h                          |  3 ++
>>>>  src/qemu/qemu_command.c                         | 48 ++++++++++++++++++-
>>>>  tests/qemuxml2argvdata/qemuxml2argv-hyperv.args |  2 +-
>>>>  tests/qemuxml2argvdata/qemuxml2argv-hyperv.xml  |  2 +
>>>>  7 files changed, 144 insertions(+), 5 deletions(-)
>>>>
>>>
>>> hv_spinlocks is not upstream yet.

All three options were added some time ago:
commit 28f52cc04d341045e810bd487a478fa9ec5f40be [1]
Author:     Vadim Rozenfeld <vrozenfe at redhat.com>
AuthorDate: 2011-12-18 22:48:13 +0200
Commit:     Marcelo Tosatti <mtosatti at redhat.com>
CommitDate: 2012-01-19 08:32:12 -0200

    hyper-v: introduce Hyper-V support infrastructure.

    [Jan: fix build with CONFIG_USER_ONLY]

    Signed-off-by: Marcelo Tosatti <mtosatti at redhat.com>

git describe: v1.0-675-g28f52cc

>>
>> Do you have any feeling for whether it is likely to be accepted,
>> and if so a possible timeframe ?
> 
> No idea, knowing that Hyper-V state in QEMU is somewhat messy I just
> grepped the three options.
> 

It accepts both hv_spinlocks and hv-spinlocks (with only the latter spelling
being greppable) since
commit 72ac2e876ddc6158f5b6d5f758d4e38c436010ed [2]
Author:     Igor Mammedov <imammedo at redhat.com>
AuthorDate: 2013-04-26 18:04:32 +0200
Commit:     Andreas Färber <afaerber at suse.de>
CommitDate: 2013-05-01 13:04:18 +0200

    target-i386: Introduce feat2prop() for CPU properties

    This helper replaces '_' with '-' in a uniform way.
    As a side effect, even custom mappings must use '-' now.

    Signed-off-by: Igor Mammedov <imammedo at redhat.com>
    [AF: Split off; operate on NUL-terminated string rather than '=' delimiter]
    Signed-off-by: Andreas Färber <afaerber at suse.de>

git describe: v1.4.0-1744-g72ac2e8

Jan

[1] http://git.qemu.org/?p=qemu.git;a=commitdiff;h=28f52cc
[2] http://git.qemu.org/?p=qemu.git;a=commitdiff;h=72ac2e8