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

2016-03-03 Thread Dave Neary
Thank you Thomas, It's great to see progress towards the plans we made in Dublin. Thanks, Dave. On 03/03/2016 06:52 PM, Thomas Monjalon wrote: > There were some discussions to form a technical board for the DPDK: > http://thread.gmane.org/gmane.comp.networking.dpdk.devel/26596 > > After

[dpdk-dev] [PATCH v2 0/3] ABI change for RETA, cmdline

2016-03-03 Thread Thomas Monjalon
2016-03-03 16:16, Adrien Mazarguil: > On Tue, Jan 12, 2016 at 11:49:06AM +0100, Nelio Laranjeiro wrote: > > Previous version of commit > > "cmdline: increase command line buffer", had side effects and was breaking > > some commands. > > > > In this version, I only applied John McNamara's solution

[dpdk-dev] [PATCH v3 00/30] i40e base driver update

2016-03-03 Thread Bruce Richardson
On Thu, Feb 18, 2016 at 10:34:06PM +0800, Helin Zhang wrote: > i40e base driver is updated, to support new X722 device IDs, and > use rx control AQ commands to read/write rx control registers. > Of cause, fixes and enhancements are added as listed as below. > > v3: > - As release_2_3.rst has

[dpdk-dev] [PATCH] hash: fix memcmp function pointer in multi-process environment

2016-03-03 Thread Dhananjaya Reddy Eadala
Hi Michael Please see my answers to your comments here. 1. Sure, I will refactor the commit log to restrict not more than 80 characters. 2. Not sure how we can ifdef at the location you mentioned. Can you please elaborate more on this. We already have similar ifdef protection to what you

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

2016-03-03 Thread Thomas Monjalon
2016-02-22 07:53, Keith Wiles: > --- /dev/null > +++ b/config/common_base > +CONFIG_RTE_EAL_IGB_UIO=y > +CONFIG_RTE_EAL_VFIO=y These options should be disabled in the base file and enabled in Linux. > +CONFIG_RTE_LIBRTE_PMD_AF_PACKET=y Idem, it should be disabled. > +CONFIG_RTE_LIBRTE_POWER=y

[dpdk-dev] [PATCH v1 0/3] virtio vector and misc

2016-03-03 Thread Santosh Shukla
On Thu, Mar 3, 2016 at 6:56 PM, Thomas Monjalon wrote: > 2016-03-02 14:11, Santosh Shukla: >> On Wed, Mar 2, 2016 at 2:02 PM, Yuanhan Liu >> wrote: >> > On Tue, Mar 01, 2016 at 03:32:17PM +0530, Santosh Shukla wrote: >> >> - 1st patch: let non-x86 arch use virtio pmd driver in non-vec >> >> -

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

2016-03-03 Thread Wiles, Keith
>2016-02-22 07:53, Keith Wiles: >> --- /dev/null >> +++ b/config/common_base >> +CONFIG_RTE_EAL_IGB_UIO=y >> +CONFIG_RTE_EAL_VFIO=y > >These options should be disabled in the base file >and enabled in Linux. > >> +CONFIG_RTE_LIBRTE_PMD_AF_PACKET=y > >Idem, it should be disabled. > >>

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

2016-03-03 Thread Ananyev, Konstantin
> > This patch set add the VxLAN & NVGRE checksum off-load support. > Both RX and TX checksum off-load can be used for VxLAN & NVGRE. > And the VxLAN port can be set, it's implemented in this patch > set either. > > v2: > - Update release note. > > v3: > - Update RX/TX offload capability. > -

[dpdk-dev] [PATCH] examples/l3fwd: em path performance fix

2016-03-03 Thread Tomasz Kulasek
It seems that for the most use cases, previous hash_multi_lookup provides better performance, and more, sequential lookup can cause significant performance drop. This patch sets previously optional hash_multi_lookup method as default. It also provides some minor optimizations such as queue drain

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-03-03 Thread Ferruh Yigit
On 3/3/2016 4:59 PM, Stephen Hemminger wrote: > On Thu, 3 Mar 2016 10:11:57 + > Ferruh Yigit wrote: > >> On 3/2/2016 10:18 PM, Jay Rolette wrote: >>> >>> On Tue, Mar 1, 2016 at 8:02 PM, Stephen Hemminger >>> mailto:stephen at networkplumber.org>> >>> wrote: >>> >>> On Mon, 29 Feb 2016

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

2016-03-03 Thread Ferruh Yigit
On 3/3/2016 5:45 PM, Ananyev, Konstantin wrote: > > >> -Original Message- >> From: Yigit, Ferruh >> Sent: Thursday, March 03, 2016 5:35 PM >> To: Ananyev, Konstantin; dev at dpdk.org >> Subject: Re: [dpdk-dev] [PATCH] igb_uio: use macros for array size >> calculation >> >> On 3/3/2016

[dpdk-dev] [PATCH] mk: add makefile extention support

2016-03-03 Thread Wiles, Keith
>2016-03-03 14:55, Wiles, Keith: >> >>2016-02-28 21:47, Wiles, Keith: >> >>> >Hi, >> >>> > >> >>> >2016-02-09 11:35, Keith Wiles: >> >>> >> Adding support to the build system to allow for Makefile.XXX >> >>> >> extention to a subtree, which already has Makefiles. These >> >>> >> Makefiles could be

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

