[dpdk-dev] [PATCH 1/2] ethdev: add buffered tx api

2016-02-09 Thread Ananyev, Konstantin
> -Original Message- > From: Kulasek, TomaszX > Sent: Tuesday, February 09, 2016 5:03 PM > To: Ananyev, Konstantin; dev at dpdk.org > Subject: RE: [dpdk-dev] [PATCH 1/2] ethdev: add buffered tx api > > > > > -Original Message- > > From: Ananyev, Konstantin > > Sent: Tuesday,

[dpdk-dev] [PATCH v2 1/3] driver/net/mpipe: support native build on tilegx platform.

2016-02-09 Thread Thomas Monjalon
2016-02-09 21:15, Liming Sun: > Looks like this patch serie has been merged into dpdk-next-net/rel_16_04. > What would be the usual way to submit changes for new comments? Would it be > incremental changes (new commit) based on previous one? Thanks. Good question. I think it's better if Bruce

[dpdk-dev] Regarding mbuf allocation/free in secondary process

2016-02-09 Thread Saravana Kumar
Hi DPDK community, I'd like to have DPDK NIC IO operations in (primary) process and execution logic in (secondary) processes. Primary process pushes NIC Rx mbufs to Secondary process through S/W ring Seconary process allocates mbuf for Tx path and pushes down to Primary process for NIC Tx I

[dpdk-dev] [PATCH v4 2/2] driver/net/mpipe: some code optimization

2016-02-09 Thread Liming Sun
This commit includes two small code optimization. 1. use __insn_fetchadd4() instead of rte_atomic32_xxx() in mpipe_dp_enter/mpipe_dp_exit to avoid the nnecessary memory fence. 2. replace the calling of rte_pktmbuf_reset() in mpipe_recv_mbuf() with specific code to avoid some duplicated

[dpdk-dev] [PATCH v4 1/2] eal/tile: add rte_vect.h and enable CONFIG_RTE_LIBRTE_LPM

2016-02-09 Thread Liming Sun
rte_vect.h was missing earlier thus LPM was disabled and l3fwd is not able to compile. This commit implements the vector api and enable LPM in the tilegx configuration by default. Signed-off-by: Liming Sun Acked-by: Zhigang Lu --- config/defconfig_tile-tilegx-linuxapp-gcc | 2 +-

[dpdk-dev] [PATCH v3 2/2] driver/net/mpipe: some code optimization

2016-02-09 Thread Liming Sun
This commit includes two small code optimization. 1. use __insn_fetchadd4() instead of rte_atomic32_xxx() in mpipe_dp_enter/mpipe_dp_exit to avoid the nnecessary memory fence. 2. replace the calling of rte_pktmbuf_reset() in mpipe_recv_mbuf() with specific code to avoid some duplicated

[dpdk-dev] [PATCH v3 1/2] eal/tile: add rte_vect.h and enable CONFIG_RTE_LIBRTE_LPM

2016-02-09 Thread Liming Sun
rte_vect.h was missing earlier thus LPM was disabled and l3fwd is not able to compile. This commit implements the vector api and enable LPM in the tilegx configuration by default. Signed-off-by: Liming Sun Acked-by: Zhigang Lu --- config/defconfig_tile-tilegx-linuxapp-gcc | 2 +-

[dpdk-dev] [PATCH v2 1/3] driver/net/mpipe: support native build on tilegx platform.

2016-02-09 Thread Thomas Monjalon
2016-02-09 18:37, Liming Sun: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > A comment about the TILE-Mx would be welcome. > Is it supported currently? > Isn't it an ARM arch? > > [lsun] Yes, it's ARM arch, but with similar mpipe driver. > It's not fully supported yet. I'll

[dpdk-dev] [PATCH v2 1/3] driver/net/mpipe: support native build on tilegx platform.

2016-02-09 Thread Liming Sun
Looks like this patch serie has been merged into dpdk-next-net/rel_16_04. What would be the usual way to submit changes for new comments? Would it be incremental changes (new commit) based on previous one? Thanks. From: Thomas Monjalon

