[dpdk-dev] [PATCH v2] virtio: fix crash loading virtio driver when devargs isn't specified

2016-06-13 Thread Huawei Xie
We skip kernel managed virtio devices, if it isn't whitelisted. Before checking if the virtio device is whitelisted, check if devargs is specified. Fixes: ac5e1d838dc1 ("virtio: skip error when probing kernel managed device") Signed-off-by: Huawei Xie Reported-by: Vincent Li --- v2: - add

[dpdk-dev] [PATCH v4 0/4] app/test: rework crypto AES unit test

2016-06-13 Thread Thomas Monjalon
> Fan Zhang (4): > app/test: categorize crypto AES test vectors into new file > app/test: add unified crypto aes test > app/test: utilize new unified crypto AES test function > app/test: add crypto AES-CBC-128 HMAC-SHA224 and HMAC-SHA384 unit > tests My first comment on v1 was: " The

[dpdk-dev] [PATCH v2] test: fix mempool perf test enq_count wraparound of 32-bit uint

2016-06-13 Thread Thomas Monjalon
> > recent CPU's can easily wrap around a 32-bit unsigned int in > > the mempool perf test. Increase to a 64-bit uint. > > > > v2: change from %lu to %"PRIu64" > > > > Signed-off-by: David Hunt > > Acked-by: Olivier Matz Applied, thanks

[dpdk-dev] [PATCH] app/test: fix bond device name too long

2016-06-13 Thread Thomas Monjalon
> > Bond device name was too long (grather than 32 signs) that cause mempool > > allocation to fail. > > > > Fixes: 92073ef961ee ("bond: unit tests") > > > > Signed-off-by: Michal Jastrzebski > > Acked-by: Bernard Iremonger Applied, thanks

[dpdk-dev] [PATCH] app/test: fix array overflow warning with gcc 4.5

2016-06-13 Thread Thomas Monjalon
2016-06-09 13:14, Tomasz Kulasek: > DPDK/app/test/test_cryptodev.c: In function ?create_snow3g_cipher_operation > _oop.clone.15?: DPDK/x86_64-native-linuxapp-gcc/include/rte_memcpy.h:796:14 > error: array subscript is above array bounds. > > In test_cryptodev.c: > 2429

[dpdk-dev] [PATCH v5 00/10] Include resources in tests

2016-06-13 Thread Thomas Monjalon
> Jan Viktorin (10): > app/test: introduce resources for tests > mk: define objcopy-specific target and arch > app/test: support resources externally linked > app/test: add functions to create files from resources > app/test: support resources archived by tar > app/test: use linked

[dpdk-dev] [PATCH] vhost: remove internal lockless enqueue

2016-06-13 Thread Huawei Xie
All other DPDK PMDs doesn't support concurrent receiving or sending packets to the same queue. The upper application should deal with this, normally through queue and core bindings. Due to historical reason, vhost internally supports concurrent lockless enqueuing packets to the same virtio queue

[dpdk-dev] Performance hit - NICs on different CPU sockets

2016-06-13 Thread Wiles, Keith
On 6/13/16, 9:07 AM, "dev on behalf of Take Ceara" wrote: >Hi, > >I'm reposting here as I didn't get any answers on the dpdk-users mailing list. > >We're working on a stateful traffic generator (www.warp17.net) using >DPDK and we would like to control two XL710 NICs (one on each socket) >to

[dpdk-dev] [PATCH] config: thunderx: select maximum supported numa nodes and lcores

2016-06-13 Thread Jerin Jacob
ThunderX platform can support dual sockets numa configuration and 48 cores per socket. Signed-off-by: Jerin Jacob --- config/defconfig_arm64-thunderx-linuxapp-gcc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/config/defconfig_arm64-thunderx-linuxapp-gcc

[dpdk-dev] [PATCH v4 19/19] maintainers: claim responsibility for the ThunderX nicvf PMD

2016-06-13 Thread Jerin Jacob
Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 3e8558f..625423f 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -336,6 +336,12 @@ M: Sony Chacko F: drivers/net/qede/ F:

[dpdk-dev] [PATCH v4 18/19] net/thunderx: updated driver documentation and release notes

2016-06-13 Thread Jerin Jacob
Updated doc/guides/nics/overview.rst, doc/guides/nics/thunderx.rst and release notes Changed "*" to "P" in overview.rst to capture the partially supported feature as "*" creating alignment issues with Sphinx table Signed-off-by: Jerin Jacob Signed-off-by: Slawomir Rosek Acked-by: John McNamara

[dpdk-dev] [PATCH v4 17/19] net/thunderx: add device start, stop and close support

2016-06-13 Thread Jerin Jacob
Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by: Zyta Szpak Signed-off-by: Slawomir Rosek Signed-off-by: Radoslaw Biernacki --- drivers/net/thunderx/nicvf_ethdev.c | 467 1 file changed, 467

[dpdk-dev] [PATCH v4 16/19] net/thunderx: add tx queue start and stop support

2016-06-13 Thread Jerin Jacob
Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by: Zyta Szpak Signed-off-by: Slawomir Rosek Signed-off-by: Radoslaw Biernacki --- drivers/net/thunderx/nicvf_ethdev.c | 59 + 1 file changed, 59

