[dpdk-dev] [dpdk-announce] release candidate 1.8.0-rc2

2014-11-27 Thread Thomas Monjalon
A new DPDK release candidate wait to be massively tested: http://dpdk.org/browse/dpdk/tag/?id=v1.8.0-rc2 This is the first release candidate (rc1 was a first step to integrate some important reworks). If everything goes well, the 1.8.0 release should be out in 2 weeks. Changelog (main cha

[dpdk-dev] [PATCH v2] bond: set {rx|tx}_offload_capa flags

2014-11-27 Thread Thomas Monjalon
2014-11-27 21:23, Declan Doherty: > v2: > rebased to HEAD > > From: Jia Yu > > Before the fix, bond device's offload capabilities are unset. This fix > takes the minimum common set of slave devices' capabilities as bond > device's capabilities. For simplicity, we ensure all slave devices > to ha

[dpdk-dev] [PATCH] enicpmd: compilation error during inclusion of vfio.h

2014-11-27 Thread Sujith Sankar
Inclusion of vfio.h was giving compilation errors if kernel version is less than 3.6.0 and if RTE_EAL_VFIO was on in config. Replaced inclusion of vfio.h with eal_vfio.h and replaced RTE_EAL_VFIO with VFIO_PRESENT in enicpmd code. Signed-off-by: Sujith Sankar --- lib/librte_pmd_enic/Makefile

[dpdk-dev] [PATCH v5 0/7] rte_hash_crc reworked to be platform-independent

2014-11-27 Thread Thomas Monjalon
2014-11-20 11:15, Yerden Zhumabekov: > These patches bring a fallback mechanism to ensure that CRC32 hash is > calculated regardless of hardware support from CPU (i.e. SSE4.2 intrinsics). > Performance is also improved by slicing data in 8 bytes. > > Patches were tested on machines either with an

[dpdk-dev] [PATCH v2] bond: set {rx|tx}_offload_capa flags

2014-11-27 Thread Doherty, Declan
Thanks Thomas, was just about to send another version :) > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Thursday, November 27, 2014 9:53 PM > To: Doherty, Declan > Cc: dev at dpdk.org; jyu at vmware.com > Subject: Re: [PATCH v2] bond: set {rx|tx}

[dpdk-dev] [PATCH v2] bond: set {rx|tx}_offload_capa flags

2014-11-27 Thread Doherty, Declan
Self nack, need to rebase again as mode 4 and 5 patches have been applied. > -Original Message- > From: Doherty, Declan > Sent: Thursday, November 27, 2014 9:24 PM > To: dev at dpdk.org > Cc: jyu at vmware.com; thomas.monjalon at 6wind.com; Doherty, Declan > Subject: [PATCH v2] bond: set {

[dpdk-dev] [PATCH v4 0/2] ADD mode 5(tlb) to link bonding pmd

2014-11-27 Thread Thomas Monjalon
> ADD mode 5(tlb) to link bonding pmd > > v4 change: > Change description of mode 5 in header. > Add description to cover letter. > > v3 change: > Rebase patch version to HEAD of orgin/master. > Unit tests moved to the separate patch v3 2/2. > > v2 change: > Add Unit Tests > Modification that up

[dpdk-dev] [PATCH v7 1/1] bond: add mode 4 support v7

2014-11-27 Thread Thomas Monjalon
> > 2014-11-27 18:01, Michal Jastrzebski: > > > From: Pawel Wodkowski > > > > > > This patch set add support for dynamic link aggregation (mode 4) to the > > > librte_pmd_bond library. This mode provides auto > > negotiation/configuration > > > of peers and well as link status changes monitoring u

[dpdk-dev] [PATCH] enicpmd: compilation error during inclusion of vfio.h

2014-11-27 Thread Thomas Monjalon
2014-11-27 19:01, Thomas Monjalon: > 2014-11-27 22:44, Sujith Sankar: > > Inclusion of vfio.h was giving compilation errors if kernel version is less > > than 3.6.0 and if RTE_EAL_VFIO was on in config. > > > > Replaced inclusion of vfio.h with eal_vfio.h and replaced RTE_EAL_VFIO with > > VFIO_PR

[dpdk-dev] [PATCH v2] bond: set {rx|tx}_offload_capa flags

2014-11-27 Thread Declan Doherty
v2: rebased to HEAD From: Jia Yu Before the fix, bond device's offload capabilities are unset. This fix takes the minimum common set of slave devices' capabilities as bond device's capabilities. For simplicity, we ensure all slave devices to have a capability before bond device can claim this ca

[dpdk-dev] how to avoid "non-matching type" in my program with dpdk

2014-11-27 Thread Linhaifeng
Hi,all when i compile my program with dpdk there is a warning found by gcc.The message is like follow.I don't know how to avoid it.Help me. /usr/include/dpdk-1.7.0/x86_64-native-linuxapp-gcc//include/rte_common.h:176: warning: cast from function call of type ?uintptr_t? to non-matching type ?vo

[dpdk-dev] [PATCH v4 4/4] doc: Various document fixes in testpmd UG

2014-11-27 Thread Pablo de Lara
Signed-off-by: Pablo de Lara --- doc/guides/testpmd_app_ug/testpmd_funcs.rst |8 +--- 1 files changed, 5 insertions(+), 3 deletions(-) diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst index 84b7bc3..bbc24a3 100644 --- a/doc/guides/te

[dpdk-dev] [PATCH v4 3/4] doc: Moved commands in testpmd UG to match testpmd command help order

2014-11-27 Thread Pablo de Lara
Moved commands in testpmd UG to set queue rates to match order in testpmd command help. Created new section "Filters" to match that same section in testpmd UG, and moved all commands related to it there. Signed-off-by: Pablo de Lara --- doc/guides/testpmd_app_ug/testpmd_funcs.rst | 1042 +++

[dpdk-dev] [PATCH v4 2/4] doc: Corrected info for tx_checksum set mask function, in testpmd UG

2014-11-27 Thread Pablo de Lara
tx_checksum set function does not use bitmask anymore, but strings Signed-off-by: Pablo de Lara --- doc/guides/testpmd_app_ug/testpmd_funcs.rst | 21 - 1 files changed, 8 insertions(+), 13 deletions(-) diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/

[dpdk-dev] [PATCH v4 1/4] doc: Added new commands in testpmd UG

2014-11-27 Thread Pablo de Lara
Added info in testpmd functions section for the following commands: - tunnel_filter add - tunnel_filter rm - rx_vxlan_port add - rx_vxlan_port rm - port stop/start queue - set port mac address filter (for VF) - tx_checksum set - tso set - tso show Signed-off-by: Pablo de Lara --- doc/guides/tes

[dpdk-dev] [PATCH v4 0/4] Updated testpmd UG due to latest changes in the app

2014-11-27 Thread Pablo de Lara
TestPMD app has been added new commands, which should be reflected in the user guide. Also, this patchset fixes several typos and changes the location of the description of some commands. Changes in v4: Fixed wrong patchset title Changes in v3: Added new missing existing command "show port rss

[dpdk-dev] [PATCH] config: disable enic driver on Power

2014-11-27 Thread Thomas Monjalon
> enic driver is giving trouble because of non-standard types : > > CC enic_res.o > In file included from > lib/librte_pmd_enic/enic_res.c:36:0: > lib/librte_pmd_enic/enic_compat.h:92:1: error: unknown type name ?u_int32_t? > static inline u_int32_t ioread32(volatile void *addr) > ^ > > Disab

[dpdk-dev] [PATCH] net: fix conflict with libc

2014-11-27 Thread Thomas Monjalon
> > It was impossible to include netinet/in.h and rte_ip.h > > because the IP protocols were redefined. > > It is removed because useless. > > > > Signed-off-by: Thomas Monjalon > > Acked by Ivan Boule Applied -- Thomas

[dpdk-dev] [PATCH v7 1/1] bond: add mode 4 support v7

2014-11-27 Thread Thomas Monjalon
2014-11-27 18:01, Michal Jastrzebski: > From: Pawel Wodkowski > > This patch set add support for dynamic link aggregation (mode 4) to the > librte_pmd_bond library. This mode provides auto negotiation/configuration > of peers and well as link status changes monitoring using out of band > LACP (

[dpdk-dev] [PATCH] enicpmd: compilation error during inclusion of vfio.h

2014-11-27 Thread Thomas Monjalon
2014-11-27 22:44, Sujith Sankar: > Inclusion of vfio.h was giving compilation errors if kernel version is less > than 3.6.0 and if RTE_EAL_VFIO was on in config. > > Replaced inclusion of vfio.h with eal_vfio.h and replaced RTE_EAL_VFIO with > VFIO_PRESENT in enicpmd code. You should add Reported

[dpdk-dev] issue with ixgbevf speed/duplex introduced with commit 8ef32003772a14c61c70b540e41c259c482c2fb6

2014-11-27 Thread Thomas Monjalon
Ping? There is still no review of this patch: http://dpdk.org/dev/patchwork/patch/1079/ And there is another patch for the same issue: http://dpdk.org/dev/patchwork/patch/1224/ 2014-11-07 17:46, Thomas Monjalon: > Hi, > > Thank you for reporting and explaining the issue. > > You

[dpdk-dev] [PATCH] app/test-pmd: Use the new rss_hf size changed in 1.7

2014-11-27 Thread Thomas Monjalon
> According to rss_hf definition in rte_eth_rss_conf, it shall be uint64 type. > Using uint16 will get truncated value, and cause incorrect output. This > fix corrected this issue. > > Signed-off-by: Jia Yu Acked and applied. -- Thomas

[dpdk-dev] [PATCH] Bond: set {rx|tx}_offload_capa flags

2014-11-27 Thread Thomas Monjalon
> > Before the fix, bond device's offload capabilities are unset. This fix > > takes the minimum common set of slave devices' capabilities as bond > > device's capabilities. For simplicity, we ensure all slave devices > > to have a capability before bond device can claim this capability, > > even i

[dpdk-dev] [PATCH v7 1/1] bond: add mode 4 support v7

2014-11-27 Thread Jastrzebski, MichalX K
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Thursday, November 27, 2014 7:09 PM > To: Jastrzebski, MichalX K > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v7 1/1] bond: add mode 4 support v7 > > 2014-11-27 18:01, Michal Jastrzebski: >

[dpdk-dev] [PATCH v2 1/2] igb_uio: compatible with upstream longterm kernel and RHEL6

2014-11-27 Thread Thomas Monjalon
2014-10-31 15:37, Jincheng Miao: > Function pci_num_vf() is introduced from upstream linux-2.6.34. So > this patch make compatible with longterm kernel linux-2.6.32.63. > > For RHEL6's kernel, although it is based on linux-2.6.32, it has > pci_num_vf() implementation. As the same with commit 11ba0

[dpdk-dev] [PATCH v7 1/1] bond: add mode 4 support v7

2014-11-27 Thread Michal Jastrzebski
From: Pawel Wodkowski This patch set add support for dynamic link aggregation (mode 4) to the librte_pmd_bond library. This mode provides auto negotiation/configuration of peers and well as link status changes monitoring using out of band LACP (link aggregation control protocol) messages. For f

[dpdk-dev] [PATCH v7 0/1] bond: add mode 4 support v7

2014-11-27 Thread Michal Jastrzebski
v7 changes - add description in API - add a warning when creating a bond mode 4 device - removed patch 2/2 with testpmd support for mode 4 v6 changes - add commit log description to link bondig mode 4 v5 changes - fix compilation errors when CONFIG_RTE_LIBRTE_PMD_BOND=n v4 changes: - fix compila

[dpdk-dev] [PATCH v4 0/2] ADD mode 5(tlb) to link bonding pmd

2014-11-27 Thread Thomas Monjalon
2014-11-27 16:33, Daniel Mrzyglod: > ADD mode 5(tlb) to link bonding pmd > > v4 change: > Change description of mode 5 in header. > Add description to cover letter. > > v3 change: > Rebase patch version to HEAD of orgin/master. > Unit tests moved to the separate patch v3 2/2. > > v2 change: > Ad

[dpdk-dev] [PATCH v2] rte_mempool_dump() crashes with NULL rte_mempool pointer.

2014-11-27 Thread Thomas Monjalon
2014-10-02 07:37, Neil Horman: > On Thu, Oct 02, 2014 at 09:47:19AM +0200, Thomas Monjalon wrote: > > 2014-10-01 17:05, Bruce Richardson: > > > On Wed, Oct 01, 2014 at 12:01:10PM -0400, Neil Horman wrote: > > > > On Wed, Oct 01, 2014 at 04:43:10PM +0100, Bruce Richardson wrote: > > > > > On Wed, Oc

[dpdk-dev] [PATCH v3 0/4] i40e VXLAN TX checksum rework

2014-11-27 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jijiang Liu > Sent: Thursday, November 27, 2014 5:03 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v3 0/4] i40e VXLAN TX checksum rework > > We have got some feedback about backward compatibility of VX

