Re: [Xen-devel] [PATCH v2] xc_cpuid_x86.c: Simplify masking conditions and remove redundant work

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Wed, 2014-09-10 at 21:02 +0800, z wrote:

> I am sorry for confusing. The latest one is my all-in-one version: 
> 
> Subject:        [PATCH v2] xc_cpuid_x86.c: Simplify masking conditions
> and remove redundant work
> Date:           Wed, 10 Sep 2014 18:29:00 +0800 (10/09/14 11:29:00)

Thanks, since Jan has already indicated he's happy with it I went to
apply it, however a build test resulted in:
xc_cpuid_x86.c: In function 'intel_xc_cpuid_policy':
xc_cpuid_x86.c:192:32: error: expected ')' before ';' token
xc_cpuid_x86.c:198:5: error: expected ';' before '}' token

I've folded in this change to fix it:
        @@ -189,7 +189,7 @@ static void intel_xc_cpuid_policy(
                 /* Only a few features are advertised in Intel's 0x80000001. */
                 regs[2] &= (bitmaskof(X86_FEATURE_LAHF_LM) |
                             bitmaskof(X86_FEATURE_3DNOWPREFETCH) |
        -                    bitmaskof(X86_FEATURE_ABM);
        +                    bitmaskof(X86_FEATURE_ABM));
                 regs[3] &= (bitmaskof(X86_FEATURE_NX) |
                             bitmaskof(X86_FEATURE_LM) |
                             bitmaskof(X86_FEATURE_SYSCALL) |

Please try and remember to compile test your incremental changes next
time ;-)
        
> Thank you for your suggestions and the rules. I left the community for
> a couple of years to work for some more closed projects. 
> 
> And I am glad to be here again. I will follow the advices you
> mentioned. 

Welcome back!

Ian.





_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel