[dpdk-dev] [PATCH 1/3] cxgbe: Fix RX performance for cxgbe PMD.

2015-07-07 Thread Thomas Monjalon
2015-07-07 22:42, Rahul Lakkireddy: > CXGBE PMD rx allocates a new mbuf everytime, which could lead to performance > hit. Instead, do bulk allocation of mbufs and re-use them. > > Also, simplify the overall rx-handler, and update its logic to fix rx perf. For such change, it would be nice to pro

[dpdk-dev] [PATCH v2 0/4] extend flow director to support L2_paylod type

2015-07-07 Thread Thomas Monjalon
> > This patch set extends flow director to support L2_paylod type in i40e > > driver. > > > > v2 change: > > - remove the flow director VF filtering from this patch to avoid breaking > > ABI. > > > > Jingjing Wu (4): > > ethdev: add struct rte_eth_l2_flow to support l2_payload flow type > >

[dpdk-dev] [PATCH 3/3] doc: Update cxgbe documentation

2015-07-07 Thread Rahul Lakkireddy
Update cxgbe documentation with the following: 1. Add instructions on how to flash firmware and config file on Chelsio T5 devices. 2. Add a list of supported devices. 3. Add instructions on how to enable and disable flow control via testpmd. Signed-off-by: Rahul Lakkireddy Signed-off-by: Kumar

[dpdk-dev] [PATCH 2/3] cxgbe: Add more supported Chelsio T5 devices.

2015-07-07 Thread Rahul Lakkireddy
Update pci id table to include more supported Chelsio T5 devices. Signed-off-by: Rahul Lakkireddy Signed-off-by: Kumar Sanghvi --- drivers/net/cxgbe/base/t4_pci_id_tbl.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/net/cxgbe/base/t4_pci_id_tbl.h b/drivers/net/cxgbe/base/t4_pc

[dpdk-dev] [PATCH 1/3] cxgbe: Fix RX performance for cxgbe PMD.

2015-07-07 Thread Rahul Lakkireddy
CXGBE PMD rx allocates a new mbuf everytime, which could lead to performance hit. Instead, do bulk allocation of mbufs and re-use them. Also, simplify the overall rx-handler, and update its logic to fix rx perf. Signed-off-by: Rahul Lakkireddy Signed-off-by: Kumar Sanghvi --- drivers/net/cxgb

[dpdk-dev] [PATCH 0/3] cxgbe: RX perf fixes, doc update, and add support for more devices

2015-07-07 Thread Rahul Lakkireddy
This series of patches do the following: 1. RX performance fixes for cxgbe Poll Mode Driver. 2. Add more supported Chelsio T5 device ids. 3. Update cxgbe documentation to include instructions on how to flash firmware and enable/disable flow control. Also, provide a list of supported Chelsio T

[dpdk-dev] [PATCH v2] mempool: improve cache search

2015-07-07 Thread Zoltan Kiss
On 02/07/15 18:07, Ananyev, Konstantin wrote: > > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Zoltan Kiss >> Sent: Wednesday, July 01, 2015 10:04 AM >> To: dev at dpdk.org >> Subject: [dpdk-dev] [PATCH v2] mempool: improve cache search >> >> The curren

[dpdk-dev] [PATCH v4 0/4] enable mirror functionality in i40e driver

2015-07-07 Thread Thomas Monjalon
> > This patch set enables mirror functionality in i40e driver, and redefines > > structure > > and macros used to configure mirror. > > > > v2 changes: > > - correct comments style > > - add doc change > > > > v3 changes: > > - change the mirror rule type to support bit mask > > - fix code

[dpdk-dev] [PATCH v3 08/12] mempool: allow config override on element alignment

2015-07-07 Thread Tony Lu
>-Original Message- >From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Bruce Richardson >Sent: Monday, July 06, 2015 11:38 PM >To: Zhigang Lu >Cc: dev at dpdk.org >Subject: Re: [dpdk-dev] [PATCH v3 08/12] mempool: allow config override on >element alignment > >On Mon, Jul 06, 2015 a

[dpdk-dev] [PATCH 4/4] bnx2x: enable PMD build

2015-07-07 Thread Stephen Hemminger
From: Stephen Hemminger This is build infrastructure changes for bnx2x driver. - enable BNX2X poll mode driver in default config. - add it to mk - put entry in MAINTAINERS Note: I intentionally did not list myself as maintainer of this driver. QLogic has discussed taking over as maintainer.

[dpdk-dev] [PATCH 3/4] bnx2x: driver support routines

2015-07-07 Thread Stephen Hemminger
From: Stephen Hemminger More code for the Broadcom/Qlogic NetExtreme II poll mode driver. Split into pieces for review and not to overwhelm mailers. Signed-off-by: Stephen Hemminger --- v3 - change copyright to QLogic --- drivers/net/bnx2x/debug.c | 113 + drivers/net/bnx2

[dpdk-dev] [PATCH 2/4] bnx2x: driver core

2015-07-07 Thread Stephen Hemminger
From: Stephen Hemminger This is the first of several parts for a new driver supporting Broadcom/Qlogic NetXtremeII 10 gigabit devices. Signed-off-by: Stephen Hemminger --- v3 - change copyright to QLogic --- drivers/net/bnx2x/bnx2x.c| 11821 + drive

[dpdk-dev] [PATCH 1/4] eal: provide functions to access PCI config

