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

2016-10-10 Thread Yuanhan Liu
On Mon, Aug 29, 2016 at 08:32:55AM +, Xu, Qian Q wrote: > I just ran a PVP test, nic receive packets then forwards to vhost PMD, and > virtio user interface. I didn't see any performance gains in this scenario. > All packet size from 64B to 1518B > performance haven't got benefit from this

[dpdk-dev] [PATCH v4 0/3] Add TSO on tunneling packet

2016-10-10 Thread Thomas Monjalon
2016-09-27 17:29, Ananyev, Konstantin: > > > From: Tan, Jianfeng > > > > Patch 1: mbuf: add Tx side tunneling type Patch 2: net/i40e: add TSO > > > > support on tunneling packet Patch 3: app/testpmd: fix Tx offload on > > > > tunneling packet > > Acked-by: Konstantin Ananyev > I think you need

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

2016-10-10 Thread Michael S. Tsirkin
On Wed, Sep 28, 2016 at 10:28:48AM +0800, Yuanhan Liu wrote: > On Tue, Sep 27, 2016 at 10:56:40PM +0300, Michael S. Tsirkin wrote: > > On Tue, Sep 27, 2016 at 11:11:58AM +0800, Yuanhan Liu wrote: > > > On Mon, Sep 26, 2016 at 10:24:55PM +0300, Michael S. Tsirkin wrote: > > > > On Mon, Sep 26, 2016

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

2016-10-10 Thread Yuanhan Liu
On Sun, Oct 09, 2016 at 12:09:07PM +, Wang, Zhihong wrote: > > > > Tested with testpmd, host: txonly, guest: rxonly > > > > size (bytes) improvement (%) > > > > 644.12 > > > > 128 6 > > > > 256 2.65 > > > > 512 -1.12

[dpdk-dev] [PATCH] app/testpmd: fix pf/vf check of flow director

2016-10-10 Thread Wenzhuo Lu
Parameters pf & vf are added into most of flow director filter CLIs. But mac-valn and tunnel filters don't have these parameters, the parameters should not be checked for mac-vlan and tunnel filters. Fixes: e6a68c013353 ("app/testpmd: extend commands for flow director in VF") Signed-off-by:

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

2016-10-10 Thread Yuanhan Liu
On Mon, Oct 10, 2016 at 02:20:22AM +0300, Michael S. Tsirkin wrote: > On Wed, Sep 28, 2016 at 10:28:48AM +0800, Yuanhan Liu wrote: > > On Tue, Sep 27, 2016 at 10:56:40PM +0300, Michael S. Tsirkin wrote: > > > On Tue, Sep 27, 2016 at 11:11:58AM +0800, Yuanhan Liu wrote: > > > > On Mon, Sep 26, 2016

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

2016-10-10 Thread Michael S. Tsirkin
On Mon, Oct 10, 2016 at 11:03:33AM +0800, Yuanhan Liu wrote: > On Mon, Oct 10, 2016 at 02:20:22AM +0300, Michael S. Tsirkin wrote: > > On Wed, Sep 28, 2016 at 10:28:48AM +0800, Yuanhan Liu wrote: > > > On Tue, Sep 27, 2016 at 10:56:40PM +0300, Michael S. Tsirkin wrote: > > > > On Tue, Sep 27, 2016

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

2016-10-10 Thread Yuanhan Liu
On Mon, Oct 10, 2016 at 06:04:32AM +0300, Michael S. Tsirkin wrote: > > > So I guess at this point, we can teach vhost-user in qemu > > > that version 1 implies any_layout but only for machine types > > > qemu 2.8 and up. It sets a bad precedent but oh well. > > > > It should work. > > > >

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

2016-10-10 Thread Yuanhan Liu
On Thu, Sep 29, 2016 at 11:21:48PM +0300, Michael S. Tsirkin wrote: > On Thu, Sep 29, 2016 at 10:05:22PM +0200, Maxime Coquelin wrote: > > > > > > On 09/29/2016 07:57 PM, Michael S. Tsirkin wrote: > Yes but two points. > > 1. why is this memset expensive? I don't have the exact answer, but

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

2016-10-10 Thread Michael S. Tsirkin
On Mon, Oct 10, 2016 at 11:37:44AM +0800, Yuanhan Liu wrote: > On Thu, Sep 29, 2016 at 11:21:48PM +0300, Michael S. Tsirkin wrote: > > On Thu, Sep 29, 2016 at 10:05:22PM +0200, Maxime Coquelin wrote: > > > > > > > > > On 09/29/2016 07:57 PM, Michael S. Tsirkin wrote: > > Yes but two points. > >

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

