[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/6] target-i386/cpu.c: coding style fixes
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [PATCH 1/6] target-i386/cpu.c: coding style fixes |
Date: |
Tue, 4 Dec 2012 20:40:37 +0100 |
On Tue, 4 Dec 2012 17:34:38 -0200
Eduardo Habkost <address@hidden> wrote:
> - Use spaces instead of tabs on cpu_x86_cpuid().
> - Use braces on 'if' statement cpu_x86_find_by_name().
>
> Signed-off-by: Eduardo Habkost <address@hidden>
> ---
> target-i386/cpu.c | 28 +++++++++++++++-------------
> 1 file changed, 15 insertions(+), 13 deletions(-)
>
> diff --git a/target-i386/cpu.c b/target-i386/cpu.c
> index c6c2ca0..7afe839 100644
> --- a/target-i386/cpu.c
> +++ b/target-i386/cpu.c
> @@ -1227,9 +1227,11 @@ static int cpu_x86_find_by_name(x86_def_t
> *x86_cpu_def, const char *cpu_model)
> uint32_t minus_7_0_ebx_features = 0;
> uint32_t numvalue;
>
> - for (def = x86_defs; def; def = def->next)
> - if (name && !strcmp(name, def->name))
> + for (def = x86_defs; def; def = def->next) {
> + if (name && !strcmp(name, def->name)) {
> break;
> + }
> + }
> if (kvm_enabled() && name && strcmp(name, "host") == 0) {
> kvm_cpu_fill_host(x86_cpu_def);
> } else if (!def) {
> @@ -1835,17 +1837,17 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index,
> uint32_t count,
> }
> break;
> case 0x8000000A:
> - if (env->cpuid_ext3_features & CPUID_EXT3_SVM) {
> - *eax = 0x00000001; /* SVM Revision */
> - *ebx = 0x00000010; /* nr of ASIDs */
> - *ecx = 0;
> - *edx = env->cpuid_svm_features; /* optional features */
> - } else {
> - *eax = 0;
> - *ebx = 0;
> - *ecx = 0;
> - *edx = 0;
> - }
> + if (env->cpuid_ext3_features & CPUID_EXT3_SVM) {
> + *eax = 0x00000001; /* SVM Revision */
> + *ebx = 0x00000010; /* nr of ASIDs */
> + *ecx = 0;
> + *edx = env->cpuid_svm_features; /* optional features */
> + } else {
> + *eax = 0;
> + *ebx = 0;
> + *ecx = 0;
> + *edx = 0;
> + }
> break;
> case 0xC0000000:
> *eax = env->cpuid_xlevel2;
> --
> 1.7.11.7
>
Reviewed-by: Igor Mammedov <address@hidden>
--
Regards,
Igor
- Re: [Qemu-devel] [PATCH 5/6] add visitor for parsing hz[KMG] input string, (continued)
[Qemu-devel] [PATCH 2/6] target-i386: cpu: separate feature string parsing from CPU model lookup, Eduardo Habkost, 2012/12/04
[Qemu-devel] [PATCH 1/6] target-i386/cpu.c: coding style fixes, Eduardo Habkost, 2012/12/04
- Re: [Qemu-devel] [PATCH 1/6] target-i386/cpu.c: coding style fixes,
Igor Mammedov <=
[Qemu-devel] [PATCH 6/6] target-i386: use visit_type_hz to parse tsc_freq property value, Eduardo Habkost, 2012/12/04
Re: [Qemu-devel] [PATCH 0/6] short x86 CPU init cleanup (v3), Andreas Färber, 2012/12/05