[dpdk-dev] [PATCH 3/3] scripts: ignore self-generated directories in validate-abi startup check

2016-03-10 Thread Panu Matilainen
On 03/10/2016 02:34 PM, Ferruh Yigit wrote: > On 3/10/2016 12:29 PM, Panu Matilainen wrote: >> On 03/10/2016 02:22 PM, Ferruh Yigit wrote: >>> On 3/10/2016 10:53 AM, Panu Matilainen wrote: When doing multiple runs of validate-abi.sh, the git status check will more often than not unnecessa

[dpdk-dev] [PATCH] vhost: Fix default value of kickfd and callfd

2016-03-10 Thread Tan, Jianfeng
On 3/10/2016 2:34 PM, Tetsuya Mukawa wrote: > On 2016/03/10 15:25, Tan, Jianfeng wrote: >> >> On 3/10/2016 2:14 PM, Tetsuya Mukawa wrote: >>> Currently, default value of kickfd and callfd is -1. >>> If the value is -1, current code guess kickfd and callfd hasn't been >>> initialized yet. And vhost

[dpdk-dev] [PATCH 2/3] scripts: avoid editing defconfig_* files in validate-abi.sh

2016-03-10 Thread Panu Matilainen
On 03/10/2016 02:25 PM, Ferruh Yigit wrote: > On 3/10/2016 10:53 AM, Panu Matilainen wrote: >> The defconfig_* files are templates which are not supposed to be >> edited, and doing so tends to leave unwanted cruft behind. Edit >> the "working copy" config instead, which is the intended DPDK way. >>

[dpdk-dev] VMware ESXi E1000 adapter EAL: Error reading from file descriptor 9: Input/output error

2016-03-10 Thread Vincent Li
Hi, I am running ubuntu 14.04 as guest with E1000 adapter in VMware ESXi, mTCP sample application get following error with dpdk-2.2.0 ./apps/example/epwget 10.1.72.17 1 - Loading mtcp configuration from : epwget.conf

[dpdk-dev] [PATCH] qede: Update overview.rst

2016-03-10 Thread Rasesh Mody
Signed-off-by: Harish Patil Signed-off-by: Rasesh Mody Signed-off-by: Sony Chacko --- doc/guides/nics/overview.rst | 70 +- 1 file changed, 35 insertions(+), 35 deletions(-) diff --git a/doc/guides/nics/overview.rst b/doc/guides/nics/overview.rst index

[dpdk-dev] [PATCH 3/3] scripts: ignore self-generated directories in validate-abi startup check

2016-03-10 Thread Panu Matilainen
On 03/10/2016 02:22 PM, Ferruh Yigit wrote: > On 3/10/2016 10:53 AM, Panu Matilainen wrote: >> When doing multiple runs of validate-abi.sh, the git status check >> will more often than not unnecessarily fail with "Working directory not >> clean" error because of the compat_result and compile target

[dpdk-dev] [PATCH v7 01/11] ethdev: add API to query packet type filling info

2016-03-10 Thread Bruce Richardson
On Thu, Mar 10, 2016 at 03:31:25AM +0800, Jianfeng Tan wrote: > Add a new API rte_eth_dev_get_ptype_info to query whether/what packet > type can be filled by given already started device or its pmd rx burst > function has already been decided). > > Signed-off-by: Jianfeng Tan > Acked-by: Konstant

[dpdk-dev] [PATCH] pipeline: use unsigned constants for left shift operations

2016-03-10 Thread Dumitrescu, Cristian
> -Original Message- > From: Panu Matilainen [mailto:pmatilai at redhat.com] > Sent: Thursday, March 10, 2016 1:50 PM > To: dev at dpdk.org > Cc: Dumitrescu, Cristian > Subject: [PATCH] pipeline: use unsigned constants for left shift operations > > Tell the compiler to use unsigned cons

[dpdk-dev] [PATCH v3 0/2] examples/l3fwd: fix using packet type blindly

2016-03-10 Thread Ananyev, Konstantin
> From: Tan, Jianfeng > Sent: Thursday, March 10, 2016 5:51 AM > To: dev at dpdk.org > Cc: Zhang, Helin; Ananyev, Konstantin; nelio.laranjeiro at 6wind.com; > adrien.mazarguil at 6wind.com; rahul.lakkireddy at chelsio.com; > pmatilai at redhat.com; Tan, Jianfeng > Subject: [PATCH v3 0/2] examples

[dpdk-dev] [PATCH] vhost: Fix default value of kickfd and callfd

2016-03-10 Thread Tan, Jianfeng
On 3/10/2016 2:14 PM, Tetsuya Mukawa wrote: > Currently, default value of kickfd and callfd is -1. > If the value is -1, current code guess kickfd and callfd hasn't been > initialized yet. And vhost library will guess the virtqueue isn't ready > for processing. > But callfd and kickfd will be set

[dpdk-dev] [PATCH v8 0/5] cryptodev API changes

2016-03-10 Thread De Lara Guarch, Pablo
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Fiona Trahe > Sent: Thursday, March 10, 2016 1:43 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v8 0/5] cryptodev API changes > > > This patch set separates the symmetric crypto operations from generi

[dpdk-dev] [PATCH] doc: add mempool mgr ABI deprication notice

2016-03-10 Thread Wiles, Keith
>On Thu, Mar 10, 2016 at 01:37:27PM +0100, Olivier MATZ wrote: >> Hi David, >> >> On 03/10/2016 12:55 PM, David Hunt wrote: >> > Announce the ABI breakage due to addition of external mempool >> > manager functionality which requires changes to rte_mempool >> > structure. >> > >> > Signed-off-by:

[dpdk-dev] [PATCH v3 2/2] config: enable vector driver by default

2016-03-10 Thread Jianfeng Tan
Previously, vector driver is not the first (default) choice for i40e, as it cannot fill packet type info for l3fwd to work well. Now there is an option for l3fwd to analysis packet type softly. So enable it by default. Signed-off-by: Jianfeng Tan --- config/common_base | 2 +-

[dpdk-dev] [PATCH v3 1/2] examples/l3fwd: fix using packet type blindly

2016-03-10 Thread Jianfeng Tan
As a example to use ptype info, l3fwd needs firstly to use rte_eth_dev_get_ptype_info() API to check if device and/or its PMD driver will parse and fill the needed packet type; if not, use the newly added option, --parse-ptype, to analyze it in the callback softly. As the mode of EXACT_MATCH uses

[dpdk-dev] [PATCH v3 0/2] examples/l3fwd: fix using packet type blindly

2016-03-10 Thread Jianfeng Tan
This patch will work on below patch series. - [PATCH v7 00/11] Add API to get packet type info v3: - em ptype check: (l4_tcp || l4_udp) -> (l4_tcp && l4_udp). - avoid rte_be_to_cpu_16 for each packet by adding proper macros. - with --parse-ptype specified, use sw parser mandatorily. - enable

[dpdk-dev] [PATCH v8 5/5] mbuf_offload: remove library

2016-03-10 Thread Fiona Trahe
From: Declan Doherty As cryptodev library does not depend on mbuf_offload library any longer, this patch removes it. Signed-off-by: Declan Doherty --- MAINTAINERS| 4 - app/test/test_cryptodev.c | 1 - app/test/test_cryptodev

[dpdk-dev] [PATCH v8 4/5] cryptodev: change burst API to be crypto op oriented

2016-03-10 Thread Fiona Trahe
From: Declan Doherty This patch modifies the crypto burst enqueue/dequeue APIs to operate on bursts rte_crypto_op's rather than the current implementation which operates on rte_mbuf bursts, this simplifies the burst processing in the crypto PMDs and the use of crypto operations in general, includ

[dpdk-dev] [PATCH v8 3/5] cryptodev: remove unused phys_addr field from key

2016-03-10 Thread Fiona Trahe
Remove unused phys_addr field from key in crypto_xform, simplifiy struct and fix knock-on impacts in l2fwd-crypto app Signed-off-by: Fiona Trahe --- examples/l2fwd-crypto/main.c | 42 ++- lib/librte_cryptodev/rte_crypto_sym.h | 16 ++--- 2 files c

[dpdk-dev] [PATCH v8 2/5] cryptodev: refactor to partition common from symmetric-specific code

2016-03-10 Thread Fiona Trahe
This patch splits symmetric specific definitions and functions away from the common crypto APIs to facilitate the future extension and expansion of the cryptodev framework, in order to allow asymmetric crypto operations to be introduced at a later date, as well as to clean the logical structure of

[dpdk-dev] [PATCH v8 1/5] cryptodev: code cleanup

2016-03-10 Thread Fiona Trahe
- Fixed >80char lines in test file - Removed unused elements from stats struct - Removed unused objects in rte_cryptodev_pmd.h - Renamed variables - Replaced leading spaces with tabs - Improved performance results display in test Signed-off-by: Fiona Trahe Signed-off-by: Declan Doherty --- app/

[dpdk-dev] [PATCH v8 0/5] cryptodev API changes

2016-03-10 Thread Fiona Trahe
This patch set separates the symmetric crypto operations from generic operations and then modifies the cryptodev burst API to accept bursts of rte_crypto_op rather than rte_mbufs. v8: - split patchset for easier review - fix broken /examples/l2fwd-crypto build in intermediate patch - split remo

[dpdk-dev] [PATCH 0/3] ethdev: add helper functions to get eth_dev and dev private data

2016-03-10 Thread Ferruh Yigit
On 3/10/2016 12:00 AM, Thomas Monjalon wrote: > 2016-02-17 14:20, Ferruh Yigit: >> This is to provide abstraction and reduce global variable access. >> >> Global variable rte_eth_devices kept exported to not break ABI. >> >> Bonding driver not selected on purpose, just it seems it is using >> rte_

[dpdk-dev] [PATCH] doc: add mempool mgr ABI deprication notice

2016-03-10 Thread Olivier MATZ
Hi David, On 03/10/2016 12:55 PM, David Hunt wrote: > Announce the ABI breakage due to addition of external mempool > manager functionality which requires changes to rte_mempool > structure. > > Signed-off-by: David Hunt Acked-by: Olivier Matz Please note we need not 2 more acks for this kind

[dpdk-dev] [PATCH] doc: add mempool mgr ABI deprication notice

2016-03-10 Thread Bruce Richardson
On Thu, Mar 10, 2016 at 01:37:27PM +0100, Olivier MATZ wrote: > Hi David, > > On 03/10/2016 12:55 PM, David Hunt wrote: > > Announce the ABI breakage due to addition of external mempool > > manager functionality which requires changes to rte_mempool > > structure. > > > > Signed-off-by: David Hun

[dpdk-dev] [PATCH 3/3] scripts: ignore self-generated directories in validate-abi startup check

2016-03-10 Thread Panu Matilainen
When doing multiple runs of validate-abi.sh, the git status check will more often than not unnecessarily fail with "Working directory not clean" error because of the compat_result and compile target directories from the previous run. Filter out the self-generated directories when checking. Signed-

