Re: [dpdk-dev] rte_port_ring and SP/MP, SC/MC flags

2017-02-01 Thread Dumitrescu, Cristian
Hi Yerden, > -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Yerden > Zhumabekov > Sent: Monday, January 30, 2017 5:43 AM > To: dev@dpdk.org > Subject: [dpdk-dev] rte_port_ring and SP/MP, SC/MC flags > > Hello, > > I'd like to use rte_port_ring abstract in my ap

Re: [dpdk-dev] [libvirt] [RFC] Vhost-user backends cross-version migration support

2017-02-01 Thread Michael S. Tsirkin
On Wed, Feb 01, 2017 at 11:41:19AM +, Daniel P. Berrange wrote: > On Wed, Feb 01, 2017 at 12:33:22PM +0100, Maxime Coquelin wrote: > > > > > > On 02/01/2017 10:43 AM, Daniel P. Berrange wrote: > > > On Wed, Feb 01, 2017 at 10:14:54AM +0100, Michal Privoznik wrote: > > > > On 02/01/2017 09:35

Re: [dpdk-dev] [PATCH v2] mbuf: use pktmbuf helper to create the pool

2017-02-01 Thread Hemant Agrawal
On 1/31/2017 3:25 PM, Olivier Matz wrote: Hi Hemant, Sorry for the delay. On Fri, 20 Jan 2017 12:41:20 +0530, Hemant Agrawal wrote: When possible, replace the uses of rte_mempool_create() with the helper provided in librte_mbuf: rte_pktmbuf_pool_create(). This is the preferred way to create

[dpdk-dev] [PATCH] doc: update release notes for enic driver

2017-02-01 Thread John Daley
Signed-off-by: John Daley --- doc/guides/rel_notes/release_17_02.rst | 6 ++ 1 file changed, 6 insertions(+) diff --git a/doc/guides/rel_notes/release_17_02.rst b/doc/guides/rel_notes/release_17_02.rst index 83519dc..e072c90 100644 --- a/doc/guides/rel_notes/release_17_02.rst +++ b/doc/guid

Re: [dpdk-dev] [dpdk-stable] [PATCH v2] net/mlx5: fix link status query

2017-02-01 Thread Ferruh Yigit
On 1/31/2017 3:42 PM, Nélio Laranjeiro wrote: > On Tue, Jan 31, 2017 at 01:45:29PM +0200, Shahaf Shuler wrote: >> Trying to query the link status through new kernel ioctl API >> ETHTOOL_GLINKSETTINGS was always failing due to kernel bug. >> The bug was fixed on version 4.9 >> this patch uses the le

Re: [dpdk-dev] [PATCH] net/sfc: multicast list set must not fail if list is empty

2017-02-01 Thread Ferruh Yigit
On 2/1/2017 8:17 AM, Andrew Rybchenko wrote: > From: Ivan Malov > > The patch is to fix sfc_set_mc_addr_list() behaviour in order > to make it accept an empty multicast address list thus making > it possible to remove multicast addresses inserted previously > > Fixes: 0fa0070e4391 ("net/sfc: sup

Re: [dpdk-dev] [PATCH] net/ixgbe: clean up rte_eth_dev_info_get

2017-02-01 Thread Ferruh Yigit
On 2/1/2017 6:10 PM, Ananyev, Konstantin wrote: > > >> -Original Message- >> From: Yigit, Ferruh >> Sent: Wednesday, February 1, 2017 5:40 PM >> To: Ananyev, Konstantin ; Lu, Wenzhuo >> ; dev@dpdk.org >> Subject: Re: [dpdk-dev] [PATCH] net/ixgbe: clean up rte_eth_dev_info_get >> >> On 2/

Re: [dpdk-dev] [dpdk-stable] [PATCH v2] net/mlx5: fix link status query

2017-02-01 Thread Ferruh Yigit
On 2/1/2017 12:57 PM, Adrien Mazarguil wrote: > On Wed, Feb 01, 2017 at 11:13:59AM +, Ferruh Yigit wrote: >> On 2/1/2017 9:07 AM, Adrien Mazarguil wrote: >>> On Wed, Feb 01, 2017 at 06:53:55AM +, Shahaf Shuler wrote: : Tuesday, January 31, 2017 6:17 PM, Ferruh Yigit: > On 1/31/2017

Re: [dpdk-dev] [PATCH] net/ixgbe: clean up rte_eth_dev_info_get

2017-02-01 Thread Ananyev, Konstantin
> -Original Message- > From: Yigit, Ferruh > Sent: Wednesday, February 1, 2017 5:40 PM > To: Ananyev, Konstantin ; Lu, Wenzhuo > ; dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH] net/ixgbe: clean up rte_eth_dev_info_get > > On 2/1/2017 4:24 PM, Ananyev, Konstantin wrote: > > Hi Wenzhuo,

