[dpdk-dev] DPDK patchwork upgrade

2016-09-07 Thread Thomas Monjalon
Hi all, Patchwork is upgraded from a pre-0.9 version to 1.1.1: http://dpdk.org/dev/patchwork This new version brings 2 new columns: - A/R/T for Acked-by/Reviewed-by/Tested-by counters. Unfortunately it does not yet parse the cover letter of a series. - S/W/F for

[dpdk-dev] Build with -Werror=declaration-after-statement

2016-09-02 Thread Thomas Monjalon
2016-09-02 13:49, ASM: > Hi folks! > > Many groups have flag -Werror=declaration-after-statement in the > their coding conventions. Dpdk have many code as static inline > function in the headers files and have trouble with it. > > For example: > * lib/librte_ether/rte_ether.h, function

[dpdk-dev] [PATCH 2/2] virtio: support IOMMU platform

2016-09-02 Thread Thomas Monjalon
2016-09-02 14:37, Jason Wang: > Virtio pmd doesn't support VFIO in the past since devices bypass IOMMU > completely. But recently, the work of making virtio device work with > IOMMU is near to complete. Good news! What are the requirements for Qemu and Linux version numbers please?

[dpdk-dev] [PATCH 1/2] net/virtio: support modern device id

2016-09-02 Thread Thomas Monjalon
2016-09-02 14:36, Jason Wang: > Spec said "The PCI Device ID is calculated by adding 0x1040 to the > Virtio Device ID". So this patch makes pmd can recognize modern virtio > net id. Please could you describe what is a modern virtio-net? > #define VIRTIO_PCI_DEVICEID_MIN 0x1000 > #define

[dpdk-dev] [dpdk-dev, RFC] drivers: advertise kmod dependencies in pmdinfo

2016-09-02 Thread Thomas Monjalon
2016-09-01 10:41, Stephen Hemminger: > Neil Horman wrote: > > On Thu, Sep 01, 2016 at 12:55:27PM +, Trahe, Fiona wrote: > > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier Matz > > > > On 08/31/2016 03:27 PM, Neil Horman wrote: > > > > > Oh, I see, so your list is a colon

[dpdk-dev] 17.02 Roadmap

2016-08-31 Thread Thomas Monjalon
2016-08-31 10:31, O'Driscoll, Tim: > Below are the features that we're planning to submit for the 17.02 > release. We'll submit a patch to update the roadmap page with this info. > > Some things will obviously change during planning/development, so we'll > provide a more detailed update in late

[dpdk-dev] [PATCH] acl: use rte_calloc for temporary memory allocation

2016-08-31 Thread Thomas Monjalon
2016-08-31 08:38, Vladyslav Buslov: > Would you accept this change as config file compile-time parameter with libc > calloc as default? > It is one line change only so it is easy to ifdef. The configuration should not be compile-time. Please think about a runtime configuration via an API.

[dpdk-dev] [PATCH] doc/guides: add info on how to enable QAT

2016-08-30 Thread Thomas Monjalon
2016-08-30 13:57, Wiles, Keith: > > 2016-08-30 14:26, Eoin Breen: > >> --- a/doc/guides/cryptodevs/qat.rst > >> +++ b/doc/guides/cryptodevs/qat.rst > >> @@ -78,6 +78,11 @@ Installation > >> To use the DPDK QAT PMD an SRIOV-enabled QAT kernel driver is required. The > >> VF devices exposed by this

[dpdk-dev] [PATCH] doc/guides: add info on how to enable QAT

2016-08-30 Thread Thomas Monjalon
2016-08-30 14:26, Eoin Breen: > --- a/doc/guides/cryptodevs/qat.rst > +++ b/doc/guides/cryptodevs/qat.rst > @@ -78,6 +78,11 @@ Installation > To use the DPDK QAT PMD an SRIOV-enabled QAT kernel driver is required. The > VF devices exposed by this driver will be used by QAT PMD. > > +To enable

[dpdk-dev] [PATCH v8 01/25] eal: define macro container_of

2016-08-30 Thread Thomas Monjalon
2016-08-30 09:57, Shreyansh Jain: > On Monday 29 August 2016 10:13 PM, Ferruh Yigit wrote: > > This gives compilation error for mlx5, because the libraries mlx depends > > defines same macro: > > /rte_common.h:338:9: error: 'container_of' macro redefined > >

[dpdk-dev] [PATCH] ntnic: add PMD driver

2016-08-29 Thread Thomas Monjalon
2016-08-29 06:22, Finn Christensen: > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > > What is the license of this driver suite? > > The driver suite is a closed-source driver, which is not free downloadable. > > > IMHO the upstream DPDK shouldn't be a platform for non-free

[dpdk-dev] [PATCH] scripts: remove useless checkpatch notes

2016-08-29 Thread Thomas Monjalon
is now more reliable because based on a very stable pattern. Signed-off-by: Thomas Monjalon --- scripts/checkpatches.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/checkpatches.sh b/scripts/checkpatches.sh index b596b4e..4addde4 100755 --- a/scripts

[dpdk-dev] [PATCH] scripts: hide double git reference check error

2016-08-29 Thread Thomas Monjalon
: fix documentation for queue start/stop") The error from the script is sufficient so the git error can be hidden. Signed-off-by: Thomas Monjalon --- scripts/check-git-log.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/check-git-log.sh b/scripts/check-git-log.sh ind

[dpdk-dev] [PATCH] ntnic: add PMD driver

2016-08-27 Thread Thomas Monjalon
2016-08-26 16:32, Finn Christensen: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > It would help a lot to describe what is the Napatech NIC Driver Suite, and > > why it is a virtual driver. > > I will elaborate on the description to make it more clear and s

[dpdk-dev] [PATCH 2/3] hash: add vectorized comparison

2016-08-27 Thread Thomas Monjalon
2016-08-26 22:34, Pablo de Lara: > From: Byron Marohn > > In lookup bulk function, the signatures of all entries > are compared against the signature of the key that is being looked up. > Now that all the signatures are together, they can be compared > with vector instructions (SSE, AVX2),

[dpdk-dev] FW: [PATCH v6 0/9] enable lpm, acl and other missing libraries in ppc64le

2016-08-26 Thread Thomas Monjalon
2016-08-26 18:55, Chao Zhu: > Thomas, > > Any comments of this patch set? Are we waiting for more acks? Yes, especially for lpm and acl, a review from respective maintainers may be needed. Konstantin? Bruce? If no review is done in 1 week, it will be applied as-is. Gowrishankar, using

[dpdk-dev] [PATCH] kni: add support for older kernels

2016-08-26 Thread Thomas Monjalon
> +#if (defined(RHEL_RELEASE_CODE) && \ > + (RHEL_RELEASE_CODE >= RHEL_RELEASE_VERSION(6, 8)) && \ > + (LINUX_VERSION_CODE < KERNEL_VERSION(2, 6, 34))) > +#undef NET_NAME_UNKNOWN > +#endif Should the title be "kni: support RHEL 6.8"?

[dpdk-dev] [PATCH] ntnic: add PMD driver

2016-08-26 Thread Thomas Monjalon
Welcome, 2016-08-26 13:44, Finn Christensen: > +NTNIC Poll Mode Driver > +== > + > +The NTNIC poll mode driver library (**librte_pmd_ntnic**) implements support > +for **Napatech NIC** 40/50 Gbps adapters. > +This PMD is implemented as a pure software virtual device and must

[dpdk-dev] [PATCH 2/5] i40e: implement vector PMD for ARM architecture

2016-08-26 Thread Thomas Monjalon
Hi Jianbo (and other developers of vectorized PMDs), 2016-08-24 15:23, Jianbo Liu: > Use ARM NEON intrinsic to implement i40e vPMD Have you tried to use the generic SIMD intrinsics? We could maintain only one vectorized implementation by using __attribute__ ((vector_size (n))) as

[dpdk-dev] [PATCH] kni: memzone info not required to get mbuf address

2016-08-24 Thread Thomas Monjalon
2016-08-04 16:54, Ferruh Yigit: > Originally mempool->mz is used to get address of the mbuf, but now > address get directly from mempool, so mempool->mz information is not > required. > > Fixes: d1d914ebbc25 ("mempool: allocate in several memory chunks by default") > > Signed-off-by: Ferruh

[dpdk-dev] [PATCH] ethtool: remove triple license information

2016-08-24 Thread Thomas Monjalon
2016-08-04 17:45, Ferruh Yigit: > On 8/1/2016 1:17 PM, Christian Ehrhardt wrote: > > License information is already in LICENSE.GPL. > > Remove two extra copies and change referred filename in the files. > > > > Signed-off-by: Christian Ehrhardt > > In patch subject, I think it is better to use

[dpdk-dev] [PATCH v2] scripts: make load-devel-config not to appear as executable

2016-08-24 Thread Thomas Monjalon
2016-08-03 16:59, Christian Ehrhardt: > *Updates in v2* > - drop the #!/bin/echo now that it is no more executable > > Quoting the first line of the script: "#! /bin/echo must be loaded with ." > Given that we should drop the .sh file ending as well as the executable > flag - both are not needed

[dpdk-dev] [PATCH v2] examples: fix unusual-interpreter

2016-08-24 Thread Thomas Monjalon
> > *update in v2* > > - use #!/usr/bin/env python as usually recommended and suggested in the > > discussion > > > > Due to regular lintian checks in Debian packaging it surfaced that these > > two scripts had a space in their #! statement which renders it to be > > human, but not shell

[dpdk-dev] [PATCH] ethdev: fix a typo in eth device API doc

2016-08-24 Thread Thomas Monjalon
2016-08-01 11:19, Nikhil Rao: > This patch fixes a typo in the eth device API doc, device > config. not stored between calls to rte_eth_dev_start/stop() > should be restored before a call to rte_eth_dev_start() > instead of after a call to rte_eth_dev_start(). > > Signed-off-by: Nikhil Rao [...]

[dpdk-dev] [PATCH] doc: Fix incorrect comment line

2016-08-24 Thread Thomas Monjalon
2016-08-17 18:36, Shreyansh Jain: > Signed-off-by: Shreyansh Jain Applied, thanks

[dpdk-dev] [PATCH] doc/versioning: add missing return value type in MAP_STATIC_SYMBOL example

2016-08-24 Thread Thomas Monjalon
2016-08-14 18:21, Mauricio Vasquez B: > The example only had as return type struct, it is actually struct rte_acl_ctx > * > > Signed-off-by: Mauricio Vasquez B Applied (minus a whitespace), thanks

[dpdk-dev] [PATCH 0/5] i40e: vector poll-mode driver on ARM64

2016-08-24 Thread Thomas Monjalon
2016-08-24 15:23, Jianbo Liu: > This patch set is to implement i40e vector PMD on ARM64. Thanks for extending ARM support. The current NIC support status is: % git grep -l 'ARM.*=.*Y' doc/guides/nics/features/ doc/guides/nics/features/ixgbe.ini doc/guides/nics/features/ixgbe_vec.ini

[dpdk-dev] [PATCH] vhost: add pmd xstats

2016-08-24 Thread Thomas Monjalon
2016-08-24 13:46, Yuanhan Liu: > On Tue, Aug 23, 2016 at 12:45:54PM +0300, Panu Matilainen wrote: > > >>>Since collecting data of vhost_update_packet_xstats will have some > > >>>effect on RX/TX performance, so, Setting compiling switch > > >>>CONFIG_RTE_LIBRTE_PMD_VHOST_UPDATE_XSTATS=n by default

[dpdk-dev] [PATCH v1] doc: fix broken link in docs

2016-08-23 Thread Thomas Monjalon
> > From: Deirdre O'Connor > > > > Fixes: 58abf6e77c6b ("doc: add contributors guide") > > > > Suggested-by: Jon Loeliger > > Signed-off-by: Deirdre O'Connor > > Acked-by: John McNamara Applied, thanks

[dpdk-dev] [PATCH v4 00/10] Fix build errors related to exported headers

