[dpdk-dev] [PATCH v4 07/32] net/qede: fix 32 bit compilation

2016-10-26 Thread Thomas Monjalon
2016-10-18 21:11, Rasesh Mody: > Fix 32 bit compilation for gcc version 4.3.4. > > Fixes: ec94dbc57362 ("qede: add base driver") > > Signed-off-by: Rasesh Mody [...] > ifeq ($(CONFIG_RTE_TOOLCHAIN_GCC),y) > +ifeq ($(shell gcc -Wno-unused-but-set-variable -Werror -E - < /dev/null > > /dev/null

[dpdk-dev] [PATCH v4 18/32] net/qede: add missing 100G link speed capability

2016-10-26 Thread Thomas Monjalon
2016-10-18 21:11, Rasesh Mody: > From: Harish Patil > > This patch fixes the missing 100G link speed advertisement > when the 100G support was initially added. > > Fixes: 2af14ca79c0a ("net/qede: support 100G") > > Signed-off-by: Harish Patil [...] > [Features] > +Speed capabilities = Y

[dpdk-dev] [PATCH v2] release 16.07.1

2016-10-26 Thread Thomas Monjalon
From: Yuanhan Liu <yuanhan@linux.intel.com> A quick download button is added for the latest stable release. Signed-off-by: Yuanhan Liu Signed-off-by: Thomas Monjalon --- v2: - buttons on one row - md5 of tarball --- content.css | 4 ++-- download.html | 10 +++--- re

[dpdk-dev] [PATCH v7 3/7] vhost: simplify mergeable Rx vring reservation

2016-10-26 Thread Thomas Monjalon
2016-10-14 17:34, Yuanhan Liu: > -static inline uint32_t __attribute__((always_inline)) > +static inline int __attribute__((always_inline)) > copy_mbuf_to_desc_mergeable(struct virtio_net *dev, struct vhost_virtqueue > *vq, > - uint16_t end_idx, struct rte_mbuf *m, > -

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

2016-10-26 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

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

2016-10-26 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

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

2016-10-26 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

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

2016-10-26 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

[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:

[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] mbuf changes

2016-10-25 Thread Thomas Monjalon
first cache lines. > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Tuesday, October 25, 2016 6:54 PM > To: Ramia, Kannan Babu > Cc: dev at dpdk.org; Olivier Matz ; Morten Br?rup > ; Ananyev, Konstantin intel.com>; Richardson, Bruce ; Wi

[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

[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

[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] 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] Project Governance and Linux Foundation

2016-10-22 Thread Thomas Monjalon
t > commit to funding. > > Jerome Tollet said that he felt it was important to have better test and > CI infrastructure, and that these cost money. He proposed that since > fd.io already has infrastructure and a lab, that this would be an > affordable option for doing this. >

[dpdk-dev] [PATCH] drivers: make driver names consistent

2016-10-19 Thread Thomas Monjalon
2016-10-19 18:05, Yuanhan Liu: > On Tue, Oct 18, 2016 at 06:46:40PM +0200, Thomas Monjalon wrote: > > 2016-10-18 22:18, Yuanhan Liu: > > > On Tue, Oct 18, 2016 at 03:42:54PM +0200, Thomas Monjalon wrote: > > > > 2016-10-18 21:06, Yuanhan Liu: > > > > >

[dpdk-dev] Project Governance and Linux Foundation

2016-10-19 Thread Thomas Monjalon
2016-10-19 09:40, Dave Neary: > On 10/19/2016 09:04 AM, O'Driscoll, Tim wrote: > > From: Jerin Jacob [mailto:jerin.jacob at caviumnetworks.com] > > > On Tue, Oct 18, 2016 at 03:27:27PM +0200, Thomas Monjalon wrote: > > > > How can we solve issues if you don't give m

[dpdk-dev] [PATCH] drivers: make driver names consistent

2016-10-18 Thread Thomas Monjalon
2016-10-18 22:18, Yuanhan Liu: > On Tue, Oct 18, 2016 at 03:42:54PM +0200, Thomas Monjalon wrote: > > 2016-10-18 21:06, Yuanhan Liu: > > > On Tue, Oct 18, 2016 at 02:50:16PM +0200, Jan Blunck wrote: > > > > >From my understanding this is a massive API breakage.

[dpdk-dev] [PATCH] drivers: make driver names consistent

2016-10-18 Thread Thomas Monjalon
2016-10-18 17:26, Jan Blunck: > On Tue, Oct 18, 2016 at 3:42 PM, Thomas Monjalon > wrote: > > 2016-10-18 21:06, Yuanhan Liu: > >> On Tue, Oct 18, 2016 at 02:50:16PM +0200, Jan Blunck wrote: > >> > >From my understanding this is a massive API breakage. T

[dpdk-dev] [PATCH v11 00/24] Introducing rte_driver/rte_device generalization

2016-10-18 Thread Thomas Monjalon
2016-10-18 09:04, Neil Horman: > On Mon, Oct 17, 2016 at 02:43:12PM +0100, Ferruh Yigit wrote: > > I caught while running ABI validation script today, I think this patch > > should increase LIBABIVER of: > > - lib/librte_cryptodev > > - lib/librte_eal > > - lib/librte_ether > > > It should, and

[dpdk-dev] [PATCH] drivers: make driver names consistent

2016-10-18 Thread Thomas Monjalon
2016-10-18 21:06, Yuanhan Liu: > On Tue, Oct 18, 2016 at 02:50:16PM +0200, Jan Blunck wrote: > > >From my understanding this is a massive API breakage. This forces all > > existing users of the virtual PMDs to change with zero benefit. Even > > if that isn't enough it also makes it impossible to

[dpdk-dev] Project Governance and Linux Foundation

2016-10-18 Thread Thomas Monjalon
2016-10-18 17:04, Jerin Jacob: > On Mon, Oct 17, 2016 at 05:23:42PM -0400, Dave Neary wrote: > > > I still hear concerns on this, and based on discussions with others who > > > put their names to the post below, they do too. I think it's a perception > > > that we need to address. > > > > I

[dpdk-dev] [dpdk-users] Project Governance and Linux Foundation

2016-10-18 Thread Thomas Monjalon
2016-10-17 14:40, O'Driscoll, Tim: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > 2016-10-17 11:52, O'Driscoll, Tim: > > > From: Hobywan Kenoby > > > > The current DPDK version can run on virtually all processors (Intel, > > &g

[dpdk-dev] [dpdk-users] Project Governance and Linux Foundation

2016-10-18 Thread Thomas Monjalon
2016-10-18 12:16, Liu Yuan: > China Mobile would like to support moving the DPDK project to the Linux > Foundation. OK, there can be some advantages to join the Linux Foundation. > DPDK is a key technology / element in the NFV. As an end user, we can foresee > the importance of DPDK in the

[dpdk-dev] [dpdk-users] Project Governance and Linux Foundation

2016-10-17 Thread Thomas Monjalon
2016-10-17 11:52, O'Driscoll, Tim: > From: Hobywan Kenoby > > The current DPDK version can run on virtually all processors (Intel, IBM > > and ARM) and leverage all NICs: is there **really** anyone questionning > > openness of the community? > > I still hear concerns on this, and based on

[dpdk-dev] [PATCH v2] kni: fix unused variable compile error

2016-10-14 Thread Thomas Monjalon
2016-10-14 17:41, Ferruh Yigit: > compile error: > CC [M] .../lib/librte_eal/linuxapp/kni/kni_misc.o > cc1: warnings being treated as errors > .../lib/librte_eal/linuxapp/kni/kni_misc.c: In function ?kni_exit_net?: > .../lib/librte_eal/linuxapp/kni/kni_misc.c:113:18: > error: unused variable

[dpdk-dev] 17.02 Roadmap

2016-10-14 Thread Thomas Monjalon
2016-10-14 10:29, Stephen Hemminger: > It seems like a lot of these feature are focused too narrowly on exposing > features that exist on specific Intel hardware. The concept of a general > purpose Dataplane Development Kit is that applications can be written that > have a generic API (like any

[dpdk-dev] [PATCH v2 1/3] lib/librte_port: enable file descriptor port support

2016-10-14 Thread Thomas Monjalon
2016-10-12 20:44, Dumitrescu, Cristian: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > This patchset was probably not tested as it does not compile. > > And it could be useless if a TAP PMD is integrated. > > I suggest to wait 17.02 cycle and see. >

[dpdk-dev] [PATCH] kni: fix unused variable compile error

2016-10-14 Thread Thomas Monjalon
2016-10-14 12:24, Ferruh Yigit: > compile error: > CC [M] .../lib/librte_eal/linuxapp/kni/kni_misc.o > cc1: warnings being treated as errors > .../lib/librte_eal/linuxapp/kni/kni_misc.c: In function ?kni_exit_net?: > .../lib/librte_eal/linuxapp/kni/kni_misc.c:113:18: > error: unused variable

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

2016-10-14 Thread Thomas Monjalon
2016-10-14 14:02, Kulasek, TomaszX: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > 2016-10-13 19:36, Tomasz Kulasek: > > > +/** > > > + * Fix pseudo header checksum for TSO and non-TSO tcp/udp packets > > before > > > + * hardware

[dpdk-dev] [dpdk-announce] release candidate 16.11-rc1

2016-10-14 Thread Thomas Monjalon
A new DPDK release candidate is ready for testing: http://dpdk.org/browse/dpdk/tag/?id=v16.11-rc1 It is the first release candidate for DPDK 16.11. It happens a bit late, though there are still some features missing. This version must be released before mid-November. Therefore we have 3

[dpdk-dev] [PATCH v8 0/2] modify callback for VF management

2016-10-14 Thread Thomas Monjalon
2016-10-10 15:34, Bernard Iremonger: > This patchset modifies the callback function for VF management. > > A third parameter has been added to the _rte_eth_dev_callback_process > function. All references to this function have been updated. > Changes have been made to the ixgbe_rcv_msg_from_vf

[dpdk-dev] [PATCH v3] drivers: prefix driver REGISTER macro with RTE PMD

2016-10-14 Thread Thomas Monjalon
2016-10-10 11:13, Shreyansh Jain: > All macros related to driver registeration renamed from DRIVER_* > to RTE_PMD_* > > This includes: > > DRIVER_REGISTER_PCI -> RTE_PMD_REGISTER_PCI > DRIVER_REGISTER_PCI_TABLE -> RTE_PMD_REGISTER_PCI_TABLE > DRIVER_REGISTER_VDEV -> RTE_PMD_REGISTER_VDEV >

[dpdk-dev] [PATCH v2 0/5] implement new Rx checksum flag

2016-10-14 Thread Thomas Monjalon
> > Xiao Wang (5): > > net/fm10k: fix Rx checksum flags > > net/fm10k: implement new Rx checksum flag > > net/e1000: implement new Rx checksum flag > > net/ixgbe: implement new Rx checksum flag > > net/i40e: implement new Rx checksum flag > > Acked-by : Jing Chen Applied directly in

[dpdk-dev] ixgbe: support checksum flags in sse vector Rx function

2016-10-14 Thread Thomas Monjalon
2016-10-06 15:00, Remy Horton: > On 07/07/2016 13:19, Olivier Matz wrote: > [..] > > Signed-off-by: Maxime Leroy > > Signed-off-by: Olivier Matz > > --- > > drivers/net/ixgbe/ixgbe_rxtx_vec_common.h | 8 ++--- > > drivers/net/ixgbe/ixgbe_rxtx_vec_neon.c | 6 > >

[dpdk-dev] [PATCH v3 00/19] KNI checkpatch cleanup

2016-10-14 Thread Thomas Monjalon
2016-09-26 16:39, Ferruh Yigit: > KNI checkpatch cleanup, mostly non-functional but cosmetic modifications. > Only functional change is related logging, switched to kernel dynamic > logging and compile time KNI debug options removed, some log message > levels updated. Applied, thanks Note that

[dpdk-dev] [PATCH] kni: add support for core_id param in single threaded mode

2016-10-13 Thread Thomas Monjalon
> > Allow binding KNI thread to specific core in single threaded mode > > by setting core_id and force_bind config parameters. > > > > Signed-off-by: Vladyslav Buslov > > Acked-by: Ferruh Yigit > > Tested-by: Ferruh Yigit Applied, thanks

[dpdk-dev] [PATCH v4 2/3] app/test: add a case to verify lpm tlb8 recycle

2016-10-13 Thread Thomas Monjalon
2016-08-08 14:40, Wei Dai: > As a bug-fix for lpm tlb8 recycle is introduced, > add a test case to verify tlb8 group is correctly > freed when it only includes a rule with depth=24. > > Signed-off-by: Wei Dai > Acked-by: Bruce Richardson Series applied with typo tlb/tbl fixed

[dpdk-dev] [PATCH v4] log: respect rte_openlog_stream calls before rte_eal_init

2016-10-13 Thread Thomas Monjalon
2016-10-12 12:38, John Ousterhout: > Before this patch, application-specific loggers could not be > installed before rte_eal_init completed (the initialization process > called rte_openlog_stream, overwriting any previously installed > logger). This made it impossible for an application to capture

[dpdk-dev] [PATCH] fix documentation error on debug functions

2016-10-13 Thread Thomas Monjalon
2016-09-26 12:28, Mcnamara, John: > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Mauricio Vasquez B > > Sent: Friday, September 2, 2016 12:02 PM > > To: dev at dpdk.org > > Cc: stephen at networkplumber.org > > Subject: [dpdk-dev] [PATCH] fix

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

2016-10-13 Thread Thomas Monjalon
Hi, 2016-10-13 19:36, Tomasz Kulasek: > Added API for `rte_eth_tx_prep` > > uint16_t rte_eth_tx_prep(uint8_t port_id, uint16_t queue_id, > struct rte_mbuf **tx_pkts, uint16_t nb_pkts) > > Added fields to the `struct rte_eth_desc_lim`: > > uint16_t nb_seg_max; > /**<

[dpdk-dev] [PATCH v3 12/12] net/virtio: add Tso support

2016-10-13 Thread Thomas Monjalon
2016-10-14 00:05, Yuanhan Liu: > On Thu, Oct 13, 2016 at 04:16:11PM +0200, Olivier Matz wrote: > > +/* When doing TSO, the IP length is not included in the pseudo header > > + * checksum of the packet given to the PMD, but for virtio it is > > + * expected. > > + */ > > +static void > >

[dpdk-dev] [PATCH] eal: avoid unnecessary conflicts over rte_config file

2016-10-13 Thread Thomas Monjalon
2016-10-13 09:20, John Ousterhout: > Hi Harry, > > But, given the existence of the --file-prefix option, isn't it already > unsafe for Collectd to check only for .rte_config? If it's important for > other programs to be able to find the config files, it seems to me that a > more robust mechanism

[dpdk-dev] [PATCH 1/1] doc: fix errors in pdump doc

2016-10-13 Thread Thomas Monjalon
> > - Fix copy/paste error in description of how to capture both rx > > & tx traffic in a single pcap file > > - Replace duplicate word with what original author presumably > > intended, such that description now makes sense > > > > Signed-off-by: Mark Kavanagh > > Acked-by: John McNamara

[dpdk-dev] [PATCH 1/1] doc: clarify usage of testpmd mac fwd mode

2016-10-13 Thread Thomas Monjalon
> > Explain default testpmd behavior in mac fwd mode to remove > > amiguity/confusion regarding user's ability to specify Ethernet addresses. > > > > Signed-off-by: Mark Kavanagh > > Acked-by: John McNamara Applied, thanks

[dpdk-dev] [PATCH v2] app/testpmd: add support to configure 25G and 50G speeds

2016-10-13 Thread Thomas Monjalon
2016-10-12 16:26, Ajit Khaparde: > Support to configure 25G and 50G speeds is missing from testpmd. > This patch also updates the testpmd user guide accordingly. > > Signed-off-by: Ajit Khaparde > -- > v2: shorten lines > 80 character limit > --- Applied, thanks

[dpdk-dev] [PATCH v2] app/testpmd: fix DCB config issue

2016-10-13 Thread Thomas Monjalon
> > An issue is found that DCB cannot be configured on ixgbe NICs. It's said the > > TX queue number is not right. > > On ixgbe the max TX queue number is not fixed, it depends on the multi- > > queue mode. > > > > This patch adds the device configuration before getting info in the DCB > >

[dpdk-dev] 17.02 Roadmap

2016-10-13 Thread Thomas Monjalon
2016-10-13 15:18, Hunt, David: > Hi Thomas, > > On 10/10/2016 9:42 PM, Thomas Monjalon wrote: > > 2016-10-10 16:13, O'Driscoll, Tim: > >> Packet Distributor Enhancements: Enhancements will be made to the Packet > >> Distributor library to improve per

[dpdk-dev] [PATCH v2] app/testpmd: fix RSS-hash-key size

2016-10-13 Thread Thomas Monjalon
> > RSS hash-key-size is retrieved from device configuration instead of > > using a fixed size of 40 bytes. > > > > Fixes: f79959ea1504 ("app/testpmd: allow to configure RSS hash key") > > > > Signed-off-by: Mohammad Abdul Awal > > Acked-by: Konstantin Ananyev Applied, thanks

[dpdk-dev] [PATCH] testpmd: fix tso with csum engine

2016-10-13 Thread Thomas Monjalon
2016-10-13 15:40, Olivier Matz: > The commit that disabled tso for small packets was broken during the > rebase. The problem is the IP checksum is not calculated in software if: > - TX IP checksum is disabled > - TSO is enabled > - the current packet is smaller than tso segment size > > When

[dpdk-dev] [PATCH 2/2] app/testpmd: use consistent vdev names

2016-10-13 Thread Thomas Monjalon
2016-10-07 02:27, De Lara Guarch, Pablo: > > > -Original Message- > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Sent: Thursday, October 06, 2016 3:34 AM > > To: De Lara Guarch, Pablo > > Cc: dev at dpdk.org > > Subject: [PATC

[dpdk-dev] [PATCH] app/test: reduce lpm6 test case size

2016-10-13 Thread Thomas Monjalon
2016-09-30 02:11, Wei Dai: > copy app/test/test_lpm6_routes.h to app/test/test_lpm6_data.h . > and then delete app/test/test_lpm6_routes.h and clear the > large_ips_table[ ] to make LPM6 test case size much smaller than > before. Also add codes in app/test/test_lpm6_data.h to generate test > data

[dpdk-dev] [PATCH v2] app/test: remove large lpm test head file

2016-10-13 Thread Thomas Monjalon
2016-09-28 01:38, Wei Dai: > remove the large file app/test/test_lpm_routes.h and add codes to > auto-generate similar large route rule talbe which keeps same depth > and IP class distribution as previous one in test_lpm_routes.h . > With the rule table auto-generated at run time, the performance

[dpdk-dev] [PATCH] test: fix hash multiwriter test

2016-10-13 Thread Thomas Monjalon
2016-10-06 23:34, Pablo de Lara: > Hash multiwriter test consists of two subtests. > If the any of the subtests fails, the overall test should fail, > but the overall test only passed if the second subtest passed, > because the return of the first subtest was being overwritten. > > Fixes:

[dpdk-dev] [PATCH v3 1/3] lib/librte_port: enable file descriptor port support

2016-10-13 Thread Thomas Monjalon
2016-10-13 10:17, Jasvinder Singh: > This patch adds File Descriptor(FD) port type (e.g. TAP port) to the > packet framework library that allows interface with the kernel network > stack. The FD port APIs are defined that allow port creation, writing > and reading packet from the kernel interface.

[dpdk-dev] [PATCH 0/2] Two offloading issues of tep_term

2016-10-13 Thread Thomas Monjalon
2016-08-04 07:58, Jianfeng Tan: > This patch set depends on: > - http://dpdk.org/ml/archives/dev/2016-August/044924.html > > Patch 1: fill tunneling type. > Patch 2: inner L4 checksum error. > > Signed-off-by: Jianfeng Tan Applied, thanks

[dpdk-dev] [PATCH] examples/l3fwd: enabling 4m hash for all 64 bit archs

2016-10-13 Thread Thomas Monjalon
2016-08-23 20:24, Hemant Agrawal: > This patch enables the support for 4 million hash entries > for all 64 bit architectures. > > Signed-off-by: Hemant Agrawal Applied, thanks

[dpdk-dev] [PATCH v2 0/2] examples/l2fwd: Add option to enable/disable MAC addresses updating

2016-10-13 Thread Thomas Monjalon
2016-09-23 15:50, Maxime Coquelin: > This series adds a new option to enable/disable MAC addresses updating in > l2fwd example. > > Doing that, we can enable basic VM 2 VM communication easily, without > external projects dependencies, nor real NIC (as with vhost example). > > Example of cli

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

2016-10-13 Thread Thomas Monjalon
Hi Tomasz, Any news? Sorry to speed up, we are very very late for RC1. 2016-10-10 16:08, Thomas Monjalon: > Hi, > > Now that the feature seems to meet a consensus, I've looked at it more > closely before integrating. Sorry if it appears like a late review. > > 2016-09-30 11:

[dpdk-dev] [PATCH] app/test: add mempool walk

2016-10-13 Thread Thomas Monjalon
2016-10-05 14:22, Olivier Matz: > Hi Thomas, > > On 10/03/2016 10:58 PM, Thomas Monjalon wrote: > > The mempool function rte_mempool_walk was not tested. > > It will print the name of all mempools. > > > > Signed-off-by: Thomas Monjalon > > Acked-by: Olivier Matz Applied

[dpdk-dev] [PATCH] examples/qos_sched: fix packets dequeue operation from ring

2016-10-13 Thread Thomas Monjalon
2016-09-01 11:11, Jasvinder Singh: > The app_worker_thread() and app_mixed_thread() use rte_ring_sc_dequeue_bulk > to dequeue packets from the ring and this imposes restriction on number of > packets in software ring to be greater than the specified value to start > actual dequeue operation, thus,

[dpdk-dev] [PATCH v2] examples: fix ip_pipeline to load PMD driver correctly

2016-10-12 Thread Thomas Monjalon
> > From: Gowrishankar Muthukrishnan > > > > v2: minor correction in patch to avoid space between -d option and driver > > path > > Acked-by: Cristian Dumitrescu Applied, thanks

[dpdk-dev] [PATCH 2/2] ip_pipeline: enable swap action in network layers configuration file

2016-10-12 Thread Thomas Monjalon
2016-08-26 22:21, Jasvinder Singh: > The network_layers configuration file (config/network_layers.cfg) > demonstrates the various network layer components such as TCP, UDP, > ICMP etc, which can be easily integrated into ip pipeline > infrastructure. > > The loopback function (implemented using

[dpdk-dev] [PATCH v2 1/3] lib/librte_port: enable file descriptor port support

2016-10-12 Thread Thomas Monjalon
2016-09-04 15:38, Jasvinder Singh: > +#define RTE_PORT_FD_READER_STATS_PKTS_IN_ADD(port, val) \ > + do { port->stats.n_pkts_in += val } while (0) > +#define RTE_PORT_FD_READER_STATS_PKTS_DROP_ADD(port, val) \ > + do { port->stats.n_pkts_drop += val } while (0) > + It does not compile

[dpdk-dev] [PATCH 2/2] examples/ip_pipeline: modify source port default parameter

2016-10-12 Thread Thomas Monjalon
> > The default value of ``file_name`` parameter of the source port structure is > > changed from ``NULL`` to ``./config/packets.pcap``. > > > > Signed-off-by: Jasvinder Singh > > Acked-by: Cristian Dumitrescu Series applied, thanks

[dpdk-dev] [PATCH 0/3] changes for enabling cuckoo hash into table library

2016-10-12 Thread Thomas Monjalon
> > From: Guruprasad Rao > > Acked-by: Cristian Dumitrescu Applied, thanks

[dpdk-dev] [PATCH v4] log: respect rte_openlog_stream calls before rte_eal_init

2016-10-12 Thread Thomas Monjalon
2016-10-12 12:38, John Ousterhout: > @@ -127,6 +125,19 @@ rte_vlog(uint32_t level, uint32_t logtype, const char > *format, va_list ap) > { > int ret; > FILE *f = rte_logs.file; > + if (f == NULL) { > + f = default_log_stream; > + if (f == NULL) { > +

[dpdk-dev] [PATCH v6 0/8] Misc enhancements in testpmd

2016-10-12 Thread Thomas Monjalon
2016-10-12 17:39, Olivier Matz: > This patchset introduces several enhancements or minor fixes > in testpmd. It is targetted for v16.11, and applies on top of > software ptype v2 patchset [1]. > > These patches are useful to validate the virtio offload > patchset [2] (to be rebased). Applied,

[dpdk-dev] [PATCH v8 2/2] app/test_pmd: add tests for new API's

2016-10-12 Thread Thomas Monjalon
2016-10-12 16:21, Iremonger, Bernard: > Hi Thomas, > > > > > > > > Subject: Re: [dpdk-dev] [PATCH v8 2/2] app/test_pmd: add tests > > > > > > for new API's > > > > > > > > > > > > 2016-10-12 16:03, Bernard Iremonger: > > > > > > > --- a/app/test-pmd/Makefile > > > > > > > +++

[dpdk-dev] [PATCH v8 2/2] app/test_pmd: add tests for new API's

2016-10-12 Thread Thomas Monjalon
2016-10-12 15:48, Iremonger, Bernard: > Hi Thomas, > > > > > Subject: Re: [dpdk-dev] [PATCH v8 2/2] app/test_pmd: add tests for new > > API's > > > > 2016-10-12 15:27, Iremonger, Bernard: > > > Hi Thomas, > > > > > > > > > > > > > Subject: Re: [dpdk-dev] [PATCH v8 2/2] app/test_pmd: add tests

[dpdk-dev] [PATCH v8 2/2] app/test_pmd: add tests for new API's

2016-10-12 Thread Thomas Monjalon
2016-10-12 15:27, Iremonger, Bernard: > Hi Thomas, > > > > > Subject: Re: [dpdk-dev] [PATCH v8 2/2] app/test_pmd: add tests for new > > API's > > > > 2016-10-12 16:03, Bernard Iremonger: > > > --- a/app/test-pmd/Makefile > > > +++ b/app/test-pmd/Makefile > > > @@ -58,6 +58,17 @@ SRCS-y +=

[dpdk-dev] [PATCH v3 0/5] vhost: optimize enqueue

2016-10-12 Thread Thomas Monjalon
Sorry guys, you lost me in the discussion. Is there some regression only on ARM? Does it need some work specifically on memcpy for ARM, or vhost for ARM? Who can work on ARM optimization? More comments below. 2016-10-12 12:22, Wang, Zhihong: > From: Yuanhan Liu [mailto:yuanhan.liu at

[dpdk-dev] [PATCH v8 2/2] app/test_pmd: add tests for new API's

2016-10-12 Thread Thomas Monjalon
2016-10-12 16:03, Bernard Iremonger: > --- a/app/test-pmd/Makefile > +++ b/app/test-pmd/Makefile > @@ -58,6 +58,17 @@ SRCS-y += csumonly.c > SRCS-y += icmpecho.c > SRCS-$(CONFIG_RTE_LIBRTE_IEEE1588) += ieee1588fwd.c > > +ifeq ($(CONFIG_RTE_BUILD_SHARED_LIB),n) >

[dpdk-dev] [PATCH] hash: fix incorrect bucket size usage

2016-10-12 Thread Thomas Monjalon
2016-10-12 02:26, Pablo de Lara: > Multiwriter insert function was using a fixed value for > the bucket size, instead of using the > RTE_HASH_BUCKET_ENTRIES macro, which value was changed > recently (making it inconsistent in this case). > > Fixes: be856325cba3 ("hash: add scalable multi-writer

[dpdk-dev] [PATCH v2] hash: fix unlimited cuckoo path

2016-10-12 Thread Thomas Monjalon
2016-10-12 01:50, Pablo de Lara: > When trying to insert a new entry, if its target bucket is full, > the alternative location (bucket) of one of the entries is checked, > to try to find an empty slot, with make_space_bucket. > This function is called every time a new bucket is checked,

[dpdk-dev] [PATCH v3] mk: gcc -march support for intel processors code names

2016-10-12 Thread Thomas Monjalon
2016-10-12 13:38, Pattan, Reshma: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > 2016-10-10 22:33, Reshma Pattan: > > > The GCC 4.9 -march option supports the intel code names for > > > processors, for example -march=silvermont, -march=broadwell. &

[dpdk-dev] [PATCH] dpdk-procinfo: free allocated xstats memory upon failure

2016-10-12 Thread Thomas Monjalon
2016-10-04 17:42, Reshma Pattan: > Some of the failures cases inside the nic_xstats_display() > function doesn't free the allocated memory for the xstats and > their names, memory is freed now. > > Fixes: e2aae1c1 ("ethdev: remove name from extended statistic fetch") > Fixes: 22561383 ("app:

[dpdk-dev] [PATCH] pdump: fix dir permissions value in mkdir call

2016-10-12 Thread Thomas Monjalon
2016-10-10 22:44, Remy Horton: > > On 10/10/2016 15:35, Reshma Pattan wrote: > [..] > > Fixes: e4ffa2d3 ("pdump: fix error handlings") > > Fixes: bdd8dcc6 ("pdump: fix default socket path") > > > > Reported-by: Jianfeng Tan > > Signed-off-by: Reshma Pattan > > --- > >

[dpdk-dev] [PATCH v3] mk: gcc -march support for intel processors code names

2016-10-12 Thread Thomas Monjalon
2016-10-10 22:33, Reshma Pattan: > The GCC 4.9 -march option supports the intel code names for processors, > for example -march=silvermont, -march=broadwell. > The RTE_MACHINE config flag can be used to pass code name to > the compiler as -march flag. > > Release notes is updated. > > Linux and

[dpdk-dev] ZUC PMD as shared library

2016-10-12 Thread Thomas Monjalon
2016-10-12 02:23, De Lara Guarch, Pablo: > Hi Thomas, > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > > > Hi Pablo, > > > > You are probably aware of the issue, but I would like to make it clear > > in case someone else run into the

[dpdk-dev] [PATCH v2] log: respect rte_openlog_stream calls before rte_eal_init

2016-10-12 Thread Thomas Monjalon
2016-10-11 14:46, John Ousterhout: > All of your suggestions look reasonable and fairly straightforward; I'll > work on a new patch that includes them. > > Given that rte_eal_log_init is a no-op (and won't even be invoked), would > it be better to remove that function completely, and even delete

[dpdk-dev] Project Governance and Linux Foundation

2016-10-12 Thread Thomas Monjalon
Hi, This is a really interesting use case. Questions below, 2016-10-12 13:44, qin.chunhua at zte.com.cn: > ZTE is supportive of improving the DPDK project governance including > moving the project to the Linux Foundation. > DPDK has been used in lots of ZTE's equipments and solutions such as

[dpdk-dev] [PATCH v5 1/8] mbuf: add function to dump ol flag list

2016-10-11 Thread Thomas Monjalon
2016-10-07 18:05, Olivier Matz: > The functions rte_get_rx_ol_flag_name() and rte_get_tx_ol_flag_name() > can dump one flag, or set of flag that are part of the same mask (ex: > PKT_TX_UDP_CKSUM, part of PKT_TX_L4_MASK). But they are not designed to > dump the list of flags contained in

[dpdk-dev] [PATCH v2] log: respect rte_openlog_stream calls before rte_eal_init

2016-10-11 Thread Thomas Monjalon
2016-10-11 09:30, John Ousterhout: > On Tue, Oct 11, 2016 at 1:08 AM, Thomas Monjalon 6wind.com> > wrote: > > > > 2016-10-10 15:39, John Ousterhout: > > > ... > > > > > > Note: I see from the code that Linux and BSD set different default > stre

[dpdk-dev] [PATCH v7 2/2] app/test_pmd: add tests for new API's

2016-10-11 Thread Thomas Monjalon
2016-10-11 15:51, Iremonger, Bernard: > Hi Thomas, > > > > > Subject: Re: [dpdk-dev] [PATCH v7 2/2] app/test_pmd: add tests for new > > API's > > > > 2016-10-11 16:09, Ferruh Yigit: > > > This will cause a compilation error for shared libraries. Because PMDs > > > not linked against

[dpdk-dev] [PATCH v3 00/16] software parser for packet type

2016-10-11 Thread Thomas Monjalon
2016-10-03 10:38, Olivier Matz: > This patchset introduces a software packet type parser. This > feature is targeted for v16.11. > > The goal here is to provide a reference implementation for packet type > parsing. This function will be used by testpmd to compare its result > with the value given

[dpdk-dev] [PATCH v7 2/2] app/test_pmd: add tests for new API's

2016-10-11 Thread Thomas Monjalon
2016-10-11 16:09, Ferruh Yigit: > This will cause a compilation error for shared libraries. Because PMDs > not linked against application when compiled as shared library but used > as plugins. > > Since it has been decided to have NIC specific APIs, we need to re-work > that approach to fix

[dpdk-dev] [PATCH 1/3] eal/drivers: prefix driver REGISTER macros with EAL

2016-10-11 Thread Thomas Monjalon
2016-10-11 09:38, Neil Horman: > This also begs the question in my mind, is it really worth changing the macro? > I really don't think it is. The registration macros are pretty descriptive as > they stand, and have already changed 3 or 4 times in the last 6 months, which > suggests to me that any

[dpdk-dev] OpenSSL libcrypto PMD name

2016-10-11 Thread Thomas Monjalon
2016-10-11 09:53, Declan Doherty: > On 10/10/16 12:36, Thomas Monjalon wrote: > > Hi, > > > > I would like to raise a naming issue in crypto. > > > > In the crypto side of DPDK, we have a library (similar to ethdev) > > for crypto API and device interface: &

[dpdk-dev] [PATCH v5 1/2] librte_ether: add protection against overwrite device data

2016-10-11 Thread Thomas Monjalon
2016-10-07 12:23, Kerlin, MarcinX: > Hi Thomas, > > > -Original Message- > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Sent: Thursday, October 06, 2016 4:53 PM > > To: Kerlin, MarcinX > > Cc: dev at dpdk.org; De Lara Guarch,

[dpdk-dev] [PATCH v2] log: respect rte_openlog_stream calls before rte_eal_init

2016-10-11 Thread Thomas Monjalon
2016-10-10 15:39, John Ousterhout: > Before this patch, application-specific loggers could not be > installed before rte_eal_init completed (the initialization process > called rte_openlog_stream, overwriting any previously installed > logger). This made it impossible for an application to capture

[dpdk-dev] [PATCH 1/3] eal/drivers: prefix driver REGISTER macros with EAL

2016-10-11 Thread Thomas Monjalon
2016-10-11 12:06, Shreyansh Jain: > On Monday 10 October 2016 06:26 PM, Neil Horman wrote: > > On Sat, Oct 08, 2016 at 01:00:59PM +, Shreyansh Jain wrote: > >> Hi Thomas, > >> > >>> -Original Message- > >>> From: Thomas Monjalon [mail

[dpdk-dev] [PATCH v3 03/16] mbuf: move packet type definitions in a new file

2016-10-11 Thread Thomas Monjalon
2016-10-11 11:01, Olivier MATZ: > Hi Thomas, > > On 10/10/2016 04:52 PM, Thomas Monjalon wrote: > > 2016-10-03 10:38, Olivier Matz: > >> The file rte_mbuf.h starts to be quite big, and next commits > >> will introduce more functions related to packet types. L

[dpdk-dev] 17.02 Roadmap

2016-10-10 Thread Thomas Monjalon
Thanks Tim for the interesting inputs. Some of them may require a dedicated thread to continue the discussion based on some preliminary specifications or drafts. 2016-10-10 16:13, O'Driscoll, Tim: > Elastic Flow Distributor: The Elastic Flow Distributor (EFD) is a flow-based > load balancing

[dpdk-dev] [PATCH v2] mk: gcc -march support for intel processors code names

2016-10-10 Thread Thomas Monjalon
2016-10-10 15:21, Pattan, Reshma: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Why not recommend GCC 4.9 and keep the graceful degradation for older > > versions, at least for one more year, even if it is not optimal for newer > > architectures? > >

[dpdk-dev] [PATCH v3 03/16] mbuf: move packet type definitions in a new file

2016-10-10 Thread Thomas Monjalon
2016-10-03 10:38, Olivier Matz: > The file rte_mbuf.h starts to be quite big, and next commits > will introduce more functions related to packet types. Let's > move them in a new file. > > Signed-off-by: Olivier Matz > --- > lib/librte_mbuf/Makefile | 2 +- >

<    1   2   3   4   5   6   7   8   9   10   >