2016-03-03 Thread Ananyev, Konstantin
> -Original Message- > From: Yigit, Ferruh > Sent: Thursday, March 03, 2016 5:35 PM > To: Ananyev, Konstantin; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] igb_uio: use macros for array size calculation > > On 3/3/2016 5:25 PM, Ananyev, Konstantin wrote: > > > > > >> -Original

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

2016-03-03 Thread Xie, Huawei
On 2/18/2016 9:48 PM, Yuanhan Liu wrote: > The current rte_vhost_dequeue_burst() implementation is a bit messy [...] > + > uint16_t > rte_vhost_dequeue_burst(struct virtio_net *dev, uint16_t queue_id, > struct rte_mempool *mbuf_pool, struct rte_mbuf **pkts, uint16_t count) > { > -

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

2016-03-03 Thread Ferruh Yigit
On 3/3/2016 5:25 PM, Ananyev, Konstantin wrote: > > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ferruh Yigit >> Sent: Thursday, March 03, 2016 5:08 PM >> To: dev at dpdk.org >> Subject: [dpdk-dev] [PATCH] igb_uio: use macros for array size calculation

[dpdk-dev] [PATCH] mk: add makefile extention support

2016-03-03 Thread Thomas Monjalon
2016-03-03 14:55, Wiles, Keith: > >>2016-02-28 21:47, Wiles, Keith: > >>> >Hi, > >>> > > >>> >2016-02-09 11:35, Keith Wiles: > >>> >> Adding support to the build system to allow for Makefile.XXX > >>> >> extention to a subtree, which already has Makefiles. These > >>> >> Makefiles could be from

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

2016-03-03 Thread Xie, Huawei
On 2/18/2016 9:48 PM, Yuanhan Liu wrote: > [...] CCed changchun, the author for the chained handling of desc and mbuf. The change makes the code more readable, but i think the following commit message is simple and enough. > > while (this_desc_is_not_drained_totally || has_next_desc) { >

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

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

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

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

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

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

[dpdk-dev] [PATCH v2 0/3] ABI change for RETA, cmdline

2016-03-03 Thread Adrien Mazarguil
On Tue, Jan 12, 2016 at 11:49:06AM +0100, Nelio Laranjeiro wrote: > Previous version of commit > "cmdline: increase command line buffer", had side effects and was breaking > some commands. > > In this version, I only applied John McNamara's solution which consists in > increasing only

[dpdk-dev] [PATCH v4 05/12] pmd/fm10k: add dev_ptype_info_get implementation

2016-03-03 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Tan, Jianfeng > Sent: Thursday, March 03, 2016 6:04 AM > To: Chen, Jing D; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v4 05/12] pmd/fm10k: add dev_ptype_info_get > implementation > > Hi, > > On

[dpdk-dev] [PATCH v2 5/5] mlx5: add VLAN insertion offload

2016-03-03 Thread Adrien Mazarguil
From: Yaacov Hazan VLAN insertion is done in software by the PMD by default unless CONFIG_RTE_LIBRTE_MLX5_VERBS_VLAN_INSERTION is enabled and Verbs provides support for hardware insertion. When enabled, this option improves performance when VLAN insertion is requested,

[dpdk-dev] [PATCH v2 4/5] mlx5: add support for HW packet padding

2016-03-03 Thread Adrien Mazarguil
From: Olga Shern Environment variable MLX5_PMD_ENABLE_PADDING enables HW packet padding in PCI bus transactions. When packet size is cache aligned and CRC stripping is enabled, 4 fewer bytes are written to the PCI bus. Enabling padding makes such packets aligned again. In

[dpdk-dev] [PATCH v2 3/5] mlx5: add RX CRC stripping configuration

2016-03-03 Thread Adrien Mazarguil
From: Olga Shern Until now, CRC was always stripped by hardware. This feature can be configured since MLNX_OFED >= 3.2. Signed-off-by: Olga Shern --- doc/guides/nics/mlx5.rst | 2 ++ doc/guides/rel_notes/release_16_04.rst | 6 ++

[dpdk-dev] [PATCH v2 2/5] mlx5: allow operation in secondary processes

2016-03-03 Thread Adrien Mazarguil
From: Or Ami Secondary processes are expected to use queues and other resources allocated by the primary, however Verbs resources can only be shared between processes when inherited through fork(). This limitation can be worked around for TX by configuring separate queues

[dpdk-dev] [PATCH v2 1/5] mlx5: add callbacks to support link (up / down) changes

2016-03-03 Thread Adrien Mazarguil
From: Or Ami Burst functions are updated to make sure applications cannot attempt to send/receive after link is brought down. Signed-off-by: Or Ami --- doc/guides/rel_notes/release_16_04.rst | 4 ++ drivers/net/mlx5/mlx5.c| 2 + drivers/net/mlx5/mlx5.h

[dpdk-dev] [PATCH v2 0/5] Implement missing features in mlx5

2016-03-03 Thread Adrien Mazarguil
This patchset adds to mlx5 a few features available in mlx4 (TX from secondary processes) or provided by Verbs (support for HW packet padding, TX VLAN insertion). Release notes and documentation are updated accordingly. Note: should be applied after "Assorted fixes for mlx4 and mlx5". Changes

[dpdk-dev] [PATCH v2 7/7] mlx4: make sure that number of RX queues is a power of 2

2016-03-03 Thread Adrien Mazarguil
From: Robin Jarry In the documentation it is specified that the hardware only supports a number of RX queues if it is a power of 2. Since ibv_exp_create_qp may not return an error when the number of queues is unsupported by hardware, sanitize the value in dev_configure.

[dpdk-dev] [PATCH v2 6/7] mlx5: fix RX checksum offload in non L3/L4 packets

2016-03-03 Thread Adrien Mazarguil
From: Yaacov Hazan Change rxq_cq_to_ol_flags() to set checksum flags according to packet type, so for non L3/L4 packets the mbuf chksum_bad flags will not be set. Fixes: 67fa62bc672d ("mlx5: support checksum offload") Signed-off-by: Yaacov Hazan ---

[dpdk-dev] [PATCH v2 5/7] mlx5: apply VLAN filtering to broadcast and IPv6 multicast flows

2016-03-03 Thread Adrien Mazarguil
Unlike promiscuous and allmulticast flows, those should remain VLAN-specific. Signed-off-by: Adrien Mazarguil --- doc/guides/rel_notes/release_16_04.rst | 4 ++ drivers/net/mlx5/mlx5_rxmode.c | 105 + drivers/net/mlx5/mlx5_rxq.c| 5 +-

[dpdk-dev] [PATCH v2 4/7] mlx5: remove redundant debugging message

2016-03-03 Thread Adrien Mazarguil
Signed-off-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5_rxmode.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/net/mlx5/mlx5_rxmode.c b/drivers/net/mlx5/mlx5_rxmode.c index bcf4231..730527e 100644 --- a/drivers/net/mlx5/mlx5_rxmode.c +++ b/drivers/net/mlx5/mlx5_rxmode.c @@

[dpdk-dev] [PATCH v2 3/7] mlx5: manage all special flow types at once

2016-03-03 Thread Adrien Mazarguil
This commit adds helpers to remove redundant code. Signed-off-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5.c | 5 + drivers/net/mlx5/mlx5.h | 2 ++ drivers/net/mlx5/mlx5_rxmode.c | 40 drivers/net/mlx5/mlx5_trigger.c | 10

[dpdk-dev] [PATCH v2 2/7] mlx5: check if port is configured as Ethernet device

2016-03-03 Thread Adrien Mazarguil
From: Or Ami If the port link layer is not Ethernet, notify the user. Signed-off-by: Or Ami --- doc/guides/rel_notes/release_16_04.rst | 5 + drivers/net/mlx5/mlx5.c| 7 +++ 2 files changed, 12 insertions(+) diff --git

[dpdk-dev] [PATCH v2 1/7] mlx5: fix possible crash during initialization

2016-03-03 Thread Adrien Mazarguil
From: Or Ami RSS configuration should not be freed when priv is NULL. Fixes: 2f97422e7759 ("mlx5: support RSS hash update and get") Signed-off-by: Or Ami --- doc/guides/rel_notes/release_16_04.rst | 4 drivers/net/mlx5/mlx5.c| 6 -- 2 files

[dpdk-dev] [PATCH v2 0/7] Assorted fixes for mlx4 and mlx5

2016-03-03 Thread Adrien Mazarguil
This patchset addresses several minor issues, release notes are updated accordingly. Note: should be applied after "Performance optimizations for mlx5 and mlx4". Changes in v2: - None, submitted again due to dependency with previous patchset. Adrien Mazarguil (3): mlx5: manage all special

[dpdk-dev] [PATCH v3 7/7] mlx: use aligned memory to register regions

2016-03-03 Thread Adrien Mazarguil
The first and last memory pool elements are usually cache-aligned but not page-aligned, particularly when using huge pages. Hardware performance can be improved significantly by registering memory regions starting and ending on page boundaries. Signed-off-by: Adrien Mazarguil ---

[dpdk-dev] [PATCH v3 6/7] mlx5: free buffers immediately after completion

2016-03-03 Thread Adrien Mazarguil
From: Nelio Laranjeiro This lowers the amount of cache misses. Signed-off-by: Nelio Laranjeiro --- drivers/net/mlx5/mlx5_rxtx.c | 35 --- 1 file changed, 20 insertions(+), 15 deletions(-) diff --git a/drivers/net/mlx5/mlx5_rxtx.c

[dpdk-dev] [PATCH v3 5/7] mlx5: avoid lkey retrieval for inlined packets

2016-03-03 Thread Adrien Mazarguil
From: Nelio Laranjeiro Improves performance as the lkey is not needed by hardware in this case. Signed-off-by: Nelio Laranjeiro --- drivers/net/mlx5/mlx5_rxtx.c | 23 +-- 1 file changed, 13 insertions(+), 10 deletions(-) diff --git

[dpdk-dev] [PATCH v3 4/7] mlx5: process offload flags only when requested

2016-03-03 Thread Adrien Mazarguil
From: Nelio Laranjeiro Improve performance by processing offloads only when requested by the application. Signed-off-by: Nelio Laranjeiro --- drivers/net/mlx5/mlx5_rxtx.c | 29 - 1 file changed, 16 insertions(+), 13 deletions(-) diff

[dpdk-dev] [PATCH v3 3/7] mlx5: remove one indirection level from RX/TX functions

2016-03-03 Thread Adrien Mazarguil
From: Nelio Laranjeiro Avoid dereferencing pointers twice to get to fast Verbs functions by storing them directly in RX/TX queue structures. Signed-off-by: Nelio Laranjeiro Signed-off-by: Yaacov Hazan --- drivers/net/mlx5/Makefile| 1 +

[dpdk-dev] [PATCH v3 2/7] mlx5: reorder TX/RX queue structure

2016-03-03 Thread Adrien Mazarguil
From: Nelio Laranjeiro Remove padding and move important fields to the beginning for better performance. Signed-off-by: Nelio Laranjeiro --- drivers/net/mlx5/mlx5_rxtx.h | 31 --- 1 file changed, 16 insertions(+), 15 deletions(-) diff

[dpdk-dev] [PATCH v3 1/7] mlx5: prefetch next TX mbuf header and data

2016-03-03 Thread Adrien Mazarguil
From: Nelio Laranjeiro This change improves performance noticeably. Signed-off-by: Nelio Laranjeiro --- drivers/net/mlx5/mlx5_rxtx.c | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/drivers/net/mlx5/mlx5_rxtx.c

[dpdk-dev] [PATCH v3 0/7] Performance optimizations for mlx5 and mlx4

2016-03-03 Thread Adrien Mazarguil
This patchset improves the mlx5 PMD performance by doing better prefetching, by reordering internal structure fields and by removing a few unnecessary operations. Note: should be applied after "Add flow director and RX VLAN stripping support" to avoid conflicts. Changes in v3: - None, submitted

[dpdk-dev] [PATCH v3 5/5] mlx5: add support for RX VLAN stripping

2016-03-03 Thread Adrien Mazarguil
From: Yaacov Hazan Allows HW to strip the 802.1Q header from incoming frames and report it through the mbuf structure. This feature requires MLNX_OFED >= 3.2. Signed-off-by: Yaacov Hazan Signed-off-by: Adrien Mazarguil --- doc/guides/nics/mlx5.rst | 2 +

[dpdk-dev] [PATCH v3 4/5] mlx5: add support for flow director

2016-03-03 Thread Adrien Mazarguil
From: Yaacov Hazan Add support for flow director filters (RTE_FDIR_MODE_PERFECT and RTE_FDIR_MODE_PERFECT_MAC_VLAN modes). This feature requires MLNX_OFED >= 3.2. Signed-off-by: Yaacov Hazan Signed-off-by: Adrien Mazarguil Signed-off-by: Raslan Darawsheh ---

[dpdk-dev] [PATCH v3 3/5] mlx5: make flow steering rule generator more generic

2016-03-03 Thread Adrien Mazarguil
From: Yaacov Hazan Upcoming flow director support will reuse this function to generate filter rules. Signed-off-by: Yaacov Hazan Signed-off-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5_mac.c| 4 ++-- drivers/net/mlx5/mlx5_rxmode.c | 5 +++--

[dpdk-dev] [PATCH v3 2/5] mlx5: add special flows (broadcast and IPv6 multicast)

2016-03-03 Thread Adrien Mazarguil
From: Yaacov Hazan Until now, broadcast frames were handled like unicast. Moving the related flow to the special flows table frees up the related unicast MAC entry. The same method is used to handle IPv6 multicast frames. Signed-off-by: Yaacov Hazan Signed-off-by: Adrien

[dpdk-dev] [PATCH v3 1/5] mlx5: refactor special flows handling

2016-03-03 Thread Adrien Mazarguil
From: Yaacov Hazan Merge redundant code by adding a static initialization table to manage promiscuous and allmulticast (special) flows. New function priv_rehash_flows() implements the logic to enable/disable relevant flows in one place from any context. Signed-off-by:

[dpdk-dev] [PATCH v3 0/5] Add flow director and RX VLAN stripping support

2016-03-03 Thread Adrien Mazarguil
To preserve compatibility with Mellanox OFED 3.1, flow director and RX VLAN stripping code is only enabled if compiled with 3.2. Changes in v3: - Fixed flow registration issue caused by missing masks in flow rules. - Fixed packet duplication with overlapping FDIR rules. - Added FDIR flush command

[dpdk-dev] [PATCH 01/10] ethdev: add a generic flow and new behavior switch to fdir

2016-03-03 Thread Olga Shern
I think what Thomas meant is that we should redesign Flow Director feature and call it something else , Mellanox is calling it "Flow Steering" . I agree that Filtering may be more generic name. We have implemented Flow Director API in Mellanox ConnectX-4 PMD (part of the DPDK 16.04 patches)

[dpdk-dev] [PATCH] eal: add missing long-options for short option arguments

2016-03-03 Thread Wiles, Keith
>On Thu, Feb 25, 2016 at 11:12 PM, Wiles, Keith >wrote: >>>On Thu, Feb 25, 2016 at 01:09:16PM -0600, Keith Wiles wrote: A number of short options for EAL are missing long options and this patch adds those missing options. The missing long options are for: -c add

[dpdk-dev] [PATCH] mk: add makefile extention support

2016-03-03 Thread Wiles, Keith
>>2016-02-28 21:47, Wiles, Keith: >>> >Hi, >>> > >>> >2016-02-09 11:35, Keith Wiles: >>> >> Adding support to the build system to allow for Makefile.XXX >>> >> extention to a subtree, which already has Makefiles. These >>> >> Makefiles could be from the autotools and others places. Using >>> >>

[dpdk-dev] [PATCH] eal: add missing long-options for short option arguments

2016-03-03 Thread Wiles, Keith
>>On Thu, Feb 25, 2016 at 01:09:16PM -0600, Keith Wiles wrote: >>> A number of short options for EAL are missing long options >>> and this patch adds those missing options. >>> >>> The missing long options are for: >>> -c add --coremask >>> -d add --driver >>> -l add --corelist >>> -m add

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

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

[dpdk-dev] [PATCH v1 0/3] virtio vector and misc

2016-03-03 Thread Thomas Monjalon
2016-03-02 14:11, Santosh Shukla: > On Wed, Mar 2, 2016 at 2:02 PM, Yuanhan Liu > wrote: > > On Tue, Mar 01, 2016 at 03:32:17PM +0530, Santosh Shukla wrote: > >> - 1st patch: let non-x86 arch use virtio pmd driver in non-vec > >> - 2nd patch: enable virtio arm support > >> - 3rd patch: update

[dpdk-dev] [PATCH v4 05/12] pmd/fm10k: add dev_ptype_info_get implementation

2016-03-03 Thread Tan, Jianfeng
Hi, On 3/3/2016 4:11 AM, Chen, Jing D wrote: > Hi, > > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jianfeng Tan > Sent: Thursday, February 25, 2016 6:09 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v4 05/12] pmd/fm10k: add dev_ptype_info_get >