[dpdk-dev] [PATCH 2/3] scripts: avoid editing defconfig_* files in validate-abi.sh

2016-03-10 Thread Panu Matilainen
The defconfig_* files are templates which are not supposed to be edited, and doing so tends to leave unwanted cruft behind. Edit the "working copy" config instead, which is the intended DPDK way. Signed-off-by: Panu Matilainen --- scripts/validate-abi.sh | 18 +- 1 file changed,

[dpdk-dev] [PATCH 1/3] scripts: support parallel building in validate-abi.sh via -j[N] option

2016-03-10 Thread Panu Matilainen
Signed-off-by: Panu Matilainen --- doc/guides/contributing/versioning.rst | 4 +++- scripts/validate-abi.sh| 13 ++--- 2 files changed, 13 insertions(+), 4 deletions(-) diff --git a/doc/guides/contributing/versioning.rst b/doc/guides/contributing/versioning.rst index ae

[dpdk-dev] ixgbevf missed stats

2016-03-10 Thread Yaron Illouz
So I can't know if there where dropps!!! Is this the same for dpdk with virtio? Can I know if there where dropps? -Original Message- From: Lu, Wenzhuo [mailto:wenzhuo...@intel.com] Sent: Thursday, March 10, 2016 2:40 PM To: Yaron Illouz ; dev at dpdk.org Subject: RE: ixgbevf missed stats

[dpdk-dev] [PATCH 1/3] scripts: support parallel building in validate-abi.sh via -j[N] option

2016-03-10 Thread Ferruh Yigit
On 3/10/2016 10:53 AM, Panu Matilainen wrote: > Signed-off-by: Panu Matilainen > --- > doc/guides/contributing/versioning.rst | 4 +++- > scripts/validate-abi.sh| 13 ++--- > 2 files changed, 13 insertions(+), 4 deletions(-) > > diff --git a/doc/guides/contributing/versi

[dpdk-dev] [PATCH 3/3] scripts: ignore self-generated directories in validate-abi startup check

2016-03-10 Thread Ferruh Yigit
On 3/10/2016 12:39 PM, Panu Matilainen wrote: > On 03/10/2016 02:34 PM, Ferruh Yigit wrote: >> On 3/10/2016 12:29 PM, Panu Matilainen wrote: >>> On 03/10/2016 02:22 PM, Ferruh Yigit wrote: On 3/10/2016 10:53 AM, Panu Matilainen wrote: > When doing multiple runs of validate-abi.sh, the git

[dpdk-dev] ixgbevf missed stats

2016-03-10 Thread Lu, Wenzhuo
Hi Yaron, The stats of PF is done by HW. The bad news is I don' find anything for VF. I also doubt if there's corresponding stats for VF. > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Yaron Illouz > Sent: Thursday, March 10, 2016 8:27 PM > To: dev at dpdk.

[dpdk-dev] [PATCH 3/3] scripts: ignore self-generated directories in validate-abi startup check

2016-03-10 Thread Ferruh Yigit
On 3/10/2016 12:29 PM, Panu Matilainen wrote: > On 03/10/2016 02:22 PM, Ferruh Yigit wrote: >> On 3/10/2016 10:53 AM, Panu Matilainen wrote: >>> When doing multiple runs of validate-abi.sh, the git status check >>> will more often than not unnecessarily fail with "Working directory not >>> clean" e

[dpdk-dev] [PATCH v3 8/8] vhost: avoid dead loop chain.

2016-03-10 Thread Yuanhan Liu
If a malicious guest forges a dead loop chain, it could lead to a dead loop of copying the desc buf to mbuf, which results to all mbuf being exhausted. Add a var nr_desc to avoid such case. Suggested-by: Huawei Xie Signed-off-by: Yuanhan Liu --- lib/librte_vhost/vhost_rxtx.c | 5 - 1 file

[dpdk-dev] [PATCH v3 7/8] vhost: do sanity check for desc->next against with vq->size

2016-03-10 Thread Yuanhan Liu
A malicious guest may easily forge some illegal vring desc buf. To make our vhost robust, we need make sure desc->next will not go beyond the vq->desc[] array. Suggested-by: Rich Lane Signed-off-by: Yuanhan Liu --- lib/librte_vhost/vhost_rxtx.c | 6 +- 1 file changed, 5 insertions(+), 1 del

[dpdk-dev] [PATCH v3 6/8] vhost: do sanity check for desc->len

2016-03-10 Thread Yuanhan Liu
We need make sure that desc->len is bigger than the size of virtio net header, otherwise, unexpected behaviour might happen due to "desc_avail" would become a huge number with for following code: desc_avail = desc->len - vq->vhost_hlen; For dequeue code path, it will try to allocate enou

[dpdk-dev] [PATCH v3 5/8] vhost: don't use unlikely for VIRTIO_NET_F_MRG_RXBUF detection

2016-03-10 Thread Yuanhan Liu
VIRTIO_NET_F_MRG_RXBUF is a default feature supported by vhost. Adding unlikely for VIRTIO_NET_F_MRG_RXBUF detection doesn't make sense to me at all. Signed-off-by: Yuanhan Liu --- lib/librte_vhost/vhost_rxtx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/librte_vhost/

[dpdk-dev] [PATCH v3 4/8] vhost: do not use rte_memcpy for virtio_hdr copy

2016-03-10 Thread Yuanhan Liu
First of all, rte_memcpy() is mostly useful for coping big packets by leveraging hardware advanced instructions like AVX. But for virtio net hdr, which is 12 bytes at most, invoking rte_memcpy() will not introduce any performance boost. And, to my suprise, rte_memcpy() is VERY huge. Since rte_memc

