[dpdk-dev] [PATCH v2] mempool: Free memzone if mempool populate phys fails

From: Nipun Gupta This patch fixes the issue of memzone not being freed incase the rte_mempool_populate_phys fails in the rte_mempool_populate_default This issue was identified when testing with OVS ~2.6 - configure the system with low memory (e.g. < 500 MB) - add bridge

[dpdk-dev] [PATCH] mempool: Free memzone if mempool populate phys fails

From: Nipun Gupta This fixes the issue of memzone not being freed, if the rte_mempool_populate_phys fails in the rte_mempool_populate_default This issue was identified when testing with OVS ~2.6 - configure the system with low memory (e.g. < 500 MB) - add bridge and dpdk

[dpdk-dev] Clarification for eth_driver changes

On 11/10/2016 11:05 AM, Shreyansh Jain wrote: > Hello David, > > On Thursday 10 November 2016 01:46 PM, David Marchand wrote: >> Hello Shreyansh, >> >> On Thu, Nov 10, 2016 at 8:26 AM, Shreyansh Jain >> wrote: >>> I need some help and clarification regarding some changes I am doing to >>>

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

On Friday 11 November 2016 05:34 PM, John McNamara wrote: [...] > -* **Improved device/driver hierarchy and generalized hotplugging** > +* **Improved device/driver hierarchy and generalized hotplugging.** > > - Device and driver relationship has been restructured by introducing generic > -

[dpdk-dev] [PATCH] doc: add sub-repositories information

Hello Ferruh, Trivial comment/suggestion: On Thursday 10 November 2016 10:57 PM, Ferruh Yigit wrote: > DPDK switched to main and sub-repositories approach, this patch > documents new approach and updates development process according. > > Signed-off-by: Ferruh Yigit > --- >

[dpdk-dev] [PATCH v1] doc: rearrange the high level documentation index

2016-11-11 13:53, Mcnamara, John: > I would prefer not to have some of these are highest level items such as > "Xen Guide", and possibly "Crypto Device Drivers". Perhaps we could push > them down a level with "Network Interface Controller Drivers" under a > "Devices and Drivers" (or similar)

[dpdk-dev] [PATCH] improve git diff

On 11/11/2016 4:21 PM, Thomas Monjalon wrote: > 2016-11-11 11:22, Ferruh Yigit: >> On 11/9/2016 3:44 PM, Thomas Monjalon wrote: >>> Sometimes git does not print the name of the function being changed >>> after @@. It happens especially after a goto label which is not indented. >>> Giving a hint

[dpdk-dev] [PATCH] improve git diff

2016-11-11 11:22, Ferruh Yigit: > On 11/9/2016 3:44 PM, Thomas Monjalon wrote: > > Sometimes git does not print the name of the function being changed > > after @@. It happens especially after a goto label which is not indented. > > Giving a hint about the languages of files .c, .h and .py > >

[dpdk-dev] [PATCH v2] mempool: Free memzone if mempool populate phys fails

Hi Hemant, On 11/11/2016 04:47 PM, Hemant Agrawal wrote: > From: Nipun Gupta > > This patch fixes the issue of memzone not being freed incase the > rte_mempool_populate_phys fails in the rte_mempool_populate_default > > This issue was identified when testing with OVS ~2.6 > - configure the

[dpdk-dev] [xen][dpdk][question] Does dpdk support xenserver 7.0 now?

Hi, I?m not sure if this is the right mailing list to ask this question, if not, please let me know. I try to compile dpdk-2.2.0 and dpdk-16.07.1 on xenserver, compilation finished, but I got error when using command `insmod rte_dom0_mm.ko`. insmod: ERROR: could not insert module

[dpdk-dev] [PATCH] doc: announce API and ABI changes for librte_eal

> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ferruh Yigit > Sent: Friday, November 11, 2016 1:05 PM > To: David Marchand ; Shreyansh Jain > > Cc: Neil Horman ; dev at dpdk.org; Thomas > Monjalon > Subject: Re: [dpdk-dev] [PATCH] doc: announce API and

[dpdk-dev] pmdinfogen issues: cross compilation for ARM fails with older host compiler

On Fri, Nov 11, 2016 at 02:48:51PM +0100, Jan Viktorin wrote: > Hello all, > > On Fri, 11 Nov 2016 10:34:39 + > Hemant Agrawal wrote: > > > Hi Neil, > >Pmdinfogen compiles with host compiler. It usages > > rte_byteorder.h of the target platform. > > This seems wierd to

[dpdk-dev] [PATCH] i40e: Fix eth_i40e_dev_init sequence on ThunderX

Hi Rao The existing rule is to try best not modify source files in the folder of base/ for each PMD. The exceptions are *_osdep.h or *_osdep.c. So for your case, please redefine the rd32/wr32 in i40e_osdep.h. (with condition compiles for ARM?, as other CPU structures may not need that.)

