Re: [dpdk-dev] [PATCH 1/7] net/qede/base: fix recovery from previous ungraceful exit

2017-08-04 Thread Mody, Rasesh
> From: Ferruh Yigit [mailto:ferruh.yi...@intel.com] > Sent: Monday, July 31, 2017 10:53 AM > To: Mody, Rasesh ; dev@dpdk.org > Cc: Dept-Eng DPDK Dev ; Thomas > Monjalon > Subject: Re: [dpdk-dev] [PATCH 1/7] net/qede/base: fix recovery from > previous ungraceful exit > > On 7/24/2017 11:19 AM, Ra

Re: [dpdk-dev] [PATCH 5/7] doc: list NPAR as supported feature in qede

2017-08-04 Thread Mody, Rasesh
> From: Rasesh Mody [mailto:rasesh.m...@cavium.com] > Sent: Monday, July 24, 2017 3:19 AM > > From: Shahed Shaikh > > Existing qede PMD code already supports NPAR feature. > So adding this in "Supported Features" section after testing it with latest > DPDK. > > Also, add myself to the list of m

Re: [dpdk-dev] [PATCH v2] examples/eventdev: fix run forever with -n param

2017-08-04 Thread Thomas Monjalon
04/08/2017 17:49, Harry van Haaren: > During the refactoring of the sample app to be more generic, the > option to set -n0 and process a huge number of packets was lost. > This commit re-adds -n0, and will process INT64_MAX number of packets. > > Fixes: adb5d5486c39 ("examples/eventdev_pipeline_sw

Re: [dpdk-dev] [dpdk-stable] [PATCH] examples/l3fwd: fix IPv6 packet type parse

2017-08-04 Thread Thomas Monjalon
04/08/2017 09:52, Qi Zhang: > Fix a typo that cause IPv6 packet type not be parsed. > > Fixes: 71a7e2424e07 ("examples/l3fwd: fix using packet type blindly") > Cc: sta...@dpdk.org > > Signed-off-by: Qi Zhang Applied, thanks

Re: [dpdk-dev] [dpdk-stable] [PATCH] net/i40e: revert fix of PF notify issue when VF not up

2017-08-04 Thread Thomas Monjalon
> > This reverts commit 035a8cf88f010c82de3040c895c801ce0849b065. > > Don't send messages to inactive VF will cause DPDK PF failing to send > > messages to kernel VF. > > With this revert, this issue will be solved. > > > > commit 035a8cf88f01 ("net/i40e: fix PF notify when VF is not up") > > Cc:

Re: [dpdk-dev] [PATCH] net/ixgbe: fix incorrect mask flag

2017-08-04 Thread Thomas Monjalon
02/08/2017 23:38, Qi Zhang: > After fail to create a flow and if this is the first flow, the > mask_added flag should be reset, or it will prevent a new flow > which require different mask be created, since the mask config > remains impact. > > Fixes: 72c135a89f8 ("net/ixgbe: create consistent fil

Re: [dpdk-dev] [PATCH] gro: fix missing includes

2017-08-04 Thread Thomas Monjalon
> > The GRO header file depends on stdint and mbuf. > > > > Spotted with devtools/check-includes.sh > > > > Fixes: e996506a1c07 ("lib/gro: add Generic Receive Offload API framework") > > > > Signed-off-by: Thomas Monjalon > > Acked-by: John McNamara Applied

Re: [dpdk-dev] [PATCH] eventdev: fix missing includes

2017-08-04 Thread Thomas Monjalon
> > The PCI helper file depends on some EAL definitions. > > Spotted with devtools/check-includes.sh: > > error: implicit declaration of function ‘rte_eal_process_type’ > > error: ‘RTE_PROC_PRIMARY’ undeclared > > error: implicit declaration of function ‘rte_socket_id’ > > > > There wa

Re: [dpdk-dev] [PATCH 1/2] ethdev: fix C11 extensions in exported header

2017-08-04 Thread Thomas Monjalon
> > Fix issues reported by check-includes.sh: > > > > rte_tm.h:473:2: error: anonymous unions are a C11 extension > > [-Werror,-Wc11-extensions] > > rte_tm.h:696:2: error: anonymous unions are a C11 extension > > [-Werror,-Wc11-extensions] > > rte_tm.h:904:2: error: anonymous unions a

Re: [dpdk-dev] [PATCH] net/ark: remove empty header file

2017-08-04 Thread Ed Czeck
On Thu, Aug 3, 2017 at 1:38 PM, Ferruh Yigit wrote: > > Signed-off-by: Ferruh Yigit > --- > doc/guides/nics/ark.rst | 3 --- > drivers/net/ark/ark_ethdev.c| 1 - > drivers/net/ark/ark_ethdev.h| 37 - > drivers/net/ark/ark_ethdev_rx.c | 1 - >

[dpdk-dev] [PATCH] eal: Copy raw strings taken from command line

2017-08-04 Thread Patrick MacArthur
Normally, command line argument strings are considered immutable, but SPDK [1] and urdma [2] construct argv arrays to pass to rte_eal_init(). These strings are allocated using malloc() and freed after DPDK initialization with free(). However, in the case of --file-prefix and --huge-dir, DPDK takes

Re: [dpdk-dev] [PATCH] eal: fix off-by-one error in hotplug add

2017-08-04 Thread Thomas Monjalon
04/08/2017 17:30, Gaetan Rivet: > snprintf returns the length it would have written had the given length > been enough, *terminating null byte excluded*. > > It will however limit the length of its writing to given length minus > one, and always put a terminating null-byte at the end of the string

[dpdk-dev] [PATCH v2] examples/eventdev: fix run forever with -n param

2017-08-04 Thread Harry van Haaren
During the refactoring of the sample app to be more generic, the option to set -n0 and process a huge number of packets was lost. This commit re-adds -n0, and will process INT64_MAX number of packets. Fixes: adb5d5486c39 ("examples/eventdev_pipeline_sw_pmd: add sample app") Signed-off-by: Harry v

[dpdk-dev] [PATCH] examples/eventdev: fix run forever with -n param

2017-08-04 Thread Harry van Haaren
During the refactoring of the sample app to be more generic, the option to set -n0 and process a huge number of packets was lost. This commit re-adds -n0, and will process INT64_MAX number of packets. Fixes: adb5d5486c39 ("examples/eventdev_pipeline_sw_pmd: add sample app") Signed-off-by: Harry v

[dpdk-dev] [PATCH v2] doc: include fail-safe PMD in release note

2017-08-04 Thread Gaetan Rivet
Signed-off-by: Gaetan Rivet --- v2: - Move the note to the proper sub-section. Thanks Ferruh doc/guides/nics/fail_safe.rst | 2 ++ doc/guides/rel_notes/release_17_08.rst | 6 +- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/doc/guides/nics/fail_safe.rst b/doc

[dpdk-dev] [PATCH] eal: fix off-by-one error in hotplug add

2017-08-04 Thread Gaetan Rivet
snprintf returns the length it would have written had the given length been enough, *terminating null byte excluded*. It will however limit the length of its writing to given length minus one, and always put a terminating null-byte at the end of the string. This must be taken into account when ca

Re: [dpdk-dev] [PATCH] doc: announce ABI change for cryptodev and ethdev

2017-08-04 Thread De Lara Guarch, Pablo
> -Original Message- > From: Akhil Goyal [mailto:akhil.go...@nxp.com] > Sent: Thursday, August 3, 2017 4:32 PM > To: dev@dpdk.org; Doherty, Declan ; > tho...@monjalon.net; Nicolau, Radu ; > avia...@mellanox.com; bor...@mellanox.com; > hemant.agra...@nxp.com; De Lara Guarch, Pablo > > Cc:

Re: [dpdk-dev] [PATCH] doc: API change notice for librte_meter

2017-08-04 Thread Thomas Monjalon
04/08/2017 17:04, Dumitrescu, Cristian: > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > 04/08/2017 16:38, Dumitrescu, Cristian: > > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > > > 04/08/2017 15:19, Cristian Dumitrescu: > > > > > +* librte_meter: The API will change to accommod

Re: [dpdk-dev] [PATCH] doc: include fail-safe PMD in release note

2017-08-04 Thread Ferruh Yigit
On 8/4/2017 4:04 PM, Gaetan Rivet wrote: > Signed-off-by: Gaetan Rivet > --- > doc/guides/nics/fail_safe.rst | 2 ++ > doc/guides/rel_notes/release_17_08.rst | 5 + > 2 files changed, 7 insertions(+) > > diff --git a/doc/guides/nics/fail_safe.rst b/doc/guides/nics/fail_safe.rst > in

Re: [dpdk-dev] [PATCH] doc: include fail-safe PMD in release note

2017-08-04 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Gaetan Rivet > Sent: Friday, August 4, 2017 4:05 PM > To: dev@dpdk.org > Cc: Yigit, Ferruh ; Gaetan Rivet > > Subject: [dpdk-dev] [PATCH] doc: include fail-safe PMD in release note > > Signed-off-by: Gaetan Rive

Re: [dpdk-dev] [PATCH] doc: API change notice for librte_meter

2017-08-04 Thread Dumitrescu, Cristian
> -Original Message- > From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Friday, August 4, 2017 3:49 PM > To: Dumitrescu, Cristian > Cc: Horton, Remy ; dev@dpdk.org; Mcnamara, > John ; Singh, Jasvinder > ; Nicolau, Radu ; > Hunt, David > Subject: Re: [PATCH] doc: API change not

[dpdk-dev] [PATCH] doc: include fail-safe PMD in release note

2017-08-04 Thread Gaetan Rivet
Signed-off-by: Gaetan Rivet --- doc/guides/nics/fail_safe.rst | 2 ++ doc/guides/rel_notes/release_17_08.rst | 5 + 2 files changed, 7 insertions(+) diff --git a/doc/guides/nics/fail_safe.rst b/doc/guides/nics/fail_safe.rst index 34ecae2..a3caf10 100644 --- a/doc/guides/nics/fail_sa

Re: [dpdk-dev] [PATCH] doc: API change notice for librte_meter

2017-08-04 Thread Thomas Monjalon
04/08/2017 16:38, Dumitrescu, Cristian: > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > 04/08/2017 15:19, Cristian Dumitrescu: > > > +* librte_meter: The API will change to accommodate configuration > > profiles. > > > + Most of the API functions will have an additional opaque parameter.

Re: [dpdk-dev] [PATCH] doc: API change notice for librte_meter

2017-08-04 Thread Dumitrescu, Cristian
> -Original Message- > From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Friday, August 4, 2017 3:29 PM > To: Dumitrescu, Cristian ; Horton, Remy > > Cc: dev@dpdk.org; Mcnamara, John ; Singh, > Jasvinder ; Nicolau, Radu > ; Hunt, David > Subject: Re: [PATCH] doc: API change not

Re: [dpdk-dev] [PATCH] doc: API change notice for librte_meter

2017-08-04 Thread Thomas Monjalon
04/08/2017 15:19, Cristian Dumitrescu: > +* librte_meter: The API will change to accommodate configuration profiles. > + Most of the API functions will have an additional opaque parameter. Why? Why opaque parameter? If you want to use it with a configuration file, you just have to implement a con

Re: [dpdk-dev] [PATCH] maintainers: volunteer to take power management

2017-08-04 Thread Bruce Richardson
On Fri, Aug 04, 2017 at 03:07:34PM +0100, David Hunt wrote: > Signed-off-by: David Hunt > --- > MAINTAINERS | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 52e048f..67e606f 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -756,6 +756,7 @@ F: lib/libr

[dpdk-dev] [PATCH v1] doc: update release notes for 17.08

2017-08-04 Thread John McNamara
Fix grammar, spelling and formatting of DPDK 17.08 release notes. Signed-off-by: John McNamara --- doc/guides/rel_notes/release_17_08.rst | 224 ++--- 1 file changed, 94 insertions(+), 130 deletions(-) diff --git a/doc/guides/rel_notes/release_17_08.rst b/doc/guides

[dpdk-dev] [PATCH] maintainers: volunteer to take power management

2017-08-04 Thread David Hunt
Signed-off-by: David Hunt --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 52e048f..67e606f 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -756,6 +756,7 @@ F: lib/librte_kvargs/ F: test/test/test_kvargs.c Power management +M: David Hunt F: l

[dpdk-dev] [PATCH v2] devtools: add script to get maintainers from patch

2017-08-04 Thread Ferruh Yigit
This is a wrapper to Linux kernel get_maintainer.pl file and only supports parsing MAINTAINERS file (no git fallback etc..) Requires DPDK_GETMAINTAINER_PATH devel config option set, please check devtools/load-devel-config. DPDK_GETMAINTAINER_PATH should be full path to the get_maintainer.pl scrip

[dpdk-dev] [RFC 10/10] net/i40e: set register for no drop

2017-08-04 Thread David Hunt
See the XL710 controller datasheet for more information on this register Signed-off-by: Nemanja Marjanovic Signed-off-by: Rory Sexton Signed-off-by: David Hunt --- drivers/net/i40e/i40e_ethdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/i40e/i40e_ethdev.c

[dpdk-dev] [RFC 06/10] examples/vm_power_mgr: add scale to medium freq fn

2017-08-04 Thread David Hunt
Signed-off-by: Nemanja Marjanovic Signed-off-by: Rory Sexton Signed-off-by: David Hunt --- examples/vm_power_manager/power_manager.c | 15 +++ examples/vm_power_manager/power_manager.h | 13 + 2 files changed, 28 insertions(+) diff --git a/examples/vm_power_manager/powe

[dpdk-dev] [RFC 09/10] examples/vm_power_mgr: set MAC address of VF

2017-08-04 Thread David Hunt
We need to set vf mac from the host, so that they will be in sync on the guest and the host. Otherwise, we'll have a random mac on the guest, and a 00:00:00:00:00:00 mac on the host. Signed-off-by: David Hunt --- examples/vm_power_manager/main.c | 58 +++- 1 f

[dpdk-dev] [RFC 02/10] net/i40e: add API to get received packet count

2017-08-04 Thread David Hunt
Signed-off-by: Nemanja Marjanovic Signed-off-by: Rory Sexton Signed-off-by: David Hunt --- drivers/net/i40e/i40e_ethdev.c | 1 + drivers/net/i40e/i40e_rxtx.c | 10 ++ drivers/net/i40e/i40e_rxtx.h | 1 + lib/librte_ether/rte_ethdev.h | 19 +++ 4 files changed, 31 i

[dpdk-dev] [RFC 08/10] examples/guest_cli: add send policy to host

2017-08-04 Thread David Hunt
Here we're adding an example of setting up a policy, and allowing the vm_cli_guest app to send it to the host using the cli command "send_policy now" Signed-off-by: Nemanja Marjanovic Signed-off-by: Rory Sexton Signed-off-by: David Hunt --- .../guest_cli/vm_power_cli_guest.c |

[dpdk-dev] [RFC 07/10] examples/vm_power_mgr: add port initialisation

2017-08-04 Thread David Hunt
We need to initialise the port's we're monitoring to be able to see the throughput. Signed-off-by: Nemanja Marjanovic Signed-off-by: David Hunt --- examples/vm_power_manager/main.c | 220 +++ 1 file changed, 220 insertions(+) diff --git a/examples/vm_power_m

[dpdk-dev] [RFC 01/10] net/i40e: add API to convert VF Id to PF Id

2017-08-04 Thread David Hunt
Need a way to convert a vf id to a pf id on the host so as to query the pf for relevant statistics which are used for the frequency changes in the vm_power_manager app. Used when profiles are passed down from the guest to the host, allowing the host to map the vfs to pfs. Signed-off-by: Nemanja Ma

[dpdk-dev] [RFC 05/10] examples/vm_power_mgr: add policy to channels

2017-08-04 Thread David Hunt
Signed-off-by: Nemanja Marjanovic Signed-off-by: Rory Sexton Signed-off-by: David Hunt --- examples/vm_power_manager/channel_monitor.c | 353 +++- examples/vm_power_manager/channel_monitor.h | 18 ++ 2 files changed, 315 insertions(+), 56 deletions(-) diff --git a/exam

[dpdk-dev] [RFC 04/10] examples/vm_power_mgr: add vcpu to pcpu mapping

2017-08-04 Thread David Hunt
Signed-off-by: Nemanja Marjanovic Signed-off-by: Rory Sexton Signed-off-by: David Hunt --- examples/vm_power_manager/channel_manager.c | 55 + examples/vm_power_manager/channel_manager.h | 16 - 2 files changed, 70 insertions(+), 1 deletion(-) diff --git a/e

[dpdk-dev] [RFC 03/10] lib/librte_power: add extra msg type for policies

2017-08-04 Thread David Hunt
Signed-off-by: Nemanja Marjanovic Signed-off-by: Rory Sexton Signed-off-by: David Hunt --- lib/librte_power/channel_commands.h | 33 + 1 file changed, 33 insertions(+) diff --git a/lib/librte_power/channel_commands.h b/lib/librte_power/channel_commands.h index

[dpdk-dev] [RFC] Policy Based Power Control for Guest

2017-08-04 Thread David Hunt
This RFC patchset adds the facility for a guest VM to send a policy down to the host that will allow the host to scale up/down cpu frequencies depending on the policy criteria independently of the DPDK app running in the guest. This differs from the previous vm_power implementation where individua

Re: [dpdk-dev] [PATCH v2] doc: deprecate Xen support

2017-08-04 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Jianfeng Tan > Sent: Thursday, August 3, 2017 9:38 PM > To: dev@dpdk.org > Cc: Tan, Jianfeng ; Thomas Monjalon > > Subject: [dpdk-dev] [PATCH v2] doc: deprecate Xen support > > Following the calls on the mailing

Re: [dpdk-dev] [PATCH] doc: remove planned devargs changes for v17.08

2017-08-04 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Gaetan Rivet > Sent: Friday, August 4, 2017 2:29 PM > To: dev@dpdk.org > Cc: Gaetan Rivet > Subject: [dpdk-dev] [PATCH] doc: remove planned devargs changes for v17.08 > > These changes have been implemented. >

Re: [dpdk-dev] [PATCH 3/3] doc: update FAQ with new testpmd default burst size

2017-08-04 Thread Mcnamara, John
> -Original Message- > From: Richardson, Bruce > Sent: Friday, August 4, 2017 1:44 PM > To: Mcnamara, John > Cc: dev@dpdk.org; Richardson, Bruce > Subject: [PATCH 3/3] doc: update FAQ with new testpmd default burst size > > Signed-off-by: Bruce Richardson Acked-by: John McNamara

Re: [dpdk-dev] [PATCH 2/3] doc: remove unneeded TLB from title of FAQ question 2

2017-08-04 Thread Mcnamara, John
> -Original Message- > From: Richardson, Bruce > Sent: Friday, August 4, 2017 1:44 PM > To: Mcnamara, John > Cc: dev@dpdk.org; Richardson, Bruce > Subject: [PATCH 2/3] doc: remove unneeded TLB from title of FAQ question 2 > > Hugepages are just that, hugepages. The TLB is a sort of cac

Re: [dpdk-dev] [PATCH 2/3] doc: remove unneeded TLB from title of FAQ question 2

2017-08-04 Thread Mcnamara, John
> -Original Message- > From: Richardson, Bruce > Sent: Friday, August 4, 2017 1:44 PM > To: Mcnamara, John > Cc: dev@dpdk.org; Richardson, Bruce > Subject: [PATCH 2/3] doc: remove unneeded TLB from title of FAQ question 2 > > Hugepages are just that, hugepages. The TLB is a sort of cac

Re: [dpdk-dev] [PATCH 1/3] doc: clarify documentation on running as non-root

2017-08-04 Thread Mcnamara, John
> -Original Message- > From: Richardson, Bruce > Sent: Friday, August 4, 2017 1:44 PM > To: Mcnamara, John > Cc: dev@dpdk.org; Richardson, Bruce > Subject: [PATCH 1/3] doc: clarify documentation on running as non-root > > Since modern kernels don't allow unprivileged processes to read

Re: [dpdk-dev] [PATCH 1/2] ethdev: fix C11 extensions in exported header

2017-08-04 Thread Dumitrescu, Cristian
> -Original Message- > From: Adrien Mazarguil [mailto:adrien.mazarg...@6wind.com] > Sent: Friday, August 4, 2017 12:56 PM > To: Thomas Monjalon > Cc: dev@dpdk.org; Dumitrescu, Cristian > Subject: [PATCH 1/2] ethdev: fix C11 extensions in exported header > > Fix issues reported by check

Re: [dpdk-dev] [PATCH v7] doc: document NIC features

2017-08-04 Thread Mcnamara, John
> -Original Message- > From: Yigit, Ferruh > Sent: Friday, August 4, 2017 2:06 PM > To: Mcnamara, John > Cc: dev@dpdk.org; Yigit, Ferruh ; Olivier Matz > > Subject: [PATCH v7] doc: document NIC features > > Document NIC features, add more information about them and add more > implement

[dpdk-dev] [PATCH 2/3] doc: remove unneeded TLB from title of FAQ question 2

2017-08-04 Thread Bruce Richardson
Hugepages are just that, hugepages. The TLB is a sort of cache for doing address translation and does not need to be referenced in the title, since "TLB hugepages" doesn't make any sense (as there are no non-TLB hugepages that you can allocate). Signed-off-by: Bruce Richardson --- doc/guides/faq

[dpdk-dev] [PATCH 3/3] doc: update FAQ with new testpmd default burst size

2017-08-04 Thread Bruce Richardson
Signed-off-by: Bruce Richardson --- doc/guides/faq/faq.rst | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/doc/guides/faq/faq.rst b/doc/guides/faq/faq.rst index f21aff4c5..dac805099 100644 --- a/doc/guides/faq/faq.rst +++ b/doc/guides/faq/faq.rst @@ -115,16 +115

[dpdk-dev] [PATCH 0/3] small documentation updates

2017-08-04 Thread Bruce Richardson
Small changes to the Linux GSG and the FAQ Bruce Richardson (3): doc: clarify documentation on running as non-root doc: remove unneeded TLB from title of FAQ question 2 doc: update FAQ with new testpmd default burst size doc/guides/faq/faq.rst | 18 +- doc/gui

[dpdk-dev] [PATCH 1/3] doc: clarify documentation on running as non-root

2017-08-04 Thread Bruce Richardson
Since modern kernels don't allow unprivileged processes to read the pagemaps file, the instructions on running as non-root are out-of-date. Add a note clarifying that they will only work with earlier kernel versions. Signed-off-by: Bruce Richardson --- doc/guides/linux_gsg/enable_func.rst | 8 ++

[dpdk-dev] [PATCH] doc: announce rte_devargs changes in 17.11

2017-08-04 Thread Gaetan Rivet
Signed-off-by: Gaetan Rivet --- doc/guides/rel_notes/deprecation.rst | 12 1 file changed, 12 insertions(+) diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst index 7024c81..9e063f1 100644 --- a/doc/guides/rel_notes/deprecation.rst +++ b/doc/gu

[dpdk-dev] [PATCH] doc: remove planned devargs changes for v17.08

2017-08-04 Thread Gaetan Rivet
These changes have been implemented. Signed-off-by: Gaetan Rivet --- doc/guides/rel_notes/deprecation.rst | 7 --- 1 file changed, 7 deletions(-) diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst index 72aa404..7024c81 100644 --- a/doc/guides/rel_note

[dpdk-dev] [PATCH] doc: API change notice for librte_meter

2017-08-04 Thread Cristian Dumitrescu
Signed-off-by: Cristian Dumitrescu Acked-by: John McNamara Acked-by: Jasvinder Singh Acked-by: Radu Nicolau Acked-by: David Hunt --- doc/guides/rel_notes/deprecation.rst | 3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/depreca

[dpdk-dev] [PATCH v7] doc: document NIC features

2017-08-04 Thread Ferruh Yigit
Document NIC features, add more information about them and add more implementation related support. Signed-off-by: Ferruh Yigit Signed-off-by: John McNamara Reviewed-by: Andrew Rybchenko Acked-by: Shreyansh Jain --- Cc: Olivier Matz v7: * Rename file * Add to the index * Minor updates. v6:

Re: [dpdk-dev] [RFC] ethdev: add ioctl-like API to control device specific features

2017-08-04 Thread Bruce Richardson
On Fri, Aug 04, 2017 at 12:58:01PM +0100, Ferruh Yigit wrote: > On 8/3/2017 8:53 PM, Thomas Monjalon wrote: > > 03/08/2017 18:15, Stephen Hemminger: > >> On Thu, 3 Aug 2017 14:21:38 +0100 > >> Bruce Richardson wrote: > >> > >>> On Thu, Aug 03, 2017 at 01:21:35PM +0100, Chilikin, Andrey wrote: > >>

Re: [dpdk-dev] [PATCH 2/2] cryptodev: fix C11 extensions in exported header

2017-08-04 Thread De Lara Guarch, Pablo
> -Original Message- > From: Adrien Mazarguil [mailto:adrien.mazarg...@6wind.com] > Sent: Friday, August 4, 2017 12:56 PM > To: Thomas Monjalon > Cc: dev@dpdk.org; De Lara Guarch, Pablo > > Subject: [PATCH 2/2] cryptodev: fix C11 extensions in exported header > > Fix issues reported by

Re: [dpdk-dev] [PATCH] eventdev: fix missing includes

2017-08-04 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Friday, August 4, 2017 11:21 AM > To: jerin.ja...@caviumnetworks.com > Cc: dev@dpdk.org > Subject: [dpdk-dev] [PATCH] eventdev: fix missing includes > > The PCI helper file depends on some

Re: [dpdk-dev] [PATCH] gro: fix missing includes

2017-08-04 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Friday, August 4, 2017 11:12 AM > To: Hu, Jiayu > Cc: dev@dpdk.org > Subject: [dpdk-dev] [PATCH] gro: fix missing includes > > The GRO header file depends on stdint and mbuf. > > Spotted

Re: [dpdk-dev] [RFC] ethdev: add ioctl-like API to control device specific features

2017-08-04 Thread Ferruh Yigit
On 8/3/2017 8:53 PM, Thomas Monjalon wrote: > 03/08/2017 18:15, Stephen Hemminger: >> On Thu, 3 Aug 2017 14:21:38 +0100 >> Bruce Richardson wrote: >> >>> On Thu, Aug 03, 2017 at 01:21:35PM +0100, Chilikin, Andrey wrote: To control some device-specific features public device-specific functions

[dpdk-dev] [PATCH 2/2] cryptodev: fix C11 extensions in exported header

2017-08-04 Thread Adrien Mazarguil
Fix issues reported by check-includes.sh: rte_crypto_sym.h:586:3: error: anonymous structs are a C11 extension [-Werror,-Wc11-extensions] rte_crypto_sym.h:510:2: error: anonymous unions are a C11 extension [-Werror,-Wc11-extensions] Fixes: b59502a5e3d0 ("cryptodev: add AEAD parameters

[dpdk-dev] [PATCH 1/2] ethdev: fix C11 extensions in exported header

2017-08-04 Thread Adrien Mazarguil
Fix issues reported by check-includes.sh: rte_tm.h:473:2: error: anonymous unions are a C11 extension [-Werror,-Wc11-extensions] rte_tm.h:696:2: error: anonymous unions are a C11 extension [-Werror,-Wc11-extensions] rte_tm.h:904:2: error: anonymous unions are a C11 extension [-We

Re: [dpdk-dev] [PATCH v6] doc: document NIC features

2017-08-04 Thread Ferruh Yigit
On 8/4/2017 12:11 PM, Mcnamara, John wrote: >> -Original Message- >> From: Thomas Monjalon [mailto:tho...@monjalon.net] >> Sent: Friday, August 4, 2017 10:32 AM >> To: Yigit, Ferruh >> Cc: dev@dpdk.org; Mcnamara, John ; Olivier Matz >> >> Subject: Re: [dpdk-dev] [PATCH v6] doc: document N

Re: [dpdk-dev] [PATCH v6] doc: document NIC features

2017-08-04 Thread Mcnamara, John
> -Original Message- > From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Friday, August 4, 2017 10:32 AM > To: Yigit, Ferruh > Cc: dev@dpdk.org; Mcnamara, John ; Olivier Matz > > Subject: Re: [dpdk-dev] [PATCH v6] doc: document NIC features > > 04/08/2017 10:56, Ferruh Yigit: >

[dpdk-dev] [PATCH v2] eventdev: fix missing includes

2017-08-04 Thread Thomas Monjalon
The PCI helper file depends on some EAL definitions. Spotted with devtools/check-includes.sh: error: implicit declaration of function ‘rte_eal_process_type’ error: ‘RTE_PROC_PRIMARY’ undeclared error: implicit declaration of function ‘rte_socket_id’ There was also this erro

Re: [dpdk-dev] [PATCH] eventdev: fix missing includes

2017-08-04 Thread Thomas Monjalon
04/08/2017 12:35, Jerin Jacob: > -Original Message- > > Date: Fri, 4 Aug 2017 12:20:35 +0200 > > From: Thomas Monjalon > > To: jerin.ja...@caviumnetworks.com > > Cc: dev@dpdk.org > > Subject: [PATCH] eventdev: fix missing includes > > X-Mailer: git-send-email 2.13.2 > > > > The PCI helpe

Re: [dpdk-dev] [PATCH v3 1/3] mk: add sensible default target with defconfig

2017-08-04 Thread Hunt, David
On 4/8/2017 11:05 AM, Thomas Monjalon wrote: 04/08/2017 11:53, Hunt, David: On 4/8/2017 10:36 AM, Thomas Monjalon wrote: 04/08/2017 10:22, Hunt, David: From: Thomas Monjalon [mailto:tho...@monjalon.net] 07/06/2017 16:37, David Hunt: Users can now use 'make defconfig' to generate a configurat

[dpdk-dev] [PATCH v5] mk: add sensible default target with defconfig

2017-08-04 Thread David Hunt
Users can now use 'make defconfig' to generate a configuration using the most appropriate defaults for the current machine. arch taken from uname -m machine defaults to native execenv is taken from uname, Linux=linuxapp, otherwise bsdapp toolchain is taken from $CC -v to see which compile

Re: [dpdk-dev] [PATCH] eventdev: fix missing includes

2017-08-04 Thread Jerin Jacob
-Original Message- > Date: Fri, 4 Aug 2017 12:20:35 +0200 > From: Thomas Monjalon > To: jerin.ja...@caviumnetworks.com > Cc: dev@dpdk.org > Subject: [PATCH] eventdev: fix missing includes > X-Mailer: git-send-email 2.13.2 > > The PCI helper file depends on some EAL definitions. > Spotted

[dpdk-dev] [PATCH v4] mk: add sensible default target with defconfig

2017-08-04 Thread David Hunt
Users can now use 'make defconfig' to generate a configuration using the most appropriate defaults for the current machine. arch taken from uname -m machine defaults to native execenv is taken from uname, Linux=linuxapp, otherwise bsdapp toolchain is taken from $CC -v to see which compile

[dpdk-dev] [PATCH] eventdev: fix missing includes

2017-08-04 Thread Thomas Monjalon
The PCI helper file depends on some EAL definitions. Spotted with devtools/check-includes.sh: error: implicit declaration of function ‘rte_eal_process_type’ error: ‘RTE_PROC_PRIMARY’ undeclared error: implicit declaration of function ‘rte_socket_id’ There was also this erro

[dpdk-dev] [PATCH] gro: fix missing includes

2017-08-04 Thread Thomas Monjalon
The GRO header file depends on stdint and mbuf. Spotted with devtools/check-includes.sh Fixes: e996506a1c07 ("lib/gro: add Generic Receive Offload API framework") Signed-off-by: Thomas Monjalon --- lib/librte_gro/rte_gro.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lib/librte_gro/r

Re: [dpdk-dev] [PATCH v6] doc: document NIC features

2017-08-04 Thread Thomas Monjalon
04/08/2017 12:04, Ferruh Yigit: > On 8/4/2017 10:32 AM, Thomas Monjalon wrote: > > 04/08/2017 10:56, Ferruh Yigit: > >> On 8/3/2017 11:57 PM, Thomas Monjalon wrote: > >>> 01/08/2017 17:23, Ferruh Yigit: > +:orphan: > + > +.. _nic_overview_features: > >>> > >>> Why have you done an or

Re: [dpdk-dev] [RFC] ethdev: add ioctl-like API to control device specific features

2017-08-04 Thread Thomas Monjalon
04/08/2017 11:59, Chilikin, Andrey: > > 03/08/2017 18:15, Stephen Hemminger: > > > On Thu, 3 Aug 2017 14:21:38 +0100 > > > Bruce Richardson wrote: > > > > > > > On Thu, Aug 03, 2017 at 01:21:35PM +0100, Chilikin, Andrey wrote: > > > > > To control some device-specific features public device-specif

Re: [dpdk-dev] [PATCH v3 1/3] mk: add sensible default target with defconfig

2017-08-04 Thread Thomas Monjalon
04/08/2017 11:53, Hunt, David: > > On 4/8/2017 10:36 AM, Thomas Monjalon wrote: > > 04/08/2017 10:22, Hunt, David: > >> From: Thomas Monjalon [mailto:tho...@monjalon.net] > >> 07/06/2017 16:37, David Hunt: > >>> Users can now use 'make defconfig' to generate a configuration using > >>> the most ap

Re: [dpdk-dev] [PATCH v6] doc: document NIC features

2017-08-04 Thread Ferruh Yigit
On 8/4/2017 10:32 AM, Thomas Monjalon wrote: > 04/08/2017 10:56, Ferruh Yigit: >> On 8/3/2017 11:57 PM, Thomas Monjalon wrote: >>> 01/08/2017 17:23, Ferruh Yigit: +:orphan: + +.. _nic_overview_features: >>> >>> Why have you done an orphan page? >>> I think it should be in doc/guides/

Re: [dpdk-dev] [RFC] ethdev: add ioctl-like API to control device specific features

2017-08-04 Thread Chilikin, Andrey
> 03/08/2017 18:15, Stephen Hemminger: > > On Thu, 3 Aug 2017 14:21:38 +0100 > > Bruce Richardson wrote: > > > > > On Thu, Aug 03, 2017 at 01:21:35PM +0100, Chilikin, Andrey wrote: > > > > To control some device-specific features public device-specific > functions > > > > rte_pmd_*.h are used. > >

[dpdk-dev] [PATCH v3] doc: add prog_guide for the GRO library

2017-08-04 Thread Jiayu Hu
Add prog_guide doc to explain the design of the GRO library. Signed-off-by: Jiayu Hu Acked-by: John McNamara --- changes in v3: - change "inputted" to "input" and update the text changes in v2: - update the text and remove changes to source_org.rst MAINTAINERS

Re: [dpdk-dev] DPDK qos support for 40G port

2017-08-04 Thread Dumitrescu, Cristian
Hi Kevin, > -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Kevin Yan > Sent: Friday, August 4, 2017 2:55 AM > To: dev@dpdk.org > Subject: [dpdk-dev] DPDK qos support for 40G port > > Hi, > I have a question about dpdk qos feature, in rte_sched.h

Re: [dpdk-dev] [PATCH v3 1/3] mk: add sensible default target with defconfig

2017-08-04 Thread Hunt, David
On 4/8/2017 10:36 AM, Thomas Monjalon wrote: 04/08/2017 10:22, Hunt, David: From: Thomas Monjalon [mailto:tho...@monjalon.net] 07/06/2017 16:37, David Hunt: Users can now use 'make defconfig' to generate a configuration using the most appropriate defaults for the current machine. arch tak

Re: [dpdk-dev] [PATCH v3 1/3] mk: add sensible default target with defconfig

2017-08-04 Thread Thomas Monjalon
04/08/2017 10:22, Hunt, David: > From: Thomas Monjalon [mailto:tho...@monjalon.net] > 07/06/2017 16:37, David Hunt: > > Users can now use 'make defconfig' to generate a configuration using > > the most appropriate defaults for the current machine. > > > > > > arch taken from uname -m > > ma

Re: [dpdk-dev] [PATCH v2] doc: add prog_guide for the GRO library

2017-08-04 Thread Jiayu Hu
Hi John, On Fri, Aug 04, 2017 at 05:11:18PM +0800, Mcnamara, John wrote: > > -Original Message- > > From: Hu, Jiayu > > Sent: Friday, August 4, 2017 9:21 AM > > To: dev@dpdk.org > > Cc: tho...@monjalon.net; Mcnamara, John ; Hu, > > Jiayu > > Subject: [PATCH v2] doc: add prog_guide for the

Re: [dpdk-dev] [PATCH v6] doc: document NIC features

2017-08-04 Thread Thomas Monjalon
04/08/2017 10:56, Ferruh Yigit: > On 8/3/2017 11:57 PM, Thomas Monjalon wrote: > > 01/08/2017 17:23, Ferruh Yigit: > >> +:orphan: > >> + > >> +.. _nic_overview_features: > > > > Why have you done an orphan page? > > I think it should be in doc/guides/nics/overview.rst > > There is a link from ni

Re: [dpdk-dev] [PATCH] doc: announce ABI change for cryptodev and ethdev

2017-08-04 Thread De Lara Guarch, Pablo
> -Original Message- > From: Akhil Goyal [mailto:akhil.go...@nxp.com] > Sent: Thursday, August 3, 2017 4:32 PM > To: dev@dpdk.org; Doherty, Declan ; > tho...@monjalon.net; Nicolau, Radu ; > avia...@mellanox.com; bor...@mellanox.com; > hemant.agra...@nxp.com; De Lara Guarch, Pablo > > Cc:

Re: [dpdk-dev] [PATCH v2] doc: add prog_guide for the GRO library

2017-08-04 Thread Mcnamara, John
> -Original Message- > From: Hu, Jiayu > Sent: Friday, August 4, 2017 9:21 AM > To: dev@dpdk.org > Cc: tho...@monjalon.net; Mcnamara, John ; Hu, > Jiayu > Subject: [PATCH v2] doc: add prog_guide for the GRO library > > Add prog_guide doc to explain the design of the GRO library. > ... >

Re: [dpdk-dev] [PATCH v6] doc: document NIC features

2017-08-04 Thread Ferruh Yigit
On 8/3/2017 11:57 PM, Thomas Monjalon wrote: > 01/08/2017 17:23, Ferruh Yigit: >> +:orphan: >> + >> +.. _nic_overview_features: > > Why have you done an orphan page? > I think it should be in doc/guides/nics/overview.rst There is a link from nics/overview.rst to this file. But this file is not i

Re: [dpdk-dev] [PATCH] net/i40e: revert fix of PF notify issue when VF not up

2017-08-04 Thread Xing, Beilei
> -Original Message- > From: Li, Xiaoyun > Sent: Friday, August 4, 2017 11:50 PM > To: Wu, Jingjing ; Xing, Beilei > Cc: dev@dpdk.org; Li, Xiaoyun ; sta...@dpdk.org > Subject: [PATCH] net/i40e: revert fix of PF notify issue when VF not up > > This reverts commit 035a8cf88f010c82de3040c895

Re: [dpdk-dev] [PATCH v3 1/3] mk: add sensible default target with defconfig

2017-08-04 Thread Hunt, David
-Original Message- From: Thomas Monjalon [mailto:tho...@monjalon.net] Sent: Thursday, 3 August, 2017 11:40 PM To: Hunt, David Cc: dev@dpdk.org; shreyansh.j...@nxp.com Subject: Re: [dpdk-dev] [PATCH v3 1/3] mk: add sensible default target with defconfig 07/06/2017 16:37, David Hunt: > U

[dpdk-dev] [PATCH v2] doc: add prog_guide for the GRO library

2017-08-04 Thread Jiayu Hu
Add prog_guide doc to explain the design of the GRO library. Signed-off-by: Jiayu Hu --- changes in v2: - update the text and remove changes to source_org.rst MAINTAINERS| 1 + .../prog_guide/generic_receive_offload_lib.rst | 159 +++

Re: [dpdk-dev] [PATCH] doc: fix a typo in kni documentation

2017-08-04 Thread Mcnamara, John
> -Original Message- > From: Jerin Jacob [mailto:jerin.ja...@caviumnetworks.com] > Sent: Friday, August 4, 2017 4:27 AM > To: dev@dpdk.org > Cc: Yigit, Ferruh ; Mcnamara, John > ; Jerin Jacob > Subject: [dpdk-dev] [PATCH] doc: fix a typo in kni documentation > > Fixes: fc1f2750a3ec ("do

[dpdk-dev] [PATCH] net/i40e: revert fix of PF notify issue when VF not up

2017-08-04 Thread Xiaoyun Li
This reverts commit 035a8cf88f010c82de3040c895c801ce0849b065. Don't send messages to inactive VF will cause DPDK PF failing to send messages to kernel VF. With this revert, this issue will be solved. commit 035a8cf88f01 ("net/i40e: fix PF notify when VF is not up") Cc: sta...@dpdk.org Signed-off-