[dpdk-dev] Proposal for a new Committer model

2016-11-29 Thread Neil Horman
On Fri, Nov 18, 2016 at 01:09:35PM -0500, Neil Horman wrote: > On Thu, Nov 17, 2016 at 09:20:50AM +, Mcnamara, John wrote: > > Repost from the moving at dpdk.org mailing list to get a wider audience. > > Original thread: > > http://dpdk.org/ml/archives/moving/2016

[dpdk-dev] Proposal for a new Committer model

2016-11-25 Thread Neil Horman
On Thu, Nov 24, 2016 at 01:53:55PM +0800, Yuanhan Liu wrote: > On Wed, Nov 23, 2016 at 03:19:19PM -0500, Neil Horman wrote: > > On Wed, Nov 23, 2016 at 11:41:20PM +0800, Yuanhan Liu wrote: > > > On Wed, Nov 23, 2016 at 09:11:54AM -0500, Neil Horman wrote: > > &g

[dpdk-dev] Proposal for a new Committer model

2016-11-25 Thread Neil Horman
On Thu, Nov 24, 2016 at 10:17:09AM +0100, Thomas Monjalon wrote: > 2016-11-23 15:13, Neil Horman: > > Can either you or thomas provide some detail as to how you are doing patch > > management between trees (details of the commands you use are what I would > > be > >

[dpdk-dev] Proposal for a new Committer model

2016-11-23 Thread Neil Horman
On Wed, Nov 23, 2016 at 02:01:44PM +, Ferruh Yigit wrote: > On 11/23/2016 1:48 PM, Neil Horman wrote: > > On Tue, Nov 22, 2016 at 08:56:23PM +, Ferruh Yigit wrote: > >> On 11/22/2016 7:52 PM, Neil Horman wrote: > >>> On Mon, Nov 21, 2016 at 09:52:41A

[dpdk-dev] Proposal for a new Committer model

2016-11-23 Thread Neil Horman
On Wed, Nov 23, 2016 at 08:21:39AM +, Mcnamara, John wrote: > > > > -Original Message- > > From: Neil Horman [mailto:nhorman at tuxdriver.com] > > Sent: Tuesday, November 22, 2016 7:52 PM > > To: Thomas Monjalon > > Cc: dev at dpdk.org; Mcnama

[dpdk-dev] Proposal for a new Committer model

2016-11-23 Thread Neil Horman
On Tue, Nov 22, 2016 at 08:56:23PM +, Ferruh Yigit wrote: > On 11/22/2016 7:52 PM, Neil Horman wrote: > > On Mon, Nov 21, 2016 at 09:52:41AM +0100, Thomas Monjalon wrote: > >> 2016-11-18 13:09, Neil Horman: > >>> A) Further promote subtree maintainership.

[dpdk-dev] Proposal for a new Committer model

2016-11-22 Thread Neil Horman
On Mon, Nov 21, 2016 at 09:52:41AM +0100, Thomas Monjalon wrote: > 2016-11-18 13:09, Neil Horman: > > A) Further promote subtree maintainership. This was a conversation that I > > proposed some time ago, but my proposed granularity was discarded in favor > > of somethi

[dpdk-dev] [PATCH] pmdinfogen: Fix pmdinfogen to select proper endianess on cross-compile

2016-11-18 Thread Neil Horman
systems. Signed-off-by: Neil Horman CC: Hemant Agrawal CC: Jerin Jacob CC: Bruce Richardson CC: Thomas Monjalon --- buildtools/pmdinfogen/pmdinfogen.h | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/buildtools/pmdinfogen/pmdinfogen.h b/buildtools/pmdinfogen

[dpdk-dev] pmdinfogen issues: cross compilation for ARM fails with older host compiler

2016-11-18 Thread Neil Horman
On Fri, Nov 18, 2016 at 04:37:38PM +, Bruce Richardson wrote: > On Fri, Nov 18, 2016 at 08:50:52AM -0500, Neil Horman wrote: > > On Fri, Nov 18, 2016 at 12:03:19PM +, Hemant Agrawal wrote: > > > > -Original Message----- > > > > From: Neil Horman

[dpdk-dev] Proposal for a new Committer model

2016-11-18 Thread Neil Horman
On Thu, Nov 17, 2016 at 09:20:50AM +, Mcnamara, John wrote: > Repost from the moving at dpdk.org mailing list to get a wider audience. > Original thread: http://dpdk.org/ml/archives/moving/2016-November/59.html > > > Hi, > > I'd like to propose a change to the DPDK committer model.

[dpdk-dev] pmdinfogen issues: cross compilation for ARM fails with older host compiler

2016-11-18 Thread Neil Horman
On Fri, Nov 18, 2016 at 12:03:19PM +, Hemant Agrawal wrote: > > -Original Message- > > From: Neil Horman [mailto:nhorman at tuxdriver.com] > > On Tue, Nov 15, 2016 at 09:34:16AM +, Hemant Agrawal wrote: > > > > On Mon, Nov 14, 2016 at 02:29

[dpdk-dev] pmdinfogen issues: cross compilation for ARM fails with older host compiler

2016-11-15 Thread Neil Horman
On Tue, Nov 15, 2016 at 09:34:16AM +, Hemant Agrawal wrote: > > On Mon, Nov 14, 2016 at 02:29:24AM +0530, Jerin Jacob wrote: > > > On Fri, Nov 11, 2016 at 10:34:39AM +, Hemant Agrawal wrote: > > > > Hi Neil, > > > >Pmdinfogen compiles with host compiler. It usages > > > >

