[dpdk-dev] [PATCH v2] lib/ether: prevent duplicate callback on list

2016-10-25 Thread Thomas Monjalon
2016-10-20 09:34, E. Scott Daniels: > This change prevents the attempt to add a structure which is > already on the callback list. If a struct with matching > parameters is found on the list, then no action is taken. If > a struct with matching parameters is found on the list, then > no action is t

[dpdk-dev] [dpdk-stable] [PATCH v2] mempool: fix search of maximum contiguous pages

2016-10-25 Thread Thomas Monjalon
2016-10-25 17:01, Olivier Matz: > From: Wei Dai > > paddr[i] + pg_sz always points to the start physical address of the > 2nd page after pddr[i], so only up to 2 pages can be combinded to > be used. With this revision, more than 2 pages can be used. > > Fixes: 84121f197187 ("mempool: store memor

[dpdk-dev] [RFC] [PATCH v2] libeventdev: event driven programming model framework for DPDK

2016-10-25 Thread Jerin Jacob
On Wed, Oct 12, 2016 at 01:00:16AM +0530, Jerin Jacob wrote: > Thanks to Intel and NXP folks for the positive and constructive feedback > I've received so far. Here is the updated RFC(v2). > > I've attempted to address as many comments as possible. > > This series adds rte_eventdev.h to the DPDK

[dpdk-dev] [PATCH] testpmd: fix fdir command on MAC and tunnel modes

2016-10-25 Thread Thomas Monjalon
2016-09-27 11:01, Frederico Cadete: > On Tue, Sep 27, 2016 at 4:42 AM, Wu, Jingjing > wrote: > > From: Frederico.Cadete- > >> The flow_director_filter commands has a pf|vf option for most modes > >> except for MAC-VLAN and tunnel. On Intel NIC's these modes are not > >> supported under virtualize

[dpdk-dev] [PATCH v2] app/testpmd: fix PF/VF check of flow director

2016-10-25 Thread Thomas Monjalon
2016-10-19 09:12, Wenzhuo Lu: > Parameters pf & vf are added into most of flow director > filter CLIs. > But mac-valn and tunnel filters don't have these parameters, > the parameters should not be checked for mac-vlan and tunnel > filters. > > Fixes: e6a68c013353 ("app/testpmd: extend commands for

[dpdk-dev] [PATCHv3] examples/l3fwd: em: use hw accelerated crc hash function for arm64

2016-10-25 Thread Thomas Monjalon
2016-10-14 16:40, Hemant Agrawal: > if machine level CRC extension are available, offload the > hash to machine provide functions e.g. armv8-a CRC extensions > support it > > Signed-off-by: Hemant Agrawal > Reviewed-by: Jerin Jacob Applied, thanks

[dpdk-dev] [PATCH v2 1/3] drivers: add name alias registration for rte_driver

2016-10-25 Thread Thomas Monjalon
2016-10-25 01:41, Yuanhan Liu: > On Mon, Oct 24, 2016 at 12:22:21PM -0400, Jan Blunck wrote: > > This adds infrastructure for drivers to allow being requested by an alias > > so that a renamed driver can still get loaded by its legacy name. > > > > Signed-off-by: Jan Blunck > > Reviewed-by: Maxim

[dpdk-dev] [PATCH v5 06/21] eal/soc: introduce very essential SoC infra definitions

2016-10-25 Thread Shreyansh Jain
On Tuesday 25 October 2016 11:06 AM, Shreyansh Jain wrote: > Hello Jan, > > On Monday 24 October 2016 09:51 PM, Jan Viktorin wrote: >> On Mon, 24 Oct 2016 17:29:25 +0530 >> Shreyansh Jain wrote: >> >>> From: Jan Viktorin >>> >>> Define initial structures and functions for the SoC infrastructure.

[dpdk-dev] [PATCH 2/2] examples/tep_term: Fix packet len for multi-seg mbuf

2016-10-25 Thread Thomas Monjalon
> > For multi-seg mbuf, ip->total_length should be pkt_len subtract > > ether len. > > > > Fixes: 4abe471ed6fc("examples/tep_term: implement VXLAN processing") > > > > Signed-off-by: Michael Qiu > > Acked-by: Jianfeng Tan Applied, thanks

[dpdk-dev] [PATCH v6 2/2] app/test_pmd: fix DCB configuration

2016-10-25 Thread Bernard Iremonger
Data Centre Bridge (DCB) configuration fails when SRIOV is enabled if nb_rxq and nb_txq are not set to 1. When dcb_mode is DCB_VT_ENABLED set nb_rxq and nb_txq to 1. The failure occurs during configuration of the ixgbe PMD when it is started, in the ixgbe_check_mq_mode function, if nb_rxq and nb_

[dpdk-dev] [PATCH v6 1/2] net/ixgbe: support multiqueue mode VMDq DCB with SRIOV

2016-10-25 Thread Bernard Iremonger
The folowing changes have been made to allow Data Centre Bridge (DCB) configuration when SRIOV is enabled. Modify ixgbe_check_mq_mode function, when SRIOV is enabled, enable mq_mode ETH_MQ_RX_VMDQ_DCB and ETH_MQ_TX_VMDQ_DCB. Modify ixgbe_dcb_tx_hw_config function, replace the struct ixgbe_hw para

[dpdk-dev] [PATCH v6 0/2] net/ixgbe: VMDq DCB with SRIOV

2016-10-25 Thread Bernard Iremonger
Changes in v6: rebase to latest master. revise commit messages. Changes in v5: fix enable/disable of the QDE bit in the PFQDE register. Changes in v4: changes to ixgbe patch following comments. Changes in v3: rebase to latest master. update commit message for ixgbe patch add testpmd patch. Chan

[dpdk-dev] [PATCH] pdump: revert PCI device name conversion

2016-10-25 Thread Reshma Pattan
Earlier ethdev library created the device names in the "bus:device.func" format hence pdump library implemented its own conversion method for changing the user passed device name format "domain:bus:device.func" to "bus:device.func" for finding the port id using device name using ethdev library call

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

2016-10-25 Thread Kulasek, TomaszX
> -Original Message- > From: Olivier Matz [mailto:olivier.matz at 6wind.com] > Sent: Tuesday, October 25, 2016 16:42 > To: Kulasek, TomaszX ; dev at dpdk.org > Cc: Ananyev, Konstantin > Subject: Re: [PATCH v10 1/6] ethdev: add Tx preparation > > Hi Tomasz, > > On 10/24/2016 06:51 PM, T

[dpdk-dev] mbuf changes

2016-10-25 Thread Shreyansh Jain
On Monday 24 October 2016 09:55 PM, Bruce Richardson wrote: > On Mon, Oct 24, 2016 at 04:11:33PM +, Wiles, Keith wrote: >> >>> On Oct 24, 2016, at 10:49 AM, Morten Br?rup >>> wrote: >>> >>> First of all: Thanks for a great DPDK Userspace 2016! >>> >>> >>> >>> Continuing the Userspace discussi

[dpdk-dev] DPDK-QoS- Using un-used bandwidth within a class

2016-10-25 Thread Dumitrescu, Cristian
Hi Sreenaath, I think you are simply hitting the known and documented performance issue with using a single pipe. The hierarchical scheduler performance is optimized for many pipes (hundreds, thousands , ?), not for single pipe. The rationale is that if you only needs handful of queues, you can

[dpdk-dev] [PATCH v2] mempool: fix search of maximum contiguous pages

2016-10-25 Thread Olivier Matz
From: Wei Dai paddr[i] + pg_sz always points to the start physical address of the 2nd page after pddr[i], so only up to 2 pages can be combinded to be used. With this revision, more than 2 pages can be used. Fixes: 84121f197187 ("mempool: store memory chunks in a list") Signed-off-by: Wei Dai

[dpdk-dev] [PATCH] mempool: fix search of maximum contiguous pages

2016-10-25 Thread Olivier Matz
Hi Thomas, On 10/25/2016 04:37 PM, Thomas Monjalon wrote: > 2016-10-13 17:05, Olivier MATZ: >> Hi Wei, >> >> On 10/13/2016 02:31 PM, Ananyev, Konstantin wrote: >>> >>> diff --git a/lib/librte_mempool/rte_mempool.c >>> b/lib/librte_mempool/rte_mempool.c >>> index 71017e1..e3e254a 1

[dpdk-dev] mbuf changes

2016-10-25 Thread Thomas Monjalon
2016-10-25 14:32, Ramia, Kannan Babu: > I didn't get your question. The only information exchanged between the stages > is mbuf pointer. So the information has to be in mbuf, whether it's part of > Meta or in private area in the packet buffer headroom is that the question > you are asking. The p

[dpdk-dev] mbuf changes

2016-10-25 Thread Olivier Matz
On 10/25/2016 04:25 PM, Morten Br?rup wrote: > It might also make sense documenting the mbuf fields in more detail > somewhere. E.g. the need for nb_segs in the NIC's TX handler. Good point, I'll do it at the same time than the first rework proposition.

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

2016-10-25 Thread Olivier Matz
Hi Tomasz, On 10/24/2016 06:51 PM, Tomasz Kulasek wrote: > Added API for `rte_eth_tx_prep` > > [...] > --- a/lib/librte_ether/rte_ethdev.h > +++ b/lib/librte_ether/rte_ethdev.h > @@ -182,6 +182,7 @@ extern "C" { > #include > #include > #include > +#include > #include "rte_ether.h" > #inc

[dpdk-dev] [PATCH] mempool: fix search of maximum contiguous pages

2016-10-25 Thread Thomas Monjalon
2016-10-13 17:05, Olivier MATZ: > Hi Wei, > > On 10/13/2016 02:31 PM, Ananyev, Konstantin wrote: > > > >> > > diff --git a/lib/librte_mempool/rte_mempool.c > > b/lib/librte_mempool/rte_mempool.c > > index 71017e1..e3e254a 100644 > > --- a/lib/librte_mempool/rte_mempool.c > > ++

[dpdk-dev] [PATCH] examples/ip_pipeline: fix gcc v6.2.1 build error

2016-10-25 Thread Thomas Monjalon
> > This patch fixes the misleading indentation error on compiling ip_pipeline > > app with gcc v6.2.1. > > > > Fixes: 3f2c9f3bb6c6 ("examples/ip_pipeline: add TAP port") > > > > Signed-off-by: Jasvinder Singh > > --- > > examples/ip_pipeline/app.h | 5 +++-- > > 1 file changed, 3 insertions(+)

[dpdk-dev] [PATCH] examples/ip_pipeline: fix freeBSD build error

2016-10-25 Thread Thomas Monjalon
> > Error log: > > CC init.o > > examples/ip_pipeline/init.c:38:22: fatal error: linux/if.h: No such file or > > directory > > #include > > ^ > > Fixes: 3f2c9f3bb6c6 ("examples/ip_pipeline: add TAP port") > > > > Signed-off-by: Jasvinder Singh > > Acked-by: Cristian Dum

[dpdk-dev] mbuf changes

2016-10-25 Thread Morten Brørup
Lots of good arguments from Konstantin below! Thomas also wrote something worth repeating: "The mbuf space must be kept jealously like a DPDK treasure :)" Maybe we should take a step away from the keyboard and consider the decision process and guidance criteria for mbuf members. Here is an exam

[dpdk-dev] [PATCH] kni: fix build with kernel 4.9

2016-10-25 Thread Thomas Monjalon
> > compile error: > > CC [M] .../lib/librte_eal/linuxapp/kni/igb_main.o > > .../lib/librte_eal/linuxapp/kni/igb_main.c:2317:21: > > error: initialization from incompatible pointer type > > [-Werror=incompatible-pointer-types] > > .ndo_set_vf_vlan = igb_ndo_set_vf_vlan, > >

[dpdk-dev] [PATCH v10 11/25] eal/pci: helpers for device name parsing/update

2016-10-25 Thread Pattan, Reshma
Hi Shreyansh, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Shreyansh Jain > Sent: Friday, September 16, 2016 5:30 AM > To: dev at dpdk.org > Cc: viktorin at rehivetech.com; David Marchand ; > hemant.agrawal at nxp.com; Thomas Monjalon > ; Shreyansh Jain >

[dpdk-dev] mbuf changes

2016-10-25 Thread Adrien Mazarguil
On Tue, Oct 25, 2016 at 02:16:29PM +0200, Morten Br?rup wrote: > Comments inline. I'm only replying to the nb_segs bits here. > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Bruce Richardson > > Sent: Tuesday, October 25, 2016 1:14 PM > > To: Adrien Maza

[dpdk-dev] mbuf changes

2016-10-25 Thread Thomas Monjalon
2016-10-25 13:04, Ramia, Kannan Babu: > Port filed is important meta information for the application use like > CGNAT vEPC functions etc. > I strongly recommend to keep the field in mind meta. Have you tried to move this field outside of the mbuf? What is the performance degradation? We need more

[dpdk-dev] mbuf changes

2016-10-25 Thread Thomas Monjalon
2016-10-25 15:00, Olivier Matz: > On 10/25/2016 12:22 PM, Morten Br?rup wrote: > > From: Ananyev, Konstantin > >> From: Bruce Richardson > >>> On Mon, Oct 24, 2016 at 11:47:16PM +0200, Morten Br?rup wrote: > From: Bruce Richardson > > On Mon, Oct 24, 2016 at 04:11:33PM +, Wiles, Keith

[dpdk-dev] mbuf changes

2016-10-25 Thread Morten Brørup
I support that! Med venlig hilsen / kind regards - Morten Br?rup > -Original Message- > From: Olivier Matz [mailto:olivier.matz at 6wind.com] > Sent: Tuesday, October 25, 2016 3:14 PM > To: Morten Br?rup; dev at dpdk.org > Subject: Re: mbuf changes > > Hi, > > On 10/24/2016 05:49 PM, M

[dpdk-dev] DPDK & ASLR

2016-10-25 Thread Samir Shah
We know that it is recommended that ASLR be turned off so that the mappings across primary and secondary remain the same. Does ASLR need to be turned off system-wide, or DPDK-processes wide? Could we use setarch/personality to disable ASLR for just the DPDK process and leave it enabled for the res

[dpdk-dev] mbuf changes

2016-10-25 Thread Olivier Matz
Hi, On 10/24/2016 05:49 PM, Morten Br?rup wrote: > And here?s something new to think about: > > m->next already reveals if there are more segments to a packet. Which > purpose does m->nb_segs serve that is not already covered by m->next? > I was asking myself the same question some time ago: ht

[dpdk-dev] mbuf changes

2016-10-25 Thread Morten Brørup
Comments inline. Med venlig hilsen / kind regards - Morten Br?rup > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier Matz > Sent: Tuesday, October 25, 2016 2:49 PM > To: Bruce Richardson; Morten Br?rup > Cc: Adrien Mazarguil; Wiles, Keith; dev at dpdk.

[dpdk-dev] rte_eth_tx_burst() hangs because of nofreedescriptors

2016-10-25 Thread yingzhi
Hi Helin, I'm doing comparison tests today, (with fragmented and small traffic). On the other hand, I checked some example code from DPDK source: load_balancer: app_lcore_io_tx: n_pkts = rte_eth_tx_burst( port,

[dpdk-dev] [PATCH] Revert "bonding: use existing enslaved device queues"

2016-10-25 Thread Bruce Richardson
On Tue, Oct 25, 2016 at 02:48:04PM +0100, Declan Doherty wrote: > On 25/10/16 13:57, Bruce Richardson wrote: > > On Mon, Oct 24, 2016 at 04:07:17PM +0100, Declan Doherty wrote: > > > On 24/10/16 15:51, Jan Blunck wrote: > > > > On Mon, Oct 24, 2016 at 7:02 AM, Declan Doherty > > > > wrote: > > > >

[dpdk-dev] mbuf changes

2016-10-25 Thread Olivier Matz
Hi, On 10/25/2016 12:22 PM, Morten Br?rup wrote: > Comments inline (at the end). > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ananyev, >> Konstantin >> Sent: Tuesday, October 25, 2016 12:03 PM >> To: Richardson, Bruce; Morten Br?rup >> Cc: Wiles, Keit

[dpdk-dev] [PATCH] Revert "bonding: use existing enslaved device queues"

2016-10-25 Thread Bruce Richardson
On Wed, Oct 19, 2016 at 10:55:25AM +0100, Bruce Richardson wrote: > On Thu, Oct 06, 2016 at 03:32:36PM +0100, Declan Doherty wrote: > > On 07/09/16 13:28, Ilya Maximets wrote: > > > This reverts commit 5b7bb2bda5519b7800f814df64d4e015282140e5. > > > > > > It is necessary to reconfigure all queues

[dpdk-dev] [PATCH] pci: Don't call probe callback if driver already loaded.

2016-10-25 Thread Ben Walker
If the user asks to probe multiple times, the probe callback should only be called on devices that don't have a driver already loaded. This is useful if a driver is registered after the execution of a program has started and the list of devices needs to be re-scanned. Signed-off-by: Ben Walker -

[dpdk-dev] mbuf changes

2016-10-25 Thread Morten Brørup
Comments inline. Med venlig hilsen / kind regards - Morten Br?rup > -Original Message- > From: Shreyansh Jain [mailto:shreyansh.jain at nxp.com] > Sent: Tuesday, October 25, 2016 1:54 PM > To: Bruce Richardson > Cc: Wiles, Keith; Morten Br?rup; dev at dpdk.org; Olivier Matz > Subject: Re

[dpdk-dev] mbuf changes

2016-10-25 Thread Olivier Matz
On 10/25/2016 02:45 PM, Bruce Richardson wrote: > On Tue, Oct 25, 2016 at 02:33:55PM +0200, Morten Br?rup wrote: >> Comments at the end. >> >> Med venlig hilsen / kind regards >> - Morten Br?rup >> >>> -Original Message- >>> From: Bruce Richardson [mailto:bruce.richardson at intel.com] >>

[dpdk-dev] [PATCH] Revert "bonding: use existing enslaved device queues"

2016-10-25 Thread Declan Doherty
On 25/10/16 13:57, Bruce Richardson wrote: > On Mon, Oct 24, 2016 at 04:07:17PM +0100, Declan Doherty wrote: >> On 24/10/16 15:51, Jan Blunck wrote: >>> On Mon, Oct 24, 2016 at 7:02 AM, Declan Doherty >>> wrote: On 14/10/16 00:37, Eric Kinzie wrote: > > On Wed Oct 12 16:24:21 +0100 20

[dpdk-dev] [PATCH 0/2] crypto/qat: performance optimisation

2016-10-25 Thread De Lara Guarch, Pablo
> -Original Message- > From: Trahe, Fiona > Sent: Monday, October 24, 2016 5:00 AM > To: dev at dpdk.org > Cc: De Lara Guarch, Pablo; Trahe, Fiona; Griffin, John; Jain, Deepak K; > Kusztal, > ArkadiuszX > Subject: [PATCH 0/2] crypto/qat: performance optimisation > > QAT PMD adjusts the

[dpdk-dev] mbuf changes

2016-10-25 Thread Morten Brørup
Comments at the end. Med venlig hilsen / kind regards - Morten Br?rup > -Original Message- > From: Bruce Richardson [mailto:bruce.richardson at intel.com] > Sent: Tuesday, October 25, 2016 2:20 PM > To: Morten Br?rup > Cc: Adrien Mazarguil; Wiles, Keith; dev at dpdk.org; Olivier Matz; Ole

[dpdk-dev] mbuf changes

2016-10-25 Thread Ramia, Kannan Babu
I didn't get your question. The only information exchanged between the stages is mbuf pointer. So the information has to be in mbuf, whether it's part of Meta or in private area in the packet buffer headroom is that the question you are asking. The private area is application specific, while I a

[dpdk-dev] mbuf changes

2016-10-25 Thread Morten Brørup
Comments inline. > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Bruce Richardson > Sent: Tuesday, October 25, 2016 1:14 PM > To: Adrien Mazarguil > Cc: Morten Br?rup; Wiles, Keith; dev at dpdk.org; Olivier Matz; Oleg > Kuporosov > Subject: Re: [dpdk-dev] mbu

[dpdk-dev] mbuf changes

2016-10-25 Thread Bruce Richardson
On Tue, Oct 25, 2016 at 03:00:39PM +0200, Olivier Matz wrote: > Hi, > > On 10/25/2016 12:22 PM, Morten Br?rup wrote: > > Comments inline (at the end). > > > >> -Original Message- > >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ananyev, > >> Konstantin > >> Sent: Tuesday, Octo

[dpdk-dev] mbuf changes

2016-10-25 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Adrien Mazarguil > Sent: Tuesday, October 25, 2016 2:48 PM > To: Morten Br?rup > Cc: Richardson, Bruce ; Wiles, Keith > ; dev at dpdk.org; Olivier Matz > ; Oleg Kuporosov > Subject: Re: [dpdk-dev] mbuf chang

[dpdk-dev] [PATCH] Revert "bonding: use existing enslaved device queues"

2016-10-25 Thread Bruce Richardson
On Mon, Oct 24, 2016 at 04:07:17PM +0100, Declan Doherty wrote: > On 24/10/16 15:51, Jan Blunck wrote: > > On Mon, Oct 24, 2016 at 7:02 AM, Declan Doherty > > wrote: > > > On 14/10/16 00:37, Eric Kinzie wrote: > > > > > > > > On Wed Oct 12 16:24:21 +0100 2016, Bruce Richardson wrote: > > > > > >

[dpdk-dev] [PATCH v2 1/2] net/i40e: fix link status change interrupt

2016-10-25 Thread Bruce Richardson
On Tue, Oct 25, 2016 at 12:19:05PM +0800, Qiming Yang wrote: > Previously, link status interrupt in i40e is achieved by checking > LINK_STAT_CHANGE_MASK in PFINT_ICR0 register which is provided only > for diagnostic use. Instead, drivers need to get the link status > change notification by using LS

[dpdk-dev] mbuf changes

2016-10-25 Thread Bruce Richardson
On Tue, Oct 25, 2016 at 02:33:55PM +0200, Morten Br?rup wrote: > Comments at the end. > > Med venlig hilsen / kind regards > - Morten Br?rup > > > -Original Message- > > From: Bruce Richardson [mailto:bruce.richardson at intel.com] > > Sent: Tuesday, October 25, 2016 2:20 PM > > To: Morte

[dpdk-dev] mbuf changes

2016-10-25 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier Matz > Sent: Tuesday, October 25, 2016 1:49 PM > To: Richardson, Bruce ; Morten Br?rup smartsharesystems.com> > Cc: Adrien Mazarguil ; Wiles, Keith > ; dev at dpdk.org; Oleg Kuporosov > > Subject: Re

[dpdk-dev] [PATCH v4] net/i40e: fix hash filter invalid issue in X722

2016-10-25 Thread Bruce Richardson
On Tue, Oct 25, 2016 at 10:22:09AM +, Wu, Jingjing wrote: > > > > -Original Message- > > From: Guo, Jia > > Sent: Tuesday, October 25, 2016 10:43 AM > > To: Zhang, Helin ; Wu, Jingjing > intel.com> > > Cc: dev at dpdk.org; Guo, Jia > > Subject: [PATCH v4] net/i40e: fix hash filter i

[dpdk-dev] [PATCH] doc: release note for ixgbe PMD API's

2016-10-25 Thread Bernard Iremonger
Signed-off-by: Bernard Iremonger --- doc/guides/rel_notes/release_16_11.rst | 4 1 file changed, 4 insertions(+) diff --git a/doc/guides/rel_notes/release_16_11.rst b/doc/guides/rel_notes/release_16_11.rst index 26cdd62..8253614 100644 --- a/doc/guides/rel_notes/release_16_11.rst +++ b/doc

[dpdk-dev] [PATCH v2] net/i40e: fix fdir configure failed issue in X710

2016-10-25 Thread Bruce Richardson
On Tue, Oct 25, 2016 at 10:28:22AM +, Wu, Jingjing wrote: > > > > -Original Message- > > From: Guo, Jia > > Sent: Tuesday, October 25, 2016 10:26 AM > > To: Zhang, Helin ; Wu, Jingjing > intel.com> > > Cc: dev at dpdk.org; Guo, Jia > > Subject: [PATCH v2] net/i40e: fix fdir configur

[dpdk-dev] mbuf changes

2016-10-25 Thread Bruce Richardson
On Tue, Oct 25, 2016 at 02:16:29PM +0200, Morten Br?rup wrote: > Comments inline. > > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Bruce Richardson > > Sent: Tuesday, October 25, 2016 1:14 PM > > To: Adrien Mazarguil > > Cc: Morten Br?rup; Wiles, Keith;

[dpdk-dev] mbuf changes

2016-10-25 Thread Bruce Richardson
On Tue, Oct 25, 2016 at 05:24:28PM +0530, Shreyansh Jain wrote: > On Monday 24 October 2016 09:55 PM, Bruce Richardson wrote: > > On Mon, Oct 24, 2016 at 04:11:33PM +, Wiles, Keith wrote: > > > > > > > On Oct 24, 2016, at 10:49 AM, Morten Br?rup > > > smartsharesystems.com> wrote: > > > > >

[dpdk-dev] mbuf changes

2016-10-25 Thread Ramia, Kannan Babu
Port filed is important meta information for the application use like CGNAT vEPC functions etc. I strongly recommend to keep the field in mind meta. Sent from my ASUS Original Message From:Olivier Matz Sent:Tue, 25 Oct 2016 18:31:36 +0530 To:Morten Br?rup ,"Ananyev, Konstantin"

[dpdk-dev] mbuf changes

2016-10-25 Thread Adrien Mazarguil
On Tue, Oct 25, 2016 at 12:11:04PM +0200, Morten Br?rup wrote: > Comments inline. > > Med venlig hilsen / kind regards > - Morten Br?rup > > > > -Original Message- > > From: Adrien Mazarguil [mailto:adrien.mazarguil at 6wind.com] > > Sent: Tuesday, October 25, 2016 11:39 AM > > To: Bruce

[dpdk-dev] [PATCH 1/3] mbuf: embedding timestamp into the packet

2016-10-25 Thread Pattan, Reshma
Hi, > -Original Message- > From: Oleg Kuporosov [mailto:olegk at mellanox.com] > Sent: Wednesday, October 19, 2016 6:40 PM > To: Pattan, Reshma ; Olivier Matz > > Cc: Richardson, Bruce ; Ananyev, Konstantin > ; dev at dpdk.org > Subject: RE: [dpdk-dev] [PATCH 1/3] mbuf: embedding timestam

[dpdk-dev] mbuf changes

2016-10-25 Thread Morten Brørup
Comments inline (at the end). > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ananyev, > Konstantin > Sent: Tuesday, October 25, 2016 12:03 PM > To: Richardson, Bruce; Morten Br?rup > Cc: Wiles, Keith; dev at dpdk.org; Olivier Matz > Subject: Re: [dpdk-dev] m

[dpdk-dev] [PATCH v2 2/2] net/i40e: fix VF bonded device link down

2016-10-25 Thread Qiming Yang
If VF device is used as slave of a bond device, it will be polled periodically through alarm. Interrupt is involved here. And then VF will send I40E_VIRTCHNL_OP_GET_LINK_STAT message to PF to query the status. The response is handled by interrupt callback. Interrupt is involved here again. That's w

[dpdk-dev] [PATCH v2 1/2] net/i40e: fix link status change interrupt

2016-10-25 Thread Qiming Yang
Previously, link status interrupt in i40e is achieved by checking LINK_STAT_CHANGE_MASK in PFINT_ICR0 register which is provided only for diagnostic use. Instead, drivers need to get the link status change notification by using LSE (Link Status Event). This patch enables LSE and calls LSC callback

[dpdk-dev] mbuf changes

2016-10-25 Thread Bruce Richardson
On Tue, Oct 25, 2016 at 01:04:44PM +0200, Adrien Mazarguil wrote: > On Tue, Oct 25, 2016 at 12:11:04PM +0200, Morten Br?rup wrote: > > Comments inline. > > > > Med venlig hilsen / kind regards > > - Morten Br?rup > > > > > > > -Original Message- > > > From: Adrien Mazarguil [mailto:adrie

[dpdk-dev] mbuf changes

2016-10-25 Thread Morten Brørup
Comments inline. Med venlig hilsen / kind regards - Morten Br?rup > -Original Message- > From: Adrien Mazarguil [mailto:adrien.mazarguil at 6wind.com] > Sent: Tuesday, October 25, 2016 11:39 AM > To: Bruce Richardson > Cc: Wiles, Keith; Morten Br?rup; dev at dpdk.org; Olivier Matz; Oleg

[dpdk-dev] mbuf changes

2016-10-25 Thread Adrien Mazarguil
On Mon, Oct 24, 2016 at 05:25:38PM +0100, Bruce Richardson wrote: > On Mon, Oct 24, 2016 at 04:11:33PM +, Wiles, Keith wrote: [...] > > > On Oct 24, 2016, at 10:49 AM, Morten Br?rup > > > wrote: [...] > > > 5. > > > > > > And here?s something new to think about: > > > > > > m->next already

[dpdk-dev] [PATCH v5 06/21] eal/soc: introduce very essential SoC infra definitions

2016-10-25 Thread Shreyansh Jain
Hello Jan, On Monday 24 October 2016 09:51 PM, Jan Viktorin wrote: > On Mon, 24 Oct 2016 17:29:25 +0530 > Shreyansh Jain wrote: > >> From: Jan Viktorin >> >> Define initial structures and functions for the SoC infrastructure. >> This patch supports only a very minimal functions for now. >> More

[dpdk-dev] [PATCH v2 2/2] net/i40e: fix VF bonded device link down

2016-10-25 Thread Wu, Jingjing
> -Original Message- > From: Yang, Qiming > Sent: Tuesday, October 25, 2016 12:19 PM > To: dev at dpdk.org > Cc: Wu, Jingjing ; Zhang, Helin intel.com>; Richardson, > Bruce ; Yang, Qiming > Subject: [PATCH v2 2/2] net/i40e: fix VF bonded device link down > > If VF device is used as sla

[dpdk-dev] [PATCH v2 1/2] net/i40e: fix link status change interrupt

2016-10-25 Thread Wu, Jingjing
> -Original Message- > From: Yang, Qiming > Sent: Tuesday, October 25, 2016 12:19 PM > To: dev at dpdk.org > Cc: Wu, Jingjing ; Zhang, Helin intel.com>; Richardson, > Bruce ; Yang, Qiming > Subject: [PATCH v2 1/2] net/i40e: fix link status change interrupt > > Previously, link status i

[dpdk-dev] [PATCH v2] net/i40e: fix fdir configure failed issue in X710

2016-10-25 Thread Wu, Jingjing
> -Original Message- > From: Guo, Jia > Sent: Tuesday, October 25, 2016 10:26 AM > To: Zhang, Helin ; Wu, Jingjing intel.com> > Cc: dev at dpdk.org; Guo, Jia > Subject: [PATCH v2] net/i40e: fix fdir configure failed issue in X710 > > Because of some register is only supported by X722,

[dpdk-dev] [PATCH v4] net/i40e: fix hash filter invalid issue in X722

2016-10-25 Thread Wu, Jingjing
> -Original Message- > From: Guo, Jia > Sent: Tuesday, October 25, 2016 10:43 AM > To: Zhang, Helin ; Wu, Jingjing intel.com> > Cc: dev at dpdk.org; Guo, Jia > Subject: [PATCH v4] net/i40e: fix hash filter invalid issue in X722 > > When verifying the Hash filtering on X722, we found a

[dpdk-dev] mbuf changes

2016-10-25 Thread Ananyev, Konstantin
Hi everyone, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Bruce Richardson > Sent: Tuesday, October 25, 2016 9:54 AM > To: Morten Br?rup > Cc: Wiles, Keith ; dev at dpdk.org; Olivier Matz > > Subject: Re: [dpdk-dev] mbuf changes > > On Mon, Oct 24, 201

[dpdk-dev] mbuf changes

2016-10-25 Thread Bruce Richardson
On Mon, Oct 24, 2016 at 11:47:16PM +0200, Morten Br?rup wrote: > Comments inline. > > Med venlig hilsen / kind regards > - Morten Br?rup > > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Bruce Richardson > > Sent: Monday, October 24, 2016 6:26 PM > > To:

[dpdk-dev] [PATCH] Revert "bonding: use existing enslaved device queues"

2016-10-25 Thread Ilya Maximets
On 24.10.2016 17:54, Jan Blunck wrote: > On Wed, Oct 19, 2016 at 5:47 AM, Ilya Maximets > wrote: >> On 18.10.2016 18:19, Jan Blunck wrote: >>> On Tue, Oct 18, 2016 at 2:49 PM, Ilya Maximets >>> wrote: On 18.10.2016 15:28, Jan Blunck wrote: > If the application already configured queues

[dpdk-dev] rte_kni_tx_burst() hangs because of no freedescriptors

2016-10-25 Thread Ananyev, Konstantin
Hi Helin, > > From: yingzhi [mailto:kaitoy at qq.com] > Sent: Monday, October 24, 2016 6:39 PM > To: Zhang, Helin > Cc: dev at dpdk.org > Subject: Re: RE: [dpdk-dev] rte_kni_tx_burst() hangs because of no > freedescriptors > > Hi Helin, > > Thanks for your response, to answer your questions: >

[dpdk-dev] 16.07.1 stable patches review and test

2016-10-25 Thread Xu, Qian Q
Yuanhan So we will get the package for testing on Oct.26th, right? Thx. -Original Message- From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Yuanhan Liu Sent: Friday, October 14, 2016 5:27 PM To: dpdk stable Cc: dev at dpdk.org Subject: [dpdk-dev] 16.07.1 stable patches review and tes

[dpdk-dev] [PATCH] doc: announce ABI change for ethtool app enhance

2016-10-25 Thread Xing, Beilei
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Qiming Yang > Sent: Sunday, October 9, 2016 11:17 AM > To: dev at dpdk.org > Cc: Yang, Qiming > Subject: [dpdk-dev] [PATCH] doc: announce ABI change for ethtool app > enhance > > This patch adds a notice that

[dpdk-dev] [PATCH] net/i40e: fix the hash filter invalid calculation in X722

2016-10-25 Thread Guo, Jia
I will refine the commit log. Other hand, since the issue is not directly related with prior patch, it just because some nic type adding request some special behavior handle. So it is reported issue fix, but may be have not corresponding fix line. Thanks jingjing's review. Best regards, Jeff

[dpdk-dev] [PATCH v2 1/3] drivers: add name alias registration for rte_driver

2016-10-25 Thread Yuanhan Liu
On Mon, Oct 24, 2016 at 12:22:21PM -0400, Jan Blunck wrote: > This adds infrastructure for drivers to allow being requested by an alias > so that a renamed driver can still get loaded by its legacy name. > > Signed-off-by: Jan Blunck > Reviewed-by: Maxime Coquelin > Tested-by: Pablo de Lara Se

[dpdk-dev] rte_kni_tx_burst() hangs because of no freedescriptors

2016-10-25 Thread Zhang, Helin
From: yingzhi [mailto:kai...@qq.com] Sent: Monday, October 24, 2016 6:39 PM To: Zhang, Helin Cc: dev at dpdk.org Subject: Re: RE: [dpdk-dev] rte_kni_tx_burst() hangs because of no freedescriptors Hi Helin, Thanks for your response, to answer your questions: 1. we send only one packet each time

[dpdk-dev] [PATCH v5 0/2] net/ixgbe: VMDq DCB with SRIOV

2016-10-25 Thread Lu, Wenzhuo
Hi, > -Original Message- > From: Iremonger, Bernard > Sent: Wednesday, October 19, 2016 6:21 PM > To: dev at dpdk.org; Shah, Rahul R; Lu, Wenzhuo > Cc: Iremonger, Bernard > Subject: [PATCH v5 0/2] net/ixgbe: VMDq DCB with SRIOV > > Changes in v5: > fix enable/disable of the QDE bit in th