[dpdk-dev] [PATCH v1] doc: rearrange the high level documentation index

> -Original Message- > From: Mcnamara, John > Sent: Friday, November 11, 2016 1:46 PM > To: dev at dpdk.org > Cc: Mcnamara, John > Subject: [PATCH v1] doc: rearrange the high level documentation index > > Rearrange the order of the high level documenation index into a more > logical

[dpdk-dev] [PATCH v1] doc: rearrange the high level documentation index

Rearrange the order of the high level documenation index into a more logical sequence for a new user. Also, improve some of the high-level document names. Signed-off-by: John McNamara --- After this patch the high-level index will look like this: DPDK documentation * Getting Started

[dpdk-dev] [PATCH v2] doc: add sub-repositories information

DPDK switched to main and sub-repositories approach, this patch documents new approach and updates development process according. Signed-off-by: Ferruh Yigit Acked-by: John McNamara --- doc/guides/contributing/patches.rst | 18 +++--- 1 file changed, 15 insertions(+), 3

[dpdk-dev] [PATCH] doc: add sub-repositories information

Hi Shreyansh, On 11/11/2016 1:15 PM, Shreyansh Jain wrote: >> +* All sub-repositories merged into main repository for -rc1 and -rc2 >> versions of the release. > 'All sub-repositories *are* merged into ...'? I will send a fixed version. Thanks, ferruh

[dpdk-dev] [PATCH] doc: announce API and ABI changes for librte_eal

On 11/10/2016 3:51 PM, David Marchand wrote: > On Thu, Nov 10, 2016 at 12:17 PM, Shreyansh Jain > wrote: >> Signed-off-by: Shreyansh Jain >> --- >> doc/guides/rel_notes/deprecation.rst | 10 ++ >> 1 file changed, 10 insertions(+) >> >> diff --git a/doc/guides/rel_notes/deprecation.rst

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

Ping. On 28.10.2016 09:14, Ilya Maximets wrote: > On 25.10.2016 09:26, Ilya Maximets wrote: >> 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

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

Fix grammar, spelling and formatting of DPDK 16.11 release notes. Signed-off-by: John McNamara --- doc/guides/rel_notes/release_16_11.rst | 171 - 1 file changed, 83 insertions(+), 88 deletions(-) diff --git a/doc/guides/rel_notes/release_16_11.rst

[dpdk-dev] [PATCH] ethdev: check number of queues less than RTE_ETHDEV_QUEUE_STAT_CNTRS

> On 11 Nov 2016, at 11:29 AM, Thomas Monjalon > wrote: > > 2016-11-11 09:16, Alejandro Lucero: >> Thomas, >> >> We are wondering if you realize this patch fixes a bug with current ethdev >> code as a device can have more than RTE_ETHDEV_QUEUE_STAT_CNTRS. >> >> Maybe the commit message is

[dpdk-dev] [PATCH v7 11/21] eal/soc: implement probing of drivers

On Thursday 10 November 2016 02:56 PM, Thomas Monjalon wrote: > 2016-11-10 14:40, Shreyansh Jain: >> On Thursday 10 November 2016 01:11 PM, Jianbo Liu wrote: >>> On 10 November 2016 at 14:10, Shreyansh Jain >>> wrote: On Thursday 10 November 2016 09:00 AM, Jianbo Liu wrote: > I'm still

[dpdk-dev] pmdinfogen issues: cross compilation for ARM fails with older host compiler

Hi Neil, Pmdinfogen compiles with host compiler. It usages rte_byteorder.h of the target platform. However, if the host compiler is older than 4.8, it will be an issue during cross compilation for some platforms. e.g. if we are compiling on x86 host for ARM, x86 host compiler will

[dpdk-dev] [PATCH] mempool: Free memzone if mempool populate phys fails