[dpdk-dev] pmdinfogen issues: cross compilation for ARM fails with older host compiler

2016-11-15 Thread Neil Horman
On Tue, Nov 15, 2016 at 09:34:16AM +, Hemant Agrawal wrote: > > On Mon, Nov 14, 2016 at 02:29:24AM +0530, Jerin Jacob wrote: > > > On Fri, Nov 11, 2016 at 10:34:39AM +, Hemant Agrawal wrote: > > > > Hi Neil, > > > >Pmdinfogen compiles with host compiler. It usages > > > >

[dpdk-dev] pmdinfogen issues: cross compilation for ARM fails with older host compiler

2016-11-14 Thread Neil Horman
On Mon, Nov 14, 2016 at 02:29:24AM +0530, Jerin Jacob wrote: > On Fri, Nov 11, 2016 at 10:34:39AM +, Hemant Agrawal wrote: > > Hi Neil, > >Pmdinfogen compiles with host compiler. It usages > > rte_byteorder.h of the target platform. > > However, if the host compiler is older

[dpdk-dev] pmdinfogen issues: cross compilation for ARM fails with older host compiler

2016-11-11 Thread Neil Horman
On Fri, Nov 11, 2016 at 02:48:51PM +0100, Jan Viktorin wrote: > Hello all, > > On Fri, 11 Nov 2016 10:34:39 + > Hemant Agrawal wrote: > > > Hi Neil, > >Pmdinfogen compiles with host compiler. It usages > > rte_byteorder.h of the target platform. > > This seems wierd to

[dpdk-dev] Red Hat 6.5 compilation failure

2016-10-31 Thread Neil Horman
On Mon, Oct 31, 2016 at 11:43:18AM +0100, N?lio Laranjeiro wrote: > Hi all, > > Am facing an issue with compilation on redhat 6.5 of DPDK v16.11-rc2, > compilation fails with: > > cc1: warnings being treated as errors > /root/dpdk/drivers/net/i40e/i40e_ethdev_vf.c: In function >

[dpdk-dev] [PATCH v11 00/24] Introducing rte_driver/rte_device generalization

2016-10-18 Thread Neil Horman
On Mon, Oct 17, 2016 at 02:43:12PM +0100, Ferruh Yigit wrote: > On 10/5/2016 12:57 PM, Shreyansh Jain wrote: > > Hi Thomas, > > > > On Tuesday 04 October 2016 01:12 PM, Thomas Monjalon wrote: > >> 2016-10-04 12:21, Shreyansh Jain: > >>> Hi Thomas, > >>> > >>> On Monday 03 October 2016 07:58 PM,

[dpdk-dev] [PATCH 1/3] eal/drivers: prefix driver REGISTER macros with EAL

2016-10-17 Thread Neil Horman
On Sat, Oct 08, 2016 at 12:47:59PM +, Shreyansh Jain wrote: > > > > -Original Message- > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Sent: Friday, October 07, 2016 7:15 PM > > To: Shreyansh Jain > > Cc: david.marchand at 6wind.com; dev at dpdk.org > > Subject:

[dpdk-dev] [PATCH 1/3] eal/drivers: prefix driver REGISTER macros with EAL

2016-10-11 Thread Neil Horman
On Tue, Oct 11, 2016 at 03:57:29PM +0200, Thomas Monjalon wrote: > 2016-10-11 09:38, Neil Horman: > > This also begs the question in my mind, is it really worth changing the > > macro? > > I really don't think it is. The registration macros are pretty descriptive > >

[dpdk-dev] [PATCH 1/3] eal/drivers: prefix driver REGISTER macros with EAL

2016-10-11 Thread Neil Horman
On Tue, Oct 11, 2016 at 12:06:27PM +0530, Shreyansh Jain wrote: > On Monday 10 October 2016 06:26 PM, Neil Horman wrote: > > On Sat, Oct 08, 2016 at 01:00:59PM +, Shreyansh Jain wrote: > > > Hi Thomas, > > > > > > > -Original Mes

[dpdk-dev] [PATCH 1/3] eal/drivers: prefix driver REGISTER macros with EAL

2016-10-10 Thread Neil Horman
On Sat, Oct 08, 2016 at 01:00:59PM +, Shreyansh Jain wrote: > Hi Thomas, > > > -Original Message- > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Sent: Friday, October 07, 2016 7:22 PM > > To: Shreyansh Jain > > Cc: david.marchand at 6wind.com; dev at dpdk.org > >

[dpdk-dev] [PATCH 1/3] eal/drivers: prefix driver REGISTER macros with EAL

2016-10-07 Thread Neil Horman
On Fri, Oct 07, 2016 at 03:51:44PM +0200, Thomas Monjalon wrote: > 2016-10-07 19:11, Shreyansh Jain: > > --- a/mk/internal/rte.compile-pre.mk > > +++ b/mk/internal/rte.compile-pre.mk > > @@ -87,7 +87,7 @@ endif > > PMDINFO_GEN = $(RTE_SDK_BIN)/app/dpdk-pmdinfogen $@ $@.pmd.c > > PMDINFO_CC =

[dpdk-dev] [PATCH v3] ntnic: add PMD driver

2016-09-12 Thread Neil Horman
On Sat, Sep 10, 2016 at 11:31:04AM -0700, Stephen Hemminger wrote: > I think that if the driver is just a shim for proprietary code, then the > vendor should just maintain it out of tree. 6wind and windriver already do > this. Agreed, this was my previous comment. Someone probably wants this

[dpdk-dev] [PATCH v3] ntnic: add PMD driver