[dpdk-dev] i40e: cannot change mtu to enable jumbo frame

2016-02-09 Thread Zhu, Heqing
Helin is still in Chinese New Year Vacation. Will the below command option help ? 4.5.9. port config - max-pkt-len Set the maximum packet length: testpmd> port config all max-pkt-len (value) This is equivalent to the --max-pkt-len command-line option. -Original Message- From: dev

[dpdk-dev] [PATCH v2 1/2] driver/net/mpipe: add rte_vect.h and enable CONFIG_RTE_LIBRTE_LPM

2016-02-09 Thread Liming Sun
Make sense. I'll split them into different patches. Thanks! Liming -Original Message- From: Bruce Richardson [mailto:bruce.richard...@intel.com] Sent: Tuesday, February 09, 2016 10:56 AM To: Liming Sun Cc: dev at dpdk.org Subject: Re: [dpdk-dev] [PATCH v2 1/2] driver/net/mpipe: add

[dpdk-dev] [PATCH v9 2/2] vhost: Add VHOST PMD

2016-02-09 Thread Tetsuya Mukawa
The patch introduces a new PMD. This PMD is implemented as thin wrapper of librte_vhost. It means librte_vhost is also needed to compile the PMD. The vhost messages will be handled only when a port is started. So start a port first, then invoke QEMU. The PMD has 2 parameters. - iface: The

[dpdk-dev] [PATCH v9 1/2] ethdev: Add a new event type to notify a queue state changed event

2016-02-09 Thread Tetsuya Mukawa
This patch adds a below event type. - RTE_ETH_EVENT_QUEUE_STATE_CHANGE This event is used for notifying a queue state changed event. Signed-off-by: Tetsuya Mukawa Acked-by: Ferruh Yigit --- lib/librte_ether/rte_ethdev.h | 2 ++ 1 file changed, 2 insertions(+) diff --git

[dpdk-dev] [PATCH v9 0/2] Add VHOST PMD

2016-02-09 Thread Tetsuya Mukawa
The patch introduces a new PMD. This PMD is implemented as thin wrapper of librte_vhost. PATCH v9 changes: - Fix a null pointer access issue implemented in v8 patch. PATCH v8 changes: - Manage ether devices list instead of internal structures list. - Remove needless NULL checking. - Replace

[dpdk-dev] [PATCH v2 1/3] driver/net/mpipe: support native build on tilegx platform.

2016-02-09 Thread Liming Sun
Thanks Thomas for the comments. Please see the response inline. Thanks, Liming -Original Message- From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] Sent: Tuesday, February 09, 2016 11:16 AM To: Liming Sun Cc: dev at dpdk.org; bruce.richardson at intel.com Subject: Re: [dpdk-dev]

[dpdk-dev] [PATCH 5/9] eal: get rid of pmd type

2016-02-09 Thread Jan Viktorin
On Fri, 29 Jan 2016 15:08:32 +0100 David Marchand wrote: > Now that we only have vdev drivers, there is no need for a pmd type in > rte_driver. > rte_driver is now purely a vdev thing, and could be renamed later > (then .init would become .probe, .uninit would become .remove). > >

[dpdk-dev] [PATCH 3/9] drivers: no more pdev drivers

2016-02-09 Thread Jan Viktorin
Maybe, a better subject? drivers: init pdev drivers in constructors On Fri, 29 Jan 2016 15:08:30 +0100 David Marchand wrote: > Now that pdev drivers have been converted to pci drivers, there is nothing > left in their init functions that can't go in a constructor. > pdev / vdev drivers init

[dpdk-dev] [PATCH 2/2] kdp: add virtual PMD for kernel slow data path communication

2016-02-09 Thread Ferruh Yigit
On Tue, Feb 09, 2016 at 05:33:55PM +, Reshma Pattan wrote: > Hi Ferruh, > Hi Reshma, > On 1/27/2016 4:32 PM, Ferruh Yigit wrote: >> This patch provides slow data path communication to the Linux kernel. >> Patch is based on librte_kni, and heavily re-uses it. >> >> The main difference is