[dpdk-dev] [PATCH] enicpmd: compilation error during inclusion of vfio.h

2014-11-27 Thread Sujith Sankar (ssujith)
On 27/11/14 10:44 pm, "Sujith Sankar (ssujith)" wrote: >Inclusion of vfio.h was giving compilation errors if kernel version is >less >than 3.6.0 and if RTE_EAL_VFIO was on in config. > >Replaced inclusion of vfio.h with eal_vfio.h and replaced RTE_EAL_VFIO >with >VFIO_PRESENT in enicpmd code.

[dpdk-dev] [PATCH 1/3] mbuf:add two TX offload flags and change three fields

2014-11-27 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ananyev, Konstantin > Sent: Thursday, November 27, 2014 2:56 PM > To: Liu, Jijiang; Olivier Matz (olivier.matz at 6wind.com) > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 1/3] mbuf:add two TX offload f

[dpdk-dev] [PATCH v4 0/2] ADD mode 5(tlb) to link bonding pmd

2014-11-27 Thread Doherty, Declan
Sorry Thomas, this was my fault, I forgot that Daniel mentioned to me earlier today that he was going to submit a new revision of the patch set with updated patch descriptions, but no code changes, due to your comments relating to the link bonding mode 4. I will ack the v4 patchset when all pa

[dpdk-dev] [PATCH v3 0/4] new sample app UG for VM power management

2014-11-27 Thread De Lara Guarch, Pablo
Self-NACK, title is incorrect > -Original Message- > From: De Lara Guarch, Pablo > Sent: Thursday, November 27, 2014 4:49 PM > To: dev at dpdk.org > Cc: De Lara Guarch, Pablo > Subject: [PATCH v3 0/4] new sample app UG for VM power management > > TestPMD app has been added new commands, w

[dpdk-dev] [PATCH v2] i40e: link flow control support

2014-11-27 Thread Thomas Monjalon
Hi, 2014-11-20 16:58, zhida zang: > From: zzang > > Add link flow control support for i40e > > Signed-off-by: zhida zang This patch is pending with open questions. Any news to submit a new version and/or acknowledge it? -- Thomas

[dpdk-dev] [PATCH v3 4/4] doc: Various document fixes in testpmd UG

2014-11-27 Thread Pablo de Lara
Signed-off-by: Pablo de Lara --- doc/guides/testpmd_app_ug/testpmd_funcs.rst |8 +--- 1 files changed, 5 insertions(+), 3 deletions(-) diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst index 84b7bc3..bbc24a3 100644 --- a/doc/guides/te

[dpdk-dev] [PATCH v3 3/4] doc: Moved commands in testpmd UG to match testpmd command help order

2014-11-27 Thread Pablo de Lara
Moved commands in testpmd UG to set queue rates to match order in testpmd command help. Created new section "Filters" to match that same section in testpmd UG, and moved all commands related to it there. Signed-off-by: Pablo de Lara --- doc/guides/testpmd_app_ug/testpmd_funcs.rst | 1042 +++

[dpdk-dev] [PATCH v3 2/4] doc: Corrected info for tx_checksum set mask function, in testpmd UG

2014-11-27 Thread Pablo de Lara
tx_checksum set function does not use bitmask anymore, but strings Signed-off-by: Pablo de Lara --- doc/guides/testpmd_app_ug/testpmd_funcs.rst | 21 - 1 files changed, 8 insertions(+), 13 deletions(-) diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/

[dpdk-dev] [PATCH v3 1/4] doc: Added new commands in testpmd UG

2014-11-27 Thread Pablo de Lara
Added info in testpmd functions section for the following commands: - tunnel_filter add - tunnel_filter rm - rx_vxlan_port add - rx_vxlan_port rm - port stop/start queue - set port mac address filter (for VF) - tx_checksum set - tso set - tso show Signed-off-by: Pablo de Lara --- doc/guides/tes

[dpdk-dev] [PATCH v3 0/4] new sample app UG for VM power management

2014-11-27 Thread Pablo de Lara
TestPMD app has been added new commands, which should be reflected in the user guide. Also, this patchset fixes several typos and changes the location of the description of some commands. Changes in v3: Added new missing existing command "show port rss reta" Added new commands from TSO support pa

[dpdk-dev] [PATCH v4 2/2] Unit tests for Mode 5 of Bonding Transmit Load balancig.

2014-11-27 Thread Daniel Mrzyglod
This Patch add unit tests for mode 5 - tlb - to the oders link bonding unit tests. Signed-off-by: Daniel Mrzyglod --- app/test/test_link_bonding.c | 499 ++- app/test/virtual_pmd.c | 6 +- 2 files changed, 502 insertions(+), 3 deletions(-) diff --

[dpdk-dev] [PATCH v4 0/2] ADD mode 5(tlb) to link bonding pmd

2014-11-27 Thread Daniel Mrzyglod
ADD mode 5(tlb) to link bonding pmd v4 change: Change description of mode 5 in header. Add description to cover letter. v3 change: Rebase patch version to HEAD of orgin/master. Unit tests moved to the separate patch v3 2/2. v2 change: Add Unit Tests Modification that updates obytes structure in

[dpdk-dev] [PATCH v6 0/3] Support configuring hash functions

2014-11-27 Thread Thomas Monjalon
Hi Helin, I think you are working on a v7 of this patchset, right? Any news? -- Thomas

[dpdk-dev] [PATCH v3 0/2] ADD mode 5(tlb) to link bonding pmd

2014-11-27 Thread Doherty, Declan
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Mrzyglod, DanielX T > Sent: Wednesday, November 26, 2014 5:18 PM > To: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v3 0/2] ADD mode 5(tlb) to link bonding pmd > > v3 change: > Rebase patch version to HEAD o

[dpdk-dev] [PATCH v4 2/2] Unit tests for Mode 5 of Bonding Transmit Load balancig.

2014-11-27 Thread Daniel Mrzyglod
This Patch add unit tests for mode 5 - tlb - to the oders link bonding unit tests. Signed-off-by: Daniel Mrzyglod --- app/test/test_link_bonding.c | 499 ++- app/test/virtual_pmd.c | 6 +- 2 files changed, 502 insertions(+), 3 deletions(-) diff --

[dpdk-dev] [PATCH v4 1/2] This patch add support of mode 5 to link bonding pmd

2014-11-27 Thread Daniel Mrzyglod
Add support for mode 5 (Transmit load balancing) into pmd driver This patch add support for Adaptive transmit load balancing (mode 5) to the librte_pmd_bond library. This mode provides an adaptive transmit load balancing. It dynamically changes the transmitting slave, according to the computed l

[dpdk-dev] [PATCH v4 0/2] ADD mode 5(tlb) to link bonding pmd

2014-11-27 Thread Daniel Mrzyglod
ADD mode 5(tlb) to link bonding pmd v4 change: Change description of mode 5 in header. Add description to cover letter. v3 change: Rebase patch version to HEAD of orgin/master. Unit tests moved to the separate patch v3 2/2. v2 change: Add Unit Tests Modification that updates obytes structure in

[dpdk-dev] [PATCH 0/3] i40e VXLAN TX checksum rework

2014-11-27 Thread Liu, Jijiang
> -Original Message- > From: Ananyev, Konstantin > Sent: Thursday, November 27, 2014 11:30 PM > To: Olivier MATZ; Liu, Jijiang; dev at dpdk.org > Subject: RE: [dpdk-dev] [PATCH 0/3] i40e VXLAN TX checksum rework > > Hi Oliver, > > > -Original Message- > > From: dev [mailto:dev-b

[dpdk-dev] [PATCH v6 0/6] enicpmd: Cisco Systems Inc. VIC Ethernet PMD

2014-11-27 Thread Thomas Monjalon
2014-11-27 04:27, Sujith Sankar: > Thanks Thomas, David and Neil ! > > I shall work on finishing the documentation. > About that, you had mentioned that you wanted it in doc/drivers/ path. > Could I send a patch with documentation in the path doc/drivers/enicpmd/ ? Yes. I'd prefer doc/drivers/eni

[dpdk-dev] [PATCH 3/3] testpmd:rework csum forward engine

2014-11-27 Thread Jijiang Liu
The changes include: 1. use the new introduced ol_flags and fields in csumonly.c file; 2. fix an issue of outer UDP checksum check; 3. fix an issue that is if the TESTPMD_TX_OFFLOAD_IP_CKSUM is not set, and the "ol_flags |= PKT_TX_IPV4" should be done in the process_inner_cksums(); Signed-off-by:

[dpdk-dev] [PATCH 2/3] i40e:PMD change for VXLAN TX checksum

2014-11-27 Thread Jijiang Liu
Rework the i40e PMD codes using the new introduced ol_flags and fields. Signed-off-by: Jijiang Liu --- lib/librte_pmd_i40e/i40e_rxtx.c | 40 +- 1 files changed, 18 insertions(+), 22 deletions(-) diff --git a/lib/librte_pmd_i40e/i40e_rxtx.c b/lib/librte_pm

[dpdk-dev] [PATCH 1/3] mbuf:add two TX offload flags and change three fields

2014-11-27 Thread Jijiang Liu
In place of removing the PKT_TX_VXLAN_CKSUM, we introduce 2 new flags: PKT_TX_OUT_IP_CKSUM, PKT_TX_UDP_TUNNEL_PKT, and a new field: l4_tun_len. Replace the inner_l2_len and the inner_l3_len field with the outer_l2_len and outer_l3_len field. PKT_TX_OUT_IP_CKSUM: is not used for non-tunnelling pa

[dpdk-dev] [PATCH 0/3] i40e VXLAN TX checksum rework

2014-11-27 Thread Jijiang Liu
We have got some feedback about backward compatibility of VXLAN TX checksum offload API with 1G/10G NIC after the i40e VXLAN TX checksum codes were applied, so we have to rework the APIs on i40e, including the changes of mbuf, i40e PMD and csum engine. The main changes in mbuf are as follows, I

[dpdk-dev] [PATCH v6 0/3] Support configuring hash functions

2014-11-27 Thread Zhang, Helin
Hi Thomas Yes, I am working on it. But still a few review comments should be dealt with. Hopefully I can send it out tomorrow. Thank you! Regards, Helin > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Thursday, November 27, 2014 11:45 PM > To: Z

[dpdk-dev] [PATCH 0/3] i40e VXLAN TX checksum rework

2014-11-27 Thread Ananyev, Konstantin
Hi Oliver, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier MATZ > Sent: Thursday, November 27, 2014 9:45 AM > To: Liu, Jijiang; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 0/3] i40e VXLAN TX checksum rework > > Hi Jijiang, > > Please find below

