[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH uq/master V2] kvm: Add CPUID support for VIA CPU
From: |
BrillyWu |
Subject: |
Re: [Qemu-devel] [PATCH uq/master V2] kvm: Add CPUID support for VIA CPU |
Date: |
Tue, 31 May 2011 15:39:43 +0800 |
Hi, Jan
> > I am sure I have checked it with the scripts/checkpatch.pl, and it
> > shows no error or warning. Maybe I should check whether my windows
> > editor and mail client can work well before sending it to
> you. I 'm
> > sorry.
>
> Sorry, you are right. Your patch revealed a bug in the checkpatch
> script.
>
> Blue, this does not trigger the missing braces warning:
Do you mean the bug is that it can not trigger missing braces warining?
It seems that there is no missing braces in my patch, but some
unnecessary braces.
>
> @@ -1035,8 +1052,17 @@ void cpu_x86_cpuid(CPUX86State *env, uin {
> /* test if maximum index reached */
> if (index & 0x80000000) {
> - if (index > env->cpuid_xlevel)
> - index = env->cpuid_level;
> + if (index > env->cpuid_xlevel) {
> + if (env->cpuid_xlevel2 > 0) {
> + /* Handle the Centaur's CPUID instruction. */
> + if (index > env->cpuid_xlevel2) {
> + index = env->cpuid_xlevel2;
> + } else if (index < 0xC0000000) {
> + index = env->cpuid_xlevel;
> + }
> + } else
> + index = env->cpuid_xlevel;
> + }
> } else {
> if (index > env->cpuid_level)
> index = env->cpuid_level;
>
>
> > OK, I will use the previous commit log, and send it to you
> in the new thread.
>
> Thanks. I think it would be fair to fix up the braces on commit now.
It looks that there are some unnecessary braces, but if I remove them,
the script/checkpatch.pl will report warnings. Could I ignore it?
BTW, I have submited a patch v3 a few minutes before withou fixing up
the braces, and I have tested it with my mail client this time, so it
could be OK now.
- Re: [Qemu-devel] [PATCH uq/master V2] kvm: Add CPUID support for VIA CPU, (continued)
- Re: [Qemu-devel] [PATCH uq/master V2] kvm: Add CPUID support for VIA CPU, Avi Kivity, 2011/05/11
- Re: [Qemu-devel] [PATCH uq/master V2] kvm: Add CPUID support for VIA CPU, Jan Kiszka, 2011/05/28
- Re: [Qemu-devel] [PATCH uq/master V2] kvm: Add CPUID support for VIA CPU, Jan Kiszka, 2011/05/28
- Re: [Qemu-devel] [PATCH uq/master V2] kvm: Add CPUID support for VIA CPU, BrillyWu, 2011/05/30
- Re: [Qemu-devel] [PATCH uq/master V2] kvm: Add CPUID support for VIA CPU, BrillyWu, 2011/05/30
- Re: [Qemu-devel] [PATCH uq/master V2] kvm: Add CPUID support for VIA CPU, Jan Kiszka, 2011/05/30
- Re: [Qemu-devel] [PATCH uq/master V2] kvm: Add CPUID support for VIA CPU, BrillyWu, 2011/05/30
- Re: [Qemu-devel] [PATCH uq/master V2] kvm: Add CPUID support for VIA CPU, Jan Kiszka, 2011/05/30
- Re: [Qemu-devel] [PATCH uq/master V2] kvm: Add CPUID support for VIA CPU, BrillyWu, 2011/05/30
- Re: [Qemu-devel] [PATCH uq/master V2] kvm: Add CPUID support for VIA CPU, Jan Kiszka, 2011/05/31
- Re: [Qemu-devel] [PATCH uq/master V2] kvm: Add CPUID support for VIA CPU,
BrillyWu <=
- Re: [Qemu-devel] [PATCH uq/master V2] kvm: Add CPUID support for VIA CPU, Jan Kiszka, 2011/05/31
- Re: [Qemu-devel] [PATCH uq/master V2] kvm: Add CPUID support for VIA CPU, BrillyWu, 2011/05/31