[dpdk-dev] [PATCH v2] I217 and I218 changes

2016-03-01 Thread Bruce Richardson
On Tue, Mar 01, 2016 at 03:37:33AM -0800, Ravi Kerur wrote: > On Mon, Feb 29, 2016 at 9:18 PM, Lu, Wenzhuo wrote: > > > Hi, > > > > > -Original Message- > > > From: Ravi Kerur [mailto:rkerur at gmail.com] > > > Sent: Tuesday, March 1, 2016 2:31 AM > > > To: Lu, Wenzhuo; dev at dpdk.org >

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-03-01 Thread Bruce Richardson
On Mon, Feb 29, 2016 at 08:33:25AM -0600, Jay Rolette wrote: > On Mon, Feb 29, 2016 at 5:06 AM, Thomas Monjalon 6wind.com> > wrote: > > > Hi, > > I totally agree with Avi's comments. > > This topic is really important for the future of DPDK. > > So I think we must give some time to continue the d

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

2016-03-01 Thread Tan, Jianfeng
Hi Konstantin, On 3/1/2016 9:51 PM, Ananyev, Konstantin wrote: > Hi Jianfeng, > >> -Original Message- >> From: Tan, Jianfeng >> Sent: Tuesday, March 01, 2016 1:24 AM >> To: dev at dpdk.org >> Cc: Zhang, Helin; Ananyev, Konstantin; nelio.laranjeiro at 6wind.com; >> adrien.mazarguil at 6win

[dpdk-dev] New driver (large patch) question.

2016-03-01 Thread Stephen Hurd
I submitted a new driver on Friday, and it was rejected for being over 300k. The rejection email suggested contacing dev-owner at dpdk.org, which I did on Monday with no reply. What's the process to submit patches larger than the mailing list size limit? -- Stephen Hurd Principal Engineer - Sof

[dpdk-dev] ixgbe TX function selection

2016-03-01 Thread Zoltan Kiss
Hi, I've noticed that ixgbe_set_tx_function() selects the non-SG function even if (dev->data->scattered_rx == 1). That seems a bit dangerous, as you can turn that on inadvertently when you don't set max_rx_pkt_len and buffer size in certain ways. I've learnt it in the hard way, as my segmented

[dpdk-dev] [PATCH v4 3/3] maintainers: claim responsibility for arm64 specific files of hash and lpm

2016-03-01 Thread Thomas Monjalon
2016-02-12 17:58, Jerin Jacob: > +F: lib/librte_lpm/rte_lpm_neon.h This line should be in the previous patch. > +F: lib/librte_hash/rte_crc_arm64.h > +F: lib/librte_hash/rte_cmp_arm64.h Yes, hash for ARM was forgotten. Please add a Fixes: line to refer to the arm enablement of librte_hash.

[dpdk-dev] [PATCH v4 2/3] lpm: add support for NEON

2016-03-01 Thread Thomas Monjalon
2016-02-12 17:58, Jerin Jacob: > # fails to compile on ARM > -CONFIG_RTE_LIBRTE_LPM=n > -CONFIG_RTE_LIBRTE_TABLE=n > -CONFIG_RTE_LIBRTE_PIPELINE=n The associated examples cannot compile. Maybe it's too early to enable them. What about updating the comment to state that only examples fail? > ---

[dpdk-dev] [PATCH v4 1/3] lpm: make rte_lpm_lookupx4 API definition architecture agnostic

2016-03-01 Thread Thomas Monjalon
2016-02-12 17:58, Jerin Jacob: > -Used architecture agnostic xmm_t to represent 128 bit SIMD variable > > -Introduced vect_* API abstraction in app/test to test rte_lpm_lookupx4 > API in architecture agnostic way > > -Moved rte_lpm_lookupx4 SSE implementation to architecture specific > rte_lpm_s

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-03-01 Thread Stephen Hemminger
On Mon, 29 Feb 2016 08:33:25 -0600 Jay Rolette wrote: > On Mon, Feb 29, 2016 at 5:06 AM, Thomas Monjalon 6wind.com> > wrote: > > > Hi, > > I totally agree with Avi's comments. > > This topic is really important for the future of DPDK. > > So I think we must give some time to continue the discus

[dpdk-dev] Multi Queue Support With Mellanox 40GbE NIC - MT27500 Family [ConnectX-3]

2016-03-01 Thread Suresh Paruchuri
Hi, We are planning to use the Multi Queue features of Mellanox NIC by using RSS to distribute packets among the multiple RX queues. We have Mellanox ConnectX-3 NIC. Seems there are lot of limitations in using RSS as there is no support for Hash Key and function update with ConnectX-3 series

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

2016-03-01 Thread Jan Medala
This is a PMD for the Amazon ethernet ENA family. The driver operates variety of ENA adapters through feature negotiation with the adapter and upgradable commands set. ENA driver handles PCI Physical and Virtual ENA functions. Signed-off-by: Evgeny Schemeilin Signed-off-by: Jan Medala Signed-off

[dpdk-dev] [PATCH v4 3/4] ena: Amazon ENA communication layer for DPDK platform

2016-03-01 Thread Jan Medala
Implementation of platform specific code for ENA communication layer. Signed-off-by: Evgeny Schemeilin Signed-off-by: Jan Medala Signed-off-by: Jakub Palider --- drivers/net/ena/base/ena_plat_dpdk.h | 208 +++ 1 file changed, 208 insertions(+) create mode 10064

[dpdk-dev] [PATCH v4 2/4] ena: Amazon ENA communication layer

2016-03-01 Thread Jan Medala
Low level common abstraction for ENA device communication. Signed-off-by: Netanel Belgazal Signed-off-by: Jan Medala Signed-off-by: Jakub Palider --- drivers/net/ena/base/ena_com.c | 2750 drivers/net/ena/base/ena_com.h | 1038 ++

[dpdk-dev] [PATCH v4 1/4] ena: Amazon ENA documentation

2016-03-01 Thread Jan Medala
Signed-off-by: Evgeny Schemeilin Signed-off-by: Alexander Matushevsky Signed-off-by: Jan Medala Signed-off-by: Jakub Palider --- MAINTAINERS | 8 ++ doc/guides/nics/ena.rst | 252 ++ doc/guides/nics/index.rst | 1 + 3 files change

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

2016-03-01 Thread Jan Medala
This drop includes additional features for Amazon ENA: * Low Latenycy Queue (LLQ) for Tx * RSS and resolved previous issues: * Improved doc * Improved style according to checkpatch script * Fixed build problems on: i686, clang, +shared, +debug Jan Medala (4): ena: Amazon ENA documentation ena:

[dpdk-dev] [RFC] avoid testpmd only working in XEN

2016-03-01 Thread Christian Ehrhardt
With LIBRTE_PMD_XENVIRT enabled testpmd is built in a way to ONLY work in XEN environments. It will surface as: PMD: gntalloc: ioctl error EAL: Error - exiting with code: 1 Cause: Creation of mbuf pool for socket 0 failed There could be a complex solution checking if this is a Xen env v

[dpdk-dev] [PATCH] app/test: fix qat autotest failure

2016-03-01 Thread Trahe, Fiona
> -Original Message- > From: Jain, Deepak K > Sent: Tuesday, March 01, 2016 4:55 PM > To: dev at dpdk.org > Cc: Griffin, John; Trahe, Fiona; Jain, Deepak K > Subject: [PATCH] app/test: fix qat autotest failure > > This patch fix the QAT autotest failure when run for multiple times. > it

[dpdk-dev] port 0 is not present on the board

2016-03-01 Thread De Lara Guarch, Pablo
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Posadas, Emerson > Sent: Tuesday, March 01, 2016 4:43 PM > To: Victor Detoni > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] port 0 is not present on the board > > Hi Victor > > Yes, the devices are already b

[dpdk-dev] rte_mbuf's packet_type field

2016-03-01 Thread Zoltan Kiss
Hi, I have a quick question about this field: how do I know if the underlying PMD supports a particular protocol parsing. Let's say I want to check for SCTP packets, looking at this field tells me EITHER the packet is SCTP (or not), OR that the hardware has no idea about SCTP. Is there a way t

[dpdk-dev] [PATCH] app/test: fix qat autotest failure

2016-03-01 Thread Deepak Kumar JAIN
This patch fix the QAT autotest failure when run for multiple times. it was caused as mbuf was not freed. Fixes: 202d375c60b (app/test: add cryptodev unit and performance tests) This patch depends on following patch: cryptodev API changes http://dpdk.org/ml/archives/dev/2016-February/034212.html

[dpdk-dev] [PATCH v2] mk: replace the combined library with a linker script

2016-03-01 Thread Panu Matilainen
On 03/01/2016 03:40 PM, Thomas Monjalon wrote: > ping > I would like to be sure nothing is forgotten in this new revision. Sorry, didn't realize you were waiting for input from me, it feels a bit strange to comment on something supposedly coming from myself :) > 2016-02-23 23:20, Thomas Monjalon

[dpdk-dev] port 0 is not present on the board

2016-03-01 Thread Posadas, Emerson
Hi Victor Yes, the devices are already binded to the DPDK driver: I have just found that my devices are not supported by DPDK, this is probably the reason l3fwd is failing. Thanks anyways EP From: Victor Detoni [mailto:victordet...@gmail.com] Sent: Tuesday, March 1, 2016 10:31 AM To: Posada

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

2016-03-01 Thread Xutao Sun
This patch add 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/guides/testpmd_app_ug

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

2016-03-01 Thread 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 7c22358..237c735 100644 --- a/driver

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

2016-03-01 Thread 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 30cbde7..0e948a1 100644 --- a/lib/librte_ether/rte_eth_ctrl.h +++ b/lib/librte_ether

[dpdk-dev] [PATCH v7 1/4] lib/ether: optimize the'rte_eth_tunnel_filter_conf' structure

2016-03-01 Thread Xutao Sun
Change the fields of outer_mac and inner_mac 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/deprecation.rst | 5 - doc/guides/rel_notes/releas

[dpdk-dev] [PATCH v7 0/4] Add tunnel filter support for IP in GRE on i40e

2016-03-01 Thread Xutao Sun
This patch set adds tunnel filter support for IP in GRE on i40e. 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 test

[dpdk-dev] port 0 is not present on the board

2016-03-01 Thread Posadas, Emerson
I see that this issue was raised a couple of months ago on this thread (http://dpdk.org/ml/archives/dev/2014-December/010661.html) But seems that the fix on that thread does not work on my DPDK setup using l3fwd application. The full output I'm getting looks as follows. # ./examples/l3fwd/build

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

2016-03-01 Thread Tan, Jianfeng
On 3/1/2016 2:29 PM, Panu Matilainen wrote: > On 02/29/2016 06:41 PM, Tan, Jianfeng wrote: >> Hi Panu, >> >> On 2/29/2016 7:34 PM, Panu Matilainen wrote: > [...] >>> >>> More importantly, to export a function you need to add an entry for it >>> in rte_ether_version.map. >> >> Oh, yes, thanks for

[dpdk-dev] [PATCH v3 2/4] kcp: add kernel control path kernel module

2016-03-01 Thread Ferruh Yigit
On 3/1/2016 1:02 AM, Stephen Hemminger wrote: > On Fri, 26 Feb 2016 14:10:39 + > Ferruh Yigit wrote: > >> +#define KCP_ERR(args...) printk(KERN_ERR "KCP: " args) >> +#define KCP_INFO(args...) printk(KERN_INFO "KCP: " args) >> + >> +#ifdef RTE_KCP_KO_DEBUG >> +#define KCP_DBG(args...) printk(K

[dpdk-dev] [PATCH v4 4/4] examples/ethtool: add control interface support to the application

2016-03-01 Thread Ferruh Yigit
Control interface APIs added into the sample application. To have the support corresponding kernel module (KCP) needs to be inserted. If kernel module is not there, application will run as it is without kernel control path support. When KCP module inserted, running application creates a virtual L

[dpdk-dev] [PATCH v4 3/4] rte_ctrl_if: add control interface library

2016-03-01 Thread Ferruh Yigit
This library gets control messages form kernelspace and forwards them to librte_ether and returns response back to the kernelspace. Library does: 1) Trigger Linux virtual interface creation 2) Initialize the netlink socket communication 3) Provides process() API to the application that does proces

[dpdk-dev] [PATCH v4 2/4] kcp: add kernel control path kernel module

2016-03-01 Thread Ferruh Yigit
This kernel module is based on KNI module, but this one is stripped version of it and only for control messages, no data transfer functionality provided. This Linux kernel module helps userspace application create virtual interfaces and when a control command issued into that virtual interface, mo

[dpdk-dev] [PATCH v4 1/4] lib/librte_ethtool: move librte_ethtool form examples to lib folder

2016-03-01 Thread Ferruh Yigit
With KCP, examples/ethtool/lib/ethtool has two users, to prevent code dublication, moving library from examples folder into lib/ folder. Signed-off-by: Ferruh Yigit Acked-by: Remy Horton --- v4: * No update --- config/common_linuxapp | 5 + doc/api/doxy-api-index.md

[dpdk-dev] [PATCH v4 0/4] Use common Linux tools to control DPDK ports

2016-03-01 Thread Ferruh Yigit
This work is to make DPDK ports more visible and to enable using common Linux tools to configure DPDK ports. Patch is based on KNI but contains only control functionality of it, also this patch does not include any Linux kernel network driver as part of it. Basically with the help of a kernel mod

[dpdk-dev] [PATCH v2] virtio: Use cpuflag for vector api

2016-03-01 Thread Santosh Shukla
On Tue, Mar 1, 2016 at 12:02 PM, Yuanhan Liu wrote: > On Tue, Mar 01, 2016 at 11:38:55AM +0530, Santosh Shukla wrote: >> On Tue, Mar 1, 2016 at 11:29 AM, Yuanhan Liu >> wrote: >> > On Mon, Feb 29, 2016 at 06:28:10PM +0530, Santosh Shukla wrote: >> >> Check cpuflag macro before using vectored api.

[dpdk-dev] [PATCH v1 3/3] guide/release: add virtio for arm feature info

2016-03-01 Thread Santosh Shukla
Signed-off-by: Santosh Shukla --- doc/guides/rel_notes/release_16_04.rst |5 + 1 file changed, 5 insertions(+) diff --git a/doc/guides/rel_notes/release_16_04.rst b/doc/guides/rel_notes/release_16_04.rst index 8273817..fc0b3bd 100644 --- a/doc/guides/rel_notes/release_16_04.rst +++ b/do

[dpdk-dev] [PATCH v1 2/3] config: enable virtio for armv7/v8

2016-03-01 Thread Santosh Shukla
removed _VIRTIO_PMD=n from arch config and let arch to use _VIRTIO_PMD from config/common_linuxapp. Signed-off-by: Santosh Shukla --- config/defconfig_arm-armv7a-linuxapp-gcc |1 - config/defconfig_arm64-armv8a-linuxapp-gcc |1 - 2 files changed, 2 deletions(-) diff --git a/config/def

[dpdk-dev] [PATCH v1 1/3] virtio: use vector rx/tx for ssse cpuflag only

2016-03-01 Thread Santosh Shukla
Temporary implementation to let virtio operate in non-vec mode for archs which doesn't support _ssse_ cpuflag. todo: 1) Move virtio_recv_pkts_vec() implementation to drivers/virtio/virtio_vec_.h file. 2) Remove use_simple_rxtx flag, so that virtio/virtio_vec_.h files to provide vectored/non-

[dpdk-dev] [PATCH v1 0/3] virtio vector and misc

2016-03-01 Thread Santosh Shukla
- 1st patch: let non-x86 arch use virtio pmd driver in non-vec - 2nd patch: enable virtio arm support - 3rd patch: update virtio for arm feature entry in release guide. Thanks. Santosh Shukla (3): virtio: use vector rx/tx for ssse cpuflag only config: enable virtio for armv7/v8 guide/releas

[dpdk-dev] [PATCH v1] virtio: Use cpuflag for vector api

2016-03-01 Thread Santosh Shukla
On Tue, Mar 1, 2016 at 2:41 PM, Qiu, Michael wrote: > On 2/26/2016 4:53 PM, Santosh Shukla wrote: >> Check cpuflag macro before using vectored api. >> -virtio_recv_pkts_vec() uses _sse3__ simd instruction for now so added >> cpuflag. >> - Also wrap other vectored freind api ie.. >> 1) virtqueue_e

[dpdk-dev] [PATCH v2] doc: introduce networking driver matrix

2016-03-01 Thread Thomas Monjalon
2016-03-01 14:53, Matej Vido: > D?a 27.01.2016 o 21:07 Thomas Monjalon nap?sal(a): > > + multiprocess aware > Could you please clarify what was meant by this "multiprocess aware" field? The short answer is "git grep RTE_PROC -- drivers/" Some initializations must be done only in primay process.

[dpdk-dev] [PATCH v4 2/4] kcp: add kernel control path kernel module

2016-03-01 Thread Stephen Hemminger
On Tue, 1 Mar 2016 15:41:58 + Ferruh Yigit wrote: > +struct kcp_ethtool_msg { > + int cmd_id; > + int port_id; > + unsigned int flag; > + char input_buffer[KCP_ETHTOOL_MSG_LEN]; > + char output_buffer[KCP_ETHTOOL_MSG_LEN]; > + int input_buffer_len; > + int output_

[dpdk-dev] [PATCH v4 2/4] kcp: add kernel control path kernel module

2016-03-01 Thread Stephen Hemminger
On Tue, 1 Mar 2016 15:41:58 + Ferruh Yigit wrote: > + > +static int kcp_net_set_mac(struct net_device *dev, void *p) > +{ > + struct sockaddr *addr = p; > + int err = 0; Unnecessary assignment. > + > + if (!is_valid_ether_addr((unsigned char *)(addr->sa_data))) No cast needed:

[dpdk-dev] [PATCH v4 2/4] kcp: add kernel control path kernel module

2016-03-01 Thread Stephen Hemminger
On Tue, 1 Mar 2016 15:41:58 + Ferruh Yigit wrote: > +#ifdef RTE_KCP_KO_DEBUG > +#define KCP_DBG(args...) pr_info(args) > +#else > +#define KCP_DBG(args...) > +#en Why not use pr_debug() which is compile enabled already and supports dynamic enabling as well.

[dpdk-dev] [PATCH v2] doc: introduce networking driver matrix

2016-03-01 Thread Matej Vido
Hi Thomas, D?a 27.01.2016 o 21:07 Thomas Monjalon nap?sal(a): > In order to better compare the drivers and check what is missing > for a common baseline, we need to fill a matrix. > > A CSS trick is used to fit the HTML page. > The PDF output needs some LaTeX wizardry. > > Signed-off-by: Thomas Mo

[dpdk-dev] [PATCH v2] mk: replace the combined library with a linker script

2016-03-01 Thread Thomas Monjalon
ping I would like to be sure nothing is forgotten in this new revision. 2016-02-23 23:20, Thomas Monjalon: > From: Panu Matilainen > > The physically linked-together combined library has been an increasing > source of problems, as was predicted when library and symbol versioning > was introduced

[dpdk-dev] [PATCH v4 1/3] fm10k: enable FTAG based forwarding

2016-03-01 Thread Stephen Hemminger
On Tue, 1 Mar 2016 13:36:39 +0800 Wang Xiao W wrote: > > +static int > +fm10k_check_ftag(struct rte_devargs *devargs) > +{ > + if (devargs == NULL) > + return 0; > + > + if (strstr(devargs->args, "enable_ftag=1") == NULL) > + return 0; > + > + return 1; > +}

[dpdk-dev] [PATCH v2] virtio: Use cpuflag for vector api

2016-03-01 Thread Yuanhan Liu
On Tue, Mar 01, 2016 at 11:38:55AM +0530, Santosh Shukla wrote: > On Tue, Mar 1, 2016 at 11:29 AM, Yuanhan Liu > wrote: > > On Mon, Feb 29, 2016 at 06:28:10PM +0530, Santosh Shukla wrote: > >> Check cpuflag macro before using vectored api. > >> -virtio_recv_pkts_vec() uses _sse3__ simd instruction

[dpdk-dev] [PATCH v3 0/2] Fix CRC32c computation

2016-03-01 Thread Thomas Monjalon
> > CRC32c computation is not valid when buffer length is not a multiple of 4 > > bytes. > > Values returned by rte_hash_crc functions does not match the one > > computed by a trivial crc32c implementation. > > > > First patch fixes crc hash function autotests, to outline the problem. > > Second p

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

2016-03-01 Thread Ananyev, Konstantin
> >> + > >> +void > >> +lpm_parse_ptype(struct rte_mbuf *m) > >> +{ > >> + struct ether_hdr *eth_hdr; > >> + uint32_t packet_type = 0; > >> + uint16_t ethertype; > >> + > >> + eth_hdr = rte_pktmbuf_mtod(m, struct ether_hdr *); > >> + ethertype = rte_be_to_cpu_16(eth_hdr->ether_type); > >> + s

[dpdk-dev] [PATCH v1] virtio: Use cpuflag for vector api

2016-03-01 Thread Yuanhan Liu
On Tue, Mar 01, 2016 at 11:40:41AM +0530, Santosh Shukla wrote: > On Tue, Mar 1, 2016 at 11:25 AM, Yuanhan Liu > wrote: > > On Mon, Feb 29, 2016 at 06:01:38PM +0530, Santosh Shukla wrote: > >> On Mon, Feb 29, 2016 at 9:57 AM, Yuanhan Liu > >> wrote: > >> > On Fri, Feb 26, 2016 at 02:21:02PM +0530

[dpdk-dev] [PATCH v2 0/7] vhost rxtx refactor

2016-03-01 Thread Yuanhan Liu
On Mon, Feb 29, 2016 at 05:06:27PM +0100, Thomas Monjalon wrote: > Hi Yuanhan > > 2016-02-18 21:49, Yuanhan Liu: > > Here is a patchset for refactoring vhost rxtx code, mainly for > > improving readability. > > This series requires to be rebased. > > And maybe you could check also the series abo

[dpdk-dev] [PATCH v2] virtio: Use cpuflag for vector api

2016-03-01 Thread Yuanhan Liu
On Mon, Feb 29, 2016 at 06:28:10PM +0530, Santosh Shukla wrote: > Check cpuflag macro before using vectored api. > -virtio_recv_pkts_vec() uses _sse3__ simd instruction for now so added > cpuflag. > - Also wrap other vectored freind api ie.. > 1) virtqueue_enqueue_recv_refill_simple > 2) virtio_rx

[dpdk-dev] [PATCH] hash: fix memcmp function pointer in multi-process

2016-03-01 Thread Dhananjaya Reddy Eadala
Hi We found a problem in dpdk-2.2 using under multi-process environment. Here is the brief description how we are using the dpdk: We have two processes proc1, proc2 using dpdk. These proc1 and proc2 are two different compiled binaries. proc1 is started as primary process and proc2 as secondary pr

[dpdk-dev] [PATCH v1] virtio: Use cpuflag for vector api

2016-03-01 Thread Yuanhan Liu
On Mon, Feb 29, 2016 at 06:01:38PM +0530, Santosh Shukla wrote: > On Mon, Feb 29, 2016 at 9:57 AM, Yuanhan Liu > wrote: > > On Fri, Feb 26, 2016 at 02:21:02PM +0530, Santosh Shukla wrote: > >> Check cpuflag macro before using vectored api. > >> -virtio_recv_pkts_vec() uses _sse3__ simd instruction

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

2016-03-01 Thread Ananyev, Konstantin
Hi Jianfeng, > -Original Message- > From: Tan, Jianfeng > Sent: Tuesday, March 01, 2016 1:24 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, Ji

[dpdk-dev] [PATCH v4 3/3] doc: update release note for fm10k FTAG support

2016-03-01 Thread Wang Xiao W
Update the 16_04 release note. Signed-off-by: Wang Xiao W --- doc/guides/rel_notes/release_16_04.rst | 2 ++ 1 file changed, 2 insertions(+) diff --git a/doc/guides/rel_notes/release_16_04.rst b/doc/guides/rel_notes/release_16_04.rst index 73494f9..c3ee8b1 100644 --- a/doc/guides/rel_notes/rel

[dpdk-dev] [PATCH v4 2/3] doc: add introduction for fm10k FTAG based forwarding

2016-03-01 Thread Wang Xiao W
Add a brief introduction on FTAG, describe what's FTAG and how it works in forwarding. Signed-off-by: Wang Xiao W --- doc/guides/nics/fm10k.rst | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/doc/guides/nics/fm10k.rst b/doc/guides/nics/fm10k.rst index dc5cb6e

[dpdk-dev] [PATCH v4 1/3] fm10k: enable FTAG based forwarding

2016-03-01 Thread Wang Xiao W
This patch enables reading sglort info into mbuf for RX and inserting an FTAG at the beginning of the packet for TX. The vlan_tci_outer field selected from rte_mbuf structure for sglort is not used in fm10k now. In FTAG based forwarding mode, the switch will forward packets according to glort info

[dpdk-dev] [PATCH v4 0/3] fm10k: enable FTAG based forwarding

2016-03-01 Thread Wang Xiao W
v4: * Removed the build time config option, used devargs to config FTAG. * Rebased on head of dpdk-next-net/rel_16_04 branch. v3: * Removed "\n" in PMD_INIT_LOG. * Returned "-ENOTSUP" instead of -1 in VF FTAG use case. v2: * Gave an error message for VF FTAG use case. * Added a notice in the doc

[dpdk-dev] [PATCH 1/5] mempool: add external mempool manager support

2016-03-01 Thread Hunt, David
Olivier, Here's my comments on your feedback. Hopefully I've covered all of it this time, and I've summarised the outstanding questions at the bottom. On 2/4/2016 2:52 PM, Olivier MATZ wrote: > >> -#ifndef RTE_LIBRTE_XEN_DOM0 >> -/* stub if DOM0 support not configured */ >> -struct rte_mempo

[dpdk-dev] port 0 is not present on the board

2016-03-01 Thread Victor Detoni
Did you try to bind nics through /tools/dpdk_nic_bind.py script? Em ter?a-feira, 1 de mar?o de 2016, Posadas, Emerson < emerson.posadas at intel.com> escreveu: > I see that this issue was raised a couple of months ago on this thread ( > http://dpdk.org/ml/archives/dev/2014-December/010661.html)

[dpdk-dev] [PATCH 3/8] drivers/net/e1000: Fix missing brackets

2016-03-01 Thread Panu Matilainen
On 02/26/2016 03:13 PM, Aaron Conole wrote: > Hi Wenzhou, > > "Lu, Wenzhuo" writes: > >> Hi Aaron, >> >> >>> -Original Message- >>> From: Aaron Conole [mailto:aconole at redhat.com] >>> Sent: Friday, February 26, 2016 2:49 AM >>> To: dev at dpdk.org >>> Cc: Lu, Wenzhuo; Zhang, Helin; Anany

[dpdk-dev] [PATCH v2 4/6] bond mode 4: allow external state machine

2016-03-01 Thread Eric Kinzie
On Thu Feb 25 15:22:35 + 2016, Iremonger, Bernard wrote: > Hi Eric, > > > > > @@ -157,6 +159,7 @@ struct rte_eth_bond_8023ad_conf { > > > uint32_t tx_period_ms; > > > uint32_t rx_marker_period_ms; > > > uint32_t update_timeout_ms; > > > + rte_eth_bond_8023ad_ext_

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

2016-03-01 Thread Xutao Sun
This patch add 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/guides/testpmd_app_ug/

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

2016-03-01 Thread Xutao Sun
Signed-off-by: Xutao Sun Signed-off-by: Jijiang Liu --- drivers/net/i40e/i40e_ethdev.c | 37 +++-- 1 file changed, 27 insertions(+), 10 deletions(-) diff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c index 7c22358..6d860ee 100644 --- a/d

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

2016-03-01 Thread 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 30cbde7..0e948a1 100644 --- a/lib/librte_ether/rte_eth_ctrl.h +++ b/lib/librte_ether

[dpdk-dev] [PATCH v6 1/4] lib/ether: optimize the'rte_eth_tunnel_filter_conf' structure

2016-03-01 Thread Xutao Sun
Change the fields of outer_mac and inner_mac 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/deprecation.rst | 5 - doc/guides/rel_notes/releas

[dpdk-dev] [PATCH v6 0/4] Add tunnel filter support for IP in GRE on i40e

2016-03-01 Thread Xutao Sun
This patch set adds tunnel filter support for IP in GRE on i40e. 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 test

[dpdk-dev] [PATCH v3] af_packet: make the device detachable

2016-03-01 Thread Panu Matilainen
On 02/29/2016 08:22 PM, Wojciech ?muda wrote: > Hi Bernard, > >> Does making rte_pmd_af_packet_devinit local result in an ABI breakage? > If someone uses it in their app, they'll be forced to change it. > However, as this function is not intentionally public and there is API > to create devices t

[dpdk-dev] [PATCH v1] virtio: Use cpuflag for vector api

2016-03-01 Thread Santosh Shukla
On Tue, Mar 1, 2016 at 11:25 AM, Yuanhan Liu wrote: > On Mon, Feb 29, 2016 at 06:01:38PM +0530, Santosh Shukla wrote: >> On Mon, Feb 29, 2016 at 9:57 AM, Yuanhan Liu >> wrote: >> > On Fri, Feb 26, 2016 at 02:21:02PM +0530, Santosh Shukla wrote: >> >> Check cpuflag macro before using vectored api.

[dpdk-dev] [PATCH v2] virtio: Use cpuflag for vector api

2016-03-01 Thread Santosh Shukla
On Tue, Mar 1, 2016 at 11:29 AM, Yuanhan Liu wrote: > On Mon, Feb 29, 2016 at 06:28:10PM +0530, Santosh Shukla wrote: >> Check cpuflag macro before using vectored api. >> -virtio_recv_pkts_vec() uses _sse3__ simd instruction for now so added >> cpuflag. >> - Also wrap other vectored freind api ie

[dpdk-dev] [PATCH v9 0/2] Add VHOST PMD

2016-03-01 Thread Tetsuya Mukawa
On 2016/03/01 11:00, Qiu, Michael wrote: > On 2/26/2016 4:36 PM, Tetsuya Mukawa wrote: >> On 2016/02/26 13:29, Tetsuya Mukawa wrote: >>> On 2016/02/25 16:51, Qiu, Michael wrote: On 2/24/2016 1:10 PM, Tetsuya Mukawa wrote: > On 2016/02/24 11:45, Qiu, Michael wrote: >> Hi, Tetsuya >

[dpdk-dev] [PATCH v4 1/3] fm10k: enable FTAG based forwarding

2016-03-01 Thread Wang, Xiao W
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Tuesday, March 1, 2016 3:36 PM > To: Wang, Xiao W > Cc: dev at dpdk.org; Chen, Jing D > Subject: Re: [dpdk-dev] [PATCH v4 1/3] fm10k: enable FTAG based forwarding > > 2016-03-01 13:36, Wang Xiao W

[dpdk-dev] [PATCH v4 4/4] virtio: return 1 to tell the upper layer we don't take over this device

2016-03-01 Thread Thomas Monjalon
2016-03-01 08:39, Xie, Huawei: > On 3/1/2016 4:24 PM, Thomas Monjalon wrote: > > 2016-03-01 07:53, Xie, Huawei: > >> On 3/1/2016 3:18 PM, Thomas Monjalon wrote: > >>> 2016-02-26 09:53, Huawei Xie: > @@ -1037,8 +1039,11 @@ eth_virtio_dev_init(struct rte_eth_dev *eth_dev) > >

[dpdk-dev] [PATCH v6 1/4] lib/ether: optimize the'rte_eth_tunnel_filter_conf' structure

2016-03-01 Thread Panu Matilainen
On 03/01/2016 05:47 AM, Xutao Sun wrote: > Change the fields of outer_mac and inner_mac 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

[dpdk-dev] [PATCH v2] examples/ip_pipeline: add link identification feature

2016-03-01 Thread Fan Zhang
This patch adds link identification feature to packet framework. To identify a link, user can use both existing port-mask option, or specify PCI device in every LINK section in the configuration file. Signed-off-by: Fan Zhang Acked-by: Cristian Dumitrescu --- *v2 To be applied on top of: [dpdk-d

[dpdk-dev] [PATCH v4 4/4] virtio: return 1 to tell the upper layer we don't take over this device

2016-03-01 Thread Xie, Huawei
On 3/1/2016 5:57 PM, Thomas Monjalon wrote: > 2016-03-01 08:39, Xie, Huawei: >> On 3/1/2016 4:24 PM, Thomas Monjalon wrote: >>> 2016-03-01 07:53, Xie, Huawei: On 3/1/2016 3:18 PM, Thomas Monjalon wrote: > 2016-02-26 09:53, Huawei Xie: >> @@ -1037,8 +1039,11 @@ eth_virtio_dev_init(struc

[dpdk-dev] [PATCH V3 4/4] bond: do not activate slave twice

2016-03-01 Thread Eric Kinzie
From: Eric Kinzie The current code for detecting link during slave addition can cause a slave interface to be activated twice -- once during slave_configure() and again at the end of __eth_bond_slave_add_lock_free(). This will either cause the active slave count to be incorrect or will cause the

[dpdk-dev] [PATCH V3 3/4] bond: active slaves with no primary

2016-03-01 Thread Eric Kinzie
From: Eric Kinzie If the link state of a slave is "up" when added, it is added to the list of active slaves but, even if it is the only slave, is not selected as the primary interface. Generally, handling of link state interrupts selects an interface to be primary, but only if the active count i

[dpdk-dev] [PATCH V3 2/4] bond mode 4: do not ignore multicast

2016-03-01 Thread Eric Kinzie
From: Eric Kinzie The bonding PMD in mode 4 puts all enslaved interfaces into promiscuous mode in order to receive LACPDUs and must filter unwanted packets after the traffic has been "collected". Allow broadcast and multicast through so that ARP and IPv6 neighbor discovery continue to work. Fix

[dpdk-dev] [PATCH V3 1/4] bond mode 4: copy entire config structure

2016-03-01 Thread Eric Kinzie
From: Eric Kinzie Copy all needed fields from the mode8023ad_private structure in bond_mode_8023ad_conf_get(). This help ensure that a subsequent call to rte_eth_bond_8023ad_setup() is not passed uninitialized data that would result in either incorrect behavior or a failed sanity check. Fixes:

[dpdk-dev] [PATCH V3 0/4] bonding: fixes and enhancements

2016-03-01 Thread Eric Kinzie
These are bug fixes and some small enhancements to allow bonding to work with external control (teamd). Please consider integrating these into DPDK 2.2 Changes in v2: - remove "bond: handle slaves with fewer queues than bonding device" - remove "bond: per-slave intermediate rx ring" Changes in v3

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

2016-03-01 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 v4 4/4] virtio: return 1 to tell the upper layer we don't take over this device

2016-03-01 Thread Thomas Monjalon
2016-03-01 07:53, Xie, Huawei: > On 3/1/2016 3:18 PM, Thomas Monjalon wrote: > > 2016-02-26 09:53, Huawei Xie: > >> @@ -1037,8 +1039,11 @@ eth_virtio_dev_init(struct rte_eth_dev *eth_dev) > >> > >>pci_dev = eth_dev->pci_dev; > >> > >> - if (vtpci_init(pci_dev, hw) < 0) > >> - retu

[dpdk-dev] [PATCH v1] virtio: Use cpuflag for vector api

2016-03-01 Thread Qiu, Michael
On 2/26/2016 4:53 PM, Santosh Shukla wrote: > Check cpuflag macro before using vectored api. > -virtio_recv_pkts_vec() uses _sse3__ simd instruction for now so added > cpuflag. > - Also wrap other vectored freind api ie.. > 1) virtqueue_enqueue_recv_refill_simple > 2) virtio_rxq_vec_setup > > todo

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

2016-03-01 Thread Thomas Monjalon
2016-03-01 08:29, Panu Matilainen: > DPDK_2.3 { > global: > > rte_eth_dev_get_ptype_info; > > local: *; > } DPDK_2.2; > > ...but if there are no other reasons to respin the series perhaps Thomas > can fixup that while applying. > > Then there's the actual version, which shoul

[dpdk-dev] [PATCH] eal: fix symbol map version number

2016-03-01 Thread Thomas Monjalon
The version 2.3 has been renamed 16.04. Fixes: 6d7de6d2e357 ("version: switch to year.month numbers") Reported-by: Panu Matilainen Signed-off-by: Thomas Monjalon --- lib/librte_eal/bsdapp/eal/rte_eal_version.map | 2 +- lib/librte_eal/linuxapp/eal/rte_eal_version.map | 2 +- 2 files changed,

[dpdk-dev] [PATCH v4 4/4] virtio: return 1 to tell the upper layer we don't take over this device

2016-03-01 Thread Xie, Huawei
On 3/1/2016 4:24 PM, Thomas Monjalon wrote: > 2016-03-01 07:53, Xie, Huawei: >> On 3/1/2016 3:18 PM, Thomas Monjalon wrote: >>> 2016-02-26 09:53, Huawei Xie: @@ -1037,8 +1039,11 @@ eth_virtio_dev_init(struct rte_eth_dev *eth_dev) pci_dev = eth_dev->pci_dev; - if (vt

[dpdk-dev] [PATCH v4 1/3] fm10k: enable FTAG based forwarding

2016-03-01 Thread Thomas Monjalon
2016-03-01 13:36, Wang Xiao W: > +static int > +fm10k_check_ftag(struct rte_devargs *devargs) > +{ > + if (devargs == NULL) > + return 0; > + > + if (strstr(devargs->args, "enable_ftag=1") == NULL) > + return 0; > + > + return 1; > +} With strstr(), ch

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

2016-03-01 Thread Panu Matilainen
On 02/29/2016 06:41 PM, Tan, Jianfeng wrote: > Hi Panu, > > On 2/29/2016 7:34 PM, Panu Matilainen wrote: [...] >> >> More importantly, to export a function you need to add an entry for it >> in rte_ether_version.map. > > Oh, yes, thanks for pointing out this, I'll change this and update > rte_ether

[dpdk-dev] [PATCH v4 4/4] virtio: return 1 to tell the upper layer we don't take over this device

2016-03-01 Thread Thomas Monjalon
Hi Huawei, 2016-02-26 09:53, Huawei Xie: > --- a/drivers/net/virtio/virtio_ethdev.c > +++ b/drivers/net/virtio/virtio_ethdev.c > @@ -1,4 +1,5 @@ > /*- > + This new line seems useless :) > * BSD LICENSE > * [...] > @@ -1037,8 +1039,11 @@ eth_virtio_dev_init(struct rte_eth_dev *eth_dev) >

[dpdk-dev] [PATCH v4 4/4] virtio: return 1 to tell the upper layer we don't take over this device

2016-03-01 Thread Xie, Huawei
On 3/1/2016 3:18 PM, Thomas Monjalon wrote: > Hi Huawei, > > 2016-02-26 09:53, Huawei Xie: >> --- a/drivers/net/virtio/virtio_ethdev.c >> +++ b/drivers/net/virtio/virtio_ethdev.c >> @@ -1,4 +1,5 @@ >> /*- >> + > This new line seems useless :) Sorry, would fix. > >> * BSD LICENSE >> * > [..

[dpdk-dev] [PATCH v2] I217 and I218 changes

2016-03-01 Thread Lu, Wenzhuo
Hi, > -Original Message- > From: Ravi Kerur [mailto:rkerur at gmail.com] > Sent: Tuesday, March 1, 2016 2:31 AM > To: Lu, Wenzhuo; dev at dpdk.org > Cc: Ravi Kerur > Subject: [PATCH v2] I217 and I218 changes > > v2: > Incorporate Wenzhou's comments > Compiled and tested (via t

[dpdk-dev] [PATCH v6 11/11] pmd/vmxnet3: add dev_ptype_info_get implementation

2016-03-01 Thread Jianfeng Tan
Signed-off-by: Jianfeng Tan --- drivers/net/vmxnet3/vmxnet3_ethdev.c | 16 1 file changed, 16 insertions(+) diff --git a/drivers/net/vmxnet3/vmxnet3_ethdev.c b/drivers/net/vmxnet3/vmxnet3_ethdev.c index c363bf6..ac120a1 100644 --- a/drivers/net/vmxnet3/vmxnet3_ethdev.c +++ b/dr

[dpdk-dev] [PATCH v6 10/11] pmd/nfp: add dev_ptype_info_get implementation

2016-03-01 Thread Jianfeng Tan
Signed-off-by: Jianfeng Tan --- drivers/net/nfp/nfp_net.c | 19 +++ 1 file changed, 19 insertions(+) diff --git a/drivers/net/nfp/nfp_net.c b/drivers/net/nfp/nfp_net.c index fd4dd39..5894a9d 100644 --- a/drivers/net/nfp/nfp_net.c +++ b/drivers/net/nfp/nfp_net.c @@ -1073,6 +1073,2

[dpdk-dev] [PATCH v6 09/11] pmd/mlx5: add dev_ptype_info_get implementation

2016-03-01 Thread Jianfeng Tan
Signed-off-by: Jianfeng Tan --- drivers/net/mlx5/mlx5.c| 1 + drivers/net/mlx5/mlx5.h| 1 + drivers/net/mlx5/mlx5_ethdev.c | 20 drivers/net/mlx5/mlx5_rxtx.c | 2 ++ 4 files changed, 24 insertions(+) diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx

[dpdk-dev] [PATCH v6 08/11] pmd/mlx4: add dev_ptype_info_get implementation

2016-03-01 Thread Jianfeng Tan
Signed-off-by: Jianfeng Tan --- drivers/net/mlx4/mlx4.c | 21 + 1 file changed, 21 insertions(+) diff --git a/drivers/net/mlx4/mlx4.c b/drivers/net/mlx4/mlx4.c index ee00151..58f4e1a 100644 --- a/drivers/net/mlx4/mlx4.c +++ b/drivers/net/mlx4/mlx4.c @@ -2835,6 +2835,8 @@ rxq_

[dpdk-dev] [PATCH v6 07/11] pmd/ixgbe: add dev_ptype_info_get implementation

2016-03-01 Thread Jianfeng Tan
Signed-off-by: Jianfeng Tan --- drivers/net/ixgbe/ixgbe_ethdev.c | 38 ++ drivers/net/ixgbe/ixgbe_ethdev.h | 2 ++ drivers/net/ixgbe/ixgbe_rxtx.c | 4 +++- 3 files changed, 43 insertions(+), 1 deletion(-) diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/dri

  1   2   >