[dpdk-dev] [PATCH] net: fix conflict with libc

2014-11-27 Thread Ivan Boule
On 11/27/2014 12:48 PM, Thomas Monjalon wrote: > It was impossible to include netinet/in.h and rte_ip.h > because the IP protocols were redefined. > It is removed because useless. > > Signed-off-by: Thomas Monjalon Acked by Ivan Boule -- Ivan Boule 6WIND Development Engineer

[dpdk-dev] [PATCH 0/3] Add RTE_ prefix to CACHE_LINE related macros

2014-11-27 Thread Thomas Monjalon
> Currently DPDK sets CACHE_LINE_SIZE value to 64 by default if the macro is > not already defined. > > FreeBSD defines a CACHE_LINE_SIZE macro in the header file: > /usr/include/machine/param.h > > These macros set different values, 64 in DPDK vs 128 in FreeBSD, causing > broken application beha

[dpdk-dev] [PATCH 1/3] mbuf:add two TX offload flags and change three fields

2014-11-27 Thread Ananyev, Konstantin
> > -Original Message- > From: Olivier MATZ [mailto:olivier.matz at 6wind.com] > Sent: Thursday, November 27, 2014 6:00 PM > To: Liu, Jijiang; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 1/3] mbuf:add two TX offload flags and change > three fields > > Hi Jijiang, > > Please see s

[dpdk-dev] [PATCH] config: disable enic driver on Power

2014-11-27 Thread Sujith Sankar (ssujith)
Hi David, I shall take a look at it get back soon after some tests. One point here is that ENICPMD will be used only on UCS servers (as Cisco VIC cards are shipped only with them) and they are all x86_64. So you could turn it off for all the other architectures. Thanks, -Sujith From: David M

[dpdk-dev] [PATCH v6 4/6] enicpmd: pmd specific code