[dpdk-dev] [PATCH v4] af_packet: make the device detachable

2016-03-03 Thread Iremonger, Bernard
> -Original Message- > From: Wojciech Zmuda [mailto:woz at semihalf.com] > Sent: Wednesday, March 2, 2016 11:56 AM > To: dev at dpdk.org > Cc: Iremonger, Bernard ; > linville at tuxdriver.com; Richardson, Bruce ; > pmatilai at redhat.com > Subject: [PATCH v4] af_packet: make the device

[dpdk-dev] [PATCH v3 3/3] app/test: add Snow3G tests

2016-03-03 Thread Deepak Kumar JAIN
Signed-off-by: Deepak Kumar JAIN --- app/test/test_cryptodev.c | 1037 +++- app/test/test_cryptodev.h |3 +- app/test/test_cryptodev_snow3g_hash_test_vectors.h | 415 app/test/test_cryptodev_snow3g_test_vectors.h

[dpdk-dev] [PATCH v3 2/3] qat: add support for Snow3G

2016-03-03 Thread Deepak Kumar JAIN
Signed-off-by: Deepak Kumar JAIN --- doc/guides/cryptodevs/qat.rst| 8 ++- doc/guides/rel_notes/release_16_04.rst | 6 ++ drivers/crypto/qat/qat_adf/qat_algs.h| 1 + drivers/crypto/qat/qat_adf/qat_algs_build_desc.c | 86 +---

