[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 1/4] target-i386: Make "level" and "xlevel" p
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [PATCH v2 1/4] target-i386: Make "level" and "xlevel" properties static |
Date: |
Thu, 9 Apr 2015 10:14:41 +0200 |
On Wed, 8 Apr 2015 16:02:40 -0300
Eduardo Habkost <address@hidden> wrote:
> Static properties require only 1 line of code, much simpler than the
> existing code that requires writing new getters/setters.
>
> Signed-off-by: Eduardo Habkost <address@hidden>
Reviewed-by: Igor Mammedov <address@hidden>
> ---
> target-i386/cpu.c | 40 ++--------------------------------------
> 1 file changed, 2 insertions(+), 38 deletions(-)
>
> diff --git a/target-i386/cpu.c b/target-i386/cpu.c
> index 03b33cf..2bbf01d 100644
> --- a/target-i386/cpu.c
> +++ b/target-i386/cpu.c
> @@ -1618,38 +1618,6 @@ static void x86_cpuid_version_set_stepping(Object
> *obj, Visitor *v,
> env->cpuid_version |= value & 0xf;
> }
>
> -static void x86_cpuid_get_level(Object *obj, Visitor *v, void *opaque,
> - const char *name, Error **errp)
> -{
> - X86CPU *cpu = X86_CPU(obj);
> -
> - visit_type_uint32(v, &cpu->env.cpuid_level, name, errp);
> -}
> -
> -static void x86_cpuid_set_level(Object *obj, Visitor *v, void *opaque,
> - const char *name, Error **errp)
> -{
> - X86CPU *cpu = X86_CPU(obj);
> -
> - visit_type_uint32(v, &cpu->env.cpuid_level, name, errp);
> -}
> -
> -static void x86_cpuid_get_xlevel(Object *obj, Visitor *v, void *opaque,
> - const char *name, Error **errp)
> -{
> - X86CPU *cpu = X86_CPU(obj);
> -
> - visit_type_uint32(v, &cpu->env.cpuid_xlevel, name, errp);
> -}
> -
> -static void x86_cpuid_set_xlevel(Object *obj, Visitor *v, void *opaque,
> - const char *name, Error **errp)
> -{
> - X86CPU *cpu = X86_CPU(obj);
> -
> - visit_type_uint32(v, &cpu->env.cpuid_xlevel, name, errp);
> -}
> -
> static char *x86_cpuid_get_vendor(Object *obj, Error **errp)
> {
> X86CPU *cpu = X86_CPU(obj);
> @@ -2900,12 +2868,6 @@ static void x86_cpu_initfn(Object *obj)
> object_property_add(obj, "stepping", "int",
> x86_cpuid_version_get_stepping,
> x86_cpuid_version_set_stepping, NULL, NULL, NULL);
> - object_property_add(obj, "level", "int",
> - x86_cpuid_get_level,
> - x86_cpuid_set_level, NULL, NULL, NULL);
> - object_property_add(obj, "xlevel", "int",
> - x86_cpuid_get_xlevel,
> - x86_cpuid_set_xlevel, NULL, NULL, NULL);
> object_property_add_str(obj, "vendor",
> x86_cpuid_get_vendor,
> x86_cpuid_set_vendor, NULL);
> @@ -2998,6 +2960,8 @@ static Property x86_cpu_properties[] = {
> DEFINE_PROP_BOOL("check", X86CPU, check_cpuid, false),
> DEFINE_PROP_BOOL("enforce", X86CPU, enforce_cpuid, false),
> DEFINE_PROP_BOOL("kvm", X86CPU, expose_kvm, true),
> + DEFINE_PROP_UINT32("level", X86CPU, env.cpuid_level, 0),
> + DEFINE_PROP_UINT32("xlevel", X86CPU, env.cpuid_xlevel, 0),
> DEFINE_PROP_END_OF_LIST()
> };
>
- [Qemu-devel] [PATCH v2 0/4] target-i386: Feature properties, sample script for -global/-readconfig, Eduardo Habkost, 2015/04/08
- [Qemu-devel] [PATCH v2 1/4] target-i386: Make "level" and "xlevel" properties static, Eduardo Habkost, 2015/04/08
- Re: [Qemu-devel] [PATCH v2 1/4] target-i386: Make "level" and "xlevel" properties static,
Igor Mammedov <=
- [Qemu-devel] [PATCH v2 2/4] target-i386: X86CPU::xlevel2 QOM property, Eduardo Habkost, 2015/04/08
- [Qemu-devel] [PATCH v2 3/4] target-i386: Register QOM properties for feature flags, Eduardo Habkost, 2015/04/08
- Re: [Qemu-devel] [PATCH v2 3/4] target-i386: Register QOM properties for feature flags, Eduardo Habkost, 2015/04/09
- Re: [Qemu-devel] [PATCH v2 3/4] target-i386: Register QOM properties for feature flags, Igor Mammedov, 2015/04/10
- Re: [Qemu-devel] [PATCH v2 3/4] target-i386: Register QOM properties for feature flags, Paolo Bonzini, 2015/04/10
- Re: [Qemu-devel] [PATCH v2 3/4] target-i386: Register QOM properties for feature flags, Andreas Färber, 2015/04/10
- Re: [Qemu-devel] [PATCH v2 3/4] target-i386: Register QOM properties for feature flags, Paolo Bonzini, 2015/04/10
- Re: [Qemu-devel] [PATCH v2 3/4] target-i386: Register QOM properties for feature flags, Andreas Färber, 2015/04/10
- Re: [Qemu-devel] [PATCH v2 3/4] target-i386: Register QOM properties for feature flags, Eduardo Habkost, 2015/04/10
[Qemu-devel] [PATCH v2 4/4] scripts: x86-cpu-model-dump script, Eduardo Habkost, 2015/04/08