2015-07-07 Thread Stephen Hemminger
From: Stephen Hemminger Some drivers need ability to access PCI config (for example for power management). This adds an abstraction to do this for both Linux and BSD. Signed-off-by: Stephen Hemminger --- v3 - put new symbols in DPDK 2.1 section --- lib/librte_eal/bsdapp/eal/eal_pci.c

[dpdk-dev] [PATCH v3 0/4] bnx2x: poll mode driver

2015-07-07 Thread Stephen Hemminger
From: Stephen Hemminger Revised version of BNX2X driver based on feedback from QLogic. Nothing major, mostly fixing references to QLogic and adding a couple more PCI ids Stephen Hemminger (4): eal: provide functions to access PCI config bnx2x: driver core bnx2x: driver support routines b

[dpdk-dev] [PATCH v4 1/4] ethdev: rename rte_eth_vmdq_mirror_conf

2015-07-07 Thread Thomas Monjalon
2015-06-26 07:03, Wu, Jingjing: > Hi, Neil > > About this patch I have an ABI concern about it. > This patch just renamed a struct rte_eth_vmdq_mirror_conf to > rte_eth_mirror_conf, the size and its elements don't change. > As my understanding, it will not break the ABI. And I also tested it. > B

[dpdk-dev] [PATCH v3 0/7] support i40e QinQ stripping and insertion

2015-07-07 Thread Thomas Monjalon
> > As i40e hardware can be reconfigured to support QinQ stripping and > > insertion, > > this patch set is to enable that together with using the reserved 16 bits in > > 'struct rte_mbuf' for the second vlan tag. Corresponding command is added > > in testpmd for testing. > > Note that no need to

[dpdk-dev] [PATCH] rte_power: fix sample vm_power_mgr not work with Haswell cpus

2015-07-07 Thread Yong Liu
From: Marvin Liu Two socket Haswell cpus on cottonwoodPass have 72 cores totally. And this number oversize the default value of maximum core number 64. Total number of channels should also increased for match core number. Signed-off-by: Marvin Liu diff --git a/examples/vm_power_manager/power_

[dpdk-dev] [PATCH v3 02/12] hash: fix compilation on non-X86 platforms

2015-07-07 Thread Tony Lu
>-Original Message- >From: Bruce Richardson [mailto:bruce.richardson at intel.com] >Sent: Monday, July 06, 2015 7:15 PM >To: Zhigang Lu >Cc: dev at dpdk.org >Subject: Re: [dpdk-dev] [PATCH v3 02/12] hash: fix compilation on non-X86 >platforms > >On Mon, Jul 06, 2015 at 04:51:27PM +0800, Z

[dpdk-dev] [PATCH v3 01/12] test: limit x86 cpuflags checks to x86 builds

2015-07-07 Thread Tony Lu
>-Original Message- >From: Bruce Richardson [mailto:bruce.richardson at intel.com] >Sent: Monday, July 06, 2015 6:53 PM >To: Zhigang Lu >Cc: dev at dpdk.org >Subject: Re: [dpdk-dev] [PATCH v3 01/12] test: limit x86 cpuflags checks to x86 >builds > >On Mon, Jul 06, 2015 at 04:51:26PM +0800

[dpdk-dev] [PATCH v2] vfio: Fix overflow while assigning vfio BAR region offset and size

2015-07-07 Thread Rahul Lakkireddy
Hi Anatoly, On Tue, Jul 07, 2015 at 09:08:08 +, Burakov, Anatoly wrote: > Hi all, > > > So, as VFIO support for 32bit systems is broken, DPDK should not configure > > VFIO in that case. > > ...Or no one should try and run VFIO on a 32-bit system, which should be > noted in documentation. I

[dpdk-dev] [PATCH 3/3] testpmd: add sport and dport configuration for sctp flow

2015-07-07 Thread Jingjing Wu
Signed-off-by: Jingjing Wu --- app/test-pmd/cmdline.c | 12 1 file changed, 12 insertions(+) diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index 8142910..45a5da0 100644 --- a/app/test-pmd/cmdline.c +++ b/app/test-pmd/cmdline.c @@ -7812,6 +7812,12 @@ cmd_flow_director_

[dpdk-dev] [PATCH 2/3] i40e: make sport and dport of sctp flow involved in match

2015-07-07 Thread Jingjing Wu
Due to the NIC's firmware update, the input set of sctp flow is changed to source IP, destination IP, source port, destination port and Verification-Tag. This patch adds the sport and dport in the programming packet of flow director. Signed-off-by: Jingjing Wu --- drivers/net/i40e/i40e_fdir.c |

[dpdk-dev] [PATCH 1/3] ethdev: change the input set of sctp flow

2015-07-07 Thread Jingjing Wu
add sport and dport into the input set of sctp flow. Signed-off-by: Jingjing Wu --- lib/librte_ether/rte_eth_ctrl.h | 8 1 file changed, 8 insertions(+) diff --git a/lib/librte_ether/rte_eth_ctrl.h b/lib/librte_ether/rte_eth_ctrl.h index 9187d6d..60e9aa6 100644 --- a/lib/librte_ether/r

[dpdk-dev] [PATCH 0/3] fix the issue sctp flow cannot be matched in FVL FDIR

2015-07-07 Thread Jingjing Wu
This patch set fixes the issue SCTP flow cannot be matched by FVL's flow director. The issue's root cause is that due to the NIC's firmware update, the input set of sctp flow are changed to source IP, destination IP, source port, destination port and Verification-Tag, which are source IP, destinati

[dpdk-dev] [PATCH v3 02/12] hash: fix compilation on non-X86 platforms

