> -----Original Message-----
> From: Thomas Monjalon [mailto:[email protected]]
> Sent: Wednesday, May 9, 2018 10:47 PM
> To: Xu, Rosen <[email protected]>
> Cc: [email protected]; Zhang, Roy Fan <[email protected]>; Doherty,
> Declan <[email protected]>; Richardson, Bruce
> <[email protected]>; [email protected]; Yigit, Ferruh
> <[email protected]>; Ananyev, Konstantin
> <[email protected]>; Zhang, Tianfei <[email protected]>;
> Liu, Song <[email protected]>; Wu, Hao <[email protected]>;
> [email protected]; Wu, Yanglong <[email protected]>
> Subject: Re: [dpdk-dev] [PATCH v10 3/3] iFPGA: Add Intel FPGA BUS Rawdev
> Driver
> 
> 09/05/2018 09:43, Xu, Rosen:
> > From: Rosen Xu <[email protected]>
> >
> > Add Intel FPGA BUS Rawdev Driver which is based on librte_rawdev
> > library.
> >
> > Signed-off-by: Rosen Xu <[email protected]>
> > Signed-off-by: Yanglong Wu <[email protected]>
> > Signed-off-by: Tianfei Zhang <[email protected]>
> > Acked-by: Shreyansh Jain <[email protected]>
> 
> I have a compilation error:
>       drivers/raw/ifpga_rawdev/base/ifpga_fme_pr.c:10:15: error:
>       instruction requires: AVX-512 ISA
> 
> because of vmovdqu64:
> 
> #if defined(RTE_ARCH_X86_64)
> static inline void copy512(const void *src, void *dst) {
>     asm volatile("vmovdqu64 (%0), %%zmm0;"
>              "vmovntdq %%zmm0, (%1);"
>              :
>              : "r"(src), "r"(dst));
> }
> #else
> static inline void copy512(const void *src, void *dst) {
>     UNUSED(src);
>     UNUSED(dst);
>     WARN_ON(1);
> }
> #endif
> 
> I suggest to fix it quickly without waiting a v11 with this:
> 
> static inline void copy512(const void *src, void *dst) { #ifdef
> CC_SUPPORT_AVX512F
>     asm volatile("vmovdqu64 (%0), %%zmm0;"
>              "vmovntdq %%zmm0, (%1);"
>              :
>              : "r"(src), "r"(dst));
> #else
>     UNUSED(src);
>     UNUSED(dst);
>     WARN_ON(1);
> #endif
> }
> 
> It does not make any runtime detection, but it's better than previously.
> 

Which linux distribution are you use? We can compile it on Ubuntu 16.04 and 
RHEL 7.4.
We will fix it on V11.

Reply via email to