2016-10-10 Thread Yuanhan Liu
On Thu, Sep 29, 2016 at 10:05:22PM +0200, Maxime Coquelin wrote: > >so doing this unconditionally would be a spec violation, but if you see > >value in this, we can add a feature bit. > Right it would be a spec violation, so it should be done conditionally. > If a feature bit is to be added, what

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

2016-10-10 Thread Yuanhan Liu
On Mon, Oct 10, 2016 at 06:46:44AM +0300, Michael S. Tsirkin wrote: > On Mon, Oct 10, 2016 at 11:37:44AM +0800, Yuanhan Liu wrote: > > On Thu, Sep 29, 2016 at 11:21:48PM +0300, Michael S. Tsirkin wrote: > > > On Thu, Sep 29, 2016 at 10:05:22PM +0200, Maxime Coquelin wrote: > > > > > > > > > > >

[dpdk-dev] [PATCH v2] i40: fix the VXLAN TSO issue

2016-10-10 Thread Wu, Jingjing
NACK. This fix has been done by a new one which is already merged: http://dpdk.org/dev/patchwork/patch/15059/ > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Zhe Tao > Sent: Thursday, July 7, 2016 12:27 PM > To: dev at dpdk.org > Cc: zhe.tao at

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

2016-10-10 Thread Yuanhan Liu
On Fri, Sep 30, 2016 at 10:16:43PM +0300, Michael S. Tsirkin wrote: > > > And the same is done is done in DPDK: > > > > > > static inline int __attribute__((always_inline)) > > > copy_desc_to_mbuf(struct virtio_net *dev, struct vring_desc *descs, > > > uint16_t max_desc, struct rte_mbuf

[dpdk-dev] [PATCH v2] i40: fix the VXLAN TSO issue

2016-10-10 Thread Yuanhan Liu
On Mon, Oct 10, 2016 at 03:58:57AM +, Wu, Jingjing wrote: > NACK. > > This fix has been done by a new one which is already merged: > http://dpdk.org/dev/patchwork/patch/15059/ Just want to let you know, that you don't have to NACK an old patchset. In the process of the patchwork, ideally,

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

2016-10-10 Thread Wang, Zhihong
> -Original Message- > From: Yuanhan Liu [mailto:yuanhan.liu at linux.intel.com] > Sent: Monday, October 10, 2016 11:59 AM > To: Michael S. Tsirkin > Cc: Maxime Coquelin ; Stephen Hemminger > ; dev at dpdk.org; qemu- > devel at nongnu.org; Wang, Zhihong > Subject: Re: [Qemu-devel]

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

2016-10-10 Thread Michael S. Tsirkin
On Mon, Oct 10, 2016 at 12:05:31PM +0800, Yuanhan Liu wrote: > On Fri, Sep 30, 2016 at 10:16:43PM +0300, Michael S. Tsirkin wrote: > > > > And the same is done is done in DPDK: > > > > > > > > static inline int __attribute__((always_inline)) > > > > copy_desc_to_mbuf(struct virtio_net *dev,

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

2016-10-10 Thread Yuanhan Liu
On Mon, Oct 10, 2016 at 07:17:06AM +0300, Michael S. Tsirkin wrote: > On Mon, Oct 10, 2016 at 12:05:31PM +0800, Yuanhan Liu wrote: > > On Fri, Sep 30, 2016 at 10:16:43PM +0300, Michael S. Tsirkin wrote: > > > > > And the same is done is done in DPDK: > > > > > > > > > > static inline int

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

2016-10-10 Thread Michael S. Tsirkin
On Mon, Oct 10, 2016 at 04:16:19AM +, Wang, Zhihong wrote: > > > > -Original Message- > > From: Yuanhan Liu [mailto:yuanhan.liu at linux.intel.com] > > Sent: Monday, October 10, 2016 11:59 AM > > To: Michael S. Tsirkin > > Cc: Maxime Coquelin ; Stephen Hemminger > > ; dev at

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

2016-10-10 Thread Michael S. Tsirkin
On Mon, Oct 10, 2016 at 12:22:09PM +0800, Yuanhan Liu wrote: > On Mon, Oct 10, 2016 at 07:17:06AM +0300, Michael S. Tsirkin wrote: > > On Mon, Oct 10, 2016 at 12:05:31PM +0800, Yuanhan Liu wrote: > > > On Fri, Sep 30, 2016 at 10:16:43PM +0300, Michael S. Tsirkin wrote: > > > > > > And the same is

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

2016-10-10 Thread Michael S. Tsirkin
On Mon, Oct 10, 2016 at 04:16:19AM +, Wang, Zhihong wrote: > > > > -Original Message- > > From: Yuanhan Liu [mailto:yuanhan.liu at linux.intel.com] > > Sent: Monday, October 10, 2016 11:59 AM > > To: Michael S. Tsirkin > > Cc: Maxime Coquelin ; Stephen Hemminger > > ; dev at

[dpdk-dev] [PATCH v2] drivers: prefix driver REGISTER macro with RTE PMD

2016-10-10 Thread Shreyansh Jain
On Monday 10 October 2016 01:20 AM, Thomas Monjalon wrote: > 2016-10-09 15:12, Shreyansh Jain: >> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] >>> 2016-10-08 23:35, Shreyansh Jain: +PMDINFO_TO_O = if grep -E 'RTE_PMD_REGISTER_PCI\([0-9a-zA-Z,_\. >>>

[dpdk-dev] [PATCH v2] drivers: prefix driver REGISTER macro with RTE PMD

2016-10-10 Thread Shreyansh Jain
On Monday 10 October 2016 10:41 AM, Shreyansh Jain wrote: > On Monday 10 October 2016 01:20 AM, Thomas Monjalon wrote: >> 2016-10-09 15:12, Shreyansh Jain: >>> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] 2016-10-08 23:35, Shreyansh Jain: > +PMDINFO_TO_O = if grep -E

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