2015-07-07 Thread Sanford, Robert
Hi Zhigang, Can you add this one-line X86 change (below)? When included by C++ code, 'key' needs an explicit type cast. You may want to put the 'const' keyword in your non-X86 change, too. -- Thanks, Robert diff --git a/lib/librte_hash/rte_jhash.h b/lib/librte_hash/rte_jhash.h index e230449.

[dpdk-dev] [PATCH v4 1/1] ring: changes to support PCI Port Hotplug

2015-07-07 Thread Bernard Iremonger
This patch depends on the Port Hotplug Framework. It implements the rte_dev_uninit_t() function for the ring pmd. Signed-off-by: Bernard Iremonger Acked-by: Bruce Richardson --- drivers/net/ring/rte_eth_ring.c | 97 ++- 1 files changed, 75 insertions(+), 22

[dpdk-dev] [PATCH v4 0/1] ring PCI Port Hotplug

2015-07-07 Thread Bernard Iremonger
Changes in V4: Rebase to latest code. Update commit line, no changes to code. Changes in V3: Rebase to use drivers/net/ring directory Handle no parameters case Changes in V2: Fix crash in the rte_pmd_ring_devuninit() function. Bernard Iremonger (1): ring: changes to support PCI Port Hotplug

[dpdk-dev] [PATCH v12 4/4] examples: new example: l2fwd-ethtool

2015-07-07 Thread Liang-Min Larry Wang
The example includes an ethtool library and two applications: one application is a non- DPDK process (nic-control) and the other is a DPDK l2fwd applicaiton (l2fwd-app). The nic-control process sends ethtool alike device management requests to l2fwd-app through a named pipe IPC. This example is des

[dpdk-dev] [PATCH v12 3/4] igb: add ops to support ethtool ops

2015-07-07 Thread Liang-Min Larry Wang
add function to support ethtool ops: - set_mac_addr - get_reg_length - get_regs - get_eeprom_length - get_eeprom - set_eeprom Signed-off-by: Liang-Min Larry Wang --- drivers/net/e1000/igb_ethdev.c | 175 drivers/net/e1000/igb_regs.h | 223 ++

[dpdk-dev] [PATCH v12 2/4] ixgbe: add ops to support ethtool ops

2015-07-07 Thread Liang-Min Larry Wang
add function to support ethtool ops: - set_mac_addr - get_reg_length - get_regs - get_eeprom_length - get_eeprom - set_eeprom Signed-off-by: Liang-Min Larry Wang --- drivers/net/ixgbe/ixgbe_ethdev.c | 178 +- drivers/net/ixgbe/ixgbe_regs.h | 376

[dpdk-dev] [PATCH v12 1/4] ethdev: add apis to support access device info

2015-07-07 Thread Liang-Min Larry Wang
add two new apis: rte_eth_dev_default_mac_addr_set and rte_eth_ethtool_dev_info to enable reading device parameters (mac, register, eeprom, pause, ring) based upon ethtool alike data parameter sepcification. Signed-off-by: Liang-Min Larry Wang --- lib/librte_ether/Makefile | 1 +

[dpdk-dev] [PATCH v12 0/4] User-space Ethtool

2015-07-07 Thread Liang-Min Larry Wang
This implementation is designed to provide a familar interface for applications that rely on kernel-space driver to support ethtool_op and net_device_op for device management. The initial implementation focuses on ops that can be implemented through existing netdev APIs. More ops will be support

[dpdk-dev] [PATCH v2] vfio: Fix overflow while assigning vfio BAR region offset and size

2015-07-07 Thread Rahul Lakkireddy
Hi Alejandro, On Mon, Jul 06, 2015 at 16:45:01 +0100, Alejandro Lucero wrote: > Hi all, > > From the kernel VFIO maintainer: > > "I suppose in the short term, mmap should not be advertised as available > on 32bit hosts. Thanks," > > So, as VFIO support for 32bit systems is broken, DPDK should

[dpdk-dev] [PATCH v2] librte_ether: release memory in uninit function.

2015-07-07 Thread Tetsuya Mukawa
On 2015/07/06 20:35, Qiu, Michael wrote: > Hi, all > > As we has gap on the memory release action to be done in which step, I > appreciate all your comments on this patch. > > Currently, the correct quit sequence for testpmd is stop() ---> > port_stop() --> port_close() --> quit(). This will lead l

[dpdk-dev] [PATCH v6 00/11] ip_pipeline: ip_pipeline application enhancements

2015-07-07 Thread Dumitrescu, Cristian
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > Sent: Tuesday, July 7, 2015 9:28 AM > To: Gajdzica, MaciejX T > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v6 00/11] ip_pipeline: ip_pipeline application > enhancements > > 2015-07-0

[dpdk-dev] [PATCH v4 3/3] bonding: free queue memory in stop function

2015-07-07 Thread Bernard Iremonger
add function bond_ethdev_free_queues() and call from the bond_ethdev_stop() function. Signed-off-by: Bernard Iremonger --- drivers/net/bonding/rte_eth_bond_pmd.c | 20 1 files changed, 20 insertions(+), 0 deletions(-) diff --git a/drivers/net/bonding/rte_eth_bond_pmd.c

[dpdk-dev] [PATCH v4 2/3] test-pmd: modified testpmd for link_bonding

