[dpdk-dev] librte_meter compilation fails on IBM Power8

[dpdk-dev] librte_meter compilation fails on IBM Power8

[Nélio Laranjeiro]

Hi Cristian, Chao,

I have encountered a compilation failure on IBM Power8 when compiling
master branch with EXTRA_CFLAGS='-O0 -g':

  /root/nl/dpdk.org/build/lib/librte_meter.a(rte_meter.o): In function `rte_meter_get_tb_params':
  /root/nl/dpdk.org/lib/librte_meter/rte_meter.c:57: undefined reference to `ceil'

Seems related to commit 43f4364d.

I don't have the time to search more deeply, I hope it can help.

Regards,

-- 
Nélio Laranjeiro
6WIND

Re: [dpdk-dev] librte_meter compilation fails on IBM Power8

[Chao Zhu]

Nelio,

I'll check. Thanks!

-----Original Message-----
From: Nélio Laranjeiro [mailto:nelio.laranjeiro@6wind.com] 
Sent: 2016年6月22日 20:31
To: Cristian Dumitrescu <cristian.dumitrescu@intel.com>; Chao Zhu
<chaozhu@linux.vnet.ibm.com>
Cc: dev@dpdk.org
Subject: librte_meter compilation fails on IBM Power8

Hi Cristian, Chao,

I have encountered a compilation failure on IBM Power8 when compiling master
branch with EXTRA_CFLAGS='-O0 -g':

  /root/nl/dpdk.org/build/lib/librte_meter.a(rte_meter.o): In function
`rte_meter_get_tb_params':
  /root/nl/dpdk.org/lib/librte_meter/rte_meter.c:57: undefined reference to
`ceil'

Seems related to commit 43f4364d.

I don't have the time to search more deeply, I hope it can help.

Regards,

--
Nélio Laranjeiro
6WIND

Re: [dpdk-dev] librte_meter compilation fails on IBM Power8

[Dumitrescu, Cristian]

> -----Original Message-----
> From: Nélio Laranjeiro [mailto:nelio.laranjeiro@6wind.com]
> Sent: Wednesday, June 22, 2016 1:31 PM
> To: Dumitrescu, Cristian <cristian.dumitrescu@intel.com>; Chao Zhu
> <chaozhu@linux.vnet.ibm.com>
> Cc: dev@dpdk.org
> Subject: librte_meter compilation fails on IBM Power8
> 
> Hi Cristian, Chao,
> 
> I have encountered a compilation failure on IBM Power8 when compiling
> master branch with EXTRA_CFLAGS='-O0 -g':
> 
>   /root/nl/dpdk.org/build/lib/librte_meter.a(rte_meter.o): In function
> `rte_meter_get_tb_params':
>   /root/nl/dpdk.org/lib/librte_meter/rte_meter.c:57: undefined reference to
> `ceil'
> 
> Seems related to commit 43f4364d.
> 
> I don't have the time to search more deeply, I hope it can help.
> 
> Regards,
> 
> --
> Nélio Laranjeiro
> 6WIND

I am not sure what the problem might be for IBM Power8.

ceil() is a function defined in math library, we include math.h header file in rte_meter.c and we also link the library properly in the Makefile by using LDLIBS += -lm, therefore I do not see any issue in the library code.

Thanks,
Cristian

Re: [dpdk-dev] librte_meter compilation fails on IBM Power8

[Chao Zhu]

I can repeat this problem by "export EXTRA_CFLAGS="-O0 -g"" on Power8. But
I'm not sure why this happens. The "-O3 -g" option works properly. I'll
investigate more.

-----Original Message-----
From: Dumitrescu, Cristian [mailto:cristian.dumitrescu@intel.com] 
Sent: 2016年6月24日 1:26
To: Nélio Laranjeiro <nelio.laranjeiro@6wind.com>; Chao Zhu <chaozhu@linux.
vnet.ibm.com>
Cc: dev@dpdk.org
Subject: RE: librte_meter compilation fails on IBM Power8