2016-08-23 Thread Thomas Monjalon
After rebasing the patchset, the compilation of each patch seems good. But the new checks fail with clang: rte_memcpy.h:814:2: error: implicit declaration of function '_mm_alignr_epi8' is invalid in C99 Other comments about the script: - it is too long (can it be

[dpdk-dev] [PATCH 0/3] remove rte_pci_dev_ids.h

2016-08-23 Thread Thomas Monjalon
2016-08-05 15:09, Ferruh Yigit: > This patchset moves remaining PCI device ids for ixgbe and igb drivers, > updates KNI code which uses these ids to use pci_device_ids from kni/ethtool > drivers, > and finally removes the rte_pci_dev_ids.h and updates document refers it. > > Ferruh Yigit (3): >

[dpdk-dev] [PATCH 3/3] eal: remove rte_pci_dev_ids.h

2016-08-23 Thread Thomas Monjalon
2016-08-08 08:20, David Marchand: > On Sat, Aug 6, 2016 at 2:55 PM, Thomas Monjalon > wrote: > > What do you think about this list? I would say we can remove it. > > Yes, we already discussed this with John. > http://dpdk.org/ml/archives/dev/2016-January/031595.html Done: h

[dpdk-dev] [PATCH] doc: remove useless file listings

2016-08-23 Thread Thomas Monjalon
> > Lists of DPDK files are hard to maintain up to date and does not bring > > much information. > > > > Signed-off-by: Thomas Monjalon > > Acked-by: John McNamara Applied

[dpdk-dev] [PATCH] doc: remove useless file listings

2016-08-23 Thread Thomas Monjalon
Lists of DPDK files are hard to maintain up to date and does not bring much information. Signed-off-by: Thomas Monjalon --- doc/guides/freebsd_gsg/build_dpdk.rst | 13 +- doc/guides/linux_gsg/build_dpdk.rst| 10 - doc/guides/prog_guide/dev_kit_build_system.rst | 57

[dpdk-dev] [PATCH] ivshmem: remove integration in dpdk

2016-08-23 Thread Thomas Monjalon
> > Following discussions on the mailing list [1] and since nobody stood up to > > implement the necessary cleanups, here is the ivshmem integration removal. > > > > There is not much to say about this patch, a lot of code is being removed. > > The default configuration file for packet_ordering

[dpdk-dev] [PATCH 6/6] examples/vhost: add an option to enable Tx zero copy

2016-08-23 Thread Thomas Monjalon
2016-08-23 16:10, Yuanhan Liu: > One thing worth noting while using Tx zero copy is the nb_tx_desc has > to be small enough so that the eth driver will hit the mbuf free > threshold easily and thus free mbuf more frequently. > > The reason behind that is, when Tx zero copy is enabled, guest Tx

[dpdk-dev] [PATCH] doc: fix old dpdk-nic-bind.py references

2016-08-23 Thread Thomas Monjalon
> > dpdk-nic-bind.py script has been renamed to dpdk-devbind.py, > > but some references to the old script have remained. > > This commit completes the renaming. > > > > Fixes: a5d7a3f77ddc ("unify tools naming") > > > > Signed-off-by: Pablo de Lara > > I like the cleanups for consistency! > >

[dpdk-dev] [PATCH v2] contigmem: zero all pages during mmap

2016-08-22 Thread Thomas Monjalon
2016-08-16 15:46, Jim Harris: > On Linux, all huge pages are zeroed by the kernel before > first access by the DPDK application. But on FreeBSD, > the contigmem driver would only zero the contiguous > memory regions during initial driver load. > > DPDK commit b78c91751 eliminated the explicit

[dpdk-dev] [PATCH] table: add missing exports

2016-08-22 Thread Thomas Monjalon
> > Signed-off-by: Aleksey Katargin > > Acked-by: Cristian Dumitrescu Fixes: 8aa327214ceb ("table: hash") Fixes: 68866e2417cc ("table: add 16-byte hash operations computed on lookup") Applied, thanks

[dpdk-dev] [PATCH] mk:fix second compile error

2016-08-22 Thread Thomas Monjalon
2016-08-09 14:01, xu, huilong: > when compile different targets on a same environment. > The second compile will failed, because test_resource obj file > can't auto clearn by makfile. I don't understand the error. Please provide commands and output.

[dpdk-dev] [PATCH 1/7] vhost: remove vhost-cuse

2016-08-22 Thread Thomas Monjalon
2016-08-18 16:48, Yuanhan Liu: > remove vhost-cuse code, including the eventfd_link kernel module that > is for vhost-cuse only. Please, remove also the deprecation notice. Thanks

[dpdk-dev] vhost [query] : support for multiple ports and non VMDQ devices in vhost switch

2016-08-22 Thread Thomas Monjalon
2016-08-18 10:36, Tan, Jianfeng: > From: Maxime Coquelin [mailto:maxime.coquelin at redhat.com] > > > On 8/17/2016 7:18 PM, Maxime Coquelin wrote: > > >> On 08/17/2016 04:33 AM, Tan, Jianfeng wrote: > > >>> b. Abstract switching logic into a framework, so that we can develop > > >>> different

[dpdk-dev] Best Practices for PMD Verification before Upstream Requests

2016-08-22 Thread Thomas Monjalon
2016-08-17 08:34, Shepard Siegel: > Atomic Rules is new to the DPDK community. We attended the DPDK Summit last > week and received terrific advice and encouragement. We are developing a > DPDK PMD for our Arkville product which is a DPDK-aware data mover, capable > of marshaling packets between

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

2016-08-22 Thread Thomas Monjalon
2016-08-22 11:46, Pattan, Reshma: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > 2016-08-15 16:02, Reshma Pattan: > > > The GCC 4.9 -march option supports the intel code names for > > > processors, for example -march=silvermont, -march=broadwell. &

[dpdk-dev] ipv4 fragmentation bug?

2016-08-22 Thread Thomas Monjalon
Hi, > 2016-08-15 20:30 GMT+03:00 ? ??? : > > While playing with function rte_ipv4_fragment_packet I found that it > > incorrectly fragments packets. > > For example if the function takes 1200 bytes packet and mtu size 1000 it > > will produces two fragments. And when those fragments

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

2016-08-22 Thread Thomas Monjalon
2016-08-15 16:02, 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. Also old gcc versions compatibility code > for

[dpdk-dev] [PATCH 4/4] net/vmxnet3: enable lro

2016-08-22 Thread Thomas Monjalon
2016-08-12 17:58, Yong Wang: > + DEV_RX_OFFLOAD_TCP_LRO; You should update doc/guides/nics/features/vmxnet3.ini accordingly. Thanks

[dpdk-dev] [PATCH v2] net/i40e: fix Rx statistic inconsistent

2016-08-22 Thread Thomas Monjalon
Hi, 2016-08-09 02:18, Zhao1, Wei: > According to your suggestion and meeting last week with harry ,I have supply > a v3 patch for this problem. > [dpdk-dev,v3] net/i40e: fix Rx statistic inconsistent , Permalink > :/dev/patchwork/patch/15161/. Please use --in-reply-to to send new version of a

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

2016-08-22 Thread Thomas Monjalon
2016-08-22 12:01, Maxime Coquelin: > I forgot to add that before this series, I think we should first fix the > windows bug. > Else we will need a dedicated fix for the stable branch. This is a funny situation :) If Zhihong had reworked the code without mentioning it is fixing a scenario with

[dpdk-dev] DPDK with VMXNET3

2016-08-21 Thread Thomas Monjalon
2016-08-21 12:01, Keren Hochman: > Hi, Hi, > I saw you removed from the git VMXNET3 user map. does this driver does not > support anymore in dpdk? thanks It is not removed, just unmaintained and moved here: http://dpdk.org/browse/old/vmxnet3-usermap/ Are you volunteer to maintain it?