2014-11-27 Thread Wodkowski, PawelX
> diff --git a/lib/librte_pmd_enic/enic_main.c b/lib/librte_pmd_enic/enic_main.c > new file mode 100644 > index 000..c047cc8 > --- /dev/null > +++ b/lib/librte_pmd_enic/enic_main.c > @@ -0,0 +1,1266 @@ > +/* > + * Copyright 2008-2014 Cisco Systems, Inc. All rights reserved. > +#ident "$Id$"

[dpdk-dev] [PATCH] doc: new sample app UG for VM power management

2014-11-27 Thread Thomas Monjalon
Hi Pablo, > This patch adds a new sample app UG, contaning explanation > of the new two sample apps added in the VM power management > patchset > > Signed-off-by: Alan Carew > Signed-off-by: Pablo de Lara > --- > .../sample_app_ug/img/vm_power_mgr_highlevel.svg | 2445 >

[dpdk-dev] [PATCH 1/3] mbuf:add two TX offload flags and change three fields

2014-11-27 Thread Liu, Jijiang
> -Original Message- > From: Olivier MATZ [mailto:olivier.matz at 6wind.com] > Sent: Thursday, November 27, 2014 6:00 PM > To: Liu, Jijiang; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 1/3] mbuf:add two TX offload flags and change > three fields > > Hi Jijiang, > > Please see some

[dpdk-dev] [PATCH 0/2] BSD timer cleanup/update

2014-11-27 Thread Thomas Monjalon
> This patch set is a quick cleanup/update of the timer in BSD. > Since BSD exposes TSC freq in sysctl, we avoid other less accurate methods. > We still provide a fallback in case we fail to obtain the value, as it is > x86 specific. > > The HPET device is not expose in BSD, therefore current code

[dpdk-dev] [PATCH] net: fix conflict with libc

2014-11-27 Thread Thomas Monjalon
It was impossible to include netinet/in.h and rte_ip.h because the IP protocols were redefined. It is removed because useless. Signed-off-by: Thomas Monjalon --- app/test-pmd/icmpecho.c | 4 ++ lib/librte_net/rte_ip.h | 112 +--- 2 files changed, 5 i

[dpdk-dev] [PATCH] config: disable enic driver on Power

2014-11-27 Thread David Marchand
Hello Sujith, Please can you have a look at this in your enic driver ? I find it weird to use these types while we could use types such as uint8_t etc... Thanks. -- David Marchand On Thu, Nov 27, 2014 at 12:42 PM, David Marchand wrote: > enic driver is giving trouble because of non-standar

[dpdk-dev] [PATCH] config: disable enic driver on Power

2014-11-27 Thread David Marchand
enic driver is giving trouble because of non-standard types : CC enic_res.o In file included from lib/librte_pmd_enic/enic_res.c:36:0: lib/librte_pmd_enic/enic_compat.h:92:1: error: unknown type name ?u_int32_t? static inline u_int32_t ioread32(volatile void *addr) ^ Disable it on Power for n

[dpdk-dev] [PATCH] scripts: fix symbol overriding in configuration files

2014-11-27 Thread David Marchand
When redefining the same symbol in configuration (basically after an inclusion), we need to undefine the previous symbol to avoid "redefined" errors. Signed-off-by: David Marchand --- scripts/gen-config-h.sh | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/scrip

[dpdk-dev] [PATCH 0/3] i40e VXLAN TX checksum rework

2014-11-27 Thread Liu, Jijiang
> -Original Message- > From: Olivier MATZ [mailto:olivier.matz at 6wind.com] > Sent: Thursday, November 27, 2014 5:45 PM > To: Liu, Jijiang; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 0/3] i40e VXLAN TX checksum rework > > Hi Jijiang, > > Please find below some comments about the

[dpdk-dev] [PATCH 0/3] i40e VXLAN TX checksum rework

2014-11-27 Thread Liu, Jijiang
> -Original Message- > From: Olivier MATZ [mailto:olivier.matz at 6wind.com] > Sent: Thursday, November 27, 2014 6:13 PM > To: Liu, Jijiang; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 0/3] i40e VXLAN TX checksum rework > > > >> We have got some feedback about backward compatibilit

[dpdk-dev] [PATCH] doc: new sample app UG for VM power management

2014-11-27 Thread Pablo de Lara
This patch adds a new sample app UG, contaning explanation of the new two sample apps added in the VM power management patchset Signed-off-by: Alan Carew Signed-off-by: Pablo de Lara --- .../sample_app_ug/img/vm_power_mgr_highlevel.svg | 2445 .../img/vm_power_mgr_vm_requ

[dpdk-dev] [PATCH 0/5] remove traces of bare metal support

2014-11-27 Thread Thomas Monjalon
2014-09-26 10:13, Neil Horman: > On Fri, Sep 26, 2014 at 04:03:57PM +0200, Thomas Monjalon wrote: > > There are some references to bare metal (i.e. without OS) support, > > especially some options to build a libc with DPDK. > > As there are currently no such support, it can be removed. > > Some com

[dpdk-dev] [PATCH 3/3] testpmd:rework csum forward engine

2014-11-27 Thread Olivier MATZ
Hi Jijiang, On 11/27/2014 09:18 AM, Jijiang Liu wrote: > The changes include: > 1. use the new introduced ol_flags and fields in csumonly.c file; > 2. fix an issue of outer UDP checksum check; > 3. fix an issue that is if the TESTPMD_TX_OFFLOAD_IP_CKSUM is not set, and > the "ol_flags |= PKT_TX_I

[dpdk-dev] [PATCH 0/3] i40e VXLAN TX checksum rework

2014-11-27 Thread Olivier MATZ
>> We have got some feedback about backward compatibility of VXLAN TX >> checksum offload API with 1G/10G NIC after the i40e VXLAN TX checksum >> codes were applied, so we have to rework the APIs on i40e, including >> the changes of mbuf, i40e PMD and csum engine. >> >> The main changes in mbuf ar

[dpdk-dev] [PATCH v4 08/13] testpmd: rework csum forward engine

2014-11-27 Thread Ananyev, Konstantin
Hi Oliver, > -Original Message- > From: Olivier MATZ [mailto:olivier.matz at 6wind.com] > Sent: Thursday, November 27, 2014 9:11 AM > To: Ananyev, Konstantin; dev at dpdk.org > Cc: Walukiewicz, Miroslaw; Liu, Jijiang; Liu, Yong; jigsaw at gmail.com; > Richardson, Bruce > Subject: Re: [PAT

[dpdk-dev] [PATCH 1/3] mbuf:add two TX offload flags and change three fields

2014-11-27 Thread Olivier MATZ
Hi Jijiang, Please see some comments below. On 11/27/2014 09:18 AM, Jijiang Liu wrote: > In place of removing the PKT_TX_VXLAN_CKSUM, we introduce 2 new flags: > PKT_TX_OUT_IP_CKSUM, PKT_TX_UDP_TUNNEL_PKT, and a new field: l4_tun_len. > Replace the inner_l2_len and the inner_l3_len field with th

[dpdk-dev] [PATCH 0/3] i40e VXLAN TX checksum rework

2014-11-27 Thread Olivier MATZ
Hi Jijiang, Please find below some comments about the specifications. The global picture looks fine to me. I've not reviewed the patch right now, but it's in the pipe. On 11/27/2014 09:18 AM, Jijiang Liu wrote: > We have got some feedback about backward compatibility of VXLAN TX checksum > offl

[dpdk-dev] [PATCH] table: hash: fix entry size of configurable key size ext and lru

2014-11-27 Thread Dumitrescu, Cristian
Acked by: Cristian Dumitrescu -Original Message- From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] Sent: Wednesday, November 26, 2014 9:18 PM To: Takayuki Usui; Dumitrescu, Cristian Cc: dev at dpdk.org; Nemeth, Balazs Subject: Re: [dpdk-dev] [PATCH] table: hash: fix entry size of

[dpdk-dev] [PATCH] table: hash: fix entry size of configurable key size ext and lru

2014-11-27 Thread Dumitrescu, Cristian
Hi Thomas, Yes, we can confirm that this patch is identical to one of the fixes provided by Balazs with his set of patches which were already applied. Sorry for not seeing this patch earlier. Thank you, Takayuki San for doing this work and sending us the patch! Regards, Cristian -Origina

[dpdk-dev] [PATCH] scripts: fix symbol overriding in configuration files

2014-11-27 Thread Thomas Monjalon
> When redefining the same symbol in configuration (basically after an > inclusion), > we need to undefine the previous symbol to avoid "redefined" errors. > > Signed-off-by: David Marchand Acked-by: Thomas Monjalon Applied Thanks -- Thomas

[dpdk-dev] [PATCH v4 08/13] testpmd: rework csum forward engine

2014-11-27 Thread Olivier MATZ
Hi Konstantin, On 11/26/2014 09:02 PM, Ananyev, Konstantin wrote: >> +/* if possible, calculate the checksum of a packet in hw or sw, >> + * depending on the testpmd command line configuration */ >> +static uint64_t >> +process_inner_cksums(void *l3_hdr, uint16_t ethertype, uint16_t l3_len, >> +

[dpdk-dev] [PATCH] kni: create KNI interface in current network namespace

2014-11-27 Thread Nicolas Dichtel
Le 21/11/2014 04:10, Takayuki Usui a ?crit : > With this patch, KNI interface (e.g. vEth0) is created in the > network namespace where the DPDK application is running. > Otherwise, all interfaces are created in the default namespace > in the host. > > Signed-off-by: Takayuki Usui > --- > lib/lib

[dpdk-dev] [PATCH 0/3] i40e VXLAN TX checksum rework

2014-11-27 Thread Liu, Jijiang
Could you review this patch today ? If possible, I hope this patch set can be included RC2. > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jijiang Liu > Sent: Thursday, November 27, 2014 4:19 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH 0/3] i40e