2015-07-07 Thread Bernard Iremonger
When the bonded port is started it also starts the slave port, but the slave port status is not set. A slave_flag has been added to struct rte_port to resolve this issue. Signed-off-by: Bernard Iremonger --- app/test-pmd/cmdline.c |4 +++- app/test-pmd/testpmd.c | 24 +-

[dpdk-dev] [PATCH v4 1/3] bonding: add support for PCI Port Hotplug

2015-07-07 Thread Bernard Iremonger
This patch depends on the Port Hotplug Framework. It implements the rte_dev_uninit_t() function for the link bonding pmd. Signed-off-by: Bernard Iremonger --- drivers/net/bonding/rte_eth_bond.h | 13 +- drivers/net/bonding/rte_eth_bond_api.c | 75 ++-

[dpdk-dev] [PATCH v4 0/3] bonding PCI Port Hotplug

2015-07-07 Thread Bernard Iremonger
Changes in V4: Rebase to latest code. Move freeing of queue memory from uninit() function to close() function. Changes in V3: Rebase to latest code. Modified testpmd to handle attach/detach of bonding pmd. Changes in V2: Rebased to use drivers/net/bonding dirctory Free rx and tx queues in uninit(

[dpdk-dev] [PATCH v3 08/12] mempool: allow config override on element alignment

2015-07-07 Thread Bruce Richardson
On Tue, Jul 07, 2015 at 05:15:41PM +0800, Tony Lu wrote: > > > >-Original Message- > >From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Bruce Richardson > >Sent: Monday, July 06, 2015 11:38 PM > >To: Zhigang Lu > >Cc: dev at dpdk.org > >Subject: Re: [dpdk-dev] [PATCH v3 08/12] mempo

[dpdk-dev] [PATCH v4 3/4] virtio: free queue memory in virtio_dev_close()

2015-07-07 Thread Stephen Hemminger
On Tue, 7 Jul 2015 10:18:06 +0100 Bernard Iremonger wrote: > Add function virtio_free_queues() and call from virtio_dev_close() > > Signed-off-by: Bernard Iremonger > --- > drivers/net/virtio/virtio_ethdev.c | 19 +++ > 1 files changed, 19 insertions(+), 0 deletions(-) > >

[dpdk-dev] [PATCH 6/6] eal/linux: avoid out of bound access

2015-07-07 Thread David Marchand
Using IBM advance toolchain on Ubuntu 14.04 (package 8.0-3), gcc is complaining about out of bound accesses. CC eal_hugepage_info.o lib/librte_eal/linuxapp/eal/eal_hugepage_info.c: In function ?eal_hugepage_info_init?: lib/librte_eal/linuxapp/eal/eal_hugepage_info.c:350:35: error: array subscrip

[dpdk-dev] [PATCH 5/6] eal/linux: indent file

2015-07-07 Thread David Marchand
With this, we should be checkpatch compliant. Signed-off-by: David Marchand --- lib/librte_eal/linuxapp/eal/eal_hugepage_info.c | 73 --- 1 file changed, 39 insertions(+), 34 deletions(-) diff --git a/lib/librte_eal/linuxapp/eal/eal_hugepage_info.c b/lib/librte_eal/linuxa

[dpdk-dev] [PATCH 4/6] eal/linux: rework while loop

2015-07-07 Thread David Marchand
Replace this while loop with a for loop and simplify error handling. Indent is broken on purpose, fixed in next commit. Signed-off-by: David Marchand --- lib/librte_eal/linuxapp/eal/eal_hugepage_info.c | 28 --- 1 file changed, 15 insertions(+), 13 deletions(-) diff --git

[dpdk-dev] [PATCH 3/6] eal/linux: cosmetic change

2015-07-07 Thread David Marchand
Prepare for checkpatch compliance. Signed-off-by: David Marchand --- lib/librte_eal/linuxapp/eal/eal_hugepage_info.c | 28 +-- 1 file changed, 16 insertions(+), 12 deletions(-) diff --git a/lib/librte_eal/linuxapp/eal/eal_hugepage_info.c b/lib/librte_eal/linuxapp/eal/eal_

[dpdk-dev] [PATCH 2/6] eal/linux: remove useless casts

2015-07-07 Thread David Marchand
Rather than cast the huge pages number returned by get_num_hugepages, rework this function so that it returns 0 when something goes wrong. And no need for casts in log. Signed-off-by: David Marchand --- lib/librte_eal/linuxapp/eal/eal_hugepage_info.c | 21 - 1 file changed,

[dpdk-dev] [PATCH 1/6] eal/linux: remove useless check on process type

2015-07-07 Thread David Marchand
The code in eal_hugepage_info.c is not reachable by secondary processes. Signed-off-by: David Marchand --- lib/librte_eal/linuxapp/eal/eal_hugepage_info.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/lib/librte_eal/linuxapp/eal/eal_hugepage_info.c b/l

[dpdk-dev] [PATCH 0/6] eal/linux: cleanup hugepage code

2015-07-07 Thread David Marchand
Here is a little patchset cleaning eal_hugepage_info.c file and fixing a bug when having more than two hugepage sizes. No functional change to be expected. -- David Marchand David Marchand (6): eal/linux: remove useless check on process type eal/linux: remove useless casts eal/linux: cosme

[dpdk-dev] [PATCH v2] librte_ether: release memory in uninit function.

2015-07-07 Thread Iremonger, Bernard
> -Original Message- > From: Tetsuya Mukawa [mailto:mukawa at igel.co.jp] > Sent: Tuesday, July 7, 2015 4:38 AM > To: dev at dpdk.org > Cc: Qiu, Michael; Iremonger, Bernard; thomas.monjalon at 6wind.com; > Ananyev, Konstantin; Stephen Hemminger; Zhang, Helin; Chen, Jing D; Neil > Horman > S

[dpdk-dev] [PATCH v2] vfio: Fix overflow while assigning vfio BAR region offset and size

2015-07-07 Thread Burakov, Anatoly
Hi Rahul, > However, unsigned long seems to be working fine for all builds. unsigned long it is then, if there aren't any other objections. Thanks, Anatoly

[dpdk-dev] [PATCH v6 00/11] ip_pipeline: ip_pipeline application enhancements

2015-07-07 Thread Thomas Monjalon
2015-07-07 10:09, Maciej Gajdzica: > Changes in v4: > - fixed build issue with gcc 5 > - fixed bugs in flow classification and firewall pipelines > > Changes in v5: > - fixed build issue with clang 3.6 > > Changes in v6: > - fixed build issues with icc 15 > - fixed build issues with gcc 32bit Pl

[dpdk-dev] [PATCH v4 4/4] test-pmd: remove call to rte_eth_promiscuous_disable() from detach_port()

2015-07-07 Thread Bernard Iremonger
At this point the stop() and close() functions have already been called. The rte_eth_promiscuous_disable() function does not return on the VM. Signed-off-by: Bernard Iremonger --- app/test-pmd/testpmd.c |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/app/test-pmd/test

[dpdk-dev] [PATCH v4 3/4] virtio: free queue memory in virtio_dev_close()

2015-07-07 Thread Bernard Iremonger
Add function virtio_free_queues() and call from virtio_dev_close() Signed-off-by: Bernard Iremonger --- drivers/net/virtio/virtio_ethdev.c | 19 +++ 1 files changed, 19 insertions(+), 0 deletions(-) diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_e

[dpdk-dev] [PATCH v4 2/4] virtio: check vq parameter in virtqueue_detatch_unused() function

2015-07-07 Thread Bernard Iremonger
If vq is NULL, there is a segmentation fault. Signed-off-by: Bernard Iremonger --- drivers/net/virtio/virtqueue.c | 14 -- 1 files changed, 8 insertions(+), 6 deletions(-) diff --git a/drivers/net/virtio/virtqueue.c b/drivers/net/virtio/virtqueue.c index 8a3005f..7f60e3e 100644 --

[dpdk-dev] [PATCH v4 1/4] virtio: add support for PCI Port Hotplug

2015-07-07 Thread Bernard Iremonger
This patch depends on the Port Hotplug Framework. It implements the eth_dev_uninit_t() function for virtio pmd. Signed-off-by: Bernard Iremonger --- drivers/net/virtio/virtio_ethdev.c | 72 --- 1 files changed, 65 insertions(+), 7 deletions(-) diff --git a/driv

[dpdk-dev] [PATCH v4 0/4] virtio: PCI Port Hotplug

2015-07-07 Thread Bernard Iremonger
This patchset contains changes to add support for PCI Port Hotplug It also contains a fix to virtqueue_detatch_unused(). It also contains a fix to testpmd to facilate testing on VM's. Changes in V4: Rebase to latest code. Move freeing of queue memory from uninit() function to close() function. Cha

[dpdk-dev] [PATCH v6 11/11] ip_pipeline: added new implementation of flow classification pipeline

2015-07-07 Thread Maciej Gajdzica
Flow classification pipeline implementation is split to two files. pipeline_flow_classification.c file handles front-end functions (cli commands parsing) pipeline_flow_classification_ops.c contains implementation of functions done by pipeline (back-end). Signed-off-by: Maciej Gajdzica --- exampl

[dpdk-dev] [PATCH v6 10/11] ip_pipeline: added new implementation of routing pipeline

2015-07-07 Thread Maciej Gajdzica
Routing pipeline implementation is split to two files. pipeline_routing.c file handles front-end functions (cli commands parsing) pipeline_routing_ops.c contains implementation of functions done by pipeline (back-end). Signed-off-by: Pawel Wodkowski --- examples/ip_pipeline/Makefile

[dpdk-dev] [PATCH v6 09/11] ip_pipeline: added new implementation of firewall pipeline

2015-07-07 Thread Maciej Gajdzica
From: Daniel Mrzyglod Firewall pipeline implementation is split to two files. pipeline_firewall.c file handles front-end functions (cli commands parsing) pipeline_firewall_ops.c contains implementation of functions done by pipeline (back-end). Signed-off-by: Daniel Mrzyglod --- examples/ip_pip

[dpdk-dev] [PATCH v6 08/11] ip_pipeline: added new implementation of passthrough pipeline

2015-07-07 Thread Maciej Gajdzica
From: Jasvinder Singh Passthrough pipeline implementation is split to two files. pipeline_passthrough.c file handles front-end functions (cli commands parsing) pipeline_passthrough_ops.c contains implementation of functions done by pipeline (back-end). Signed-off-by: Jasvinder Singh --- exampl

[dpdk-dev] [PATCH v6 07/11] ip_pipeline: moved config files to separate folder

2015-07-07 Thread Maciej Gajdzica
Created new folder for config(.cfg) and script(.sh) files. Signed-off-by: Maciej Gajdzica --- examples/ip_pipeline/config/ip_pipeline.cfg |9 +++ examples/ip_pipeline/config/ip_pipeline.sh |5 ++ examples/ip_pipeline/config/tm_profile.cfg | 105 +++ examples/ip

[dpdk-dev] [PATCH v6 06/11] ip_pipeline: added application thread

2015-07-07 Thread Maciej Gajdzica
Application thread runs pipelines on assigned cores. Signed-off-by: Maciej Gajdzica --- examples/ip_pipeline/Makefile |1 + examples/ip_pipeline/main.c |6 +++ examples/ip_pipeline/thread.c | 110 + 3 files changed, 117 insertions(+) create mod

[dpdk-dev] [PATCH v6 05/11] ip_pipeline: added master pipeline

2015-07-07 Thread Maciej Gajdzica
From: Jasvinder Singh Master pipeline is responsible for command line handling and communicationg with all other pipelines via message queues. Removed cmdline.c file, as its functionality will be split over multiple pipeline files. Signed-off-by: Jasvinder Singh --- examples/ip_pipeline/Makefi

[dpdk-dev] [PATCH v6 04/11] ip_pipeline: moved pipelines to separate folder

2015-07-07 Thread Maciej Gajdzica
Moved pipelines to separate folder, removed not needed pipelines and modified Makefile to match that change. Signed-off-by: Maciej Gajdzica --- examples/ip_pipeline/Makefile |9 +- examples/ip_pipeline/pipeline/pipeline_firewall.c | 313 + .../pipeline/pipe

[dpdk-dev] [PATCH v6 03/11] ip_pipeline: modified init to match new params struct

2015-07-07 Thread Maciej Gajdzica
After changes in config parser, app params struct is changed and requires modifications in initialization procedures. Signed-off-by: Maciej Gajdzica --- examples/ip_pipeline/Makefile |1 + examples/ip_pipeline/init.c | 1643 ++--- examples/ip_pipeline/ma

[dpdk-dev] [PATCH v6 02/11] ip_pipeline: added config checks

2015-07-07 Thread Maciej Gajdzica
From: Jasvinder Singh After loading configuration from a file, data integrity is checked. Signed-off-by: Jasvinder Singh --- examples/ip_pipeline/Makefile |1 + examples/ip_pipeline/config_check.c | 399 +++ examples/ip_pipeline/main.c |2

[dpdk-dev] [PATCH v6 01/11] ip_pipeline: add parsing for config files with new syntax

2015-07-07 Thread Maciej Gajdzica
From: Pawel Wodkowski New syntax of config files is needed for ip_pipeline example enhancements. Some old files are temporarily disabled in the Makefile. It is part of a bigger change. Signed-off-by: Pawel Wodkowski --- examples/ip_pipeline/Makefile | 17 +- examples/ip_pipeline/app

[dpdk-dev] [PATCH v6 00/11] ip_pipeline: ip_pipeline application enhancements

2015-07-07 Thread Maciej Gajdzica
This patchset enhances functionality of ip_pipeline application. New config file syntax is introduced, so parser is changed. Changed structure of the application. Now every global variable is stored in app_struct in app.h. Syntax of pipeline cli commands was changed. Implementation of cli commands

[dpdk-dev] [PATCH v8 12/12] eal: Consolidate rte_eal_pci_probe/close_one_driver() of linuxapp and bsdapp

2015-07-07 Thread David Marchand
On Mon, Jul 6, 2015 at 8:24 AM, Tetsuya Mukawa wrote: > From: "Tetsuya.Mukawa" > > This patch consolidates below functions, and implements these in common > eal code. > - rte_eal_pci_probe_one_driver() > - rte_eal_pci_close_one_driver() > > Signed-off-by: Tetsuya Mukawa > --- > lib/librte_ea

[dpdk-dev] [PATCH v8 10/12] eal: Consolidate pci uio functions of linuxapp and bsdapp

2015-07-07 Thread David Marchand
On Mon, Jul 6, 2015 at 8:24 AM, Tetsuya Mukawa wrote: > From: "Tetsuya.Mukawa" > > The patch consolidates below functions, and implement these > in eal_common_pci_uio.c. > - pci_uio_map_secondary() > - pci_uio_map_resource() > - pci_uio_unmap() > - pci_uio_find_resource() > - pci_uio_unmap_

[dpdk-dev] [PATCH v8 11/12] eal: Consolidate pci_map/unmap_device() of linuxapp and bsdapp

2015-07-07 Thread David Marchand
On Mon, Jul 6, 2015 at 8:24 AM, Tetsuya Mukawa wrote: > From: "Tetsuya.Mukawa" > > The patch consolidates below functions, and implemented in common > eal code. > - pci_map_device() > - pci_unmap_device() > > Signed-off-by: Tetsuya Mukawa > --- > lib/librte_eal/bsdapp/eal/eal_pci.c | 3

[dpdk-dev] [PATCH v8 09/12] eal: Consolidate pci_map/unmap_resource() of linuxapp and bsdapp

2015-07-07 Thread David Marchand
On Mon, Jul 6, 2015 at 8:24 AM, Tetsuya Mukawa wrote: > From: "Tetsuya.Mukawa" > > The patch consolidates below functions, and implemented in common > eal code. > - pci_map_resource() > - pci_unmap_resource() > > Signed-off-by: Tetsuya Mukawa > --- > lib/librte_eal/bsdapp/eal/eal_pci.c

[dpdk-dev] [PATCH v8 08/12] eal: Consolidate pci_map and mapped_pci_resource of linuxapp and bsdapp

2015-07-07 Thread David Marchand
On Mon, Jul 6, 2015 at 8:24 AM, Tetsuya Mukawa wrote: > From: "Tetsuya.Mukawa" > > This patch consolidates below structures, and defines them in common code. > - struct pci_map > - strucy mapped_pci_resouces > > Signed-off-by: Tetsuya Mukawa > --- > lib/librte_eal/bsdapp/eal/eal_pci.c

[dpdk-dev] [PATCH v8 07/12] eal: Add pci_uio_map_resource_by_index()

2015-07-07 Thread David Marchand
On Mon, Jul 6, 2015 at 8:24 AM, Tetsuya Mukawa wrote: > From: "Tetsuya.Mukawa" > > This patch adds a new function called pci_uio_map_resource_by_index(). > The function hides how to map uio resource in linuxapp and bsdapp. > With the function, pci_uio_map_resource() will be more abstracted. > >

[dpdk-dev] [PATCH v8 06/12] eal: Add pci_uio_alloc_resource()

2015-07-07 Thread David Marchand
On Mon, Jul 6, 2015 at 8:24 AM, Tetsuya Mukawa wrote: > From: "Tetsuya.Mukawa" > > This patch adds a new function called pci_uio_alloc_resource(). > The function hides how to prepare uio resource in linuxapp and bsdapp. > With the function, pci_uio_map_resource() will be more abstracted. > > Sig

[dpdk-dev] [PATCH v8 05/12] eal: Fix uio mapping differences between linuxapp and bsdapp

2015-07-07 Thread David Marchand
On Mon, Jul 6, 2015 at 8:24 AM, Tetsuya Mukawa wrote: > diff --git a/lib/librte_eal/bsdapp/eal/eal_pci.c > b/lib/librte_eal/bsdapp/eal/eal_pci.c > index 21d1e66..92d9886 100644 > --- a/lib/librte_eal/bsdapp/eal/eal_pci.c > +++ b/lib/librte_eal/bsdapp/eal/eal_pci.c > > @@ -170,19 +154,33 @@ pci_ui

[dpdk-dev] [PATCH v8 04/12] eal/bsdapp: Change names of pci related data structure

2015-07-07 Thread David Marchand
On Mon, Jul 6, 2015 at 8:24 AM, Tetsuya Mukawa wrote: > From: "Tetsuya.Mukawa" > > To merge pci code of linuxapp and bsdapp, this patch changes names > like below. > - uio_map to pci_map > - uio_resource to mapped_pci_resource > - uio_res_list to mapped_pci_res_list > > Signed-off-by: Tetsuya

[dpdk-dev] [PATCH v8 03/12] eal: Fix memory leaks and needless increment of pci_map_addr

2015-07-07 Thread David Marchand
Hello Tetsuya, Little comment below for this patch. On Mon, Jul 6, 2015 at 8:24 AM, Tetsuya Mukawa wrote: > > diff --git a/lib/librte_eal/bsdapp/eal/eal_pci.c > b/lib/librte_eal/bsdapp/eal/eal_pci.c > index a63d450..63758c7 100644 > --- a/lib/librte_eal/bsdapp/eal/eal_pci.c > +++ b/lib/librte_e

[dpdk-dev] [PATCH v8 02/12] eal: Close file descriptor of uio configuration

2015-07-07 Thread David Marchand
On Mon, Jul 6, 2015 at 8:24 AM, Tetsuya Mukawa wrote: > From: "Tetsuya.Mukawa" > > When pci_uio_unmap_resource() is called, a file descriptor that is used > for uio configuration should be closed. > > Signed-off-by: Tetsuya Mukawa > Acked-by: Stephen Hemminger > --- > lib/librte_eal/linuxapp/

[dpdk-dev] [PATCH v8 01/12] eal: Fix coding style of eal_pci.c and eal_pci_uio.c

2015-07-07 Thread David Marchand
On Mon, Jul 6, 2015 at 8:24 AM, Tetsuya Mukawa wrote: > From: "Tetsuya.Mukawa" > > This patch fixes coding style of below files in linuxapp and bsdapp. > - eal_pci.c > - eal_pci_uio.c > > Signed-off-by: Tetsuya Mukawa > Acked-by: Stephen Hemminger > Acked-by: David Marchand I think I alre

[dpdk-dev] [PATCH 0/2] Native uio-based PMD for Mellanox ConnectX-3 devices

2015-07-07 Thread Pavel Odintsov
Hello! Sorry for off topic. But this question is very important for me. Olga, could I achieve line rate with Mellanox Cards with 64b packets for 40GE? On Tue, Jul 7, 2015 at 9:50 AM, Olga Shern wrote: > Hi Keunhong, > > I disagree with you regarding the performance of ConnectX-3 PMD driver base

[dpdk-dev] [PATCH] mk: enable next abi in static libs

2015-07-07 Thread Neil Horman
On Tue, Jul 07, 2015 at 05:46:08AM -0700, Thomas Monjalon wrote: > Thanks Neil, we are making good progress. > > 2015-07-07 07:14, Neil Horman: > > On Mon, Jul 06, 2015 at 11:44:59PM +0200, Thomas Monjalon wrote: > > > 2015-07-06 14:22, Neil Horman: > > > > On Mon, Jul 06, 2015 at 03:49:50PM +0200

[dpdk-dev] [PATCH v6 00/11] ip_pipeline: ip_pipeline application enhancements

2015-07-07 Thread Dumitrescu, Cristian
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Maciej Gajdzica > Sent: Tuesday, July 7, 2015 9:09 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v6 00/11] ip_pipeline: ip_pipeline application > enhancements > > Changes in v6: > - fixed build issues