[dpdk-dev] [PATCH v4 15/19] net/thunderx: add rx queue start and stop support

2016-06-13 Thread Jerin Jacob
Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by: Zyta Szpak Signed-off-by: Slawomir Rosek Signed-off-by: Radoslaw Biernacki --- drivers/net/thunderx/nicvf_ethdev.c | 167

[dpdk-dev] [PATCH v4 14/19] net/thunderx: add dev_supported_ptypes_get and rx_queue_count support

2016-06-13 Thread Jerin Jacob
Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by: Zyta Szpak Signed-off-by: Slawomir Rosek Signed-off-by: Radoslaw Biernacki --- drivers/net/thunderx/nicvf_ethdev.c | 41 +

[dpdk-dev] [PATCH v4 13/19] net/thunderx: add single and multi segment rx functions

2016-06-13 Thread Jerin Jacob
Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by: Zyta Szpak Signed-off-by: Slawomir Rosek Signed-off-by: Radoslaw Biernacki Reviewed-by: Ferruh Yigit --- drivers/net/thunderx/nicvf_ethdev.h | 33 drivers/net/thunderx/nicvf_rxtx.c

[dpdk-dev] [PATCH v4 12/19] net/thunderx: add single and multi segment tx functions

2016-06-13 Thread Jerin Jacob
Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by: Zyta Szpak Signed-off-by: Slawomir Rosek Signed-off-by: Radoslaw Biernacki --- drivers/net/thunderx/Makefile | 2 + drivers/net/thunderx/nicvf_ethdev.c | 5 +-

[dpdk-dev] [PATCH v4 11/19] net/thunderx: add stats support

2016-06-13 Thread Jerin Jacob
Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by: Zyta Szpak Signed-off-by: Slawomir Rosek Signed-off-by: Radoslaw Biernacki Reviewed-by: Ferruh Yigit --- drivers/net/thunderx/nicvf_ethdev.c | 66 + 1

[dpdk-dev] [PATCH v4 10/19] net/thunderx: add mtu_set and promiscuous_enable support

2016-06-13 Thread Jerin Jacob
Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by: Zyta Szpak Signed-off-by: Slawomir Rosek Signed-off-by: Radoslaw Biernacki Reviewed-by: Ferruh Yigit --- drivers/net/thunderx/nicvf_ethdev.c | 51 +

[dpdk-dev] [PATCH v4 09/19] net/thunderx: add rss and reta query and update support

2016-06-13 Thread Jerin Jacob
Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by: Zyta Szpak Signed-off-by: Slawomir Rosek Signed-off-by: Radoslaw Biernacki Reviewed-by: Ferruh Yigit --- drivers/net/thunderx/nicvf_ethdev.c | 172 1

[dpdk-dev] [PATCH v4 08/19] net/thunderx: add tx_queue_setup/release support

2016-06-13 Thread Jerin Jacob
Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by: Zyta Szpak Signed-off-by: Slawomir Rosek Signed-off-by: Radoslaw Biernacki --- drivers/net/thunderx/nicvf_ethdev.c | 175 1 file changed, 175

[dpdk-dev] [PATCH v4 07/19] net/thunderx: add rx_queue_setup/release support

2016-06-13 Thread Jerin Jacob
Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by: Zyta Szpak Signed-off-by: Slawomir Rosek Signed-off-by: Radoslaw Biernacki Reviewed-by: Ferruh Yigit --- drivers/net/thunderx/nicvf_ethdev.c | 136

[dpdk-dev] [PATCH v4 06/19] net/thunderx: add dev_infos_get support

2016-06-13 Thread Jerin Jacob
Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by: Zyta Szpak Signed-off-by: Slawomir Rosek Signed-off-by: Radoslaw Biernacki Reviewed-by: Ferruh Yigit --- drivers/net/thunderx/nicvf_ethdev.c | 45 +

[dpdk-dev] [PATCH v4 05/19] net/thunderx: add dev_configure support

2016-06-13 Thread Jerin Jacob
Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by: Zyta Szpak Signed-off-by: Slawomir Rosek Signed-off-by: Radoslaw Biernacki Reviewed-by: Ferruh Yigit --- drivers/net/thunderx/nicvf_ethdev.c | 78 + 1

[dpdk-dev] [PATCH v4 04/19] net/thunderx: add get_reg and get_reg_length support

2016-06-13 Thread Jerin Jacob
Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by: Zyta Szpak Signed-off-by: Slawomir Rosek Signed-off-by: Radoslaw Biernacki --- drivers/net/thunderx/nicvf_ethdev.c | 27 +++ 1 file changed, 27 insertions(+) diff

[dpdk-dev] [PATCH v4 03/19] net/thunderx: add link status and link update support

2016-06-13 Thread Jerin Jacob
Extended the nicvf_interrupt function to respond NIC_MBOX_MSG_BGX_LINK_CHANGE mbox message from PF and update struct rte_eth_link accordingly. Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by: Zyta Szpak Signed-off-by: Slawomir Rosek

[dpdk-dev] [PATCH v4 02/19] net/thunderx: add pmd skeleton

2016-06-13 Thread Jerin Jacob
Introduce driver initialization and enable build infrastructure for nicvf pmd driver. By default, It is enabled only for defconfig_arm64-thunderx-* config as it is an inbuilt NIC device. Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by:

[dpdk-dev] [PATCH v4 01/19] net/thunderx/base: add hardware API for ThunderX nicvf inbuilt NIC

2016-06-13 Thread Jerin Jacob
Adds hardware specific API for ThunderX nicvf inbuilt NIC device under drivers/net/thunderx/nicvf/base directory. Signed-off-by: Jerin Jacob Signed-off-by: Maciej Czekaj Signed-off-by: Kamil Rytarowski Signed-off-by: Zyta Szpak Signed-off-by: Slawomir Rosek Signed-off-by: Radoslaw Biernacki

[dpdk-dev] [PATCH v4 00/19] DPDK PMD for ThunderX NIC device

2016-06-13 Thread Jerin Jacob
This patch set provides the initial version of DPDK PMD for the built-in NIC device in Cavium ThunderX SoC family. Implemented features and ThunderX nicvf PMD documentation added in doc/guides/nics/overview.rst and doc/guides/nics/thunderx.rst respectively in this patch set. These patches are

[dpdk-dev] Duplicate config symbols

2016-06-13 Thread Thomas Monjalon
2016-06-13 17:09, Christian Ehrhardt: > On Mon, Jun 13, 2016 at 3:47 PM, Thomas Monjalon 6wind.com> > wrote: > > > 2016-06-13 13:50, Christian Ehrhardt: > > > I wondered multiple times now when changing a config symbol that some of > > > them are in the .config file multiple times. > > > I

[dpdk-dev] [PATCH] port: add kni interface support

2016-06-13 Thread Ethan
Hi Cristian, I've got your comments. Thank you for review the code from a DPDK newbie. :-) I plan to submit a new patch to fix all during this week hopefully. There are four places I'd like to discuss further: 1. Dedicated lcore for kni kernel thread First of all, it is a bug to add kni kernel

[dpdk-dev] [PATCH v3 18/20] thunderx/config: set max numa node to two

2016-06-13 Thread Jerin Jacob
On Wed, Jun 08, 2016 at 06:54:43PM +0100, Ferruh Yigit wrote: > On 6/7/2016 5:40 PM, Jerin Jacob wrote: > > Signed-off-by: Jerin Jacob > > --- > > config/defconfig_arm64-thunderx-linuxapp-gcc | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git

[dpdk-dev] [PATCH] virtio: fix allocating virtnet_rx not mem aligned

2016-06-13 Thread Yuanhan Liu
On Mon, Jun 13, 2016 at 10:06:22AM +, Tan, Jianfeng wrote: > > > > -Original Message- > > From: Yuanhan Liu [mailto:yuanhan.liu at linux.intel.com] > > Sent: Monday, June 13, 2016 5:52 PM > > To: Tan, Jianfeng > > Cc: dev at dpdk.org; Xie, Huawei > > Subject: Re: [dpdk-dev] [PATCH]

[dpdk-dev] [PATCH v2 1/4] virtio-user: use virtual address in cq

2016-06-13 Thread Yuanhan Liu
On Mon, Jun 13, 2016 at 06:43:40AM +, Jianfeng Tan wrote: > Change to use virtio_net_hdr_mem instead of physical addr of memzone > when sending contrl queue commands. The virtio_net_hdr_mem has been > initialized to use virtual address under the case of virtio-user. I'd suggest to squash this

[dpdk-dev] [PATCH] virtio: fix allocating virtnet_rx not mem aligned

2016-06-13 Thread Tan, Jianfeng
On 6/13/2016 5:21 PM, Yuanhan Liu wrote: > On Sun, Jun 12, 2016 at 02:29:42PM +, Jianfeng Tan wrote: >> Compile DPDK with clang, below line in virtio_rxtx.c could be >> optimized with four "VMOVAPS ymm, m256". >>memset(>fake_mbuf, 0, sizeof(rxvq->fake_mbuf)); >> >> This instruction

[dpdk-dev] [PATCH v2 4/4] virtio-user: handle ctrl-q in driver

2016-06-13 Thread Yuanhan Liu
On Mon, Jun 13, 2016 at 06:43:43AM +, Jianfeng Tan wrote: > In virtio-user driver, when notify ctrl-queue, invoke API of > virtio-user device emulation to handle ctrl-q command. > > Besides, multi-queue requires ctrl-queue and ctrl-queue will be > enabled automatically when multi-queue is

[dpdk-dev] [PATCH v2] mbuf: new flag when Vlan is stripped

2016-06-13 Thread Olivier Matz
Hi Konstantin, On 06/13/2016 04:42 PM, Ananyev, Konstantin wrote: >> The behavior of PKT_RX_VLAN_PKT was not very well defined, resulting in >> PMDs not advertising the same flags in similar conditions. >> >> Following discussion in [1], introduce 2 new flags PKT_RX_VLAN_STRIPPED >> and

[dpdk-dev] [PATCH v3 08/20] vhost: introduce new API to export numa node

2016-06-13 Thread Yuanhan Liu
On Thu, Jun 09, 2016 at 12:45:00PM +0800, Yuanhan Liu wrote: > On Wed, Jun 08, 2016 at 02:51:33PM -0700, Rich Lane wrote: > > On Mon, Jun 6, 2016 at 8:51 PM, Yuanhan Liu > > wrote: > > > > @@ -248,14 +248,9 @@ new_device(struct virtio_net *dev) > > ? ? ? ? internal =