2016-09-12 Thread Neil Horman
On Sat, Sep 10, 2016 at 07:58:38AM +, Finn Christensen wrote: > > -Original Message- > > From: Neil Horman [mailto:nhorman at tuxdriver.com] > > Sent: 9. september 2016 15:51 > > To: Finn Christensen > > Cc: dev at dpdk.org; thomas.monj

[dpdk-dev] [PATCH v3] ntnic: add PMD driver

2016-09-09 Thread Neil Horman
On Fri, Sep 09, 2016 at 12:48:38PM +, Finn Christensen wrote: > This is the Napatech NTNIC Poll Mode Driver (PMD) for DPDK. > > This patch adds support for Napatech NICs to DPDK. This is the > initial implementation. > > Signed-off-by: Finn Christensen > --- > v3: > * Removed the need for

[dpdk-dev] [PATCH v2] ntnic: add PMD driver

2016-09-08 Thread Neil Horman
On Thu, Sep 08, 2016 at 11:14:24AM +, Finn Christensen wrote: > This is the Napatech NTNIC Poll Mode Driver (PMD) for DPDK. > > This patch adds support for Napatech NICs to DPDK. This is the > initial implementation. > > Signed-off-by: Finn Christensen > --- > v2: > * Added information

[dpdk-dev] [dpdk-dev, RFC] drivers: advertise kmod dependencies in pmdinfo

2016-09-02 Thread Neil Horman
On Fri, Sep 02, 2016 at 01:52:46PM +, Trahe, Fiona wrote: > > > > -Original Message- > > From: Neil Horman [mailto:nhorman at tuxdriver.com] > > Sent: Friday, September 2, 2016 2:33 PM > > To: Trahe, Fiona > > Cc: Stephen Hemminger ; dev at

[dpdk-dev] [dpdk-dev, RFC] drivers: advertise kmod dependencies in pmdinfo

2016-09-02 Thread Neil Horman
On Fri, Sep 02, 2016 at 09:19:26AM +, Trahe, Fiona wrote: > > > > -Original Message- > > From: Neil Horman [mailto:nhorman at tuxdriver.com] > > Sent: Thursday, September 1, 2016 8:16 PM > > To: Stephen Hemminger > > Cc: Trahe, Fiona ; dev at

[dpdk-dev] [dpdk-dev, RFC] drivers: advertise kmod dependencies in pmdinfo

2016-09-01 Thread Neil Horman
On Thu, Sep 01, 2016 at 10:41:22AM -0700, Stephen Hemminger wrote: > On Thu, 1 Sep 2016 13:35:19 -0400 > Neil Horman wrote: > > > On Thu, Sep 01, 2016 at 12:55:27PM +, Trahe, Fiona wrote: > > > Hi Neil and Olivier, > > > > > > > -Origin

[dpdk-dev] [dpdk-dev, RFC] drivers: advertise kmod dependencies in pmdinfo

2016-09-01 Thread Neil Horman
On Thu, Sep 01, 2016 at 12:55:27PM +, Trahe, Fiona wrote: > Hi Neil and Olivier, > > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier Matz > > Sent: Wednesday, August 31, 2016 2:40 PM > > To: Neil Horman > >

[dpdk-dev] [dpdk-dev, RFC] drivers: advertise kmod dependencies in pmdinfo

2016-08-31 Thread Neil Horman
On Wed, Aug 31, 2016 at 11:21:18AM +0200, Olivier Matz wrote: > Hi Neil, > > On 08/30/2016 03:23 PM, Neil Horman wrote: > > On Fri, Aug 26, 2016 at 03:20:46PM +0200, Olivier Matz wrote: > >> Add a new macro DRIVER_REGISTER_KMOD_DEP() that allows a driver to > >> d

[dpdk-dev] [dpdk-dev, RFC] drivers: advertise kmod dependencies in pmdinfo

2016-08-30 Thread Neil Horman
On Fri, Aug 26, 2016 at 03:20:46PM +0200, Olivier Matz wrote: > Add a new macro DRIVER_REGISTER_KMOD_DEP() that allows a driver to > declare the list of kernel modules required to run properly. > > Today, most PCI drivers require uio/vfio. > > Signed-off-by: Olivier Matz > > --- > In this RFC,

[dpdk-dev] [PATCH v1] dpdk-devbind.py: Virtio interface issue.

2016-08-30 Thread Neil Horman
On Mon, Aug 29, 2016 at 11:16:35PM +, Dey, Souvik wrote: > Hi, > > I already followed the 100% python way and submitted the v3 of this patch. > http://dpdk.org/dev/patchwork/patch/15378/ > How will your patch be different in solving the issue. There will always be > multiple ways to solving

[dpdk-dev] [RFC 0/4] Use Google Test as DPDK unit test framework

2016-08-05 Thread Neil Horman
On Fri, Aug 05, 2016 at 10:11:56AM +0100, Remy Horton wrote: > > On 05/08/2016 08:41, Yerden Zhumabekov wrote: > [..] > > We use cmocka.org for tests. Written in C. It has support for: > > * mocking; > > * setup/teardown; > > * asserts; > > * test groups. > > > > Output is nicely formatted. > >

[dpdk-dev] [RFC 0/4] Use Google Test as DPDK unit test framework

2016-08-03 Thread Neil Horman
On Wed, Aug 03, 2016 at 09:57:39AM +, Doherty, Declan wrote: > > > > -Original Message- > ... > > You are not advocating but the unit test must be written in C++. > > I don't think it is a good idea to force people to write and maintain the > > tests > > in a different language than