[dpdk-dev] i40e: cannot change mtu to enable jumbo frame

2016-02-09 Thread Julien Meunier
Hello Helin, I tried to send jumbo frames to a i40e card. However, I observed that all frames are dropped. Moreover, set_mtu function is not implemented on i40e PMD. > testpmd --log-level 8 --huge-dir=/mnt/huge -n 4 -l 2,18 --socket-mem 1024,1024 -w :02:00.0 -w :02:00.2 -- -i

[dpdk-dev] [PATCH 2/2] kdp: add virtual PMD for kernel slow data path communication

2016-02-09 Thread Reshma Pattan
Hi Ferruh, On 1/27/2016 4:32 PM, Ferruh Yigit wrote: > This patch provides slow data path communication to the Linux kernel. > Patch is based on librte_kni, and heavily re-uses it. > > The main difference is librte_kni library converted into a PMD, to > provide ease of use for applications. > >

[dpdk-dev] DPDK i40evf problem in receving packet

2016-02-09 Thread Saurabh Mishra
Hi Qian -- Any suggestions? This is bit urgent. /Saurabh On Sat, Feb 6, 2016 at 9:22 AM, Saurabh Mishra wrote: > Hi Qian -- > > > Here's the data from Host: > > [root at oscompute3 ~]# ethtool -i p3p1 > > driver: i40e > > version: 1.0.11-k > > firmware-version: f4.40 a1.4 n04.53 e80001dc0 > >

[dpdk-dev] [PATCH] doc: drop old naming of the project