[dpdk-dev] [PATCH 0/2] Native uio-based PMD for Mellanox ConnectX-3 devices

2015-07-07 Thread Olga Shern
Hi Pavel, A new Mellanox NIC, ConnectX-4 can achieve line arte with 64b packet for 40G link We will have PMD that supports this NIC in DPDK 2.2 . By the way, this is also an advantage of using verbs API, with almost 0% effort we can support a new NIC Best Regards, Olga -Original Message

[dpdk-dev] [PATCH] mk: enable next abi in static libs

2015-07-07 Thread Neil Horman
On Tue, Jul 07, 2015 at 05:46:08AM -0700, Thomas Monjalon wrote: > Thanks Neil, we are making good progress. > > 2015-07-07 07:14, Neil Horman: > > On Mon, Jul 06, 2015 at 11:44:59PM +0200, Thomas Monjalon wrote: > > > 2015-07-06 14:22, Neil Horman: > > > > On Mon, Jul 06, 2015 at 03:49:50PM +0200

[dpdk-dev] [PATCH v2] vfio: Fix overflow while assigning vfio BAR region offset and size

2015-07-07 Thread Burakov, Anatoly
Hi all, > So, as VFIO support for 32bit systems is broken, DPDK should not configure > VFIO in that case. ...Or no one should try and run VFIO on a 32-bit system, which should be noted in documentation. I'm a bit wary of adding special handling in this case. Does making the offset off_t fix th