[dpdk-dev] [PATCH] examples/vhost: fix corrupted vdev tailq list

2016-06-13 Thread Yuanhan Liu
On Tue, Jun 07, 2016 at 11:32:56AM +0800, Yuanhan Liu wrote: > There are two tailq lists, one for logging all vhost devices, another > one for logging vhost devices distributed on a specific core. However, > there is just one tailq entry, named "next", to chain the two list, > which is wrong and

[dpdk-dev] [PATCH] virtio: fix allocating virtnet_rx not mem aligned

2016-06-13 Thread Yuanhan Liu
On Mon, Jun 13, 2016 at 05:21:01PM +0800, Yuanhan Liu wrote: > On Sun, Jun 12, 2016 at 02:29:42PM +, Jianfeng Tan wrote: > > Compile DPDK with clang, below line in virtio_rxtx.c could be > > optimized with four "VMOVAPS ymm, m256". > > memset(>fake_mbuf, 0, sizeof(rxvq->fake_mbuf)); > > > >

[dpdk-dev] dpdk and sriov nic without igb_uio

2016-06-13 Thread nirmoy das
Hi .., I am trying to find a way to use dpdk in a VM with sriov nic without using igb_uio. uio_pci_generic: failed with "No IRQ assigned to device: no support for interrupts?". vfio-pci also doesn't work on linux < 4.5 and need dpdk > 16.04. Is this a deadend for dpdk+sriov If I have linux <

[dpdk-dev] [PATCH] virito: fix reuse index in nested loop

2016-06-13 Thread Yuanhan Liu
On Mon, Jun 13, 2016 at 09:11:57AM +, Tan, Jianfeng wrote: > > > > -Original Message- > > From: Yuanhan Liu [mailto:yuanhan.liu at linux.intel.com] > > Sent: Monday, June 13, 2016 4:58 PM > > To: Tan, Jianfeng > > Cc: dev at dpdk.org; Xie, Huawei > > Subject: Re: [PATCH] virito: fix

[dpdk-dev] [PATCH 29/29] ixgbe/base: update README

2016-06-13 Thread Ferruh Yigit
On 5/6/2016 7:07 AM, Beilei Xing wrote: > update README. It can be good to give some context here, like updated base driver version and new base driver features listed... > Besides, update release notes. > > Signed-off-by: Beilei Xing > --- > doc/guides/rel_notes/release_16_07.rst | 11

[dpdk-dev] [PATCH 18/29] ixgbe/base: update swfw semaphore function

2016-06-13 Thread Ferruh Yigit
On 5/6/2016 7:07 AM, Beilei Xing wrote: > For X540 and forward it is possible if a system reset occur at the > right time to leave the SWFW semaphore high. This new function will > attempt to grab and release the semaphore. If the grab times out it > will still release the semaphore placing it in

[dpdk-dev] [PATCH 28/29] ixgbe/base: define if enable crosstalk work around

2016-06-13 Thread Ferruh Yigit
On 5/6/2016 7:07 AM, Beilei Xing wrote: > A work around for a new crosstalk erratum that causes link flap in > emtry cages has beed introduced. So this patch defines the bit in typo: emtry, beed > NVM that will tell software if this work around is needed. > > Signed-off-by: Beilei Xing > ---

[dpdk-dev] [PATCH 21/29] ixgbe/base: smplify add/remove VLANs

2016-06-13 Thread Ferruh Yigit
On 5/6/2016 7:07 AM, Beilei Xing wrote: > This patch simplifies the adding and removing VLANs from > VFTA/VLVF/VLVFB registers. The logic to determine registers has > been simplified to (vid / 32) and (1 - vid / 32). Many conditional > paths and checks are no longer needed with this patch. > >

[dpdk-dev] [PATCH 18/29] ixgbe/base: update swfw semaphore function

2016-06-13 Thread Ferruh Yigit
On 5/6/2016 7:07 AM, Beilei Xing wrote: > For X540 and forward it is possible if a system reset occur at the > right time to leave the SWFW semaphore high. This new function will > attempt to grab and release the semaphore. If the grab times out it > will still release the semaphore placing it in

[dpdk-dev] [PATCH] virtio: fix allocating virtnet_rx not mem aligned

2016-06-13 Thread Yuanhan Liu
On Sun, Jun 12, 2016 at 02:29:42PM +, Jianfeng Tan wrote: > Compile DPDK with clang, below line in virtio_rxtx.c could be > optimized with four "VMOVAPS ymm, m256". > memset(>fake_mbuf, 0, sizeof(rxvq->fake_mbuf)); > > This instruction requires memory address is 32-byte aligned. > Or, it

[dpdk-dev] [RFC] mk: filter duplicate configuration entries

2016-06-13 Thread Christian Ehrhardt
Due to the hierarchy and the demand to keep the base config shoing all options some options end up multiple times in the .config file. A suggested solution was to filter for duplicates at the end of the actual config step which is implemented here. Signed-off-by: Christian Ehrhardt ---

[dpdk-dev] Duplicate config symbols