2016-02-09 Thread Bruce Richardson
On Mon, Feb 08, 2016 at 02:39:02PM +0100, Thomas Monjalon wrote: > It was requested by Intel, more than one year ago, to replace the name > "Intel DPDK" by "DPDK". > Some references to the old name were still in some docs and code comments, > leading to confusion. > > Fixes: ac8ada004c12 ("doc:

[dpdk-dev] [PATCH v2 1/3] driver/net/mpipe: support native build on tilegx platform.

2016-02-09 Thread Thomas Monjalon
Hi, Sorry for being late for commenting. 2016-01-08 09:30, Liming Sun: > -EZchip TILE-Gx > +EZchip TILE-Gx/Mx A comment about the TILE-Mx would be welcome. Is it supported currently? Isn't it an ARM arch? > M: Zhigang Lu > +M: Liming Sun > F: lib/librte_eal/common/include/arch/tile/ > F:

[dpdk-dev] [PATCH v2] af_packet: make the device detachable

2016-02-09 Thread Wojciech Zmuda
Implement rte_pmd_af_packet_devuninit() exposed through struct rte_driver.uninit() and set dev_flags to RTE_ETH_DEV_DETACHABLE, to allow af_packet device deinitialization with API function rte_eth_dev_detach(). This fixes memory leak by freeing memory allocated during initialization. During device

[dpdk-dev] [PATCH v7 2/2] eal/linux: Add support for handling built-in kernel modules

2016-02-09 Thread Kamil Rytarowski
Thank you! W dniu 09.02.2016 o 15:56, Thomas Monjalon pisze: > 2016-01-28 14:13, krytarowski at caviumnetworks.com: >> From: Kamil Rytarowski >> >> Currently rte_eal_check_module() detects Linux kernel modules via reading >> /proc/modules. Built-in ones aren't listed there and therefore they are

[dpdk-dev] [PATCH 1/2] ethdev: add buffered tx api

2016-02-09 Thread Kulasek, TomaszX
> -Original Message- > From: Ananyev, Konstantin > Sent: Tuesday, February 2, 2016 14:50 > To: Kulasek, TomaszX ; dev at dpdk.org > Subject: RE: [dpdk-dev] [PATCH 1/2] ethdev: add buffered tx api > > Hi Tomasz, > > > -Original Message- > > From: Kulasek, TomaszX > > Sent:

[dpdk-dev] [PATCH v2] af_packet: make the device detachable

2016-02-09 Thread Bruce Richardson
On Tue, Feb 09, 2016 at 05:09:06PM +0100, Wojciech Zmuda wrote: > Implement rte_pmd_af_packet_devuninit() exposed through struct > rte_driver.uninit() and set dev_flags to RTE_ETH_DEV_DETACHABLE, > to allow af_packet device deinitialization with API function > rte_eth_dev_detach(). This fixes

[dpdk-dev] [PATCH v2 2/2] driver/net/mpipe: fix the crash/hung issue when testpmd quits

2016-02-09 Thread Bruce Richardson
On Fri, Jan 08, 2016 at 09:39:08AM -0500, Liming Sun wrote: > 1. Fixed the compiling issue of the ethtool example on tilegx >platform. > 2. Fixed the hung/crash issue when quitting testpmd under high >traffic rate. The buffer error bit needs to be checked before >processing the idesc

[dpdk-dev] [PATCH v7 2/2] eal/linux: Add support for handling built-in kernel modules

2016-02-09 Thread Thomas Monjalon
2016-01-28 14:13, krytarowski at caviumnetworks.com: > From: Kamil Rytarowski > > Currently rte_eal_check_module() detects Linux kernel modules via reading > /proc/modules. Built-in ones aren't listed there and therefore they are not > being found by the script. > > Add support for checking

[dpdk-dev] [PATCH v2 1/2] driver/net/mpipe: add rte_vect.h and enable CONFIG_RTE_LIBRTE_LPM

2016-02-09 Thread Bruce Richardson
On Fri, Jan 08, 2016 at 09:39:07AM -0500, Liming Sun wrote: > rte_vect.h was missing earlier thus LPM was disabled and l3fwd is > not able to compile. This commit implements the vector api and > enable LPM in the tilegx configuration by default. It also includes > a minor optimization to use

[dpdk-dev] [PATCH v2 1/3] driver/net/mpipe: support native build on tilegx platform.

2016-02-09 Thread Bruce Richardson
On Fri, Jan 08, 2016 at 09:30:36AM -0500, Liming Sun wrote: > This submit updates the CROSS setting to support native build on > TileGx platform. It also enable the combined library by default. > > Signed-off-by: Liming Sun > Acked-by: Zhigang Lu Series applied to dpdk-next-net/rel_16_04.

[dpdk-dev] [PATCH v2] docs: add statistics read frequency to fm10k guide

2016-02-09 Thread Harry van Haaren
This patch documents that the statistics of fm10k based NICs must be read regularly in order to avoid an undetected 32 bit integer-overflow. Signed-off-by: Harry van Haaren --- v2: -Moved notes to Known Issues section of Release Notes. doc/guides/nics/fm10k.rst| 16

[dpdk-dev] [PATCH] docs: add statistics read frequency to fm10k guide

2016-02-09 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > Sent: Monday, February 8, 2016 4:01 PM > To: Van Haaren, Harry > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] docs: add statistics read frequency to > fm10k guide > > 2016-02-08

[dpdk-dev] [PATCH] tools: fix syntax errors and add support for Python 3

2016-02-09 Thread Thomas Monjalon
2016-01-27 20:59, yurai: > This patch fixes syntax errors from tools/setup.sh during binding ethernet > device > on systems where Python 3 is default. Backward compability with Python 2 is > preserved. > > Signed-off-by: Dawid Jurczak Applied, thanks

[dpdk-dev] [PATCH v2 4/4] virtio: use pci ioport api

2016-02-09 Thread Tetsuya Mukawa
On 2016/02/07 16:48, David Marchand wrote: > Move all os / arch specifics to eal. > > Signed-off-by: David Marchand > --- > drivers/net/virtio/virtio_pci.c | 339 > +++- > drivers/net/virtio/virtio_pci.h | 38 + > 2 files changed, 55 insertions(+), 322

[dpdk-dev] [PATCH] tools: Fix typo in dpdk_nic_bind.py script.

2016-02-09 Thread Thomas Monjalon
2016-02-09 11:10, Bruce Richardson: > On Mon, Feb 08, 2016 at 04:33:46PM -0800, Jeff Shaw wrote: > > We should call sys.exit(), not divide sys by exit(). Obviously :) > > Signed-off-by: Jeff Shaw > > Acked-by: Bruce Richardson Applied, thanks