[dpdk-dev] [PATCH 0/3] fix the issue sctp flow cannot be matched in FVL FDIR

2015-07-07 Thread Liu, Yong
Hi, > -Original Message- > From: Wu, Jingjing > Sent: Tuesday, July 07, 2015 3:58 PM > To: dev at dpdk.org > Cc: Wu, Jingjing; Liu, Yong; Xu, HuilongX > Subject: [PATCH 0/3] fix the issue sctp flow cannot be matched in FVL FDIR > > This patch set fixes the issue SCTP flow cannot be matched

[dpdk-dev] [PATCH] mk: enable next abi in static libs

2015-07-07 Thread Neil Horman
On Mon, Jul 06, 2015 at 11:44:59PM +0200, Thomas Monjalon wrote: > 2015-07-06 14:22, Neil Horman: > > On Mon, Jul 06, 2015 at 03:49:50PM +0200, Thomas Monjalon wrote: > > > 2015-07-06 09:35, Neil Horman: > > > > On Mon, Jul 06, 2015 at 03:18:51PM +0200, Thomas Monjalon wrote: > > > > > Any comment

[dpdk-dev] [PATCH 0/2] Native uio-based PMD for Mellanox ConnectX-3 devices

2015-07-07 Thread Olga Shern
Hi Keunhong, I disagree with you regarding the performance of ConnectX-3 PMD driver based on verbs. We get line rate of 40G link with message size smaller than 512B. In MLNX_OFED 3.0 we have presented a new verbs, called accelerated verbs, these verbs improves by more than 100% the performance

