Hello Thomas.
Thanks for your quick response.

On 02/12/2016 03:37 PM, Thomas Monjalon wrote:
> 2016-02-12 12:05, Fernando Seiti Furusato:
>> Add a proper ifeq statement to set the mcpu as needed for ppc64el, as
>> the only one originally set is not valid for ppc architectures.
>
> What is the benefit of using the default machine config, compared to
> the power8 one?
>
> Don't you think the default machine should be renamed core2?

I think it would be better indeed. Thanks for pointing that out.

>
> [...]
>> +ifeq (ppc64le,$(shell uname -m))
>> +        MACHINE_CFLAGS += -mcpu=power8
>
> Why this flag is not set in mk/machine/power8/rte.vars.mk ?
>

This and what observed above would make a better patch.
Let me try those.

Thanks!
-- 

Fernando Seiti Furusato
IBM Linux Technology Center

Reply via email to