[dpdk-dev] [RFC 0/4] Use Google Test as DPDK unit test framework

2016-08-03 Thread Neil Horman
On Tue, Aug 02, 2016 at 11:52:24PM +0200, Thomas Monjalon wrote: > 2016-08-02 21:37, Declan Doherty: > > I've been trying out using google test as a possible replacement for our > > unit > > test framework and have put to together this series of patches with help > > from > > Anatoly as RFC to

[dpdk-dev] [RFC 3/4] eal: add command line option to log output to stdout

2016-08-03 Thread Neil Horman
On Tue, Aug 02, 2016 at 09:37:48PM +0100, Declan Doherty wrote: > Adds new command line options which allows the user to stop > application echoing log output to stdout, logs are still > written to syslog. > > Signed-off-by: Declan Doherty > --- > lib/librte_eal/common/eal_common_log.c | 14

[dpdk-dev] [PATCH] validate_abi: build faster by augmenting make with job count

2016-08-01 Thread Neil Horman
On Mon, Aug 01, 2016 at 04:16:12PM +, Wiles, Keith wrote: > > >> Neil > >> > > Sorry for the delayed response, I've been on vacation. > > > >> Your modified copy of make has no bearing on the topic we are taking about > >> customers using dpdk in standard distros right? > >> > > !? I

[dpdk-dev] [PATCH] validate_abi: build faster by augmenting make with job count

2016-08-01 Thread Neil Horman
On Sun, Jul 24, 2016 at 06:08:00PM +, Wiles, Keith wrote: > > > Sent from my iPhone > > > On Jul 21, 2016, at 1:34 PM, Neil Horman wrote: > > > >> On Thu, Jul 21, 2016 at 03:22:45PM +, Wiles, Keith wrote: > >> > >>

[dpdk-dev] [PATCH] validate_abi: build faster by augmenting make with job count

2016-07-21 Thread Neil Horman
On Thu, Jul 21, 2016 at 03:22:45PM +, Wiles, Keith wrote: > > > On Jul 21, 2016, at 10:06 AM, Neil Horman wrote: > > > > On Thu, Jul 21, 2016 at 02:09:19PM +, Wiles, Keith wrote: > >> > >>> On Jul 21, 2016, at 8:54 AM, Neil Horman wrote: >

[dpdk-dev] [PATCH] validate_abi: build faster by augmenting make with job count

2016-07-21 Thread Neil Horman
On Thu, Jul 21, 2016 at 02:09:19PM +, Wiles, Keith wrote: > > > On Jul 21, 2016, at 8:54 AM, Neil Horman wrote: > > > > On Wed, Jul 20, 2016 at 10:32:28PM +, Wiles, Keith wrote: > >> > >>> On Jul 20, 2016, at 3:16 PM, Neil Horman wrote: >

[dpdk-dev] [PATCH] validate_abi: build faster by augmenting make with job count

2016-07-21 Thread Neil Horman
On Wed, Jul 20, 2016 at 10:32:28PM +, Wiles, Keith wrote: > > > On Jul 20, 2016, at 3:16 PM, Neil Horman wrote: > > > > On Wed, Jul 20, 2016 at 07:47:32PM +, Wiles, Keith wrote: > >> > >>> On Jul 20, 2016, at 12:48 PM, Neil Horman wrote: >

[dpdk-dev] [PATCH] validate_abi: build faster by augmenting make with job count

2016-07-20 Thread Neil Horman
On Wed, Jul 20, 2016 at 07:47:32PM +, Wiles, Keith wrote: > > > On Jul 20, 2016, at 12:48 PM, Neil Horman wrote: > > > > On Wed, Jul 20, 2016 at 07:40:49PM +0200, Thomas Monjalon wrote: > >> 2016-07-20 13:09, Neil Horman: > >>> From: Neil Hor

[dpdk-dev] [PATCH v2] validate_abi: build faster by augmenting make with job count

2016-07-20 Thread Neil Horman
is set to the number of online cpus. Signed-off-by: Neil Horman CC: Thomas Monjalon CC: "Mcnamara, John" --- Change notes v2) switch variable to DPDK_MAKE_JOBS make provision for no existance of lscpu --- scripts/validate-abi.sh | 15 +-- 1 file changed, 13 insert

[dpdk-dev] [PATCH] validate_abi: build faster by augmenting make with job count

2016-07-20 Thread Neil Horman
On Wed, Jul 20, 2016 at 07:40:49PM +0200, Thomas Monjalon wrote: > 2016-07-20 13:09, Neil Horman: > > From: Neil Horman > > > > John Mcnamara and I were discussing enhacing the validate_abi script to > > build > > the dpdk tree faster with multiple jobs.

[dpdk-dev] [PATCH] validate_abi: build faster by augmenting make with job count

2016-07-20 Thread Neil Horman
From: Neil Horman <nhor...@redhat.com> John Mcnamara and I were discussing enhacing the validate_abi script to build the dpdk tree faster with multiple jobs. Theres no reason not to do it, so this implements that requirement. It uses a MAKE_JOBS variable that can be set by the user to

[dpdk-dev] [PATCH 0/5] Driver register fixes

2016-07-09 Thread Neil Horman
| 7 ++- > drivers/net/xenvirt/rte_eth_xenvirt.c | 5 +++-- > 21 files changed, 67 insertions(+), 47 deletions(-) > > -- > 2.7.4 > > Series Acked-by: Neil Horman

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