[dpdk-dev] [PATCH v3 3/8] vhost: refactor virtio_dev_merge_rx

2016-03-10 Thread Yuanhan Liu
Current virtio_dev_merge_rx() implementation just looks like the old rte_vhost_dequeue_burst(), full of twisted logic, that you can see same code block in quite many different places. However, the logic of virtio_dev_merge_rx() is quite similar to virtio_dev_rx(). The big difference is that the m

[dpdk-dev] [PATCH v3 2/8] vhost: refactor virtio_dev_rx

2016-03-10 Thread Yuanhan Liu
This is a simple refactor, as there isn't any twisted logic in old code. Here I just broke the code and introduced two helper functions, reserve_avail_buf() and copy_mbuf_to_desc() to make the code more readable. Also, it saves nearly 1K bytes of binary code size. Signed-off-by: Yuanhan Liu ---

[dpdk-dev] [PATCH v3 1/8] vhost: refactor rte_vhost_dequeue_burst

2016-03-10 Thread Yuanhan Liu
The current rte_vhost_dequeue_burst() implementation is a bit messy and logic twisted. And you could see repeat code here and there. However, rte_vhost_dequeue_burst() acutally does a simple job: copy the packet data from vring desc to mbuf. What's tricky here is: - desc buff could be chained (by

[dpdk-dev] [PATCH v3 0/8] vhost rxtx refactor and fixes

2016-03-10 Thread Yuanhan Liu
v3: - quite few minor changes, including using likely/unlikely when possible. - Added a new patch 8 to avoid desc dead loop chain The first 3 patches refactor 3 major functions at vhost_rxtx.c. It simplifies the code logic, making it more readable. OTOH, it reduces binary code size, due

[dpdk-dev] ixgbevf missed stats

2016-03-10 Thread Yaron Illouz
Hi There is no statistics of missed packet for ixgbevf in a vm. How can I get these stats? In ixgbe it is called imissed

[dpdk-dev] [PATCH v3 1/1] drivers/net/i40e: add ethdev functions

2016-03-10 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Remy Horton > Sent: Wednesday, March 9, 2016 1:29 PM > To: Zhang, Helin > Cc: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v3 1/1] drivers/net/i40e: add ethdev functions > > Implements driver support for dumpin

[dpdk-dev] [PATCH 2/3] scripts: avoid editing defconfig_* files in validate-abi.sh

2016-03-10 Thread Ferruh Yigit
On 3/10/2016 10:53 AM, Panu Matilainen wrote: > The defconfig_* files are templates which are not supposed to be > edited, and doing so tends to leave unwanted cruft behind. Edit > the "working copy" config instead, which is the intended DPDK way. > > Signed-off-by: Panu Matilainen > --- > scrip

[dpdk-dev] [PATCH 3/3] scripts: ignore self-generated directories in validate-abi startup check

2016-03-10 Thread Ferruh Yigit
On 3/10/2016 10:53 AM, Panu Matilainen wrote: > When doing multiple runs of validate-abi.sh, the git status check > will more often than not unnecessarily fail with "Working directory not > clean" error because of the compat_result and compile target directories > from the previous run. Filter out

[dpdk-dev] [PATCH v3 3/3] vhost: fix vq realloc at numa_realloc

2016-03-10 Thread Yuanhan Liu
vq is allocated on pairs, hence we should do pair reallocation at numa_realloc() as well, otherwise an error like following occurs while do numa reallocation: VHOST_CONFIG: reallocate vq from 0 to 1 node PANIC in rte_free(): Fatal error: Invalid memory The reason we don't catch it is

[dpdk-dev] [PATCH v3 2/3] vhost: simplify numa_realloc

2016-03-10 Thread Yuanhan Liu
We could first check if we need realloc vq or not, if so, reallocate it. We then do similar to vhost dev realloc. This could get rid of the tons of repeated "if (realloc_dev)" and "if (realloc_vq)" statements, therefore, makes code a bit more readable. Signed-off-by: Yuanhan Liu Acked-by: Huawei

[dpdk-dev] [PATCH v3 1/3] vhost: get rid of linked list dev

2016-03-10 Thread Yuanhan Liu
While we use a single linked list to maintain all devices, we could use a static array to achieve the same goal, just like what we did to maintain the eth devices with rte_eth_devices array. This could simplifies the code a bit. Signed-off-by: Yuanhan Liu Acked-by: Huawei Xie --- lib/librte_vho

[dpdk-dev] [PATCH v3 0/3] vhost: virtio-net.c cleanups and fixes

2016-03-10 Thread Yuanhan Liu
This short series includes two cleanup patch, and one bug fix patch. v3: - code rebase --- Yuanhan Liu (3): vhost: get rid of linked list dev vhost: simplify numa_realloc vhost: fix vq realloc at numa_realloc lib/librte_vhost/virtio-net.c | 276 +-

[dpdk-dev] [PATCH, v2] mempool: avoid memory waste with large pagesize

2016-03-10 Thread Olivier MATZ
>> It still does not work. When CONFIG_RTE_LIBRTE_MEMPOOL_DEBUG=y: >> >> mp = rte_mempool_create("test", 128, >> 64, 0, 0, NULL, NULL, NULL, NULL, SOCKET_ID_ANY, 0); >> rte_mempool_dump(stdout, mp); >> >> populated_size=128 >> header_size=64 >> elt_size=64 >> trailer_size=64 >> total

[dpdk-dev] [PATCH v5 0/4] DPDK polling-mode driver for Amazon Elastic Network Adapters (ENA)

2016-03-10 Thread Bruce Richardson
On Wed, Mar 09, 2016 at 12:34:51PM +0100, Jan M?dala wrote: > I'd like to kindly request for review and comments. > > Jan Hi Jan, I've run checkpatch on these patches and below is some of the output that you might want to look at fixing. As well as these, there were a *lot* of warnings about li

[dpdk-dev] [PATCH v3 2/2] examples: rework to use buffered tx

2016-03-10 Thread Tomasz Kulasek
The internal buffering of packets for TX in sample apps is no longer needed, so this patchset also replaces this code with calls to the new rte_eth_tx_buffer* APIs in: * l2fwd-jobstats * l2fwd-keepalive * l2fwd * l3fwd-acl * l3fwd-power * link_status_interrupt * client_server_mp * l2fwd_fork * pac

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

2016-03-10 Thread Tomasz Kulasek
Many sample apps include internal buffering for single-packet-at-a-time operation. Since this is such a common paradigm, this functionality is better suited to being implemented in the ethdev API. The new APIs in the ethdev library are: * rte_eth_tx_buffer_init - initialize buffer * rte_eth_tx_buf

[dpdk-dev] [PATCH v3 0/2] add support for buffered tx to ethdev

2016-03-10 Thread Tomasz Kulasek
Many sample apps include internal buffering for single-packet-at-a-time operation. Since this is such a common paradigm, this functionality is better suited to being implemented in the ethdev API. The new APIs in the ethdev library are: * rte_eth_tx_buffer_init - initialize buffer * rte_eth_tx_buf

[dpdk-dev] [PATCH] doc: add mempool mgr ABI deprication notice

2016-03-10 Thread David Hunt
Announce the ABI breakage due to addition of external mempool manager functionality which requires changes to rte_mempool structure. Signed-off-by: David Hunt --- doc/guides/rel_notes/deprecation.rst | 10 ++ 1 file changed, 10 insertions(+) diff --git a/doc/guides/rel_notes/deprecation

[dpdk-dev] [PATCH v5 2/2] i40evf: support to report pf reset event

2016-03-10 Thread Jingjing Wu
When Linux PF and DPDK VF are used for i40e PMD, In case of PF reset, interrupt will go via adminq event, VF need be informed the event, a callback mechanism is introduced by VF. This will allow VF to invoke callback when reset happens. Users can register a callback for this interrupt event like:

[dpdk-dev] [PATCH v5 1/2] i40evf: allocate virtchnl cmd buffer for each vf

2016-03-10 Thread Jingjing Wu
Currently, i40evf PMD uses a global static buffer to send virtchnl command to host driver. It is shared by multi VFs. This patch changed to allocate virtchnl cmd buffer for each VF. Signed-off-by: Jingjing Wu Acked-by: Helin Zhang --- drivers/net/i40e/i40e_ethdev.h| 2 + drivers/net/i40e/

[dpdk-dev] [PATCH v5 0/2] i40evf: pf reset event report

2016-03-10 Thread Jingjing Wu
v5 changes: - doc rewording and format fixing. v4 changes: - rebase on latest dpdk-next-net/rel_16_04 branch (commit 0f9564a0e4f2) v3 changes: - commit log doc rewording. - rebase on latest dpdk-next-net/rel_16_04 branch. - remove few useless line. - adjust interval and increase times for w

[dpdk-dev] [PATCH v3 0/2] add support for buffered tx to ethdev

2016-03-10 Thread Ananyev, Konstantin
> Many sample apps include internal buffering for single-packet-at-a-time > operation. Since this is such a common paradigm, this functionality is > better suited to being implemented in the ethdev API. > > The new APIs in the ethdev library are: > * rte_eth_tx_buffer_init - initialize buffer > *

[dpdk-dev] [PATCH v4 12/12] testpmd: extend flow director commands

2016-03-10 Thread Jingjing Wu
This patch extended commands for filter's input set changing. It added vlan as filter's input fields. Signed-off-by: Jingjing Wu Acked-by: Helin Zhang --- app/test-pmd/cmdline.c | 6 +++--- doc/guides/testpmd_app_ug/testpmd_funcs.rst | 2 +- 2 files changed, 4 insertions(+)

[dpdk-dev] [PATCH v4 11/12] i40e: extend flow director to filter by vlan id

2016-03-10 Thread Jingjing Wu
This patch extended flow director to select vlan id as filter's input set and program the filter rule with vlan id. Signed-off-by: Jingjing Wu Acked-by: Helin Zhang --- doc/guides/rel_notes/release_16_04.rst | 2 ++ drivers/net/i40e/i40e_ethdev.c | 11 drivers/net/i40e/i40e_fd

[dpdk-dev] [PATCH v4 10/12] i40e: fix VLAN bitmasks for input set

2016-03-10 Thread Jingjing Wu
From: Andrey Chilikin This patch adds missing VLAN bitmask for inner frame in case of tunneling and fixes VLAN tags bitmasks for single or outer frame in case of tunneling. Fixes: 98f055707685 ("i40e: configure input fields for RSS or flow director") Signed-off-by: Andrey Chilikin Signed-off-b

[dpdk-dev] [PATCH v4 09/12] testpmd: extend flow director commands

2016-03-10 Thread Jingjing Wu
This patch extended commands for filter's input set changing. It added GRE/Vxlan Tunnel as filter's input fields. Signed-off-by: Jingjing Wu Acked-by: Helin Zhang --- app/test-pmd/cmdline.c | 27 +-- doc/guides/testpmd_app_ug/testpmd_funcs.rst | 22 +

[dpdk-dev] [PATCH v4 08/12] i40e: extend flow director to filter by tunnel ID

2016-03-10 Thread Jingjing Wu
This patch extended flow director to select Vxlan/GRE tunnel ID as filter's input set and program the filter rule with the defined tunnel type. Signed-off-by: Jingjing Wu Acked-by: Helin Zhang --- drivers/net/i40e/i40e_ethdev.c | 11 +++ drivers/net/i40e/i40e_fdir.c | 150 +++

[dpdk-dev] [PATCH v4 07/12] librte_ether: extend flow director struct

2016-03-10 Thread Jingjing Wu
This patch changed rte_eth_fdir_flow from union to struct to support more packets formats, for example, Vxlan and GRE tunnel packets with IP inner frame. This patch also add new RTE_FDIR_TUNNEL_TYPE_GRE enum. Signed-off-by: Jingjing Wu Acked-by: Helin Zhang --- doc/guides/rel_notes/deprecation

[dpdk-dev] [PATCH v4 06/12] testpmd: extend input set related commands

2016-03-10 Thread Jingjing Wu
This patch extended commands for filter's input set changing. It added tos, protocol and ttl as filter's input fields, and remove the words selection from flex payloads for flow director. Signed-off-by: Jingjing Wu Acked-by: Helin Zhang --- app/test-pmd/cmdline.c | 100

[dpdk-dev] [PATCH v4 05/12] i40e: extend flow director to filter by IP Header

2016-03-10 Thread Jingjing Wu
This patch extended flow director to select more IP Header fields as filter input set. Signed-off-by: Jingjing Wu Acked-by: Helin Zhang --- drivers/net/i40e/i40e_ethdev.c | 69 ++ drivers/net/i40e/i40e_fdir.c | 26 +++- 2 files changed, 75 i

[dpdk-dev] [PATCH v4 04/12] i40e: restore default setting on input set

2016-03-10 Thread Jingjing Wu
This patch added a new function to set the input set to default when initialization. Signed-off-by: Jingjing Wu Acked-by: Helin Zhang --- drivers/net/i40e/i40e_ethdev.c | 56 ++ 1 file changed, 56 insertions(+) diff --git a/drivers/net/i40e/i40e_ethdev.c

[dpdk-dev] [PATCH v4 03/12] i40e: remove flex payload from input selection

2016-03-10 Thread Jingjing Wu
In this patch, flex payload is removed from valid fdir input set values. It is because all flex payload configuration can be set in struct rte_fdir_conf during device configure phase. And it is a more flexible configuration including flexpayload's selection, input set selection by word and mask set

[dpdk-dev] [PATCH v4 02/12] i40e: split function for hash and fdir input

2016-03-10 Thread Jingjing Wu
This patch split function for input set changing of hash and fdir to avoid multiple check on different situation. Signed-off-by: Jingjing Wu Acked-by: Helin Zhang --- drivers/net/i40e/i40e_ethdev.c | 235 + drivers/net/i40e/i40e_ethdev.h | 11 +- drivers

[dpdk-dev] [PATCH v4 01/12] ethdev: extend flow director for input selection

2016-03-10 Thread Jingjing Wu
This patch added RTE_ETH_INPUT_SET_L3_IP4_TTL, RTE_ETH_INPUT_SET_L3_IP6_HOP_LIMITS input field type and extended struct rte_eth_ipv4_flow and rte_eth_ipv6_flow to support filtering by tos, protocol and ttl. Signed-off-by: Jingjing Wu Acked-by: Helin Zhang --- lib/librte_ether/rte_eth_ctrl.h | 8

[dpdk-dev] [PATCH v4 00/12] extend flow director fields in i40e driver

2016-03-10 Thread Jingjing Wu
v4 changes: - rebase to latest dpdk-next-net/rel_16_04. - comments on new fields in API structure. v3 changes: - rebase to latest dpdk-next-net/rel_16_04(commit: 0f9564a0e4f2) - use AQ rx control register read/write for some registers - remove few useless lines - patch title rewording v2 ch

[dpdk-dev] [PATCH v4 0/3] clean-up on virtual PMDs

2016-03-10 Thread Bruce Richardson
On Fri, Feb 26, 2016 at 04:58:06PM +, Ferruh Yigit wrote: > This is a clean-up patch, no defect fixed, no functional difference > expected. > > Patch removes duplicated fields between data->dev_private > and data (struct rte_eth_dev_data) for pcap and null PMDs. > For ring, renames some varia

[dpdk-dev] [PATCH v4 2/3] ring: variable rename and code cleanup

2016-03-10 Thread Bruce Richardson
On Fri, Feb 26, 2016 at 04:58:08PM +, Ferruh Yigit wrote: > Rename nb_rx/tx_queues fields in internals struct to max_rx/tx_queues > Updated fields required to keep max queue numbers configured. For current > queue number requirements data->nb_rx/tx_queues fields used. > > Some checkpatch corre

[dpdk-dev] [PATCH v8 4/4] app/test-pmd: test tunnel filter for IP in GRE

2016-03-10 Thread Jingjing Wu
From: Xutao Sun This patch added some options in tunnel_filter command to test IP in GRE packet classification on i40e. Update the testpmd documentation. Signed-off-by: Xutao Sun Signed-off-by: Jijiang Liu --- app/test-pmd/cmdline.c | 32 +- doc/gu

[dpdk-dev] [PATCH v8 3/4] driver/i40e: implement tunnel filter for IP in GRE