[dpdk-dev] [PATCH v3 1/3] crypto: add cipher/auth only support

2016-03-03 Thread Deepak Kumar JAIN
Refactored the existing functionality into modular form to support the cipher/auth only functionalities. Signed-off-by: Deepak Kumar JAIN --- drivers/crypto/qat/qat_adf/qat_algs.h| 18 +- drivers/crypto/qat/qat_adf/qat_algs_build_desc.c | 210 ---

[dpdk-dev] [PATCH v3 0/3] Snow3G support for Intel Quick Assist Devices

2016-03-03 Thread Deepak Kumar JAIN
This patchset contains fixes and refactoring for Snow3G(UEA2 and UIA2) wireless algorithm for Intel Quick Assist devices. QAT PMD previously supported only cipher/hash alg-chaining for AES/SHA. The code has been refactored to also support cipher-only and hash only (for Snow3G only)

[dpdk-dev] [PATCH v3] mk: stop on warning only in developer build

2016-03-03 Thread Panu Matilainen
On 03/03/2016 12:36 PM, Thomas Monjalon wrote: > 2016-03-02 22:04, Bruce Richardson: >> On Wed, Mar 02, 2016 at 03:22:23PM +0100, Thomas Monjalon wrote: >>> From: Panu Matilainen >>> >>> Add RTE_DEVEL_BUILD make-variable which can be used to do things >>> differently when doing development vs

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-03-03 Thread Panu Matilainen
On 03/03/2016 12:05 PM, Ferruh Yigit wrote: > On 3/3/2016 8:31 AM, Panu Matilainen wrote: >> On 03/03/2016 12:35 AM, Thomas Monjalon wrote: >>> 2016-03-02 12:21, Thomas Monjalon: 2016-03-02 11:47, Vincent JARDIN: > Le 02/03/2016 09:27, Panu Matilainen a ?crit : I'd like to see

