Re: [PATCH RFC v2 4/7] x86emul/test: remove unused macros

[Wei Liu <wei.liu2@citrix.com>]

On Fri, Dec 09, 2016 at 06:15:04AM -0700, Jan Beulich wrote:
> >>> On 09.12.16 at 13:23, <wei.liu2@citrix.com> wrote:
> > --- a/tools/tests/x86_emulator/test_x86_emulator.c
> > +++ b/tools/tests/x86_emulator/test_x86_emulator.c
> > @@ -23,15 +23,6 @@ static const struct {
> >  #endif
> >  };
> >  
> > -/* EFLAGS bit definitions. */
> > -#define EFLG_OF (1<<11)
> > -#define EFLG_DF (1<<10)
> > -#define EFLG_SF (1<<7)
> > -#define EFLG_ZF (1<<6)
> > -#define EFLG_AF (1<<4)
> > -#define EFLG_PF (1<<2)
> > -#define EFLG_CF (1<<0)
> 
> Please don't - we rather should start making use of them.
> 
> > @@ -140,18 +124,6 @@ static inline uint64_t xgetbv(uint32_t xcr)
> >      (ecx & (1U << 28)) != 0; \
> >  })
> >  
> > -#define cpu_has_avx2 ({ \
> > -    unsigned int eax = 1, ebx, ecx = 0; \
> > -    emul_test_cpuid(&eax, &ebx, &ecx, &eax, NULL); \
> > -    if ( !(ecx & (1U << 27)) || ((xgetbv(0) & 6) != 6) ) \
> > -        ebx = 0; \
> > -    else { \
> > -        eax = 7, ecx = 0; \
> > -        emul_test_cpuid(&eax, &ebx, &ecx, &eax, NULL); \
> > -    } \
> > -    (ebx & (1U << 5)) != 0; \
> > -})
> 
> Please don't - I already have a patch queued needing this.
> 

No problem. I will see if this patch is still needed, given your comment
on previous patch.

Wei.

> Jan
> 

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