Re: [dpdk-dev] Further fun with ABI tracking

2017-02-23 Thread Christian Ehrhardt
On Thu, Feb 23, 2017 at 7:48 PM, Ferruh Yigit wrote: > Can you please describe this option more? > Of course, I happy about any engagement/discussion to have on that. Much better than silently staying in queue. > Does is mean for each DPDK release, distro will release all libraries? > First o

Re: [dpdk-dev] [PATCH 0/8] QoS features on i40e

2017-02-23 Thread Lu, Wenzhuo
Hi Stephen, > -Original Message- > From: Stephen Hemminger [mailto:step...@networkplumber.org] > Sent: Friday, February 24, 2017 2:55 PM > To: Lu, Wenzhuo > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH 0/8] QoS features on i40e > > On Fri, 24 Feb 2017 11:24:27 +0800 > Wenzhuo Lu wro

Re: [dpdk-dev] [PATCH 0/8] QoS features on i40e

2017-02-23 Thread Stephen Hemminger
On Fri, 24 Feb 2017 11:24:27 +0800 Wenzhuo Lu wrote: > This patch set enables several QoS features on i40e. > 1, VF max bandwidth setting. > 2, TC min bandwidth setting on a VF. > 3, TC max bandwidth setting on a VF. > 4, TC TX scheduling mode setting. > As there're no new interface between PF an

Re: [dpdk-dev] [PATCH] net/ixgbevf: reset hardware when stopping port

