[dpdk-dev] [PATCH 2/4] eventdev: implement the northbound APIs

2016-11-22 Thread Eads, Gage
> -Original Message- > From: Jerin Jacob [mailto:jerin.jacob at caviumnetworks.com] > Sent: Tuesday, November 22, 2016 2:00 PM > To: Eads, Gage > Cc: dev at dpdk.org; Richardson, Bruce ; Van > Haaren, Harry ; hemant.agrawal at nxp.com > Subject: Re: [dpdk-dev] [PATCH 2/4]

[dpdk-dev] dpdk/vpp and cross-version migration for vhost

2016-11-22 Thread Yuanhan Liu
On Thu, Nov 17, 2016 at 07:37:09PM +0200, Michael S. Tsirkin wrote: > On Thu, Nov 17, 2016 at 05:49:36PM +0800, Yuanhan Liu wrote: > > On Thu, Nov 17, 2016 at 09:47:09AM +0100, Maxime Coquelin wrote: > > > > > > > > > On 11/17/2016 09:29 AM, Yuanhan Liu wrote: > > > >As usaual, sorry for late

[dpdk-dev] Proposal for a new Committer model

2016-11-22 Thread Ferruh Yigit
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. This was a conversation that I >>> proposed some time ago, but my proposed granularity was discarded in

[dpdk-dev] Adding API to force freeing consumed buffers in TX ring

2016-11-22 Thread Wiles, Keith
> On Nov 21, 2016, at 9:25 AM, Richardson, Bruce intel.com> wrote: > > On Mon, Nov 21, 2016 at 04:06:32PM +0100, Olivier Matz wrote: >> Hi, >> >> On 11/21/2016 03:33 PM, Wiles, Keith wrote: >>> On Nov 21, 2016, at 4:48 AM, Damjan Marion (damarion) >>> cisco.com> wrote:

[dpdk-dev] [PATCH 2/4] eventdev: implement the northbound APIs

2016-11-22 Thread Eads, Gage
> -Original Message- > From: Jerin Jacob [mailto:jerin.jacob at caviumnetworks.com] > Sent: Tuesday, November 22, 2016 12:19 PM > To: Eads, Gage > Cc: dev at dpdk.org; Richardson, Bruce ; Van > Haaren, Harry ; hemant.agrawal at nxp.com > Subject: Re: [dpdk-dev] [PATCH 2/4]

[dpdk-dev] [PATCH v2 3/8] drivers: Use ETH_DEV_PCI_DEV() helper

2016-11-22 Thread Shreyansh Jain
On Monday 21 November 2016 10:25 PM, Jan Blunck wrote: > The drivers should not directly access the rte_eth_dev->pci_dev but use > a macro instead. This is a preparation for replacing the pci_dev with > a struct rte_device member in the future. > > Signed-off-by: Jan Blunck > --- >

[dpdk-dev] [PATCH v2 1/8] eal: define container_of macro

2016-11-22 Thread Shreyansh Jain
On Monday 21 November 2016 10:25 PM, Jan Blunck wrote: > This macro is based on Jan Viktorin's original patch but also checks the > type of the passed pointer against the type of the member. > > Signed-off-by: Jan Viktorin > Signed-off-by: Shreyansh Jain > [jblunck at infradead.org: add type

[dpdk-dev] [PATCH 0/2] l2fwd/l3fwd: rework long options parsing

2016-11-22 Thread Olivier Matz
Hi, On 11/22/2016 02:52 PM, Olivier Matz wrote: > These 2 patches were part of this RFC, which will not be integrated: > http://dpdk.org/ml/archives/dev/2016-September/046974.html > > It does not bring any functional change, it just reworks the way long > options are parsed in l2fwd and l3fwd to

[dpdk-dev] [PATCH 1/2] l3fwd: rework long options parsing

2016-11-22 Thread Olivier Matz
Avoid the use of several strncpy() since getopt is able to map a long option with an id, which can be matched in the same switch/case than short options. Signed-off-by: Olivier Matz --- examples/l3fwd/main.c | 169 ++ 1 file changed, 87