[dpdk-dev] [PATCH] eal: fix remaining checks for other 64bits architectures

2014-11-27 Thread Thomas Monjalon
> > RTE_ARCH_X86_64 can not be used as a way to determine if we are building for > > 64bits cpus. Instead, RTE_ARCH_64 should be used. > > > > Signed-off-by: David Marchand > > Acked-by: Chao Zhu Applied Thanks -- Thomas

[dpdk-dev] [PATCH v3 08/13] testpmd: rework csum forward engine

2014-11-27 Thread Liu, Jijiang
> -Original Message- > From: Ananyev, Konstantin > Sent: Thursday, November 27, 2014 12:35 AM > To: Olivier MATZ; dev at dpdk.org > Cc: Walukiewicz, Miroslaw; Liu, Jijiang; Liu, Yong; jigsaw at gmail.com; > Richardson, > Bruce > Subject: RE: [PATCH v3 08/13] testpmd: rework csum forward

[dpdk-dev] [PATCH v4 08/13] testpmd: rework csum forward engine

2014-11-27 Thread Liu, Jijiang
> -Original Message- > From: Ananyev, Konstantin > Sent: Thursday, November 27, 2014 4:03 AM > To: Olivier Matz; dev at dpdk.org > Cc: Walukiewicz, Miroslaw; Liu, Jijiang; Liu, Yong; jigsaw at gmail.com; > Richardson, > Bruce > Subject: RE: [PATCH v4 08/13] testpmd: rework csum forward e

[dpdk-dev] [PATCH v3 1/2] ethdev: new structure of Ethertype Filter for filter_ctrl api

2014-11-27 Thread Qiu, Michael
On 11/13/2014 8:51 PM, Jingjing Wu wrote: > From: "jingjing.wu" > > A new structure of ethertype filter is defined in rte_eth_ctrl.h > for filter_ctrl api > > Signed-off-by: jingjing.wu > --- > lib/librte_ether/rte_eth_ctrl.h | 20 > 1 file changed, 20 insertions(+) > > diff

[dpdk-dev] [PATCH v6 0/6] enicpmd: Cisco Systems Inc. VIC Ethernet PMD

2014-11-27 Thread Sujith Sankar (ssujith)
Thanks Thomas, David and Neil ! I shall work on finishing the documentation. About that, you had mentioned that you wanted it in doc/drivers/ path. Could I send a patch with documentation in the path doc/drivers/enicpmd/ ? Thanks, -Sujith On 27/11/14 3:41 am, "Thomas Monjalon" wrote: >> > ENIC