2016-06-13 Thread Christian Ehrhardt
On Mon, Jun 13, 2016 at 3:47 PM, Thomas Monjalon wrote: > 2016-06-13 13:50, Christian Ehrhardt: > > I wondered multiple times now when changing a config symbol that some of > > them are in the .config file multiple times. > > I totally feel like I'm overlooking something, but still it might be >

[dpdk-dev] [PATCH v5 10/10] app/test: do not dump PCI devices in blacklist test

2016-06-13 Thread Jan Viktorin
Dumping of devices in a unittest is useless. Instead, test whether the test has been set up well - i.e. there are no devices. Signed-off-by: Jan Viktorin --- app/test/test_pci.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/test/test_pci.c b/app/test/test_pci.c index

[dpdk-dev] [PATCH v5 09/10] eal/pci: allow to override sysfs

2016-06-13 Thread Jan Viktorin
The SYSFS_PCI_DEVICES is a constant that makes the PCI testing difficult as it points to an absolute path. We remove using this constant and introducing a function pci_get_sysfs_path that gives the same value. However, the user can pass a SYSFS_PCI_DEVICES env variable to override the path. It is

[dpdk-dev] [PATCH v5 08/10] app/test: convert current pci_test into a single test case

2016-06-13 Thread Jan Viktorin
The current test_pci is just a single test case that tests the blacklisting of devices. Rename it to test_pci_blacklist and call it from the test_pci. The setup and cleanup are moved out of the test_pci_blacklist entirely to cover all other tests. Signed-off-by: Jan Viktorin --- v4 * the

[dpdk-dev] [PATCH v5 07/10] app/test: extract test_pci_setup and test_pci_cleanup

2016-06-13 Thread Jan Viktorin
Signed-off-by: Jan Viktorin --- app/test/test_pci.c | 47 ++- 1 file changed, 34 insertions(+), 13 deletions(-) diff --git a/app/test/test_pci.c b/app/test/test_pci.c index 8b7c8bb..50078a0 100644 --- a/app/test/test_pci.c +++ b/app/test/test_pci.c @@

[dpdk-dev] [PATCH v5 06/10] app/test: use linked list to store PCI drivers

2016-06-13 Thread Jan Viktorin
The test unregisters all drivers before start. The drivers were stored into a fixed-sized array. This is inflexible. This patch change this to utilize a linked list for the same purpose. Signed-off-by: Jan Viktorin --- v3: * fixed commit message * used "backup" to describe the

[dpdk-dev] [PATCH v5 05/10] app/test: support resources archived by tar

2016-06-13 Thread Jan Viktorin
When a more complex resource (a file hierarchy) is needed, packing every single file as a single resource would be very ineffective. For that purpose, it is possible to pack the files into a tar archive, extract it before test from the resource and finally clean up all the created files. This

[dpdk-dev] [PATCH v5 04/10] app/test: add functions to create files from resources

2016-06-13 Thread Jan Viktorin
A resource can be written into the target filesystem by calling resource_fwrite or resource_fwrite_file. Such file can be created before a test is started and removed after the test finishes. Signed-off-by: Jan Viktorin --- app/test/resource.c | 35 +++

[dpdk-dev] [PATCH v5 03/10] app/test: support resources externally linked

2016-06-13 Thread Jan Viktorin
To include resources from other source that the C source code we can take advantage of the objcopy behaviour, i.e. packing of an arbitrary file as an object file that is linked to the target program. A linked object file is always accessible as a pair extern const char beg_; extern const char

[dpdk-dev] [PATCH v5 02/10] mk: define objcopy-specific target and arch

2016-06-13 Thread Jan Viktorin
The program objcopy uses non-standard conventions to name the target and arch. Define the values for supported architecturesi (tile and ppc_64 are missing). Signed-off-by: Jan Viktorin --- v4: * fixed commit message (removed FIXME) --- mk/arch/arm/rte.vars.mk | 5 +

[dpdk-dev] [PATCH v5 01/10] app/test: introduce resources for tests

2016-06-13 Thread Jan Viktorin
Certain internal mechanisms of DPDK access different file system structures (e.g. /sys/bus/pci/devices). It is difficult to test those cases automatically by a unit test when such path is not hard-coded and there is no simple way how to distribute fake ones with the current testing environment.

[dpdk-dev] [PATCH v5 00/10] Include resources in tests

2016-06-13 Thread Jan Viktorin
Hello, the fifth version fixing the libarchive API usage. Regards Jan --- v1: * included 5 patches improving the PCI tests * fixed using of non-existing RTE_INIT macro v2: * Makefile macro resource renamed to linked_resource * introduced macro linked_tar_resource * added more comments *

[dpdk-dev] [PATCH 0/7] Miscellaneous fixes for mlx4 and mlx5

2016-06-13 Thread Bruce Richardson
On Wed, Jun 08, 2016 at 11:43:24AM +0200, Nelio Laranjeiro wrote: > Various minor fixes for mlx4 (ConnectX-3) and mlx5 (ConnectX-4). > > Adrien Mazarguil (4): > mlx: ensure MTU update is effective > mlx: retrieve mbuf size through proper API > mlx5: fix RX VLAN stripping capability check >

[dpdk-dev] [PATCH] virito: fix reuse index in nested loop