[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 something that hasn't worked as

[dpdk-dev] [PATCH 0/2] l2fwd/l3fwd: rework long options parsing

2016-11-22 Thread Olivier Matz
These 2 patches were part of this RFC, which will not be integrated: http://dpdk.org/ml/archives/dev/2016-September/046974.html It does not bring any functional change, it just reworks the way long options are parsed in l2fwd and l3fwd to avoid uneeded strcmp() calls and to ease the addition of a

[dpdk-dev] [PATCH 0/4] libeventdev API and northbound implementation

2016-11-22 Thread Shreyansh Jain
On Tuesday 22 November 2016 07:30 AM, Yuanhan Liu wrote: > On Sat, Nov 19, 2016 at 12:57:15AM +0530, Jerin Jacob wrote: >> On Fri, Nov 18, 2016 at 04:04:29PM +, Bruce Richardson wrote: >>> +Thomas >>> >>> On Fri, Nov 18, 2016 at 03:25:18PM +, Bruce Richardson wrote: On Fri, Nov 18,

[dpdk-dev] [PATCH] virtio: tx with can_push when VERSION_1 is set

2016-11-22 Thread Maxime Coquelin
Hi Pierre, On 11/22/2016 10:54 AM, Pierre Pfister (ppfister) wrote: > Hello Maxime, > >> Le 9 nov. 2016 ? 15:51, Maxime Coquelin a >> ?crit : >> >> Hi Pierre, >> >> On 11/09/2016 01:42 PM, Pierre Pfister (ppfister) wrote: >>> Hello Maxime, >>> >>> Sorry for the late reply. >>> >>> Le 8

[dpdk-dev] [RFC PATCH 0/7] RFC: EventDev Software PMD

2016-11-22 Thread Richardson, Bruce
> -Original Message- > From: Jerin Jacob [mailto:jerin.jacob at caviumnetworks.com] > Sent: Monday, November 21, 2016 8:19 PM > To: Richardson, Bruce > Cc: Van Haaren, Harry ; dev at dpdk.org > Subject: Re: [dpdk-dev] [RFC PATCH 0/7] RFC: EventDev Software PMD > > On Mon, Nov 21, 2016

[dpdk-dev] [PATCH 7/7] examples/eventdev_pipeline: adding example

2016-11-22 Thread Richardson, Bruce
> -Original Message- > From: Jerin Jacob [mailto:jerin.jacob at caviumnetworks.com] > Sent: Tuesday, November 22, 2016 6:02 AM > To: Van Haaren, Harry > Cc: dev at dpdk.org; Eads, Gage ; Richardson, Bruce > > Subject: Re: [dpdk-dev] [PATCH 7/7] examples/eventdev_pipeline: adding >

[dpdk-dev] [PATCH v2] i40e: Fix eth_i40e_dev_init sequence on ThunderX

2016-11-22 Thread Bruce Richardson
On Tue, Nov 22, 2016 at 03:46:38AM +0530, Jerin Jacob wrote: > On Sun, Nov 20, 2016 at 11:21:43PM +, Ananyev, Konstantin wrote: > > Hi > > > > > > i40e_asq_send_command: rd32 & wr32 under ThunderX gives unpredictable > > >results. To solve this include rte memory

[dpdk-dev] [PATCH v2 1/8] eal: define container_of macro

2016-11-22 Thread Jan Viktorin
On Tue, 22 Nov 2016 11:26:50 + Shreyansh Jain wrote: > > -Original Message- > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Sent: Tuesday, November 22, 2016 3:50 PM > > To: Shreyansh Jain ; Jan Blunck > > > > Cc: dev at dpdk.org; david.marchand at 6wind.com; Jan

[dpdk-dev] [PATCH v2 1/8] eal: define container_of macro

2016-11-22 Thread Shreyansh Jain
> -Original Message- > From: Jan Viktorin [mailto:viktorin at rehivetech.com] > Sent: Tuesday, November 22, 2016 6:03 PM > To: Shreyansh Jain > Cc: Thomas Monjalon ; Jan Blunck > ; dev at dpdk.org; david.marchand at 6wind.com > Subject: Re: [dpdk-dev] [PATCH v2 1/8] eal: define

[dpdk-dev] [PATCH] mk: remove make target for examples

2016-11-22 Thread Ferruh Yigit
On 11/22/2016 9:38 AM, Thomas Monjalon wrote: > 2016-11-22 00:34, Ferruh Yigit: >> On 11/21/2016 11:47 PM, Thomas Monjalon wrote: >>> The command >>> make examples >>> works only if target directories have the exact name of configs. >>> >>> It is more flexible to use >>> make -C examples

[dpdk-dev] [PATCH v2] drivers: advertise kmod dependencies in pmdinfo

2016-11-22 Thread Olivier Matz
Hi Adrien, On 11/22/2016 11:27 AM, Adrien Mazarguil wrote: > Hi Olivier, > > Neither mlx4 nor mlx5 depend on igb/uio/vfio modules, please see below. > > On Tue, Nov 22, 2016 at 10:50:57AM +0100, Olivier Matz wrote: >> Add a new macro RTE_PMD_REGISTER_KMOD_DEP() that allows a driver to >>

[dpdk-dev] [PATCH v2] drivers: advertise kmod dependencies in pmdinfo

2016-11-22 Thread Adrien Mazarguil
Hi Olivier, Neither mlx4 nor mlx5 depend on igb/uio/vfio modules, please see below. On Tue, Nov 22, 2016 at 10:50:57AM +0100, Olivier Matz wrote: > Add a new macro RTE_PMD_REGISTER_KMOD_DEP() that allows a driver to > declare the list of kernel modules required to run properly. > > Today, most

[dpdk-dev] [PATCH v2 1/8] eal: define container_of macro

2016-11-22 Thread Shreyansh Jain
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Tuesday, November 22, 2016 3:50 PM > To: Shreyansh Jain ; Jan Blunck > > Cc: dev at dpdk.org; david.marchand at 6wind.com; Jan Viktorin > > Subject: Re: [dpdk-dev] [PATCH v2 1/8] eal: define

[dpdk-dev] [PATCH v2 1/8] eal: define container_of macro

2016-11-22 Thread Thomas Monjalon
2016-11-22 15:33, Shreyansh Jain: > On Monday 21 November 2016 10:25 PM, Jan Blunck wrote: > > This macro is based on Jan Viktorin's original patch but also checks the > > type of the passed pointer against the type of the member. > > > > Signed-off-by: Jan Viktorin > > Signed-off-by: Shreyansh

[dpdk-dev] [PATCH] mempool: fix Api documentation

2016-11-22 Thread Olivier Matz
A previous commit changed the local_cache table into a pointer, reducing the size of the rte_mempool structure. Fix the API comment of rte_mempool_create() related to this modification. Fixes: 213af31e0960 ("mempool: reduce structure size if no cache needed") Signed-off-by: Olivier Matz ---

[dpdk-dev] [PATCH v2] drivers: advertise kmod dependencies in pmdinfo

2016-11-22 Thread Olivier Matz
Add a new macro RTE_PMD_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 Acked-by: Fiona Trahe --- v1 -> v2:

[dpdk-dev] [PATCH] mk: remove make target for examples

2016-11-22 Thread Thomas Monjalon
2016-11-22 00:34, Ferruh Yigit: > On 11/21/2016 11:47 PM, Thomas Monjalon wrote: > > The command > > make examples > > works only if target directories have the exact name of configs. > > > > It is more flexible to use > > make -C examples RTE_SDK=$(pwd) RTE_TARGET=build > > > >

[dpdk-dev] [PATCH v3 2/2] mempool: pktmbuf pool default fallback for mempool ops error

2016-11-22 Thread Olivier Matz
Hi Hemant, Back on this topic, please see some comments below. On 11/07/2016 01:30 PM, Hemant Agrawal wrote: > Hi Olivier, > >> -Original Message- >> From: Olivier Matz [mailto:olivier.matz at 6wind.com] >> Sent: Friday, October 14, 2016 5:41 PM >>> On 9/22/2016 6:42 PM, Hemant

[dpdk-dev] [PATCH] virtio: tx with can_push when VERSION_1 is set

2016-11-22 Thread Pierre Pfister (ppfister)
Hello Maxime, > Le 9 nov. 2016 ? 15:51, Maxime Coquelin a > ?crit : > > Hi Pierre, > > On 11/09/2016 01:42 PM, Pierre Pfister (ppfister) wrote: >> Hello Maxime, >> >> Sorry for the late reply. >> >> >>> Le 8 nov. 2016 ? 10:44, Maxime Coquelin a >>> ?crit : >>> >>> Hi Pierre, >>> >>> On

[dpdk-dev] [PATCH v2] examples/ethtool: fix bug in drvinfo callback

2016-11-22 Thread Qiming Yang
Function pcmd_drvinfo_callback uses struct info to get the ethtool information of each port. Struct info will store the information of previous port until this information be updated. This patch fixes this issue. Fixes: bda68ab9d1e7 ("examples/ethtool: add user-space ethtool sample application")

[dpdk-dev] [RFC PATCH 0/7] RFC: EventDev Software PMD

2016-11-22 Thread Jerin Jacob
On Mon, Nov 21, 2016 at 09:48:56AM +, Bruce Richardson wrote: > On Sat, Nov 19, 2016 at 03:53:25AM +0530, Jerin Jacob wrote: > > On Thu, Nov 17, 2016 at 10:05:07AM +, Bruce Richardson wrote: > > > > 2) device stats API can be based on capability, HW implementations may > > > > not > > > >