2016-11-11 20:12, Hemant Agrawal: > From: Nipun Gupta > > This fixes the issue of memzone not being freed, if the > rte_mempool_populate_phys fails in the rte_mempool_populate_default > > This issue was identified when testing with OVS ~2.6 > - configure the system with low memory (e.g. < 500

[dpdk-dev] [PATCH] ethdev: check number of queues less than RTE_ETHDEV_QUEUE_STAT_CNTRS

2016-11-11 09:16, Alejandro Lucero: > Thomas, > > We are wondering if you realize this patch fixes a bug with current ethdev > code as a device can have more than RTE_ETHDEV_QUEUE_STAT_CNTRS. > > Maybe the commit message is giving the wrong impression and as you > commented, it should just focus

[dpdk-dev] [PATCH] doc: add sub-repositories information

> -Original Message- > From: Yigit, Ferruh > Sent: Thursday, November 10, 2016 5:27 PM > To: dev at dpdk.org > Cc: Mcnamara, John ; Thomas Monjalon > > Subject: [PATCH] doc: add sub-repositories information > > DPDK switched to main and sub-repositories approach, this patch documents >

[dpdk-dev] [PATCH v4] latencystats: added new library for latency stats

On 07/11/2016 21:14, Reshma Pattan wrote: [..] > Signed-off-by: Reshma Pattan Reviewed-by: Remy Horton > +static void > +metrics_display(int port_id) > +{ > +struct rte_stat_value *stats; > +struct rte_metric_name *names; Note that rte_stats_value is being renamed to

[dpdk-dev] [PATCH] doc: remove iomem and ioport handling in igb_uio

On 22/09/2016 13:44, Jianfeng Tan wrote: [..] > > Suggested-by: Yigit, Ferruh > Signed-off-by: Jianfeng Tan Acked-by: Remy Horton > +++ b/doc/guides/rel_notes/deprecation.rst > @@ -57,3 +57,8 @@ Deprecation Notices > * API will change for ``rte_port_source_params`` and

[dpdk-dev] [PATCH v2] net/qede: fix unknown speed errmsg for 25G link

- Fix to use bitmapped values in NVM configuration for speed capability advertisement. This issue is specific to 25G NIC since it is capable of 25G and 10G speeds. - Update feature list. Fixes: 64c239b7f8b7 ("net/qede: fix advertising link speed capability") Signed-off-by: Harish Patil ---

[dpdk-dev] [PATCH] ethdev: check number of queues less than RTE_ETHDEV_QUEUE_STAT_CNTRS

On Fri, Nov 11, 2016 at 9:29 AM, Thomas Monjalon wrote: > 2016-11-11 09:16, Alejandro Lucero: > > Thomas, > > > > We are wondering if you realize this patch fixes a bug with current > ethdev > > code as a device can have more than RTE_ETHDEV_QUEUE_STAT_CNTRS. > > > > Maybe the commit message is

[dpdk-dev] [PATCH] ethdev: check number of queues less than RTE_ETHDEV_QUEUE_STAT_CNTRS

Thomas, We are wondering if you realize this patch fixes a bug with current ethdev code as a device can have more than RTE_ETHDEV_QUEUE_STAT_CNTRS. Maybe the commit message is giving the wrong impression and as you commented, it should just focus on the bug it fixes and to leave for another

[dpdk-dev] [PATCH] maintainers: claim responsability for xen

Hi Thomas and Konrad, On 11/11/2016 2:59 AM, Konrad Rzeszutek Wilk wrote: > On Wed, Nov 9, 2016 at 5:03 PM, Thomas Monjalon > wrote: >> 2016-11-07 07:38, Jianfeng Tan: >>> As some users are still using xen as the hypervisor, I suggest to >>> continue support for xen in DPDK. And from 16.11, I

[dpdk-dev] [PATCH] net/ixgbe: fix link never come up problem with x552

> > From: zhao wei > > > > The links never coming up with DPDK16.11 when bring up x552 NIC, device id > > is > > 15ac.This is caused by delete some code which casing removes X550em SFP iXFI > > setup for the drivers in function ixgbe_setup_mac_link_sfp_x550em().Fix > > methord is recover the

[dpdk-dev] [PATCH 0/2] net/thunderx: add 83xx SoC support

2016-11-08 12:01, Jerin Jacob: > CN83xx is 24 core version of ThunderX ARMv8 SoC with integrated > octeon style packet and crypto accelerators. > > The standard NIC block used in 88xx/81xx also included in 83xx. > This patchset adds support for existing standard NIC block on 83xx by > adding new

[dpdk-dev] [PATCH v3] doc: arm64: document DPDK application profiling methods

> > Signed-off-by: Jerin Jacob > > Signed-off-by: John McNamara > > Acked-by: Jianbo Liu Applied, thanks

[dpdk-dev] [PATCH] doc: postpone ABI changes for Tx prepare

> > The changes for the feature "Tx prepare" should be made in version 17.02. > > > > Signed-off-by: Thomas Monjalon > > Acked-by: Tomasz Kulasek Applied

[dpdk-dev] [PATCH] doc: move testpmd guide with other tools

2016-11-10 23:18, Wiles, Keith: > > On Nov 10, 2016, at 5:02 PM, Thomas Monjalon > > wrote: > > 2016-11-10 16:11, Mcnamara, John: > >> The problem is that TestPMD is a bit of an outlier. It isn't a sample > >> application and it isn't really a test application despite the name (it is > >> more

[dpdk-dev] [PATCH] doc: fix l3fwd mode selection from compile to run time

> > The l3fwd application route lookup mode can be selected at run time but > > not at compile time. This patch corrects the statement in the doc. > > > > Fixes: d0dff9ba ("doc: sample application user guide") > > > > Signed-off-by: Reshma Pattan > > Acked-by: John McNamara Applied, thanks