[dpdk-dev] [PATCH v6 1/5] lib/librte_ether: change function name of tunnel port config

2016-03-03 Thread Panu Matilainen
On 03/03/2016 03:22 AM, Wenzhuo Lu wrote: > The names of function for tunnel port configuration are not > accurate. They're tunnel_add/del, better change them to > tunnel_port_add/del. > As it may be an ABI change if change the names directly, the > new functions are added but not remove the old

[dpdk-dev] New driver (large patch) question.

2016-03-03 Thread Stephen Hurd
Unfortunately no. On Wed, Mar 2, 2016 at 9:53 PM, Qiu, Michael wrote: > On 3/3/2016 7:11 AM, Stephen Hurd wrote: > > On Wed, Mar 2, 2016 at 2:15 PM, Thomas Monjalon < > thomas.monjalon at 6wind.com> > > wrote: > > > >>> The comments in it are the only publicly available > >>> documentation on

[dpdk-dev] [PATCH] eal: fix symbol map version number

2016-03-03 Thread Thomas Monjalon
> The version 2.3 has been renamed 16.04. > > Fixes: 6d7de6d2e357 ("version: switch to year.month numbers") > > Reported-by: Panu Matilainen > Signed-off-by: Thomas Monjalon Applied

[dpdk-dev] [PATCH] mk: fix error message