2016-07-09 Thread Neil Horman
On Fri, Jul 08, 2016 at 07:03:10PM +, Mcnamara, John wrote: > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Neil Horman > > Sent: Friday, July 8, 2016 2:42 PM > > To: Thomas Monjalon > > Cc: De Lara Guarch

[dpdk-dev] [PATCH v3 08/10] eal: remove PCI include from generic driver header

2016-07-08 Thread Neil Horman
On Fri, Jul 08, 2016 at 04:42:22PM +0200, Thomas Monjalon wrote: > Remove include of rte_pci.h in the generic header rte_dev.h. > > Fixes: cb6696d22023 ("drivers: update registration macro usage") > > Suggested-by: David Marchand > Signed-off-by: Thomas Monj

[dpdk-dev] [PATCH v3 06/10] pmdinfogen: fix build warnings

2016-07-08 Thread Neil Horman
gs? set but not used > ?sec_name? defined but not used > ?get_symbol_index? defined but not used > pointer of type ?void *? used in arithmetic > > Fixes: 98b0fdb0ffc6 ("pmdinfogen: add buildtools and pmdinfogen utility") > > Signed-off-by: Thomas Monjalon Acked-by: Neil Horman

[dpdk-dev] [PATCH v3 04/10] mk: fix driver build with installed SDK

2016-07-08 Thread Neil Horman
") > > Signed-off-by: Thomas Monjalon Acked-by: Neil Horman

[dpdk-dev] [PATCH 07/11] pmdinfogen: fix build warnings

2016-07-08 Thread Neil Horman
On Thu, Jul 07, 2016 at 09:25:27PM +, Mcnamara, John wrote: > > > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Neil Horman > > Sent: Thursday, July 7, 2016 6:55 PM > > To: Thomas Monjalon > > Cc: dev at dpdk.or