> -----Original Message-----
> From: Nélio Laranjeiro [mailto:nelio.laranjeiro@6wind.com]
> Sent: Wednesday, June 22, 2016 1:31 PM
> To: Dumitrescu, Cristian <cristian.dumitrescu@intel.com>; Chao Zhu 
> <chaozhu@linux.vnet.ibm.com>
> Cc: dev@dpdk.org
> Subject: librte_meter compilation fails on IBM Power8
> 
> Hi Cristian, Chao,
> 
> I have encountered a compilation failure on IBM Power8 when compiling 
> master branch with EXTRA_CFLAGS='-O0 -g':
> 
>   /root/nl/dpdk.org/build/lib/librte_meter.a(rte_meter.o): In function
> `rte_meter_get_tb_params':
>   /root/nl/dpdk.org/lib/librte_meter/rte_meter.c:57: undefined 
> reference to `ceil'
> 
> Seems related to commit 43f4364d.
> 
> I don't have the time to search more deeply, I hope it can help.
> 
> Regards,
> 
> --
> Nélio Laranjeiro
> 6WIND

I am not sure what the problem might be for IBM Power8.

ceil() is a function defined in math library, we include math.h header file
in rte_meter.c and we also link the library properly in the Makefile by
using LDLIBS += -lm, therefore I do not see any issue in the library code.

Thanks,
Cristian

Re: [dpdk-dev] librte_meter compilation fails on IBM Power8

[gowrishankar]

I remember I came across this problem some time back (not sure on which 
tip of
master), but certainly it is no more appearing now (atleast today tip 
65965546 ..)
Just an update.

Thanks,
Gowrishankar
On Friday 24 June 2016 03:19 PM, Chao Zhu wrote:
> I can repeat this problem by "export EXTRA_CFLAGS="-O0 -g"" on Power8. But
> I'm not sure why this happens. The "-O3 -g" option works properly. I'll
> investigate more.
>
> -----Original Message-----
> From: Dumitrescu, Cristian [mailto:cristian.dumitrescu@intel.com]
> Sent: 2016年6月24日 1:26
> To: Nélio Laranjeiro <nelio.laranjeiro@6wind.com>; Chao Zhu <chaozhu@linux.
> vnet.ibm.com>
> Cc: dev@dpdk.org
> Subject: RE: librte_meter compilation fails on IBM Power8
>
>
>
>> -----Original Message-----
>> From: Nélio Laranjeiro [mailto:nelio.laranjeiro@6wind.com]
>> Sent: Wednesday, June 22, 2016 1:31 PM
>> To: Dumitrescu, Cristian <cristian.dumitrescu@intel.com>; Chao Zhu
>> <chaozhu@linux.vnet.ibm.com>
>> Cc: dev@dpdk.org
>> Subject: librte_meter compilation fails on IBM Power8
>>
>> Hi Cristian, Chao,
>>
>> I have encountered a compilation failure on IBM Power8 when compiling
>> master branch with EXTRA_CFLAGS='-O0 -g':
>>
>>    /root/nl/dpdk.org/build/lib/librte_meter.a(rte_meter.o): In function
>> `rte_meter_get_tb_params':
>>    /root/nl/dpdk.org/lib/librte_meter/rte_meter.c:57: undefined
>> reference to `ceil'
>>
>> Seems related to commit 43f4364d.
>>
>> I don't have the time to search more deeply, I hope it can help.
>>
>> Regards,
>>
>> --
>> Nélio Laranjeiro
>> 6WIND
> I am not sure what the problem might be for IBM Power8.
>
> ceil() is a function defined in math library, we include math.h header file
> in rte_meter.c and we also link the library properly in the Makefile by
> using LDLIBS += -lm, therefore I do not see any issue in the library code.
>
> Thanks,
> Cristian
>
>
>