From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44337) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VkzTC-000640-UV for qemu-devel@nongnu.org; Mon, 25 Nov 2013 11:57:15 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VkzT8-0002nl-4h for qemu-devel@nongnu.org; Mon, 25 Nov 2013 11:57:10 -0500 Received: from mx1.redhat.com ([209.132.183.28]:21986) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VkzT7-0002mm-SP for qemu-devel@nongnu.org; Mon, 25 Nov 2013 11:57:06 -0500 Date: Mon, 25 Nov 2013 17:56:40 +0100 From: Igor Mammedov Message-ID: <20131125175640.078b4ee6@thinkpad> In-Reply-To: <1373927181-24247-1-git-send-email-imammedo@redhat.com> References: <1373927181-24247-1-git-send-email-imammedo@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH qom-cpu 00/21 v9] target-i386: convert CPU features into properties List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Andreas =?ISO-8859-1?B?RuRyYmVy?= On Tue, 16 Jul 2013 00:25:53 +0200 Igor Mammedov 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. Hello Andreas, 1.7 development is pretty much closed now, I hope you'd be able to find time to pull this early in 1.8 development cycle. > > 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 > > -- > 1.8.3.1 > > -- Regards, Igor