[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

[dpdk-dev] [PATCH 09/11] eal: move PCI table macro

2016-07-08 Thread Neil Horman
On Fri, Jul 08, 2016 at 04:03:40PM +0200, Thomas Monjalon wrote: > 2016-07-08 09:56, Neil Horman: > > On Fri, Jul 08, 2016 at 10:49:25AM +0200, David Marchand wrote: > > > Hello Thomas, Neil, > > > > > > (will be back in a couple of days, th

[dpdk-dev] [PATCH 09/11] eal: move PCI table macro

2016-07-08 Thread Neil Horman
On Fri, Jul 08, 2016 at 10:49:25AM +0200, David Marchand wrote: > Hello Thomas, Neil, > > (will be back in a couple of days, thanks Thomas for pointing this thread) > > > On Thu, Jul 7, 2016 at 6:25 PM, Thomas Monjalon > wrote: > > 2016-07-07 12:11, Neil Horman: >

[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 >

[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; Ri

[dpdk-dev] [PATCH 08/11] pmdinfogen: fix usage message

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 06:24:16PM +0200, Thomas Monjalon wrote: > 2016-07-07 12:05, Neil Horman: > > On Thu, Jul 07, 2016 at 05:36:27PM +0200, Thomas Monjalon wrote: > > > - "usage: pmdinfo \n"); > > > + "usage: pmdin

[dpdk-dev] [PATCH 03/11] mk: fix build dependency of drivers on pmdinfogen

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 06:21:49PM +0200, Thomas Monjalon wrote: > 2016-07-07 11:56, Neil Horman: > > On Thu, Jul 07, 2016 at 05:36:22PM +0200, Thomas Monjalon wrote: > > > When compiling the drivers, some code is generated with pmdinfogen. > > > A fresh parallel

[dpdk-dev] [PATCH 11/11] maintainers: add section for pmdinfo

2016-07-07 Thread Neil Horman
complete. Sound good? Neil > > 2016-07-07 17:36, Thomas Monjalon: > > The author of this feature is Neil Horman. > > > > Signed-off-by: Thomas Monjalon > > --- > > +Driver information > > +F: buildtools/pmdinfogen/ > > +F: tools/pmdinfo.py > > >

[dpdk-dev] [PATCH 11/11] maintainers: add section for pmdinfo

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 05:36:30PM +0200, Thomas Monjalon wrote: > The author of this feature is Neil Horman. > > Signed-off-by: Thomas Monjalon > --- > MAINTAINERS | 4 > 1 file changed, 4 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > in

[dpdk-dev] [PATCH 10/11] doc: fix syntax in pmdinfogen guide

2016-07-07 Thread Neil Horman
format: > > -.. code-block:: C > +.. code-block:: c > > static char ="PMD_INFO_STRING=\"{'name' : '', > ...}\""; > > -- > 2.7.0 > > Acked-by: Neil Horman

[dpdk-dev] [PATCH 09/11] eal: move PCI table macro

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 05:36:28PM +0200, Thomas Monjalon wrote: > Remove include of rte_pci.h in the generic header rte_dev.h > and move the macro DRIVER_REGISTER_PCI_TABLE in rte_pci.h. > > Fixes: cb6696d22023 ("drivers: update registration macro usage") > > Signed-off-by: Thomas Monjalon >

[dpdk-dev] [PATCH 08/11] pmdinfogen: fix usage message

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 05:36:27PM +0200, Thomas Monjalon wrote: > The name of the tool is pmdinfogen. > > Fixes: 98b0fdb0ffc6 ("pmdinfogen: add buildtools and pmdinfogen utility") > > Signed-off-by: Thomas Monjalon > --- > buildtools/pmdinfogen/pmdinfogen.c | 2 +- > 1 file changed, 1

[dpdk-dev] [PATCH 06/11] mk: fix verbose pmdinfogen run

2016-07-07 Thread Neil Horman
$OBJF; \ > - mv -f \$$OBJF.o \$$OBJF; \ > - fi; \ > - fi;" && \ > + $(PMDINFO_TO_O) && \ > echo $(C_TO_O_CMD) > $(call obj2cmd,$(@)) && \ > sed 's,'$@':,dep_'$@' =,' $(call obj2dep,$(@)).tmp > $(call > obj2dep,$(@)) && \ > rm -f $(call obj2dep,$(@)).tmp > > - > # return an empty string if string are equal > compare = $(strip $(subst $(1),,$(2)) $(subst $(2),,$(1))) > > -- > 2.7.0 > > Acked-by: Neil Horman

[dpdk-dev] [PATCH 04/11] mk: remove traces of hostapp build directory

2016-07-07 Thread Neil Horman
t try to rebuild .config > $(RTE_OUTPUT)/include/rte_config.h: $(RTE_OUTPUT)/.config > $(Q)rm -rf $(RTE_OUTPUT)/include $(RTE_OUTPUT)/app \ > - $(RTE_OUTPUT)/hostapp $(RTE_OUTPUT)/lib \ > + $(RTE_OUTPUT)/lib \ > $(RTE_OUTPUT)/hostlib $(RTE_OUTPUT)/kmod $(RTE_OUTPUT)/build > $(Q)mkdir -p $(RTE_OUTPUT)/include > $(Q)$(RTE_SDK)/scripts/gen-config-h.sh $(RTE_OUTPUT)/.config \ > -- > 2.7.0 > > Acked-by: Neil Horman

[dpdk-dev] [PATCH 03/11] mk: fix build dependency of drivers on pmdinfogen

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 05:36:22PM +0200, Thomas Monjalon wrote: > When compiling the drivers, some code is generated with pmdinfogen. > A fresh parallel build can fail if a driver is compiled before pmdinfogen: > build/buildtools/dpdk-pmdinfogen: Permission denied > > There was a

[dpdk-dev] [PATCH 02/11] crypto: fix parameters registration

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 05:36:21PM +0200, Thomas Monjalon wrote: > The name aesni_gcm was used to register parameters of several > crypto drivers. > > Fixes: cb6696d22023 ("drivers: update registration macro usage") > > Signed-off-by: Thomas Monjalon > --- >

[dpdk-dev] [PATCH 01/11] drivers: fix build with new register macro

2016-07-07 Thread Neil Horman
ruct rte_driver rte_mlx5_driver = { > .init = rte_mlx5_pmd_init, > }; > > -PMD_REGISTER_DRIVER(rte_mlx5_driveri, mlx5) > +PMD_REGISTER_DRIVER(rte_mlx5_driver, mlx5); > DRIVER_REGISTER_PCI_TABLE(mlx5, mlx5_pci_id_map); > -- > 2.7.0 > > Acked-by: Neil Horman

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

2016-07-07 Thread Neil Horman
don't have hardware for any other device. Also not build tested on snow3g or kasumi pmd because building those requires external libraries that appear to not be open source licensed. Signed-off-by: Neil Horman CC: Bruce Richardson CC: Thomas Monjalon CC: "De Lara Guarch, Pablo" C

[dpdk-dev] [PATCHv8 2/6] drivers: Update driver registration macro usage

2016-07-07 Thread Neil Horman
On Thu, Jul 07, 2016 at 11:00:59AM +, De Lara Guarch, Pablo wrote: > > > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Neil Horman > > Sent: Friday, June 17, 2016 7:46 PM > > To: dev at dpdk.org > > Cc: Ne

[dpdk-dev] [PATCH v9 0/7] export PMD infos

2016-07-05 Thread Neil Horman
On Mon, Jul 04, 2016 at 10:10:37PM +0200, Thomas Monjalon wrote: > 2016-07-04 12:41, Neil Horman: > > On Mon, Jul 04, 2016 at 03:10:14PM +0200, Thomas Monjalon wrote: > > > Hi Neil, > > > > > > I don't really understand why you don't accept I contribute to thi

[dpdk-dev] [PATCH v9 0/7] export PMD infos

2016-07-04 Thread Neil Horman
trary to what I just spent I don't even know how many weeks arguing with you and others about. Given your role as maintainer, it is tantamout to a 'my way or the highway' position. > 2016-07-04 08:34, Neil Horman: > > On Mon, Jul 04, 2016 at 03:13:58AM +0200, Thomas Monjalon wrote

[dpdk-dev] [PATCH v9 0/7] export PMD infos

2016-07-04 Thread Neil Horman
gt; > Because of (4) the other items should be pretty easy to implement, as its > just a > matter of modifying the macros to export the info, pmdinfo to encode it to > json, > and pmd_hw_support.py to read it. I'm not adding them now, as theres alot of > rote work to do to get some of them in place

[dpdk-dev] [PATCHv8 5/6] pmdinfo.py: Add tool to query binaries for hw and other support information

2016-06-29 Thread Neil Horman
chive, you have to unarchive it first (ar x). It would be the same if you had bound them together with tar or zip. > DPDK doesn't to my knowledge have any coding guidelines for Python, so the > comments below should be considered advisory rather than merge-blocking > issues. > > > On 17

[dpdk-dev] [PATCHv8 0/6] Implement pmd hardware support exports

2016-06-29 Thread Neil Horman
On Wed, Jun 29, 2016 at 10:18:26AM +0100, Remy Horton wrote: > Morning, > > Compile error with virtio_user_ethdev.c - looks like a new addition (commit > ce2eabdd43ec) that also needs to be converted. > > Regards, > > ..Remy > diff --git a/drivers/net/virtio/virtio_user_ethdev.c

[dpdk-dev] [PATCH v5 10/17] ethdev: get rid of eth driver register callback

2016-06-22 Thread Neil Horman
On Wed, Jun 22, 2016 at 02:36:29PM +0530, Shreyansh Jain wrote: > Now that all pdev are pci drivers, we don't need to register ethdev drivers > through a dedicated channel. > > Signed-off-by: David Marchand > Signed-off-by: Shreyansh Jain > --- > lib/librte_ether/rte_ethdev.c | 22

[dpdk-dev] [PATCH v5 09/17] crypto: get rid of crypto driver register callback

2016-06-22 Thread Neil Horman
On Wed, Jun 22, 2016 at 02:36:28PM +0530, Shreyansh Jain wrote: > Now that all pdev are pci drivers, we don't need to register crypto drivers > through a dedicated channel. > > Signed-off-by: David Marchand > Signed-off-by: Shreyansh Jain > --- > lib/librte_cryptodev/rte_cryptodev.c

[dpdk-dev] [PATCHv8 6/6] doc: Add prog_guide section documenting pmdinfo script

2016-06-17 Thread Neil Horman
Information on pmdinfogen may be useful to 3rd party driver developers. Include documentation on what it does Signed-off-by: Neil Horman Acked-by: Panu Matilainen Acked-by: John McNamara CC: Bruce Richardson CC: Thomas Monjalon CC: Stephen Hemminger CC: Panu Matilainen --- doc/guides

[dpdk-dev] [PATCHv8 5/6] pmdinfo.py: Add tool to query binaries for hw and other support information

2016-06-17 Thread Neil Horman
) pretty, where a user supplied pci.ids file is used to print out vendor and device strings Signed-off-by: Neil Horman Acked-by: Panu Matilainen CC: Bruce Richardson CC: Thomas Monjalon CC: Stephen Hemminger CC: Panu Matilainen --- mk/rte.sdkinstall.mk | 2 + tools/pmdinfo.py | 629

[dpdk-dev] [PATCHv8 4/6] Makefile: Do post processing on objects that register a driver

2016-06-17 Thread Neil Horman
into the parent object that it came from. This effectively just adds the json string into the string table of the object that defines the PMD to the outside world. Signed-off-by: Neil Horman Acked-by: Panu Matilainen CC: Bruce Richardson CC: Thomas Monjalon CC: Stephen Hemminger CC: Panu

[dpdk-dev] [PATCHv8 3/6] eal: Add an export symbol to expose the autoload path to external tools

2016-06-17 Thread Neil Horman
stallation directory (or static binary), and scan the associated path (if found) for pmd DSO's and report on their hardware support. Signed-off-by: Neil Horman Acked-by: Panu Matilainen CC: Bruce Richardson CC: Thomas Monjalon CC: Stephen Hemminger CC: Panu Matilainen --- lib/librte_

[dpdk-dev] [PATCHv8 2/6] drivers: Update driver registration macro usage

2016-06-17 Thread Neil Horman
create a json string to make hardware support info discoverable later. Signed-off-by: Neil Horman Acked-by: Panu Matilainen CC: Bruce Richardson CC: Thomas Monjalon CC: Stephen Hemminger CC: Panu Matilainen --- drivers/Makefile | 2 ++ drivers/crypto/aesni_gcm/ae

[dpdk-dev] [PATCHv8 1/6] pmdinfogen: Add buildtools and pmdinfogen utility

2016-06-17 Thread Neil Horman
hich the C was generated. This creates an entry in the string table of the object that can inform a later tool about hardware support. Signed-off-by: Neil Horman CC: Bruce Richardson CC: Thomas Monjalon CC: Stephen Hemminger CC: Panu Matilainen Acked-by: Panu Matilainen --- G

[dpdk-dev] [PATCHv8 0/6] Implement pmd hardware support exports

2016-06-17 Thread Neil Horman
ixed some typos in doc v8) * Modified symlink for pmdinfo to use dpdk_ * Added dpdk_pmdinfogen for pmdinfogen binary Signed-off-by: Neil Horman Acked-by: Panu Matilainen CC: Bruce Richardson CC: Thomas Monjalon CC: Stephen Hemminger CC: Panu Matilainen

[dpdk-dev] [PATCHv7 1/6] pmdinfogen: Add buildtools and pmdinfogen utility

2016-06-16 Thread Neil Horman
On Thu, Jun 16, 2016 at 05:06:49PM +0300, Panu Matilainen wrote: > On 06/16/2016 04:33 PM, Neil Horman wrote: > > On Thu, Jun 16, 2016 at 03:29:57PM +0300, Panu Matilainen wrote: > > > On 06/09/2016 08:46 PM, Neil Horman wrote: > > > > pmdinfogen is a tool used to par

[dpdk-dev] [PATCHv7 1/6] pmdinfogen: Add buildtools and pmdinfogen utility

2016-06-16 Thread Neil Horman
On Thu, Jun 16, 2016 at 03:29:57PM +0300, Panu Matilainen wrote: > On 06/09/2016 08:46 PM, Neil Horman wrote: > > pmdinfogen is a tool used to parse object files and build json strings for > > use in later determining hardware support in a dso or application binary. > > pmdi

[dpdk-dev] random pkt generator PMD

2016-06-15 Thread Neil Horman
On Wed, Jun 15, 2016 at 03:43:56PM +0600, Yerden Zhumabekov wrote: > Hello everybody, > > DPDK already got a number of PMDs for various eth devices, it even has PMD > emulations for backends such as pcap, sw rings etc. > > I've been thinking about the idea of having PMD which would generate

[dpdk-dev] [PATCH v8 0/8] add packet capture framework

2016-06-10 Thread Neil Horman
On Fri, Jun 10, 2016 at 05:18:46PM +0100, Reshma Pattan wrote: > This patch set include below changes > > 1)Changes to librte_ether. > 2)A new library librte_pdump added for packet capture framework. > 3)A new app/pdump tool added for packet capturing. > 4)Test pmd changes done to initialize