[dpdk-dev] [PATCH v2] doc: introduce networking driver matrix

2016-02-09 Thread Thomas Monjalon
2016-01-27 21:07, Thomas Monjalon: > In order to better compare the drivers and check what is missing > for a common baseline, we need to fill a matrix. > > A CSS trick is used to fit the HTML page. > The PDF output needs some LaTeX wizardry. > > Signed-off-by: Thomas Monjalon > --- > v2: add

[dpdk-dev] [PATCH v2] doc: minor correction in document

2016-02-09 Thread Thomas Monjalon
2016-02-02 13:11, Ferruh Yigit: > * remove outdated chapter reference to Multi-process support. > Fixes: fc1f2750a3ec ("doc: programmers guide") > > * html output converts "--" to "-", this is wrong when explaining the > command arguments, used fixed width quotes for them. > > v2: > * for

[dpdk-dev] [PATCH v2] e1000: enable promiscuous and allmulticast support for VF

2016-02-09 Thread Yury Kylulin
Enable promiscuous and allmulticast mode control from the VF using rte_eth_promiscuous_enable()/rte_eth_promiscuous_disable() and rte_eth_allmulticast_enable()/rte_eth_allmulticast_disable(). For promiscuous mode host/PF igb driver should be built with IGB_ENABLE_VF_PROMISC. For allmulticast

[dpdk-dev] [PATCH] doc: add a further ACL example

2016-02-09 Thread Thomas Monjalon
> > Add a further ACL example where the elements of the search key are not > > entirely fitting into the 4 consecutive bytes of all input fields. > > > > Signed-off-by: Antonio Fischetti > > Acked-by: John McNamara Applied, thanks

[dpdk-dev] [PATCH] mk: add makefile extention support

2016-02-09 Thread Keith Wiles
Adding support to the build system to allow for Makefile.XXX extention to a subtree, which already has Makefiles. These Makefiles could be from the autotools and others places. Using the Makefile extention RTE_MKFILE_SUFFIX in a makefile subtree using 'export RTE_MKFILE_SUFFIX=.XXX' to use

[dpdk-dev] [PATCH v2] mempool: reduce rte_mempool structure size

2016-02-09 Thread Keith Wiles
Patch v2 to add some comments and setup for RTE_NEXT_ABI changes. The rte_mempool structure is changed, which will cause an ABI change for this structure. Providing backward compat is not reasonable here as this structure is used in multiple defines/inlines. Allow mempool cache support to be

[dpdk-dev] DPDK Community Call - Linux Foundation

2016-02-09 Thread Dave Neary
Is there any substantive feedback on this? It seems like silence = consensus (or that the people who might care more are not on this list). My suggestion is to go to the next stage, start drafting a membership agreement and identify potential members to recruit. Will the Linux Foundation take

[dpdk-dev] [PATCH] tools: Fix typo in dpdk_nic_bind.py script.

2016-02-09 Thread Bruce Richardson
On Mon, Feb 08, 2016 at 04:33:46PM -0800, Jeff Shaw wrote: > We should call sys.exit(), not divide sys by exit(). > > Signed-off-by: Jeff Shaw Acked-by: Bruce Richardson

[dpdk-dev] [PATCH] mlx4: fix unneeded function error with clang v3.6

2016-02-09 Thread Bruce Richardson
On Tue, Feb 09, 2016 at 09:39:26AM +0100, Adrien Mazarguil wrote: > On Mon, Feb 08, 2016 at 10:11:49PM +, Bruce Richardson wrote: > > When compiling with clang 3.6, the mlx4 driver gives the following error > > message about an unneeded function. > > > > CC mlx4.o > >

[dpdk-dev] [PATCH v7 2/2] vhost: Add VHOST PMD