2016-03-03 Thread Thomas Monjalon
2016-03-02 16:52, Thomas Monjalon: > When specifying a wrong directory with RTE_SDK and RTE_TARGET > to build an application, the error message about missing config > file was wrong. > > Fixes: 6b62a72a70d0 ("mk: install a standard cutomizable tree") > > Reported-by: Steeven Lee >

[dpdk-dev] [PATCH v3] mk: stop on warning only in developer build

2016-03-03 Thread Thomas Monjalon
2016-03-02 22:04, Bruce Richardson: > On Wed, Mar 02, 2016 at 03:22:23PM +0100, Thomas Monjalon wrote: > > From: Panu Matilainen > > > > Add RTE_DEVEL_BUILD make-variable which can be used to do things > > differently when doing development vs building a release, > > autodetected from source

[dpdk-dev] Clarification on latency at NIC line rate..

2016-03-03 Thread Saravana Kumar
Hi DPDK community, I have been experimenting with dpdk eth_tx send on ixgbe pmd (82599 NIC) I am seeing the following behavior When i pass 1500 bytes at 9.99GB traffic -- latency looks fine (25 microseconds) but when i pass 1500 bytes hitting 10GB traffic(line rate)

[dpdk-dev] [PATCH v3 1/2] librte_pipeline: add support for packet redirection at action handlers

2016-03-03 Thread Panu Matilainen
On 03/02/2016 10:41 PM, Jasvinder Singh wrote: > Currently, there is no mechanism that allows the pipeline ports (in/out) and > table action handlers to override the default forwarding decision (as > previously configured per input port or in the table entry). Therefore, new > pipeline API

[dpdk-dev] [PATCH] nfp: fix how tx checksum is advertised to firmware

