14/01/2019 16:49, Ferruh Yigit:
> AVX512 was disabled for GCC because of Bugzilla issue 97 [1],
> the GCC defect submitted for the issue [2] highlighted that this is
> a known binutils version 2.30 issue.
> 
> Narrowed the scope of no-avx512 to the this specific binutils version.
> 
> [1]
> https://bugs.dpdk.org/show_bug.cgi?id=97
> 
> [2]
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88096
> 
> Fixes: 8d07c82b239f ("mk: disable gcc AVX512F support")
> Cc: sta...@dpdk.org
> 
> Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com>
> ---
> Cc: Tom Barbette <barbe...@kth.se>
> Cc: Thomas Monjalon <tho...@monjalon.net>
> Cc: Yongseok Koh <ys...@mellanox.com>
> Cc: Konstantin Ananyev <konstantin.anan...@intel.com>
> Cc: Bruce Richardson <bruce.richard...@intel.com>
> 
> v2:
> * Added warning message (print only once for eal)
> * Moved decision details to compat file, kept execution in cpuflags
> 
> v3:
> * replace 'ld' with '$(LD)' for cross build
> * added meson support
> 
> v4:
> * Document this limitation in release notes
> Cc: Vipin Varghese <vipin.vargh...@intel.com>
> 
> v5:
> * Grammar fix
> * remove $(lastword.. ) from makefile
> * add 'OR' between possible workarounds

Acked-by: Thomas Monjalon <tho...@monjalon.net>

Applied, thanks



Reply via email to