[dpdk-dev] [PATCH] mk: enable next abi in static libs

2015-07-07 Thread Thomas Monjalon
Thanks Neil, we are making good progress. 2015-07-07 07:14, Neil Horman: > On Mon, Jul 06, 2015 at 11:44:59PM +0200, Thomas Monjalon wrote: > > 2015-07-06 14:22, Neil Horman: > > > On Mon, Jul 06, 2015 at 03:49:50PM +0200, Thomas Monjalon wrote: > > > > 2015-07-06 09:35, Neil Horman: > > > > > On

[dpdk-dev] [PATCH 0/2] Native uio-based PMD for Mellanox ConnectX-3 devices

2015-07-07 Thread Keunhong Lee
We found that optimizing fragmentation configuration of mlx4 driver performs as fast as native PMD. I think we have to re-consider using native driver rather than ib driver. Keunhong. 2015-07-07 1:14 GMT+09:00 Thomas Monjalon : > 2015-07-07 00:57, Keunhong Lee: > > Answer 1. UIO based driver is

[dpdk-dev] [PATCH] virtio: fix the vq size issue

2015-07-07 Thread Ouyang, Changchun
> -Original Message- > From: Ouyang, Changchun > Sent: Wednesday, July 1, 2015 3:49 PM > To: dev at dpdk.org > Cc: Cao, Waterman; Xu, Qian Q; Ouyang, Changchun > Subject: [PATCH] virtio: fix the vq size issue > > This commit breaks virtio basic packets rx functionality: > d78deadae4dca2

[dpdk-dev] [PATCH v2 0/3] Fix vhost startup issue

2015-07-07 Thread Ouyang, Changchun
> -Original Message- > From: Ouyang, Changchun > Sent: Monday, July 6, 2015 10:27 AM > To: dev at dpdk.org > Cc: Xie, Huawei; Cao, Waterman; Xu, Qian Q; Ouyang, Changchun > Subject: [PATCH v2 0/3] Fix vhost startup issue > > The patch set fix vhost sample fails to start up in second time:

[dpdk-dev] [PATCH 0/2] Native uio-based PMD for Mellanox ConnectX-3 devices

2015-07-07 Thread Keunhong Lee
Answer 1. UIO based driver is faster then ib based driver. It can saturate 40G link with MTU sized packets using a single thread while ib wrapper cannot. Answer 2. Sorry, I missed that. I'll make a new patch email with my real name. Question 1. Is it OK if I separate GPL-based and BSD-based code