2016-03-03 Thread Alejandro Lucero
Even with tx checksum offload available, do not set the flag by default. Fixes: b812daadad0d (\"nfp: add Rx and Tx\") Signed-off-by: Alejandro Lucero --- drivers/net/nfp/nfp_net.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/net/nfp/nfp_net.c

[dpdk-dev] [PATCH 1/7] lib/librte_ether: Add 2/2.5/25/50Gbps link speeds

2016-03-03 Thread Simon Kågström
On 2016-03-03 10:28, Thomas Monjalon wrote: > 2016-03-03 08:53, Simon K?gstr?m: >> I realize this is a more general question, but is it really meaningful >> to have macros for all possible link speeds? We're working on a PMD >> driver with a channelized interface exposed as DPDK ports. Each

[dpdk-dev] [PATCH] nfp: tx checksum offload fixes

2016-03-03 Thread Thomas Monjalon
Hi Alejandro, Please start the title with a verb (fix here), describe briefly the bug, and add a Fixes: tag. More info in this doc: http://dpdk.org/doc/guides/contributing/patches.html#commit-messages-subject-line 2016-03-03 10:08, Alejandro Lucero: > Signed-off-by: Alejandro Lucero > --- >

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-03-03 Thread Thomas Monjalon
2016-03-03 10:05, Ferruh Yigit: > On 3/3/2016 8:31 AM, Panu Matilainen wrote: > > On 03/03/2016 12:35 AM, Thomas Monjalon wrote: > >> 2016-03-02 12:21, Thomas Monjalon: > >>> 2016-03-02 11:47, Vincent JARDIN: > Le 02/03/2016 09:27, Panu Matilainen a ?crit : > >>> I'd like to see these be

[dpdk-dev] [PATCH v2] nfp: fix variable type in tx checksum offload

2016-03-03 Thread Alejandro Lucero
The mbuf ol_flags field was changed to uin64_t with DPDK version 1.8 Fixes: b812daadad0d (\"nfp: add Rx and Tx\") Signed-off-by: Alejandro Lucero --- drivers/net/nfp/nfp_net.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/nfp/nfp_net.c

[dpdk-dev] [PATCH v4] librte_pipeline: add support for packet redirection at action handlers

2016-03-03 Thread Jasvinder Singh
Currently, there is no mechanism that allows the pipeline ports (in/out) and table action handlers to override the default forwarding decision (as previously configured per input port or in the table entry). Therefore, new pipeline API functions have been added which allows action handlers to

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-03-03 Thread Panu Matilainen
On 03/03/2016 12:35 AM, Thomas Monjalon wrote: > 2016-03-02 12:21, Thomas Monjalon: >> 2016-03-02 11:47, Vincent JARDIN: >>> Le 02/03/2016 09:27, Panu Matilainen a ?crit : >> I'd like to see these be merged. >> >> Jay > > The code is really not ready. I am okay with cooperative

[dpdk-dev] [PATCH 1/7] lib/librte_ether: Add 2/2.5/25/50Gbps link speeds

2016-03-03 Thread Thomas Monjalon
Hi, 2016-03-03 08:53, Simon K?gstr?m: > Hi! > > On 2016-03-03 05:08, Stephen Hurd wrote: > > Add additional ETH_LINK_SPEED_* macros for 2, 2.5, 25, and 50 Gbps links Stephen, you could be interested in the rework done by Marc Sune: http://dpdk.org/dev/patchwork/patch/10919/ > I realize

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-03-03 Thread Ferruh Yigit
On 3/2/2016 10:18 PM, Jay Rolette wrote: > > On Tue, Mar 1, 2016 at 8:02 PM, Stephen Hemminger > mailto:stephen at networkplumber.org>> wrote: > > On Mon, 29 Feb 2016 08:33:25 -0600 > Jay Rolette mailto:rolette at infiniteio.com>> > wrote: > > > On Mon, Feb 29, 2016 at 5:06 AM,

[dpdk-dev] [PATCH v6 0/6] interrupt mode for fm10k

2016-03-03 Thread Thomas Monjalon
2016-03-03 08:36, Liu, Yong: > Hi Thomas, > In patchwork website, this patch set seem that has been applied. > http://dpdk.org/dev/patchwork/patch/10381/ > > But I can't find it neither in dpdk repo nor in dpdk-next-net repo. > Could you check with that? It's here:

[dpdk-dev] [PATCH] nfp: tx checksum offload fixes

2016-03-03 Thread Alejandro Lucero
Signed-off-by: Alejandro Lucero --- drivers/net/nfp/nfp_net.c |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/net/nfp/nfp_net.c b/drivers/net/nfp/nfp_net.c index fd4dd39..6078e9f 100644 --- a/drivers/net/nfp/nfp_net.c +++ b/drivers/net/nfp/nfp_net.c @@ -1522,7

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-03-03 Thread Ferruh Yigit
On 3/3/2016 8:31 AM, Panu Matilainen wrote: > On 03/03/2016 12:35 AM, Thomas Monjalon wrote: >> 2016-03-02 12:21, Thomas Monjalon: >>> 2016-03-02 11:47, Vincent JARDIN: Le 02/03/2016 09:27, Panu Matilainen a ?crit : >>> I'd like to see these be merged. >>> >>> Jay >> >>

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

2016-03-03 Thread Iremonger, Bernard
Hi Wenzhuo, > -Original Message- > From: Lu, Wenzhuo > Sent: Thursday, March 3, 2016 2:34 AM > To: Iremonger, Bernard ; dev at dpdk.org > Subject: RE: [dpdk-dev] [PATCH] e1000: fix setting of VF MAC address > > Hi Bernard, > > > > -Original Message- > > From: dev

[dpdk-dev] [PATCH v3 1/2] librte_pipeline: add support for packet redirection at action handlers

2016-03-03 Thread Singh, Jasvinder
> -Original Message- > From: Panu Matilainen [mailto:pmatilai at redhat.com] > Sent: Thursday, March 3, 2016 9:35 AM > To: Singh, Jasvinder ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v3 1/2] librte_pipeline: add support for > packet redirection at action handlers > > On

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

2016-03-03 Thread Wenzhuo Lu
The patch add VxLAN & NVGRE TX checksum off-load. When the flag of outer IP header checksum offload is set, we'll set the context descriptor to enable this checksum off-load. Also update release note for VxLAN & NVGRE checksum off-load support. Signed-off-by: Wenzhuo Lu ---

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

2016-03-03 Thread Wenzhuo Lu
X550 will do VxLAN & NVGRE RX checksum off-load automatically. This patch exposes the result of the checksum off-load. Signed-off-by: Wenzhuo Lu --- drivers/net/ixgbe/ixgbe_ethdev.c | 4 drivers/net/ixgbe/ixgbe_rxtx.c | 11 ++- lib/librte_ether/rte_ethdev.h| 1 +

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

2016-03-03 Thread Wenzhuo Lu
Add UDP tunnel port add/del support on ixgbe. Now only support VxLAN port configuration. Although according to the specification the VxLAN port has a default value 4789, it can be changed. We support VxLAN port configuration to meet the change. Note, the default value of VxLAN port in ixgbe NICs

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

2016-03-03 Thread Wenzhuo Lu
As the names of tunnel port config functions are not accurate, change them from tunnel_add/del to tunnel_port_add/del. And support both the old and new rte ops. Signed-off-by: Wenzhuo Lu --- drivers/net/i40e/i40e_ethdev.c | 22 -- 1 file changed, 12 insertions(+), 10

[dpdk-dev] [PATCH v6 1/5] lib/librte_ether: change function name of tunnel port config

2016-03-03 Thread Wenzhuo Lu
The names of function for tunnel port configuration are not accurate. They're tunnel_add/del, better change them to tunnel_port_add/del. As it may be an ABI change if change the names directly, the new functions are added but not remove the old ones. The old ones will be removed in the next

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

2016-03-03 Thread Wenzhuo Lu
This patch set add the VxLAN & NVGRE checksum off-load support. Both RX and TX checksum off-load can be used for VxLAN & NVGRE. And the VxLAN port can be set, it's implemented in this patch set either. v2: - Update release note. v3: - Update RX/TX offload capability. - Reuse PKT_RX_EIP_CKSUM_BAD

[dpdk-dev] [PATCH v6 0/6] interrupt mode for fm10k

2016-03-03 Thread Liu, Yong
Thanks Thomas, found it. > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Thursday, March 03, 2016 5:11 PM > To: Liu, Yong > Cc: He, Shaopeng; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v6 0/6] interrupt mode for fm10k > > 2016-03-03 08:36,

