[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-26 Thread Gonzalez Monroy, Sergio
On 18/03/2015 12:59, Thomas Monjalon wrote: > Hi Sergio, > > Thank you for explaining the situation. > > 2015-03-18 12:11, Gonzalez Monroy, Sergio: >> Given that the patch to remove combined libraries is not welcome, I'll >> try to explain the current situation so we can agree on the way forward. >

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-26 Thread Neil Horman
On Thu, Mar 26, 2015 at 08:52:29AM +, Gonzalez Monroy, Sergio wrote: > On 18/03/2015 12:59, Thomas Monjalon wrote: > >Hi Sergio, > > > >Thank you for explaining the situation. > > > >2015-03-18 12:11, Gonzalez Monroy, Sergio: > >>Given that the patch to remove combined libraries is not welcome,

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-18 Thread Stefan Puiu
Hi, On Wed, Mar 18, 2015 at 2:59 PM, Thomas Monjalon wrote: > Hi Sergio, > > Thank you for explaining the situation. > > 2015-03-18 12:11, Gonzalez Monroy, Sergio: >> Given that the patch to remove combined libraries is not welcome, I'll >> try to explain the current situation so we can agree on

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-18 Thread Gonzalez Monroy, Sergio
On 18/03/2015 15:30, Stefan Puiu wrote: > Hi, > > On Wed, Mar 18, 2015 at 2:59 PM, Thomas Monjalon > wrote: >> Hi Sergio, >> >> Thank you for explaining the situation. >> >> 2015-03-18 12:11, Gonzalez Monroy, Sergio: >>> Given that the patch to remove combined libraries is not welcome, I'll >>> tr

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-18 Thread Thomas Monjalon
Hi Sergio, Thank you for explaining the situation. 2015-03-18 12:11, Gonzalez Monroy, Sergio: > Given that the patch to remove combined libraries is not welcome, I'll > try to explain the current situation so we can agree on the way forward. > > Currently we have build config option for shared

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-18 Thread Neil Horman
On Wed, Mar 18, 2015 at 05:30:12PM +0200, Stefan Puiu wrote: > Hi, > > On Wed, Mar 18, 2015 at 2:59 PM, Thomas Monjalon > wrote: > > Hi Sergio, > > > > Thank you for explaining the situation. > > > > 2015-03-18 12:11, Gonzalez Monroy, Sergio: > >> Given that the patch to remove combined libraries

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-18 Thread Neil Horman
On Wed, Mar 18, 2015 at 12:11:50PM +, Gonzalez Monroy, Sergio wrote: > On 12/03/2015 16:27, Sergio Gonzalez Monroy wrote: > >Remove CONFIG_RTE_BUILD_COMBINE_LIBS and CONFIG_RTE_LIBNAME. > > > >Signed-off-by: Sergio Gonzalez Monroy > >--- > > config/common_bsdapp| 6 -

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-18 Thread Gonzalez Monroy, Sergio
On 12/03/2015 16:27, Sergio Gonzalez Monroy wrote: > Remove CONFIG_RTE_BUILD_COMBINE_LIBS and CONFIG_RTE_LIBNAME. > > Signed-off-by: Sergio Gonzalez Monroy > --- > config/common_bsdapp| 6 -- > config/common_linuxapp | 6 -- > config/defconfig_p

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Gonzalez Monroy, Sergio
On 13/03/2015 16:07, Stephen Hemminger wrote: > On Thu, 12 Mar 2015 16:27:58 + > Sergio Gonzalez Monroy wrote: > >> Remove CONFIG_RTE_BUILD_COMBINE_LIBS and CONFIG_RTE_LIBNAME. >> >> Signed-off-by: Sergio Gonzalez Monroy >> --- > NAK. The combined library is good and useful for those who want

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Stefan Puiu
Hi, 2 cents from a DPDK library user - I make 2 changes to the default linux+gcc configuration: combine libraries and build shared libraries (since I want 2 instances of the app, it didn't make sense to me to link statically). I tried working with the individual libs, but adding all of them with -

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Gonzalez Monroy, Sergio
On 13/03/2015 15:18, Neil Horman wrote: > On Fri, Mar 13, 2015 at 04:12:35PM +0200, Stefan Puiu wrote: >> Hi, >> >> 2 cents from a DPDK library user - I make 2 changes to the default >> linux+gcc configuration: combine libraries and build shared libraries >> (since I want 2 instances of the app, it

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Gonzalez Monroy, Sergio
On 13/03/2015 13:16, Kavanagh, Mark B wrote: > >> -Original Message- >> From: Gonzalez Monroy, Sergio >> Sent: Friday, March 13, 2015 11:49 AM >> To: Kavanagh, Mark B >> Cc: dev at dpdk.org >> Subject: Re: [dpdk-dev] [PATCH v2 1/4] mk: Remove c

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Kavanagh, Mark B
>-Original Message- >From: Gonzalez Monroy, Sergio >Sent: Friday, March 13, 2015 11:49 AM >To: Kavanagh, Mark B >Cc: dev at dpdk.org >Subject: Re: [dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related >options > >On 13/03/2015 11:34, Kavanagh, Mark

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Neil Horman
On Fri, Mar 13, 2015 at 09:07:59AM -0700, Stephen Hemminger wrote: > On Thu, 12 Mar 2015 16:27:58 + > Sergio Gonzalez Monroy wrote: > > > Remove CONFIG_RTE_BUILD_COMBINE_LIBS and CONFIG_RTE_LIBNAME. > > > > Signed-off-by: Sergio Gonzalez Monroy > > --- > > NAK. The combined library is good

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Neil Horman
On Fri, Mar 13, 2015 at 03:28:00PM +, Gonzalez Monroy, Sergio wrote: > On 13/03/2015 15:18, Neil Horman wrote: > >On Fri, Mar 13, 2015 at 04:12:35PM +0200, Stefan Puiu wrote: > >>Hi, > >> > >>2 cents from a DPDK library user - I make 2 changes to the default > >>linux+gcc configuration: combine

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Gonzalez Monroy, Sergio
On 13/03/2015 11:34, Kavanagh, Mark B wrote: >> On 13/03/2015 10:49, Kavanagh, Mark B wrote: --- config/common_bsdapp| 6 -- config/common_linuxapp | 6 -- config/defconfig_ppc_64-power8-linuxapp-gcc | 2 - lib/Makefile

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Kavanagh, Mark B
>On 13/03/2015 10:49, Kavanagh, Mark B wrote: >>> --- >>> config/common_bsdapp| 6 -- >>> config/common_linuxapp | 6 -- >>> config/defconfig_ppc_64-power8-linuxapp-gcc | 2 - >>> lib/Makefile| 1 - >>> mk/rte.app.mk

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Gonzalez Monroy, Sergio
On 13/03/2015 10:49, Kavanagh, Mark B wrote: >> --- >> config/common_bsdapp| 6 -- >> config/common_linuxapp | 6 -- >> config/defconfig_ppc_64-power8-linuxapp-gcc | 2 - >> lib/Makefile| 1 - >> mk/rte.app.mk

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Neil Horman
On Fri, Mar 13, 2015 at 04:12:35PM +0200, Stefan Puiu wrote: > Hi, > > 2 cents from a DPDK library user - I make 2 changes to the default > linux+gcc configuration: combine libraries and build shared libraries > (since I want 2 instances of the app, it didn't make sense to me to > link statically)

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Kavanagh, Mark B
>--- > config/common_bsdapp| 6 -- > config/common_linuxapp | 6 -- > config/defconfig_ppc_64-power8-linuxapp-gcc | 2 - > lib/Makefile| 1 - > mk/rte.app.mk | 12 > mk/rte.lib.mk

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Neil Horman
On Fri, Mar 13, 2015 at 11:48:59AM +, Gonzalez Monroy, Sergio wrote: > On 13/03/2015 11:34, Kavanagh, Mark B wrote: > >>On 13/03/2015 10:49, Kavanagh, Mark B wrote: > --- > config/common_bsdapp| 6 -- > config/common_linuxapp | 6 -- >

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Stephen Hemminger
On Thu, 12 Mar 2015 16:27:58 + Sergio Gonzalez Monroy wrote: > Remove CONFIG_RTE_BUILD_COMBINE_LIBS and CONFIG_RTE_LIBNAME. > > Signed-off-by: Sergio Gonzalez Monroy > --- NAK. The combined library is good and useful for those who want simplicity and build with static library. It is not c

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-12 Thread Sergio Gonzalez Monroy
Remove CONFIG_RTE_BUILD_COMBINE_LIBS and CONFIG_RTE_LIBNAME. Signed-off-by: Sergio Gonzalez Monroy --- config/common_bsdapp| 6 -- config/common_linuxapp | 6 -- config/defconfig_ppc_64-power8-linuxapp-gcc | 2 - lib/Makefile