[dpdk-dev] [PATCH] crypto: normalize cryptodev pmd names with macros

2016-07-09 Thread Mcnamara, John
; > > Sent: Friday, July 8, 2016 2:42 PM > > > To: Thomas Monjalon > > > Cc: De Lara Guarch, Pablo ; > > > dev at dpdk.org; Richardson, Bruce ; > > > Stephen Hemminger ; Panu Matilainen > > > > > > Subject: Re: [dpdk-dev] [PATCH] crypto: normal

[dpdk-dev] [PATCH] crypto: normalize cryptodev pmd names with macros

2016-07-09 Thread Neil Horman
, Pablo ; dev at > > dpdk.org; > > Richardson, Bruce ; Stephen Hemminger > > ; Panu Matilainen > > Subject: Re: [dpdk-dev] [PATCH] crypto: normalize cryptodev pmd names with > > macros > > > > On Fri, Jul 08, 2016 at 02:40:01PM +0200, Thomas Monjalon wrote: > > &

[dpdk-dev] [PATCH] crypto: normalize cryptodev pmd names with macros

2016-07-08 Thread Mcnamara, John
> Subject: Re: [dpdk-dev] [PATCH] crypto: normalize cryptodev pmd names with > macros > > On Fri, Jul 08, 2016 at 02:40:01PM +0200, Thomas Monjalon wrote: > > 2016-07-08 08:17, Neil Horman: > > > I suppose thats fine, sure, though I'm really not comfortable with >

[dpdk-dev] [PATCH] crypto: normalize cryptodev pmd names with macros

2016-07-08 Thread Thomas Monjalon
2016-07-08 08:17, Neil Horman: > I suppose thats fine, sure, though I'm really not comfortable with an open > source project requiring what appears to be non-open source components > (though I > can't really tell what the snow3g or kasumi license is). Yes you're right. IMHO, it is acceptable beca

[dpdk-dev] [PATCH] crypto: normalize cryptodev pmd names with macros

2016-07-08 Thread De Lara Guarch, Pablo
> -Original Message- > From: Neil Horman [mailto:nhorman at tuxdriver.com] > Sent: Friday, July 08, 2016 1:18 PM > To: De Lara Guarch, Pablo > Cc: dev at dpdk.org; Richardson, Bruce; Thomas Monjalon; Stephen > Hemminger; Panu Matilainen > Subject: Re: [PATCH] crypto: normalize cryptodev p

[dpdk-dev] [PATCH] crypto: normalize cryptodev pmd names with macros

2016-07-08 Thread Thomas Monjalon
2016-07-07 11:37, Neil Horman: > -PMD_REGISTER_DRIVER(cryptodev_aesni_mb_pmd_drvi, aesni_mb); > -DRIVER_REGISTER_PARAM_STRING(aesni_gcm, "max_nb_queue_pairs= " > +PMD_REGISTER_DRIVER(cryptodev_aesni_mb_pmd_drv, CRYPTODEV_NAME_AESNI_MB_PMD); > +DRIVER_REGISTER_PARAM_STRING(CRYPTODEV_NAME_AESNI_MB_PM

[dpdk-dev] [PATCH] crypto: normalize cryptodev pmd names with macros

2016-07-08 Thread Neil Horman
On Fri, Jul 08, 2016 at 02:00:20PM +, De Lara Guarch, Pablo wrote: > > > > -Original Message- > > From: Neil Horman [mailto:nhorman at tuxdriver.com] > > Sent: Friday, July 08, 2016 1:18 PM > > To: De Lara Guarch, Pablo > > Cc: dev at dpdk.org; Richardson, Bruce; Thomas Monjalon; Step

[dpdk-dev] [PATCH] crypto: normalize cryptodev pmd names with macros

2016-07-08 Thread Neil Horman
On Fri, Jul 08, 2016 at 02:40:01PM +0200, Thomas Monjalon wrote: > 2016-07-08 08:17, Neil Horman: > > I suppose thats fine, sure, though I'm really not comfortable with an open > > source project requiring what appears to be non-open source components > > (though I > > can't really tell what the s

[dpdk-dev] [PATCH] crypto: normalize cryptodev pmd names with macros

2016-07-08 Thread De Lara Guarch, Pablo
Hi Neil, > -Original Message- > From: Neil Horman [mailto:nhorman at tuxdriver.com] > Sent: Thursday, July 07, 2016 4:38 PM > To: dev at dpdk.org > Cc: Neil Horman; Richardson, Bruce; Thomas Monjalon; De Lara Guarch, > Pablo; Stephen Hemminger; Panu Matilainen > Subject: [PATCH] crypto: no

[dpdk-dev] [PATCH] crypto: normalize cryptodev pmd names with macros

2016-07-08 Thread Neil Horman
On Fri, Jul 08, 2016 at 09:09:10AM +, De Lara Guarch, Pablo wrote: > Hi Neil, > > > -Original Message- > > From: Neil Horman [mailto:nhorman at tuxdriver.com] > > Sent: Thursday, July 07, 2016 4:38 PM > > To: dev at dpdk.org > > Cc: Neil Horman; Richardson, Bruce; Thomas Monjalon; De L

[dpdk-dev] [PATCH] crypto: normalize cryptodev pmd names with macros

2016-07-07 Thread Neil Horman
Recently reported, the introduction of pmd information exports led to a breakage of cryptodev unit tests because the test infrastructure relies on the cryptodev names being available in macros. This patch fixes the pmd naming to use the macro names. Note that the macro names were already pre-stri