[dpdk-dev] [PATCH v2] app/test: add AES GCM performance test to cryptodev

2016-09-26 Thread De Lara Guarch, Pablo
Hi Arek, > -Original Message- > From: Jain, Deepak K > Sent: Monday, September 26, 2016 12:43 PM > To: Kusztal, ArkadiuszX; dev at dpdk.org > Cc: Trahe, Fiona; De Lara Guarch, Pablo; Griffin, John > Subject: RE: [PATCH v2] app/test: add AES GCM performance test to > cryptodev > > > > >

[dpdk-dev] [PATCH] app/test: remove unnecessary conditional

2016-09-26 Thread Pablo de Lara
Regardless the result of the conditional, the true and false statements were the same, so the conditional can be removed. Signed-off-by: Pablo de Lara --- app/test/test_cryptodev.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/app/test/test_cryptodev.c b/app/test/test_c

[dpdk-dev] [PATCH v2 7/7] net/vhost: add an option to enable dequeue zero copy

2016-09-26 Thread Maxime Coquelin
On 09/23/2016 06:13 AM, Yuanhan Liu wrote: > Add an option, dequeue-zero-copy, to enable this feature in vhost-pmd. > > Signed-off-by: Yuanhan Liu > --- > drivers/net/vhost/rte_eth_vhost.c | 13 + > 1 file changed, 13 insertions(+) Reviewed-by: Maxime Coquelin Thanks, Maxime

[dpdk-dev] [PATCH v2 6/7] examples/vhost: add an option to enable dequeue zero copy

2016-09-26 Thread Maxime Coquelin
On 09/23/2016 06:13 AM, Yuanhan Liu wrote: > Add an option, --dequeue-zero-copy, to enable dequeue zero copy. > > One thing worth noting while using dequeue zero copy is the nb_tx_desc > has to be small enough so that the eth driver will hit the mbuf free > threshold easily and thus free mbuf mor

[dpdk-dev] [PATCH v2 5/7] vhost: add a flag to enable dequeue zero copy

2016-09-26 Thread Maxime Coquelin
On 09/23/2016 06:13 AM, Yuanhan Liu wrote: > Dequeue zero copy is disabled by default. Here add a new flag > ``RTE_VHOST_USER_DEQUEUE_ZERO_COPY`` to explictily enable it. > > Signed-off-by: Yuanhan Liu > --- > > v2: - update release log > - doc dequeue zero copy in detail > --- > doc/guides

[dpdk-dev] [PATCH] cryptodev: fix compilation error in SUSE 11 SP2