2016-06-13 Thread Yuanhan Liu
On Sun, Jun 12, 2016 at 10:05:45AM +, Jianfeng Tan wrote: > This patches fixes problem of reusing index of outmost loop in nested > loops. This bug will lead to failure when starting a multi queue > virtio device: rx queues (except from the first one) cannot be started, > expecially their

[dpdk-dev] [PATCH v4 8/8] doc: update xstats documentation

2016-06-13 Thread Remy Horton
Signed-off-by: Remy Horton --- doc/guides/prog_guide/poll_mode_drv.rst | 25 + 1 file changed, 21 insertions(+), 4 deletions(-) diff --git a/doc/guides/prog_guide/poll_mode_drv.rst b/doc/guides/prog_guide/poll_mode_drv.rst index 7698692..802fb8f 100644 ---

[dpdk-dev] [PATCH v4 7/8] rte: change xstats usage to new API

2016-06-13 Thread Remy Horton
The current extended ethernet statistics fetching involve doing several string operations, which causes performance issues if there are lots of statistics and/or network interfaces. This patch changes the test-pmd and proc_info applications to use the new xstats API, and removes deprecated code

[dpdk-dev] [PATCH v4 6/8] drivers/net/virtio: change xstats to use integer ids

2016-06-13 Thread Remy Horton
The current extended ethernet statistics fetching involve doing several string operations, which causes performance issues if there are lots of statistics and/or network interfaces. This patch changes the virtio driver to use the new API that seperates name string and value queries.

[dpdk-dev] [PATCH v4 5/8] drivers/net/i40e: change xstats to use integer ids

2016-06-13 Thread Remy Horton
The current extended ethernet statistics fetching involve doing several string operations, which causes performance issues if there are lots of statistics and/or network interfaces. This patch changes the i40e driver to use the new API that seperates name string and value queries. Signed-off-by:

[dpdk-dev] [PATCH v4 4/8] drivers/net/fm10k: change xstats to use integer ids

2016-06-13 Thread Remy Horton
The current extended ethernet statistics fetching involve doing several string operations, which causes performance issues if there are lots of statistics and/or network interfaces. This patch changes the fm10k driver to use the new API that seperates name string and value queries. Signed-off-by:

[dpdk-dev] [PATCH v4 3/8] drivers/net/e1000: change xstats to use integer ids

2016-06-13 Thread Remy Horton
The current extended ethernet statistics fetching involve doing several string operations, which causes performance issues if there are lots of statistics and/or network interfaces. This patch changes the e1000 driver to use the new API that seperates name string and value queries. Signed-off-by:

[dpdk-dev] [PATCH v4 2/8] drivers/net/ixgbe: change xstats to use integer ids

2016-06-13 Thread Remy Horton
The current extended ethernet statistics fetching involve doing several string operations, which causes performance issues if there are lots of statistics and/or network interfaces. This patch changes the ixgbe driver to use the new API that seperates name string and value queries. Signed-off-by:

[dpdk-dev] [PATCH v4 1/8] rte: change xstats to use integer ids

2016-06-13 Thread Remy Horton
Signed-off-by: Remy Horton --- lib/librte_ether/rte_ethdev.c | 94 +++--- lib/librte_ether/rte_ethdev.h | 35 + lib/librte_ether/rte_ether_version.map | 7 +++ 3 files changed, 128 insertions(+), 8 deletions(-) diff --git

[dpdk-dev] [PATCH v4 0/8] Remove string operations from xstats

2016-06-13 Thread Remy Horton
The current extended ethernet statistics fetching involve doing several string operations, which causes performance issues if there are lots of statistics and/or network interfaces. This patchset changes the API for xstats to use integer identifiers instead of strings and implements this new API

[dpdk-dev] [PATCH v3 1/2] ethdev: add callback to get register size in bytes

2016-06-13 Thread Remy Horton
On 12/06/2016 15:51, Zyta Szpak wrote: > I would prefer having only one function rte_eth_dev_get_regs() > which returns length and width if data is NULL. > The first call is a parameter request before buffer allocation, > and the second call fills the buffer. > > We can

[dpdk-dev] dpdk and sriov nic without igb_uio

2016-06-13 Thread Bruce Richardson
On Mon, Jun 13, 2016 at 05:40:39PM +0200, nirmoy das wrote: > Hi .., > > I am trying to find a way to use dpdk in a VM with sriov nic > without using igb_uio. > > uio_pci_generic: failed with "No IRQ assigned to device: no support for > interrupts?". > vfio-pci also doesn't work on linux < 4.5

[dpdk-dev] [PATCH v4 00/19] DPDK PMD for ThunderX NIC device

2016-06-13 Thread Bruce Richardson
On Mon, Jun 13, 2016 at 07:25:24PM +0530, Jerin Jacob wrote: > This patch set provides the initial version of DPDK PMD for the > built-in NIC device in Cavium ThunderX SoC family. > > Implemented features and ThunderX nicvf PMD documentation added > in doc/guides/nics/overview.rst and

[dpdk-dev] [PATCH v4 05/10] app/test: support resources archived by tar