2016-10-10 Thread Akhil Goyal
On 10/8/2016 3:06 AM, De Lara Guarch, Pablo wrote: > Hi Akhil, > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of >> akhil.goyal at nxp.com >> Sent: Friday, October 07, 2016 10:06 AM >> To: Kusztal, ArkadiuszX; Doherty, Declan >> Cc: Griffin, John; Trahe,

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

2016-10-10 Thread Jianbo Liu
On 10 October 2016 at 10:44, Yuanhan Liu wrote: > On Sun, Oct 09, 2016 at 12:09:07PM +, Wang, Zhihong wrote: >> > > > Tested with testpmd, host: txonly, guest: rxonly >> > > > size (bytes) improvement (%) >> > > > 644.12 >> > > > 128 6 >> > > > 256

[dpdk-dev] [PATCH v3] drivers: prefix driver REGISTER macro with RTE PMD

2016-10-10 Thread Shreyansh Jain
All macros related to driver registeration renamed from DRIVER_* to RTE_PMD_* This includes: DRIVER_REGISTER_PCI -> RTE_PMD_REGISTER_PCI DRIVER_REGISTER_PCI_TABLE -> RTE_PMD_REGISTER_PCI_TABLE DRIVER_REGISTER_VDEV -> RTE_PMD_REGISTER_VDEV DRIVER_REGISTER_PARAM_STRING ->

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

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

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

