[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH qom-cpu 00/21 v9] target-i386: convert CPU featu
From: |
Vadim Rozenfeld |
Subject: |
Re: [Qemu-devel] [PATCH qom-cpu 00/21 v9] target-i386: convert CPU features into properties |
Date: |
Tue, 15 Oct 2013 08:27:11 -0400 (EDT) |
----- Original Message -----
From: "Igor Mammedov" <address@hidden>
To: "Igor Mammedov" <address@hidden>
Cc: "Anthony Liguori" <address@hidden>, "Eduardo Habkost" <address@hidden>,
address@hidden, "Vadim Rozenfeld" <address@hidden>, "Paolo Bonzini"
<address@hidden>, "Andreas Färber" <address@hidden>
Sent: Monday, October 14, 2013 3:09:11 PM
Subject: Re: [Qemu-devel] [PATCH qom-cpu 00/21 v9] target-i386: convert CPU
features into properties
On Tue, 16 Jul 2013 00:25:53 +0200
Igor Mammedov <address@hidden> wrote:
> It's reordered and rebased v8 plus CPUID feature bits conversion to properties
> and cleanups that are removing unused anymore *_feature_name arrays.
>
> dynamic => static properties conversion is still making sense as cleanup of
> initfn(), consolidating all properties in one place and making uniform
> property setters signatures, so it was kept.
>
> hyperv and dynamic => static properties conversion are covered by virt-test's
> qemu_cpu test group.
>
> On top of that, CPUID feature bits conversion and cleanups it's allowed.
>
> git for testing: https://github.com/imammedo/qemu/tree/x86-cpu-properties.v9
>
> v8-v9:
> * reorder hyperv/check/force and cpu_x86_parse_featurestr() cleanups to be
> the first
> * when converting dynamic properties inline custom DEFINE_PROP_FOO directly in
> cpu_x86_properties property array. (req: afaerber)
> * change "f-" CPUID feature bits prefix to "feat-" (req: afaerber)
> * replace F_* macros with a single X86CPU_FEAT() macro and simplify it setting
> default value to 0, intended for defaults to be set by initfn()
> * kvmclock: fix endless loop on compat kvmclock2 append
> * converted check_features_against_host() and x86_cpu_list() to use
> static properties definition for feature name lookups, removing along the
> way
> unused anymore *_feature_name arrays.
>
> v7-v8:
> * split out dynamic properties convertion patch into per property patches
> to simplify review
> * drop feature bits convertion
>
> v6-v7:
> * convert globals check_cpuid, enforce_cpuid and hyperv_* to fields of
> CPUState
> * Make PropertyInfo-s static
> * maintain legacy kvmclock semantic in cpu_x86_parse_featurestr()
> * existing properties code are not moved around, just fixed signatures where
> it's needed and used visitors.
>
> v5-v6:
> * when converting feature names to property names, replace '_' with '-'
> * separate patches converting existing dynamic properties into one, were
> squashed into one [1/9] and change tested with virt-test(next).
> * patches that were touching +-foo features are squashed into one [9/9],
> to avoid behavior change between them(f-kvmclock property).
> * the rest of conversions were basicaly rebased on top of current qom-cpu-next
> tree, with small corrections
>
> Igor Mammedov (20):
> target-i386: Move hyperv_* static globals to X86CPU
> target-i386: convert 'hv_spinlocks' to static property
> target-i386: convert 'hv_relaxed' to static property
> target-i386: convert 'hv_vapic' to static property
> target-i386: convert 'check' and 'enforce' to static properties
> target-i386: cleanup 'foo' feature handling'
> target-i386: cleanup 'foo=val' feature handling
> target-i386: cpu: convert 'level' to static property
> target-i386: cpu: convert 'xlevel' to static property
> target-i386: cpu: convert 'family' to static property
> target-i386: cpu: convert 'model' to static property
> target-i386: cpu: convert 'stepping' to static property
> target-i386: cpu: convert 'vendor' to static property
> target-i386: cpu: convert 'model-id' to static property
> target-i386: cpu: convert 'tsc-frequency' to static property
> target-i386: set [+-]feature using static properties
> qdev: introduce qdev_prop_find_bit()
> target-i386: use static properties in check_features_against_host() to
> print CPUID feature names
> target-i386: use static properties to list CPUID features
> target-i386: remove unused *_feature_name arrays
>
> hw/core/qdev-properties.c | 15 +
> include/hw/qdev-properties.h | 13 +
> target-i386/Makefile.objs | 2 +-
> target-i386/cpu-qom.h | 6 +
> target-i386/cpu.c | 705
> +++++++++++++++++++++----------------------
> target-i386/cpu.h | 4 +
> target-i386/hyperv.c | 64 ----
> target-i386/hyperv.h | 45 ---
> target-i386/kvm.c | 36 ++-
> 9 files changed, 415 insertions(+), 475 deletions(-)
> delete mode 100644 target-i386/hyperv.c
> delete mode 100644 target-i386/hyperv.h
>
ping?
Hi Igor,
Looks fine with me.
Best regards,
Vadim.