[dpdk-dev] Patchwork Link is Broken

2016-08-19 Thread Thomas Monjalon
2016-08-19 09:47, Jon Loeliger: > Hey folks, > > Over on this page: > > http://dpdk.org/doc/guides/contributing/patches.html > > in section 5.1, paragraph 3 has a link to Patchwork that is broken. > That or perhaps someone was indicating an opinion: > >

[dpdk-dev] [dpdk-users] rte_zmalloc() returning non-zeroed memory on FreeBSD

2016-08-11 Thread Thomas Monjalon
Hi, 2016-08-10 23:30, Verkamp, Daniel: > It seems that with DPDK 16.07, rte_zmalloc() and related functions no > longer return zeroed memory reliably on FreeBSD. > > I notice that commit b78c9175118f7d61022ddc5c62ce54a1bd73cea5 ("mem: do > not zero out memory on zmalloc") removed the explicit

[dpdk-dev] DPDK support for on system bus connected embedded MAC

2016-08-06 Thread Thomas Monjalon
Hi, 2016-08-05 16:26, Venkat Karthik K: > Hello, > > I am planning to develop Poll Mode Driver for new embedded MAC which is > connected on system bus. Could you please provide more details about what you are working on? > I am looking into available poll mode drivers > support for different

[dpdk-dev] [PATCH 3/3] eal: remove rte_pci_dev_ids.h

2016-08-06 Thread Thomas Monjalon
2016-08-05 15:09, Ferruh Yigit: > All PCI device ids moved to drivers, it is safe to delete > rte_pci_dev_ids.h Good move. Thanks Ferruh! > --- a/doc/guides/prog_guide/dev_kit_build_system.rst > +++ b/doc/guides/prog_guide/dev_kit_build_system.rst > @@ -93,20 +93,20 @@ Each build directory

[dpdk-dev] test: failed to clone dpdk source in windows

2016-08-05 Thread Thomas Monjalon
2016-08-05 09:44, Thomas Monjalon: > 2016-08-05 10:09, linhaifeng: > > hi,thomas > > > > Could you change the name of file in directory > > app/test/test_pci_sysfs/bus/pci/devices/ ? > > I think somebody like us also cann't access internet in liunux.Windows not

[dpdk-dev] test: failed to clone dpdk source in windows

2016-08-05 Thread Thomas Monjalon
2016-08-05 10:09, linhaifeng: > hi,thomas > > Could you change the name of file in directory > app/test/test_pci_sysfs/bus/pci/devices/ ? > I think somebody like us also cann't access internet in liunux.Windows not > support file name > include ':'. Not sure to understand the use case. This is

[dpdk-dev] [RFC 0/4] Use Google Test as DPDK unit test framework

2016-08-05 Thread Thomas Monjalon
2016-08-04 19:55, Wiles, Keith: > > > On Aug 4, 2016, at 2:47 PM, Jim Murphy wrote: > > > > Hi, > > > > We are looking at using our existing test environment for our DPDK > > applications that will run on our build servers. Hughpages therefore is an > > issue. What is involved in running DPDK

[dpdk-dev] [PATCH] doc: announce ivshmem support removal

2016-08-04 Thread Thomas Monjalon
2016-08-04 17:00, : > Hi Thomas, > > I'm going to develop a NFV based carrier system with SPP because it's able > to chain VMs in high-performance and I think it might be the best solution > for service function chaining. Without ivshmem, throuput between VMs is > largely decreased. So we are

[dpdk-dev] [PATCH] version: 16.11-rc0

2016-08-03 Thread Thomas Monjalon
Now that we have a new blank release notes, a new generated NIC features table, and the obsolete code removed, we can start applying some patches for 16.11. Version number 16.11-rc0 applied. I wish you a happy start of release cycle :)

[dpdk-dev] [PATCH] version: 16.11-rc0

2016-08-03 Thread Thomas Monjalon
Signed-off-by: Thomas Monjalon --- lib/librte_eal/common/include/rte_version.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/librte_eal/common/include/rte_version.h b/lib/librte_eal/common/include/rte_version.h index 615deb7..2c20cb3 100644 --- a/lib/librte_eal

[dpdk-dev] [PATCH] mbuf: remove deprecated internal function

2016-08-03 Thread Thomas Monjalon
2016-08-01 15:04, Thomas Monjalon: > The function __rte_mbuf_raw_alloc was reserved for internal use and > has been deprecated in favor of the public function rte_mbuf_raw_alloc. > It can be safely removed now. > > Signed-off-by: Thomas Monjalon Applied

[dpdk-dev] [PATCH v2] log: remove history dump

2016-08-03 Thread Thomas Monjalon
2016-07-29 16:50, David Marchand: > On Fri, Jul 29, 2016 at 3:50 PM, Thomas Monjalon > wrote: > > The log history feature was deprecated in 16.07. > > The remaining empty functions are removed in 16.11. > > > > Signed-off-by: Thomas Monjalon > > --- > >

[dpdk-dev] [PATCH] doc: postpone mempool ABI breakage

