[dpdk-dev] [PATCH] mk: remove module compilation noise

2016-09-23 Thread Thomas Monjalon
2016-09-08 13:33, Ferruh Yigit: > Following log generated by Linux kernel Makefiles: > (cat /dev/null; echo > kernel/.../build/lib/librte_eal/linuxapp/igb_uio/igb_uio.ko;) > > .../build/lib/librte_eal/linuxapp/igb_uio/modules.order > > This happens because $(Q) used for both Linux and DPDK makef

[dpdk-dev] [PATCH] mk: fix verbosity zero

2016-09-23 Thread Thomas Monjalon
Verbosity is considered enabled when $V is not empty. So V=0 and V=1 are equivalent. It is fixed by unsetting V when it is 0. A side effect is to fix kernel module compilation verbosity which is set to 0 when V is empty. Reported-by: Ferruh Yigit Signed-off-by: Thomas Monjalon --- mk/rte.sdkro

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

2016-09-23 Thread Zhang Qi
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 Zhang Qi (2): net/i40e: use PHY type to check PHY capability net/i40e: enable 25G device drivers/net/i40e/i40e_ethdev.c | 54 +

[dpdk-dev] [PATCH v3 1/2] net/i40e: use PHY type to check PHY capability

2016-09-23 Thread Zhang Qi
In previous code, we use device ID to check PHY capability which is not extensible since there is always new device be added. Now we use PHY type to detect PHY capability. PHY type encoded all link speed the device support, it is read out through aq command "get_phy_capability" at init stage. Sign

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

2016-09-23 Thread Zhang Qi
Add code branch for 25G link speed, so 25G device will be functional. Signed-off-by: Zhang Qi --- drivers/net/i40e/i40e_ethdev.c | 23 ++- drivers/net/i40e/i40e_ethdev.h | 6 ++ 2 files changed, 24 insertions(+), 5 deletions(-) diff --git a/drivers/net/i40e/i40e_ethdev.

[dpdk-dev] [PATCH 29/39] net/ixgbe/base: report autoneg supported for X550

2016-09-23 Thread Wang, Xiao W
Hi Ferruh, > -Original Message- > From: Yigit, Ferruh > Sent: Tuesday, September 20, 2016 1:08 AM > To: Wang, Xiao W ; Lu, Wenzhuo > > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 29/39] net/ixgbe/base: report autoneg > supported for X550 > > On 8/27/2016 4:48 PM, Xiao Wang wrote

[dpdk-dev] [PATCH v3] i40e: enable i40e pmd on ARM platform

2016-09-23 Thread Zhang, Qi Z
Hi > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jianbo Liu > Sent: Friday, August 5, 2016 5:06 PM > To: dev at dpdk.org; Zhang, Helin ; Wu, Jingjing > > Cc: Jianbo Liu > Subject: [dpdk-dev] [PATCH v3] i40e: enable i40e pmd on ARM platform > > And add re

[dpdk-dev] [PATCH 05/12] net/i40e/base: enable 25G get PHY abilities

2016-09-23 Thread Zhang, Qi Z
Hi: > -Original Message- > From: Yigit, Ferruh > Sent: Friday, September 23, 2016 12:54 AM > To: Zhang, Qi Z ; Wu, Jingjing intel.com>; > Zhang, Helin > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 05/12] net/i40e/base: enable 25G get PHY > abilities > > On 8/25/2016 9:05 PM, Zh

[dpdk-dev] 答复: [Pktgen] can we set ethType for RANGE_PKT?

2016-09-23 Thread Caianning
I am using pktgen ver3.0.09. Also, I checked 3.0.14, the same thing there. Thanks. -- ???: Wiles, Keith [mailto:keith.wiles at intel.com] : 2016?9?23? 5:03 ???: Caianning ??: dev at dpdk.org; Houzhipeng ??: Re: [Pktgen] can we set ethType for RANGE_PKT? Regards, Keith > On Sep

[dpdk-dev] [PATCH] pci:fix missing free

2016-09-23 Thread zhouyangchao
Signed-off-by: zhouyangchao --- lib/librte_eal/common/eal_common_pci.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/lib/librte_eal/common/eal_common_pci.c b/lib/librte_eal/common/eal_common_pci.c index 7248c38..eb44998 100644 --- a/lib/librte_eal/common/eal_common_pci

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

2016-09-23 Thread Wang, Zhihong
> -Original Message- > From: Jianbo Liu [mailto:jianbo.liu at linaro.org] > Sent: Thursday, September 22, 2016 10:42 PM > To: Wang, Zhihong > Cc: Yuanhan Liu ; Maxime Coquelin > ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v3 0/5] vhost: optimize enqueue > > On 22 September 2016 a

[dpdk-dev] [PATCH 05/39] net/ixgbe/base: support VF multicast promiscuous

2016-09-23 Thread Wang, Xiao W
Hi Ferruh, > -Original Message- > From: Yigit, Ferruh > Sent: Tuesday, September 20, 2016 1:06 AM > To: Wang, Xiao W ; Lu, Wenzhuo > > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 05/39] net/ixgbe/base: support VF multicast > promiscuous > > On 8/27/2016 4:47 PM, Xiao Wang wrote:

[dpdk-dev] [PATCH 16/39] net/ixgbe/base: bump mailbox version

2016-09-23 Thread Wang, Xiao W
Hi Ferruh, > -Original Message- > From: Yigit, Ferruh > Sent: Tuesday, September 20, 2016 1:03 AM > To: Wang, Xiao W ; Lu, Wenzhuo > > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 16/39] net/ixgbe/base: bump mailbox version > > On 8/27/2016 4:47 PM, Xiao Wang wrote: > > This patc

[dpdk-dev] [PATCH v5 0/2] net/vhost: add pmd xstats support

2016-09-23 Thread Yuanhan Liu
On Thu, Sep 22, 2016 at 04:19:07PM +0800, Zhiyong Yang wrote: > Patch 1 moves all stats counters to a new defined struct vhost_stats, > in order to manage all stats counters in a consistent way. > > Patch 2 adds the pmd xstats support. Applied to dpdk-next-virtio. Thanks. --yliu

[dpdk-dev] [PATCH 0/6] vhost: add Tx zero copy support

2016-09-23 Thread Yuanhan Liu
On Mon, Aug 29, 2016 at 08:57:52AM +, Xu, Qian Q wrote: > Btw, some good news: if I run a simple dequeue workload (running rxonly in > vhost-pmd and runnin txonly in guest testpmd), it yields ~50% performance > boost for packet size 1518B, but this case is without NIC. > And similar case as

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

2016-09-23 Thread Yuanhan Liu
v2: - renamed "tx zero copy" to "dequeue zero copy", to reduce confusions. - hnadle the case that a desc buf might across 2 host phys pages - use MAP_POPULATE to let kernel populate the page tables - updated release note - doc-ed the limitations for the vm2nic case - merge 2 con

[dpdk-dev] [PATCH v2 1/7] vhost: simplify memory regions handling

2016-09-23 Thread Yuanhan Liu
Due to history reason (that vhost-cuse comes before vhost-user), some fields for maintaining the vhost-user memory mappings (such as mmapped address and size, with those we then can unmap on destroy) are kept in "orig_region_map" struct, a structure that is defined only in vhost-user source file.

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

2016-09-23 Thread Yuanhan Liu
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_virt2phy() could yield proper physical address. Signed

[dpdk-dev] [PATCH v2 3/7] vhost: introduce last avail idx for dequeue

2016-09-23 Thread Yuanhan Liu
So far, we retrieve both the used ring and avail ring idx by the var last_used_idx; it won't be a problem because the used ring is updated immediately after those avail entries are consumed. But that's not true when dequeue zero copy is enabled, that used ring is updated only when the mbuf is cons

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

2016-09-23 Thread Yuanhan Liu
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_burst. Because we don't do the copy here, an update of

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

2016-09-23 Thread Yuanhan Liu
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/prog_guide/vhost_lib.rst| 35 +

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

2016-09-23 Thread Yuanhan Liu
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 more frequently. The reason behind that is, when dequeue

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

2016-09-23 Thread Yuanhan Liu
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(+) diff --git a/drivers/net/vhost/rte_eth_vhost.c b/drivers/net/vhost/rte_eth_vhost.c index 7539cd4..61b3dfc

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

2016-09-23 Thread Yuanhan Liu
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 > > > > > > > port id, > > > > > > > we could call some specific functions of the driver not >

[dpdk-dev] [PATCH 17/39] net/ixgbe/base: access IOSF by host interface

2016-09-23 Thread Wang, Xiao W
Hi Ferruh, > -Original Message- > From: Yigit, Ferruh > Sent: Tuesday, September 20, 2016 1:04 AM > To: Wang, Xiao W ; Lu, Wenzhuo > > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 17/39] net/ixgbe/base: access IOSF by host > interface > > On 8/27/2016 4:48 PM, Xiao Wang wrote: >

[dpdk-dev] [PATCH] net/i40e: add compile barrier in i40e vPMD

2016-09-23 Thread Zhang Qi
We detected the out of order packet desc read issue with GCC6.1. This happened after we apply some other patch. Actually by adding some dummy code will also make this happen. So add compile barrier should be a clean solution. The performance test show no impact with this patch. This does make sense

[dpdk-dev] [PATCH 18/39] net/ixgbe/base: fix check on NACK

2016-09-23 Thread Wang, Xiao W
Hi Ferruh, > -Original Message- > From: Yigit, Ferruh > Sent: Tuesday, September 20, 2016 1:07 AM > To: Wang, Xiao W ; Lu, Wenzhuo > > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 18/39] net/ixgbe/base: fix check on NACK > > On 8/27/2016 4:48 PM, Xiao Wang wrote: > > Previously w

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

