[dpdk-dev] [PATCH] mk: fix acl library static linking

2016-06-30 Thread Thomas Monjalon
2016-06-30 15:24, Ananyev, Konstantin: > Hi Thomas, [...] > > So are you going to apply this patch? > Right now acl just can't be used properly in case of static library build. Got it. I have suggested a small rework in another mail. Then I'll apply it promptly.

[dpdk-dev] [PATCH] mk: fix acl library static linking

2016-06-30 Thread Thomas Monjalon
2016-06-30 15:02, Sergio Gonzalez Monroy: > On 30/06/2016 13:44, Thomas Monjalon wrote: > > 2016-06-30 13:04, Sergio Gonzalez Monroy: > >> On 30/06/2016 12:38, Thomas Monjalon wrote: > >>> Does it need to be commented in rte.app.mk? > >>> The other libs are in whole-archive to support dlopen of dri

[dpdk-dev] [PATCH] mk: fix acl library static linking

2016-06-30 Thread Sergio Gonzalez Monroy
On 30/06/2016 16:28, Thomas Monjalon wrote: > 2016-06-30 15:02, Sergio Gonzalez Monroy: >> On 30/06/2016 13:44, Thomas Monjalon wrote: >>> 2016-06-30 13:04, Sergio Gonzalez Monroy: On 30/06/2016 12:38, Thomas Monjalon wrote: > Does it need to be commented in rte.app.mk? > The other lib

[dpdk-dev] [PATCH] mk: fix acl library static linking

2016-06-30 Thread Ananyev, Konstantin
Hi Thomas, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Sergio Gonzalez Monroy > Sent: Thursday, June 30, 2016 3:02 PM > To: Thomas Monjalon > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] mk: fix acl library static linking >

[dpdk-dev] [PATCH] mk: fix acl library static linking

2016-06-30 Thread Sergio Gonzalez Monroy
On 30/06/2016 13:44, Thomas Monjalon wrote: > 2016-06-30 13:04, Sergio Gonzalez Monroy: >> On 30/06/2016 12:38, Thomas Monjalon wrote: >>> Does it need to be commented in rte.app.mk? >>> The other libs are in whole-archive to support dlopen of drivers. >>> But the problem here is not because of a d

[dpdk-dev] [PATCH] mk: fix acl library static linking

2016-06-30 Thread Thomas Monjalon
2016-06-30 13:04, Sergio Gonzalez Monroy: > On 30/06/2016 12:38, Thomas Monjalon wrote: > > Does it need to be commented in rte.app.mk? > > The other libs are in whole-archive to support dlopen of drivers. > > But the problem here is not because of a driver use. > > There seem to be a bunch of lib

[dpdk-dev] [PATCH] mk: fix acl library static linking

2016-06-30 Thread Thomas Monjalon
2016-06-30 12:10, Sergio Gonzalez Monroy: > Since below commit, ACL library is outside the scope of --whole-archive > and ACL autotest fails. > > RTE>>acl_autotest > ACL: allocation of 25166728 bytes on socket 9 for ACL_acl_ctx failed > ACL: rte_acl_add_rules(acl_ctx): rule #1 is invalid >

[dpdk-dev] [PATCH] mk: fix acl library static linking

2016-06-30 Thread Sergio Gonzalez Monroy
On 30/06/2016 12:38, Thomas Monjalon wrote: > 2016-06-30 12:10, Sergio Gonzalez Monroy: >> Since below commit, ACL library is outside the scope of --whole-archive >> and ACL autotest fails. >> >>RTE>>acl_autotest >>ACL: allocation of 25166728 bytes on socket 9 for ACL_acl_ctx failed >>A

[dpdk-dev] [PATCH] mk: fix acl library static linking

2016-06-30 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Sergio Gonzalez Monroy > Sent: Thursday, June 30, 2016 12:10 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] mk: fix acl library static linking > > Since below commit, ACL library

[dpdk-dev] [PATCH] mk: fix acl library static linking

2016-06-30 Thread Sergio Gonzalez Monroy
Since below commit, ACL library is outside the scope of --whole-archive and ACL autotest fails. RTE>>acl_autotest ACL: allocation of 25166728 bytes on socket 9 for ACL_acl_ctx failed ACL: rte_acl_add_rules(acl_ctx): rule #1 is invalid Line 1584: SSE classify with zero categories failed!