[dpdk-dev] [PATCHv7 6/6] doc: Add prog_guide section documenting pmdinfo script

2016-06-09 Thread Neil Horman
Information on pmdinfogen may be useful to 3rd party driver developers. Include documentation on what it does Signed-off-by: Neil Horman CC: Bruce Richardson CC: Thomas Monjalon CC: Stephen Hemminger CC: Panu Matilainen --- doc/guides/prog_guide/dev_kit_build_system.rst | 43

[dpdk-dev] [PATCHv7 5/6] pmdinfo.py: Add tool to query binaries for hw and other support information

2016-06-09 Thread Neil Horman
) pretty, where a user supplied pci.ids file is used to print out vendor and device strings Signed-off-by: Neil Horman CC: Bruce Richardson CC: Thomas Monjalon CC: Stephen Hemminger CC: Panu Matilainen --- mk/rte.sdkinstall.mk | 2 + tools/pmdinfo.py | 629

[dpdk-dev] [PATCHv7 4/6] Makefile: Do post processing on objects that register a driver

2016-06-09 Thread Neil Horman
into the parent object that it came from. This effectively just adds the json string into the string table of the object that defines the PMD to the outside world. Signed-off-by: Neil Horman CC: Bruce Richardson CC: Thomas Monjalon CC: Stephen Hemminger CC: Panu Matilainen --- mk/internal