2016-08-03 Thread Thomas Monjalon
2016-07-29 15:41, Thomas Monjalon: > It was planned to remove some mempool functions which are deprecated > since 16.07. > As no other mempool ABI change is planned in 16.11, it is better > to postpone and group every mempool ABI changes in 17.02. > > Signed-off-by: Thomas Monjalon Applied

[dpdk-dev] [PATCH v3] doc: autogenerate nic overview table from ini files

2016-08-03 Thread Thomas Monjalon
2016-08-03 15:32, Bruce Richardson: > On Mon, Aug 01, 2016 at 11:37:03PM +0200, Thomas Monjalon wrote: > > Hi John, > > > > 2016-07-29 12:59, John McNamara: > > > Convert the NIC feature table in the overview doc into a set of ini > > > files and add functio

[dpdk-dev] [RFC] scripts: make load-devel-config not to appear as executable

2016-08-03 Thread Thomas Monjalon
2016-08-03 11:42, Christian Ehrhardt: > On Wed, Aug 3, 2016 at 11:26 AM, Bruce Richardson < > bruce.richardson at intel.com> wrote: > > > Definitely no objection on the file mode change. > > > > For the dropping of the .sh extension, I don't think it matters much. > > However, > > given that .sh

[dpdk-dev] [PATCH] i40e: enable i40e pmd on ARM platform

2016-08-03 Thread Thomas Monjalon
2016-08-03 14:02, Jianbo Liu: > I think we can place a compiling condition before rte_rmb() to avoid > performance decrease on x86. > For example: #if defined(RTE_ARCH_ARM) || defined(RTE_ARCH_ARM64) Please could you explain why a memory barrier would be needed on ARM but not on x86? What about

[dpdk-dev] eal: map shared config into exact same address as primary process (for freebsd)

2016-08-03 Thread Thomas Monjalon
Hi, 2016-08-02 15:31, txcy uio: > Hello > > I am getting a segmentation fault on freebsd 10 due to the shared config > mismatch between the primary and secondary process - > > The following commit seems to have fixed this issue long time back on Linux > but was never ported to freebsd. >

[dpdk-dev] [RFC 0/4] Use Google Test as DPDK unit test framework

2016-08-03 Thread Thomas Monjalon
2016-08-02 21:37, Declan Doherty: > I've been trying out using google test as a possible replacement for our unit > test framework and have put to together this series of patches with help from > Anatoly as RFC to get peoples thoughts on migrating to google test. Thanks for exploring new

[dpdk-dev] [PATCH v2] lpm: remove redundant check when adding lpm rule

2016-08-03 Thread Thomas Monjalon
2016-08-02 17:04, Bruce Richardson: > Having to make this change twice shows up the fact that we are still carrying > around some version changes for older releases. Given that we are now past the > 16.07 release, the old code can probably be removed. Any volunteers to maybe > do up a patch for

[dpdk-dev] [RFC] scripts: make load-devel-config not to appear as executable

2016-08-02 Thread Thomas Monjalon
2016-08-02 15:54, Christian Ehrhardt: > Quoting the first line of the script: "#! /bin/echo must be loaded with ." > Given that we should drop the .sh file ending as well as the executable > flag - both are not needed to source the file. Hmmm, it is still a file containing some shell commands,

[dpdk-dev] DPDK Community Survey - about to close

2016-08-02 Thread Thomas Monjalon
Just 2 minutes for DPDK ;) http://surveymonkey.com/r/DPDK_Community_Survey 2016-08-02 14:39, Thomas Monjalon: > Hi all, > > That's the first time a DPDK survey is published. > It will help us in our future progress to decide what are the most > important stuff to work on.

[dpdk-dev] [dpdk-announce] DPDK Community Survey - about to close

2016-08-02 Thread Thomas Monjalon
Hi all, That's the first time a DPDK survey is published. It will help us in our future progress to decide what are the most important stuff to work on. Please do not wait to fill it out. It closes on August 4. Thanks for taking 2 minutes now to give your feedback. When you will have done your

[dpdk-dev] [PATCH v3 4/4] eal: fix end character check in --lcores argument

2016-08-02 Thread Thomas Monjalon
2016-08-02 08:22, Dai, Wei: > Hi, Thomas, Yigit > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > 2016-07-28 16:26, Ferruh Yigit: > > > On 7/27/2016 12:27 PM, Wei Dai wrote: > > > > With --lcores 'a-b at c-d', eal_parse_cores() fails becau

[dpdk-dev] [PATCH v3] doc: autogenerate nic overview table from ini files

2016-08-02 Thread Thomas Monjalon
Hi John, 2016-07-29 12:59, John McNamara: > Convert the NIC feature table in the overview doc into a set of ini > files and add functions into the Sphinx conf.py file to auto-generate > them back into an RST table. I have not reviewed the Python code which generate the RST table. It works so it

[dpdk-dev] [PATCH] examples: fix unusual-interpreter

