[dpdk-stable] [PATCH] mk: fix scope of disabling AVX512F support

[Thomas Monjalon]

19/12/2018 21:20, Ferruh Yigit:
> On 12/19/2018 7:58 PM, Thomas Monjalon wrote:
> > 19/12/2018 20:29, 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.
> > [...]
> >>  # disable AVX512F support of gcc as a workaround for Bug 97
> >>  ifeq ($(CONFIG_RTE_TOOLCHAIN_GCC),y)
> >> +ifneq ($(filter 2.30%,$(LD_VERSION)),)
> >>  MACHINE_CFLAGS += -mno-avx512f
> > 
> > I think we should print a warning here.
> > There is a function $(warning) or $(info) for such case.
> 
> I can add but it prints warning per component, so it prints a lot.

I think we can choose to print only when compiling EAL
by checking $(findstring eal,$(lastword $(MAKEFILE_LIST)))

One more comment: it would be good to have this fix with meson too.