[dpdk-dev] [PATCH] vhost: fix segfault on bad descriptor address.

2016-07-10 Thread Yuanhan Liu
On Fri, Jul 08, 2016 at 02:48:56PM +0300, Ilya Maximets wrote: > > Another point is that crash constantly happens on queue_id=3 (second RX > queue) in > my scenario. It is newly allocated virtqueue while reconfiguration from rxq=1 > to > rxq=2. That's a valuable message: what's your DPDK HEAD

[dpdk-dev] igb_uio: BUG: unable to handle kernel NULL pointer dereference at 0000000000000200

2016-07-10 Thread Vinay Y S
Using dpdk-16.04 on Intel 82599 NIC on Ubuntu 16.04. Running testpmd interactive bound to two vf interfaces from each of the two ports on the 82599 nic. While testpmd was running I decided to bind to one of the pf instead of vf. So I did a ifdown on second PF (eno2). Then I bound it to igb_uio

[dpdk-dev] [PATCH v3 00/16] Make VFIO support less dependent on PCI

2016-07-10 Thread Thomas Monjalon
> Jan Viktorin (16): > vfio: fix include of eal_private.h to be local > vfio: move VFIO-specific stuff to eal_vfio.h > vfio: move common vfio constants to eal_vfio.h > vfio: move vfio_iommu_type and dma_map functions to eal_vfio > vfio: generalize pci_vfio_set_iommu_type > vfio:

[dpdk-dev] [PATCH] ixgbe: support checksum flags in sse vector Rx function

2016-07-10 Thread Olivier Matz
Hi Bruce, On 07/08/2016 10:42 PM, Bruce Richardson wrote: > On Thu, Jul 07, 2016 at 02:19:02PM +0200, Olivier Matz wrote: >> Update desc_to_olflags_v() to set PKT_RX_IP_CKSUM_BAD and >> PKT_RX_L4_CKSUM_BAD in the ol_fags of the mbuf. >> {...} > > Given this looks a significant change to a

[dpdk-dev] [PATCH v3 06/16] vfio: generalize pci_vfio_has_supported_extensions

2016-07-10 Thread Thomas Monjalon
There is a small error in this patch: 2016-07-04 17:16, Jan Viktorin: > The pci_vfio_has_supported_extensions is not PCI-specific and it is a private > function of the eal_pci_vfio.c. We just rename the function and make it > available even for non-PCI devices. |...] |> ---

[dpdk-dev] [PATCH v2] app/test: fix hexdump length of cipher/plaintexts