2016-08-01 Thread Thomas Monjalon
2016-08-01 15:12, Christian Ehrhardt: > On Mon, Aug 1, 2016 at 2:50 PM, Thomas Monjalon > wrote: > > 2016-08-01 14:28, Christian Ehrhardt: > > > Due to regular lintian checks in Debian packaging it surfaced that these > > > two scripts had a space in th

[dpdk-dev] [PATCH] mbuf: remove deprecated internal function

2016-08-01 Thread Thomas Monjalon
The function __rte_mbuf_raw_alloc was reserved for internal use and has been deprecated in favor of the public function rte_mbuf_raw_alloc. It can be safely removed now. Signed-off-by: Thomas Monjalon --- lib/librte_mbuf/rte_mbuf.h | 7 --- 1 file changed, 7 deletions(-) diff --git a/lib

[dpdk-dev] [PATCH] examples: fix unusual-interpreter

2016-08-01 Thread Thomas Monjalon
2016-08-01 14:28, Christian Ehrhardt: > Due to regular lintian checks in Debian packaging it surfaced that these > two scripts had a space in their #! statement which renders it to be > human, but not shell readable. [...] > -#! /usr/bin/python2 > +#!/usr/bin/python2 I think we can have a space

[dpdk-dev] Application framework vs. library

2016-08-01 Thread Thomas Monjalon
Hi, Thanks for explaining your context. Your concerns are known under the term "usability enhancements". As they require some API changes, we won't make them in the release 16.11. But we must work on it now to be able to announce the API changes for 17.02 before the 16.11 release. 2016-08-01

[dpdk-dev] [PATCH v2] log: remove history dump

2016-07-29 Thread Thomas Monjalon
The log history feature was deprecated in 16.07. The remaining empty functions are removed in 16.11. Signed-off-by: Thomas Monjalon --- v2: fix LIBABIVER and compilation of test --- app/test/test.h | 5 +++- doc/guides/rel_notes/deprecation.rst| 3 --- doc/guides

[dpdk-dev] [PATCH] doc: postpone mempool ABI breakage

2016-07-29 Thread Thomas Monjalon
It was planned to remove some mempool functions which are deprecated since 16.07. As no other mempool ABI change is planned in 16.11, it is better to postpone and group every mempool ABI changes in 17.02. Signed-off-by: Thomas Monjalon --- doc/guides/rel_notes/deprecation.rst | 15

[dpdk-dev] [PATCH] log: remove history dump

2016-07-29 Thread Thomas Monjalon
The log history feature was deprecated in 16.07. The remaining empty functions are removed in 16.11. Signed-off-by: Thomas Monjalon --- doc/guides/rel_notes/deprecation.rst| 3 --- doc/guides/rel_notes/release_16_11.rst | 4 +++- lib/librte_eal/common/eal_common_log.c | 19

[dpdk-dev] [PATCH v1] doc: add template release notes for 16.11

2016-07-29 Thread Thomas Monjalon
2016-07-29 12:23, John McNamara: > Add template release notes for DPDK 16.11 with inline > comments and explanations of the various sections. > > Signed-off-by: John McNamara Applied as the first patch for 16.11, thanks :)

[dpdk-dev] [dpdk-announce] DPDK 16.07 released

2016-07-29 Thread Thomas Monjalon
Once again, a great release from the impressive DPDK community: http://fast.dpdk.org/rel/dpdk-16.07.tar.xz The statistics are awesome: 955 patches from 115 authors 839 files changed, 127162 insertions(+), 24668 deletions(-) There are 50 new contributors (including

[dpdk-dev] [PATCH v2] doc: announce ABI change of struct rte_port_source_params and rte_port_sink_params

2016-07-28 Thread Thomas Monjalon
2016-07-27 12:42, Thomas Monjalon: > 2016-07-27 10:08, Dumitrescu, Cristian: > > As Thomas mentioned, today is probably the last day to discuss ABI changes. > > This one is pretty small and straightforward, any issues with it? > > > > Panu had a concern that the cha

[dpdk-dev] doc: deprecate vhost-cuse

2016-07-28 Thread Thomas Monjalon
us > > > to do all kinds of extending easier. > > > > > > So here to mark vhost-cuse as deprecated in this release and will be > > > removed in the next release (v16.11). > > > > > > Signed-off-by: Yuanhan Liu > > > Acked-by: Ciara Loftus > > > Acked-by: Thomas Monjalon > > > Acked-by: Rich Lane > > Acked-by: Jan Viktorin > Acked-by: Christian Ehrhardt Applied, thanks

[dpdk-dev] [PATCH v2] doc: announce ABI change for mbuf structure

2016-07-28 Thread Thomas Monjalon
> > For 16.11, the mbuf structure will be modified implying ABI breakage. > > Some discussions already took place here: > > http://www.dpdk.org/dev/patchwork/patch/12878/ > > > > Signed-off-by: Olivier Matz > > Acked-by: Thomas Monjalon Acked-by: Konstantin

[dpdk-dev] [PATCH] doc: announce driver name changes