2016-10-10 Thread Jianbo Liu
On 10 October 2016 at 14:22, Wang, Zhihong wrote: > > >> -Original Message- >> From: Jianbo Liu [mailto:jianbo.liu at linaro.org] >> Sent: Monday, October 10, 2016 1:32 PM >> To: Yuanhan Liu >> Cc: Wang, Zhihong ; Maxime Coquelin >> ; dev at dpdk.org >> Subject: Re: [dpdk-dev] [PATCH v3

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

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

[dpdk-dev] [PATCH 0/3] fix flow director mask issues

2016-10-10 Thread Wenzhuo Lu
There're 2 issues about flow director mask. 1, MAC address mask is not supported in mac-vlan mode. 2, All the parameter are defined as big endian, but they're not treated approriately. This patch set is used to fix these 2 issues. Wenzhuo Lu (3): ixgbe: fix wrong flow director mask

[dpdk-dev] [PATCH 1/3] ixgbe: fix wrong flow director mask

2016-10-10 Thread Wenzhuo Lu
In mac-vlan mode, MAC address mask is not supported by HW. The MAC address mask should not be set in mac-vlan mode. Instead, only set it in tunnel mode. Fixes: 82fb702077f6 ("ixgbe: support new flow director modes for X550") Signed-off-by: Wenzhuo Lu --- drivers/net/ixgbe/ixgbe_fdir.c | 10

[dpdk-dev] [PATCH 2/3] app/testpmd: fix flow director endian issue

2016-10-10 Thread Wenzhuo Lu
The vlan mask and tunnel id mask of flow director are defined as big endian. So they should be converted. When the mask is printed, the parameters are not converted either. Fixes: 53b2bb9b7ea7 ("app/testpmd: new flow director commands") Signed-off-by: Wenzhuo Lu --- app/test-pmd/cmdline.c | 6

[dpdk-dev] [PATCH 3/3] app/testpmd: fix wrong flow director mask

2016-10-10 Thread Wenzhuo Lu
In mac-vlan mode, MAC address mask is not supported by HW. The MAC address mask should not be set in mac-vlan mode. Remove this parameter from the CLI. Fixes: 53b2bb9b7ea7 ("app/testpmd: new flow director commands") Signed-off-by: Wenzhuo Lu --- app/test-pmd/cmdline.c | 5

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

2016-10-10 Thread Yuanhan Liu
On Sun, Oct 09, 2016 at 06:46:44PM +0800, linhaifeng wrote: > ? 2016/8/23 16:10, Yuanhan Liu ??: > > The basic idea of Tx zero copy is, instead of copying data from the > > desc buf, here we let the mbuf reference the desc buf addr directly. > > Is there problem when push vlan to the mbuf which

[dpdk-dev] [PATCH 1/1] fix the number of operations in libcrypto test

2016-10-10 Thread Marcin Kerlin
This patch reduce the number of total operations from 1M to 10K, because test is taking too long time now. Fixes: ffbe3be0d4b5 ("app/test: add libcrypto") Signed-off-by: Marcin Kerlin --- app/test/test_cryptodev_perf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[dpdk-dev] Project Governance and Linux Foundation

2016-10-10 Thread O'Driscoll, Tim
This email is being sent on behalf of: Cavium, Cisco, Intel, NXP & Red Hat. Since its creation as an open source project in 2013, DPDK has grown significantly. The number of DPDK users, contributors, commercial products that use DPDK and open source projects that depend on it have all

[dpdk-dev] [RFC v2] Generic flow director/filtering/classification API

2016-10-10 Thread Zhao1, Wei
Hi Adrien Mazarguil, In your v2 version of rte_flow.txt , there is an action type RTE_FLOW_ACTION_TYPE_MARK, but there is no definition of struct rte_flow_action_mark. And there is an definition of struct rte_flow_action_id. Is it a typo or other usage? Thank you. struct rte_flow_action_id

[dpdk-dev] [PATCH 1/2] ethdev: fix hotplug attach

2016-10-10 Thread Thomas Monjalon
2016-10-07 15:01, David Marchand: > If a pci probe operation creates a port but, for any reason, fails to > finish this operation and decides to delete the newly created port, then > the last created port id can not be trusted anymore and any subsequent > attach operations will fail. > > This

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

2016-10-10 Thread Xu, Qian Q
Good to know. I will try v3. BTW, on the master branch, seems vhost PMD is broken. When we run --vdev 'eth_vhost0,', then it will report the error that the driver is not supported. V16.07 is OK, but I haven't got time to do git bisect. -Original Message- From: Yuanhan Liu

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

2016-10-10 Thread Maxime Coquelin
Hi Xu, On 10/10/2016 12:12 PM, Xu, Qian Q wrote: > Good to know. I will try v3. BTW, on the master branch, seems vhost PMD is > broken. When we run --vdev 'eth_vhost0,', then it will report the error > that the driver is not supported. V16.07 is OK, but I haven't got time to do > git

[dpdk-dev] [PATCH v2 14/15] ethdev: Support VFs on the different PCI domains

2016-10-10 Thread Ferruh Yigit
Hi Kamil, On 9/30/2016 1:05 PM, Kamil Rytarowski wrote: > It's possible to have the same numbers for bus, device id and function, > therefore we need to differentiate on domain. > > This enables DPDK with multiple VFs on ThunderX 2-socket hardware. > > Signed-off-by: Maciej Czekaj >

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

2016-10-10 Thread Xu, Qian Q
Oh, thx for the info, it's better to have some documentation update in R16.11 release notes. -Original Message- From: Maxime Coquelin [mailto:maxime.coque...@redhat.com] Sent: Monday, October 10, 2016 11:14 AM To: Xu, Qian Q ; Yuanhan Liu Cc: dev at dpdk.org Subject: Re: [dpdk-dev]

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

2016-10-10 Thread Xu, Qian Q
I'm a little concerned if it's a correct way to change the name from release to release, some users may use eth_vhost for the driver, and found it was not working. Do we need also make the consistency b/w releases? What's the name difference b/w eth and net? Other PMDs are not virtual devices.

[dpdk-dev] DPDK Community Survey - about to close

2016-10-10 Thread Glynn, Michael J
-Original Message- From: users [mailto:users-boun...@dpdk.org] On Behalf Of Glynn, Michael J Sent: Monday, August 29, 2016 6:14 PM To: users at dpdk.org; dev at dpdk.org; announce at dpdk.org Subject: [dpdk-users] FW: DPDK Community Survey - about to close > ... > > Regarding

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

2016-10-10 Thread Kavanagh, Mark B
> >On Thu, Sep 29, 2016 at 04:31:30PM -0400, Dey wrote: >> /* >> * dev_ops for virtio, bare necessities for basic operation >> */ >> @@ -677,7 +685,6 @@ static const struct eth_dev_ops virtio_eth_dev_ops = { >> .allmulticast_enable = virtio_dev_allmulticast_enable, >>

[dpdk-dev] OpenSSL libcrypto PMD name

2016-10-10 Thread Thomas Monjalon
Hi, I would like to raise a naming issue in crypto. In the crypto side of DPDK, we have a library (similar to ethdev) for crypto API and device interface: http://dpdk.org/browse/dpdk/tree/lib/librte_cryptodev There are also some drivers (which are some libraries):

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

2016-10-10 Thread Maxime Coquelin
On 10/10/2016 12:22 PM, Xu, Qian Q wrote: > Oh, thx for the info, it's better to have some documentation update in R16.11 > release notes. I'm not the author of this change, just faced the same situation as yours a a user. But I agree that if documentation is not aligned, it should be updated.

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

2016-10-10 Thread Sergio Gonzalez Monroy
On 07/10/2016 21:53, De Lara Guarch, Pablo wrote: >> -Original Message- >> From: Akhil Goyal [mailto:akhil.goyal at nxp.com] >> Sent: Tuesday, October 04, 2016 11:33 PM >> To: De Lara Guarch, Pablo; Gonzalez Monroy, Sergio; dev at dpdk.org >> Subject: Re: [PATCH] examples/ipsec-secgw:

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

2016-10-10 Thread Ferruh Yigit
Hi Pattan, On 10/5/2016 11:38 AM, Pattan, Reshma wrote: > Hi Ferruh, > > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ferruh Yigit >> Sent: Thursday, September 15, 2016 4:46 PM >> To: dev at dpdk.org >> Cc: Stephen Hemminger ; Ferruh Yigit >> >>

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

2016-10-10 Thread Maxime Coquelin
On 10/10/2016 06:22 AM, Yuanhan Liu wrote: > On Mon, Oct 10, 2016 at 07:17:06AM +0300, Michael S. Tsirkin wrote: >> On Mon, Oct 10, 2016 at 12:05:31PM +0800, Yuanhan Liu wrote: >>> On Fri, Sep 30, 2016 at 10:16:43PM +0300, Michael S. Tsirkin wrote: >> And the same is done is done in DPDK:

[dpdk-dev] [PATCH 1/3] eal/drivers: prefix driver REGISTER macros with EAL

2016-10-10 Thread Neil Horman
On Sat, Oct 08, 2016 at 01:00:59PM +, Shreyansh Jain wrote: > Hi Thomas, > > > -Original Message- > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Sent: Friday, October 07, 2016 7:22 PM > > To: Shreyansh Jain > > Cc: david.marchand at 6wind.com; dev at dpdk.org > >

[dpdk-dev] [PATCH v2 14/15] ethdev: Support VFs on the different PCI domains

2016-10-10 Thread Kamil Rytarowski
W dniu 10.10.2016 o 12:19, Ferruh Yigit pisze: > Hi Kamil, > > On 9/30/2016 1:05 PM, Kamil Rytarowski wrote: >> It's possible to have the same numbers for bus, device id and function, >> therefore we need to differentiate on domain. >> >> This enables DPDK with multiple VFs on ThunderX 2-socket

[dpdk-dev] [RFC v2] Generic flow director/filtering/classification API

2016-10-10 Thread Adrien Mazarguil
Hi Wei, On Mon, Oct 10, 2016 at 09:42:53AM +, Zhao1, Wei wrote: > Hi Adrien Mazarguil, > > In your v2 version of rte_flow.txt , there is an action type > RTE_FLOW_ACTION_TYPE_MARK, but there is no definition of struct > rte_flow_action_mark. > And there is an definition of struct

[dpdk-dev] [PATCH v3] net/kni: add KNI PMD

2016-10-10 Thread Ferruh Yigit
Add KNI PMD which wraps librte_kni for ease of use. KNI PMD can be used as any regular PMD to send / receive packets to the Linux networking stack. Signed-off-by: Ferruh Yigit --- v3: * rebase on top of latest master v2: * updated driver name eth_kni -> net_kni --- config/common_base

[dpdk-dev] [PATCH] net/i40e: add additional prefetch instructions for bulk rx

2016-10-10 Thread Wu, Jingjing
> -Original Message- > From: Yigit, Ferruh > Sent: Wednesday, September 14, 2016 9:25 PM > To: Vladyslav Buslov ; Zhang, Helin > ; Wu, Jingjing > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] net/i40e: add additional prefetch > instructions for bulk rx > > On 7/14/2016 6:27

[dpdk-dev] [PATCH v2 14/15] ethdev: Support VFs on the different PCI domains

2016-10-10 Thread Ferruh Yigit
On 10/10/2016 2:01 PM, Kamil Rytarowski wrote: > > > W dniu 10.10.2016 o 12:19, Ferruh Yigit pisze: >> Hi Kamil, >> >> On 9/30/2016 1:05 PM, Kamil Rytarowski wrote: >>> It's possible to have the same numbers for bus, device id and function, >>> therefore we need to differentiate on domain. >>>

[dpdk-dev] [PATCH] net/mlx5: fix init on secondary process

2016-10-10 Thread Ferruh Yigit
On 9/28/2016 3:24 PM, Olivier Gournet wrote: > Fixes: 1d88ba171942 ("net/mlx5: refactor Tx data path") > > Signed-off-by: Olivier Gournet > --- Adding maintainer to the thread

[dpdk-dev] [PATCH v4 4/6] i40e: add Tx preparation

2016-10-10 Thread Wu, Jingjing
> #include "i40e_logs.h" > #include "base/i40e_prototype.h" > @@ -79,6 +81,17 @@ > PKT_TX_TCP_SEG | \ > PKT_TX_OUTER_IP_CKSUM) > > +#define I40E_TX_OFFLOAD_MASK ( \ > + PKT_TX_IP_CKSUM | \ > + PKT_TX_L4_MASK |\ >

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

2016-10-10 Thread Thomas Monjalon
Hi, Now that the feature seems to meet a consensus, I've looked at it more closely before integrating. Sorry if it appears like a late review. 2016-09-30 11:00, Tomasz Kulasek: > Added API for `rte_eth_tx_prep` I would love to read the usability and performance considerations here. No need for

[dpdk-dev] [PATCH] net/enic: update enic guide and add warning for invalid conf

2016-10-10 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of John Daley > Sent: Thursday, September 29, 2016 9:55 PM > To: Richardson, Bruce > Cc: dev at dpdk.org; Nelson Escobar > Subject: [dpdk-dev] [PATCH] net/enic: update enic guide and add warning > for invalid

[dpdk-dev] [PATCH v2] mk: gcc -march support for intel processors code names

2016-10-10 Thread Thomas Monjalon
2016-09-28 10:54, Liu, Yong: > Tested-by: Yong Liu > FangFang Wei [...] >Description: Build test on different distributions >Command / instruction: > Verify build pass on listed distributions. > > OS GCCKernel > Ubuntu 16.04

[dpdk-dev] [PATCH v7 2/2] net/ixgbe: add callback to user app on VF to PF mbox msg

2016-10-10 Thread Iremonger, Bernard
Hi Thomas, > Subject: Re: [dpdk-dev] [PATCH v7 2/2] net/ixgbe: add callback to user app > on VF to PF mbox msg > > 2016-10-07 17:46, Bernard Iremonger: > > +struct rte_pmd_ixgbe_mb_event_param { > > + uint16_t vfid; /**< Virtual Function number */ > > + uint16_t msg_type; /**< message

[dpdk-dev] [PATCH v8 0/2] modify callback for VF management

2016-10-10 Thread Bernard Iremonger
This patchset modifies the callback function for VF management. A third parameter has been added to the _rte_eth_dev_callback_process function. All references to this function have been updated. Changes have been made to the ixgbe_rcv_msg_from_vf function to use the new callback parameter. This

[dpdk-dev] [PATCH v8 1/2] librte_ether: modify internal callback function

2016-10-10 Thread Bernard Iremonger
add cb_arg parameter to the _rte_eth_dev_callback_process function. Adding a parameter to this function allows passing information to the application when an eth device event occurs such as a VF to PF message. This allows the application to decide if a particular function is permitted.

[dpdk-dev] [PATCH v8 2/2] net/ixgbe: add callback to user app on VF to PF mbox msg

2016-10-10 Thread Bernard Iremonger
call _rte_eth_dev_callback_process from ixgbe_rcv_msg_from_vf function. The callback asks the user application if it is allowed to perform the function. If the cb_param.retval is RTE_PMD_IXGBE_MB_EVENT_PROCEED then continue, if 0, do nothing and send ACK to VF if > 1, do nothing and send NAK to

[dpdk-dev] [PATCH] pdump: fix dir permissions value in mkdir call

2016-10-10 Thread Reshma Pattan
From: Reshma Pattan Inside the function pdump_get_socket_path(), pdump socket directories are created using mkdir() call with permissions 700, which was assigning wrong permissions to the directories i.e. "d-w-r-xr-T" instead of drwx---. The reason is mkdir() call

[dpdk-dev] [PATCH v2] net/i40e: remove weak symbols

2016-10-10 Thread Wu, Jingjing
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Zoltan Kiss > Sent: Thursday, July 21, 2016 1:11 AM > To: dev at dpdk.org > Cc: Matias Elo ; Gonzalez Monroy, Sergio > ; Yigit, Ferruh intel.com>; > damarion at cisco.com; thomas.monjalon at 6wind.com >

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

2016-10-10 Thread Yuanhan Liu
On Mon, Oct 10, 2016 at 02:40:44PM +0200, Maxime Coquelin wrote: > >>>At that time, a packet always use 2 descs. Since indirect desc is > >>>enabled (by default) now, the assumption is not true then. What's > >>>worse, it might even slow things a bit down. That should also be > >>>part of the

[dpdk-dev] [PATCH v3 03/16] mbuf: move packet type definitions in a new file

2016-10-10 Thread Thomas Monjalon
2016-10-03 10:38, Olivier Matz: > The file rte_mbuf.h starts to be quite big, and next commits > will introduce more functions related to packet types. Let's > move them in a new file. > > Signed-off-by: Olivier Matz > --- > lib/librte_mbuf/Makefile | 2 +- >

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

2016-10-10 Thread Maxime Coquelin
On 10/10/2016 04:42 PM, Yuanhan Liu wrote: > On Mon, Oct 10, 2016 at 02:40:44PM +0200, Maxime Coquelin wrote: > At that time, a packet always use 2 descs. Since indirect desc is > enabled (by default) now, the assumption is not true then. What's > worse, it might even slow things a

[dpdk-dev] [PATCH 1/4] testpmd: Add support to configure 25G and 50G speeds

2016-10-10 Thread Ferruh Yigit
Hi Ajit, On 9/29/2016 6:03 PM, Ajit Khaparde wrote: > Support to configure 25G and 50G speeds is missing from testpmd. > This patch also updates the testpmd user guide accordingly. > > Signed-off-by: Ajit Khaparde This patch seems not really part of the patchset for bnxt driver, but standalone

[dpdk-dev] [PATCH v2] mk: gcc -march support for intel processors code names

2016-10-10 Thread Pattan, Reshma
Hi Thomas > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Monday, October 10, 2016 3:26 PM > To: Liu, Yong ; Pattan, Reshma > > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2] mk: gcc -march support for intel processors > code names >

[dpdk-dev] [PATCH 2/4] bnxt: Fix a segfault encountered during KNI exit

2016-10-10 Thread Ferruh Yigit
On 9/29/2016 8:06 PM, Ajit Khaparde wrote: > On Thu, Sep 29, 2016 at 12:41 PM, Ferruh Yigit >wrote: > > On 9/29/2016 6:03 PM, Ajit Khaparde wrote: > > The bnxt PMD is running into a segfault when exiting out of KNI > > KNI? > > examples/kni > >

[dpdk-dev] [PATCH] app/test: fix crypto mbuf pool size

2016-10-10 Thread Trahe, Fiona
Hi Piotr, > -Original Message- > From: Azarewicz, PiotrX T > Sent: Friday, October 7, 2016 12:35 PM > To: Trahe, Fiona > Cc: dev at dpdk.org > Subject: [PATCH] app/test: fix crypto mbuf pool size > > This patch fix that created pool for crypto mbufs may be too big in some >

[dpdk-dev] [PATCH 3/4] bnxt: Add support for Async Link Notification

2016-10-10 Thread Ferruh Yigit
Hi Ajit, On 9/29/2016 6:03 PM, Ajit Khaparde wrote: > This patch adds support to get Link notification asynchronously. > The HW sends Async notifications on default completion ring. The > PMD processes these notifications and logs a message appropriately. > > Signed-off-by: Ajit Khaparde > ---

[dpdk-dev] 17.02 Roadmap

2016-10-10 Thread O'Driscoll, Tim
We published our initial roadmap for 17.02 at the end of August. Since then we've been doing more detailed planning and would like to provide an update on the features that we plan to submit for this release. This is our current plan, which should hopefully remain fairly stable now: Consistent

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

2016-10-10 Thread De Lara Guarch, Pablo
Hi Akhil, > -Original Message- [...] > Hi Pablo, > > I have already rebased this patch to dpdk-next-crypto subtree. Please > let me know if there is any issue. > Here is my git log > 779f1301a382b6b9e2877fd0357bba33d1242d65 test_cryptodev_perf: IV and > digest should be stored at a

[dpdk-dev] [PATCH] net/i40e: add additional prefetch instructions for bulk rx

2016-10-10 Thread Vladyslav Buslov
> -Original Message- > From: Wu, Jingjing [mailto:jingjing.wu at intel.com] > Sent: Monday, October 10, 2016 4:26 PM > To: Yigit, Ferruh; Vladyslav Buslov; Zhang, Helin > Cc: dev at dpdk.org > Subject: RE: [dpdk-dev] [PATCH] net/i40e: add additional prefetch > instructions for bulk rx > >

[dpdk-dev] [PATCH v2] mk: gcc -march support for intel processors code names

2016-10-10 Thread Thomas Monjalon
2016-10-10 15:21, Pattan, Reshma: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Why not recommend GCC 4.9 and keep the graceful degradation for older > > versions, at least for one more year, even if it is not optimal for newer > > architectures? > > I am ok with the idea, so

[dpdk-dev] [PATCH v4 4/6] i40e: add Tx preparation

2016-10-10 Thread Kulasek, TomaszX
Hi Jingjing, > -Original Message- > From: Wu, Jingjing > Sent: Monday, October 10, 2016 16:03 > To: Kulasek, TomaszX ; dev at dpdk.org > Cc: Ananyev, Konstantin ; Kulasek, TomaszX > > Subject: RE: [dpdk-dev] [PATCH v4 4/6] i40e: add Tx preparation > > > #include "i40e_logs.h" > >

[dpdk-dev] [PATCH 1/4] testpmd: Add support to configure 25G and 50G speeds

2016-10-10 Thread Ajit Khaparde
On Mon, Oct 10, 2016 at 10:01 AM, Ferruh Yigit wrote: > Hi Ajit, > > On 9/29/2016 6:03 PM, Ajit Khaparde wrote: > > Support to configure 25G and 50G speeds is missing from testpmd. > > This patch also updates the testpmd user guide accordingly. > > > > Signed-off-by: Ajit Khaparde > > This

[dpdk-dev] [PATCH 2/4] bnxt: Fix a segfault encountered during KNI exit

2016-10-10 Thread Ajit Khaparde
On Mon, Oct 10, 2016 at 10:24 AM, Ferruh Yigit wrote: > On 9/29/2016 8:06 PM, Ajit Khaparde wrote: > > On Thu, Sep 29, 2016 at 12:41 PM, Ferruh Yigit > >wrote: > > > > On 9/29/2016 6:03 PM, Ajit Khaparde wrote: > > > The bnxt PMD is running into a

[dpdk-dev] [PATCH v2 4/4] net/bnxt: add ULL suffix to constant 1 for bit shift in bnxt_mac_addr_remove_op

2016-10-10 Thread Ajit Khaparde
On Thu, Sep 29, 2016 at 12:39 PM, John W. Linville wrote: > Some(?) compilers will treat the unmarked constant 1 as a 32-bit > integer, but the shift operation is in a loop that could run up to > 63 times -- undefined behavior! > > Coverity issue: 127546 > Fixes: 778b759ba10e ("net/bnxt: add MAC

[dpdk-dev] 17.02 Roadmap

2016-10-10 Thread Thomas Monjalon
Thanks Tim for the interesting inputs. Some of them may require a dedicated thread to continue the discussion based on some preliminary specifications or drafts. 2016-10-10 16:13, O'Driscoll, Tim: > Elastic Flow Distributor: The Elastic Flow Distributor (EFD) is a flow-based > load balancing

[dpdk-dev] [PATCH v3] mk: gcc -march support for intel processors code names

2016-10-10 Thread Reshma Pattan
The GCC 4.9 -march option supports the intel code names for processors, for example -march=silvermont, -march=broadwell. The RTE_MACHINE config flag can be used to pass code name to the compiler as -march flag. Release notes is updated. Linux and FreeBSD getting started guides are updated with

[dpdk-dev] [PATCH] pdump: fix dir permissions value in mkdir call

2016-10-10 Thread Remy Horton
On 10/10/2016 15:35, Reshma Pattan wrote: [..] > Fixes: e4ffa2d3 ("pdump: fix error handlings") > Fixes: bdd8dcc6 ("pdump: fix default socket path") > > Reported-by: Jianfeng Tan > Signed-off-by: Reshma Pattan > --- > lib/librte_pdump/rte_pdump.c | 4 ++-- > 1 file changed, 2 insertions(+), 2

[dpdk-dev] [PATCH v2] log: respect rte_openlog_stream calls before rte_eal_init

2016-10-10 Thread John Ousterhout
Before this patch, application-specific loggers could not be installed before rte_eal_init completed (the initialization process called rte_openlog_stream, overwriting any previously installed logger). This made it impossible for an application to capture the initial log messages generated during

[dpdk-dev] [PATCH 3/4] bnxt: Add support for Async Link Notification

2016-10-10 Thread Ajit Khaparde
On Mon, Oct 10, 2016 at 10:40 AM, Ferruh Yigit wrote: > Hi Ajit, > > On 9/29/2016 6:03 PM, Ajit Khaparde wrote: > > This patch adds support to get Link notification asynchronously. > > The HW sends Async notifications on default completion ring. The > > PMD processes these notifications and logs

[dpdk-dev] DPDK patchwork upgrade

2016-10-10 Thread Stephen Finucane
> 2016-09-07 16:40, De Lara Guarch, Pablo: > > I am looking at it, and I see two issues: > > - There are some patches that have been acked, but I don't see that in > > patchwork > > (e.g. http://dpdk.org/dev/patchwork/patch/15381/) > > Yes the new columns are empty. > I guess they will be filled