2016-09-26 Thread Pablo de Lara
This commit fixes following build error, which happens in SUSE 11 SP2, with gcc 4.5.1: In file included from lib/librte_cryptodev/rte_cryptodev.c:71:0: lib/librte_cryptodev/rte_cryptodev_pmd.h:76:7: error: flexible array member in otherwise empty struct Fixes: 347a1e037fd3 ("lib: use C99 syntax f

[dpdk-dev] [PATCH v2 4/7] vhost: add dequeue zero copy

2016-09-26 Thread Maxime Coquelin
On 09/23/2016 06:13 AM, Yuanhan Liu wrote: > The basic idea of dequeue zero copy is, instead of copying data from > the desc buf, here we let the mbuf reference the desc buf addr directly. > > Doing so, however, has one major issue: we can't update the used ring > at the end of rte_vhost_dequeue_

[dpdk-dev] [PATCH v2] app/test: improve error message in crypto test code

2016-09-26 Thread De Lara Guarch, Pablo
> -Original Message- > From: Trahe, Fiona > Sent: Monday, September 26, 2016 5:57 AM > To: dev at dpdk.org > Cc: De Lara Guarch, Pablo; Trahe, Fiona > Subject: [PATCH v2] app/test: improve error message in crypto test code > > Improve error message if crypto PMD build is not enabled in c

[dpdk-dev] [PATCH 1/2] vhost: enable any layout feature

2016-09-26 Thread Michael S. Tsirkin
On Mon, Sep 26, 2016 at 11:01:58AM -0700, Stephen Hemminger wrote: > I assume that if using Version 1 that the bit will be ignored > Therein lies a problem. If dpdk tweaks flags, updating it will break guest migration. One way is to require that users specify all flags fully when creating the vi

[dpdk-dev] [PATCH v2 2/7] vhost: get guest/host physical address mappings

2016-09-26 Thread Maxime Coquelin
On 09/23/2016 06:13 AM, Yuanhan Liu wrote: > So that we can convert a guest physical address to host physical > address, which will be used in later Tx zero copy implementation. > > MAP_POPULATE is set while mmaping guest memory regions, to make > sure the page tables are setup and then rte_mem_v

[dpdk-dev] [PATCH] test_cryptodev_perf: IV and digest should be stored at a DMAeble address

2016-09-26 Thread akhil.go...@nxp.com
From: Akhil Goyal For physical crypto devices, IV and digest are processed by the crypto device which need the contents to be written on some DMA able address. So in order to do that, IV and digest are accomodated in the packet. Signed-off-by: Akhil Goyal --- app/test/test_cryptodev_perf.c |

[dpdk-dev] [PATCH] app/test: Remove hard coding for nb_queue_pairs in test_cryptodev

2016-09-26 Thread akhil.go...@nxp.com
From: Akhil Goyal nb_queue_pairs should not be hard coded with device specific number. It should be retrieved from the device infos. Also in ut_setup, ts_params->conf.nb_queue_pairs is already set in testsuite_setup and we are not modifying it. Signed-off-by: Akhil Goyal --- app/test/test_cryp

[dpdk-dev] [PATCH] examples/ipsec-secgw: Update checksum while decrementing ttl

2016-09-26 Thread akhil.go...@nxp.com
From: Akhil Goyal In IPsec-secgw application when TTL is decremented in IP header before forwarding the packet, checksum needs to be updated. In this patch an incremental checksum is added. Other applications(like l3fwd) are also doing so. Signed-off-by: Akhil Goyal --- examples/ipsec-secgw/i

[dpdk-dev] [PATCH v3] vhost: Add indirect descriptors support to the TX path

2016-09-26 Thread Yuanhan Liu
On Mon, Sep 26, 2016 at 03:25:35PM +0300, Michael S. Tsirkin wrote: > On Mon, Sep 26, 2016 at 11:03:54AM +0800, Yuanhan Liu wrote: > > On Fri, Sep 23, 2016 at 01:24:14PM -0700, Stephen Hemminger wrote: > > > On Fri, 23 Sep 2016 21:22:23 +0300 > > > "Michael S. Tsirkin" wrote: > > > > > > > On Fri

[dpdk-dev] [v2 04/15] bnxt: Update Broadcom PMD driver documentation

2016-09-26 Thread Mcnamara, John
>> Reformatting the text as follows would looks better in the rendered output: ? > Do you want me to resend the patch (series). > Or you can manage it w/o that? It isn't worth resubmitting the patchset for this. If you do have to resubmit for other reasons you can make these changes as well. So:

[dpdk-dev] [PATCH 13/13] net/thunderx: document secondary queue set support

2016-09-26 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Kamil Rytarowski > Sent: Friday, August 26, 2016 5:54 PM > To: dev at dpdk.org > Cc: maciej.czekaj at caviumnetworks.com; zyta.szpak at semihalf.com; > slawomir.rosek at semihalf.com; rad at semihalf.com; > jerin

[dpdk-dev] [PATCH v2] app/test: add AES GCM performance test to cryptodev

2016-09-26 Thread Jain, Deepak K
> -Original Message- > From: Kusztal, ArkadiuszX > Sent: Friday, September 23, 2016 11:30 AM > To: dev at dpdk.org > Cc: Trahe, Fiona ; Jain, Deepak K > ; De Lara Guarch, Pablo > ; Griffin, John intel.com>; > Kusztal, ArkadiuszX > Subject: [PATCH v2] app/test: add AES GCM performance te

[dpdk-dev] [PATCH] app/test: Remove hard coding for nb_queue_pairs in test_cryptodev

2016-09-26 Thread De Lara Guarch, Pablo
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of > akhil.goyal at nxp.com > Sent: Monday, September 26, 2016 9:33 AM > To: dev at dpdk.org > Cc: Akhil Goyal > Subject: [dpdk-dev] [PATCH] app/test: Remove hard coding for > nb_queue_pairs in test_cryptodev > > F

[dpdk-dev] [v2 04/15] bnxt: Update Broadcom PMD driver documentation

2016-09-26 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ajit Khaparde > Sent: Monday, September 26, 2016 5:19 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [v2 04/15] bnxt: Update Broadcom PMD driver > documentation > > Update doc/guides/nics/bnxt.rst to indicate th

[dpdk-dev] [PATCH] app/test: fix compilation error when debug mode enabled

2016-09-26 Thread De Lara Guarch, Pablo
Hi Arek, > -Original Message- > From: Kusztal, ArkadiuszX > Sent: Monday, September 26, 2016 6:53 AM > To: dev at dpdk.org > Cc: Trahe, Fiona; Jain, Deepak K; De Lara Guarch, Pablo; Griffin, John; > Kusztal, > ArkadiuszX > Subject: [PATCH] app/test: fix compilation error when debug mode e

[dpdk-dev] [RFC PATCH v2 3/5] librte_ether: add API's for VF management

2016-09-26 Thread Thomas Monjalon
2016-09-26 15:37, Iremonger, Bernard: > Hi Thomas, Bruce, > > > > > Subject: Re: [dpdk-dev] [RFC PATCH v2 3/5] librte_ether: add API's for VF > > management > > > > 2016-09-23 17:02, Iremonger, Bernard: > > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > > > 2016-09-23 09:53,

[dpdk-dev] [PATCH] net/vhost: Add function to retreive the 'vid' for a given port id

2016-09-26 Thread Thomas Monjalon
2016-09-26 16:24, Iremonger, Bernard: > Hi Bruce, Thomas, > > > > > Subject: Re: [dpdk-dev] [PATCH] net/vhost: Add function to retreive the > > 'vid' > > for a given port id > > > > On Mon, Sep 26, 2016 at 04:26:27PM +0200, Thomas Monjalon wrote: > > > 2016-09-26 14:18, Bruce Richardson: > > >

[dpdk-dev] [PATCH v2 1/4] doc: move tool guides in their own subdirectory

2016-09-26 Thread Christian Ehrhardt
On Mon, Sep 26, 2016 at 5:13 PM, Mcnamara, John wrote: > > P.S. Could you mark the V1 version as "superseded" in PatchWork. Done I'd have thought it does that automatically. But maybe it failed to identify them as belonging together. Thanks for the hint - cleanup is always good. -- Christi

[dpdk-dev] [v2 00/15] bnxt patch series

2016-09-26 Thread Ferruh Yigit
Hi Ajit, On 9/26/2016 5:13 PM, Ajit Khaparde wrote: > Please consider applying this patch series. > > v1->v2: Update the bnxt PMD to use a newer HWRM API > has been split into smaller patches for easier review. > > Ajit Khaparde (15): > bnxt: Add support for Broadcom StrataGX Communication Pro

[dpdk-dev] [PATCH v2 2/2] doc: l2fwd: document new --[no-]mac-updating option

2016-09-26 Thread Maxime Coquelin
On 09/26/2016 05:36 PM, Mcnamara, John wrote: > > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Mcnamara, John >> Sent: Monday, September 26, 2016 11:33 AM >> To: Maxime Coquelin ; Richardson, Bruce >> ; De Lara Guarch, Pablo >> ; dev at dpdk.org >> Subj

[dpdk-dev] [RFC] latencystats: added new library for latency stats

2016-09-26 Thread Reshma Pattan
Library is designed to calculate latency stats and report them to the application when queried. Library measures minimum, average, maximum latencies and jitter in nano seconds. Current implementation supports global latency stats, i.e. per application stats. APIs: Added APIs to initialize and un

[dpdk-dev] [PATCH] app/test: remove large lpm test head file

2016-09-26 Thread Wei Dai
remove the large file app/test/test_lpm_routes.h and add codes to auto-generate similar large route rule talbe which keeps same depth and IP class distribution as previous one in test_lpm_routes.h . With the rule table auto-generated at run time, the performance of looking up keep similar to that f

[dpdk-dev] [PATCH v5 0/2] net/i40e: enable 25G device

2016-09-26 Thread Ferruh Yigit
Hi Qi, On 9/26/2016 12:25 AM, Zhang Qi wrote: > The patch enable devices that support 25G link speed. > It is based on previous 25G base driver update: > http://dpdk.org/ml/archives/dev/2016-August/045569.html > > v5: > - no chnage, just for adding the missing change log. > > v4: > - correct ret

[dpdk-dev] [PATCH] net/vhost: Add function to retreive the 'vid' for a given port id

2016-09-26 Thread Wiles, Keith
Sent from my iPhone On Sep 26, 2016, at 9:24 AM, Iremonger, Bernard mailto:bernard.iremonger at intel.com>> wrote: Hi Bruce, Thomas, Subject: Re: [dpdk-dev] [PATCH] net/vhost: Add function to retreive the 'vid' for a given port id On Mon, Sep 26, 2016 at 04:26:27PM +0200, Thomas Monjalon wr

[dpdk-dev] [PATCH v3 2/2] app/testpmd: improve handling of multiprocess

2016-09-26 Thread Marcin Kerlin
Added lookup for pool name because secondary process should attach to mempool created by primary process rather than create new one. Added function free_shared_dev_data() used at the exit of the testpmd. This causes detach devices data from array rte_eth_dev_data[] shared between all processes. Th

[dpdk-dev] [PATCH v3 1/2] librte_ether: ensure not overwrite device data in mp app

2016-09-26 Thread Marcin Kerlin
Added prevention not overwrite device data in array rte_eth_dev_data[]. Secondary process appends in the first free place rather than at the beginning. This behavior prevents overwriting devices data of primary process by secondary process. Signed-off-by: Marcin Kerlin --- lib/librte_ether/rte_e

[dpdk-dev] [PATCH v3 0/2] app/testpmd: improve multiprocess support

2016-09-26 Thread Marcin Kerlin
This patch ensure not overwrite device data in the multiprocess application. 1)Changes in the library introduces continuity in array rte_eth_dev_data[] shared between all processes. Secondary process adds new entries in free space instead of overwriting existing entries. 2)Changes in application