2016-09-23 Thread Yuanhan Liu
On Tue, Aug 23, 2016 at 03:40:58PM +0800, Yuanhan Liu wrote: > 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((v

[dpdk-dev] [PATCH v2] net/virtio: fix build error with clang

2016-09-23 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 how the following error comes with clang: err

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

2016-09-23 Thread Yuanhan Liu
On Thu, Sep 22, 2016 at 04:54:57PM +0100, Arek Kusztal wrote: > This patch adds AES Galois Counter Mode test case for cryptodev > QAT tests. Test is performed with different buffer sizes, burst > size of 32 and 128b key. Test vectors are placed in > app/test/test_cryptodev_perf_vectors.h file. > >

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

2016-09-23 Thread Yuanhan Liu
On Fri, Sep 23, 2016 at 02:32:57PM +0800, Yuanhan Liu wrote: > On Thu, Sep 22, 2016 at 04:54:57PM +0100, Arek Kusztal wrote: > > This patch adds AES Galois Counter Mode test case for cryptodev > > QAT tests. Test is performed with different buffer sizes, burst > > size of 32 and 128b key. Test vect

[dpdk-dev] [PATCH v2] net/virtio: fix build error with clang

2016-09-23 Thread Jerin Jacob
On Fri, Sep 23, 2016 at 02:15:09PM +0800, Yuanhan Liu wrote: Hi Yuanhan, Thanks for this patch. > 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_p

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

2016-09-23 Thread jozma...@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_callback_register(void(*userfunc)(unsigned)); When userf

[dpdk-dev] [PATCH 1/2] i40e: Add packet_type metadata in the i40e vPMD

2016-09-23 Thread Zhang, Qi Z
Hi: > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jeff Shaw > Sent: Friday, July 15, 2016 12:59 AM > To: dev at dpdk.org; Zhang, Helin ; Wu, Jingjing > ; damarion at cisco.com > Subject: [dpdk-dev] [PATCH 1/2] i40e: Add packet_type metadata in the i40e > vP

[dpdk-dev] [PATCH v2] net/virtio: fix build error with clang

2016-09-23 Thread Yuanhan Liu
On Fri, Sep 23, 2016 at 12:05:14PM +0530, Jerin Jacob wrote: > On Fri, Sep 23, 2016 at 02:15:09PM +0800, Yuanhan Liu wrote: > > Hi Yuanhan, > > Thanks for this patch. > > > Interestingly, clang and gcc has different prototype for _mm_prefetch(). > > For gcc, we have > > > >_mm_prefetch (con

[dpdk-dev] [PATCH 33/39] net/ixgbe/base: add X550em_a FW ALEF support

2016-09-23 Thread Wang, Xiao W
Hi Wenzhuo, > -Original Message- > From: Lu, Wenzhuo > Sent: Thursday, September 22, 2016 10:57 AM > To: Yigit, Ferruh ; Wang, Xiao W > > Cc: dev at dpdk.org > Subject: RE: [dpdk-dev] [PATCH 33/39] net/ixgbe/base: add X550em_a FW ALEF > support > > Hi Xiao, > > > > -Original Messag

[dpdk-dev] [PATCH] eal: remove single file segments related code

2016-09-23 Thread Yuanhan Liu
Commit c711ccb30987 ("ivshmem: remove library and its EAL integration") removed ivshmem support, but seems David forgot to remove the another piece of code: code for RTE_EAL_SINGLE_FILE_SEGMENTS, which is introduced when ivshmem was firstly added. Now ivshmem was removed and a simple git grep show

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

2016-09-23 Thread Maxime Coquelin
Indirect descriptors are usually supported by virtio-net devices, allowing to dispatch a larger number of requests. When the virtio device sends a packet using indirect descriptors, only one slot is used in the ring, even for large packets. The main effect is to improve the 0% packet loss benchma

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

2016-09-23 Thread Yuanhan Liu
On Fri, Sep 23, 2016 at 09:16:49AM +0200, Maxime Coquelin wrote: > + if (vq->desc[desc_indexes[i]].flags & VRING_DESC_F_INDIRECT) { > + if (unlikely(!(dev->features & > + (1ULL << > VIRTIO_RING_F_INDIRECT_DESC { > +

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

2016-09-23 Thread Maxime Coquelin
On 09/23/2016 09:29 AM, Yuanhan Liu wrote: > On Fri, Sep 23, 2016 at 09:16:49AM +0200, Maxime Coquelin wrote: >> +if (vq->desc[desc_indexes[i]].flags & VRING_DESC_F_INDIRECT) { >> +if (unlikely(!(dev->features & >> +(1ULL << >>

[dpdk-dev] [PATCH] Tools: Fix issue with virtio interface names

2016-09-23 Thread Yuanhan Liu
On Thu, Sep 22, 2016 at 04:46:52PM +0200, Thomas Monjalon wrote: > 2016-09-22 10:34, Mussar, Gary: > > I am new at submitting patches to dpdk.org. This patch has been acked and > > patchwork claims success on build it: > > http://www.dpdk.org/dev/patchwork/patch/15595/ > > > > Other users state

[dpdk-dev] [PATCH v2 0/8] IPsec Enhancements

2016-09-23 Thread Sergio Gonzalez Monroy
This patch set mainly adds support for AES-GCM and AES-CTR. It also updates the IV generation method for AES-CBC mode using the forward function instead of randomly generating the IV. v2: - Update releas notes. - Initialize salt values, GCM/CTR key length now is 20B, 16B key and 4 LSB salt.

[dpdk-dev] [PATCH v2 1/7] examples/ipsec-secgw: change CBC IV generation

2016-09-23 Thread Sergio Gonzalez Monroy
NIST SP800-38A recommends two methods to generate unpredictable IVs (Initilisation Vector) for CBC mode: 1) Apply the forward function to a nonce (ie. counter) 2) Use a FIPS-approved random number generator This patch implements the first recommended method by using the forward function to generat

[dpdk-dev] [PATCH v2 2/7] examples/ipsec-secgw: reset crypto operation status

2016-09-23 Thread Sergio Gonzalez Monroy
Signed-off-by: Sergio Gonzalez Monroy --- examples/ipsec-secgw/ipsec.c | 1 + 1 file changed, 1 insertion(+) diff --git a/examples/ipsec-secgw/ipsec.c b/examples/ipsec-secgw/ipsec.c index 1e87d0d..f49143b 100644 --- a/examples/ipsec-secgw/ipsec.c +++ b/examples/ipsec-secgw/ipsec.c @@ -124,6 +124

[dpdk-dev] [PATCH v2 3/7] examples/ipsec-secgw: add AES-GCM support

2016-09-23 Thread Sergio Gonzalez Monroy
Add support for AES-GCM (Galois-Counter Mode). RFC4106: The Use of Galois-Counter Mode (GCM) in IPSec ESP. Signed-off-by: Sergio Gonzalez Monroy --- examples/ipsec-secgw/esp.c | 59 ++-- examples/ipsec-secgw/ipsec.h | 9 +++ examples/ipsec-secgw/sa

[dpdk-dev] [PATCH v2 4/7] examples/ipsec-secgw: enable AES-CTR mode

2016-09-23 Thread Sergio Gonzalez Monroy
RFC3686: Using AES Counter (CTR) Mode With IPsec ESP.` Signed-off-by: Sergio Gonzalez Monroy --- examples/ipsec-secgw/esp.c | 2 ++ examples/ipsec-secgw/sa.c | 7 +++ 2 files changed, 9 insertions(+) diff --git a/examples/ipsec-secgw/esp.c b/examples/ipsec-secgw/esp.c index 7ee53da..ec5a2e

[dpdk-dev] [PATCH v2 5/7] examples/ipsec-secgw: check sp only when setup

2016-09-23 Thread Sergio Gonzalez Monroy
Application will segfault if there is IPv4 or IPv6 and no SP/ACL rules for IPv4 or IPv6 respectively. Avoid checking the ACL/SP in such cases. Fixes: 906257e965b7 ("examples/ipsec-secgw: support IPv6") Signed-off-by: Sergio Gonzalez Monroy --- examples/ipsec-secgw/ipsec-secgw.c | 4 ++-- 1 fil

[dpdk-dev] [PATCH v2 6/7] examples/ipsec-secgw: cryptodev queue size macro

2016-09-23 Thread Sergio Gonzalez Monroy
Signed-off-by: Sergio Gonzalez Monroy --- examples/ipsec-secgw/ipsec-secgw.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/examples/ipsec-secgw/ipsec-secgw.c b/examples/ipsec-secgw/ipsec-secgw.c index 9eee96f..5a4c9b7 100644 --- a/examples/ipsec-secgw/ipsec-secgw.c +++ b/

[dpdk-dev] [PATCH v2 7/7] examples/ipsec-secgw: initialize sa salt

2016-09-23 Thread Sergio Gonzalez Monroy
This patch initializes the salt value used by the following cipher algorithms: - CBC: random salt - GCM/CTR: the key required is 20B, and the last 4B are used as salt. Signed-off-by: Sergio Gonzalez Monroy --- examples/ipsec-secgw/sa.c | 17 ++--- 1 file changed, 14 insertions(+), 3

[dpdk-dev] [PATCH v2 8/8] examples/ipsec-secgw: update release notes

2016-09-23 Thread Sergio Gonzalez Monroy
Signed-off-by: Sergio Gonzalez Monroy --- doc/guides/rel_notes/release_16_11.rst | 9 + 1 file changed, 9 insertions(+) diff --git a/doc/guides/rel_notes/release_16_11.rst b/doc/guides/rel_notes/release_16_11.rst index 373053a..12f507b 100644 --- a/doc/guides/rel_notes/release_16_11.rst

[dpdk-dev] [PATCH v8 1/2] examples/ipsec-secgw: add configuration file support

2016-09-23 Thread Sergio Gonzalez Monroy
On 21/09/2016 13:05, Fan Zhang wrote: > This patch adds the configuration file support to ipsec_secgw > sample application. Instead of hard-coded rules, the users can > specify their own SP, SA, and routing rules in the configuration > file. An command line option "-f" is added to pass the > config

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

2016-09-23 Thread Yuanhan Liu
On Wed, Sep 21, 2016 at 06:45:05PM -0700, Stephen Hemminger wrote: > On Thu, 22 Sep 2016 00:08:38 + > "Dey, Souvik" wrote: > > > Answers inline. > > > > -- > > Regards, > > Souvik > > > > -Original Message- > > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > > Sent

[dpdk-dev] [PATCH v8 2/2] examples/ipsec-secgw: add sample configuration files

2016-09-23 Thread Sergio Gonzalez Monroy
On 21/09/2016 13:05, Fan Zhang wrote: > This patch adds two sample configuration files to ipsec-secgw sample > application. The sample configuration files shows how to setup > back-to-back systems that would forward traffic through an IPsec > tunnel. > > Signed-off-by: Fan Zhang > --- > examples

[dpdk-dev] [PATCH 35/39] net/ixgbe/base: hold semaphore for shadow RAM access

2016-09-23 Thread Wang, Xiao W
Hi Ferruh, > -Original Message- > From: Yigit, Ferruh > Sent: Tuesday, September 20, 2016 1:08 AM > To: Wang, Xiao W ; Lu, Wenzhuo > > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 35/39] net/ixgbe/base: hold semaphore for > shadow RAM access > > On 8/27/2016 4:48 PM, Xiao Wang wr

[dpdk-dev] [PATCH 1/2] i40e: Add packet_type metadata in the i40e vPMD

2016-09-23 Thread Thomas Monjalon
2016-09-23 06:40, Zhang, Qi Z: > > From: Damjan Marion > > > > The ptype is decoded from the rx descriptor and stored in the packet type > > field > > in the mbuf using the same function as the non-vector driver. > > > > Signed-off-by: Damjan Marion > > Signed-off-by: Jeff Shaw > > Acted-by:

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

2016-09-23 Thread Maxime Coquelin
Indirect descriptors are usually supported by virtio-net devices, allowing to dispatch a larger number of requests. When the virtio device sends a packet using indirect descriptors, only one slot is used in the ring, even for large packets. The main effect is to improve the 0% packet loss benchma

[dpdk-dev] [PATCH v2] net/virtio: fix build error with clang

2016-09-23 Thread Thomas Monjalon
2016-09-23 14:15, Yuanhan Liu: > - _mm_prefetch((const void *)rused, _MM_HINT_T0); > + rte_prefetch0(rused); There are other occurences of _mm_prefetch in other drivers. It could deserve a patch.

[dpdk-dev] [PATCH] Tools: Fix issue with virtio interface names

2016-09-23 Thread Thomas Monjalon
2016-09-23 15:36, Yuanhan Liu: > On Thu, Sep 22, 2016 at 04:46:52PM +0200, Thomas Monjalon wrote: > > 2016-09-22 10:34, Mussar, Gary: > > > I am new at submitting patches to dpdk.org. This patch has been acked and > > > patchwork claims success on build it: > > > http://www.dpdk.org/dev/patchwork

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

2016-09-23 Thread Thomas Monjalon
2016-09-23 12:26, Yuanhan Liu: > 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 port id, > > > > > > > > we could call som

[dpdk-dev] [PATCH] pci:fix missing free

2016-09-23 Thread Thomas Monjalon
Hi, 2016-09-23 10:06, zhouyangchao: > Signed-off-by: zhouyangchao Is your first name Yangchao and your surname Zhou? If so, it would be: Signed-off-by: Yangchao Zhou

[dpdk-dev] [PATCH] eal: remove single file segments related code

2016-09-23 Thread Thomas Monjalon
2016-09-23 15:10, Yuanhan Liu: > Commit c711ccb30987 ("ivshmem: remove library and its EAL integration") > removed ivshmem support, but seems David forgot to remove the another > piece of code: code for RTE_EAL_SINGLE_FILE_SEGMENTS, which is introduced > when ivshmem was firstly added. It is not a

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

2016-09-23 Thread Jain, Deepak K
> -Original Message- > From: Yuanhan Liu [mailto:yuanhan.liu at linux.intel.com] > Sent: Friday, September 23, 2016 7:33 AM > To: Kusztal, ArkadiuszX > Cc: dev at dpdk.org; Trahe, Fiona ; Jain, Deepak K > ; De Lara Guarch, Pablo > ; Griffin, John > Subject: Re: [dpdk-dev] [PATCH] app/te

[dpdk-dev] [PATCH] mk: remove module compilation noise

2016-09-23 Thread Ferruh Yigit
On 9/23/2016 12:35 AM, Thomas Monjalon wrote: > 2016-09-08 13:33, Ferruh Yigit: >> Following log generated by Linux kernel Makefiles: >> (cat /dev/null; echo >> kernel/.../build/lib/librte_eal/linuxapp/igb_uio/igb_uio.ko;) > >> .../build/lib/librte_eal/linuxapp/igb_uio/modules.order >> >> This ha

[dpdk-dev] [PATCH] mk: fix verbosity zero

2016-09-23 Thread Ferruh Yigit
On 9/23/2016 12:42 AM, Thomas Monjalon wrote: > Verbosity is considered enabled when $V is not empty. > So V=0 and V=1 are equivalent. > It is fixed by unsetting V when it is 0. > > A side effect is to fix kernel module compilation verbosity > which is set to 0 when V is empty. > > Reported-by: F

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

2016-09-23 Thread Kusztal, ArkadiuszX
Hi Yuanhan, For the first error as Deepak has said it depend on another patch. For the 32b build there will be a v2. Thanks, Arek -Original Message- From: Jain, Deepak K Sent: Friday, September 23, 2016 10:03 AM To: Yuanhan Liu ; Kusztal, ArkadiuszX Cc: dev at dpdk.org; Trahe, Fiona

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

2016-09-23 Thread Kavanagh, Mark B
>Subject: Re: [PATCH v6] net/virtio: add set_mtu in virtio > >On Wed, Sep 21, 2016 at 06:45:05PM -0700, Stephen Hemminger wrote: >> On Thu, 22 Sep 2016 00:08:38 + >> "Dey, Souvik" wrote: >> >> > Answers inline. >> > >> > -- >> > Regards, >> > Souvik >> > >> > -Original Message- >> > Fr

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

2016-09-23 Thread Yuanhan Liu
On Fri, Sep 23, 2016 at 09:02:49AM +, Jain, Deepak K wrote: > > > > -Original Message- > > From: Yuanhan Liu [mailto:yuanhan.liu at linux.intel.com] > > Sent: Friday, September 23, 2016 7:33 AM > > To: Kusztal, ArkadiuszX > > Cc: dev at dpdk.org; Trahe, Fiona ; Jain, Deepak K > > ; D

[dpdk-dev] [PATCH] eal: remove single file segments related code

2016-09-23 Thread Yuanhan Liu
On Fri, Sep 23, 2016 at 10:50:06AM +0200, Thomas Monjalon wrote: > 2016-09-23 15:10, Yuanhan Liu: > > Commit c711ccb30987 ("ivshmem: remove library and its EAL integration") > > removed ivshmem support, but seems David forgot to remove the another > > piece of code: code for RTE_EAL_SINGLE_FILE_SEG

[dpdk-dev] [PATCH] net/mlx5: support Mellanox OFED 3.4

2016-09-23 Thread Adrien Mazarguil
On Wed, Sep 21, 2016 at 03:48:12PM +0200, Nelio Laranjeiro wrote: > Some macros are renamed by Mellanox OFED 3.4. > > Signed-off-by: Nelio Laranjeiro Acked-by: Adrien Mazarguil -- Adrien Mazarguil 6WIND

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

2016-09-23 Thread Yuanhan Liu
On Fri, Sep 23, 2016 at 10:43:20AM +0200, Thomas Monjalon wrote: > 2016-09-23 12:26, Yuanhan Liu: > > 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 whi

[dpdk-dev] [PATCH] net/mlx5: return RSS hash result in mbuf

2016-09-23 Thread Adrien Mazarguil
On Wed, Sep 14, 2016 at 10:16:05AM +0200, Nelio Laranjeiro wrote: > Signed-off-by: Nelio Laranjeiro Acked-by: Adrien Mazarguil > --- > drivers/net/mlx5/mlx5_rxq.c | 1 + > drivers/net/mlx5/mlx5_rxtx.c | 6 +- > drivers/net/mlx5/mlx5_rxtx.h | 2 ++ > 3 files changed, 8 insertions(+), 1 del

[dpdk-dev] [PATCH v2] net/virtio: fix build error with clang

2016-09-23 Thread Yuanhan Liu
On Fri, Sep 23, 2016 at 10:31:12AM +0200, Thomas Monjalon wrote: > 2016-09-23 14:15, Yuanhan Liu: > > - _mm_prefetch((const void *)rused, _MM_HINT_T0); > > + rte_prefetch0(rused); > > There are other occurences of _mm_prefetch in other drivers. > It could deserve a patch. Indeed. I could make

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

2016-09-23 Thread Bruce Richardson
On Thu, Sep 22, 2016 at 07:04:37PM +0200, Thomas Monjalon wrote: > 2016-09-15 16:46, Iremonger, Bernard: > > > > > Do we really need to expose VF specific functions here? > > > > > It can be generic(PF/VF) function indexed only through port_id. > > > > > (example: as rte_eth_dev_set_vlan_anti_spoof

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

2016-09-23 Thread Loftus, Ciara
> On Fri, Sep 23, 2016 at 10:43:20AM +0200, Thomas Monjalon wrote: > > 2016-09-23 12:26, Yuanhan Liu: > > > 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

[dpdk-dev] [PATCH] mk: fix verbosity zero

2016-09-23 Thread Thomas Monjalon
2016-09-23 10:06, Ferruh Yigit: > On 9/23/2016 12:42 AM, Thomas Monjalon wrote: > > Verbosity is considered enabled when $V is not empty. > > So V=0 and V=1 are equivalent. > > It is fixed by unsetting V when it is 0. > > > > A side effect is to fix kernel module compilation verbosity > > which is

[dpdk-dev] [PATCH] mk: fix verbosity zero

2016-09-23 Thread Ferruh Yigit
On 9/23/2016 12:42 AM, Thomas Monjalon wrote: > Verbosity is considered enabled when $V is not empty. > So V=0 and V=1 are equivalent. > It is fixed by unsetting V when it is 0. > > A side effect is to fix kernel module compilation verbosity > which is set to 0 when V is empty. > > Reported-by: F

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

2016-09-23 Thread Thomas Monjalon
2016-09-23 10:20, Bruce Richardson: > On Thu, Sep 22, 2016 at 07:04:37PM +0200, Thomas Monjalon wrote: > > 2016-09-15 16:46, Iremonger, Bernard: > > > > > > Do we really need to expose VF specific functions here? > > > > > > It can be generic(PF/VF) function indexed only through port_id. > > > > >

[dpdk-dev] [PATCH v2 1/6] ethdev: add Tx preparation

2016-09-23 Thread Ananyev, Konstantin
Hi Jerin, > > Hi Konstantin, > > > > > Hi Jerin, > > > > > > > > > > Hi Jerin, > > > > > > Hi Konstantin, > > > > > > > > > > > > > > > > > > > > > > > > > > > [...] > > > > > > > > > > > > > > + > > > > > > > > +#ifdef RTE_ETHDEV_TX_PREP > > > > > > > > > > > > > > Sorry for being a bit late on

[dpdk-dev] [PATCH] Tools: Fix issue with virtio interface names

2016-09-23 Thread Yuanhan Liu
On Fri, Sep 02, 2016 at 03:23:34PM +, Mcnamara, John wrote: > > > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Gary Mussar > > Sent: Friday, September 2, 2016 2:17 PM > > To: dev at dpdk.org > > Cc: Gary Mussar > > Subject: [dpdk-dev] [PATCH] Tools

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

2016-09-23 Thread Richardson, Bruce
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Friday, September 23, 2016 10:36 AM > To: Richardson, Bruce > Cc: Iremonger, Bernard ; dev at dpdk.org; > Jerin > Jacob ; Shah, Rahul R > ; Lu, Wenzhuo ; azelezniak > > Subject: Re: [dpdk-dev] [RFC

[dpdk-dev] [PATCH] eal: remove single file segments related code

2016-09-23 Thread Tan, Jianfeng
> -Original Message- > From: Yuanhan Liu [mailto:yuanhan.liu at linux.intel.com] > Sent: Friday, September 23, 2016 5:15 PM > To: Thomas Monjalon > Cc: dev at dpdk.org; David Marchand; Tan, Jianfeng > Subject: Re: [PATCH] eal: remove single file segments related code > > On Fri, Sep 23,

[dpdk-dev] [PATCH v2 1/6] ethdev: add Tx preparation

2016-09-23 Thread Jerin Jacob
On Fri, Sep 23, 2016 at 09:41:52AM +, Ananyev, Konstantin wrote: Hi Konstantin, > Hi Jerin, > > > > > Hi Konstantin, > > > > > > > > Hi Jerin, > > > > > > > > > > > > > Hi Jerin, > > > > > > > > Hi Konstantin, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [...] > > > > > >

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

2016-09-23 Thread Arek Kusztal
This patch adds AES Galois Counter Mode performance test case for cryptodev QAT tests. Test is performed with different buffer sizes, burst size of 32 and 128b key. Test vectors are placed in app/test/test_cryptodev_perf_vectors.h file. This patch depends on the following patches/patchsets: "app/

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

2016-09-23 Thread Bruce Richardson
On Fri, Sep 23, 2016 at 11:36:21AM +0200, Thomas Monjalon wrote: > 2016-09-23 10:20, Bruce Richardson: > > On Thu, Sep 22, 2016 at 07:04:37PM +0200, Thomas Monjalon wrote: > > > 2016-09-15 16:46, Iremonger, Bernard: > > > > > > > Do we really need to expose VF specific functions here? > > > > > > >

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

2016-09-23 Thread 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 this patch set. > > I think you can do bottleneck analysis on ARM to see what's slowing down the > perf, there might be

[dpdk-dev] [PATCH] net/i40e: fix null pointer dereferences when using VMDQ+RSS

2016-09-23 Thread Bruce Richardson
On Thu, Sep 22, 2016 at 08:20:25AM +, Wu, Jingjing wrote: > > > > -Original Message- > > From: Yigit, Ferruh > > Sent: Wednesday, September 14, 2016 10:00 PM > > To: Rich Lane; dev at dpdk.org > > Cc: Zhang, Helin; Wu, Jingjing > > Subject: Re: [dpdk-dev] [PATCH] net/i40e: fix null po

[dpdk-dev] [PATCH v3] net/i40e: fix parsing QinQ packets type issue

2016-09-23 Thread Bruce Richardson
On Thu, Sep 22, 2016 at 07:40:20AM +, Wu, Jingjing wrote: > > > > -Original Message- > > From: Xing, Beilei > > Sent: Monday, September 12, 2016 5:42 PM > > To: Wu, Jingjing > > Cc: dev at dpdk.org; Xing, Beilei > > Subject: [PATCH v3] net/i40e: fix parsing QinQ packets type issue > >

[dpdk-dev] [PATCH] doc: announce ABI changes in filtering support

2016-09-23 Thread Your Name
From: Laura Stroe This patch adds a notice that the ABI for filter types functionality will be enhanced in the 17.02 release with new operation available to manipulate the tunnel filters: replace filter types. Signed-off-by: Laura Stroe --- doc/guides/rel_notes/deprecation.rst | 9 + 1

[dpdk-dev] [PATCH 00/12] net/i40e: share code update

2016-09-23 Thread Bruce Richardson
On Thu, Aug 25, 2016 at 04:05:06PM -0400, Zhang Qi wrote: > The patch enable 25G PHY support, > add WoL and proxy configure feature for X722 > and other features for all i40e familiy. > > Zhang Qi (12): > net/i40e/base: add function to clear default VSI > net/i40e/base: fix UDP packet header

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

2016-09-23 Thread Ferruh Yigit
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 deletions(-) diff --git a/scripts/check-git-log.sh b/scripts/check-git-log.s

[dpdk-dev] [PATCH 0/3] changes for enabling cuckoo hash into table library

2016-09-23 Thread Dumitrescu, Cristian
> -Original Message- > From: Chokkalingam, SankarX > Sent: Thursday, September 22, 2016 11:12 AM > To: dev at dpdk.org > Cc: Dumitrescu, Cristian ; Singh, Jasvinder > ; Chokkalingam, SankarX > ; Rao, GuruprasadX > > Subject: [PATCH 0/3] changes for enabling cuckoo hash into table library

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

2016-09-23 Thread Thomas Monjalon
2016-09-23 09:53, Richardson, Bruce: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > 2016-09-23 10:20, Bruce Richardson: > > > On Thu, Sep 22, 2016 at 07:04:37PM +0200, Thomas Monjalon wrote: > > > > 2016-09-15 16:46, Iremonger, Bernard: > > > > > > > > Do we really need to expose

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

2016-09-23 Thread Thomas Monjalon
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 this patch set. > > > > I think you can do bottleneck analysis on ARM to s

[dpdk-dev] [PATCH v2 0/2] examples/l2fwd: Add option to enable/disable MAC addresses updating

2016-09-23 Thread Maxime Coquelin
This series adds a new option to enable/disable MAC addresses updating in l2fwd example. Doing that, we can enable basic VM 2 VM communication easily, without external projects dependencies, nor real NIC (as with vhost example). Example of cli with vhost-user: #l2fwd -c f --socket-mem=1024 \

[dpdk-dev] [PATCH v2 1/2] examples/l2fwd: Add new option to enable/disable MAC addresses updating

2016-09-23 Thread Maxime Coquelin
l2fwd could be useful for testing virtual devices without the need of physical ones. To achieve this, this patch adds a new option to enable/disable the MAC addresses updating done at forwarding time: --[no-]mac-updating By default, MAC address updating remains enabled, to keep consistency with p

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

2016-09-23 Thread Maxime Coquelin
This patch documents the new l2fwd option, which provides a way to disable the MAC addresses updating, enabling the use of l2fwd for basic VM to VM communication. Acked-by: John McNamara Signed-off-by: Maxime Coquelin --- doc/guides/sample_app_ug/img/l2_fwd_vm2vm.svg | 311

[dpdk-dev] [PATCH 0/2] examples/l2fwd: Add option to enable/disable MAC addresses tweaking

2016-09-23 Thread Maxime Coquelin
On 09/21/2016 11:53 PM, De Lara Guarch, Pablo wrote: > > >> -Original Message- >> From: Maxime Coquelin [mailto:maxime.coquelin at redhat.com] >> Sent: Tuesday, July 26, 2016 8:56 AM >> To: Richardson, Bruce; De Lara Guarch, Pablo >> Cc: dev at dpdk.org; Maxime Coquelin >> Subject: [PATCH

[dpdk-dev] [PATCH 0/2] examples/l2fwd: Add option to enable/disable MAC addresses tweaking

2016-09-23 Thread Maxime Coquelin
Hi John, On 08/30/2016 05:42 PM, Mcnamara, John wrote: >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Maxime Coquelin >> Sent: Tuesday, July 26, 2016 4:56 PM >> To: Richardson, Bruce ; De Lara Guarch, Pablo >> >> Cc: dev at dpdk.org; Maxime Coquelin >> S

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

2016-09-23 Thread Thomas Monjalon
2016-09-23 08:39, jozmarti at cisco.com: > --- a/lib/librte_eal/common/include/generic/rte_cycles.h > +++ b/lib/librte_eal/common/include/generic/rte_cycles.h > @@ -180,15 +180,17 @@ rte_get_timer_hz(void) > default: rte_panic("Invalid timer source specified\n"); > } > } > - > /** > +

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

2016-09-23 Thread Thomas Monjalon
2016-09-22 14:11, Kerlin, MarcinX: > Hi Reshma, > > From: Pattan, Reshma > > > > Hi Marcin, > > > > > /** > > > * @internal > > > + * Returns a shared device data slot specified by the unique identifier > > > name. > > > + * > > > + * @paramname > > > + * The pointer to the Unique i

  1   2   >