[dpdk-dev] [dpdk-announce] technical board

2016-03-04 Thread Thomas Monjalon
There were some discussions to form a technical board for the DPDK: http://thread.gmane.org/gmane.comp.networking.dpdk.devel/26596 After a first meeting of this group, its scope and operation mode have been approved: http://dpdk.org/browse/tools/dpdk-web/commit/?id=5833c070 So it

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

2016-03-04 Thread Tetsuya Mukawa
On 2016/03/02 11:24, Qiu, Michael wrote: > On 3/1/2016 10:19 AM, Tetsuya Mukawa wrote: >> 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: > [...] > > BTW, I have set the frontend mergeable off. > I

[dpdk-dev] [PATCH] e1000: fix setting of VF MAC address

2016-03-04 Thread Lu, Wenzhuo
Hi, > > > Signed-off-by: Bernard Iremonger Acked-by: Wenzhuo Lu > > > --- > > > drivers/net/e1000/igb_ethdev.c | 12 +++- > > > drivers/net/e1000/igb_pf.c | 8 +--- > > > 2 files changed, 16 insertions(+), 4 deletions(-) > > > > > > diff --git a/drivers/net/e1000/igb_ethdev.c

[dpdk-dev] [PATCH v4 7/7] app/testpmd: add CLIs for E-tag operation

2016-03-04 Thread He, Shaopeng
Hi Wenzhuo, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Wenzhuo Lu > Sent: Thursday, February 18, 2016 10:46 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v4 7/7] app/testpmd: add CLIs for E-tag > operation > > Add the CLIs to support the E-tag

[dpdk-dev] [PATCH v4 6/7] ixgbe: support l2 tunnel operation

2016-03-04 Thread He, Shaopeng
Hi Wenzhuo, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Wenzhuo Lu > Sent: Thursday, February 18, 2016 10:46 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v4 6/7] ixgbe: support l2 tunnel operation > > Add support of l2 tunnel operation. >

[dpdk-dev] [PATCH v4 3/7] ixgbe: support l2 tunnel config

2016-03-04 Thread He, Shaopeng
Hi Wenzhuo, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Wenzhuo Lu > Sent: Thursday, February 18, 2016 10:46 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v4 3/7] ixgbe: support l2 tunnel config > > Add support of l2 tunnel configuration. >

[dpdk-dev] ixgbe TX function selection

2016-03-04 Thread Lu, Wenzhuo
Hi Zoltan, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Zoltan Kiss > Sent: Wednesday, March 2, 2016 3:19 AM > To: dev at dpdk.org > Subject: [dpdk-dev] ixgbe TX function selection > > Hi, > > I've noticed that ixgbe_set_tx_function() selects the non-SG

[dpdk-dev] [PATCH v4 5/7] lib/librte_ether: support new l2 tunnel operation

2016-03-04 Thread He, Shaopeng
Hi Wenzhuo, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Wenzhuo Lu > Sent: Thursday, February 18, 2016 10:46 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v4 5/7] lib/librte_ether: support new l2 tunnel > operation > > Add functions to support

[dpdk-dev] [PATCH v2 1/7] vhost: refactor rte_vhost_dequeue_burst

2016-03-04 Thread Yuanhan Liu
On Thu, Mar 03, 2016 at 05:19:42PM +, Xie, Huawei wrote: > On 2/18/2016 9:48 PM, Yuanhan Liu wrote: > > [...] > CCed changchun, the author for the chained handling of desc and mbuf. > The change makes the code more readable, but i think the following > commit message is simple and enough.

[dpdk-dev] [PATCH v2 1/7] vhost: refactor rte_vhost_dequeue_burst