[dpdk-dev] [PATCH v3 19/19] kni: move kernel version ifdefs to compat header

2016-09-26 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/compat.h| 12 lib/librte_eal/linuxapp/kni/kni_vhost.c | 16 +--- 2 files changed, 17 insertions(+), 11 deletions(-) diff --git a/lib/librte_eal/linuxapp/kni/compat.h b/lib/librte_eal/linuxapp/kni/compat.h i

[dpdk-dev] [PATCH v3 18/19] kni: prefer uint32_t to unsigned int

2016-09-26 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/kni_dev.h | 8 lib/librte_eal/linuxapp/kni/kni_fifo.h | 28 ++-- lib/librte_eal/linuxapp/kni/kni_misc.c | 17 +++-- lib/librte_eal/linuxapp/kni/kni_net.c | 24

[dpdk-dev] [PATCH v3 17/19] kni: updated log messages

2016-09-26 Thread Ferruh Yigit
Remove some function entrance logs and changed log level of some logs. Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/kni_misc.c | 5 - lib/librte_eal/linuxapp/kni/kni_net.c | 6 ++ 2 files changed, 2 insertions(+), 9 deletions(-) diff --git a/lib/librte_eal/linuxapp/kni/k

[dpdk-dev] [PATCH v3 16/19] kni: remove compile time debug configuration

2016-09-26 Thread Ferruh Yigit
Since switched to kernel dynamic debugging it is possible to remove compile time debug log configuration. Signed-off-by: Ferruh Yigit --- config/common_base | 3 --- lib/librte_eal/linuxapp/kni/kni_dev.h | 18 - lib/librte_eal/linuxapp/kni/kni_misc.c | 8 +++

[dpdk-dev] [PATCH v3 15/19] kni: move functions to eliminate function declarations

2016-09-26 Thread Ferruh Yigit
Function implementations kept same. Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/kni_misc.c | 311 - lib/librte_eal/linuxapp/kni/kni_net.c | 295 +++ 2 files changed, 293 insertions(+), 313 deletions(-) diff --git a/lib

[dpdk-dev] [PATCH v3 14/19] kni: remove unnecessary 'out of memory' message

2016-09-26 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/kni_net.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/lib/librte_eal/linuxapp/kni/kni_net.c b/lib/librte_eal/linuxapp/kni/kni_net.c index 6328e1e..5182b2f 100644 --- a/lib/librte_eal/linuxapp/kni/kni_net.c +

[dpdk-dev] [PATCH v3 13/19] kni: update kernel logging

2016-09-26 Thread Ferruh Yigit
Switch to dynamic logging functions. Depending kernel configuration this may cause previously visible logs disappear. How to enable dynamic logging: https://www.kernel.org/doc/Documentation/dynamic-debug-howto.txt Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/kni_dev.h | 13

[dpdk-dev] [PATCH v3 12/19] kni: prefer ether_addr_copy to memcpy

2016-09-26 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/compat.h | 4 lib/librte_eal/linuxapp/kni/kni_misc.c | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/lib/librte_eal/linuxapp/kni/compat.h b/lib/librte_eal/linuxapp/kni/compat.h index d79d626..9ae50a7 100644

[dpdk-dev] [PATCH v3 11/19] kni: prefer min_t to min

2016-09-26 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/kni_net.c | 6 +++--- lib/librte_eal/linuxapp/kni/kni_vhost.c | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/librte_eal/linuxapp/kni/kni_net.c b/lib/librte_eal/linuxapp/kni/kni_net.c index bae81e6..04a3d63

[dpdk-dev] [PATCH v3 10/19] kni: macros with complex values should be enclosed in parentheses

2016-09-26 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/compat.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/librte_eal/linuxapp/kni/compat.h b/lib/librte_eal/linuxapp/kni/compat.h index 962a4e7..d79d626 100644 --- a/lib/librte_eal/linuxapp/kni/compat.h +++ b/lib/

[dpdk-dev] [PATCH v3 09/19] kni: do not use assignment in if condition

2016-09-26 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/kni_vhost.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/lib/librte_eal/linuxapp/kni/kni_vhost.c b/lib/librte_eal/linuxapp/kni/kni_vhost.c index bef4889..eacfe3f 100644 --- a/lib/librte_eal/linuxapp/kni

[dpdk-dev] [PATCH v3 08/19] kni: trailing statements should be on next line

2016-09-26 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/kni_vhost.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/lib/librte_eal/linuxapp/kni/kni_vhost.c b/lib/librte_eal/linuxapp/kni/kni_vhost.c index ec39538..bef4889 100644 --- a/lib/librte_eal/linuxapp/kni/kni

[dpdk-dev] [PATCH v3 07/19] kni: comparisons should place the constant on the right

2016-09-26 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/kni_vhost.c | 26 +++--- 1 file changed, 11 insertions(+), 15 deletions(-) diff --git a/lib/librte_eal/linuxapp/kni/kni_vhost.c b/lib/librte_eal/linuxapp/kni/kni_vhost.c index f1345c3..ec39538 100644 --- a/lib/librt

[dpdk-dev] [PATCH v3 06/19] kni: remove useless return

2016-09-26 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/kni_net.c | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/librte_eal/linuxapp/kni/kni_net.c b/lib/librte_eal/linuxapp/kni/kni_net.c index 937dd09..bae81e6 100644 --- a/lib/librte_eal/linuxapp/kni/kni_net.c +++ b/lib/librte_eal/li

[dpdk-dev] [PATCH v3 05/19] kni: prefer unsigned int to unsigned

2016-09-26 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/kni_dev.h | 4 ++-- lib/librte_eal/linuxapp/kni/kni_fifo.h | 28 ++-- lib/librte_eal/linuxapp/kni/kni_net.c | 22 +++--- lib/librte_eal/linuxapp/kni/kni_vhost.c | 12 ++-- 4 files cha

[dpdk-dev] [PATCH v3 04/19] kni: whitespace, indentation, long line corrections

2016-09-26 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/kni_dev.h | 11 +++--- lib/librte_eal/linuxapp/kni/kni_ethtool.c | 39 +++- lib/librte_eal/linuxapp/kni/kni_fifo.h| 2 +- lib/librte_eal/linuxapp/kni/kni_misc.c| 20 ++- lib/librte_eal/linuxapp/kni/kn

[dpdk-dev] [PATCH v3 03/19] kni: make static struct const

2016-09-26 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/kni_misc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/librte_eal/linuxapp/kni/kni_misc.c b/lib/librte_eal/linuxapp/kni/kni_misc.c index f051595..37e9295 100644 --- a/lib/librte_eal/linuxapp/kni/kni_misc.c ++

[dpdk-dev] [PATCH v3 02/19] kni: uninitialize global variables

2016-09-26 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/kni_misc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/librte_eal/linuxapp/kni/kni_misc.c b/lib/librte_eal/linuxapp/kni/kni_misc.c index 113fc20..f051595 100644 --- a/lib/librte_eal/linuxapp/kni/kni_mis

[dpdk-dev] [PATCH v3 01/19] kni: move externs to the header file

2016-09-26 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- v2: keep variable externs in .c file --- lib/librte_eal/linuxapp/kni/kni_dev.h | 11 +++ lib/librte_eal/linuxapp/kni/kni_misc.c | 14 ++ lib/librte_eal/linuxapp/kni/kni_vhost.c | 11 +-- 3 files changed, 14 insertions(+), 22 deletion

[dpdk-dev] [PATCH v3 00/19] KNI checkpatch cleanup

2016-09-26 Thread Ferruh Yigit
KNI checkpatch cleanup, mostly non-functional but cosmetic modifications. Only functional change is related logging, switched to kernel dynamic logging and compile time KNI debug options removed, some log message levels updated. v3: rebased on latest master depends on http://dpdk.org/dev/patchwork

[dpdk-dev] [PATCH] net/vhost: Add function to retreive the 'vid' for a given port id

2016-09-26 Thread Thomas Monjalon
2016-09-26 14:18, Bruce Richardson: > On Mon, Sep 26, 2016 at 03:12:01PM +0200, Thomas Monjalon wrote: > > 2016-09-23 21:23, Wiles, Keith: > > > On Sep 23, 2016, at 12:26 AM, Yuanhan Liu > > linux.intel.com> wrote: > > > > On Thu, Sep 22, 2016 at 06:43:55PM +0200, Thomas Monjalon wrote: > > >

[dpdk-dev] [PATCH] net/vhost: Add function to retreive the 'vid' for a given port id

2016-09-26 Thread Iremonger, Bernard
Hi Bruce, Thomas, > Subject: Re: [dpdk-dev] [PATCH] net/vhost: Add function to retreive the 'vid' > for a given port id > > On Mon, Sep 26, 2016 at 04:26:27PM +0200, Thomas Monjalon wrote: > > 2016-09-26 14:18, Bruce Richardson: > > > On Mon, Sep 26, 2016 at 03:12:01PM +0200, Thomas Monjalon wr

[dpdk-dev] pci-passthrough using vfio-pci

2016-09-26 Thread S.V. Nagaharish K.
I have a quad port X710 nic card, and trying to use two ports in a VM using pci-passthrough and other two ports in the Host. After doing pci-passthrough using vfio-pci, all the pci devices are showing under devices handled by dpdk PMD. But the dpdk initialization in the host is failing, becau

[dpdk-dev] [RFC PATCH v2 3/5] librte_ether: add API's for VF management

2016-09-26 Thread Iremonger, Bernard
Hi Thomas, Bruce, > Subject: Re: [dpdk-dev] [RFC PATCH v2 3/5] librte_ether: add API's for VF > management > > 2016-09-23 17:02, Iremonger, Bernard: > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > > 2016-09-23 09:53, Richardson, Bruce: > > > > From: Thomas Monjalon [mailto:t

[dpdk-dev] [PATCH v2 2/2] doc: l2fwd: document new --[no-]mac-updating option

2016-09-26 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Mcnamara, John > Sent: Monday, September 26, 2016 11:33 AM > To: Maxime Coquelin ; Richardson, Bruce > ; De Lara Guarch, Pablo > ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 2/2] doc: l2fwd: document n

[dpdk-dev] [PATCH 16/17] doc: update qede pmd documentation

2016-09-26 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Rasesh Mody > Sent: Saturday, August 27, 2016 7:29 AM > To: dev at dpdk.org > Cc: Dept-EngDPDKDev at qlogic.com; Rasesh Mody > Subject: [dpdk-dev] [PATCH 16/17] doc: update qede pmd documentation > > Signed-o

[dpdk-dev] [PATCH] net/vhost: Add function to retreive the 'vid' for a given port id

2016-09-26 Thread Bruce Richardson
On Mon, Sep 26, 2016 at 04:26:27PM +0200, Thomas Monjalon wrote: > 2016-09-26 14:18, Bruce Richardson: > > On Mon, Sep 26, 2016 at 03:12:01PM +0200, Thomas Monjalon wrote: > > > 2016-09-23 21:23, Wiles, Keith: > > > > On Sep 23, 2016, at 12:26 AM, Yuanhan Liu > > > linux.intel.com> wrote: > > > >

[dpdk-dev] [PATCH v2] doc: add limitations for i40e PMD

2016-09-26 Thread Mcnamara, John
> -Original Message- > From: Wu, Jingjing > Sent: Friday, September 16, 2016 6:06 PM > To: dev at dpdk.org > Cc: Wu, Jingjing ; Xing, Beilei > ; Mcnamara, John > Subject: [PATCH v2] doc: add limitations for i40e PMD > > This patch adds "Limitations or Known issues" section for i40e PMD,

[dpdk-dev] [PATCH v3] vhost: Add indirect descriptors support to the TX path

2016-09-26 Thread Michael S. Tsirkin
On Mon, Sep 26, 2016 at 11:03:54AM +0800, Yuanhan Liu wrote: > On Fri, Sep 23, 2016 at 01:24:14PM -0700, Stephen Hemminger wrote: > > On Fri, 23 Sep 2016 21:22:23 +0300 > > "Michael S. Tsirkin" wrote: > > > > > On Fri, Sep 23, 2016 at 08:16:09PM +0200, Maxime Coquelin wrote: > > > > > > > > > >

[dpdk-dev] [PATCH v2 4/4] doc: add basic invocation info for dpdk-devbind

2016-09-26 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Christian Ehrhardt > Sent: Wednesday, August 31, 2016 8:50 AM > To: christian.ehrhardt at canonical.com; thomas.monjalon at 6wind.com; > dev at dpdk.org > Subject: [dpdk-dev] [PATCH v2 4/4] doc: add basic invoc

[dpdk-dev] [PATCH v2 3/4] doc: add basic invocation info for dpdk-pmdinfo

2016-09-26 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Christian Ehrhardt > Sent: Wednesday, August 31, 2016 8:50 AM > To: christian.ehrhardt at canonical.com; thomas.monjalon at 6wind.com; > dev at dpdk.org > Subject: [dpdk-dev] [PATCH v2 3/4] doc: add basic invoc

[dpdk-dev] [PATCH v2 2/4] doc: rendering and installation of man pages

2016-09-26 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Christian Ehrhardt > Sent: Wednesday, August 31, 2016 8:50 AM > To: christian.ehrhardt at canonical.com; thomas.monjalon at 6wind.com; > dev at dpdk.org > Subject: [dpdk-dev] [PATCH v2 2/4] doc: rendering and i

[dpdk-dev] [PATCH v2 1/4] doc: move tool guides in their own subdirectory

2016-09-26 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Christian Ehrhardt > Sent: Wednesday, August 31, 2016 8:50 AM > To: christian.ehrhardt at canonical.com; thomas.monjalon at 6wind.com; > dev at dpdk.org > Subject: [dpdk-dev] [PATCH v2 1/4] doc: move tool guide

[dpdk-dev] [PATCH] net/vhost: Add function to retreive the 'vid' for a given port id

2016-09-26 Thread Thomas Monjalon
2016-09-23 21:23, Wiles, Keith: > On Sep 23, 2016, at 12:26 AM, Yuanhan Liu > wrote: > > On Thu, Sep 22, 2016 at 06:43:55PM +0200, Thomas Monjalon wrote: > There could be a similar need in other PMD. > If we can get an opaque identifier of the device which is not the >

[dpdk-dev] [PATCH v2 1/2] librte_ether: ensure not overwrite device data in mp app

2016-09-26 Thread Kerlin, MarcinX
Hi Thomas, > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Friday, September 23, 2016 4:13 PM > To: Kerlin, MarcinX > Cc: Pattan, Reshma ; dev at dpdk.org; De Lara > Guarch, Pablo > Subject: Re: [dpdk-dev] [PATCH v2 1/2] librte_ether: ensure not

[dpdk-dev] [PATCH] kni: add support for core_id param in single threaded mode

2016-09-26 Thread Ferruh Yigit
On 9/24/2016 2:13 PM, Vladyslav Buslov wrote: > Allow binding KNI thread to specific core in single threaded mode > by setting core_id and force_bind config parameters. > > Signed-off-by: Vladyslav Buslov > Acked-by: Ferruh Yigit Tested-by: Ferruh Yigit

[dpdk-dev] [PATCH] app/test: fix compilation error when debug mode enabled

2016-09-26 Thread Arek Kusztal
This patch fixes compilation error in test_cryptodev.c when RTE_LOG_DEBUG is specified Fixes: ba0dda7a7100 ("app/test: add GMAC authentication tests to cryptodev tests") Signed-off-by: Arek Kusztal --- app/test/test_cryptodev.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --g

[dpdk-dev] [PATCH v3] rte_delay_us can be replaced with user function

2016-09-26 Thread Thomas Monjalon
2016-09-26 10:35, jozmarti at cisco.com: > From: Jozef Martiniak > > When running single-core, some drivers tend to call rte_delay_us for a > long time, and that is causing packet drops. > To avoid this, rte_delay_us can be replaced with user-defined delay > function with: > > void rte_delay_us_

[dpdk-dev] [PATCH 2/2] net/virtio: enable any layout feature

2016-09-26 Thread Yuanhan Liu
The feature VIRTIO_F_ANY_LAYOUT was actually supported by commit dd856dfcb9e7 ("virtio: use any layout on Tx"). But it's not enabled. Here this patch enables it. Signed-off-by: Yuanhan Liu --- drivers/net/virtio/virtio_ethdev.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/virt

[dpdk-dev] [PATCH 1/2] vhost: enable any layout feature

2016-09-26 Thread Yuanhan Liu
The VIRTIO_F_ANY_LAYOUT feature allows virtio-net header and packet data in single vring desc if possible. Before that, it is assumed they will always take two descs. DPDK vhost removes this assumption since commit bc7f87a2c19f ("vhost: refactor dequeueing"), meaning we actually support this featu

[dpdk-dev] [PATCH 0/2] enables vhost/virtio any layout feature

2016-09-26 Thread Yuanhan Liu
The feature is actually supported both in virtio PMD and vhost lib. We just haven't enabled it yet. This patchset simply enables it. --- Yuanhan Liu (2): vhost: enable any layout feature net/virtio: enable any layout feature drivers/net/virtio/virtio_ethdev.h | 1 + lib/librte_vhost/vhost.c

[dpdk-dev] [v2 04/15] bnxt: Update Broadcom PMD driver documentation

2016-09-26 Thread Ajit Khaparde
On Mon, Sep 26, 2016 at 2:33 PM, Mcnamara, John wrote: > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ajit Khaparde > > Sent: Monday, September 26, 2016 5:19 PM > > To: dev at dpdk.org > > Subject: [dpdk-dev] [v2 04/15] bnxt: Update Broadcom PMD driver

[dpdk-dev] [PATCH] examples/ipsec-secgw: Update checksum while decrementing ttl

2016-09-26 Thread Sergio Gonzalez Monroy
Hi Akhil, This application relies on checksum offload in both outbound and inbound paths (PKT_TX_IP_CKSUM flag). Because we assume that we always forward the packet in both paths, we decrement the ttl in both inbound and outbound. You seem to only increment (recalculate) the checksum of the inn

[dpdk-dev] [PATCH] net/vhost: Add function to retreive the 'vid' for a given port id

2016-09-26 Thread Bruce Richardson
On Mon, Sep 26, 2016 at 03:12:01PM +0200, Thomas Monjalon wrote: > 2016-09-23 21:23, Wiles, Keith: > > On Sep 23, 2016, at 12:26 AM, Yuanhan Liu > > wrote: > > > On Thu, Sep 22, 2016 at 06:43:55PM +0200, Thomas Monjalon wrote: > > There could be a similar need in other PMD. > >

[dpdk-dev] [PATCH] app/test: fix compilation error when debug mode enabled

2016-09-26 Thread Jain, Deepak K
> -Original Message- > From: Kusztal, ArkadiuszX > Sent: Monday, September 26, 2016 2:53 PM > To: dev at dpdk.org > Cc: Trahe, Fiona ; Jain, Deepak K > ; De Lara Guarch, Pablo > ; Griffin, John intel.com>; > Kusztal, ArkadiuszX > Subject: [PATCH] app/test: fix compilation error when deb

[dpdk-dev] [PATCH v2] app/test: improve error message in crypto test code

2016-09-26 Thread Fiona Trahe
Improve error message if crypto PMD build is not enabled in config file Signed-off-by: Fiona Trahe --- app/test/test_cryptodev.c | 33 + app/test/test_cryptodev_perf.c | 18 ++ 2 files changed, 51 insertions(+) diff --git a/app/test/test_cryp

[dpdk-dev] [PATCH] app/test: remove large lpm test head file

2016-09-26 Thread Dai, Wei
Hi, Bruce How about your test result for this patch ? Especially on performance of rule looking-up ? There are also some replies for your comments as below. Thanks / Wei > -Original Message- > From: Richardson, Bruce > Sent: Monday, September 26, 2016 6:07 PM > To: Dai, Wei > Cc: dev a

[dpdk-dev] [PATCH v5 3/3] app/testpmd: support tunneled TSO in csum fwd engine

2016-09-26 Thread Jianfeng Tan
Add a new command "tunnel_tso set " to enable segmentation offload and set MSS to tso_segsz. Another command, "tunnel_tso show " is added to show tunneled packet MSS. Result 0 means tunnel_tso is disabled. The original commands, "tso set " and "tso show " are only reponsible for non-tunneled pac

[dpdk-dev] [PATCH v3 0/5] vhost: optimize enqueue

2016-09-26 Thread Jianbo Liu
On 26 September 2016 at 13:37, Luke Gorrie wrote: > On 22 September 2016 at 11:01, Jianbo Liu wrote: >> >> Tested with testpmd, host: txonly, guest: rxonly >> size (bytes) improvement (%) >> 644.12 >> 128 6 >> 256 2.65 >> 512

[dpdk-dev] [PATCH v3 0/5] vhost: optimize enqueue

2016-09-26 Thread Jianbo Liu
On 26 September 2016 at 13:25, Wang, Zhihong wrote: > > >> -Original Message- >> From: Jianbo Liu [mailto:jianbo.liu at linaro.org] >> Sent: Monday, September 26, 2016 1:13 PM >> To: Wang, Zhihong >> Cc: Thomas Monjalon ; dev at dpdk.org; Yuanhan >> Liu ; Maxime Coquelin >> >> Subject: R

[dpdk-dev] [PATCH v3 0/5] vhost: optimize enqueue

2016-09-26 Thread Jianbo Liu
On 25 September 2016 at 13:41, Wang, Zhihong wrote: > > >> -Original Message- >> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] >> Sent: Friday, September 23, 2016 9:41 PM >> To: Jianbo Liu >> Cc: dev at dpdk.org; Wang, Zhihong ; Yuanhan Liu >> ; Maxime Coquelin >> > Th

[dpdk-dev] [PATCH] app/test: remove large lpm test head file

2016-09-26 Thread Dai, Wei
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Monday, September 26, 2016 5:56 PM > To: Dai, Wei > Cc: dev at dpdk.org; Richardson, Bruce > Subject: Re: [dpdk-dev] [PATCH] app/test: remove large lpm test head file > > 2016-09-26 17:37, Wei Dai:

[dpdk-dev] Using rte_eth_link_get_nowait

2016-09-26 Thread Keren Hochman
Hi, My application needs to know if port is up, but without wasting 9 seconds (using rte_eth_link_get). I tried to register to callback: lsi_event_callback as suggested in the DPDK examples but I did not receive the event RTE_ETH_EVENT_INTR_LSC. Thanks, Keren.

[dpdk-dev] [PATCH v3] net: fix build error with clang

2016-09-26 Thread Yuanhan Liu
Interestingly, clang and gcc has different prototype for _mm_prefetch(). For gcc, we have _mm_prefetch (const void *__P, enum _mm_hint __I) While for clang, it's #define _mm_prefetch(a, sel) (__builtin_prefetch((void *)(a), 0, (sel))) That's how the following error comes with clang: e

[dpdk-dev] [PATCH] fix documentation error on debug functions

2016-09-26 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Mauricio Vasquez B > Sent: Friday, September 2, 2016 12:02 PM > To: dev at dpdk.org > Cc: stephen at networkplumber.org > Subject: [dpdk-dev] [PATCH] fix documentation error on debug functions > > Previous patch

[dpdk-dev] [PATCH 1/1] doc: clarify usage of testpmd mac fwd mode

2016-09-26 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Mark Kavanagh > Sent: Friday, September 9, 2016 5:16 PM > To: dev at dpdk.org > Cc: Kavanagh, Mark B > Subject: [dpdk-dev] [PATCH 1/1] doc: clarify usage of testpmd mac fwd mode > > Explain default testpmd be

[dpdk-dev] [PATCH v3 0/5] vhost: optimize enqueue

2016-09-26 Thread Jianbo Liu
Hi Thomas, On 23 September 2016 at 21:41, Thomas Monjalon wrote: > 2016-09-23 18:41, Jianbo Liu: >> On 23 September 2016 at 10:56, Wang, Zhihong >> wrote: >> . >> > This is expected because the 2nd patch is just a baseline and all >> > optimization >> > patches are organized in the rest of

[dpdk-dev] [RFC][PATCH V2 1/3] examples/vhost: Add vswitch (generic switch) framework

2016-09-26 Thread Yuanhan Liu
Besides the VMDq proposal, I got few more comments for you. On Mon, Sep 05, 2016 at 04:24:29PM +0530, Pankaj Chauhan wrote: > Introduce support for a generic framework for handling of switching between > physical and vhost devices. The vswitch framework introduces the following > concept: > > 1.

[dpdk-dev] [v2 07/15] bnxt: Update the bnxt PMD to use the 1.5.1 HWRM API

2016-09-26 Thread Ajit Khaparde
Update the PMD to use the 1.5.1 HWRM API. Most of the changes in this patch and the following patches are white spaces and rearrangement of the lines - hopefully a onetime change owing to the usage of a different autogenerated file. Other than that, the following fields have been renamed: 1) rx_er

[dpdk-dev] [PATCH] app/test: remove large lpm test head file

2016-09-26 Thread Thomas Monjalon
2016-09-26 17:37, Wei Dai: > remove the large file app/test/test_lpm_routes.h and add codes to > auto-generate similar large route rule talbe which keeps same depth > and IP class distribution as previous one in test_lpm_routes.h . > With the rule table auto-generated at run time, the performance >

[dpdk-dev] [RFC][PATCH V2 1/3] examples/vhost: Add vswitch (generic switch) framework

2016-09-26 Thread Yuanhan Liu
On Tue, Sep 20, 2016 at 02:28:17PM +0530, Pankaj Chauhan wrote: > On 9/19/2016 8:13 PM, Yuanhan Liu wrote: > >Firstly, sorry for being late on this discussion: I just got a chance > >to follow what you guys were talking about. > > > >On Tue, Sep 13, 2016 at 02:51:31PM +0800, Tan, Jianfeng wrote: >

[dpdk-dev] [PATCH] eal: check cpu flags at init

2016-09-26 Thread Aaron Conole
Flavio Leitner writes: > An application might be linked to DPDK but not really use it, > so move the cpu flag check to the EAL initialization instead. > > Signed-off-by: Flavio Leitner > --- > lib/librte_eal/bsdapp/eal/eal.c | 3 +++ > lib/librte_eal/common/eal_common_cpuflags.c | 6

[dpdk-dev] [PATCH v2] scripts: add more git log checks

2016-09-26 Thread Yuanhan Liu
On Fri, Sep 23, 2016 at 01:47:27PM +0100, Ferruh Yigit wrote: > Alphabetically sorted items to check and > added git log capitalization checks for LRO, NIC and PMD > > Signed-off-by: Ferruh Yigit > --- > scripts/check-git-log.sh | 21 - > 1 file changed, 12 insertions(+), 9 d

[dpdk-dev] [PATCH v6] net/virtio: add set_mtu in virtio

2016-09-26 Thread Yuanhan Liu
Hi Stephen, Are you okay with this change? --yliu On Fri, Sep 23, 2016 at 03:17:37PM +, Dey, Souvik wrote: > Hi Liu/Mark/Stephen, > > I have tried to modify the code with all of your latest > comments. > Do let me know if this looks fine or you have more comments. >

[dpdk-dev] [v2 15/15] bnxt: Update struct definitions to use the 1.5.1 HWRM API

2016-09-26 Thread Ajit Khaparde
Update the PMD to use the 1.5.1 HWRM API. Most of the changes in the patch are white spaces and rearrangement of the lines - a onetime change owing to the usage of a different autogenerated file. Structures updated: hwrm_stat_ctx_alloc_input, hwrm_stat_ctx_alloc_output, hwrm_stat_ctx_free_input, h

[dpdk-dev] [v2 14/15] bnxt: Update the HWRM filter related structures to version 1.5.1

2016-09-26 Thread Ajit Khaparde
Update the PMD to use packet filtering related structures as per the 1.5.1 version of the HWRM API. Most of the changes in the patch are white spaces and rearrangement of the lines - a onetime change owing to the usage of a different autogenerated file. Structures being updated: hwrm_cfa_l2_filter

[dpdk-dev] [v2 13/15] bnxt: Update the HWRM ring related structures to version 1.5.1

2016-09-26 Thread Ajit Khaparde
Update the PMD to use HWRM ring related structures as per the 1.5.1 version of the HWRM API. Most of the changes in the patch are white spaces and rearrangement of the lines - a onetime change owing to the usage of a different autogenerated file. Structures being updated: hwrm_ring_alloc_input, h

[dpdk-dev] [v2 12/15] bnxt: Update the VNIC related structures to the 1.5.1 HWRM version

2016-09-26 Thread Ajit Khaparde
Update the PMD to use VNIC related structures as per the 1.5.1 HWRM API. Most of the changes in the patch are white spaces and rearrangement of the lines - a onetime change owing to the usage of a different autogenerated file. Structures being updated: hwrm_vnic_alloc_input, hwrm_vnic_alloc_output

  1   2   >