2017-02-23 Thread Lu, Wenzhuo
Hi Olivier, Fengtian, > -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Olivier Matz > Sent: Thursday, January 12, 2017 12:52 AM > To: dev@dpdk.org; Zhang, Helin; Ananyev, Konstantin > Cc: Guo Fengtian; sta...@dpdk.org; David Marchand > Subject: [dpdk-dev] [PATCH

[dpdk-dev] [PATCH 8/8] app/testpmd: set TC strict link priority mode

2017-02-23 Thread Wenzhuo Lu
Add a CLI to set some TCs' strict link priority mode on a physical port. Signed-off-by: Wenzhuo Lu --- app/test-pmd/cmdline.c | 62 + doc/guides/testpmd_app_ug/testpmd_funcs.rst | 7 2 files changed, 69 insertions(+) diff --git a/app/test-p

[dpdk-dev] [PATCH 7/8] app/testpmd: set VF TC TX max bandwidth

2017-02-23 Thread Wenzhuo Lu
Add CLI to set a specific TC's max bandwidth on a specific VF from PF. Signed-off-by: Wenzhuo Lu --- app/test-pmd/cmdline.c | 64 + doc/guides/testpmd_app_ug/testpmd_funcs.rst | 7 2 files changed, 71 insertions(+) diff --git a/app/test-pmd

[dpdk-dev] [PATCH 6/8] app/testpmd: set VF TC TX min bandwidth

2017-02-23 Thread Wenzhuo Lu
Add CLI to set TCs' min bandwidth on a specific VF from PF. Signed-off-by: Wenzhuo Lu --- app/test-pmd/cmdline.c | 121 doc/guides/testpmd_app_ug/testpmd_funcs.rst | 7 ++ 2 files changed, 128 insertions(+) diff --git a/app/test-pmd/cmdline.c

[dpdk-dev] [PATCH 3/8] net/i40e: set VF TC max bandwidth from PF

2017-02-23 Thread Wenzhuo Lu
Set a specific TC's max bandwidth on a VF. User can call the API to set VF TC's max bandwidth from PF. Signed-off-by: Wenzhuo Lu --- doc/guides/nics/i40e.rst | 11 +++ doc/guides/rel_notes/release_17_05.rst| 5 ++ drivers/net/i40e/i40e_ethdev.c| 118 ++

[dpdk-dev] [PATCH 5/8] app/testpmd: set VF TX max bandwidth

2017-02-23 Thread Wenzhuo Lu
Add CLI to set a specific VF's TX max bandwidth from PF. Signed-off-by: Wenzhuo Lu --- app/test-pmd/cmdline.c | 96 + doc/guides/testpmd_app_ug/testpmd_funcs.rst | 7 +++ 2 files changed, 103 insertions(+) diff --git a/app/test-pmd/cmdline.c b/a

[dpdk-dev] [PATCH 4/8] net/i40e: set TC strict priority mode

2017-02-23 Thread Wenzhuo Lu
Set some TCs to strict priority mode. It's a global setting on a physical port. Signed-off-by: Wenzhuo Lu --- doc/guides/nics/i40e.rst | 10 +++ doc/guides/rel_notes/release_17_05.rst| 7 ++ drivers/net/i40e/i40e_ethdev.c| 119 ++ d

[dpdk-dev] [PATCH 2/8] net/i40e: allocate VF TC bandwidth from PF

2017-02-23 Thread Wenzhuo Lu
Allocate all TCs' relative bandwidth (percentage) on a specific VF. It can be taken as relative min bandwidth setting. User can call the API to set VF TC's min bandwidth from PF. Signed-off-by: Wenzhuo Lu --- doc/guides/rel_notes/release_17_05.rst| 5 ++ drivers/net/i40e/i40e_ethdev.c

[dpdk-dev] [PATCH 1/8] net/i40e: set VF max bandwidth from PF

2017-02-23 Thread Wenzhuo Lu
Support setting VF max bandwidth from PF. User can call the API on PF to set a specific VF's max bandwidth. Signed-off-by: Wenzhuo Lu --- doc/guides/rel_notes/release_17_05.rst| 3 + drivers/net/i40e/i40e_ethdev.c| 94 +++ drivers/net/i40e/rte_pmd_i40

[dpdk-dev] [PATCH 0/8] QoS features on i40e

2017-02-23 Thread Wenzhuo Lu
This patch set enables several QoS features on i40e. 1, VF max bandwidth setting. 2, TC min bandwidth setting on a VF. 3, TC max bandwidth setting on a VF. 4, TC TX scheduling mode setting. As there're no new interface between PF and VF defined, all the settings for VF are done on PF. PF acts as a

[dpdk-dev] [PATCH] net/i40e: enable statistic reset for VF

2017-02-23 Thread Qi Zhang
The patch implements the dev_ops "stats_reset" for VF. Signed-off-by: Qi Zhang --- drivers/net/i40e/i40e_ethdev_vf.c | 34 ++ drivers/net/i40e/i40e_pf.c| 25 + drivers/net/i40e/i40e_pf.h| 1 + 3 files changed, 60 insertions

Re: [dpdk-dev] [PATCH v2] e1000/base: fix multicast setting in VF

2017-02-23 Thread Lu, Wenzhuo
Hi, > -Original Message- > From: Yong Wang [mailto:wang.yon...@zte.com.cn] > Sent: Tuesday, February 21, 2017 5:33 PM > To: Lu, Wenzhuo > Cc: dev@dpdk.org; Yong Wang > Subject: [PATCH v2] e1000/base: fix multicast setting in VF > > In function e1000_update_mc_addr_list_vf(), "msgbuf[0]" i

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

2017-02-23 Thread Martinx - ジェームズ
On 23 February 2017 at 03:49, Thomas Monjalon wrote: > 2017-02-20 15:33, Joao Martins: > > On 02/17/2017 04:07 PM, Konrad Rzeszutek Wilk wrote: > > > On Thu, Feb 16, 2017 at 10:51:44PM +0100, Vincent JARDIN wrote: > > >> Le 16/02/2017 à 14:36, Konrad Rzeszutek Wilk a écrit : > > Is it time n

Re: [dpdk-dev] Further fun with ABI tracking

2017-02-23 Thread Ferruh Yigit
On 2/22/2017 1:12 PM, Christian Ehrhardt wrote: > On Tue, Feb 14, 2017 at 9:31 PM, Jan Blunck wrote: > >>> 1. Downstreams to insert Major version into soname >>> Distributions could insert the DPDK major version (like 16.11) into the >>> soname and package names. A common example of this is libbo

Re: [dpdk-dev] [PATCH 36/50] net/liquidio: add API to alloc and send command

2017-02-23 Thread Shijith Thotton
On Thursday 23 February 2017 08:03 PM, Ferruh Yigit wrote: > On 2/21/2017 9:26 AM, Shijith Thotton wrote: > > Can it be "control command"? > > add API to alloc and send _control command_ ? > > Yes. Will modify commit title. >> Add APIs to allocate and send control command to device. >> >> S

Re: [dpdk-dev] [PATCH 16/50] net/liquidio: add APIs to allocate and free IQ

2017-02-23 Thread Shijith Thotton
On Thursday 23 February 2017 08:00 PM, Ferruh Yigit wrote: > On 2/21/2017 9:26 AM, Shijith Thotton wrote: >> Add APIs to allocate and free instruction queue. Allocates instruction >> queue 0 initially to send device configurations commands and later re- >> allocates as per application requirement

Re: [dpdk-dev] [PATCH 06/50] net/liquidio: liquidio VF PMD Driver registration

2017-02-23 Thread Shijith Thotton
On Thursday 23 February 2017 07:59 PM, Ferruh Yigit wrote: > On 2/21/2017 9:26 AM, Shijith Thotton wrote: >> Register LiquidIO PMD (net_liovf) and define APIs to init and >> uninit. >> >> Signed-off-by: Shijith Thotton >> Signed-off-by: Jerin Jacob >> Signed-off-by: Derek Chickles >> Signed-of

Re: [dpdk-dev] [PATCH 05/50] maintainers: claim responsibility for LiquidIO PMD

2017-02-23 Thread Shijith Thotton
On Thursday 23 February 2017 07:58 PM, Ferruh Yigit wrote: > On 2/21/2017 9:26 AM, Shijith Thotton wrote: >> Signed-off-by: Shijith Thotton >> Signed-off-by: Derek Chickles >> --- >> MAINTAINERS | 7 +++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/MAINTAINERS b/MAINTAINERS >> ind

[dpdk-dev] [PATCH v1 09/14] ring: allow dequeue fns to return remaining entry count

2017-02-23 Thread Bruce Richardson
Add an extra parameter to the ring dequeue burst/bulk functions so that those functions can optionally return the amount of remaining objs in the ring. This information can be used by applications in a number of ways, for instance, with single-consumer queues, it provides a max dequeue size which i

[dpdk-dev] [PATCH v1 13/14] ring: create common function for updating tail idx

2017-02-23 Thread Bruce Richardson
Both producer and consumer use the same logic for updating the tail index so merge into a single function. Signed-off-by: Bruce Richardson --- lib/librte_ring/rte_ring.h | 32 +++- 1 file changed, 15 insertions(+), 17 deletions(-) diff --git a/lib/librte_ring/rte_rin

[dpdk-dev] [PATCH v1 12/14] ring: separate out head index manipulation for enq/deq

2017-02-23 Thread Bruce Richardson
We can write a single common function for head manipulation for enq and a common one for deq, allowing us to have a single worker function for enq and deq, rather than two of each. Update all other inline functions to use the new functions. Signed-off-by: Bruce Richardson --- lib/librte_ring/rte

[dpdk-dev] [PATCH v1 14/14] ring: make ring struct and enq/deq macros type agnostic

2017-02-23 Thread Bruce Richardson
Modify the enqueue and dequeue macros to support copying any type of object by passing in the exact object type. Rather than using the "ring" structure member of rte_ring, which is of type "array of void *", instead have the macros take the start of the ring a a pointer value, thereby leaving the r

[dpdk-dev] [PATCH v1 11/14] ring: reduce scope of local variables

2017-02-23 Thread Bruce Richardson
The local variable i is only used for loop control so define it in the enqueue and dequeue blocks directly, rather than at the function level. Signed-off-by: Bruce Richardson --- lib/librte_ring/rte_ring.h | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/lib/librte_ring/

[dpdk-dev] [PATCH v1 10/14] examples/quota_watermark: use ring space for watermarks

2017-02-23 Thread Bruce Richardson
Now that the enqueue function returns the amount of space in the ring, we can use that to replace the old watermark functionality. Update the example app to do so, and re-enable it in the examples Makefile. Signed-off-by: Bruce Richardson --- examples/Makefile | 2 +- ex

[dpdk-dev] [PATCH v1 08/14] ring: allow enqueue fns to return free space value

2017-02-23 Thread Bruce Richardson
Add an extra parameter to the ring enqueue burst/bulk functions so that those functions can optionally return the amount of free space in the ring. This information can be used by applications in a number of ways, for instance, with single-producer queues, it provides a max enqueue size which is gu

[dpdk-dev] [PATCH v1 07/14] ring: make bulk and burst fn return vals consistent

2017-02-23 Thread Bruce Richardson
The bulk fns for rings returns 0 for all elements enqueued and negative for no space. Change that to make them consistent with the burst functions in returning the number of elements enqueued/dequeued, i.e. 0 or N. This change also allows the return value from enq/deq to be used directly without a

[dpdk-dev] [PATCH v1 06/14] ring: remove watermark support

2017-02-23 Thread Bruce Richardson
Remove the watermark support. A future commit will add support for having enqueue functions return the amount of free space in the ring, which will allow applications to implement their own watermark checks, while also being more useful to the app. Signed-off-by: Bruce Richardson --- app/test/co

[dpdk-dev] [PATCH v1 04/14] ring: remove debug setting

2017-02-23 Thread Bruce Richardson
The debug option only provided statistics to the user, most of which could be tracked by the application itself. Remove this as a compile time option, and feature, simplifying the code. Signed-off-by: Bruce Richardson --- app/test/test_ring.c | 410 -

[dpdk-dev] [PATCH v1 05/14] ring: remove the yield when waiting for tail update

2017-02-23 Thread Bruce Richardson
There was a compile time setting to enable a ring to yield when it entered a loop in mp or mc rings waiting for the tail pointer update. Build time settings are not recommended for enabling/disabling features, and since this was off by default, remove it completely. If needed, a runtime enabled equ

[dpdk-dev] [PATCH v1 00/14] refactor and cleanup of rte_ring

2017-02-23 Thread Bruce Richardson
NOTE: this set depends on the v2 cleanup set sent previously. http://dpdk.org/ml/archives/dev/2017-February/thread.html#58200 This patchset make a set of, sometimes non-backward compatible, cleanup changes to the rte_ring code in order to improve it. The resulting code is shorter, since th

[dpdk-dev] [PATCH v1 03/14] ring: eliminate duplication of size and mask fields

2017-02-23 Thread Bruce Richardson
The size and mask fields are duplicated in both the producer and consumer data structures. Move them out of that into the top level structure so they are not duplicated. Signed-off-by: Bruce Richardson --- app/test/test_ring.c | 6 +++--- lib/librte_ring/rte_ring.c | 20 ++

[dpdk-dev] [PATCH v1 02/14] ring: create common structure for prod and cons metadata

2017-02-23 Thread Bruce Richardson
create a common structure to hold the metadata for the producer and the consumer, since both need essentially the same information - the head and tail values, the ring size and mask. Signed-off-by: Bruce Richardson --- lib/librte_ring/rte_ring.h | 32 1 file chan

[dpdk-dev] [PATCH v1 01/14] ring: remove split cacheline build setting

2017-02-23 Thread Bruce Richardson
Users compiling DPDK should not need to know or care about the arrangement of cachelines in the rte_ring structure. Therefore just remove the build option and set the structures to be always split. For improved performance use 128B rather than 64B alignment since it stops the producer and consumer

Re: [dpdk-dev] [PATCH v2 0/5] minor fixes and cleanup

2017-02-23 Thread Bruce Richardson
On Thu, Feb 23, 2017 at 04:41:58PM +, Bruce Richardson wrote: > This patchset contains some minor fixes and improvements which > I found in the context of preparing a new patchset for the > rte_ring code. Having these merged separately reduces the > number of patches in the ring set so should m

[dpdk-dev] Regarding phys port id in mellanox NIC

2017-02-23 Thread Ramakrishnan, Kuralamudhan
Hello, I have a question regarding the phys port id in mellanox NIC, is this feature deprecated in the latest mlnx-en-dpdk driver ? Issue: I am working on the Ethernet controller: Mellanox Technologies MT27520 Family [ConnectX-3 Pro] NIC. I used to read the /sys/class/net/enp6s0/phys_port_id f

Re: [dpdk-dev] [dpdk-users] Problem running dpdk-qat example in SR-IOV mode

2017-02-23 Thread Thomas Monjalon
2017-02-23 16:18, De Lara Guarch, Pablo: > > From: Thomas Monjalon > > 2017-02-20 11:53, Trahe, Fiona: > > > The dpdk-qat example app is an earlier mechanism of wrapping a QAT > > driver into DPDK - it's not using the cryptodev API, and depends on both a > > kernel driver and user-space libs from a

Re: [dpdk-dev] [PATCH 50/50] doc: added documents

2017-02-23 Thread Mcnamara, John
Hi, Thanks for the doc. Some comments below. > -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Shijith Thotton > Sent: Tuesday, February 21, 2017 9:27 AM > To: dev@dpdk.org > Cc: Jerin Jacob ; Derek Chickles > > Subject: [dpdk-dev] [PATCH 50/50] doc: added docu

[dpdk-dev] [PATCH v2 4/5] examples/quota_watermark: correct code indentation

2017-02-23 Thread Bruce Richardson
The code indentation in the example app files used spaces rather than tabs for indentation, and as such did not conform to DPDK conventions. This left those modifying the code in a bind - to fix things on a line by line basis so as to avoid checkpatch errors, or to keep things consistent within the

[dpdk-dev] [PATCH v2 3/5] crypto/null: use ring size function

2017-02-23 Thread Bruce Richardson
Rather than reading the size directly from the ring structure, use the dedicated function for that purpose. Signed-off-by: Bruce Richardson --- drivers/crypto/null/null_crypto_pmd_ops.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/crypto/null/null_crypto_pmd_ops.c

[dpdk-dev] [PATCH v2 5/5] examples/quota_watermark: fix requirement for 2M pages

2017-02-23 Thread Bruce Richardson
The sample app was forcing the shared memory block for high/low watermarks to be placed in a memzone on 2M pages. This prevented it from running on systems with just 1G pages, so remove the flag forcing 2M pages. Fixes: 1d6c3ee3321a ("examples/quota_watermark: initial import") CC: sta...@dpdk.org

[dpdk-dev] [PATCH v2 1/5] app/pdump: fix duplicate macro definition

2017-02-23 Thread Bruce Richardson
RTE_RING_SZ_MASK is redefined here with the original definition in rte_ring.h. Since rte_ring.h is already included, just remove the duplicate definition here. Fixes: caa7028276b8 ("app/pdump: add tool for packet capturing") Signed-off-by: Bruce Richardson --- app/pdump/main.c | 1 - 1 file cha

[dpdk-dev] [PATCH v2 2/5] ring: add a function to return the ring size

2017-02-23 Thread Bruce Richardson
Applications and other libraries should not be reading inside the rte_ring structure directly to get the ring size. Instead add a fn to allow it to be queried. Signed-off-by: Bruce Richardson --- lib/librte_ring/rte_ring.h | 14 ++ 1 file changed, 14 insertions(+) diff --git a/lib/l

[dpdk-dev] [PATCH v2 0/5] minor fixes and cleanup

2017-02-23 Thread Bruce Richardson
This patchset contains some minor fixes and improvements which I found in the context of preparing a new patchset for the rte_ring code. Having these merged separately reduces the number of patches in the ring set so should make everyone's life easier. V2 Changes: new patch added with extra fix fo

Re: [dpdk-dev] [dpdk-users] Problem running dpdk-qat example in SR-IOV mode

2017-02-23 Thread De Lara Guarch, Pablo
Hi Thomas, > -Original Message- > From: users [mailto:users-boun...@dpdk.org] On Behalf Of Thomas > Monjalon > Sent: Monday, February 20, 2017 1:11 PM > To: Trahe, Fiona > Cc: dev@dpdk.org; Pankaj Joshi; us...@dpdk.org > Subject: Re: [dpdk-users] [dpdk-dev] Problem running dpdk-qat example

Re: [dpdk-dev] Regarding phys port id in mellanox NIC

2017-02-23 Thread Adrien Mazarguil
Hi Kural, On Thu, Feb 23, 2017 at 11:23:37AM +, Ramakrishnan, Kuralamudhan wrote: > Hello, > > I have a question regarding the phys port id in mellanox NIC, is this feature > deprecated in the latest mlnx-en-dpdk driver ? > > Issue: > > I am working on the Ethernet controller: Mellanox Tec

[dpdk-dev] [PATCH v2] crypto/openssl: add DES DOCSIS BPI support

2017-02-23 Thread Pablo de Lara
Adds support in OpenSSL for algorithm following the DOCSIS specification, which combines DES-CBC for full DES blocks (8 bytes) and DES-CFB for last runt block (less than 8 bytes). Signed-off-by: Pablo de Lara --- This patch depends on patch http://dpdk.org/dev/patchwork/patch/20691/. Changes in

[dpdk-dev] [PATCH v2] crypto/aesni_mb: add AES DOCSIS BPI support

2017-02-23 Thread Pablo de Lara
Underlying IPSec Multi buffer library implements DOCSIS specification, so this commit adds support for this new feature, which combines AES-CBC for full AES blocks (16 bytes) and AES-CFB for last runt block (less than 16 bytes). Signed-off-by: Pablo de Lara --- This patch depends on patch http://

Re: [dpdk-dev] [PATCH 50/50] doc: added documents

2017-02-23 Thread Ferruh Yigit
On 2/21/2017 9:27 AM, Shijith Thotton wrote: > Added doc/guides/nics/liquidio.rst and > doc/guides/nics/features/liquidio.ini. Updated release notes. > > Signed-off-by: Shijith Thotton > Signed-off-by: Jerin Jacob > Signed-off-by: Derek Chickles > --- > doc/guides/nics/features/liquidio.ini |

Re: [dpdk-dev] [PATCH 40/50] net/liquidio: add API to set MTU

2017-02-23 Thread Ferruh Yigit
On 2/21/2017 9:26 AM, Shijith Thotton wrote: > Signed-off-by: Shijith Thotton > Signed-off-by: Jerin Jacob > Signed-off-by: Derek Chickles > Signed-off-by: Venkat Koppula > Signed-off-by: Mallesham Jatharakonda > --- <...> > > +dev_mtu_updation_error: > +rte_eal_alarm_cancel(lio_sync_link_s

Re: [dpdk-dev] [PATCH 36/50] net/liquidio: add API to alloc and send command

2017-02-23 Thread Ferruh Yigit
On 2/21/2017 9:26 AM, Shijith Thotton wrote: Can it be "control command"? add API to alloc and send _control command_ ? > Add APIs to allocate and send control command to device. > > Signed-off-by: Shijith Thotton > Signed-off-by: Jerin Jacob > Signed-off-by: Derek Chickles > Signed-off-by:

[dpdk-dev] [PATCH] crypto/openssl: add DES DOCSIS BPI support

2017-02-23 Thread Pablo de Lara
Adds support in OpenSSL for algorithm following the DOCSIS specification, which combines DES-CBC for full DES blocks (8 bytes) and DES-CFB for last runt block (not multiple of 8 bytes). Signed-off-by: Pablo de Lara --- app/test/test_cryptodev.c| 18 +++ app/test/test_cry

Re: [dpdk-dev] [PATCH 30/50] net/liquidio: add Tx data path for single segment

2017-02-23 Thread Ferruh Yigit
On 2/21/2017 9:26 AM, Shijith Thotton wrote: > Signed-off-by: Shijith Thotton > Signed-off-by: Jerin Jacob > Signed-off-by: Derek Chickles > Signed-off-by: Venkat Koppula > Signed-off-by: Mallesham Jatharakonda > --- <...> > @@ -543,6 +544,7 @@ static int lio_dev_configure(struct rte_eth_dev

Re: [dpdk-dev] [PATCH 28/50] net/liquidio: add APIs for sg list

2017-02-23 Thread Ferruh Yigit
On 2/21/2017 9:26 AM, Shijith Thotton wrote: > SG list is used while sending packets with multiple segments. May be good to use "Scatter-Gather List" in commit log to make it clear, and since it is an abbreviation, uppercase "SG" can be used in patch title. > > Signed-off-by: Shijith Thotton >

Re: [dpdk-dev] [PATCH 16/50] net/liquidio: add APIs to allocate and free IQ

2017-02-23 Thread Ferruh Yigit
On 2/21/2017 9:26 AM, Shijith Thotton wrote: > Add APIs to allocate and free instruction queue. Allocates instruction > queue 0 initially to send device configurations commands and later re- > allocates as per application requirement during tx queue setup. Can you please describe the relevant enti

Re: [dpdk-dev] [PATCH 06/50] net/liquidio: liquidio VF PMD Driver registration

2017-02-23 Thread Ferruh Yigit
On 2/21/2017 9:26 AM, Shijith Thotton wrote: > Register LiquidIO PMD (net_liovf) and define APIs to init and > uninit. > > Signed-off-by: Shijith Thotton > Signed-off-by: Jerin Jacob > Signed-off-by: Derek Chickles > Signed-off-by: Venkat Koppula > Signed-off-by: Mallesham Jatharakonda <...>

Re: [dpdk-dev] [PATCH 05/50] maintainers: claim responsibility for LiquidIO PMD

2017-02-23 Thread Ferruh Yigit
On 2/21/2017 9:26 AM, Shijith Thotton wrote: > Signed-off-by: Shijith Thotton > Signed-off-by: Derek Chickles > --- > MAINTAINERS | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index b4617fc..a63b7f7 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @

Re: [dpdk-dev] [PATCH 07/50] net/liquidio: added Makefile

2017-02-23 Thread Ferruh Yigit
On 2/21/2017 9:26 AM, Shijith Thotton wrote: > Added Makefile and made build changes. > > Signed-off-by: Shijith Thotton > Signed-off-by: Jerin Jacob > Signed-off-by: Derek Chickles > Signed-off-by: Venkat Koppula > Signed-off-by: Mallesham Jatharakonda > --- Thanks for splitting into functi

[dpdk-dev] [PATCH v2] cryptodev: enable BPI for Cablelabs DOCSIS security spec

2017-02-23 Thread Pablo de Lara
Extend the DPDK cryptodev API to enable processing of packets according to the Baseline Privacy Interface Plus (BPI+) Specification described in the security specification of the Cablelabs Data-over-Cable Service Interface Specification (DOCSIS). Brief summary of BPI+ symmetric cryptography requir

Re: [dpdk-dev] [PATCH 1/3] doc: set alignments in NIC overview table

2017-02-23 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Monday, February 20, 2017 9:27 PM > To: dev@dpdk.org > Subject: [dpdk-dev] [PATCH 1/3] doc: set alignments in NIC overview table > > Some CSS alignments were not explicitly set. > The poin

Re: [dpdk-dev] [PATCH 3/3] doc: highlight pointed column in NIC overview table

2017-02-23 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Monday, February 20, 2017 9:27 PM > To: dev@dpdk.org > Subject: [dpdk-dev] [PATCH 3/3] doc: highlight pointed column in NIC > overview table Acked-by: John McNamara

Re: [dpdk-dev] [PATCH 2/3] doc: highlight pointed row in NIC overview table

2017-02-23 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Monday, February 20, 2017 9:27 PM > To: dev@dpdk.org > Subject: [dpdk-dev] [PATCH 2/3] doc: highlight pointed row in NIC overview > table > > Every cells of a feature row are painted in ye

Re: [dpdk-dev] [PATCH 1/3] doc: set alignments in NIC overview table

2017-02-23 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Monday, February 20, 2017 9:27 PM > To: dev@dpdk.org > Subject: [dpdk-dev] [PATCH 1/3] doc: set alignments in NIC overview table > > Some CSS alignments were not explicitly set. > The poin

[dpdk-dev] [PATCH 2/4] cryptodev: add algorithm string parsers

2017-02-23 Thread Pablo de Lara
Adds functions to get the cipher/authentication algorithm enums, given a string. This is useful for applications which gets the algorithm required from the user, to have a common string-enum mapping. Signed-off-by: Pablo de Lara --- lib/librte_cryptodev/rte_cryptodev.c | 34 +++

[dpdk-dev] [PATCH 4/4] examples/l2fwd-crypto: use cryptodev algorithm parser

2017-02-23 Thread Pablo de Lara
L2fwd-crypto app was creating an array of strings for the supported algorithms, which was different from the strings that are now in cryptodev. Use the new API in cryptodev to parse the string from the user, to get the algorithm enum, instead, so it is not necessary to add a new supported algorith

[dpdk-dev] [PATCH 0/4] New crypto algorithm string parser API

2017-02-23 Thread Pablo de Lara
Last release, an array with strings for the supported algorithms by the cryptodev library was added and used in the crypto-perf app. This patchset creates a new API to parse strings from the user, to select the desired algorithm (using the array above), which can be used by any application, making

[dpdk-dev] [PATCH 3/4] app/crypto-perf: use cryptodev algorithm parser

2017-02-23 Thread Pablo de Lara
Instead of going through the array of supported algorithms in the app, to get the algorithm enum, use the new API in cryptodev to parse this string, so it is not necessary to add a new supported algorithm in the cryptodev library and this app. Signed-off-by: Pablo de Lara --- app/test-crypto-per

[dpdk-dev] [PATCH 1/4] cryptodev: add missing algorithm strings

2017-02-23 Thread Pablo de Lara
DES-CBC and AUTH NULL algorithms were missing in the array of algorithm strings. Signed-off-by: Pablo de Lara --- lib/librte_cryptodev/rte_cryptodev.c | 4 1 file changed, 4 insertions(+) diff --git a/lib/librte_cryptodev/rte_cryptodev.c b/lib/librte_cryptodev/rte_cryptodev.c index a64320

[dpdk-dev] [PATCH] net/virtio-user: fix multi-process issue

2017-02-23 Thread Ami Sabo
Secondary process doesn't properly attach to the rte_eth_device initialized by the primary process. Accessing device from secondary process (e.g. via rte_eth_rx_burst), causes process to crash. because rte_eth_dev_data is not properly set. The issue was flood by 'commit 7f95f78a8aea ("ethdev: cle

Re: [dpdk-dev] [PATCH 0/3] cross focus in NIC overview HTML table

2017-02-23 Thread Ferruh Yigit
On 2/20/2017 9:26 PM, Thomas Monjalon wrote: > In order to make the NIC overview easier to read > (http://dpdk.org/doc/guides/nics/overview.html) > the row and column pointed by the mouse are highlighted. > > Thomas Monjalon (3): > doc: set alignments in NIC overview table > doc: highlight poi

Re: [dpdk-dev] [PATCH v4] i40e: implement vector PMD for altivec

2017-02-23 Thread Ferruh Yigit
On 2/23/2017 4:39 AM, Gowrishankar wrote: > From: Gowrishankar Muthukrishnan > > This patch enables i40e driver in powerpc along with its altivec > intrinsic support. > > Changes: > v4 - docs and config update. > v3 - minor corrections for coding style standard. > v2 - minor corrections for gcc

Re: [dpdk-dev] [PATCH v2 1/2] net/mlx4: split the definitions to the header file

2017-02-23 Thread Vasily Philipov
Hi Ferruh, > -Original Message- > From: Ferruh Yigit [mailto:ferruh.yi...@intel.com] > Sent: Wednesday, February 22, 2017 21:05 > To: Vasily Philipov ; dev@dpdk.org > Cc: Adrien Mazarguil ; Nélio Laranjeiro > > Subject: Re: [dpdk-dev] [PATCH v2 1/2] net/mlx4: split the definitions to the

Re: [dpdk-dev] [PATCH] net/bnx2x: Fix transmit queue free threshold

2017-02-23 Thread Ferruh Yigit
On 2/23/2017 12:26 AM, Harish Patil wrote: >> >> The default tx_free_thresh is potentially larger than the allocated queue >> which will result in TX queue cleanup never happening. To fix this, >> lower the default free threshold and ensure that the free threshold is >> never greater than the maxi

Re: [dpdk-dev] [PATCH 00/50] LiquidIO PMD

2017-02-23 Thread Ferruh Yigit
On 2/22/2017 4:56 AM, Shijith Thotton wrote: > > > On Wednesday 22 February 2017 01:52 AM, Stephen Hemminger wrote: >> On Tue, 21 Feb 2017 14:56:15 +0530 >> Shijith Thotton wrote: >> >>> The patch series provides initial version of virtual function poll mode >>> driver for Cavium LiquidIO II ser

Re: [dpdk-dev] [PATCH v2] net/virtio-user: fix multi-process issue

2017-02-23 Thread Ami Sabo
Mistakenly sent only partial diff. Resent the all patch (from the original code base) as a reply to the first message in the thread. -- Ami -Original Message- From: Yuanhan Liu [mailto:yuanhan@linux.intel.com] Sent: Thursday, February 23, 2017 4:55 AM To: Ami Sabo Cc: dev@dpdk.org

[dpdk-dev] [PATCH] Signed-off-by: Ami Sabo

2017-02-23 Thread Ami Sabo
net/virtio-user: fix multi-process issue Secondary process doesn't properly attach to the rte_eth_device initialized by the primary process. Accessing device from secondary process (e.g. via rte_eth_rx_burst), causes process to crash. because rte_eth_dev_data is not properly set. The issue was f

Re: [dpdk-dev] [PATCH 0/4] catch up TILE-Gx support in DPDK

2017-02-23 Thread Chris Metcalf
Lining please go ahead since you are the maintainer. Note I adjusted maintainers in my patch so probably best to wait for that series to be applied then just remove the maintainers entry along with all the tilegx mpipe stuff. Sent from the waterpark while waiting for a kid :) On Feb 21, 2017,

Re: [dpdk-dev] [PATCH 0/4] catch up TILE-Gx support in DPDK

2017-02-23 Thread Liming Sun
>> So do you mean that the TILE-Gx maintainers officially give up on their role? >> Then please update the MAINTAINERS file. Yes. Please update the MAINTAINERS file as needed. Chris Metcalf is out this week. I could submit a separate patch for the MAINTAINERS file if it's required. Or we could wa

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

2017-02-23 Thread Thomas Monjalon
2017-02-20 15:33, Joao Martins: > On 02/17/2017 04:07 PM, Konrad Rzeszutek Wilk wrote: > > On Thu, Feb 16, 2017 at 10:51:44PM +0100, Vincent JARDIN wrote: > >> Le 16/02/2017 à 14:36, Konrad Rzeszutek Wilk a écrit : > Is it time now to officially remove Dom0 support? > >>> So we do have an prot

[dpdk-dev] [PATCH] vhost: fix multiple queue not being enabled for older kernels

2017-02-23 Thread Yuanhan Liu
Some macros (say VIRTIO_NET_F_MQ) are needed for enabling multiple queue, however they are introduced since kernel v3.8, meaning build error happens if we build DPDK vhost on those platforms. 71dfdbe66a66 ("vhost: fix build with kernel < 3.8") meant to fix it, but in a wrong way: it completely dis

Re: [dpdk-dev] [PATCH v10 0/7] Expanded statistics reporting

2017-02-23 Thread Thomas Monjalon
2017-02-23 07:09, Remy Horton: > > On 16/02/2017 10:53, Thomas Monjalon wrote: > > 2017-02-03 10:33, Remy Horton: > [..] > > I think there are three remaining questions: > > - When the metrics computation are done? (in which thread?) > > Actual calculation is not done by libmetrics itself - it on