2016-07-10 Thread Deepak Kumar Jain
From: Pablo de Lara Plaintexts and ciphertexts are dumped when debugging is enabled, using TEST_HEXDUMP. For Snow3G and KASUMI, their lengths are in bits, but TEST_HEXDUMP uses bytes, so lenghts are passed in bytes now. Fixes: 47df73a1a62f ("app/test: use hexdump

[dpdk-dev] [PATCH 0/3] more commits checks

2016-07-10 Thread Thomas Monjalon
2016-07-06 12:38, Thomas Monjalon: > The script check-git-log.sh helps to keep a consistent git history. > Scrolling/parsing the history is easier when we are used to some > rules. > > N?lio Laranjeiro (1): > scripts: check headline of drivers commits > > Thomas Monjalon (2): > scripts:

[dpdk-dev] [PATCH] scripts: fix commit check for empty list

2016-07-10 Thread Thomas Monjalon
2016-07-08 19:50, Thomas Monjalon: > When running check-git-log.sh on a clean tree it was complaining > of a wrong empty headline because '^[^:]*$' was matching. > It is fixed by matching at least one character with +. > > Signed-off-by: Thomas Monjalon Applied

[dpdk-dev] [PATCH] scripts: fix commit check of first word

2016-07-10 Thread Thomas Monjalon
> > The first word of each commit message is checked. > > But when the commit range was greater than 1, only the latest commit > > was checked. > > It is fixed by checking each commit separately. > > > > Fixes: 9c24780f0d5e ("scripts: check first word of commit messages") > > > > Signed-off-by:

[dpdk-dev] [PATCH 0/3] fix dependencies on librte_hash

2016-07-10 Thread Thomas Monjalon
2016-07-06 16:44, Thomas Monjalon: > Thomas Monjalon (3): > net/bnx2x: remove unneeded dependency on hash library > table: remove unneeded dependency on hash library > examples: fix dependencies on hash library Applied

[dpdk-dev] [PATCH] net/enic: remove useless assert macro

2016-07-10 Thread Thomas Monjalon
> > The macro ENIC_ASSERT does the same thing as RTE_ASSERT, thus it can be > > removed. > > > > Signed-off-by: Thomas Monjalon > Acked-by: John Daley Applied

[dpdk-dev] [PATCH] app/test: remove useless clock estimation

2016-07-10 Thread Thomas Monjalon
2016-07-06 17:22, Thomas Monjalon: > The function get_machclk_freq() in RED test can be replaced by > rte_get_timer_hz() which already estimates the clock frequency at EAL init. > > Signed-off-by: Thomas Monjalon Applied

[dpdk-dev] [PATCH v4] igb_uio: fix possible mmap failure for Linux >= v4.5

2016-07-10 Thread Thomas Monjalon
2016-07-05 16:00, Ferruh Yigit: > mmap the iomem range of the PCI device fails for kernels that > enabled CONFIG_IO_STRICT_DEVMEM option: > > EAL: pci_map_resource(): > cannot mmap(39, 0x7f1c5180, 0x10, 0x0): > Invalid argument (0x) > >

[dpdk-dev] [PATCH 2/2] malloc: no need to zero out memory on zmalloc

2016-07-10 Thread Thomas Monjalon
2016-07-05 12:01, Sergio Gonzalez Monroy: > Zeroing out memory on rte_zmalloc_socket is not required anymore since all > allocated memory is already zeroed. > > Signed-off-by: Sergio Gonzalez Monroy Series applied, thanks This patch will need special attention when validating this release.

[dpdk-dev] [PATCH v4] eal/linuxapp: fix resource leak

2016-07-10 Thread Thomas Monjalon
2016-07-06 11:48, Sergio Gonzalez Monroy: > On 06/07/2016 11:44, Daniel Mrzyglod wrote: > > Current code does not munmap 'hugepage' mapping (hugepage info file) on > > function exit, leaking resources. > > > > Coverity issue: 97920 > > Fixes: b6a468ad41d5 ("memory: add --socket-mem option") > > >

[dpdk-dev] [PATCH] pdump: close client socket

2016-07-10 Thread Thomas Monjalon
> > Close the client socket before returning on error. > > > > Coverity issue: 127555 > > > > Fixes: f3c1829130ac ("pdump: check missing home environment variable") > > > > Signed-off-by: Reshma Pattan > > Acked-by: John McNamara Applied, thanks

[dpdk-dev] [PATCH] app/test: fix hexdump length of cipher/plaintexts

2016-07-10 Thread Thomas Monjalon
2016-07-09 16:05, Pablo de Lara: > Plaintexts and ciphertexts are dumped when debugging is enabled, > using TEST_HEXDUMP. For Snow3G and KASUMI, their lengths are in bits, > but TEST_HEXDUMP uses bytes, so lenghts are passed in bytes now. There is an error:

[dpdk-dev] [PATCH] app/test: fix missing operation initialization

2016-07-10 Thread Thomas Monjalon
2016-07-10 11:20, Deepak Kumar Jain: > Initializing the authentication op parameter. > > Fixes: eec136f3c54f ("aesni_gcm: add driver for AES-GCM crypto operations") > > Signed-off-by: Deepak Kumar Jain Applied, thanks

[dpdk-dev] [PATCH] qat: fix for digest verification use case

2016-07-10 Thread Thomas Monjalon
2016-07-10 11:21, Deepak Kumar Jain: > This fixes the cases in which operation was Digest verify. > > Fixes: e25200fbb45df ("crypto: add cipher/auth only support") > > Signed-off-by: Deepak Kumar Jain Applied, thanks

[dpdk-dev] [PATCH] kasumi: restrict cipher bit-level ops

2016-07-10 Thread Thomas Monjalon
2016-07-09 17:35, Pablo de Lara: > KASUMI PMD only supports bit-level cipher operations > when destination buffer is different from the source > (out of place operations). This commit adds a check > in the code to prevent the user from trying to perform > in-place bit-level ciphering. > > Fixes:

[dpdk-dev] [PATCH] net/enic: fix crash when changing number of Rx or Tx queues

2016-07-10 Thread John Daley
The check that all Tx and Rx queues were set up was not adequate when reconfiguring with a different number of queues. Only the number of completion queues (CQs) was being used to make the determination, but the CQ array is shared between the underlying Rx and Tx queues. Check that the internal

[dpdk-dev] [PATCH v2 6/6] l3fwd: add altivec support for em_hash_key

2016-07-10 Thread Gowrishankar
From: gowrishankar This patch adds ppc64le port for em_mask_key function. Signed-off-by: Gowrishankar --- examples/l3fwd/l3fwd_em.c | 8 1 file changed, 8 insertions(+) diff --git a/examples/l3fwd/l3fwd_em.c b/examples/l3fwd/l3fwd_em.c index

[dpdk-dev] [PATCH v2 5/6] sched: enable sched library for ppc64le

2016-07-10 Thread Gowrishankar
From: gowrishankar This patch enables librte_sched in ppc64le. Signed-off-by: Gowrishankar --- config/defconfig_ppc_64-power8-linuxapp-gcc | 2 -- 1 file changed, 2 deletions(-) diff --git a/config/defconfig_ppc_64-power8-linuxapp-gcc

[dpdk-dev] [PATCH v2 4/6] table: cache align rte_bucket_4_8

2016-07-10 Thread Gowrishankar
From: gowrishankar Align rte_bucket_4_8 for cache line. Signed-off-by: Gowrishankar --- lib/librte_table/rte_table_hash_key8.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/librte_table/rte_table_hash_key8.c

[dpdk-dev] [PATCH v2 3/6] ip_pipeline: fix lcore mapping for varying SMT threads as in ppc64

2016-07-10 Thread Gowrishankar
From: gowrishankar offline lcore would still refer to original core id and this has to be considered while creating cpu core mask. Signed-off-by: Gowrishankar --- config/defconfig_ppc_64-power8-linuxapp-gcc | 3 --- examples/ip_pipeline/cpu_core_map.c

[dpdk-dev] [PATCH v2 2/6] acl: add altivec intrinsics for dpdk acl on ppc_64

2016-07-10 Thread Gowrishankar
From: gowrishankar This patch adds port for ACL library in ppc64le. Signed-off-by: Gowrishankar --- app/test-acl/main.c | 4 + config/defconfig_ppc_64-power8-linuxapp-gcc | 1 - lib/librte_acl/Makefile | 2 +

[dpdk-dev] [PATCH v2 1/6] lpm: add altivec intrinsics for dpdk lpm on ppc_64

2016-07-10 Thread Gowrishankar
From: gowrishankar This patch adds ppc64le port for LPM library in DPDK. Signed-off-by: Gowrishankar --- app/test/test_xmmt_ops.h | 16 +++ config/defconfig_ppc_64-power8-linuxapp-gcc| 1 -

[dpdk-dev] [PATCH v2 0/6] enable lpm, acl and other missing libraries in ppc64le

2016-07-10 Thread Gowrishankar
From: gowrishankar This patchset enables LPM, ACL and other few missing libs in ppc64le and also address few patches in related examples (ip_pipeline and l3fwd). Test report: LPM and ACL unit tests verified as in patch set v1. Same results as before observed.

[dpdk-dev] [PATCH] qat: fix for digest verification use case

2016-07-10 Thread Deepak Kumar Jain
This fixes the cases in which operation was Digest verify. Fixes: e25200fbb45df ("crypto: add cipher/auth only support") Signed-off-by: Deepak Kumar Jain --- drivers/crypto/qat/qat_adf/qat_algs.h| 3 ++- drivers/crypto/qat/qat_adf/qat_algs_build_desc.c | 10 +-

[dpdk-dev] [PATCH] app/test: fix missing operation initialization

2016-07-10 Thread Deepak Kumar Jain
Initializing the authentication op parameter. Fixes: eec136f3c54f ("aesni_gcm: add driver for AES-GCM crypto operations") Signed-off-by: Deepak Kumar Jain --- app/test/test_cryptodev.c | 1 + 1 file changed, 1 insertion(+) diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c

[dpdk-dev] [PATCH 0/2] enable pmdinfo support on FreeBSD

2016-07-10 Thread Thomas Monjalon
2016-07-08 21:37, Bruce Richardson: > This patchset fixes pmdinfogen and pmdinfo.py so that they can work on > FreeBSD. It's been tested with static binaries compiled using clang and gcc > on FreeBSD, as well as gcc-generated dynamic libraries. For some reason > clang wasn't able to build dynamic

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

2016-07-10 Thread Thomas Monjalon
2016-07-09 09:45, Neil Horman: > On Sat, Jul 09, 2016 at 02:06:09AM +0100, Pablo de Lara wrote: > > This patchset makes some extra changes after the modification > > of the PMD_REGISTER_DRIVER macro, including renaming back > > some virtual device driver names (virtio could not be renamed) > > and

[dpdk-dev] [PATCH v1] ethdev: fix xstats id mismatch

2016-07-10 Thread Thomas Monjalon
2016-07-08 16:44, Remy Horton: > When fetching xstats values the driver specific parameters are > placed after the generic ones, but when fetching xstats names > the driver specific parameter names came first. This patch fixes > the resulting id mismatch between names and values. > > Fixes: