[dpdk-dev] [ovs-discuss] vswitches performance comparison

2015-07-21 Thread Gray, Mark D
> > I'd like to hope that's my methodology problem, but I just follow the > installation guide without any customization. > > Hi Mark, do you have any performance data share with us? Maybe we are > using different type of workloads, like I mentioned I am using typical data > center workload, I

[dpdk-dev] [ovs-discuss] vswitches performance comparison

2015-07-21 Thread Gray, Mark D
> -Original Message- > From: discuss [mailto:discuss-bounces at openvswitch.org] On Behalf Of Jun > Xiao > Sent: Tuesday, July 21, 2015 7:01 PM > To: discuss; dev > Subject: [ovs-discuss] vswitches performance comparison > > After CloudNetEngine vswitch technical preview is launched, we

[dpdk-dev] [PATCH 1/1] ip_pipeline: fixed parsing cpu cores

2015-07-21 Thread Maciej Gajdzica
This patch fixes parsing value of core variable in pipeline config. Before not every combination of cores (c), sockets (s) and hyperthreading (h) was parsed correctly. Signed-off-by: Maciej Gajdzica --- examples/ip_pipeline/config_parse.c | 12 ++-- 1 file changed, 6 insertions(+), 6

[dpdk-dev] [PATCH] ixgbe: fix check for split packets

2015-07-21 Thread Bruce Richardson
The check for split packets to be reassembled in the vector ixgbe PMD was incorrectly only checking the first 16 elements of the array instead of all 32. This is fixed by changing the uint32_t values to be uint64_t instead. Fixes: cf4b4708a88a ("ixgbe: improve slow-path perf with vector scattered

[dpdk-dev] ixgbe_recv_scattered_pkts_vec split_flags question

2015-07-21 Thread Bruce Richardson
On Tue, Jul 21, 2015 at 04:09:59PM +0100, Zoltan Kiss wrote: > Hi, > > I have a question regarding split_flags in this question. It's defined as an > array of 1 byte unsigned ints: > > uint8_t split_flags[RTE_IXGBE_VPMD_RX_BURST] = {0}; > > RTE_IXGBE_VPMD_RX_BURST is 32, so it will be 32 bytes.

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

2015-07-21 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 file changed, 20 insertions(+) diff --git a/drivers/net/bonding/rte_eth_bond_pmd.c

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

2015-07-21 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 | 22 --

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

2015-07-21 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 | 78 ++--

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

2015-07-21 Thread Bernard Iremonger
Changes in V6: Rebase. add rte_eth_bond_free() to rte_eth_bond_version.map Changes in V5: Rebase. Refactor initialisation of static structures. 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.

[dpdk-dev] ixgbe_recv_scattered_pkts_vec split_flags question

2015-07-21 Thread Zoltan Kiss
Hi, I have a question regarding split_flags in this question. It's defined as an array of 1 byte unsigned ints: uint8_t split_flags[RTE_IXGBE_VPMD_RX_BURST] = {0}; RTE_IXGBE_VPMD_RX_BURST is 32, so it will be 32 bytes. Then we cast it into a pointer for 4 byte values, and check the first 4

[dpdk-dev] [PATCH 1/4] doc: rename ABI chapter to deprecation

2015-07-21 Thread Thomas Monjalon
2015-07-21 13:20, Dumitrescu, Cristian: > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > > --- a/doc/guides/rel_notes/abi.rst > > +++ b/doc/guides/rel_notes/deprecation.rst > > There are some pending doc patches on ABI changes that have been sent and > ack-ed prior to

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

2015-07-21 Thread Iremonger, Bernard
Hi Thomas, > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Saturday, July 18, 2015 9:28 PM > To: Iremonger, Bernard > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v4 1/1] ring: changes to support PCI Port > Hotplug > > 2015-07-07 14:09,

[dpdk-dev] [PATCH 1/1] ip_pipeline: fixed parsing cpu cores

2015-07-21 Thread Dumitrescu, Cristian
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Maciej Gajdzica > Sent: Tuesday, July 21, 2015 3:39 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH 1/1] ip_pipeline: fixed parsing cpu cores > > This patch fixes parsing value of core variable in

[dpdk-dev] Non-working TX IP checksum offload

2015-07-21 Thread Angela Czubak
Hi Andriy, thank you, that solved it. Regards, Angela Andriy Berestovskyy wrote on 07/17/2015 04:37:45 PM: > From: Andriy Berestovskyy > To: Angela Czubak/Poland/Contr/IBM at IBMPL, > Cc: dev at dpdk.org > Date: 07/17/2015 04:38 PM > Subject: Re: [dpdk-dev] Non-working TX IP checksum

[dpdk-dev] [PATCH] hash: move field hash_func_init_val in rte_hash struct

2015-07-21 Thread Pablo de Lara
In order to keep the ABI consistent with the old hash library, hash_func_init_val field has been moved, so it remains at the same offset as previously, since hash_func and hash_func_init_val are fields accesed by the public function rte_hash_hash and must keep the same offset as older versions.

[dpdk-dev] vswitches performance comparison

2015-07-21 Thread Stephen Hemminger
On Wed, 22 Jul 2015 02:00:42 +0800 "Jun Xiao" wrote: > After CloudNetEngine vswitch technical preview is launched, we received quite > a few queries on vswitches performance comparison, but we cannot simply give a > test result on our platform because performance varies on different H/Ws and >

[dpdk-dev] [PATCH 1/4] doc: rename ABI chapter to deprecation

2015-07-21 Thread Dumitrescu, Cristian
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > Sent: Sunday, July 19, 2015 11:52 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH 1/4] doc: rename ABI chapter to deprecation > > This chapter is for ABI and API. That's why a renaming

[dpdk-dev] [PATCH v2 1/1] ixgbe: Fix phy errors in stats->ierrors

2015-07-21 Thread Liu, Yong
Tested-by: Marvin Liu > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Lu, Wenzhuo > Sent: Tuesday, July 21, 2015 1:41 PM > To: Tahhan, Maryam; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 1/1] ixgbe: Fix phy errors in stats- > >ierrors > > Hi, > >

[dpdk-dev] Does PCI hotplug work with IVSHMEM?

2015-07-21 Thread Fulvio Risso
Dear all, we're adding dynamically an IVSHMEM device on a VM that is already running, but apparently this is not visible in DPDK, while it is correctly recognized by the Guest OS. This is the list of steps we execute: 1) Launch a new Guest VM with Qemu 2) Create a new IVSHMEM metadata file

[dpdk-dev] [PATCH v2] malloc: fix combined lib build

2015-07-21 Thread Sergio Gonzalez Monroy
Malloc was moved to the EAL and dummy malloc library was left to not break apps that had a librte_malloc.so dependency. Note that the dummy library will be removed in the next release. When building a combined library, all objects are copied to the same directory before creating the library

[dpdk-dev] [PATCH] malloc: fix combined lib build

2015-07-21 Thread Gonzalez Monroy, Sergio
On 21/07/2015 11:18, Sergio Gonzalez Monroy wrote: > Malloc was moved to the EAL and dummy malloc library was left > to not brake apps that had a librte_malloc.so dependency. > Note that the dummy library will be removed in the next release. > > When building a combined library, all objects are

[dpdk-dev] [PATCH] malloc: fix combined lib build

2015-07-21 Thread Sergio Gonzalez Monroy
Malloc was moved to the EAL and dummy malloc library was left to not brake apps that had a librte_malloc.so dependency. Note that the dummy library will be removed in the next release. When building a combined library, all objects are copied to the same directory before creating the library

[dpdk-dev] [PATCH] DPDK fails to build into one single library ("CONFIG_RTE_BUILD_COMBINE_LIBS=y").

2015-07-21 Thread Gonzalez Monroy, Sergio
On 21/07/2015 10:49, Rauta, Alin wrote: > The reason why I took into account as a solution the commenting of that line > is that LIBRTE_EAL already has rte_malloc integrated. > So, I'm not sure why the line is needed ... > /Alin That line is needed so when we create shared libs it creates a

[dpdk-dev] [PATCH v3 0/3] cxgbe: Fix compilation and enable FreeBSD support for CXGBE PMD

2015-07-21 Thread Bruce Richardson
On Mon, Jul 20, 2015 at 11:01:34PM +0530, Rahul Lakkireddy wrote: > This series of patches fix compilation and enable CXGBE poll mode driver for > FreeBSD. The first patch fixes a limitation of nic_uio that only binds to > devices present in rte_pci_dev_ids.h. The second patch does the actual >

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

2015-07-21 Thread Iremonger, Bernard
Hi Thomas, > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Saturday, July 18, 2015 9:40 PM > To: Iremonger, Bernard > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v5 1/3] bonding: add support for PCI Port > Hotplug > > 2015-07-15 16:32,

[dpdk-dev] ethdev cleanup following hotplug changes

2015-07-21 Thread Thomas Monjalon
2015-07-21 08:49, Tetsuya Mukawa: > 2015-07-17 18:31 GMT+09:00 Thomas Monjalon : > > Any news about this comment in ethdev? > > > > * TODO: > > * rte_eal_vdev_init() should return port_id, > > * And rte_eth_dev_save() and rte_eth_dev_get_changed_port() > > * should be removed. */ > > > >

[dpdk-dev] [PATCH] DPDK fails to build into one single library ("CONFIG_RTE_BUILD_COMBINE_LIBS=y").

2015-07-21 Thread Rauta, Alin
The reason why I took into account as a solution the commenting of that line is that LIBRTE_EAL already has rte_malloc integrated. So, I'm not sure why the line is needed ... /Alin -Original Message- From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] Sent: Monday, July 20, 2015

[dpdk-dev] Creation of ACL context with multiple tries

2015-07-21 Thread Ananyev, Konstantin
Hi Vartha, > Hi All, > Need your inputs on the following queries on ACL contexts. > > 1. Has anyone tried creating ACL context with multiple tries? > 2. The document/code mentioned that we can have the maximum of 8 > tries(RTE_ACL_MAX_TRIES) per ACL context. > What is the use case for

[dpdk-dev] ethdev cleanup following hotplug changes

2015-07-21 Thread Tetsuya Mukawa
2015-07-17 18:31 GMT+09:00 Thomas Monjalon : > Hi Tetsuya, Hi Thomas, Sorry for late reply. I am out of office now, and will back to tomorrow. > > Any news about this comment in ethdev? > > * TODO: > * rte_eal_vdev_init() should return port_id, > * And rte_eth_dev_save() and

[dpdk-dev] [PATCH v3 1/3] nic_uio: Fix to allow any device to be bound to nic_uio

2015-07-21 Thread David Marchand
On Mon, Jul 20, 2015 at 7:31 PM, Rahul Lakkireddy < rahul.lakkireddy at chelsio.com> wrote: > nic_uio requires the pci ids to be present in rte_pci_dev_ids.h in order to > bind the devices to nic_uio. However, it's better to remove this > whitelist of > pci ids, and instead rely on

[dpdk-dev] [PATCH v2] fm10k: add missing newline to debug log

2015-07-21 Thread Chen, Jing D
> -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > Sent: Friday, July 17, 2015 5:07 AM > To: Chen, Jing D > Cc: dev at dpdk.org; Stephen Hemminger > Subject: [PATCH v2] fm10k: add missing newline to debug log > > If FM10K_DEBUG_DRIVER is enabled, then

[dpdk-dev] [PATCH] fm10k: remove useless code

2015-07-21 Thread Chen, Jing D
> -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > Sent: Friday, July 17, 2015 6:05 AM > To: Chen, Jing D > Cc: dev at dpdk.org; Stephen Hemminger > Subject: [PATCH] fm10k: remove useless code > > The return in fm10k_dev_handle_fault has useless

[dpdk-dev] [PATCH] fm10K: fix interrupt fault handling

2015-07-21 Thread Chen, Jing D
Hi, Stephen, > -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > Sent: Friday, July 17, 2015 8:33 AM > To: Qiu, Michael; Chen, Jing D > Cc: dev at dpdk.org; Stephen Hemminger > Subject: [PATCH] fm10K: fix interrupt fault handling > > The fm10k driver

[dpdk-dev] [PATCH v2 1/1] ixgbe: Fix phy errors in stats->ierrors

2015-07-21 Thread Lu, Wenzhuo
Hi, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Maryam Tahhan > Sent: Monday, July 20, 2015 8:28 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v2 1/1] ixgbe: Fix phy errors in stats->ierrors > > Fixes: f6bf669b990004dc69231476d9fcf0b80574e568.

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

2015-07-21 Thread Ouyang, Changchun
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Monday, July 20, 2015 6:42 PM > To: Ouyang, Changchun > Cc: Xu, Qian Q; Stephen Hemminger; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] virtio: fix the vq size issue > > 2015-07-20 06:18,

[dpdk-dev] [PATCH v2 1/2] virtio: fix queue size and number of descriptors

2015-07-21 Thread Ouyang, Changchun
> -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > Sent: Tuesday, July 21, 2015 2:41 AM > To: Ouyang, Changchun > Cc: dev at dpdk.org; Stephen Hemminger > Subject: [PATCH v2 1/2] virtio: fix queue size and number of descriptors > > The virtual queue

[dpdk-dev] [PATCH v2 0/2] virtio: fixes for 2.1-rc1

2015-07-21 Thread Ouyang, Changchun
> -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > Sent: Tuesday, July 21, 2015 2:41 AM > To: Ouyang, Changchun > Cc: dev at dpdk.org; Stephen Hemminger > Subject: [PATCH v2 0/2] virtio: fixes for 2.1-rc1 I think v2 here should be v3, as you have

[dpdk-dev] [PATCH v2 2/2] virtio: small cleanups

2015-07-21 Thread Ouyang, Changchun
> -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > Sent: Tuesday, July 21, 2015 2:41 AM > To: Ouyang, Changchun > Cc: dev at dpdk.org; Stephen Hemminger > Subject: [PATCH v2 2/2] virtio: small cleanups > > Some minor cleanups. > * pass constant to

[dpdk-dev] jumbo frame support for 82583V

2015-07-21 Thread Lu, Wenzhuo
Hi Klaus, > -Original Message- > From: Klaus Degner [mailto:kd at allegro-packets.com] > Sent: Monday, July 20, 2015 5:21 PM > To: Lu, Wenzhuo > Cc: dev at dpdk.org > Subject: Re: jumbo frame support for 82583V > > Hi Wenzhuo, > > Hi Klaus, > > > >> -Original Message- > >> From:

[dpdk-dev] driver initialization in DPDK 2.0 built into a shared library.

2015-07-21 Thread Polevoy, Igor
After linking the shared library using ?whole-archive and ?start-group and allowing for multiple definitions also with ?z,muldefs I have the situation when init is called two times and I?m getting the core created inside the tailqinitfn_rte_uio_tailq (eal_common_tailqs.c:153). Commenting out

[dpdk-dev] [PATCH v3 3/3] doc: Update documentation to reflect FreeBSD support for CXGBE PMD

2015-07-21 Thread Rahul Lakkireddy
Update cxgbe documentation to include support for FreeBSD: 1. Add instructions on how to compile CXGBE PMD in FreeBSD. 2. Add instructions on how to flash firmware image on Chelsio T5 cards in FreeBSD. 3. Add sample application usage for FreeBSD. 4. Add an extra step to reload kernel module in

[dpdk-dev] [PATCH v3 2/3] cxgbe: Enable and fix FreeBSD compilation for CXGBE PMD

2015-07-21 Thread Rahul Lakkireddy
Fix "MACRO redefined" and "function redefined" compilation errors in FreeBSD by adding CXGBE prefix to them. Also remove reference to a linux header linux/if_ether.h and use DPDK macros directly. Finally, enable CXGBE PMD for FreeBSD. Signed-off-by: Rahul Lakkireddy Signed-off-by: Kumar

[dpdk-dev] [PATCH v3 1/3] nic_uio: Fix to allow any device to be bound to nic_uio

2015-07-21 Thread Rahul Lakkireddy
nic_uio requires the pci ids to be present in rte_pci_dev_ids.h in order to bind the devices to nic_uio. However, it's better to remove this whitelist of pci ids, and instead rely on hw.nic_uio.bdfs kenv parameter to allow binding any device to nic_uio. Suggested-by: David Marchand

[dpdk-dev] [PATCH v3 0/3] cxgbe: Fix compilation and enable FreeBSD support for CXGBE PMD

2015-07-21 Thread Rahul Lakkireddy
This series of patches fix compilation and enable CXGBE poll mode driver for FreeBSD. The first patch fixes a limitation of nic_uio that only binds to devices present in rte_pci_dev_ids.h. The second patch does the actual compilation fix and enabling of CXGBE PMD for FreeBSD. The last patch