2016-06-13 Thread Jan Viktorin
On Mon, 13 Jun 2016 16:40:09 +0200 Thomas Monjalon wrote: > 2016-06-13 10:12, Jan Viktorin: > > +static int do_copy(struct archive *r, struct archive *w) > > +{ > > + const void *buf; > > + size_t len; > > + off_t off; > > + int ret; > > + > > + while (1) { > > + ret =

[dpdk-dev] [PATCH v4 05/10] app/test: support resources archived by tar

2016-06-13 Thread Thomas Monjalon
2016-06-13 10:12, Jan Viktorin: > +static int do_copy(struct archive *r, struct archive *w) > +{ > + const void *buf; > + size_t len; > + off_t off; > + int ret; > + > + while (1) { > + ret = archive_read_data_block(r, , , ); There is an error in 32-bit

[dpdk-dev] [PATCH v2] mbuf: new flag when Vlan is stripped

2016-06-13 Thread Ananyev, Konstantin
Hi Olivier, > > Hi Konstantin, > > On 06/13/2016 04:42 PM, Ananyev, Konstantin wrote: > >> The behavior of PKT_RX_VLAN_PKT was not very well defined, resulting in > >> PMDs not advertising the same flags in similar conditions. > >> > >> Following discussion in [1], introduce 2 new flags

[dpdk-dev] [PATCH v1 19/28] eal/soc: add drv_flags

2016-06-13 Thread Jan Viktorin
On Mon, 13 Jun 2016 14:21:40 + Shreyansh Jain wrote: > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jan Viktorin > > Sent: Friday, May 06, 2016 7:18 PM > > To: dev at dpdk.org > > Cc: Jan Viktorin ; David Marchand > > ; Thomas Monjalon >

[dpdk-dev] [PATCH v1 07/28] eal/soc: add rte_eal_soc_register/unregister logic

2016-06-13 Thread Jan Viktorin
On Mon, 13 Jun 2016 14:19:39 + Shreyansh Jain wrote: > Another trivial comment inlined: > > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jan Viktorin > > Sent: Friday, May 06, 2016 7:18 PM > > To: dev at dpdk.org > > Cc: Jan Viktorin ; David

[dpdk-dev] [PATCH v1 02/28] eal: extract function eal_parse_sysfs_valuef

2016-06-13 Thread Jan Viktorin
On Mon, 13 Jun 2016 14:18:40 + Shreyansh Jain wrote: > Hi Jan, > > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jan Viktorin > > Sent: Friday, May 06, 2016 7:18 PM > > To: dev at dpdk.org > > Cc: Jan Viktorin ; David Marchand > > ; Thomas Monjalon

[dpdk-dev] [PATCH v2 0/2] mk: fix more library underlinking

2016-06-13 Thread Thomas Monjalon
2016-06-13 13:06, Christian Ehrhardt: > After several cleanups libraries are now linked against the libs they > refer as DEPDIR which is great. > But that now also made it visible that some references where still missing. > > The following two patches try to fix that: > [PATCH v2 1/2] mk: fix

[dpdk-dev] [PATCH v2 0/6] reduce overlinking in applications

2016-06-13 Thread Thomas Monjalon
2016-06-10 15:19, Thomas Monjalon: > This is a respin of the ideas of Christian and Ferruh to limit > the static application size or dynamic links to shared libraries. > > It also brings some clean-up in rte.app.mk. > > Ferruh Yigit (2): > mk: prevent overlinking in applications > mk: reduce

[dpdk-dev] [PATCH v4 01/19] net/thunderx/base: add hardware API for ThunderX nicvf inbuilt NIC

2016-06-13 Thread Bruce Richardson
On Mon, Jun 13, 2016 at 07:25:25PM +0530, Jerin Jacob wrote: > Adds hardware specific API for ThunderX nicvf inbuilt NIC device under > drivers/net/thunderx/nicvf/base directory. > Hi Jerin, we are trying to move away from huge drops of shared code in a single patchfile, so as to make the

[dpdk-dev] [PATCH v3 4/9] app/testpmd: remove fwd_config_setup from fwd_config_display

2016-06-13 Thread Iremonger, Bernard
Hi Pablo, > > Subject: [PATCH v3 4/9] app/testpmd: remove fwd_config_setup from > > fwd_config_display > > > > Remove fwd_config_setup from fwd_config_display and check that > > forwarding has been setup before displaying forwarding configuration. > > Add call to fwd_config_setup for corelist,

[dpdk-dev] Performance hit - NICs on different CPU sockets

2016-06-13 Thread Take Ceara
Hi, I'm reposting here as I didn't get any answers on the dpdk-users mailing list. We're working on a stateful traffic generator (www.warp17.net) using DPDK and we would like to control two XL710 NICs (one on each socket) to maximize CPU usage. It looks that we run into the following limitation:

[dpdk-dev] [PATCH v10 3/3] i40e: add floating VEB extension support

2016-06-13 Thread Zhe Tao
To enable this feature, the user should pass a devargs parameter to the EAL like "-w 84:00.0,enable_floatingVEB=1", and the application will make sure the PMD will use the floating VEB feature for all the VFs created by this PF device. Also you can specifiy which VF need to connect to this

[dpdk-dev] [PATCH v10 2/3] i40e: add floating VEB support in i40e

2016-06-13 Thread Zhe Tao
This patch add the support for floating VEB in i40e. All the VFs VSIs can decide whether to connect to the legacy VEB/VEPA or the floating VEB. When connect to the floating VEB a new floating VEB is created. Now all the VFs need to connect to floating VEB or legacy VEB, cannot connect to both of

[dpdk-dev] [PATCH v10 1/3] i40e: support floating VEB config

2016-06-13 Thread Zhe Tao
Add the new floating related argument option in the devarg. Using this parameter, all the samples can decide whether to use legacy VEB/VEPA or floating VEB. To enable this feature, the user should pass a devargs parameter to the EAL like "-w 84:00.0,enable_floatingVEB=1", and the application will

[dpdk-dev] [PATCH v10 0/3] i40e: add floating VEB support for i40e

2016-06-13 Thread Zhe Tao
This patch-set add the support for floating VEB in i40e. All the VFs VSIs can decide whether to connect to the legacy VEB/VEPA or the floating VEB. When connect to the floating VEB a new floating VEB is created. Now all the VFs need to connect to floating VEB or legacy VEB, cannot connect to both

[dpdk-dev] [PATCH v3 4/9] app/testpmd: remove fwd_config_setup from fwd_config_display

2016-06-13 Thread De Lara Guarch, Pablo
Hi Bernard, > -Original Message- > From: Iremonger, Bernard > Sent: Sunday, June 12, 2016 4:23 PM > To: dev at dpdk.org > Cc: De Lara Guarch, Pablo; Iremonger, Bernard > Subject: [PATCH v3 4/9] app/testpmd: remove fwd_config_setup from > fwd_config_display > > Remove fwd_config_setup

[dpdk-dev] [PATCH] mbuf: remove unused Rx error flags

2016-06-13 Thread Thomas Monjalon
> > > Just a note, I think you'll need to rebase your patch with latest code. > > > enic PMD fails to compile. > > > > Indeed, this patch should be applied after John's enic series. Latest > > one is there: > > http://dpdk.org/ml/archives/dev/2016-June/040183.html > > > > Please Bruce/Thomas,

[dpdk-dev] Duplicate config symbols

2016-06-13 Thread Thomas Monjalon
2016-06-13 13:50, Christian Ehrhardt: > I wondered multiple times now when changing a config symbol that some of > them are in the .config file multiple times. > I totally feel like I'm overlooking something, but still it might be worth > to ask. [...] > Is there any reason to do so or is this an

[dpdk-dev] [PATCH 22/29] ixgbe/base: add bypassing VLVF

2016-06-13 Thread Ferruh Yigit
On 5/6/2016 7:07 AM, Beilei Xing wrote: > This patch adds support for the VLVF to be bypassed when adding or > removing a VFTA entry. The PF can utilize the default pool while > preserving the VLVF for the VFs use. > Meanwhile, update corresponding VF ops and drivers where corresponding > ops is

[dpdk-dev] [PATCH] ethdev: clarify the origin of mbufs

2016-06-13 Thread Olivier Matz
Following the discussion on dpdk-users [1], enhance the API documentation of rte_eth_tx_burst() to specify that the mbufs have to be allocated from a pool. [1] http://dpdk.org/ml/archives/users/2016-June/000618.html Reported-by: Wu Xiaoban Signed-off-by: Olivier Matz ---

[dpdk-dev] Performance hit - NICs on different CPU sockets

2016-06-13 Thread Bruce Richardson
On Mon, Jun 13, 2016 at 04:07:37PM +0200, Take Ceara wrote: > Hi, > > I'm reposting here as I didn't get any answers on the dpdk-users mailing list. > > We're working on a stateful traffic generator (www.warp17.net) using > DPDK and we would like to control two XL710 NICs (one on each socket) >

[dpdk-dev] [PATCH v4 1/2] enic: fix seg fault when releasing queues

2016-06-13 Thread Bruce Richardson
On Sat, Jun 11, 2016 at 10:27:04AM -0700, John Daley wrote: > If device configuration failed due to a lack of resources, such as > if more queues are requested than are available, the queue release > functions are called with NULL pointers which were being dereferenced. > > Skip releasing queues

[dpdk-dev] [PATCH v4 4/4] app/test: add crypto AES-CBC-128 HMAC-SHA224 and HMAC-SHA384 unit tests

2016-06-13 Thread Fan Zhang
This patch adds the HMAC-SHA224 and HMAC-SHA384 digest generation and verification tests to crypto Signed-off-by: Fan Zhang --- app/test/test_cryptodev_aes.c | 26 app/test/test_cryptodev_aes.h | 296 ++ 2 files changed, 322 insertions(+) diff

[dpdk-dev] [PATCH v4 3/4] app/test: utilize new unified crypto AES test function

2016-06-13 Thread Fan Zhang
This patch replaces the AES test code with new unified crypto AES test function. Signed-off-by: Fan Zhang --- app/test/test_cryptodev.c | 1613 ++-- app/test/test_cryptodev_aes_ctr_test_vectors.h | 257 2 files changed, 122 insertions(+), 1748

[dpdk-dev] [PATCH v4 2/4] app/test: add unified crypto aes test

2016-06-13 Thread Fan Zhang
This patch adds a new crypto AES unified test function. Signed-off-by: Fan Zhang --- app/test/Makefile | 1 + app/test/test_cryptodev_aes.c | 663 ++ app/test/test_cryptodev_aes.h | 6 + 3 files changed, 670 insertions(+) create mode

  1   2   3   >