Re: [dpdk-dev] [PATCH 1/6] net/tap: use correct tap name

2017-02-01 Thread Ferruh Yigit
On 2/1/2017 3:55 PM, Wiles, Keith wrote: > >> On Feb 1, 2017, at 9:40 AM, Pascal Mazon wrote: >> >> On 02/01/2017 04:25 PM, Wiles, Keith wrote: >>> On Feb 1, 2017, at 2:11 AM, Pascal Mazon wrote: On 02/01/2017 12:29 AM, Wiles, Keith wrote: > >> On Jan 31, 2017, at 10:39 AM

Re: [dpdk-dev] [PATCH] net/ixgbe: clean up rte_eth_dev_info_get

2017-02-01 Thread Ferruh Yigit
On 2/1/2017 4:24 PM, Ananyev, Konstantin wrote: > Hi Wenzhuo, > >> -Original Message- >> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Wenzhuo Lu >> Sent: Wednesday, January 25, 2017 2:39 AM >> To: dev@dpdk.org >> Cc: Lu, Wenzhuo >> Subject: [dpdk-dev] [PATCH] net/ixgbe: clean up r

Re: [dpdk-dev] [PATCH 2/7] ethdev: reduce goto's in attach/detach

2017-02-01 Thread Ferruh Yigit
On 1/10/2017 8:40 AM, Thomas Monjalon wrote: > Hi Stephen, > > Please use --in-reply-to to keep v1 and v2 in the same thread. > > Comment below > > > 2017-01-09 15:30, Stephen Hemminger: >> int >> rte_eth_dev_detach(uint8_t port_id, char *name) >> { >> -int ret = -1; >> +int ret; >>

[dpdk-dev] bugs and glitches in rte_cryptodev_devices_get

2017-02-01 Thread Stephen Hemminger
The function rte_cryptodev_devices_get has several issues. I was just going to fix it, but think it need to be explained. One potentially serious one (reported by coverity) is: *** CID 141067:(BAD_COMPARE) /lib/librte_cryptodev/rte_cryptodev.c: 503 in rte_cryptodev_devices_get() 497

Re: [dpdk-dev] [PATCH 6/7] ixgbe: remove useless return

2017-02-01 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Stephen Hemminger > Sent: Monday, January 9, 2017 11:30 PM > To: dev@dpdk.org > Cc: Stephen Hemminger > Subject: [dpdk-dev] [PATCH 6/7] ixgbe: remove useless return > > If all goto's lead to a return, then bette

Re: [dpdk-dev] [PATCH] net/ixgbe: clean up rte_eth_dev_info_get

2017-02-01 Thread Ananyev, Konstantin
Hi Wenzhuo, > -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Wenzhuo Lu > Sent: Wednesday, January 25, 2017 2:39 AM > To: dev@dpdk.org > Cc: Lu, Wenzhuo > Subject: [dpdk-dev] [PATCH] net/ixgbe: clean up rte_eth_dev_info_get > > It'not appropriate to call rte_et

Re: [dpdk-dev] [PATCH 1/2] net/ixgbe: calculate the correct number of received packets in bulk alloc function

2017-02-01 Thread Ananyev, Konstantin
Hi, > -Original Message- > From: Jianbo Liu [mailto:jianbo@linaro.org] > Sent: Monday, December 19, 2016 6:09 AM > To: dev@dpdk.org; Zhang, Helin ; Ananyev, Konstantin > ; > jerin.ja...@caviumnetworks.com > Cc: Jianbo Liu > Subject: [PATCH 1/2] net/ixgbe: calculate the correct number

Re: [dpdk-dev] [PATCH 1/6] net/tap: use correct tap name

2017-02-01 Thread Wiles, Keith
> On Feb 1, 2017, at 9:40 AM, Pascal Mazon wrote: > > On 02/01/2017 04:25 PM, Wiles, Keith wrote: >> >>> On Feb 1, 2017, at 2:11 AM, Pascal Mazon wrote: >>> >>> On 02/01/2017 12:29 AM, Wiles, Keith wrote: > On Jan 31, 2017, at 10:39 AM, Pascal Mazon wrote: > > On 01/31/201

Re: [dpdk-dev] [PATCH 1/6] net/tap: use correct tap name

2017-02-01 Thread Pascal Mazon
On 02/01/2017 04:25 PM, Wiles, Keith wrote: On Feb 1, 2017, at 2:11 AM, Pascal Mazon wrote: On 02/01/2017 12:29 AM, Wiles, Keith wrote: On Jan 31, 2017, at 10:39 AM, Pascal Mazon wrote: On 01/31/2017 05:06 PM, Wiles, Keith wrote:> Looking at the changes to set the link up/down and the a

Re: [dpdk-dev] [PATCH 1/6] net/tap: use correct tap name

2017-02-01 Thread Wiles, Keith
> On Feb 1, 2017, at 2:11 AM, Pascal Mazon wrote: > > On 02/01/2017 12:29 AM, Wiles, Keith wrote: >> >>> On Jan 31, 2017, at 10:39 AM, Pascal Mazon wrote: >>> >>> On 01/31/2017 05:06 PM, Wiles, Keith wrote:> Looking at the changes to set the link up/down and the adding the two func

Re: [dpdk-dev] [PATCH 25/25] rte_eal_init: add info about rte_errno codes

2017-02-01 Thread Bruce Richardson
On Wed, Feb 01, 2017 at 01:06:03PM +0100, Jan Blunck wrote: > On Wed, Feb 1, 2017 at 11:54 AM, Adrien Mazarguil > wrote: > > On Mon, Jan 30, 2017 at 09:19:29PM +0100, Thomas Monjalon wrote: > >> 2017-01-30 13:38, Aaron Conole: > >> > Stephen Hemminger writes: > >> > > Bruce Richardson wrote: > >

Re: [dpdk-dev] [PATCH] net/ixgbe: ensure link status is updated

2017-02-01 Thread Thomas Monjalon
2016-12-08 17:10, Ferruh Yigit: > On 11/28/2016 5:27 PM, Laurent Hardy wrote: > > Hello, > > Is there anyone available to review this patch ? > > Adding more developers into cc. This patch is dying: more than 2 months without any review! A tip to help ixgbe maintainers in their task, please chec

[dpdk-dev] [PATCH] app/test-crypto-perf: fix gcc compilation under FreeBSD

2017-02-01 Thread Daniel Mrzyglod
this fix error: implicit declaration of function 'getline' Signed-off-by: Daniel Mrzyglod --- app/test-crypto-perf/cperf_test_vector_parsing.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/app/test-crypto-perf/cperf_test_vector_parsing.c b/app/test-crypto-perf/cperf_test_vector_parsing

Re: [dpdk-dev] [dpdk-stable] [PATCH v2] net/mlx5: fix link status query

2017-02-01 Thread Adrien Mazarguil
On Wed, Feb 01, 2017 at 11:13:59AM +, Ferruh Yigit wrote: > On 2/1/2017 9:07 AM, Adrien Mazarguil wrote: > > On Wed, Feb 01, 2017 at 06:53:55AM +, Shahaf Shuler wrote: > >> : Tuesday, January 31, 2017 6:17 PM, Ferruh Yigit: > >>> On 1/31/2017 11:45 AM, Shahaf Shuler wrote: > Trying to

Re: [dpdk-dev] [PATCH 25/25] rte_eal_init: add info about rte_errno codes

2017-02-01 Thread Jan Blunck
On Wed, Feb 1, 2017 at 11:54 AM, Adrien Mazarguil wrote: > On Mon, Jan 30, 2017 at 09:19:29PM +0100, Thomas Monjalon wrote: >> 2017-01-30 13:38, Aaron Conole: >> > Stephen Hemminger writes: >> > > Bruce Richardson wrote: >> > >> On Fri, Jan 27, 2017 at 08:33:46AM -0800, Stephen Hemminger wrote:

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

2017-02-01 Thread Pablo de Lara
Underlying IPSec Multi buffer library implements DOCSIS specification, so this commit adds support for this new feature. Signed-off-by: Pablo de Lara --- This patch depends on following patch and it is targetting 17.05 release: http://dpdk.org/dev/patchwork/patch/19732/ app/test-crypto-perf/c

Re: [dpdk-dev] [RFC 1/2] doc: introduction to prgdev

2017-02-01 Thread Jan Blunck
On Fri, Jan 20, 2017 at 4:21 AM, Chen Jing D(Mark) wrote: > This is the documentation to describe what prgdev is, how to use > prgdev API and accomplish an image download. > > Signed-off-by: Chen Jing D(Mark) > --- > doc/guides/prog_guide/prgdev_lib.rst | 457 >

Re: [dpdk-dev] [libvirt] [RFC] Vhost-user backends cross-version migration support

2017-02-01 Thread Daniel P. Berrange
On Wed, Feb 01, 2017 at 12:33:22PM +0100, Maxime Coquelin wrote: > > > On 02/01/2017 10:43 AM, Daniel P. Berrange wrote: > > On Wed, Feb 01, 2017 at 10:14:54AM +0100, Michal Privoznik wrote: > > > On 02/01/2017 09:35 AM, Maxime Coquelin wrote: > > > > > > Solution 3: Libvirt queries OVS for vhos

Re: [dpdk-dev] [libvirt] [RFC] Vhost-user backends cross-version migration support

2017-02-01 Thread Maxime Coquelin
On 02/01/2017 10:43 AM, Daniel P. Berrange wrote: On Wed, Feb 01, 2017 at 10:14:54AM +0100, Michal Privoznik wrote: On 02/01/2017 09:35 AM, Maxime Coquelin wrote: Solution 3: Libvirt queries OVS for vhost backend version string: *OK* =

Re: [dpdk-dev] [dpdk-stable] [PATCH v2] net/mlx5: fix link status query

2017-02-01 Thread Ferruh Yigit
On 2/1/2017 9:07 AM, Adrien Mazarguil wrote: > On Wed, Feb 01, 2017 at 06:53:55AM +, Shahaf Shuler wrote: >> : Tuesday, January 31, 2017 6:17 PM, Ferruh Yigit: >>> On 1/31/2017 11:45 AM, Shahaf Shuler wrote: Trying to query the link status through new kernel ioctl API ETHTOOL_GLINKSET

Re: [dpdk-dev] [PATCH 1/3] net/mlx5: use flow API default masks

2017-02-01 Thread Ferruh Yigit
On 1/31/2017 4:02 PM, Nelio Laranjeiro wrote: > Default masks were introduced in the API after its implementation in this > PMD. > > Signed-off-by: Nelio Laranjeiro > Acked-by: Adrien Mazarguil Series applied to dpdk-next-net/master, thanks.

Re: [dpdk-dev] [PATCH v3] net/mlx5: fix link status is always inconsist

2017-02-01 Thread Ferruh Yigit
On 1/31/2017 3:40 PM, Nélio Laranjeiro wrote: > On Tue, Jan 31, 2017 at 03:13:38PM +0200, Shahaf Shuler wrote: >> Query the link status can end up in an inconsist state where the port is >> down but it is reporting speed. For that another query is scheduled for a >> later time. >> A race condition

Re: [dpdk-dev] [PATCH 25/25] rte_eal_init: add info about rte_errno codes

2017-02-01 Thread Adrien Mazarguil
On Mon, Jan 30, 2017 at 09:19:29PM +0100, Thomas Monjalon wrote: > 2017-01-30 13:38, Aaron Conole: > > Stephen Hemminger writes: > > > Bruce Richardson wrote: > > >> On Fri, Jan 27, 2017 at 08:33:46AM -0800, Stephen Hemminger wrote: > > >> > Why use rte_errno? > > >> > Most DPDK calls just return

Re: [dpdk-dev] [libvirt] [RFC] Vhost-user backends cross-version migration support

2017-02-01 Thread Daniel P. Berrange
On Wed, Feb 01, 2017 at 10:14:54AM +0100, Michal Privoznik wrote: > On 02/01/2017 09:35 AM, Maxime Coquelin wrote: > > Solution 3: Libvirt queries OVS for vhost backend version string: *OK* > > == > > > > > > The idea is to hav

Re: [dpdk-dev] [libvirt] [RFC] Vhost-user backends cross-version migration support

2017-02-01 Thread Michal Privoznik
On 02/01/2017 09:35 AM, Maxime Coquelin wrote: > Hi, > > Few months ago, Michael reported a problem about migrating VMs relying > on vhost-user between hosts supporting different backend versions: > - Message-Id: <20161011173526-mutt-send-email-...@kernel.org> > - https://lists.gnu.org/archive/

Re: [dpdk-dev] [dpdk-stable] [PATCH v2] net/mlx5: fix link status query

2017-02-01 Thread Adrien Mazarguil
On Wed, Feb 01, 2017 at 06:53:55AM +, Shahaf Shuler wrote: > : Tuesday, January 31, 2017 6:17 PM, Ferruh Yigit: > > On 1/31/2017 11:45 AM, Shahaf Shuler wrote: > > > Trying to query the link status through new kernel ioctl API > > > ETHTOOL_GLINKSETTINGS was always failing due to kernel bug. >

Re: [dpdk-dev] [PATCH v2 02/15] eventdev: add APIs for extended stats

2017-02-01 Thread Van Haaren, Harry
> From: Eads, Gage > Sent: Tuesday, January 31, 2017 8:50 PM > To: Van Haaren, Harry ; dev@dpdk.org > Cc: Richardson, Bruce ; Jerin Jacob > > Subject: Re: [dpdk-dev] [PATCH v2 02/15] eventdev: add APIs for extended stats > Hi Harry, > Looks like the function names in *_version.map don't quite

[dpdk-dev] [RFC] Vhost-user backends cross-version migration support

2017-02-01 Thread Maxime Coquelin
Hi, Few months ago, Michael reported a problem about migrating VMs relying on vhost-user between hosts supporting different backend versions: - Message-Id: <20161011173526-mutt-send-email-...@kernel.org> - https://lists.gnu.org/archive/html/qemu-devel/2016-10/msg03026.html The goal of this t

[dpdk-dev] [PATCH] net/sfc: multicast list set must not fail if list is empty

2017-02-01 Thread Andrew Rybchenko
From: Ivan Malov The patch is to fix sfc_set_mc_addr_list() behaviour in order to make it accept an empty multicast address list thus making it possible to remove multicast addresses inserted previously Fixes: 0fa0070e4391 ("net/sfc: support multicast addresses list controls") Signed-off-by: Iv

Re: [dpdk-dev] [PATCH 1/6] net/tap: use correct tap name

2017-02-01 Thread Pascal Mazon
On 02/01/2017 12:29 AM, Wiles, Keith wrote: On Jan 31, 2017, at 10:39 AM, Pascal Mazon wrote: On 01/31/2017 05:06 PM, Wiles, Keith wrote:> Looking at the changes to set the link up/down and the adding the two functions. I noticed in the stop/start routines I was set the link in DPDK and no