[dpdk-dev] [PATCHv7 3/6] eal: Add an export symbol to expose the autoload path to external tools

2016-06-09 Thread Neil Horman
stallation directory (or static binary), and scan the associated path (if found) for pmd DSO's and report on their hardware support. Signed-off-by: Neil Horman CC: Bruce Richardson CC: Thomas Monjalon CC: Stephen Hemminger CC: Panu Matilainen --- lib/librte_eal/common/eal_common_opt

[dpdk-dev] [PATCHv7 2/6] drivers: Update driver registration macro usage

2016-06-09 Thread Neil Horman
create a json string to make hardware support info discoverable later. Signed-off-by: Neil Horman CC: Bruce Richardson CC: Thomas Monjalon CC: Stephen Hemminger CC: Panu Matilainen --- drivers/Makefile | 2 ++ drivers/crypto/aesni_gcm/aesni_gcm_pmd.c | 4 +++- dri

[dpdk-dev] [PATCHv7 1/6] pmdinfogen: Add buildtools and pmdinfogen utility

2016-06-09 Thread Neil Horman
hich the C was generated. This creates an entry in the string table of the object that can inform a later tool about hardware support. Signed-off-by: Neil Horman CC: Bruce Richardson CC: Thomas Monjalon CC: Stephen Hemminger CC: Panu Matilainen --- GNUmakefile

[dpdk-dev] [PATCHv7 0/6] Implement pmd hardware support exports

2016-06-09 Thread Neil Horman
some typos in doc Signed-off-by: Neil Horman CC: Bruce Richardson CC: Thomas Monjalon CC: Stephen Hemminger CC: Panu Matilainen

[dpdk-dev] [PATCHv6 7/7] doc: Add prog_guide section documenting pmdinfo script

2016-06-09 Thread Neil Horman
On Wed, Jun 08, 2016 at 05:14:05PM +, Mcnamara, John wrote: > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Neil Horman > > Sent: Tuesday, May 31, 2016 2:58 PM > > To: dev at dpdk.org > > Cc: Neil Horman ; Richardso

[dpdk-dev] [PATCHv6 1/7] pmdinfogen: Add buildtools and pmdinfogen utility

2016-06-07 Thread Neil Horman
On Tue, Jun 07, 2016 at 03:24:55PM +0200, Thomas Monjalon wrote: > 2016-06-07 09:03, Neil Horman: > > On Tue, Jun 07, 2016 at 02:53:36PM +0200, Thomas Monjalon wrote: > > > 2016-06-07 08:04, Neil Horman: > > > > On Tue, Jun 07, 2016 at 11:57:42AM +0200, Thomas Monjal

[dpdk-dev] [PATCHv6 1/7] pmdinfogen: Add buildtools and pmdinfogen utility

2016-06-07 Thread Neil Horman
On Tue, Jun 07, 2016 at 02:53:36PM +0200, Thomas Monjalon wrote: > 2016-06-07 08:04, Neil Horman: > > On Tue, Jun 07, 2016 at 11:57:42AM +0200, Thomas Monjalon wrote: > > > 2016-05-31 09:57, Neil Horman: > > > > +++ b/buildtools/Makefile > > > >

[dpdk-dev] [PATCHv6 0/7] Implement pmd hardware support exports

2016-06-07 Thread Neil Horman
On Tue, Jun 07, 2016 at 11:34:56AM +0200, Thomas Monjalon wrote: > Hi Neil, > > You are creating a new directory buildtools/ which depends on mk/ I don't know what this means. Every part of the dpdk that builds code depends on the infrastructure in mk/ (save for the scripts below) > We also

  1   2   3   4   5   6   7   8   9   10   >