2016-03-04 Thread Yuanhan Liu
On Thu, Mar 03, 2016 at 04:30:42PM +, Xie, Huawei wrote: > On 2/18/2016 9:48 PM, Yuanhan Liu wrote: > > + mbuf_avail = 0; > > + mbuf_offset = 0; > > + while (desc_avail || (desc->flags & VRING_DESC_F_NEXT) != 0) { > > + /* This desc reachs to its end, get the next one */ > > +

[dpdk-dev] [PATCH v3 5/6] virtio: Add support for qtest virtio-net PMD

2016-03-04 Thread Tan, Jianfeng
Hi Tetsuya, Seems that this patch is too long. Is it possible to split into multiple commits? On 2/22/2016 4:17 PM, Tetsuya Mukawa wrote: > The patch adds a new virtio-net PMD configuration that allows the PMD to > work on host as if the PMD is in VM. > Here is new configuration for virtio-net

[dpdk-dev] [PATCH v2 1/7] vhost: refactor rte_vhost_dequeue_burst

2016-03-04 Thread Yuanhan Liu
On Thu, Mar 03, 2016 at 04:21:19PM +, Xie, Huawei wrote: > On 2/18/2016 9:48 PM, Yuanhan Liu wrote: > > The current rte_vhost_dequeue_burst() implementation is a bit messy > > and logic twisted. And you could see repeat code here and there: it > > invokes rte_pktmbuf_alloc() three times at

[dpdk-dev] [PATCH v3 3/6] EAL: Add new EAL "--range-virtaddr" option

2016-03-04 Thread Tan, Jianfeng
Hi Tetsuya, On 2/22/2016 4:17 PM, Tetsuya Mukawa wrote: > The option specifies how to mmap EAL memory. > If the option is specified like '--range-virtaddr=-', > EAL will check /proc/maps, then tries to find free region between addr1 > and addr2. If a region is found, EAL will treat it as if

[dpdk-dev] [PATCH v2 1/7] vhost: refactor rte_vhost_dequeue_burst

2016-03-04 Thread Yuanhan Liu
On Thu, Mar 03, 2016 at 05:40:14PM +, Xie, Huawei wrote: > On 2/18/2016 9:48 PM, Yuanhan Liu wrote: > > The current rte_vhost_dequeue_burst() implementation is a bit messy > [...] > > + > > uint16_t > > rte_vhost_dequeue_burst(struct virtio_net *dev, uint16_t queue_id, > > struct

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

2016-03-04 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 v7 1/5] lib/librte_ether: change function name of tunnel port config

2016-03-04 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

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

2016-03-04 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 --- drivers/net/i40e/i40e_ethdev.c | 22 -- 1 file changed, 12 insertions(+), 10

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

2016-03-04 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

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

2016-03-04 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 --- drivers/net/ixgbe/ixgbe_ethdev.c | 4 drivers/net/ixgbe/ixgbe_rxtx.c | 11 ++- lib/librte_ether/rte_ethdev.h| 1 +

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

2016-03-04 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. Signed-off-by: Wenzhuo Lu ---

[dpdk-dev] [PATCH v4 7/7] app/testpmd: add CLIs for E-tag operation

2016-03-04 Thread Lu, Wenzhuo
Hi Shaopeng, > -Original Message- > From: He, Shaopeng > Sent: Friday, March 4, 2016 9:46 AM > To: Lu, Wenzhuo; dev at dpdk.org > Subject: RE: [dpdk-dev] [PATCH v4 7/7] app/testpmd: add CLIs for E-tag > operation > > Hi Wenzhuo, > > > -Original Message- > > From: dev

[dpdk-dev] [PATCH v4 6/7] ixgbe: support l2 tunnel operation

2016-03-04 Thread Lu, Wenzhuo
Hi Shaopeng, > -Original Message- > From: He, Shaopeng > Sent: Friday, March 4, 2016 9:47 AM > To: Lu, Wenzhuo; dev at dpdk.org > Subject: RE: [dpdk-dev] [PATCH v4 6/7] ixgbe: support l2 tunnel operation > > Hi Wenzhuo, > > > -Original Message- > > From: dev [mailto:dev-bounces

[dpdk-dev] [PATCH v4 3/7] ixgbe: support l2 tunnel config

2016-03-04 Thread Lu, Wenzhuo
Hi Shaopeng, > -Original Message- > From: He, Shaopeng > Sent: Friday, March 4, 2016 9:47 AM > To: Lu, Wenzhuo; dev at dpdk.org > Subject: RE: [dpdk-dev] [PATCH v4 3/7] ixgbe: support l2 tunnel config > > Hi Wenzhuo, > > > -Original Message- > > From: dev [mailto:dev-bounces at

[dpdk-dev] [PATCH v4 5/7] lib/librte_ether: support new l2 tunnel operation

2016-03-04 Thread Lu, Wenzhuo
Hi Shaopeng, > -Original Message- > From: He, Shaopeng > Sent: Friday, March 4, 2016 9:47 AM > To: Lu, Wenzhuo; dev at dpdk.org > Subject: RE: [dpdk-dev] [PATCH v4 5/7] lib/librte_ether: support new l2 tunnel > operation > > Hi Wenzhuo, > > > -Original Message- > > From: dev

[dpdk-dev] i40e & ixgbe xmit issue: txe->next_id

2016-03-04 Thread Zhe Tao
Hi all, I have a question about why we need the txe->next_id field in the i40e Tx function? >From the current implementation, all the txe is initialized and free >sequentially, so we don't need this "next_id" field in our TX function, and when we decide the "last_id" for txe, we assume the txe is

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

2016-03-04 Thread Tetsuya Mukawa
The patch introduces a new PMD. This PMD is implemented as thin wrapper of librte_vhost. PATCH v10 changes: - Rebase on latest master. - Fix DPDK version number(2.3 to 16.04) - Set port id to mbuf while receiving packets. PATCH v9 changes: - Fix a null pointer access issue implemented in v8

[dpdk-dev] [PATCH v10 1/2] ethdev: Add a new event type to notify a queue state changed event

2016-03-04 Thread Tetsuya Mukawa
This patch adds a below event type. - RTE_ETH_EVENT_QUEUE_STATE_CHANGE This event is used for notifying a queue state changed event. Signed-off-by: Tetsuya Mukawa Acked-by: Ferruh Yigit --- lib/librte_ether/rte_ethdev.h | 2 ++ 1 file changed, 2 insertions(+) diff --git

[dpdk-dev] [PATCH v10 2/2] vhost: Add VHOST PMD

2016-03-04 Thread Tetsuya Mukawa
The patch introduces a new PMD. This PMD is implemented as thin wrapper of librte_vhost. It means librte_vhost is also needed to compile the PMD. The vhost messages will be handled only when a port is started. So start a port first, then invoke QEMU. The PMD has 2 parameters. - iface: The

[dpdk-dev] [PATCH v3 5/6] virtio: Add support for qtest virtio-net PMD

2016-03-04 Thread Tetsuya Mukawa
On 2016/03/04 11:18, Tan, Jianfeng wrote: > Hi Tetsuya, > > Seems that this patch is too long. Is it possible to split into > multiple commits? Hi Jianfeng, Sure, will do. > > On 2/22/2016 4:17 PM, Tetsuya Mukawa wrote: >> The patch adds a new virtio-net PMD configuration that allows the PMD to

[dpdk-dev] [PATCH] ixgbe: fix ixgbevf RX/TX function assignment

2016-03-04 Thread Wu, Jingjing
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Zhe Tao > Sent: Sunday, February 28, 2016 11:08 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] ixgbe: fix ixgbevf RX/TX function assignment > > For the secondary process of DPDK to initialize ixgbevf,

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

2016-03-04 Thread Liu, Yong
Tested-by: Yong Liu - Tested Branch: dpdk-next-net - Tested Commit: 5fa83b5398e26af7537b09605432fcb3d0cc1d41 - OS: Fedora21 3.17.4-301.fc21.x86_64 - GCC: gcc version 4.9.2 20150212 - CPU: Intel(R) Xeon(R) CPU D-1540 @ 2.00GHz - NIC: Intel Corporation Device Sageville [8086:15ad] - Default

[dpdk-dev] [PATCH v3 5/6] virtio: Add support for qtest virtio-net PMD

2016-03-04 Thread Tan, Jianfeng
Hi Tetsuya, On 3/4/2016 1:05 PM, Tetsuya Mukawa wrote: > On 2016/03/04 11:18, Tan, Jianfeng wrote: >> Hi Tetsuya, >> >> Seems that this patch is too long. Is it possible to split into >> multiple commits? > Hi Jianfeng, > > Sure, will do. > >> On 2/22/2016 4:17 PM, Tetsuya Mukawa wrote: >>> The

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

2016-03-04 Thread Xie, Huawei
On 2/23/2016 12:23 AM, Tom Kiely wrote: > Hi, > Sorry I missed the last few messages until now. I'm happy with > just removing the "if". Kyle, when you say you fixed it, do you mean > that you will push the patch or have already done so ? >Thanks, >Tom Could you please send the

[dpdk-dev] [PATCH v2 0/5] virtio: Tx performance improvements

2016-03-04 Thread Xie, Huawei
On 1/14/2016 9:49 PM, Xie, Huawei wrote: > On 1/6/2016 8:04 PM, Thomas Monjalon wrote: >> 2016-01-05 08:10, Xie, Huawei: >>> On 10/26/2015 10:06 PM, Xie, Huawei wrote: On 10/19/2015 1:16 PM, Stephen Hemminger wrote: > This is a tested version of the virtio Tx performance improvements

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

2016-03-04 Thread Qiu, Michael
On 3/2/2016 10:48 AM, Yuanhan Liu wrote: > On Wed, Mar 02, 2016 at 02:10:14AM +, Qiu, Michael wrote: >> On 3/1/2016 5:46 PM, Santosh Shukla wrote: >>> 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

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

2016-03-04 Thread Christian Ehrhardt
Hi, I expected more of a discussion fallout as there would be so different approaches to solve it (with the explicit xs_open stuff I mentioned, but that I consider overengineering). I had this running a while now and I can only say for me it works. A few internal people working with Xen acked it

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

2016-03-04 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 With LIBRTE_PMD_XENVIRT enabled this now tries the xen style

[dpdk-dev] [dpdk-announce] technical board

2016-03-04 Thread St Leger, Jim
Nice work by this TechBoard getting this established after the Userspace discussion in Dublin in October. -Original Message- From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon Sent: Thursday, March 3, 2016 4:53 PM To: announce at dpdk.org Subject: [dpdk-dev]

[dpdk-dev] [PATCH v5 1/2] enic: receive path performance improvements

2016-03-04 Thread John Daley
From: johndale This is a wholesale replacement of the Enic PMD receive path in order to improve performance and code clarity. The changes are: - Simplify and reduce code path length of receive function. - Put most of the fast-path receive funtions in one file. - Reduce the

[dpdk-dev] [PATCH v5 2/2] enic: remove unused functions

2016-03-04 Thread John Daley
The clang compiler warned about a half dozen static inline functions which were not used. Remove them. Signed-off-by: John Daley --- drivers/net/enic/enic_main.c | 10 -- drivers/net/enic/enic_rx.c | 35 --- 2 files changed, 45 deletions(-) diff --git

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

2016-03-04 Thread Tom Kiely
Sure. Tom On 03/04/2016 06:16 AM, Xie, Huawei wrote: > On 2/23/2016 12:23 AM, Tom Kiely wrote: >> Hi, >> Sorry I missed the last few messages until now. I'm happy with >> just removing the "if". Kyle, when you say you fixed it, do you mean >> that you will push the patch or have already

[dpdk-dev] [PATCH] examples/dpdk_qat: Fix RX queue start number to the one just received the packets

2016-03-04 Thread Wu, Jingjing
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Zhe Tao > Sent: Tuesday, September 29, 2015 7:08 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] examples/dpdk_qat: Fix RX queue start > number to the one just received the packets > > Every time we

[dpdk-dev] [PATCH v10 2/2] vhost: Add VHOST PMD

2016-03-04 Thread Yuanhan Liu
On Fri, Mar 04, 2016 at 01:17:42PM +0900, Tetsuya Mukawa wrote: > The patch introduces a new PMD. This PMD is implemented as thin wrapper > of librte_vhost. It means librte_vhost is also needed to compile the PMD. > The vhost messages will be handled only when a port is started. So start > a port

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

2016-03-04 Thread Olivier MATZ
Hi David, On 02/29/2016 12:11 PM, Hunt, David wrote: >> Also, I'm sorry but it seems that several comments or question I've made >> in http://dpdk.org/ml/archives/dev/2016-February/032706.html are >> not addressed. >> >> Examples: >> - putting some part of the patch in separate commits >> -

[dpdk-dev] [PATCH 2/6] mempool: add stack (lifo) based external mempool handler

2016-03-04 Thread Olivier MATZ
Hi David, On 02/29/2016 12:04 PM, Hunt, David wrote: > > On 2/19/2016 1:31 PM, Olivier MATZ wrote: >> Hi David, >> >> On 02/16/2016 03:48 PM, David Hunt wrote: >>> adds a simple stack based mempool handler >>> >>> Signed-off-by: David Hunt >>> --- >>> lib/librte_mempool/Makefile|

[dpdk-dev] [PATCH] cmdline: include missing cmdline_parse.h

2016-03-04 Thread Olivier MATZ
Hi Marc, On 03/03/2016 12:52 AM, Marc Sune wrote: > cmdline_parse_*.h headers use struct cmdline_token_hdr / > cmdline_parse_token_hdr_t which is defined in cmdline_parse.h, but > do not include it, forcing manual inclusion. > > This commit includes cmdline_parse.h in all cmdline_parse_*.h.

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

2016-03-04 Thread Olivier MATZ
Hi David, >> >>> @@ -622,6 +607,10 @@ rte_mempool_xmem_create(const char *name, >>> unsigned n, unsigned elt_size, >>> >>> mp->elt_va_end = mp->elt_va_start; >>> >>> +/* Parameters are setup. Call the mempool handler alloc */ >>> +if ((rte_mempool_ext_alloc(mp, name, n, socket_id,

[dpdk-dev] [PATCH v5] mbuf: introduce common functions to get dma address

2016-03-04 Thread Olivier Matz
From: Ravi Kerur Macros RTE_MBUF_DATA_DMA_ADDR and RTE_MBUF_DATA_DMA_ADDR_DEFAULT are defined in each PMD driver file. Convert macros to inline functions and move them to common lib/librte_mbuf/rte_mbuf.h file. PMD drivers include rte_mbuf.h file directly/indirectly hence no

[dpdk-dev] [PATCH 1/3] add tx crc disable flag

2016-03-04 Thread Olivier MATZ
Hi Paul, On 02/15/2016 05:55 PM, Paul Emmerich wrote: > Signed-off-by: Paul Emmerich > --- > lib/librte_mbuf/rte_mbuf.c | 1 + > lib/librte_mbuf/rte_mbuf.h | 6 ++ > 2 files changed, 7 insertions(+) > > diff --git a/lib/librte_mbuf/rte_mbuf.c b/lib/librte_mbuf/rte_mbuf.c > index

[dpdk-dev] [PATCH v4 0/7] support E-tag offloading and forwarding on Intel X550 NIC

2016-03-04 Thread Liu, Yong
Tested-by: Yong Liu - Tested Branch: dpdk-next-net - Tested Commit: 5fa83b5398e26af7537b09605432fcb3d0cc1d41 - OS: Fedora21 3.17.4-301.fc21.x86_64 - GCC: gcc version 4.9.2 20150212 - CPU: Intel(R) Xeon(R) CPU D-1540 @ 2.00GHz - NIC: Intel Corporation Device Sageville [8086:15ad] - Default

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

2016-03-04 Thread Liu, Yong
Sorry, sent to incorrect mail thread. Please ignore this tested-by email. > -Original Message- > From: Liu, Yong > Sent: Friday, March 04, 2016 1:45 PM > To: Lu, Wenzhuo; dev at dpdk.org > Subject: RE: [dpdk-dev] [PATCH v7 0/5] Support VxLAN & NVGRE checksum > off-load on X550 > >

[dpdk-dev] [PATCH] config: remove duplicate configuration information

2016-03-04 Thread Traynor, Kevin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > Sent: Thursday, March 3, 2016 6:38 PM > To: Wiles, Keith > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] config: remove duplicate configuration > information > > 2016-02-22 07:53,

[dpdk-dev] rte_mbuf's packet_type field

2016-03-04 Thread Olivier MATZ
Hi Zoltan, On 03/01/2016 06:15 PM, Zoltan Kiss wrote: > 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

[dpdk-dev] [PATCH] config: remove duplicate configuration information

2016-03-04 Thread Panu Matilainen
On 03/03/2016 08:37 PM, Thomas Monjalon wrote: > 2016-02-22 07:53, Keith Wiles: >> --- /dev/null >> +++ b/config/common_base >> +CONFIG_RTE_EAL_IGB_UIO=y >> +CONFIG_RTE_EAL_VFIO=y > > These options should be disabled in the base file > and enabled in Linux. > >> +CONFIG_RTE_LIBRTE_PMD_AF_PACKET=y

[dpdk-dev] [PATCH] config: remove duplicate configuration information

2016-03-04 Thread Panu Matilainen
On 03/04/2016 11:28 AM, Traynor, Kevin wrote: > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon >> Sent: Thursday, March 3, 2016 6:38 PM >> To: Wiles, Keith >> Cc: dev at dpdk.org >> Subject: Re: [dpdk-dev] [PATCH] config: remove duplicate

[dpdk-dev] [PATCH v3 5/6] virtio: Add support for qtest virtio-net PMD

2016-03-04 Thread Tetsuya Mukawa
On 2016/03/04 15:10, Tan, Jianfeng wrote: > Hi Tetsuya, > > On 3/4/2016 1:05 PM, Tetsuya Mukawa wrote: >> On 2016/03/04 11:18, Tan, Jianfeng wrote: >>> Hi Tetsuya, >>> >>> Seems that this patch is too long. Is it possible to split into >>> multiple commits? >> Hi Jianfeng, >> >> Sure, will do. >>

[dpdk-dev] [PATCH v10 2/2] vhost: Add VHOST PMD

2016-03-04 Thread Tetsuya Mukawa
On 2016/03/04 17:39, Yuanhan Liu wrote: > On Fri, Mar 04, 2016 at 01:17:42PM +0900, Tetsuya Mukawa wrote: >> The patch introduces a new PMD. This PMD is implemented as thin wrapper >> of librte_vhost. It means librte_vhost is also needed to compile the PMD. >> The vhost messages will be handled

[dpdk-dev] [PATCH] config: remove duplicate configuration information

2016-03-04 Thread Traynor, Kevin
> -Original Message- > From: Panu Matilainen [mailto:pmatilai at redhat.com] > Sent: Friday, March 4, 2016 9:39 AM > To: Traynor, Kevin ; Thomas Monjalon > ; Wiles, Keith > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] config: remove duplicate configuration > information > > On

[dpdk-dev] [PATCH] config: remove duplicate configuration information

2016-03-04 Thread Panu Matilainen
On 03/03/2016 08:37 PM, Thomas Monjalon wrote: >> --- a/config/defconfig_x86_64-native-bsdapp-clang >> +++ b/config/defconfig_x86_64-native-bsdapp-clang >> @@ -37,6 +37,7 @@ CONFIG_RTE_MACHINE="native" >> CONFIG_RTE_ARCH="x86_64" >> CONFIG_RTE_ARCH_X86_64=y >> CONFIG_RTE_ARCH_X86=y >>

[dpdk-dev] [PATCH] config: remove duplicate configuration information

2016-03-04 Thread Thomas Monjalon
2016-03-04 09:58, Traynor, Kevin: > From: Panu Matilainen [mailto:pmatilai at redhat.com] > > On 03/04/2016 11:28 AM, Traynor, Kevin wrote: > > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > > >> 2016-02-22 07:53, Keith Wiles: > > >>> +CONFIG_RTE_LIBRTE_VHOST=y > > >>

[dpdk-dev] [PATCH] lpm6: fix use after free of lpm in rte_lpm6_create

2016-03-04 Thread Christian Ehrhardt
In certain autotests lpm->max_rules turned out to be non initialized. That was caused by a failing allocation for lpm->rules_tbl in rte_lpm6_create. It then left the function via goto exit with lpm freed, but still a pointer value being set. In case of an allocation failure it resets lpm to NULL

[dpdk-dev] rte_mbuf's packet_type field

2016-03-04 Thread Ananyev, Konstantin
Hi, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier MATZ > Sent: Friday, March 04, 2016 9:29 AM > To: Zoltan Kiss; dev at dpdk.org > Subject: Re: [dpdk-dev] rte_mbuf's packet_type field > > Hi Zoltan, > > On 03/01/2016 06:15 PM, Zoltan Kiss wrote:

[dpdk-dev] FOSDEM videos

2016-03-04 Thread Thomas Monjalon
The videos of the conference FOSDEM 2016 are available. This is my list of DPDK related talks: * Saying again that "Userspace is not slow" https://fosdem.org/2016/schedule/event/linux_petascale_storage/ * OPNFV https://fosdem.org/2016/schedule/event/telco_on_free_software/ *

[dpdk-dev] [PATCH v2] igb_uio: use macros for array size calculation

2016-03-04 Thread Ferruh Yigit
Minor code cleanup. Remove array size calculations and remove unnecessary assignment. Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/igb_uio/igb_uio.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/librte_eal/linuxapp/igb_uio/igb_uio.c

[dpdk-dev] [PATCH v2] igb_uio: use macros for array size calculation

2016-03-04 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ferruh Yigit > Sent: Friday, March 04, 2016 11:21 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v2] igb_uio: use macros for array size calculation > > Minor code cleanup. > Remove array size

[dpdk-dev] ixgbe TX function selection

2016-03-04 Thread Zoltan Kiss
On 04/03/16 01:47, Lu, Wenzhuo wrote: > Hi Zoltan, > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Zoltan Kiss >> Sent: Wednesday, March 2, 2016 3:19 AM >> To: dev at dpdk.org >> Subject: [dpdk-dev] ixgbe TX function selection >> >> Hi, >> >> I've

[dpdk-dev] [PATCH] igb_uio: cast private data to correct struct type

2016-03-04 Thread Ananyev, Konstantin
> Subject: [dpdk-dev] [PATCH] igb_uio: cast private data to correct struct type > > Fixes: af75078fece3 ("first public release") > > This was working fine because addresses of two structs are same: > > struct A { > struct B b; > } a; > > As above sample "a" and "b" has same address. > >

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

2016-03-04 Thread Kyle Larose
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 wrote: >>> >>> Hi, >>> Sorry I missed the last few messages until now. I'm happy with >>> just removing the "if". Kyle, when you say you

[dpdk-dev] [PATCH] lpm/lpm6: fix missing free of rules_tbl and lpm

2016-03-04 Thread Christian Ehrhardt
lpm6 autotests failed with the default alloc of 512M Memory. While >=2500M was a workaround it became clear while debugging that it had a leak. One could see a lot of output like: LPM Test tests6[i]: FAIL LPM: LPM memory allocation failed It turned out that in rte_lpm6_free - lpm might not be

[dpdk-dev] rte_mbuf's packet_type field

2016-03-04 Thread Zoltan Kiss
On 04/03/16 10:58, Ananyev, Konstantin wrote: > Hi, > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier MATZ >> Sent: Friday, March 04, 2016 9:29 AM >> To: Zoltan Kiss; dev at dpdk.org >> Subject: Re: [dpdk-dev] rte_mbuf's packet_type field >> >> Hi

[dpdk-dev] [PATCH] config: remove duplicate configuration information

2016-03-04 Thread Wiles, Keith
> >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon >> Sent: Thursday, March 3, 2016 6:38 PM >> To: Wiles, Keith >> Cc: dev at dpdk.org >> Subject: Re: [dpdk-dev] [PATCH] config: remove duplicate configuration >> information >> >> 2016-02-22

[dpdk-dev] [PATCH] config: remove duplicate configuration information

2016-03-04 Thread Wiles, Keith
>On 03/03/2016 08:37 PM, Thomas Monjalon wrote: >>> --- a/config/defconfig_x86_64-native-bsdapp-clang >>> +++ b/config/defconfig_x86_64-native-bsdapp-clang >>> @@ -37,6 +37,7 @@ CONFIG_RTE_MACHINE="native" >>> CONFIG_RTE_ARCH="x86_64" >>> CONFIG_RTE_ARCH_X86_64=y >>> CONFIG_RTE_ARCH_X86=y

[dpdk-dev] [PATCH] cmdline: include missing cmdline_parse.h

2016-03-04 Thread Thomas Monjalon
2016-03-04 10:05, Olivier MATZ: > Hi Marc, > > On 03/03/2016 12:52 AM, Marc Sune wrote: > > cmdline_parse_*.h headers use struct cmdline_token_hdr / > > cmdline_parse_token_hdr_t which is defined in cmdline_parse.h, but > > do not include it, forcing manual inclusion. > > > > This commit

[dpdk-dev] [PATCH] config: remove duplicate configuration information

2016-03-04 Thread Wiles, Keith
>>> >>>--- a/config/defconfig_x86_64-native-bsdapp-clang >>>+++ b/config/defconfig_x86_64-native-bsdapp-clang >>>@@ -37,6 +37,7 @@ CONFIG_RTE_MACHINE="native" >>> CONFIG_RTE_ARCH="x86_64" >>> CONFIG_RTE_ARCH_X86_64=y >>> CONFIG_RTE_ARCH_X86=y >>>+CONFIG_RTE_ARCH_64=y >>> >>>

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

2016-03-04 Thread Thomas Monjalon
2016-02-29 16:52, Declan Doherty: > From: 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 >

[dpdk-dev] [PATCH v5] mbuf: introduce common functions to get dma address

2016-03-04 Thread Thomas Monjalon
2016-03-04 10:09, Olivier Matz: > From: Ravi Kerur > > Macros RTE_MBUF_DATA_DMA_ADDR and RTE_MBUF_DATA_DMA_ADDR_DEFAULT > are defined in each PMD driver file. Convert macros to inline > functions and move them to common lib/librte_mbuf/rte_mbuf.h file. > PMD drivers include rte_mbuf.h file

[dpdk-dev] [PATCH] lpm: merge tbl24 and tbl8 structures

2016-03-04 Thread Thomas Monjalon
2015-11-24 14:25, Bruce Richardson: > The tbl8 and tbl24 structures were essentially identical except for > slightly different names for one or two fields. Merge these two > structures into a single structure definition. > > Two fields have been renamed as part of this change: the > "ext_entry"

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

2016-03-04 Thread Thomas Monjalon
2015-11-29 10:46, Stephen Hemminger: > This is the last round of sched updates for 2.2. It is based > on code changes (extensively) tested by QA and used in the vRouter. > > Stephen Hemminger (3): > rte_sched: keep track of RED drops > rte_sched: introduce reciprocal divide > rte_sched:

[dpdk-dev] [PATCH] lpm: merge tbl24 and tbl8 structures

2016-03-04 Thread Jerin Jacob
On Fri, Mar 04, 2016 at 03:57:43PM +0100, Thomas Monjalon wrote: > 2015-11-24 14:25, Bruce Richardson: > > The tbl8 and tbl24 structures were essentially identical except for > > slightly different names for one or two fields. Merge these two > > structures into a single structure definition. > >

[dpdk-dev] [PATCH] maintainer: claim responsibility for igb_uio

2016-03-04 Thread Ferruh Yigit
igb_iuo has no maintainer, claim responsibility for igb_uio Signed-off-by: Ferruh Yigit --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 628bc05..e253bf7 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -164,6 +164,7 @@ F:

[dpdk-dev] [PATCH] maintainer: claim responsibility for igb_uio

2016-03-04 Thread Zhang, Helin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ferruh Yigit > Sent: Friday, March 4, 2016 11:07 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] maintainer: claim responsibility for igb_uio > > igb_iuo has no maintainer, claim responsibility for

[dpdk-dev] [PATCH] lpm: merge tbl24 and tbl8 structures

2016-03-04 Thread Kobylinski, MichalX
> -Original Message- > From: Jerin Jacob [mailto:jerin.jacob at caviumnetworks.com] > Sent: Friday, March 4, 2016 4:04 PM > To: Thomas Monjalon > Cc: Richardson, Bruce ; dev at dpdk.org; > Kobylinski, MichalX > Subject: Re: [dpdk-dev] [PATCH] lpm: merge tbl24 and tbl8 structures >

[dpdk-dev] [PATCH] eal/linux: fix compilation error CONFIG_RTE_LIBEAL_USE_HPET

2016-03-04 Thread Thomas Monjalon
2016-02-11 00:05, Vincent JARDIN: > When CONFIG_RTE_LIBEAL_USE_HPET=y is set, eal_timer.c does not compile > anymore. Just add simple missing include. > > Signed-off-by: Vincent JARDIN It has been fixed in another patch: http://dpdk.org/browse/dpdk/commit/?id=3560681d68

[dpdk-dev] [PATCH v2 0/2] Add missing ethdev driver support

2016-03-04 Thread Remy Horton
Several rte_eth_dev_* functions are currently only supported by the ixgbe NIC driver. This patchset adds driver support for some of these functions to the i40e and vmxnet3 drivers. -- This patchset depends on: * http://dpdk.org/dev/patchwork/patch/10654/ v2 changes: * Rebased to origin/master

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

2016-03-04 Thread Remy Horton
Implements driver support for dumping of EEPROM and registers, and the setting of MAC address. Signed-off-by: Remy Horton --- doc/guides/rel_notes/release_16_04.rst |5 + drivers/net/i40e/i40e_ethdev.c | 116 drivers/net/i40e/i40e_regs.h | 1000

[dpdk-dev] [PATCH v2 2/2] drivers/net/vmxnet3: add ethdev functions

2016-03-04 Thread Remy Horton
Implements driver support for setting of MAC address. Signed-off-by: Remy Horton --- doc/guides/rel_notes/release_16_04.rst | 4 drivers/net/vmxnet3/vmxnet3_ethdev.c | 19 +++ 2 files changed, 23 insertions(+) diff --git a/doc/guides/rel_notes/release_16_04.rst

[dpdk-dev] [PATCH] config: remove duplicate configuration information

2016-03-04 Thread Thomas Monjalon
2016-03-04 14:44, Wiles, Keith: > >>> > >>>--- a/config/defconfig_x86_64-native-bsdapp-clang > >>>+++ b/config/defconfig_x86_64-native-bsdapp-clang > >>>@@ -37,6 +37,7 @@ CONFIG_RTE_MACHINE="native" > >>> CONFIG_RTE_ARCH="x86_64" > >>> CONFIG_RTE_ARCH_X86_64=y > >>> CONFIG_RTE_ARCH_X86=y >

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

2016-03-04 Thread Jianfeng Tan
This patch will work on below patch series. - [PATCH v6 00/11] Add API to get packet type info 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

[dpdk-dev] [PATCH v2] ethdev: fix byte order inconsistence between fdir flow and mask

2016-03-04 Thread Thomas Monjalon
> > Fixed issue of byte order in ethdev library that the structure for setting > > fdir's > > mask and flow entry is inconsist and made inputs of mask be in big endian. > > > > Fixes: 76c6f89e80d4 ("ixgbe: support new flow director masks") > > Fixes: 2d4c1a9ea2ac ("ethdev: add new flow director

[dpdk-dev] [PATCH v2] config: remove duplicate configuration information

2016-03-04 Thread Keith Wiles
In order to cleanup the configuration files some and reduce the number of duplicate configuration information. Add a new file called common_base which contains just about all of the configuration lines in one place. Then have the common_bsdapp, common_linuxapp files include this one file. Then in

[dpdk-dev] [PATCH] config: add missing CONFIG_RTE_ARCH_64 configurations

2016-03-04 Thread Keith Wiles
Signed-off-by: Keith Wiles --- config/defconfig_x86_64-native-bsdapp-clang | 1 + config/defconfig_x86_64-native-bsdapp-gcc | 1 + 2 files changed, 2 insertions(+) diff --git a/config/defconfig_x86_64-native-bsdapp-clang b/config/defconfig_x86_64-native-bsdapp-clang index d2baf2c..8b870b3

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

2016-03-04 Thread Jianfeng Tan
This patch adds option, --avail-cores, to use lcores which are available by calling pthread_getaffinity_np() to narrow down detected cores before parsing coremask (-c), corelist (-l), and coremap (--lcores). Test example: $ taskset 0xc ./examples/helloworld/build/helloworld \

[dpdk-dev] [PATCH] bonding: fix crash when no slave devices

2016-03-04 Thread Ferruh Yigit
On 2/23/2016 12:13 PM, Bernard Iremonger wrote: > If a bonded device is created when there are no slave devices > there is loop in bond_ethdev_promiscous_enable() which results > in a segmentation fault. > I have applied a similar fix to bond_ethdev_promiscous_disable() > where a similar loop

[dpdk-dev] [PATCH] bonding: fix crash when no slave devices

2016-03-04 Thread Iremonger, Bernard
Hi Ferruh, > -Original Message- > From: Yigit, Ferruh > Sent: Friday, March 4, 2016 5:14 PM > To: Iremonger, Bernard ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] bonding: fix crash when no slave devices > > On 2/23/2016 12:13 PM, Bernard Iremonger wrote: > > If a bonded device is