2016-03-10 Thread Jingjing Wu
From: Xutao Sun Signed-off-by: Xutao Sun Signed-off-by: Jijiang Liu --- drivers/net/i40e/i40e_ethdev.c | 33 - 1 file changed, 24 insertions(+), 9 deletions(-) diff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c index 127ed72..3b9bf77 1

[dpdk-dev] [PATCH v8 2/4] lib/ether: add IP in GRE type

2016-03-10 Thread Jingjing Wu
From: Xutao Sun Signed-off-by: Xutao Sun Signed-off-by: Jijiang Liu --- lib/librte_ether/rte_eth_ctrl.h | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/librte_ether/rte_eth_ctrl.h b/lib/librte_ether/rte_eth_ctrl.h index 736cfc1..70635e9 100644 --- a/lib/librte_ether/rte_eth_ctrl.h +++

[dpdk-dev] [PATCH v8 1/4] lib/ether: optimize struct rte_eth_tunnel_filter_conf

2016-03-10 Thread Jingjing Wu
From: Xutao Sun Change the fields of outer_mac and inner_mac in struct rte_eth_tunnel_filter_conf from pointer to struct in order to keep the code's readability. Signed-off-by: Xutao Sun Signed-off-by: Jijiang Liu --- app/test-pmd/cmdline.c | 6 -- doc/guides/rel_notes/de

[dpdk-dev] [PATCH v8 0/4] This patch set adds tunnel filter support for IP in GRE on i40e.

2016-03-10 Thread Jingjing Wu
From: Xutao Sun v2 changes: Fix the byte order problem. v3 changes: Remove the deprecation notice and update the release notes. v4 changes: Modify the mistakes in cmdline.c in the old patch. v5 changes: Fix type errors and update the testpmd documentation. v6 changes: Use internal v

[dpdk-dev] [RFC 10/35] eal: introduce RTE_DECONST macro

2016-03-10 Thread Olivier MATZ
Hi Bruce, On 03/10/2016 10:26 AM, Bruce Richardson wrote: > On Thu, Mar 10, 2016 at 09:29:03AM +0100, Olivier MATZ wrote: If you have a better alternative, without duplicating the code, I'll be happy to learn. >>> >>> I really don't like this dropping of const either, but I do see the pr

[dpdk-dev] [PATCH, v2] mempool: avoid memory waste with large pagesize

2016-03-10 Thread Ferruh Yigit
On 3/10/2016 8:37 AM, Olivier MATZ wrote: > Hello, > > On 03/09/2016 10:12 PM, Stephen Hemminger wrote: >> If page size is large (like 64K on ARM) and object size is small >> then don't waste lots of memory by rounding up to page size. >> Instead, round up so that 1 or more objects all fit in a pa

[dpdk-dev] [PATCH 0/3] sched: patches for 2.2

2016-03-10 Thread Stephen Hemminger
Why does this need to be reassigned to Intel. That is not how the DPDK works. Please leave the original copyright holders on the file. On Thu, Mar 10, 2016 at 10:41 AM, Dumitrescu, Cristian < cristian.dumitrescu at intel.com> wrote: > > > > -Original Message- > > From: Stephen Hemminger [

[dpdk-dev] [PATCH v9 5/5] ixgbe: support VxLAN & NVGRE TX checksum off-load

2016-03-10 Thread Wenzhuo Lu
The patch add VxLAN & NVGRE TX checksum off-load. When the flag of outer IP header checksum offload is set, we'll set the context descriptor to enable this checksum off-load. Also update release note for VxLAN & NVGRE checksum off-load support and ABI change. Signed-off-by: Wenzhuo Lu Acked-by:

[dpdk-dev] [PATCH v9 4/5] ixgbe: support VxLAN & NVGRE RX checksum off-load

2016-03-10 Thread Wenzhuo Lu
X550 will do VxLAN & NVGRE RX checksum off-load automatically. This patch exposes the result of the checksum off-load. Signed-off-by: Wenzhuo Lu Acked-by: Konstantin Ananyev --- drivers/net/ixgbe/ixgbe_ethdev.c | 4 drivers/net/ixgbe/ixgbe_rxtx.c | 11 ++- lib/librte_ether/rte_e

[dpdk-dev] [PATCH v9 3/5] ixgbe: support UDP tunnel port config

2016-03-10 Thread Wenzhuo Lu
Add UDP tunnel port add/del support on ixgbe. Now only support VxLAN port configuration. Although according to the specification the VxLAN port has a default value 4789, it can be changed. We support VxLAN port configuration to meet the change. Note, the default value of VxLAN port in ixgbe NICs is

[dpdk-dev] [PATCH v9 2/5] i40e: rename the tunnel port config functions

2016-03-10 Thread Wenzhuo Lu
As the names of tunnel port config functions are not accurate, change them from tunnel_add/del to tunnel_port_add/del. And support both the old and new rte ops. Signed-off-by: Wenzhuo Lu Acked-by: Konstantin Ananyev --- drivers/net/i40e/i40e_ethdev.c | 22 -- 1 file changed,

[dpdk-dev] [PATCH v9 1/5] lib/librte_ether: change function name of tunnel port config

2016-03-10 Thread Wenzhuo Lu
The names of function for tunnel port configuration are not accurate. They're tunnel_add/del, better change them to tunnel_port_add/del. As it may be an ABI change if change the names directly, the new functions are added but not remove the old ones. The old ones will be removed in the next release

[dpdk-dev] [PATCH v9 0/5] Support VxLAN & NVGRE checksum off-load on X550

2016-03-10 Thread Wenzhuo Lu
This patch set add the VxLAN & NVGRE checksum off-load support. Both RX and TX checksum off-load can be used for VxLAN & NVGRE. And the VxLAN port can be set, it's implemented in this patch set either. v2: - Update release note. v3: - Update RX/TX offload capability. - Reuse PKT_RX_EIP_CKSUM_BAD

[dpdk-dev] [PATCH 1/2] bnx2x: fix error handling in bnx2x_loop_obtain_resources()

2016-03-10 Thread Bruce Richardson
On Fri, Mar 04, 2016 at 10:28:44PM +, Rasesh Mody wrote: > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Chas Williams > > Sent: Wednesday, December 30, 2015 4:38 PM > > > > From: "Charles (Chas) Williams" > > > > bnx2x_loop_obtain_resources() returns a struct containing the stat

[dpdk-dev] [PATCH 2/2] bnx2x: Determine rx/tx queue sizes sooner

2016-03-10 Thread Bruce Richardson
On Fri, Mar 04, 2016 at 10:28:57PM +, Rasesh Mody wrote: > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Chas Williams > > Sent: Wednesday, December 30, 2015 4:38 PM > > > > From: "Charles (Chas) Williams" > > > > The VF needs to determine the queues sizes before .dev_infos_get s

[dpdk-dev] [PATCH v7 1/2] cryptodev: API tidy and changes to support future extensions

2016-03-10 Thread Trahe, Fiona
Hi Thomas, > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Tuesday, March 08, 2016 2:11 PM > To: Trahe, Fiona > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v7 1/2] cryptodev: API tidy and changes to > support future extensions > > Hi, >

