[dpdk-dev] [PATCHv2] librte_acl make it build/work for 'default' target

2014-08-12 Thread Thomas Monjalon
Hi all, 2014-08-08 10:30, Neil Horman: > On Fri, Aug 08, 2014 at 01:09:34PM +, Ananyev, Konstantin wrote: > > > > Also I think user should have an ability to change default classify > > > > code path without modifying/rebuilding acl library. > > > I agree, but both the methods we are advocat

[dpdk-dev] [PATCHv2] librte_acl make it build/work for 'default' target

2014-08-08 Thread Ananyev, Konstantin
> From: Neil Horman [mailto:nhorman at tuxdriver.com] > Sent: Friday, August 08, 2014 1:25 PM > To: Ananyev, Konstantin > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCHv2] librte_acl make it build/work for 'default' > target > > On Fri, Aug 08,

[dpdk-dev] [PATCHv2] librte_acl make it build/work for 'default' target

2014-08-08 Thread Ananyev, Konstantin
> From: Neil Horman [mailto:nhorman at tuxdriver.com] > Sent: Thursday, August 07, 2014 9:12 PM > To: Ananyev, Konstantin > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCHv2] librte_acl make it build/work for 'default' > target > > On Thu, Aug 07, 20

[dpdk-dev] [PATCHv2] librte_acl make it build/work for 'default' target

2014-08-08 Thread Neil Horman
On Fri, Aug 08, 2014 at 01:09:34PM +, Ananyev, Konstantin wrote: > > > From: Neil Horman [mailto:nhorman at tuxdriver.com] > > Sent: Friday, August 08, 2014 1:25 PM > > To: Ananyev, Konstantin > > Cc: dev at dpdk.org > > Subject: Re: [dpdk-dev] [PATCHv2]

[dpdk-dev] [PATCHv2] librte_acl make it build/work for 'default' target

2014-08-08 Thread Neil Horman
On Fri, Aug 08, 2014 at 11:49:58AM +, Ananyev, Konstantin wrote: > > From: Neil Horman [mailto:nhorman at tuxdriver.com] > > Sent: Thursday, August 07, 2014 9:12 PM > > To: Ananyev, Konstantin > > Cc: dev at dpdk.org > > Subject: Re: [dpdk-dev] [PATCHv2] lib

[dpdk-dev] [PATCHv2] librte_acl make it build/work for 'default' target

2014-08-07 Thread Vincent JARDIN
What's about using function versioning attributes too: https://gcc.gnu.org/wiki/FunctionMultiVersioning ? Le 7 ao?t 2014 22:11, "Neil Horman" a ?crit : > > On Thu, Aug 07, 2014 at 07:31:03PM +0100, Konstantin Ananyev wrote: > > Make ACL library to build/work on 'default' architecture: > > - mak

[dpdk-dev] [PATCHv2] librte_acl make it build/work for 'default' target

2014-08-07 Thread Neil Horman
On Thu, Aug 07, 2014 at 02:28:47PM -0700, Chris Wright wrote: > * Vincent JARDIN (vincent.jardin at 6wind.com) wrote: > > What's about using function versioning attributes too: > > > > https://gcc.gnu.org/wiki/FunctionMultiVersioning > > > > ? > > Neat, hadn't seen that gcc feature before, but:

[dpdk-dev] [PATCHv2] librte_acl make it build/work for 'default' target

2014-08-07 Thread Konstantin Ananyev
Make ACL library to build/work on 'default' architecture: - make rte_acl_classify_scalar really scalar (make sure it wouldn't use sse4 instrincts through resolve_priority()). - Provide two versions of rte_acl_classify code path: rte_acl_classify_sse() - could be build and used only on systems wi

[dpdk-dev] [PATCHv2] librte_acl make it build/work for 'default' target

2014-08-07 Thread Neil Horman
On Thu, Aug 07, 2014 at 07:31:03PM +0100, Konstantin Ananyev wrote: > Make ACL library to build/work on 'default' architecture: > - make rte_acl_classify_scalar really scalar > (make sure it wouldn't use sse4 instrincts through resolve_priority()). > - Provide two versions of rte_acl_classify code

[dpdk-dev] [PATCHv2] librte_acl make it build/work for 'default' target

2014-08-07 Thread Chris Wright
* Vincent JARDIN (vincent.jardin at 6wind.com) wrote: > What's about using function versioning attributes too: > > https://gcc.gnu.org/wiki/FunctionMultiVersioning > > ? Neat, hadn't seen that gcc feature before, but: "This support is available in GCC 4.8 and later. Support is only availab