2016-02-09 Thread Tetsuya Mukawa
On 2016/02/08 18:42, Ferruh Yigit wrote: > On Fri, Feb 05, 2016 at 03:28:37PM +0900, Tetsuya Mukawa wrote: >> On 2016/02/04 20:17, Ferruh Yigit wrote: >>> On Thu, Feb 04, 2016 at 04:26:31PM +0900, Tetsuya Mukawa wrote: >>> >>> Hi Tetsuya, >>> The patch introduces a new PMD. This PMD is

[dpdk-dev] [PATCH 1/2] kdp: add kernel data path kernel module

2016-02-09 Thread Ferruh Yigit
On Mon, Feb 08, 2016 at 05:14:54PM +, Reshma Pattan wrote: Hi Reshma, > > > On 1/27/2016 4:32 PM, Ferruh Yigit wrote: >> This kernel module is based on KNI module, but this one is stripped >> version of it and only for data messages, no control functionality >> provided. >> >> FIFO

[dpdk-dev] [PATCH v2] hash: fix CRC32c computation

2016-02-09 Thread Didier Pallard
As demonstrated by the following code, CRC32c computation is not valid when buffer length is not a multiple of 4 bytes: (Output obtained by code below) CRC of 1 NULL bytes expected: 0x527d5351 soft: 527d5351 rte accelerated: 48674bc7 rte soft: 48674bc7 CRC of 2 NULL bytes expected:

[dpdk-dev] [PATCH v8 4/4] eal/linux: vfio: add pci ioport support

2016-02-09 Thread David Marchand
On Mon, Feb 8, 2016 at 3:13 PM, Burakov, Anatoly wrote: >> Include vfio map/rd/wr support for pci ioport. >> >> Signed-off-by: Santosh Shukla >> --- >> v7->v8: >> - Remove rte_pci_ioport malloc and rte_free()/unmap() func from v7. >> - removed umap from git header. >> >>

[dpdk-dev] [PATCH 2/9] pci: register all pdev as pci drivers

2016-02-09 Thread David Marchand
On Mon, Feb 8, 2016 at 12:03 PM, Jan Viktorin wrote: > On Fri, 29 Jan 2016 15:08:29 +0100 > David Marchand wrote: > >> pdev drivers are actually pci drivers. >> Wrappers for ethdev and crypto pci drivers, that assume a 1 to 1 >> association between pci device and upper device, have been added

[dpdk-dev] [PATCH] mlx4: fix unneeded function error with clang v3.6

2016-02-09 Thread Adrien Mazarguil
On Mon, Feb 08, 2016 at 10:11:49PM +, Bruce Richardson wrote: > When compiling with clang 3.6, the mlx4 driver gives the following error > message about an unneeded function. > > CC mlx4.o > .../drivers/net/mlx4/mlx4.c:136:20: fatal error: function > 'wr_id_t_check' is not needed and

[dpdk-dev] [PATCH v2 0/9] pci cleanup and blacklist rework

2016-02-09 Thread David Marchand
Hello Jan, On Mon, Feb 8, 2016 at 2:31 PM, Jan Viktorin wrote: > I am confused a bit. I started to review the "[PATCH 0/9] prepare for > rte_device > / rte_driver" series and then I've noticed there are 2 patch series having > "pci: > no need for dynamic tailq init" patch there. But then,

[dpdk-dev] [PATCH] af_packet: make the device detachable

2016-02-09 Thread Iremonger, Bernard
Hi Wojciech > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Wojciech Zmuda > > Sent: Tuesday, January 5, 2016 2:05 PM > > To: dev at dpdk.org > > Subject: [dpdk-dev] [PATCH] af_packet: make the device detachable > > > > Fix memory leak when detaching

[dpdk-dev] [PATCH v2 4/4] virtio: use pci ioport api

2016-02-09 Thread David Marchand
On Tue, Feb 9, 2016 at 4:52 AM, Tetsuya Mukawa wrote: > On 2016/02/07 16:48, David Marchand wrote: > One more legacy_virtio_resource_init() is defined like below. > > #ifdef RTE_EXEC_ENV_LINUXAPP > > #else > > static int > legacy_virtio_resource_init(struct rte_pci_device *pci_dev __rte_unused)