[dpdk-dev] [PATCH v7 2/2] cryptodev: change burst API to be crypto op oriented

2016-03-10 Thread Trahe, Fiona
Hi Thomas, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Trahe, Fiona > Sent: Wednesday, March 09, 2016 12:56 PM > To: Thomas Monjalon > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v7 2/2] cryptodev: change burst API to be > crypto > op oriented >

[dpdk-dev] [PATCH] virtio: fix rx ring descriptor starvation

2016-03-10 Thread Kyle Larose
On Wed, Mar 9, 2016 at 4:37 PM, Bruce Richardson wrote: > On Fri, Mar 04, 2016 at 08:25:07AM -0500, Kyle Larose wrote: >> On Fri, Mar 4, 2016 at 3:11 AM, Tom Kiely wrote: >> > Sure. >> >Tom >> > >> > >> > On 03/04/2016 06:16 AM, Xie, Huawei wrote: >> >> >> >> On 2/23/2016 12:23 AM, Tom Kiely

[dpdk-dev] [PATCH v2] virtio: fix rx ring descriptor starvation

2016-03-10 Thread Kyle Larose
Virtio has an mbuf descriptor ring containing mbufs to be used for receiving traffic. When the host queues traffic to be sent to the guest, it consumes these descriptors. If none exist, it discards the packet. The virtio pmd allocates mbufs to the descriptor ring every time it succesfully receives

[dpdk-dev] [PATCH, v2] mempool: avoid memory waste with large pagesize

2016-03-10 Thread Olivier MATZ
Hello, On 03/09/2016 10:12 PM, Stephen Hemminger wrote: > If page size is large (like 64K on ARM) and object size is small > then don't waste lots of memory by rounding up to page size. > Instead, round up so that 1 or more objects all fit in a page. > > This preserves the requirement that an obj

[dpdk-dev] [PATCH] eal: add option --avail-cores to detect lcores

2016-03-10 Thread Tan, Jianfeng
On 3/10/2016 3:33 AM, Ananyev, Konstantin wrote: > >> On 3/8/2016 4:54 PM, Panu Matilainen wrote: >>> On 03/04/2016 12:05 PM, Jianfeng Tan wrote: This patch adds option, --avail-cores, to use lcores which are available by calling pthread_geta

[dpdk-dev] [RFC 10/35] eal: introduce RTE_DECONST macro

2016-03-10 Thread Olivier MATZ
>> If you have a better alternative, without duplicating the code, >> I'll be happy to learn. > > I really don't like this dropping of const either, but I do see the problem. > I'd nearly rather see two copies of the function than start dropping the const > in such a way. I don't think duplicatin

[dpdk-dev] [RFC 10/35] eal: introduce RTE_DECONST macro

2016-03-10 Thread Bruce Richardson
On Thu, Mar 10, 2016 at 09:29:03AM +0100, Olivier MATZ wrote: > >> If you have a better alternative, without duplicating the code, > >> I'll be happy to learn. > > > > I really don't like this dropping of const either, but I do see the problem. > > I'd nearly rather see two copies of the function

[dpdk-dev] [RFC 10/35] eal: introduce RTE_DECONST macro

2016-03-10 Thread Olivier MATZ
> I would rather have the mempool_audit code take a non-const argument. > The macro method sets a bad precedent and will encourage more bad code. > Plus code checkers are likely to flag any such usage as suspect. Doing that would imply dropping the const qualifier in several functions: - rte_mem

[dpdk-dev] [PATCH] example/ipsec-secgw: ipsec security gateway

2016-03-10 Thread Jerin Jacob
On Wed, Mar 09, 2016 at 11:54:50PM +, Sergio Gonzalez Monroy wrote: > On 01/02/2016 11:26, Jerin Jacob wrote: > >On Mon, Feb 01, 2016 at 11:09:16AM +, Sergio Gonzalez Monroy wrote: > >>On 31/01/2016 14:39, Jerin Jacob wrote: > >>>On Fri, Jan 29, 2016 at 08:29:12PM +, Sergio Gonzalez Mon

<    1   2   3   >