[dpdk-dev] [PATCH v5 0/2] cryptodev API changes

2016-03-04 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. V5: - updates .map file - removes EXPERIMENTAL label from rte_cryptodev.h V4: - Fixes for issues introduced in

[dpdk-dev] [PATCH v5 1/2] 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, i

2016-03-04 Thread Fiona Trahe
Signed-off-by: Fiona Trahe --- app/test/test_cryptodev.c | 164 +++--- app/test/test_cryptodev_perf.c | 79 +-- drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c | 44 +- drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c | 6 +-

[dpdk-dev] [PATCH v5 2/2] 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

2016-03-04 Thread Fiona Trahe
The changes also continues the separatation of the symmetric operation parameters from the more general operation parameters, this will simplify the integration of asymmetric crypto operations in the future. As well as the changes to the crypto APIs this patch adds functions for managing

[dpdk-dev] [PATCH v2] config: remove duplicate configuration information

2016-03-04 Thread Thomas Monjalon
2016-03-04 11:01, Keith Wiles: > +# > +# Combine to one single library > +# > +CONFIG_RTE_BUILD_COMBINE_LIBS=n It has been removed.

[dpdk-dev] [PATCH v5 0/2] cryptodev API changes

2016-03-04 Thread Trahe, Fiona
> -Original Message- > From: Trahe, Fiona > Sent: Friday, March 04, 2016 5:18 PM > To: dev at dpdk.org > Cc: Doherty, Declan; Trahe, Fiona > Subject: [PATCH v5 0/2] cryptodev API changes > > This patch set separates the symmetric crypto operations from generic > operations and then

[dpdk-dev] [PATCH v5 0/2] cryptodev API changes

2016-03-04 Thread Thomas Monjalon
2016-03-04 17:17, 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. > > V5: > - updates .map file > - removes EXPERIMENTAL label from

  1   2   >