[dpdk-dev] [PATCH 0/4] libeventdev API and northbound implementation

2016-11-22 Thread Thomas Monjalon
2016-11-21 09:57, Bruce Richardson: > On Mon, Nov 21, 2016 at 10:40:50AM +0100, Thomas Monjalon wrote: > > Are you asking for a temporary tree? > > If yes, please tell its name and its committers, it will be done. > > Yes, we are asking for a new tree, but I would not assume it is > temporary -

[dpdk-dev] [PATCH 2/4] eventdev: implement the northbound APIs

2016-11-22 Thread Jerin Jacob
On Tue, Nov 22, 2016 at 12:43:58AM +0530, Jerin Jacob wrote: > On Mon, Nov 21, 2016 at 05:45:51PM +, Eads, Gage wrote: > > Hi Jerin, > > > > I did a quick review and overall this implementation looks good. I noticed > > just one issue in rte_event_queue_setup(): the check of > >

[dpdk-dev] [PATCH] mk: remove make target for examples

2016-11-22 Thread Thomas Monjalon
The command make examples works only if target directories have the exact name of configs. It is more flexible to use make -C examples RTE_SDK=$(pwd) RTE_TARGET=build Signed-off-by: Thomas Monjalon --- mk/rte.sdkexamples.mk | 77 ---

[dpdk-dev] [PATCH 2/4] eventdev: implement the northbound APIs

2016-11-22 Thread Jerin Jacob
On Mon, Nov 21, 2016 at 05:45:51PM +, Eads, Gage wrote: > Hi Jerin, > > I did a quick review and overall this implementation looks good. I noticed > just one issue in rte_event_queue_setup(): the check of > nb_atomic_order_sequences is being applied to atomic-type queues, but that > field

[dpdk-dev] [PATCH] mk: remove make target for examples

2016-11-22 Thread Ferruh Yigit
On 11/21/2016 11:47 PM, Thomas Monjalon wrote: > The command > make examples > works only if target directories have the exact name of configs. > > It is more flexible to use > make -C examples RTE_SDK=$(pwd) RTE_TARGET=build > > Signed-off-by: Thomas Monjalon Instead of removing examples