2016-07-28 Thread Thomas Monjalon
> > > Signed-off-by: Pablo de Lara > > Acked-by: Jerin Jacob Acked-by: Thomas Monjalon Acked-by: Maxime Coquelin Applied, thanks

[dpdk-dev] doc: announce renaming of ethdev library

2016-07-28 Thread Thomas Monjalon
the library filename as every other libraries. > > > > > > Signed-off-by: Thomas Monjalon > > > > > Acked-by: Jan Viktorin Acked-by: Christian Ehrhardt Acked-by: Konstantin Ananyev Applied

[dpdk-dev] [PATCH v2] doc: announce ABI change for rte_eth_dev structure

2016-07-28 Thread Thomas Monjalon
2016-07-28 21:55, Jerin Jacob: > On Thu, Jul 28, 2016 at 04:52:45PM +0200, Thomas Monjalon wrote: > > 2016-07-28 19:29, Jerin Jacob: > > > Above things worries me, I wouldn't have cared if the changes are not > > > comes > > > in fastpath and I don't think this

[dpdk-dev] [PATCH] doc: remove deprecation notice related to new flow types

2016-07-28 Thread Thomas Monjalon
2016-07-28 15:45, Rahul Lakkireddy: > Remove deprecation notice pertaining to introduction of new flow > types in favor of a more generic filtering infrastructure proposal. > > Signed-off-by: Rahul Lakkireddy > Signed-off-by: Kumar Sanghvi Applied, thanks

[dpdk-dev] [PATCH v3 4/4] eal: fix end character check in --lcores argument

2016-07-28 Thread Thomas Monjalon
2016-07-28 16:26, Ferruh Yigit: > On 7/27/2016 12:27 PM, Wei Dai wrote: > > With --lcores 'a-b at c-d', eal_parse_cores() fails because > > eal_parse_set() fails due to the next character after > > lcore set a-b, which is '@'and not ',' or '\0'. > > There is also a right check immediately > >

[dpdk-dev] [PATCH] doc: fix incorrect path to testpmd app

2016-07-28 Thread Thomas Monjalon
> Signed-off-by: Shreyansh Jain > Acked-by: John McNamara Applied, thanks

[dpdk-dev] [PATCH v2] maintainers: update email address for vhost and null PMD

2016-07-28 Thread Thomas Monjalon
> Signed-off-by: Tetsuya Mukawa Applied, thanks

[dpdk-dev] [PATCH v2] doc: improve wording of new features section

2016-07-28 Thread Thomas Monjalon
2016-07-27 14:26, John McNamara: > Improve the wording of some text in the "new features" section of > the release notes. > > Signed-off-by: Bruce Richardson > Signed-off-by: John McNamara Applied, thanks

[dpdk-dev] [PATCH v2] doc: update guide and release notes for mlx5

2016-07-28 Thread Thomas Monjalon
> > Signed-off-by: Olga Shern > > Acked-by: John McNamara Applied, thanks

[dpdk-dev] [PATCH v3] doc: add known issue about promiscuous mode for I40e VF

2016-07-28 Thread Thomas Monjalon
> > When use i40e linux kernel driver as host driver and DPDK handler the i40e > > VF, the promiscuous mode doesn't work in i40e VF. It is not supported by > > DPDK i40e VF driver right now. > > > > Signed-off-by: Jeff Guo > > Acked-by: John McNamara Applied, thanks

[dpdk-dev] [PATCH v2] ethdev: fix documentation for queue start/stop

2016-07-28 Thread Thomas Monjalon
> > Fix documentation for rte_eth_dev_tx/rx_queue_start/stop() functions > > > > Fixes: 2de9f8551ff9 ("ethdev: fix documentation for queue start/stop") > > > > Signed-off-by: Nikhil Rao > > Acked-by: John McNamara Applied, thanks

[dpdk-dev] [PATCH] examples/ip_pipeline: fix wrong values in flow.cfg

2016-07-28 Thread Thomas Monjalon
2016-07-27 02:35, Sankar Chokkalingam: > This configuration is example configuration for flow classification. > This fix changes the offset and mask value to compute the hash correctly. > This fix does not involve code change and do not impact compilation, > build and performance. > > Fixes:

[dpdk-dev] [PATCH v2] doc: announce ABI change for rte_eth_dev structure

2016-07-28 Thread Thomas Monjalon
2016-07-28 19:29, Jerin Jacob: > Above things worries me, I wouldn't have cared if the changes are not comes > in fastpath and I don't think this sort of issues will never get fixed any > time > soon in this community. > > So I given up. I feel something goes wrong here but I cannot understand

[dpdk-dev] [PATCH] net/i40e: fix setting RSS in i40e_recv_scattered_pkts

2016-07-28 Thread Thomas Monjalon
> > From: Dumitru Ceara > > > > The driver is incorrectly setting the RSS field in the last mbuf in the > > packet chain instead of the first. Moreover, the last mbuf might have > > already been freed if it only contained the Ethernet CRC. > > > > Also, fix the call to i40e_rxd_build_fdir to

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