[dpdk-dev] [PATCH 1/7] lib/librte_ether: Add 2/2.5/25/50Gbps link speeds

2016-03-03 Thread Simon Kågström
Hi! On 2016-03-03 05:08, Stephen Hurd wrote: > Add additional ETH_LINK_SPEED_* macros for 2, 2.5, 25, and 50 Gbps links > > Signed-off-by: Stephen Hurd > --- > lib/librte_ether/rte_ethdev.h | 4 > 1 file changed, 4 insertions(+) > > diff --git a/lib/librte_ether/rte_ethdev.h

[dpdk-dev] [PATCH v6 0/6] interrupt mode for fm10k

2016-03-03 Thread Liu, Yong
Hi Thomas, In patchwork website, this patch set seem that has been applied. http://dpdk.org/dev/patchwork/patch/10381/ But I can't find it neither in dpdk repo nor in dpdk-next-net repo. Could you check with that? > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On

[dpdk-dev] [PATCH 1/4] ixgbe: support UDP tunnel add/del

2016-03-03 Thread Lu, Wenzhuo
Hi Michael, > -Original Message- > From: Qiu, Michael > Sent: Thursday, March 3, 2016 2:58 PM > To: Lu, Wenzhuo; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 1/4] ixgbe: support UDP tunnel add/del > > On 1/11/2016 3:08 PM, Wenzhuo Lu wrote: > > Add UDP tunnel add/del support on

[dpdk-dev] [PATCH 1/4] ixgbe: support UDP tunnel add/del

2016-03-03 Thread Qiu, Michael
On 1/11/2016 3:08 PM, Wenzhuo Lu wrote: > Add UDP tunnel add/del support on ixgbe. Now it only support > VxLAN port configuration. > Although the VxLAN port has a default value 4789, it can be > changed. We support VxLAN port configuration to meet the > change. > Note, the default value of VxLAN

[dpdk-dev] New driver (large patch) question.

2016-03-03 Thread Qiu, Michael
On 3/3/2016 7:11 AM, Stephen Hurd wrote: > On Wed, Mar 2, 2016 at 2:15 PM, Thomas Monjalon > wrote: > >>> The comments in it are the only publicly available >>> documentation on the hardware I'm aware of. >> So you must keep the comments. >> > That's my goal, but the comments are well over the

[dpdk-dev] [PATCH] hash: fix memcmp function pointer in multi-process environment

2016-03-03 Thread Qiu, Michael
On 3/3/2016 11:36 AM, Dhana Eadala wrote: > We found a problem in dpdk-2.2 using under multi-process environment. > Here is the brief description how we are using the dpdk: > > We have two processes proc1, proc2 using dpdk. These proc1 and proc2 are two > different compiled binaries. > proc1 is

[dpdk-dev] [PATCH v2] ixgbe: support multicast promiscuous mode on VF

2016-03-03 Thread Wang, Xiao W
Hi, > > > + > > > + err = mbx->ops.write_posted(hw, msgbuf, 2, 0); > > > + if (err) > > > + return err; > > > + > > > + err = mbx->ops.read_posted(hw, msgbuf, 2, 0); > > > > Is it more reasonable to read a message of size 1 than 2? Pf side only > > write 1 word into mbx. > Thanks for the

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

2016-03-03 Thread Lu, Wenzhuo
Hi Bernard, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Bernard Iremonger > Sent: Thursday, March 3, 2016 12:09 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] e1000: fix setting of VF MAC address > > Allow reprogramming of the RAR with a zero

  1   2   >